Skip to content

Commit

Permalink
Reverted to more concise time check; added logs to better surface fai…
Browse files Browse the repository at this point in the history
…lure reasons. (#35)

Co-authored-by: Olivér Falvai <[email protected]>
  • Loading branch information
matrangam and ofalvai authored Apr 14, 2022
1 parent d96325b commit 4820d57
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions step.go
Original file line number Diff line number Diff line change
Expand Up @@ -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)
}
}

Expand Down

0 comments on commit 4820d57

Please sign in to comment.