Update @jupyterlab/galata

.pre-commit-config.yaml

@@ -29,7 +29,7 @@ repos:
     rev: 'v2.2.6'
     hooks:
       - id: codespell
-        args: ['-L', 'hart,noteable']
+        args: ['-L', 'hart,noteable', '--skip', "*.spec.ts"]
         exclude: |
           (?x)^(
               yarn.lock|

ui-tests/package.json

@@ -15,8 +15,8 @@
     "test:update": "playwright test --update-snapshots"
   },
   "dependencies": {
-    "@jupyterlab/galata": "~5.2.0-rc.0",
-    "@playwright/test": "^1.33.0",
+    "@jupyterlab/galata": "~5.2.0",
+    "@playwright/test": "^1.44.0",
     "rimraf": "^3.0.2"
   }
 }

ui-tests/test/editor.spec.ts

@@ -5,7 +5,7 @@ import path from 'path';
 
 import { test } from './fixtures';
 
-import { expect } from '@playwright/test';
+import { expect } from '@jupyterlab/galata';
 
 const FILE = 'environment.yml';
 

ui-tests/test/filebrowser.spec.ts

@@ -3,7 +3,7 @@
 
 import path from 'path';
 
-import { expect } from '@playwright/test';
+import { expect } from '@jupyterlab/galata';
 
 import { test } from './fixtures';
 
@@ -36,7 +36,7 @@
     const toolbar = page.getByRole('toolbar');
 
     ['Rename', 'Delete', 'Open', 'Download', 'Delete'].forEach(async (text) => {
       expect(toolbar.getByText(text)).toBeVisible();
     });
   });
 

ui-tests/test/general.spec.ts

@@ -3,7 +3,7 @@
 
 import path from 'path';
 
-import { expect } from '@playwright/test';
+import { expect } from '@jupyterlab/galata';
 
 import { test } from './fixtures';
 

ui-tests/test/layout.spec.ts

@@ -3,7 +3,7 @@
 
 import path from 'path';
 
-import { expect } from '@playwright/test';
+import { expect } from '@jupyterlab/galata';
 
 import { galata } from '@jupyterlab/galata';
 

ui-tests/test/links.spec.ts

@@ -3,7 +3,7 @@
 
 import path from 'path';
 
-import { expect } from '@playwright/test';
+import { expect } from '@jupyterlab/galata';
 
 import { test } from './fixtures';
 

ui-tests/test/menus.spec.ts

@@ -5,7 +5,7 @@ import path from 'path';
 
 import { test } from './fixtures';
 
-import { expect } from '@playwright/test';
+import { expect } from '@jupyterlab/galata';
 import { waitForKernelReady } from './utils';
 
 const NOTEBOOK = 'empty.ipynb';

ui-tests/test/mobile.spec.ts

@@ -1,15 +1,30 @@
 // Copyright (c) Jupyter Development Team.
 // Distributed under the terms of the Modified BSD License.
 
-import { expect } from '@playwright/test';
+import { expect, galata } from '@jupyterlab/galata';
 
 import { test } from './fixtures';
 
 import { hideAddCellButton, waitForKernelReady } from './utils';
 
-test.use({ autoGoto: false, viewport: { width: 524, height: 800 } });
+test.use({
+  autoGoto: false,
+  viewport: { width: 524, height: 800 },
+  tmpPath: 'mobile-layout',
+});
 
 test.describe('Mobile', () => {
+  // manually create the test directory since tmpPath is set to a fixed value
+  test.beforeAll(async ({ request, tmpPath }) => {
+    const contents = galata.newContentsHelper(request);
+    await contents.createDirectory(tmpPath);
+  });
+
+  test.afterAll(async ({ request, tmpPath }) => {
+    const contents = galata.newContentsHelper(request);
+    await contents.deleteDirectory(tmpPath);
+  });
+
   test('The layout should be more compact on the file browser page', async ({
     page,
     tmpPath,
@@ -29,9 +44,9 @@
     await page.goto(`tree/${tmpPath}`);
 
     // Create a new notebook
+    await page.click('text="New"');
     const [notebook] = await Promise.all([
       page.waitForEvent('popup'),
-      page.click('text="New"'),
       page.click('text="Python 3 (ipykernel)"'),
     ]);
 

ui-tests/test/notebook.spec.ts

@@ -3,7 +3,7 @@
 
 import path from 'path';
 
-import { expect } from '@playwright/test';
+import { expect } from '@jupyterlab/galata';
 
 import { test } from './fixtures';
 

ui-tests/test/settings.spec.ts

@@ -3,11 +3,21 @@
 
 import { test } from './fixtures';
 
-import { expect } from '@playwright/test';
+import { expect, galata } from '@jupyterlab/galata';
 
-test.use({ autoGoto: false });
+test.use({ autoGoto: false, tmpPath: 'settings' });
 
 test.describe('Settings', () => {
+  test.beforeAll(async ({ request, tmpPath }) => {
+    const contents = galata.newContentsHelper(request);
+    await contents.createDirectory(tmpPath);
+  });
+
+  test.afterAll(async ({ request, tmpPath }) => {
+    const contents = galata.newContentsHelper(request);
+    await contents.deleteDirectory(tmpPath);
+  });
+
   test('Should be persisted after reloading the page', async ({
     page,
     tmpPath,

ui-tests/test/smoke.spec.ts

@@ -1,4 +1,4 @@
-import { expect } from '@playwright/test';
+import { expect } from '@jupyterlab/galata';
 
 import { test } from './fixtures';
 
@@ -24,9 +24,9 @@ test.describe('Smoke', () => {
     await console.waitForSelector('.jp-CodeConsole');
 
     // Create a new notebook
+    await page.click('text="New"');
     const [notebook] = await Promise.all([
       page.waitForEvent('popup'),
-      page.click('text="New"'),
       page.click('text="Python 3 (ipykernel)"'),
     ]);
 

ui-tests/test/tree.spec.ts

@@ -3,7 +3,7 @@
 
 import { test } from './fixtures';
 
-import { expect } from '@playwright/test';
+import { expect } from '@jupyterlab/galata';
 
 const SUBFOLDER = 'subfolder';