Added tests for 100% coverage

Added test to make sure no process.domain change happens
hapijs · Jul 30, 2015 · bfc0c0b · bfc0c0b
1 parent 1d8e779
commit bfc0c0b
Showing 1 changed file with 85 additions and 2 deletions.
diff --git a/test/policy.js b/test/policy.js
@@ -4,7 +4,7 @@ var Catbox = require('..');
 var Code = require('code');
 var Lab = require('lab');
 var Import = require('./import');
-
+var Domain = require('domain');
 
 // Declare internals
 
@@ -18,7 +18,6 @@ var describe = lab.experiment;
 var it = lab.test;
 var expect = Code.expect;
 
-
 describe('Policy', function () {
 
     it('returns cached item', function (done) {
@@ -1281,6 +1280,90 @@ describe('Policy', function () {
                 });
             });
         });
+
+        describe('get()', function(){
+            it('it only binds if domain exists', function (done) {
+                var policy = new Catbox.Policy({
+                    expiresIn: 1000,
+                    staleIn: 100,
+                    generateFunc: function (id, next) {
+                        setTimeout(function(){
+                            return next(null, true);
+                        }, 20);
+                    },
+                    staleTimeout: 50
+                }, new Catbox.Client(Import), 'test');
+
+                var tests = 0;
+                var completed = 0;
+
+                var checkAndDone = process.domain.bind(function (expected, actual) {    // Bind back to the lab domain
+                    expect(actual).to.not.exist();
+                    expect(expected).to.not.exist();
+                    expect(actual).to.not.equal(expected, process.domain);      // This should be the lab domain
+
+                    if (tests === completed) {
+                        done();
+                    }
+                });
+
+                var test = function(domain){
+                    tests++;
+
+                    Domain.create().run(function () {
+                        process.domain = domain;
+
+                        policy.get('', function (err, result) {
+                            completed++;
+                            checkAndDone(domain, process.domain);
+                        });
+                    });
+                };
+
+                test(null);
+            });
+
+            it('it returns with the correct process domain', function (done) {
+                var policy = new Catbox.Policy({
+                    expiresIn: 1000,
+                    staleIn: 100,
+                    generateFunc: function (id, next) {
+                        setTimeout(function(){
+                            return next(null, true);
+                        }, 20);
+                    },
+                    staleTimeout: 50
+                }, new Catbox.Client(Import), 'test');
+
+                var tests = 0;
+                var completed = 0;
+
+                var checkAndDone = process.domain.bind(function (expected, actual) {
+                    expect(actual).to.equal(expected);
+
+                    if (tests === completed) {
+                        done();
+                    }
+                });
+
+                var test = function(domain){
+                    tests++;
+
+                    Domain.create().run(function () {
+                        process.domain.name = domain;
+
+                        policy.get('', function (err, result) {
+                            completed++;
+                            checkAndDone(domain, process.domain.name);
+                        });
+                    });
+                };
+
+                for (var i = 0; i < 10; ++i){
+                    test(i);
+                }
+            });
+        });
     });
 
     describe('drop()', function () {