@neoskop/aviation-client - npm Package Compare versions

Comparing version 0.2.0 to 0.2.1

dist/model/aviation-feature.js

@@ -41,8 +41,11 @@ "use strict";
         if (this._functionEnabled) {
-            var hostname = void 0;
+            var hostname = '';
+            var preview = false;
             if (request) {
-                hostname = request.headers.host;
+                hostname = request.headers['host'];
+                preview = request.headers['x-aviation-preview'] && request.headers['x-aviation-preview'] == 'true';
             }
             else if (typeof window != 'undefined') {
                 hostname = window.location.hostname;
+                preview = window.localStorage.getItem('aviationPreview') && window.localStorage.getItem('aviationPreview') == 'true';
             }
@@ -49,0 +52,0 @@ var returnValue = eval('(function(){' + this._functionCode + '})()');

dist/test/aviation-feature.spec.js

@@ -6,4 +6,12 @@ "use strict";
 var httpMocks = require("node-mocks-http");
+var jsdom = require("jsdom");
+var dom = new jsdom.JSDOM('');
 before(function () {
     chai.should();
+    global['document'] = dom;
+    global['window'] = dom.window;
+    var nodeLocalStorage = require('node-localstorage');
+    var LocalStorage = nodeLocalStorage.LocalStorage;
+    global['localStorage'] = new LocalStorage('./tmp');
+    global['window'].localStorage = global['localStorage'];
 });
@@ -71,3 +79,41 @@ describe('Aviation feature', function () {
     });
+    it('should allow the function to check the hostname in browser context', function () {
+        var sut = new aviation_feature_1.AviationFeature({
+            name: 'foo',
+            enabled: false,
+            functionCode: 'return hostname == "foo"',
+            functionEnabled: true
+        });
+        dom.reconfigure({ url: "https://foo/bar" });
+        return sut.evaluate().should.be.true;
+    });
+    it('should should be testable in node.js contexts', function () {
+        var sut = new aviation_feature_1.AviationFeature({
+            name: 'foo',
+            enabled: false,
+            functionCode: 'return preview',
+            functionEnabled: true
+        });
+        var req = httpMocks.createRequest({
+            url: 'https://foo',
+            headers: {
+                Host: 'foo',
+                'X-Aviation-Preview': 'true'
+            }
+        });
+        return sut.evaluate(req).should.be.true;
+    });
+    it('should should be testable in browser contexts', function () {
+        var sut = new aviation_feature_1.AviationFeature({
+            name: 'foo',
+            enabled: false,
+            functionCode: 'return preview',
+            functionEnabled: true
+        });
+        window.localStorage.setItem('aviationPreview', 'true');
+        var result = sut.evaluate().should.be.true;
+        window.localStorage.removeItem('aviationPreview');
+        return result;
+    });
 });
 //# sourceMappingURL=aviation-feature.spec.js.map

package.json

 {
   "name": "@neoskop/aviation-client",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "main": "dist/index.js",
@@ -22,3 +22,3 @@ "typings": "dist/index.d.ts",
   "devDependencies": {
-    "@types/chai": "^4.0.0",
+    "@types/chai": "^4.0.1",
     "@types/chai-as-promised": "^0.0.31",
@@ -37,4 +37,4 @@ "@types/express": "^4.0.36",
     "ts-node": "^3.0.6",
-    "typescript": "^2.3.4"
+    "typescript": "^2.4.1"
   }
 }

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

32077

increased by13.6%

Lines of code

445

increased by12.37%

No dependency changes