Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow .xctestrun equivalent in config.json #360

Merged
merged 10 commits into from
Nov 19, 2019
Merged

Allow .xctestrun equivalent in config.json #360

merged 10 commits into from
Nov 19, 2019

Conversation

bdittmer
Copy link
Contributor

Allow for the the config file to contain info usually stored in a .xctestrunfile run file.

Example config: https://gist.github.com/bdittmer/9c131fa94ca36daf8c70e3014c87c51e
Example run w/ config: https://gist.github.com/bdittmer/9ce90fece7308d7192f6c4f6bbd98e7a

Addresses #358

@bdittmer bdittmer requested a review from ob September 17, 2019 20:13
Copy link
Member

@ob ob left a comment

Choose a reason for hiding this comment

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

This looks good but I think it would be worth converting the other code path that pulls the test plan from an xctestrun file into a BPTestPlan in order to not have two different code paths... what do you think?

bp/src/BPConfiguration.m Show resolved Hide resolved
@ob
Copy link
Member

ob commented Sep 17, 2019

Also, it could use some docs about the new keys in the config file. Oh, and some tests

Copy link
Member

@ob ob left a comment

Choose a reason for hiding this comment

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

LGTM, left some comments

bp/src/BPConfiguration.m Outdated Show resolved Hide resolved
bp/src/BPConfiguration.m Show resolved Hide resolved
bp/src/BPConfiguration.m Outdated Show resolved Hide resolved
bptestrunner/bluepill_batch_test.bzl Show resolved Hide resolved
Brian Dittmer and others added 2 commits November 13, 2019 13:20
More changes

Add additional keys

Clean up BPTestPlan code.

Validate BPTestPlans

Bazel batch tests (#389)

* Bazel support for Bluepill batch test mode.

Batch test mode is the equivilent of an xctestrun file that's built by
  Xcode from a xcscheme. It has information to run a set of test targets.
  Bluepill will read a test plan json, which has configurations of each
  test targets, then split and pass it to BP sessions to run the tests.

Changes:
Added bluepill_batch_test rule
Added bluepill_batch_test_runner.template.sh, the rule will generate the sh
  from itto setup workspace for bluepill then call it.
Changed BPConfiguration to accept a test plan file.

* Added support for time estimate json support in Bazel batch test rule

* Corrected batch test runner script to return the right RC and copy logs

Remove test outputs after copying
Copy link
Member

@ob ob left a comment

Choose a reason for hiding this comment

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

LGTM, left a few comments :-)

bptestrunner/bluepill_batch_test_runner.template.sh Outdated Show resolved Hide resolved
bptestrunner/bluepill_batch_test_runner.template.sh Outdated Show resolved Hide resolved
bptestrunner/bluepill_batch_test_runner.template.sh Outdated Show resolved Hide resolved
bptestrunner/bluepill_batch_test_runner.template.sh Outdated Show resolved Hide resolved
bptestrunner/bluepill_batch_test_runner.template.sh Outdated Show resolved Hide resolved
Copy link
Member

@ob ob left a comment

Choose a reason for hiding this comment

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

LGTM, 🚢

bptestrunner/bluepill_batch_test_runner.template.sh Outdated Show resolved Hide resolved
@chenxiao0228 chenxiao0228 merged commit 97f36f9 into master Nov 19, 2019
@chenxiao0228 chenxiao0228 deleted the bp-358 branch November 19, 2019 00:22
chenxiao0228 pushed a commit that referenced this pull request Nov 19, 2019
Bazel support for Bluepill batch test mode.

Batch test mode is the equivilent of an xctestrun file that's built by
  Xcode from a xcscheme. It has information to run a set of test targets.
  Bluepill will read a test plan json, which has configurations of each
  test targets, then split and pass it to BP sessions to run the tests.
Added bluepill_batch_test rule
Added bluepill_batch_test_runner.template.sh, the rule will generate the sh
  from itto setup workspace for bluepill then call it.
Changed BPConfiguration to accept a test plan file.
Added support for time estimate json support in Bazel batch test rule

(cherry picked from commit 97f36f9)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants