test: restore Function#length property in wrapped mocha test function…

…s when using continuation-local-storage with node 8 (googleapis#553) PR-URL: googleapis#553
BenCrux · Sep 10, 2017 · 6f9614a · 6f9614a
1 parent e744614
commit 6f9614a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/test/plugins/common.js b/test/plugins/common.js
@@ -26,7 +26,7 @@ var semver = require('semver');
 var logger = require('@google-cloud/common').logger;
 var trace = require('../..');
 var cls = require('../../src/cls.js');
-if (semver.satisfies(process.version, '>=8')) {
+if (semver.satisfies(process.version, '>=8') && process.env.GCLOUD_TRACE_NEW_CONTEXT) {
   // Monkeypatch the monkeypatcher
   var oldIt = global.it;
   global.it = function it(title, fn) {

diff --git a/test/test-cls-ah.js b/test/test-cls-ah.js
@@ -19,11 +19,9 @@ var assert = require('assert');
 var http = require('http');
 var semver = require('semver');
 
-if (semver.satisfies(process.version, '<8')) {
+if (semver.satisfies(process.version, '<8') || !process.env.GCLOUD_TRACE_NEW_CONTEXT) {
   console.log('Skipping cls-ah tests on node version without async hooks');
   return;
-} else {
-  assert.ok(process.env.GCLOUD_TRACE_NEW_CONTEXT);
 }
 
 var cls = require('../src/cls-ah.js');