fix: pass spread args instead of args; add tests

src/logger.ts

@@ -65,32 +65,32 @@ export class Logger {
 
     this.logger = consoleLogLevel({
       stderr: true,
-      prefix: opts && opts.tag ? opts.tag : '@google-cloud/trace-agent',
+      prefix: opts && opts.tag ? opts.tag : 'unknown',
       level: levelName as ConsoleLogLevel
     });
   }
 
   error(...args: Array<{}>): void {
     if (this.logger) {
-      this.logger.error(args);
+      this.logger.error(...args);
     }
   }
 
   warn(...args: Array<{}>): void {
     if (this.logger) {
-      this.logger.warn(args);
+      this.logger.warn(...args);
     }
   }
 
   debug(...args: Array<{}>): void {
     if (this.logger) {
-      this.logger.debug(args);
+      this.logger.debug(...args);
     }
   }
 
   info(...args: Array<{}>): void {
     if (this.logger) {
-      this.logger.info(args);
+      this.logger.info(...args);
     }
   }
 }

test/logger.ts

@@ -32,7 +32,7 @@ export class TestLogger extends Logger {
       new OriginalLogger({level: LEVELS[PASS_THROUGH_LOG_LEVEL]});
 
   constructor(options?: Partial<LoggerConfig>) {
-    super(options);
+    super(Object.assign({tag: '@google-cloud/trace-agent'}, options));
   }
 
   private makeLoggerFn(logLevel: keyof Logger): LoggerFunction<this> {

test/plugins/common.ts

@@ -30,10 +30,10 @@ import { StackdriverTracer } from '../../src/trace-api';
 import { traceWriter } from '../../src/trace-writer';
 import * as TracingPolicy from '../../src/tracing-policy';
 import { SpanType } from '../../src/constants';
+import { TestLogger } from '../logger';
 
 var semver = require('semver');
 
-var logger = require('@google-cloud/common').logger;
 var trace = require('../../..');
 if (semver.satisfies(process.version, '>=8')) {
   // Monkeypatch Mocha's it() to create a fresh context with each test case.
@@ -75,7 +75,7 @@ shimmer.wrap(trace, 'start', function(original) {
       ignoreContextHeader: false,
       rootSpanNameOverride: (name: string) => name,
       samplingRate: 0
-    }, logger());
+    }, new TestLogger());
     testTraceAgent.policy = new TracingPolicy.TraceAllPolicy();
     return result;
   };

test/test-env-log-level.ts

@@ -14,12 +14,10 @@
  * limitations under the License.
  */
 
-import * as common from '@google-cloud/common';
 import * as assert from 'assert';
 import * as shimmer from 'shimmer';
 
 import * as logger from '../src/logger';
-import {FORCE_NEW} from '../src/util';
 
 import {TestLogger} from './logger';
 import * as traceTestModule from './trace';
@@ -69,6 +67,6 @@ describe('should respect environment variables', () => {
     assert.strictEqual(logLevel, logger.LEVELS[0]);
     process.env.GCLOUD_TRACE_LOGLEVEL = '300';
     traceTestModule.start();
-    assert.strictEqual(logLevel, logger.LEVELS[5]);
+    assert.strictEqual(logLevel, logger.LEVELS[4]);
   });
 });

test/test-modules-loaded-before-agent.ts

@@ -14,7 +14,6 @@
  * limitations under the License.
  */
 
-import * as common from '@google-cloud/common';
 import * as assert from 'assert';
 import * as shimmer from 'shimmer';
 

test/trace.ts

@@ -34,7 +34,6 @@
  * Most tests should include this file instead of the main module root.
  */
 
-import * as common from '@google-cloud/common';
 import * as assert from 'assert';
 import * as shimmer from 'shimmer';