From 4820d57e5eb982c6701b73be7ff3e0c6256bc574 Mon Sep 17 00:00:00 2001
From: Michael Matranga <michael.matranga@bitrise.io>
Date: Thu, 14 Apr 2022 11:05:49 -0400
Subject: [PATCH] Reverted to more concise time check; added logs to better
 surface failure reasons. (#35)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-authored-by: Olivér Falvai <oliver.falvai@bitrise.io>
---
 step.go | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/step.go b/step.go
index f61d460..a942d16 100644
--- a/step.go
+++ b/step.go
@@ -454,9 +454,10 @@ func (b TestBuilder) findTestBundle(opts findTestBundleOpts) (testBundle, error)
 		}
 		buildStartTime := opts.BuildInterval.start
 		buildEndTime := opts.BuildInterval.end
-		if (info.ModTime().After(buildStartTime) && info.ModTime().Before(buildEndTime)) ||
-			info.ModTime().Equal(buildStartTime) || info.ModTime().Equal(buildEndTime) {
+		if !info.ModTime().Before(buildStartTime) && !info.ModTime().After(buildEndTime) {
 			buildXCTestrunPths = append(buildXCTestrunPths, xctestrunPth)
+		} else {
+			log.Printf("xctestrun: %s was created at %s, which is outside of the window %s - %s ", xctestrunPth, info.ModTime(), buildStartTime, buildEndTime)
 		}
 	}