MilkyShots/MilkyShots
4
0
Fork 0
Code Issues 45 Pull Requests 1 Packages Projects Releases Wiki Activity

fix: replace remaining jsonPath calls in asset visibility tests

Browse Source 
Test 75 admin assertion: allow null email for server-scanned assets
(Guid.Empty placeholder has no email). Removes all remaining jsonPath
calls that could NPE on null resolved values.
This commit is contained in:
Samuele Lorefice
2026-07-15 15:03:50 +02:00
parent a3ce98a984
commit cace8640e9
1 changed files with 11 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
Lactose/WepApiTest.http
View File

@@ -1119,13 +1119,20 @@ Authorization: Bearer {{admin_token}}
client.assert(response.status === 200)
});
client.test("Admin sees original fileName", function () {
client.assert(jsonPath(response.body, "$.fileName") != null)
client.assert(response.body.fileName != null)
});
client.test("Admin sees uploader email", function () {
client.assert(jsonPath(response.body, "$.uploadedBy.email") != null)
client.test("Admin sees uploader email (or placeholder if server-scanned)", function () {
// uploadedBy is always present (real user or "deleted" placeholder).
// If it's a real uploader (non-empty GUID), email must be visible.
// If server-scanned (Guid.Empty placeholder), email can be null.
var uploader = response.body.uploadedBy;
client.assert(uploader != null, "uploadedBy field must be present");
if (uploader.id !== "00000000-0000-0000-0000-000000000000") {
client.assert(uploader.email != null, "real uploader email should be visible to admin")
}
});
client.test("Admin sees asset visibility", function () {
client.assert(jsonPath(response.body, "$.visibility") != null)
client.assert(response.body.visibility != null)
});
} else {
client.test("No assets in database — skipping asset visibility tests", function () {