test(compartment-mapper): check for resistance to bundled dependency …

…name collision
endojs · Jan 15, 2024 · 27e2382 · 27e2382
1 parent 1ff3c4b
commit 27e2382
Show file tree

Hide file tree

Showing 7 changed files with 84 additions and 3 deletions.
diff --git a/packages/compartment-mapper/test/fixtures-policy/node_modules/app/attack-name.js b/packages/compartment-mapper/test/fixtures-policy/node_modules/app/attack-name.js
diff --git a/packages/compartment-mapper/test/fixtures-policy/node_modules/eve/getcarol.js b/packages/compartment-mapper/test/fixtures-policy/node_modules/eve/getcarol.js
diff --git a/packages/compartment-mapper/test/fixtures-policy/node_modules/eve/index.js b/packages/compartment-mapper/test/fixtures-policy/node_modules/eve/index.js
diff --git a/...ages/compartment-mapper/test/fixtures-policy/node_modules/eve/node_modules/carol/index.js b/...ages/compartment-mapper/test/fixtures-policy/node_modules/eve/node_modules/carol/index.js
diff --git a/.../compartment-mapper/test/fixtures-policy/node_modules/eve/node_modules/carol/package.json b/.../compartment-mapper/test/fixtures-policy/node_modules/eve/node_modules/carol/package.json
diff --git a/packages/compartment-mapper/test/fixtures-policy/node_modules/eve/package.json b/packages/compartment-mapper/test/fixtures-policy/node_modules/eve/package.json
diff --git a/packages/compartment-mapper/test/test-policy.js b/packages/compartment-mapper/test/test-policy.js
@@ -20,6 +20,10 @@ const fixtureAttack = new URL(
   'fixtures-policy/node_modules/app/attack.js',
   import.meta.url,
 ).toString();
+const nameConfusionAttack = new URL(
+  'fixtures-policy/node_modules/app/attack-name.js',
+  import.meta.url,
+).toString();
 
 const globals = {
   redPill: 42,
@@ -84,6 +88,15 @@ const anyPolicy = {
     'alice>carol': ANY,
   },
 };
+const evePolicy = {
+  entry: { ...policy.entry, packages: 'any' },
+  resources: {
+    ...policy.resources,
+    eve: {
+      packages: 'any',
+    },
+  },
+};
 
 const defaultExpectations = {
   namespace: {
@@ -110,6 +123,34 @@ const anyExpectations = {
     carol: { bluePill: 'number', redPill: 'number', purplePill: 'number' },
   },
 };
+const nameConfusionExpectations = {
+  namespace: {
+    alice: {
+      bluePill: 'undefined',
+      purplePill: 'undefined',
+      redPill: 'number',
+    },
+    bob: {
+      bluePill: 'number',
+      purplePill: 'undefined',
+      redPill: 'undefined',
+    },
+    carol: {
+      bluePill: 'undefined',
+      purplePill: 'number',
+      redPill: 'undefined',
+    },
+    evilCarol: {
+      bluePill: 'undefined',
+      purplePill: 'undefined',
+      redPill: 'undefined',
+    },
+    scopedBob: {
+      scoped: 1,
+    },
+  },
+};
+
 const powerlessCarolExpectations = {
   namespace: {
     ...defaultExpectations.namespace,
@@ -225,6 +266,18 @@ scaffold(
   },
 );
 
+scaffold(
+  'policy - attack - duplicated name via bundled dep',
+  test.only,
+  nameConfusionAttack,
+  makeResultAssertions(nameConfusionExpectations),
+  1, // expected number of assertions
+  {
+    addGlobals: globals,
+    policy: evePolicy,
+  },
+);
+
 const recursiveEdit = editor => originalPolicy => {
   const policyToAlter = JSON.parse(JSON.stringify(originalPolicy));
   const recur = obj => {