Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -89,6 +89,7 @@ A full list supported options are listed here.
| videos-directory | n/a | Directory where videos of test runs will be saved. If not provided, videos are not recorded. | N | n/a |
| video-paths | -V | A list of videos that will be saved in the simulators. | N | n/a |
| image-paths | -I | A list of images that will be saved in the simulators. | N | n/a |
|test-time-estimates-json| -j | Path of the input file with test execution time estimates | N | n/a |
| unsafe-skip-xcode-version-check | | Skip Xcode version check | N | NO |
| retry-app-crash-tests | | Retry tests that crashed app and consider it non-fatal if it passes on retry. | N | false |

Expand Down
4 changes: 4 additions & 0 deletions bluepill/src/BPPacker.m
Original file line number Diff line number Diff line change
Expand Up @@ -146,6 +146,10 @@ @implementation BPPacker
if (splitExecTime + [testTimes[test] doubleValue] <= optimalBundleTime) {
splitExecTime += [testTimes[test] doubleValue];
} else {
if ([bundles count] == [[config numSims] integerValue] - 1) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add tests for this change.

// Put all of the remaining tests into the last bundle
i = (int)[bundleTestsToRun count];
}
break;
}
}
Expand Down