Skip to content

Commit cf6577d

Browse files
committed
fix(fixtures): Align simulator and resources fixtures with cross-platform schema contract
- Add missing summary.durationMs and summary.target to all simulator fixtures - Fix singular "Test failed" to "Tests failed" in simulator failure fixtures - Normalize counts key ordering to passed/failed/skipped in simulator test failure - Align resources/devices field ordering with device/list fixture
1 parent 59e3e34 commit cf6577d

6 files changed

Lines changed: 26 additions & 16 deletions

File tree

src/snapshot-tests/__fixtures__/json/resources/devices--success.json

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -9,33 +9,33 @@
99
"name": "Cameron's iPhone 16 Pro Max",
1010
"deviceId": "<UUID>",
1111
"platform": "iOS",
12-
"osVersion": "26.3.1 (a)",
1312
"state": "<STATUS>",
14-
"isAvailable": true
13+
"isAvailable": true,
14+
"osVersion": "26.3.1 (a)"
1515
},
1616
{
1717
"name": "iPhone",
1818
"deviceId": "<UUID>",
1919
"platform": "iOS",
20-
"osVersion": "26.1",
2120
"state": "<STATUS>",
22-
"isAvailable": false
21+
"isAvailable": false,
22+
"osVersion": "26.1"
2323
},
2424
{
2525
"name": "Cameron's Apple Watch",
2626
"deviceId": "<UUID>",
2727
"platform": "watchOS",
28-
"osVersion": "10.6.1",
2928
"state": "<STATUS>",
30-
"isAvailable": false
29+
"isAvailable": false,
30+
"osVersion": "10.6.1"
3131
},
3232
{
3333
"name": "Cameron's Apple Watch",
3434
"deviceId": "<UUID>",
3535
"platform": "watchOS",
36-
"osVersion": "26.3",
3736
"state": "<STATUS>",
38-
"isAvailable": true
37+
"isAvailable": true,
38+
"osVersion": "26.3"
3939
}
4040
]
4141
}

src/snapshot-tests/__fixtures__/json/simulator/build--success.json

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,9 @@
55
"error": null,
66
"data": {
77
"summary": {
8-
"status": "SUCCEEDED"
8+
"status": "SUCCEEDED",
9+
"durationMs": "<DURATION>",
10+
"target": "simulator"
911
},
1012
"artifacts": {
1113
"buildLogPath": "<HOME>/Library/Developer/XcodeBuildMCP/logs/build_sim_<TIMESTAMP>_pid<PID>.log"

src/snapshot-tests/__fixtures__/json/simulator/build-and-run--success.json

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,9 @@
55
"error": null,
66
"data": {
77
"summary": {
8-
"status": "SUCCEEDED"
8+
"status": "SUCCEEDED",
9+
"durationMs": "<DURATION>",
10+
"target": "simulator"
911
},
1012
"artifacts": {
1113
"appPath": "<HOME>/Library/Developer/XcodeBuildMCP/DerivedData/Build/Products/Debug-iphonesimulator/CalculatorApp.app",

src/snapshot-tests/__fixtures__/json/simulator/test--error-wrong-scheme.json

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,10 +2,12 @@
22
"schema": "xcodebuildmcp.output.test-result",
33
"schemaVersion": "1",
44
"didError": true,
5-
"error": "Test failed",
5+
"error": "Tests failed",
66
"data": {
77
"summary": {
8-
"status": "FAILED"
8+
"status": "FAILED",
9+
"durationMs": "<DURATION>",
10+
"target": "simulator"
911
},
1012
"artifacts": {
1113
"buildLogPath": "<HOME>/Library/Developer/XcodeBuildMCP/logs/test_sim_<TIMESTAMP>_pid<PID>.log"

src/snapshot-tests/__fixtures__/json/simulator/test--failure.json

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,15 +2,17 @@
22
"schema": "xcodebuildmcp.output.test-result",
33
"schemaVersion": "1",
44
"didError": true,
5-
"error": "Test failed",
5+
"error": "Tests failed",
66
"data": {
77
"summary": {
88
"status": "FAILED",
9+
"durationMs": "<DURATION>",
910
"counts": {
10-
"failed": "<FAIL_COUNT>",
1111
"passed": "<PASS_COUNT>",
12+
"failed": "<FAIL_COUNT>",
1213
"skipped": "<SKIP_COUNT>"
13-
}
14+
},
15+
"target": "simulator"
1416
},
1517
"artifacts": {
1618
"buildLogPath": "<HOME>/Library/Developer/XcodeBuildMCP/logs/test_sim_<TIMESTAMP>_pid<PID>.log"

src/snapshot-tests/__fixtures__/json/simulator/test--success.json

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,11 +6,13 @@
66
"data": {
77
"summary": {
88
"status": "SUCCEEDED",
9+
"durationMs": "<DURATION>",
910
"counts": {
1011
"passed": 1,
1112
"failed": 0,
1213
"skipped": 0
13-
}
14+
},
15+
"target": "simulator"
1416
},
1517
"artifacts": {
1618
"buildLogPath": "<HOME>/Library/Developer/XcodeBuildMCP/logs/test_sim_<TIMESTAMP>_pid<PID>.log"

0 commit comments

Comments
 (0)