BB2-3882: Add selenium coverage to ensure v2 SMART App scopes are ret… (#1380)

* BB2-3882: Add selenium coverage to ensure v2 SMART App scopes are returned

* Revert env var

* Address feedback and update a scope to correct format

* Update last test to use constants as well

Author: JamesDemeryNava

apps/dot_ext/tests/demographic_scopes_test_cases.py

@@ -29,14 +29,14 @@
     'patient/Coverage.r',
     'patient/Coverage.s',
     'patient/Coverage.rs',
-    'patient/launch',
+    'launch/patient',
     'openid'
 ]
 V2_SCOPES_ALL_CONDENSED = [
     'patient/Patient.rs',
     'patient/ExplanationOfBenefit.rs',
     'patient/Coverage.rs',
-    'patient/launch',
+    'launch/patient',
     'openid'
 ]
 V2_SCOPES_NON_DEMOGRAPHIC = [
@@ -46,13 +46,13 @@
     'patient/Coverage.r',
     'patient/Coverage.s',
     'patient/Coverage.rs',
-    'patient/launch',
+    'launch/patient',
     'openid'
 ]
 V2_SCOPES_NON_DEMOGRAPHIC_CONDENSED = [
     'patient/ExplanationOfBenefit.rs',
     'patient/Coverage.rs',
-    'patient/launch',
+    'launch/patient',
     'openid'
 ]
 

apps/integration_tests/selenium_cases.py

@@ -157,6 +157,14 @@ class Action(Enum):
 # API versions
 API_V2 = "v2"
 
+# v2 SMART APP scope constants
+X_PATH_FOR_USER_SCOPES = "//*[@id='main-content']/div/div/div/pre"
+COVERAGE_SCOPE = "patient/Coverage.rs"
+PATIENT_SCOPE = "patient/Patient.rs"
+EOB_SCOPE = "patient/ExplanationOfBenefit.rs"
+OPENID_SCOPE = "openid"
+LAUNCH_SCOPE = "launch/patient"
+
 BROWSERBACK = {
     "display": "Back to FHIR resource page",
     "action": Action.BACK,
@@ -513,6 +521,34 @@ class Action(Enum):
     BROWSERBACK,
 ]
 
+SEQ_CHECK_SCOPES = [
+    {
+        "display": "Check that v2 SMART APP scopes were returned",
+        "action": Action.CONTAIN_TEXT,
+        "params": [20, By.XPATH, X_PATH_FOR_USER_SCOPES, COVERAGE_SCOPE]
+    },
+    {
+        "display": "Check that v2 SMART APP scopes were returned",
+        "action": Action.CONTAIN_TEXT,
+        "params": [20, By.XPATH, X_PATH_FOR_USER_SCOPES, PATIENT_SCOPE]
+    },
+    {
+        "display": "Check that v2 SMART APP scopes were returned",
+        "action": Action.CONTAIN_TEXT,
+        "params": [20, By.XPATH, X_PATH_FOR_USER_SCOPES, EOB_SCOPE]
+    },
+    {
+        "display": "Check that v2 SMART APP scopes were returned",
+        "action": Action.CONTAIN_TEXT,
+        "params": [20, By.XPATH, X_PATH_FOR_USER_SCOPES, OPENID_SCOPE]
+    },
+    {
+        "display": "Check that v2 SMART APP scopes were returned",
+        "action": Action.CONTAIN_TEXT,
+        "params": [20, By.XPATH, X_PATH_FOR_USER_SCOPES, LAUNCH_SCOPE]
+    }
+]
+
 TESTS = {
     "auth_grant_pkce_fhir_calls": [
         {"sequence": SEQ_AUTHORIZE_PKCE_START},
@@ -565,6 +601,15 @@ class Action(Enum):
         # the 'approve' and 'deny' button click not using locale based text
         # so it is lang agnostic
         CLICK_AGREE_ACCESS
+    ],
+    "authorize_get_v2_scopes": [
+        {"sequence": SEQ_AUTHORIZE_PKCE_START},
+        CALL_LOGIN,
+        WAIT_SECONDS,
+        WAIT_SECONDS,
+        CLICK_AGREE_ACCESS,
+        # Check the different scopes that have been returned
+        {"sequence": SEQ_CHECK_SCOPES}
     ]
 }
 

apps/integration_tests/selenium_tests.py

@@ -60,3 +60,18 @@ def test_authorize_lang_english_button(self):
             print("Skip test " + test_name + " - only for new perm screen.")
         self._testclient_home()
         self._print_testcase_banner(test_name, api_ver, step[0], self.use_mslsx, False)
+
+    '''
+    Test authorizing through the test client, using v2 URLs, and ensuring all of the
+    SMART App v2 Scopes are available within the returned token
+    '''
+    def test_v2_authorization_and_scopes(self):
+        step = [0]
+        test_name = "authorize_get_v2_scopes"
+        api_ver = API_V2
+        self._print_testcase_banner(test_name, api_ver, step[0], self.use_mslsx, True)
+
+        self._play(TESTS[test_name], step, api_ver=api_ver)
+
+        self._testclient_home()
+        self._print_testcase_banner(test_name, api_ver, step[0], self.use_mslsx, False)