fix(deps): update dependency @opencensus/propagation-stackdriver to v0.0.14 (#1045)

Author: renovate[bot]

package.json

@@ -94,7 +94,7 @@
   },
   "dependencies": {
     "@google-cloud/common": "^2.0.0",
-    "@opencensus/propagation-stackdriver": "0.0.13",
+    "@opencensus/propagation-stackdriver": "0.0.14",
     "builtin-modules": "^3.0.0",
     "console-log-level": "^1.4.0",
     "continuation-local-storage": "^3.2.1",

src/cls.ts

@@ -119,9 +119,7 @@ export class TraceCLS implements CLS<RootContext> {
       case TraceCLSMechanism.ASYNC_HOOKS:
         if (!asyncHooksAvailable) {
           throw new Error(
-            `CLS mechanism [${
-              config.mechanism
-            }] is not compatible with Node <8.`
+            `CLS mechanism [${config.mechanism}] is not compatible with Node <8.`
           );
         }
         this.CLSClass = AsyncHooksCLS;

src/plugins/plugin-http.ts

@@ -228,27 +228,23 @@ function patchHttp(http: HttpModule, api: Tracer) {
   if (semver.satisfies(process.version, '>=8.0.0')) {
     // http.get in Node 8 calls the private copy of request rather than the one
     // we have patched on module.export, so patch get as well.
-    shimmer.wrap(
-      http,
-      'get',
-      (): typeof http.get => {
-        // Re-implement http.get. This needs to be done (instead of using
-        // makeRequestTrace to patch it) because we need to set the trace
-        // context header before the returned ClientRequest is ended.
-        // The Node.js docs state that the only differences between request and
-        // get are that (1) get defaults to the HTTP GET method and (2) the
-        // returned request object is ended immediately.
-        // The former is already true (at least in supported Node versions up to
-        // v9), so we simply follow the latter.
-        // Ref:
-        // https://nodejs.org/dist/latest/docs/api/http.html#http_http_get_options_callback
-        return function getTrace(options, callback) {
-          const req = http.request(options, callback);
-          req.end();
-          return req;
-        };
-      }
-    );
+    shimmer.wrap(http, 'get', (): typeof http.get => {
+      // Re-implement http.get. This needs to be done (instead of using
+      // makeRequestTrace to patch it) because we need to set the trace
+      // context header before the returned ClientRequest is ended.
+      // The Node.js docs state that the only differences between request and
+      // get are that (1) get defaults to the HTTP GET method and (2) the
+      // returned request object is ended immediately.
+      // The former is already true (at least in supported Node versions up to
+      // v9), so we simply follow the latter.
+      // Ref:
+      // https://nodejs.org/dist/latest/docs/api/http.html#http_http_get_options_callback
+      return function getTrace(options, callback) {
+        const req = http.request(options, callback);
+        req.end();
+        return req;
+      };
+    });
   }
 }
 

src/trace-api.ts

@@ -201,9 +201,7 @@ export class StackdriverTracer implements Tracer {
     const rootSpan = cls.get().getContext();
     if (rootSpan.type === SpanType.ROOT && !rootSpan.span.endTime) {
       this.logger!.warn(
-        `TraceApi#runInRootSpan: [${
-          this.pluginNameToLog
-        }] Cannot create nested root spans.`
+        `TraceApi#runInRootSpan: [${this.pluginNameToLog}] Cannot create nested root spans.`
       );
       return fn(UNCORRELATED_ROOT_SPAN);
     }
@@ -319,11 +317,7 @@ export class StackdriverTracer implements Tracer {
         // seems to have some value, but isn't representable. The user probably
         // needs a custom outer span that encompasses the entirety of work.
         this.logger!.warn(
-          `TraceApi#createChildSpan: [${
-            this.pluginNameToLog
-          }] Creating phantom child span [${options.name}] because root span [${
-            rootSpan.span.name
-          }] was already closed.`
+          `TraceApi#createChildSpan: [${this.pluginNameToLog}] Creating phantom child span [${options.name}] because root span [${rootSpan.span.name}] was already closed.`
         );
         return UNCORRELATED_CHILD_SPAN;
       }
@@ -385,9 +379,7 @@ export class StackdriverTracer implements Tracer {
         skipFrames: options.skipFrames ? options.skipFrames + 1 : 1,
       });
       this.logger!.info(
-        `TraceApi#createChildSpan: [${
-          this.pluginNameToLog
-        }] Created child span [${options.name}]`
+        `TraceApi#createChildSpan: [${this.pluginNameToLog}] Created child span [${options.name}]`
       );
       return childContext;
     } else if (rootSpan.type === SpanType.UNSAMPLED) {
@@ -398,11 +390,7 @@ export class StackdriverTracer implements Tracer {
     } else {
       // Context was lost.
       this.logger!.warn(
-        `TraceApi#createChildSpan: [${
-          this.pluginNameToLog
-        }] Creating phantom child span [${
-          options.name
-        }] because there is no root span.`
+        `TraceApi#createChildSpan: [${this.pluginNameToLog}] Creating phantom child span [${options.name}] because there is no root span.`
       );
       return UNCORRELATED_CHILD_SPAN;
     }

src/trace-plugin-loader.ts

@@ -518,9 +518,7 @@ export class PluginLoader {
           }
         } catch (e) {
           this.logger.error(
-            `PluginLoader#getVersion: [${pjsonPath}] An error occurred while retrieving version string. ${
-              e.message
-            }`
+            `PluginLoader#getVersion: [${pjsonPath}] An error occurred while retrieving version string. ${e.message}`
           );
           version = null;
         }

test/plugins/test-cls-bluebird.ts

@@ -163,9 +163,7 @@ describe('Patch plugin for bluebird', () => {
 
   // tslint:disable-next-line:no-any
   testCases.forEach((testCase: TestCase<any>) => {
-    it(`enables context propagation in the same way as native promises for test case: ${
-      testCase.description
-    }`, async () => {
+    it(`enables context propagation in the same way as native promises for test case: ${testCase.description}`, async () => {
       const actual = (await getTracesForPromiseImplementation(
         testCase.makePromise,
         testCase.thenFn

test/plugins/test-trace-http.ts

@@ -249,9 +249,7 @@ for (const nodule of Object.keys(servers) as Array<keyof typeof servers>) {
       });
 
       for (const testCase of testCases) {
-        it(`creates spans with accurate timespans when ${
-          testCase.description
-        }`, async () => {
+        it(`creates spans with accurate timespans when ${testCase.description}`, async () => {
           let recordedTime = 0;
           await testTraceModule
             .get()

test/test-cls-ah.ts

@@ -200,9 +200,7 @@ maybeSkip(describe)('AsyncHooks-based CLS', () => {
     for (const testCase of testCases) {
       const skipIfTestSpecifies = !!testCase.skip ? it.skip : it;
       skipIfTestSpecifies(
-        `Doesn't retain stale references when running ${
-          testCase.description
-        } in a context`,
+        `Doesn't retain stale references when running ${testCase.description} in a context`,
         async () => {
           createdPromiseIDs.length = 0;
           try {

test/utils.ts

@@ -85,9 +85,7 @@ export function assertSpanDuration(span: TraceSpan, bounds: [number, number?]) {
   assert.ok(
     spanDuration >= lowerBound - ASSERT_SPAN_TIME_TOLERANCE_MS &&
       spanDuration <= upperBound + ASSERT_SPAN_TIME_TOLERANCE_MS,
-    `Span duration of ${spanDuration} ms is not in the acceptable expected range of [${
-      bounds[0]
-    }, ${bounds[1]}] ms (w/ ${ASSERT_SPAN_TIME_TOLERANCE_MS} ms leniency)`
+    `Span duration of ${spanDuration} ms is not in the acceptable expected range of [${bounds[0]}, ${bounds[1]}] ms (w/ ${ASSERT_SPAN_TIME_TOLERANCE_MS} ms leniency)`
   );
 }
 

test/web-frameworks/connect.ts

@@ -40,9 +40,7 @@ export class Connect3 implements WebFramework {
   addHandler(options: WebFrameworkAddHandlerOptions): void {
     if (!options.hasResponse && !options.blocking) {
       throw new Error(
-        `${
-          this.constructor.name
-        } wrapper for testing doesn't support non-blocking handlers.`
+        `${this.constructor.name} wrapper for testing doesn't support non-blocking handlers.`
       );
     }
     this.app.use(