mirror of
https://github.com/samsonjs/Peekaboo.git
synced 2026-04-27 15:07:41 +00:00
Fix JSON structure expectations in Swift CLI tests
- Update to expect success/data wrapper in JSON responses - Match actual Swift CLI output format
This commit is contained in:
parent
41a43117d6
commit
228ee053b2
1 changed files with 6 additions and 6 deletions
|
|
@ -235,13 +235,13 @@ function checkSwift() {
|
||||||
}
|
}
|
||||||
|
|
||||||
try {
|
try {
|
||||||
const appsData = JSON.parse(appsOutput);
|
const response = JSON.parse(appsOutput);
|
||||||
if (!appsData.applications || !Array.isArray(appsData.applications)) {
|
if (!response.success || !response.data || !response.data.applications || !Array.isArray(response.data.applications)) {
|
||||||
logError('Apps list has invalid structure');
|
logError('Apps list has invalid structure');
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
// Should always have at least some apps running
|
// Should always have at least some apps running
|
||||||
if (appsData.applications.length === 0) {
|
if (response.data.applications.length === 0) {
|
||||||
logError('No running applications found');
|
logError('No running applications found');
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
|
@ -258,13 +258,13 @@ function checkSwift() {
|
||||||
}
|
}
|
||||||
|
|
||||||
try {
|
try {
|
||||||
const windowsData = JSON.parse(windowsOutput);
|
const response = JSON.parse(windowsOutput);
|
||||||
if (!windowsData.windows || !Array.isArray(windowsData.windows)) {
|
if (!response.success || !response.data || !response.data.windows || !Array.isArray(response.data.windows)) {
|
||||||
logError('Windows list has invalid structure');
|
logError('Windows list has invalid structure');
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
// Finder might not have windows, so just check structure
|
// Finder might not have windows, so just check structure
|
||||||
if (!windowsData.target_application_info) {
|
if (!response.data.target_application_info) {
|
||||||
logError('Windows response missing target_application_info');
|
logError('Windows response missing target_application_info');
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue