Add Test Plans #716
Add Test Plans #716
Conversation
|
|
||
| static var pathProperties: [PathProperty] { | ||
| [ | ||
| .string("testPlans") |
yonaskolb
Nov 10, 2019
Author
Owner
This file is getting large, gonna split it up
This file is getting large, gonna split it up
|
LGTM |
| @@ -194,10 +194,18 @@ public class SchemeGenerator { | |||
| let launchVariables = scheme.run.flatMap { $0.environmentVariables.isEmpty ? nil : $0.environmentVariables } | |||
| let profileVariables = scheme.profile.flatMap { $0.environmentVariables.isEmpty ? nil : $0.environmentVariables } | |||
|
|
|||
| var testPlans: [XCScheme.TestPlanReference]? | |||
giginet
Nov 10, 2019
•
Collaborator
You can use let
let testPlanns: [XCScheme.TestPlanReference]? = scheme.test?.testPlans.map { plans in
return plans.enumerated().map { index, path in
.init(reference: "container:\(path)", default: index == 0)
}
}
You can use let
let testPlanns: [XCScheme.TestPlanReference]? = scheme.test?.testPlans.map { plans in
return plans.enumerated().map { index, path in
.init(reference: "container:\(path)", default: index == 0)
}
}
yonaskolb
Nov 10, 2019
Author
Owner
Yeah there’s a few ways to do it. The property has to be nil if there are no test plans though, as that’s what XcodeProj expects
Yeah there’s a few ways to do it. The property has to be nil if there are no test plans though, as that’s what XcodeProj expects
giginet
Nov 11, 2019
Collaborator
Points of my comment was avoiding using mutable variables as we can.
Type should be [XCScheme.TestPlanReference]? Sorry.
Points of my comment was avoiding using mutable variables as we can.
Type should be [XCScheme.TestPlanReference]? Sorry.
|
@yonaskolb any updates on release for this? would love to start migrating to test plans. |
|
I really like to use XcodeGen but our project is build up on test plans. Is here any update. |
|
Really looking forward to this! Thanks for your effort on this feature! <3 |
|
Looking forward to this! Any ETA on when this will become available? |
|
hey @yonaskolb / @giginet is there anything one can do to help out finializing this? I would like to give it a try and pick this up. |
|
Went ahead and created an up-to-date version of the branch, tried my best to resolve the conflicts, but i really need some hints what the actual code was lacking. :D Edit 1: Started with the simplest one: Adding documentation Edit 2: Guess I understand the to-dos now. First up ill add the option to define a default test plan, after that ill try to add spec validation. |
|
Would love this feature! |
Adds test plan support
Resolves #684
Possible changes:
testPlanswhich are then referenced in schemes, similar to the newprojectReferences?