Refactoring test classes to the new clases names

Author: silva-fabio

packages/opencensus-core/package.json

@@ -4,6 +4,7 @@
   "description": "OpenCensus is a toolkit for collecting application performance and behavior data.",
   "main": "build/src/index.js",
   "types": "build/src/index.d.js",
+  "repository": "census-instrumentation/opencensus-node",
   "scripts": {
     "test": "npm run script run-unit-tests",
     "compile-all": "npm run script compile-auto",

packages/opencensus-core/src/trace/instrumentation/types.ts

@@ -14,24 +14,24 @@
  * limitations under the License.
  */
 
-export interface Plugin <TM> {
-    applyPatch(module: any, manager: TM, version: string): void; 
+export interface Plugin <T> {
+    applyPatch(module: any, tracer: T, version: string): void; 
 }
 
-export abstract class BasePlugin <TM>  {
+export abstract class BasePlugin <T>  {
 
     public module: any;
     public moduleName: string;
-    public traceManager: TM;
+    public tracer: T;
     public version: string;
 
     constructor (moduleName: string) {
         this.moduleName = moduleName;
     }
 
-    setPluginContext(http: any, traceManager: TM, version: string) {
+    setPluginContext(http: any, tracer: T, version: string) {
         this.module = http;
-        this.traceManager = traceManager;
+        this.tracer = tracer;
         this.version = version;     
     }
 

packages/opencensus-core/test/test-span.ts

@@ -14,8 +14,8 @@
  * limitations under the License.
  */
 
-import { Span } from '../src/trace/span';
-import { Trace } from '../src/trace/trace';
+import { Span } from '../src/trace/model/span';
+import { Trace } from '../src/trace/model/trace';
 
 var assert = require('assert');
 

packages/opencensus-core/test/test-trace.ts

@@ -14,8 +14,8 @@
  * limitations under the License.
  */
 
-import { Trace } from '../src/trace/trace';
-import { Span } from '../src/trace/span';
+import { Trace } from '../src/trace/model/trace';
+import { Span } from '../src/trace/model/span';
 
 let assert = require('assert');
 

…pencensus-core/test/test-tracemanager.ts …ages/opencensus-core/test/test-tracer.tspackages/opencensus-core/test/test-tracemanager.ts renamed to packages/opencensus-core/test/test-tracer.ts packages/opencensus-core/test/test-tracemanager.ts renamed to packages/opencensus-core/test/test-tracer.ts

@@ -14,48 +14,55 @@
  * limitations under the License.
  */
 
-import { TraceManager } from '../src/trace/tracemanager';
-import { Trace } from '../src/trace/trace';
-import { Span } from '../src/trace/span';
+import { Tracer } from '../src/trace/model/tracer';
+import { Trace } from '../src/trace/model/trace';
+import { Span } from '../src/trace/model/span';
+import { Exporter } from '../src/exporters/exporter';
 
 let assert = require('assert');
 
-describe('TraceManager', function () {
-  describe('new TraceManager()', function () {
-    it('should be a TraceManager instance', function () {
-      let traceManager = new TraceManager();
-      assert.ok(traceManager instanceof TraceManager);
+class NoopExporter implements Exporter {
+  emit(trace: Trace) {}
+}
+
+let noopExporter = new NoopExporter();
+
+describe('Tracer', function () {
+  describe('new Tracer()', function () {
+    it('should be a Tracer instance', function () {
+      let tracer = new Tracer(noopExporter);
+      assert.ok(tracer instanceof Tracer);
     });
   });
 
   describe('start()', function () {
-    let traceManager = new TraceManager();
-    let traceManagerStarted = traceManager.start();
+    let tracer = new Tracer(noopExporter);
+    let tracerStarted = tracer.start();
 
-    it('should return a TraceManager instance', function () {
-      assert.ok(traceManagerStarted instanceof TraceManager);
+    it('should return a tracer instance', function () {
+      assert.ok(tracerStarted instanceof Tracer);
     });
 
     it('should set true on active property', function () {
-      assert.ok(traceManagerStarted.active);
+      assert.ok(tracerStarted.active);
     });
   });
 
   describe('startTrace()', function () {
-    let traceManager;
+    let tracer;
     let trace;
 
     before(() => {
-      traceManager = new TraceManager();
-      trace = traceManager.startTrace();
+      tracer = new Tracer(noopExporter);
+      trace = tracer.startTrace();
     })
 
     it('should return a Trace instance', function () {
       assert.ok(trace instanceof Trace);
     });
 
     it('the new trace was set as current trace', function () {
-      assert.equal(traceManager.currentTrace.id, trace.id);
+      assert.equal(tracer.currentTrace.id, trace.id);
     });
 
     it('the new trace was started', function () {
@@ -65,26 +72,26 @@ describe('TraceManager', function () {
 
   describe('endTrace()', function () {
     it('the current trace was ended', function () {
-      let traceManager = new TraceManager();
-      let trace = traceManager.startTrace();
-      traceManager.endTrace();
+      let tracer = new Tracer(noopExporter);
+      let trace = tracer.startTrace();
+      tracer.endTrace();
       assert.ok(trace.ended);
     });
   });
 
   describe('clearCurrentTrace()', function () {
     it('the current trace is null', function () {
-      let traceManager = new TraceManager();
-      let trace = traceManager.startTrace();
-      traceManager.clearCurrentTrace();
-      assert.ok(traceManager.currentTrace == null);
+      let tracer = new Tracer(noopExporter);
+      let trace = tracer.startTrace();
+      tracer.clearCurrentTrace();
+      assert.ok(tracer.currentTrace == null);
     });
   });
 
   describe('startSpan()', function () {
-    let traceManager = new TraceManager();
-    let trace = traceManager.startTrace();
-    let span = traceManager.startSpan("spanName", "spanType");
+    let tracer = new Tracer(noopExporter);
+    let trace = tracer.startTrace();
+    let span = tracer.startSpan("spanName", "spanType");
     it('should return a Span instance', function () {
       assert.ok(span instanceof Span);
     });