Fix tests for server

Author: joepio

data-browser/src/hooks/useSavedDrives.ts

@@ -5,7 +5,8 @@ import { useSettings } from '../helpers/AppSettings';
 
 const rootDrives = [
   window.location.origin,
-  ...(isDev() ? ['https://atomicdata.dev', 'http://localhost:9883'] : []),
+  'https://atomicdata.dev',
+  ...(isDev() ? ['http://localhost:9883'] : []),
 ];
 
 const arrayOpts = {

data-browser/src/routes/SettingsServer/WSIndicator.tsx

@@ -56,6 +56,7 @@ function getIndicatorState(
   }
 }
 
+/** Shows the status of a WebSocket connection for some resource. */
 export function WSIndicator({
   subject,
   className,

data-browser/tests/e2e.spec.ts

@@ -32,6 +32,7 @@ const publicReadRight =
   '[data-test="right-public"] input[type="checkbox"] >> nth=0';
 const contextMenu = '[data-test="context-menu"]';
 const addressBar = '[data-test="address-bar"]';
+const defaultDevServer = 'http://localhost:9883';
 
 test.describe('data-browser', async () => {
   test.beforeEach(async ({ page }) => {
@@ -573,7 +574,10 @@ async function openNewSubjectWindow(browser: Browser, url: string) {
   const context2 = await browser.newContext();
   const page = await context2.newPage();
   await page.goto(frontEndUrl);
-  await changeDrive(serverUrl, page);
+  // Only when we run on `localhost` we don't need to change drive during tests
+  if (serverUrl !== defaultDevServer) {
+    await changeDrive(serverUrl, page);
+  }
   await openSubject(page, url);
   await page.setViewportSize({ width: 1000, height: 400 });