fixup: explicit throw, skip test for web, will need to skip more

Author: Anemy

packages/compass-e2e-tests/tests/connection.test.ts

@@ -329,7 +329,8 @@ describe('Connection string', function () {
   });
 
   it('can connect to an Atlas replicaset without srv', async function () {
-    if (!hasAtlasEnvironmentVariables()) {
+    if (!hasAtlasEnvironmentVariables() || TEST_COMPASS_WEB) {
+      // Skip this on web because it doesn't support saslprep.
       return this.skip();
     }
 
@@ -340,15 +341,13 @@ describe('Connection string', function () {
 
     const connectionString = await resolveMongodbSrv(withSRV);
     await browser.connectWithConnectionString(connectionString);
-    if (!TEST_COMPASS_WEB) {
-      const result = await browser.shellEval(
-        connectionNameFromString(connectionString),
-        'db.runCommand({ connectionStatus: 1 })',
-        true
-      );
-      assertNotError(result);
-      expect(result).to.have.property('ok', 1);
-    }
+    const result = await browser.shellEval(
+      connectionNameFromString(connectionString),
+      'db.runCommand({ connectionStatus: 1 })',
+      true
+    );
+    assertNotError(result);
+    expect(result).to.have.property('ok', 1);
   });
 
   it('can connect to an Atlas cluster with a direct connection', async function () {

packages/compass-web/polyfills/@mongodb-js/saslprep/index.ts

@@ -1,7 +1,3 @@
 export function saslprep(): void {
-  // no-op
+  throw new Error('saslprep is not supported in this environment');
 }
-
-export default {
-  saslprep,
-};