Skip to content

Commit 7143877

Browse files
Pollepsjoepio
authored andcommitted
#299 Improve form test
1 parent ed5ddb9 commit 7143877

File tree

2 files changed

+25
-6
lines changed

2 files changed

+25
-6
lines changed

data-browser/src/components/forms/NewForm/useNewForm.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ export const useNewForm = (
6262
store
6363
.renameSubject(oldSubject, defferedSubjectValue)
6464
.then(() => {
65-
// setSubject(defferedSubjectValue);
65+
setSubject(defferedSubjectValue);
6666
})
6767
.catch(e => setSubjectErr(e));
6868
}, [defferedSubjectValue, resource]);

data-browser/tests/e2e.spec.ts

Lines changed: 24 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -422,12 +422,11 @@ test.describe('data-browser', async () => {
422422
await signIn(page);
423423
await page.locator(`${currentDriveTitle} > text=localhost`);
424424

425-
const dropdownId = await page
426-
.locator(sideBarDriveSwitcher)
427-
.getAttribute('aria-controls');
428-
429425
await page.click(sideBarDriveSwitcher);
430426
// temp disable for trailing slash
427+
// const dropdownId = await page
428+
// .locator(sideBarDriveSwitcher)
429+
// .getAttribute('aria-controls');
431430
// await page.click(`[id="${dropdownId}"] >> text=Atomic Data`);
432431
// await expect(page.locator(currentDriveTitle)).toHaveText('Atomic Data');
433432

@@ -469,10 +468,15 @@ test.describe('data-browser', async () => {
469468
await signIn(page);
470469
await newDrive(page);
471470
await newResource('class', page);
471+
const shortnameInput = '[data-test="input-shortname"]';
472472
// Try entering a wrong slug
473-
await page.click('[data-test="input-shortname"]');
473+
await page.click(shortnameInput);
474474
await page.keyboard.type('not valid');
475475
await expect(page.locator('text=Not a valid slug')).toBeVisible();
476+
await page.locator(shortnameInput).fill('');
477+
await page.keyboard.type('is-valid');
478+
await expect(page.locator('text=Not a valid slug')).not.toBeVisible();
479+
476480
// Add a new property
477481
await page.click(
478482
'[placeholder="Select a property or enter a property URL..."]',
@@ -491,6 +495,21 @@ test.describe('data-browser', async () => {
491495
await page.click('[data-test="input-recommends-add-resource"]');
492496
await page.locator('text=append').click();
493497
await expect(page.locator('text=https://atomicdata.dev')).not.toBeVisible();
498+
499+
// Try to save without a description
500+
page.locator('button:has-text("Save")').click();
501+
await expect(
502+
page.locator(
503+
'text=Property https://atomicdata.dev/properties/description missing',
504+
),
505+
).toBeVisible();
506+
507+
// Add a description
508+
await page.click('textarea[name="yamdeContent"]');
509+
await page.keyboard.type('This is a test class');
510+
await page.click('button:has-text("Save")');
511+
512+
await expect(page.locator('text=Resource Saved')).toBeVisible();
494513
});
495514

496515
test('sidebar subresource', async ({ page }) => {

0 commit comments

Comments
 (0)