Skip to content

Commit 53a5083

Browse files
committed
test(V2): require synced non-home project live coverage
Fail live project ownership tests when subscribed non-home project data is missing, then verify update and delete attempts are rejected.
1 parent 4bc04e4 commit 53a5083

1 file changed

Lines changed: 16 additions & 8 deletions

File tree

tests/v2/live-api/project-validation.live.spec.js

Lines changed: 16 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,15 @@ const findNonHomeProject = async (request, homeOrgId) => {
4646
return null;
4747
};
4848

49+
const requireNonHomeProject = async (request, homeOrgId) => {
50+
const nonHomeProject = await findNonHomeProject(request, homeOrgId);
51+
expect(
52+
nonHomeProject,
53+
'Expected at least one synced project from another subscribed organization',
54+
).to.exist;
55+
return nonHomeProject;
56+
};
57+
4958
const buildProjectUpdateData = (record, overrides = {}) => ({
5059
projectRegistryName: record.projectRegistryName,
5160
projectId: record.projectId,
@@ -218,10 +227,7 @@ describe('Project Live API Validation Tests', function () {
218227
});
219228
describe('Step 7: PUT Request Tests', function () {
220229
it('should reject updating a project not owned by the home organization', async function () {
221-
const nonHomeProject = await findNonHomeProject(request, homeOrgId);
222-
if (!nonHomeProject) {
223-
this.skip();
224-
}
230+
const nonHomeProject = await requireNonHomeProject(request, homeOrgId);
225231

226232
const updateData = buildProjectUpdateData(nonHomeProject, {
227233
projectName: `Should Not Update ${Date.now()}`,
@@ -353,6 +359,11 @@ describe('Project Live API Validation Tests', function () {
353359
}
354360
});
355361

362+
it('should include synced project data from another organization', async function () {
363+
const nonHomeProject = await requireNonHomeProject(request, homeOrgId);
364+
expect(nonHomeProject.orgUid).to.not.equal(homeOrgId);
365+
});
366+
356367
it('should support search functionality', async function () {
357368
// Test search if supported by endpoint
358369
const response = await request
@@ -365,10 +376,7 @@ describe('Project Live API Validation Tests', function () {
365376
});
366377
describe('Step 9: DELETE Request Tests', function () {
367378
it('should reject deleting a project not owned by the home organization', async function () {
368-
const nonHomeProject = await findNonHomeProject(request, homeOrgId);
369-
if (!nonHomeProject) {
370-
this.skip();
371-
}
379+
const nonHomeProject = await requireNonHomeProject(request, homeOrgId);
372380

373381
try {
374382
const response = await request

0 commit comments

Comments
 (0)