test: crash on missing allowScripts entry

Author: dominykas

test/fixtures/index.js

@@ -20,6 +20,7 @@ exports.setup = (main, deps) => {
     Rimraf.sync(cwd);
     Mkdirp.sync(cwd);
     Fs.copyFileSync(Path.join(__dirname, `${main}.json`), Path.join(cwd, 'package.json'));
+    Fs.writeFileSync(Path.join(cwd, 'res.txt'), '');
     delete require.cache[Path.join(cwd, 'package.json')];
 
     deps.forEach((dep) => {
@@ -43,8 +44,11 @@ exports.setup = (main, deps) => {
     });
 
     return {
-        expectedResult: Fs.readFileSync(Path.join(__dirname, `${main}.txt`)).toString().trim(),
-        getResults: () => {
+        getExpectedResult: () => {
+
+            return Fs.readFileSync(Path.join(__dirname, `${main}.txt`)).toString().trim();
+        },
+        getActualResult: () => {
 
             return Fs.readFileSync(output).toString().trim();
         },

test/fixtures/not-in-allowed.json

@@ -0,0 +1,13 @@
+{
+  "private": true,
+  "name": "@example/not-in-allowed",
+  "version": "0.0.0",
+  "dependencies": {
+    "@example/with-install-script": "*",
+    "@example/with-postinstall-script": "*",
+    "@example/without-scripts": "*"
+  },
+  "allowScripts": {
+    "@example/with-postinstall-script": "*"
+  }
+}

test/index.js

@@ -41,9 +41,24 @@ describe('allow-scripts', () => {
 
             await Allow.run({});
 
-            expect(fixture.getResults()).to.equal(fixture.expectedResult);
+            expect(fixture.getActualResult()).to.equal(fixture.getExpectedResult());
             expect(fixture.getLog()).not.to.contain('without-scripts');
             expect(fixture.getLog()).not.to.contain('without-install-scripts');
         });
+
+        it('crashes on script not in allowed list', async () => {
+
+            const fixture = Fixtures.setup('not-in-allowed', [
+                'with-install-script',
+                'with-postinstall-script',
+                'without-scripts'
+            ]);
+
+            await expect(Allow.run({})).to.reject('No entry for @example/with-install-script');
+
+            expect(fixture.getActualResult()).not.to.contain('with-postinstall-script');
+            expect(fixture.getActualResult()).to.equal('');
+            expect(fixture.getLog()).to.equal('');
+        });
     });
 });