Add logic to allow for flexible error status code (#1045)

* add logic to allow for flexible error status code * change logic * Update OAuth2Test.js * fix logic * fix logic * try grabbing text and comparing expected error --------- Co-authored-by: Hara Prasad <haraprasadj@uchicago.edu>
uc-cdis · Jun 28, 2024 · e3c44d1 · e3c44d1
1 parent 2e3033e
commit e3c44d1
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/services/apis/fence/fenceTasks.js b/services/apis/fence/fenceTasks.js
@@ -515,7 +515,8 @@ module.exports = {
         }
       // }
     } else {
-      I.seeTextEquals('Unauthorized', 'h2');
+      let errorMessage = await I.grabTextFrom('//div[@class="error-page__status-code-text"]/h2');
+      expect(errorMessage).to.be.oneOf(["Bad Request", "Unauthorized"]);
     }
     // I.saveScreenshot('consent_auth_code_flow.png');
     const urlStr = await I.grabCurrentUrl();

diff --git a/suites/apis/OAuth2Test.js b/suites/apis/OAuth2Test.js
@@ -29,7 +29,7 @@ Scenario('Authorization code flow: Test that successfully generates code @reqGoo
 
 Scenario('Authorization code flow: Test that fail to generate code due to not provided openid scope @reqGoogle', async ({ fence }) => {
   const resULR = await fence.do.getConsentCode(
-    fence.props.clients.client.id, 'code', 'user', 'ok', false,
+    fence.props.clients.client.id, 'code', 'user', 'ok', false
   );
   fence.ask.assertNotContainSubStr(resULR, ['code=']);
 });