test: when nothing allowed; when semver range is invalid

Author: dominykas

package.json

@@ -7,7 +7,7 @@
     "url": "https://github.com/dominykas/allow-scripts.git"
   },
   "scripts": {
-    "test": "lab -L -t 83 -m 5000"
+    "test": "lab -L -t 86 -m 5000"
   },
   "bin": {
     "allow-scripts": "./bin/allow-scripts.js"

test/fixtures/allowed-semver.json

@@ -1,6 +1,6 @@
 {
   "private": true,
-  "name": "@example/allowed-false",
+  "name": "@example/allowed-semver",
   "version": "0.0.0",
   "dependencies": {
     "@example/with-install-script": "*",

test/fixtures/invalid-semver.json

@@ -0,0 +1,14 @@
+{
+  "private": true,
+  "name": "@example/invalid-semver",
+  "version": "0.0.0",
+  "dependencies": {
+    "@example/with-install-script": "*",
+    "@example/with-postinstall-script": "*",
+    "@example/without-scripts": "*"
+  },
+  "allowScripts": {
+    "@example/with-install-script": "not-a-semver-range",
+    "@example/with-postinstall-script": "*"
+  }
+}

test/fixtures/nothing-allowed.json

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

test/index.js

@@ -85,5 +85,33 @@ describe('allow-scripts', () => {
             expect(fixture.getActualResult()).to.equal('postinstall from with-postinstall-script');
             expect(fixture.getLog()).to.contain('skip node_modules/@example/with-install-script (because 0.0.0 is outside of allowed range: 1.x.x)');
         });
+
+        it('crashes on invalid semver range', async () => {
+
+            const fixture = Fixtures.setup('invalid-semver', [
+                'with-install-script',
+                'with-postinstall-script',
+                'without-scripts'
+            ]);
+
+            await expect(Allow.run({})).to.reject('Invalid version range in allowScripts[@example/with-install-script]: not-a-semver-range');
+
+            expect(fixture.getActualResult()).to.equal('');
+            expect(fixture.getLog()).to.equal('');
+        });
+
+        it('crashes on missing allowScripts section', async () => {
+
+            const fixture = Fixtures.setup('nothing-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()).to.equal('');
+            expect(fixture.getLog()).to.equal('');
+        });
     });
 });