add todos

snowflakedb · Dec 4, 2024 · bff75dd · bff75dd
1 parent 6445bec
commit bff75dd
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/ci/test_authentication.sh b/ci/test_authentication.sh
@@ -13,5 +13,5 @@ docker run \
   -v $(cd $THIS_DIR/.. && pwd):/mnt/host \
   -v $WORKSPACE:/mnt/workspace \
   --rm \
-  nexus.int.snowflakecomputing.com:8086/docker/snowdrivers-test-external-browser:2 \
+  nexus.int.snowflakecomputing.com:8086/docker/snowdrivers-test-external-browser:3 \
   "/mnt/host/ci/container/test_authentication.sh"
diff --git a/test/authentication/testKeyPair.js b/test/authentication/testKeyPair.js
@@ -73,6 +73,7 @@ describe('Key-pair authentication', function () {
       await authTest.verifyConnectionIsUp();
     });
 
+    //todo SNOW-1844747 improve error message
     it('Invalid private key password', async function () {
       const connectionOption = { ...connParameters.keypairEncryptedPrivateKeyPath, privateKeyPass: 'invalid' };
       authTest.createConnection(connectionOption);

diff --git a/test/authentication/testOkta.js b/test/authentication/testOkta.js
@@ -28,11 +28,12 @@ describe('Okta authentication', function () {
     await authTest.verifyConnectionIsNotUp();
   });
 
-  it.skip('Wrong okta url', async function () {
+  //todo SNOW-1844747 improve error message
+  it('Wrong okta url', async function () {
     const connectionOption = { ...connParameters.okta, authenticator: 'https://testinvalidaccoount.com' };
     authTest.createConnection(connectionOption);
     await authTest.connectAsync();
-    authTest.verifyErrorWasThrown('');
+    authTest.verifyErrorWasThrown('Cannot read properties of null (reading \'ssoUrl\')');
     await authTest.verifyConnectionIsNotUp();
   });
 });