Decouple superset and opensrp states

.env.sample

@@ -202,4 +202,6 @@ REACT_APP_SUPERSET_MDA_LITE_REPORTING_INDICATOR_STOPS=kenya2021
 REACT_APP_CLIENTS_LIST_BATCH_SIZE=100
 
 REACT_APP_AUTO_ENABLE_NAVIGATION_TO='/assign,/teams'
-REACT_APP_REVOKE_COMPLETE_AND_RETIRED_PLANS_TEAM_ASSIGNMENTS=false
+REACT_APP_REVOKE_COMPLETE_AND_RETIRED_PLANS_TEAM_ASSIGNMENTS=false
+
+REACT_APP_SUPERSET_OAUTH_STATE=opensrp

docker/confd_env.toml

@@ -120,4 +120,5 @@ keys = [
   "/react/app/superset/mda/lite/reporting/ward/geojson/slice",
   "/react/app/show/map/at/jurisdiction/level",
   "/react/app/clients/list/batch/size",
+  "/react/app/superset/oauth/state",
 ]

docker/config.js.tmpl

@@ -117,4 +117,5 @@ window._env_ = {
     "REACT_APP_SUPERSET_MDA_LITE_REPORTING_WARD_GEOJSON_SLICE": "{{ getv "/react/app/superset/mda/lite/reporting/ward/geojson/slice" "0" }}",
     "REACT_APP_SHOW_MAP_AT_JURISDICTION_LEVEL": "{{ getv "/react/app/show/map/at/jurisdiction/level" "99" }}",
     "REACT_APP_CLIENTS_LIST_BATCH_SIZE": "{{ getv "/react/app/clients/list/batch/size" "100" }}",
+    "REACT_APP_SUPERSET_OAUTH_STATE": "{{ getv "/react/app/superset/oauth/state" "opensrp"}}",
 }

docs/env.md

@@ -221,3 +221,9 @@ Below is a list of currently supported environment variables:
   - OpenSRP update plan endpoint parameter (`revoke_assignments`) value. Determines if team assignments for complete and retired plans will be revoked or not.
   - Options: 'true' and 'false'. Any other value defaults to 'false'.
   - default: 'true'
+
+- **REACT_APP_SUPERSET_OAUTH_STATE**
+
+  - **Optional**
+  - Value of superset provider oauth state
+  - Defaults to the value of `REACT_APP_OPENSRP_OAUTH_STATE` if not provided.

src/configs/env.ts

@@ -401,6 +401,9 @@ export type OPENSRP_USER_URL = typeof OPENSRP_USER_URL;
 export const OPENSRP_OAUTH_STATE = setEnv('REACT_APP_OPENSRP_OAUTH_STATE', 'opensrp');
 export type OPENSRP_OAUTH_STATE = typeof OPENSRP_OAUTH_STATE;
 
+export const SUPERSET_OAUTH_STATE = setEnv('REACT_APP_SUPERSET_OAUTH_STATE', '');
+export type SUPERSET_OAUTH_STATE = typeof SUPERSET_OAUTH_STATE;
+
 export const OPENSRP_LOGOUT_URL = setEnv(
   'REACT_APP_OPENSRP_LOGOUT_URL',
   'https://reveal-stage.smartregister.org/opensrp/logout.do'

src/services/superset/index.ts

@@ -2,7 +2,12 @@ import reducerRegistry from '@onaio/redux-reducer-registry';
 import { getOauthProviderState } from '@onaio/session-reducer';
 import superset, { SupersetConnectorConfig } from '@onaio/superset-connector';
 import { Dictionary } from '@onaio/utils';
-import { OPENSRP_OAUTH_STATE, SUPERSET_API_BASE, SUPERSET_API_ENDPOINT } from '../../configs/env';
+import {
+  OPENSRP_OAUTH_STATE,
+  SUPERSET_API_BASE,
+  SUPERSET_API_ENDPOINT,
+  SUPERSET_OAUTH_STATE,
+} from '../../configs/env';
 import { ERROR_PERMISSION_DENIED } from '../../configs/lang';
 import { SUPERSET_ACCESS_DENIED_MESSAGE } from '../../constants';
 import { displayError } from '../../helpers/errors';
@@ -60,7 +65,8 @@ const supersetFetch = async (
      * to enable the use of multiple providers, we store the provider name in the
      * `state` parameter so that we can use it here
      */
-    provider: getOauthProviderState(store.getState()) || OPENSRP_OAUTH_STATE,
+    provider:
+      SUPERSET_OAUTH_STATE || getOauthProviderState(store.getState()) || OPENSRP_OAUTH_STATE,
     token: '',
   };
 

src/services/superset/tests/index.test.ts

@@ -117,6 +117,32 @@ describe('services/superset', () => {
 
     expect(processDataMock).toHaveBeenCalledWith(fixtures.sliceResponse);
   });
+
+  it('sets superset oauth state correctly', async () => {
+    store.dispatch(resetSuperset());
+
+    const envModule = require('../../../configs/env');
+    const supersetState = 'superset-oauth-state';
+    envModule.SUPERSET_OAUTH_STATE = supersetState;
+
+    const authZMock = jest.spyOn(superset, 'authZ');
+
+    fetchMock.get(`${SUPERSET_API_BASE}oauth-authorized/${supersetState}`, JSON.stringify({}));
+    fetchMock.get(
+      `${SUPERSET_API_BASE}superset/slice_json/1337`,
+      JSON.stringify(fixtures.sliceResponse)
+    );
+
+    await supersetFetch('1337');
+
+    expect(authZMock.mock.calls[0][0]).toEqual({
+      base: 'http://localhost',
+      endpoint: 'slice',
+      extraPath: '1337',
+      provider: supersetState,
+      token: '',
+    });
+  });
 });
 
 describe('services/superset/fetchCallback', () => {