don't skip annotating node unless fully bypassed (#620)

Author: jedireza

lib/coverage.js

@@ -109,7 +109,7 @@ internals.instrument = function (filename) {
 
         // Coverage status
 
-        if (bypass[node.range[0]]) {
+        if (bypass[node.range[0]] && bypass[node.range[1]]) {
             return;
         }
 

test/coverage.js

@@ -140,6 +140,19 @@ describe('Coverage', () => {
         done();
     });
 
+    it('bypasses marked code and reports misses correctly', (done) => {
+
+        const Test = require('./coverage/bypass-misses');
+        Test.method(1);
+
+        const cov = Lab.coverage.analyze({ coveragePath: Path.join(__dirname, 'coverage/bypass-misses') });
+        expect(Math.floor(cov.percent)).to.equal(93);
+        expect(cov.sloc).to.equal(15);
+        expect(cov.misses).to.equal(1);
+        expect(cov.hits).to.equal(14);
+        done();
+    });
+
     it('ignores non-matching files', (done) => {
 
         require('./coverage/exclude/ignore');

test/coverage/bypass-misses.js

@@ -0,0 +1,27 @@
+'use strict';
+
+// Load modules
+
+
+// Declare internals
+
+const internals = {};
+
+/*$lab:coverage:off$*/const noop = function () {};
+
+const /*$lab:coverage:on$*/FiveMath = function () {
+
+    this.addFive = function (value) {
+
+        return value + 5;
+    };
+
+    this.subtractFive = function (value) {
+
+      return value - 5;
+    };
+};
+
+const fiveMath = new FiveMath();
+
+exports.method = fiveMath.addFive;