Merge pull request #4610 from snyk/chore/delete-integrated-iac-check-…

…for-fedramp-env chore: remove integrated IaC FF check in FedRAMP env [CFG-2413]
snyk · May 23, 2023 · 13fcece · 13fcece
2 parents eb21374 + 08e1b6f
commit 13fcece
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 22 deletions.
diff --git a/src/lib/feature-flags/index.ts b/src/lib/feature-flags/index.ts
@@ -28,13 +28,6 @@ export async function hasFeatureFlag(
   featureFlag: string,
   options: Options,
 ): Promise<boolean | undefined> {
-  if (
-    featureFlag === 'iacIntegratedExperience' &&
-    config.API.includes('snykgov')
-  ) {
-    return true;
-  }
-
   const { code, error, ok } = await isFeatureFlagSupportedForOrg(
     featureFlag,
     options.org,

diff --git a/test/jest/unit/lib/feature-flags/feature-flags.spec.ts b/test/jest/unit/lib/feature-flags/feature-flags.spec.ts
@@ -1,14 +1,7 @@
-import config from '../../../../../src/lib/config';
 import { hasFeatureFlag } from '../../../../../src/lib/feature-flags';
 import * as request from '../../../../../src/lib/request';
 
 describe('hasFeatureFlag fn', () => {
-  const configApiDefault = config.API;
-
-  afterAll(() => {
-    config.API = configApiDefault;
-  });
-
   it.each`
     hasFlag  | expected
     ${true}  | ${true}
@@ -41,12 +34,4 @@ describe('hasFeatureFlag fn', () => {
       hasFeatureFlag('test-ff', { path: 'test-path' }),
     ).rejects.toThrowError('Forbidden');
   });
-
-  it('should return iacIntegratedExperience feature flag being true for FedRAMP', async () => {
-    config.API = 'https://app.snykgov.io/api/v1';
-    const result = await hasFeatureFlag('iacIntegratedExperience', {
-      path: 'test-path',
-    });
-    expect(result).toEqual(true);
-  });
 });