chord: merge commit

Author: luanamartinssantos

packages/opencensus-core/scripts/index.ts

@@ -14,13 +14,8 @@ const {
   TRACE_SYSTEM_TEST_ENCRYPTED_CREDENTIALS_IV
 } = process.env;
 
-import { checkInstall } from './check-install';
 import { compile } from './compile';
-import { encryptCredentials, decryptCredentials } from './credentials';
-import { initTestFixtures } from './init-test-fixtures';
-import { reportCoverage } from './report-coverage';
 import { runTests } from './run-tests';
-import { testNonInterference } from './test-non-interference';
 import { BUILD_DIRECTORY, existsP, spawnP } from './utils';
 
 // The identifying string in the service account credentials file path.
@@ -73,40 +68,6 @@ async function run(steps: string[]) {
             timeout: 4000
           });
           break;
-        case 'run-unit-tests-with-coverage':
-          await runTests({
-            includeGlobs: [
-              `${BUILD_DIRECTORY}/test/test-*.js`,
-              `${BUILD_DIRECTORY}/test/plugins/test-*.js`
-            ],
-            excludeGlobs: unitTestExcludeGlobs,
-            rootDir: BUILD_DIRECTORY,
-            coverage: true,
-            timeout: 4000
-          });
-          break;
-        case 'run-system-tests':
-          await spawnP(
-            'npm', ['install'], { cwd: 'system-test' }
-          );
-          if (CI_PULL_REQUEST && !(await existsP('node-team-test-d0b0be11c23d.json'))) {
-            console.log('> Not running system tests in PRs');
-          } else {
-            await runTests({
-              includeGlobs: [
-                `system-test/*.js`,
-              ],
-              rootDir: '.',
-              coverage: false
-            });
-          }
-          break;
-        case 'report-coverage':
-          await reportCoverage();
-          break;
-        case 'test-non-interference':
-          await testNonInterference();
-          break;
         default:
           console.log(`> ${step}: not found`);
           break;

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

@@ -1,10 +1,70 @@
 import { Trace } from '../src/trace/trace';
+import { Span } from '../src/trace/span';
 
-var assert = require('assert');
+let assert = require('assert');
 
-describe('Trace', function() {
-  it('should be a Trace instance', function() {
-    var trace = new Trace();
-    assert.ok(trace instanceof Trace);
+describe('Trace', function () {
+  describe('new Trace()', function () {    
+    it('should be a Trace instance', function () {
+      let trace = new Trace();
+      assert.ok(trace instanceof Trace);
+    });
   });
+
+  describe('start()', function () {
+    it('trace was started', function () {
+      let trace = new Trace();
+      trace.start();
+      assert.ok(trace.started);
+    });
+  });
+
+  describe('startSpan()', function () {
+    let trace = new Trace();
+    trace.start()
+    let span = trace.startSpan("spanName", "spanType");
+    it('should be a Span instance', function () {
+      assert.ok(span instanceof Span);
+    });
+
+    it('span was started', function () {
+      assert.ok(span.started);
+    });
+  });
+
+  describe('end()', function () {
+    it('trace was ended', function () {
+      let trace = new Trace();
+      trace.start()
+      trace.end();
+      assert.ok(trace.ended);
+    });
+  });
+
+  describe('end() before trace started', function () {
+    it('trace was not ended', function () {
+      let trace = new Trace();
+      trace.end();
+      assert.ok(!trace.ended);
+    });
+  });
+
+  describe('startSpan() before trace started', function () {
+    it('should return null', function () {
+      let trace = new Trace();
+      let span = trace.startSpan("spanName", "spanType");
+      assert.ok(span == null);
+    });
+  });
+
+  describe('startSpan() after trace ended', function () {
+    it('should return null', function () {
+      let trace = new Trace();
+      trace.start()
+      trace.end();
+      let span = trace.startSpan("spanName", "spanType");
+      assert.ok(span == null);
+    });
+  });
+
 });