Minor improvements on tests

Author: zonia3000

tests/v2/admin_groups.spec.js

@@ -95,7 +95,7 @@ test('Admin groups management', async ({ page }) => {
 		}
 		await modal.getByRole('button', { name: 'Add' }).click();
 		await waitModalClosed(page);
-		finalCount = selectableGroups.length + 2;
+		finalCount = selectableGroups1 + 2;
 		await expect(groupBadges).toHaveCount(finalCount);
 		await expect(page.getByRole('button', { name: 'Add group' })).not.toBeVisible();
 	});

tests/v2/admin_jobs.spec.js

@@ -100,7 +100,9 @@ test('Execute a job and show it on the job tables [v2]', async ({ page, request
 
 	await test.step('Search running jobs', async () => {
 		await page.getByText('Reset').click();
-		await page.selectOption('#status', 'submitted');
+		await page.getByRole('combobox', { name: 'Status' }).selectOption('submitted');
+		await page.getByRole('spinbutton', { name: 'Project Id' }).fill(workflow2.projectId || '');
+		await page.getByRole('spinbutton', { name: 'Workflow Id' }).fill(workflow2.workflowId || '');
 		await search(page);
 		const statuses = page.locator('table tbody tr td:nth-child(2)');
 		await expect(statuses).toHaveCount(1);

tests/v2/run_mock_tasks.spec.js

@@ -259,6 +259,16 @@ test('Run mock tasks [v2]', async ({ page, workflow }) => {
 		expect(await page.locator('table tbody tr').count()).toEqual(5);
 	});
 
+	await test.step('Ensure jobs are ended', async () => {
+		const running = await page.getByRole('row', { name: 'submitted' }).count();
+		if (running > 0) {
+			console.warn(
+				`WARNING: waiting for the completion of a job that should have already been completed!`
+			);
+			await expect(page.getByRole('row', { name: 'submitted' })).toHaveCount(0, { timeout: 10000 });
+		}
+	});
+
 	/** @type {string|null} */
 	let jobId = null;
 	await test.step('Open Info modal', async () => {