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

…name collision

packages/compartment-mapper/test/scaffold.js

@@ -291,13 +291,15 @@ export function scaffold(
       };
 
       const sourceMaps = new Set();
+      const sourceMapLog = [];
       const sourceMapHook = (sourceMap, { sha512 }) => {
         sourceMaps.add(sha512);
-        t.log(sha512, sourceMap);
+        sourceMapLog.push(['+', sha512, sourceMap]);
       };
 
       const computeSourceMapLocation = ({ sha512 }) => {
         sourceMaps.delete(sha512);
+        sourceMapLog.push(['-', sha512]);
         return `${sha512}.map.json`;
       };
 
@@ -324,8 +326,10 @@ export function scaffold(
       // in a way that is difficult to generalize since not all test paths
       // reach here.
       if (sourceMaps.size !== 0) {
+        sourceMapLog.forEach(l => t.log(...l));
+        t.log('left-over', sourceMaps);
         throw new Error(
-          'The bundler and importer should agree on source map count',
+          `The bundler and importer should agree on source map count but they differ by ${sourceMaps.size}`,
         );
       }
 

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',
+    },
+    evilAlice: {
+      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,
+  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 => {