Skip to content

Commit

Permalink
remove xctest simulators from schema
Browse files Browse the repository at this point in the history
  • Loading branch information
alexplischke committed Jan 23, 2025
1 parent 5659ce7 commit 9a7a596
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 69 deletions.
34 changes: 0 additions & 34 deletions api/saucectl.schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -2529,35 +2529,6 @@
}
}
},
"simulators": {
"description": "Defines details for running this suite on virtual devices using a simulator.",
"type": "array",
"items": {
"properties": {
"name": {
"description": "The name of the simulator. To ensure name accuracy, check the list of supported virtual devices (https://app.saucelabs.com/live/web-testing/virtual).",
"type": "string"
},
"orientation": {
"$ref": "#/allOf/1/then/properties/suites/items/properties/emulators/items/properties/orientation"
},
"platformVersions": {
"description": "The set of one or more versions of the device platform on which to run the test suite.",
"type": "array",
"minItems": 1
},
"armRequired": {
"description": "If set to true, the simulator will run on an ARM-based Mac. If set to false, the simulator will run on an Intel-based Mac.",
"type": "boolean"
}
},
"required": [
"name",
"platformVersions"
],
"additionalProperties": false
}
},
"devices": {
"description": "Define details for running this suite on real devices.",
"type": "array",
Expand Down Expand Up @@ -2645,11 +2616,6 @@
}
},
"anyOf": [
{
"required": [
"simulators"
]
},
{
"required": [
"devices"
Expand Down
34 changes: 0 additions & 34 deletions api/v1alpha/framework/xctest.schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -182,35 +182,6 @@
}
}
},
"simulators": {
"description": "Defines details for running this suite on virtual devices using a simulator.",
"type": "array",
"items": {
"properties": {
"name": {
"description": "The name of the simulator. To ensure name accuracy, check the list of supported virtual devices (https://app.saucelabs.com/live/web-testing/virtual).",
"type": "string"
},
"orientation": {
"$ref": "../subschema/common.schema.json#/definitions/orientation"
},
"platformVersions": {
"description": "The set of one or more versions of the device platform on which to run the test suite.",
"type": "array",
"minItems": 1
},
"armRequired": {
"description": "If set to true, the simulator will run on an ARM-based Mac. If set to false, the simulator will run on an Intel-based Mac.",
"type": "boolean"
}
},
"required": [
"name",
"platformVersions"
],
"additionalProperties": false
}
},
"devices": {
"description": "Define details for running this suite on real devices.",
"type": "array",
Expand Down Expand Up @@ -298,11 +269,6 @@
}
},
"anyOf": [
{
"required": [
"simulators"
]
},
{
"required": [
"devices"
Expand Down
6 changes: 5 additions & 1 deletion internal/cmd/ini/initializer.go
Original file line number Diff line number Diff line change
Expand Up @@ -1033,7 +1033,11 @@ func (ini *initializer) initializeBatchXctest(f *pflag.FlagSet) []error {
}
}
validAppExt := []string{".app"}
validAppExt = append(validAppExt, ".ipa")
if f.Changed("simulator") {
validAppExt = append(validAppExt, ".zip")
} else {
validAppExt = append(validAppExt, ".ipa")
}
if ini.cfg.app != "" {
verifier := extValidator(validAppExt)
if err = verifier(ini.cfg.app); err != nil {
Expand Down

0 comments on commit 9a7a596

Please sign in to comment.