fix: fixed typescript code style

Author: djonathascardoso

packages/opencensus-core/src/internal/cls-ah.ts

@@ -14,10 +14,10 @@
  * limitations under the License.
  */
 
-import * as asyncHook from 'async_hooks'
-import {Context, Func, Namespace as CLSNamespace} from 'continuation-local-storage'
-import {EventEmitter} from 'events'
-import * as shimmer from 'shimmer'
+import * as asyncHook from 'async_hooks';
+import {Context, Func, Namespace as CLSNamespace} from 'continuation-local-storage';
+import {EventEmitter} from 'events';
+import * as shimmer from 'shimmer';
 
 const wrappedSymbol = Symbol('context_wrapped');
 let contexts: {[asyncId: number]: Context;} = {};

packages/opencensus-core/src/internal/cls.ts

@@ -14,20 +14,23 @@
  * limitations under the License.
  */
 
-import * as CLS from 'continuation-local-storage'
-import * as semver from 'semver'
-import { debug } from './util'
+import * as CLS from 'continuation-local-storage';
+import * as semver from 'semver';
+
+import * as cls_ah from './cls-ah';
+import {debug} from './util';
 
 export type Namespace = CLS.Namespace;
 export type Func<T> = CLS.Func<T>;
 
-const useAsyncHooks: boolean = semver.satisfies(process.version, '>=8') ;//&&
-   // !!process.env.GCLOUD_TRACE_NEW_CONTEXT;
+const useAsyncHooks: boolean = semver.satisfies(
+    process.version, '>=8');  //&&
+                              // !!process.env.GCLOUD_TRACE_NEW_CONTEXT;
 
-debug('useAsyncHooks = %s',useAsyncHooks);
+debug('useAsyncHooks = %s', useAsyncHooks);
 
 const cls: typeof CLS =
-    useAsyncHooks ? require('./cls-ah') : require('continuation-local-storage');
+    useAsyncHooks ? cls_ah : CLS;
 
 
 const TRACE_NAMESPACE = 'opencensus.io';
@@ -51,5 +54,3 @@ export function destroyNamespace(): void {
 export function getNamespace(): CLS.Namespace {
   return cls.getNamespace(TRACE_NAMESPACE);
 }
-
-

packages/opencensus-core/src/trace/model/root-span.ts

@@ -18,11 +18,11 @@ import * as uuid from 'uuid';
 
 import {Clock} from '../../internal/clock';
 import {debug} from '../../internal/util';
-import {OnEndSpanEventListener, RootSpan, TraceContext, TraceOptions, Span, Tracer} from './types';
 
 import {SpanImpl} from './span';
 import {SpanBaseModel} from './spanbasemodel';
 import {TracerImpl} from './tracer';
+import {OnEndSpanEventListener, RootSpan, Span, TraceContext, TraceOptions, Tracer} from './types';
 
 /** Defines a root span */
 export class RootSpanImpl extends SpanBaseModel implements RootSpan {
@@ -45,21 +45,25 @@ export class RootSpanImpl extends SpanBaseModel implements RootSpan {
     this.spansLocal = [];
   }
 
-  get spans() {
+  /** Get span list from rootspan instance */
+  get spans(): Span[] {
     return this.spansLocal;
   }
 
-  get traceId() {
+  /** Get trace id from rootspan instance */
+  get traceId(): string {
     return this.traceIdLocal;
   }
 
+  /** Start a rootspan instance */
   start() {
     super.start();
     debug(
         'starting %s  %o', this.className,
         {traceId: this.traceId, id: this.id, parentSpanId: this.parentSpanId});
   }
 
+  /** End a rootspan instance */
   end() {
     super.end();
 
@@ -72,6 +76,10 @@ export class RootSpanImpl extends SpanBaseModel implements RootSpan {
     this.tracer.onEndSpan(this);
   }
 
+  /**
+   * Event called when a span ended
+   * @param span Span ended
+   */
   onEndSpan(span: Span) {
     debug('ending span  %o', {
       id: span.id,
@@ -83,6 +91,12 @@ export class RootSpanImpl extends SpanBaseModel implements RootSpan {
     });
   }
 
+  /**
+   * Start a new span linked with the rootspan
+   * @param name Span name
+   * @param type Span type
+   * @param parentSpanId Span parent ID
+   */
   startSpan(name: string, type: string, parentSpanId?: string) {
     if (!this.started) {
       debug(

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

@@ -22,14 +22,14 @@ import {SpanImpl} from '../src/trace/model/span';
 import {TracerImpl} from '../src/trace/model/tracer';
 import {Span} from '../src/trace/model/types';
 
-let tracer = new TracerImpl();
+const tracer = new TracerImpl();
 
-describe('Span', function() {
+describe('Span', () => {
   /**
    * Should create a span
    */
-  describe('new Span()', function() {
-    it('should create a Span instance', function() {
+  describe('new Span()', () => {
+    it('should create a Span instance', () => {
       const rootSpan = new RootSpanImpl(tracer);
       const span = new SpanImpl(rootSpan);
       assert.ok(span instanceof SpanImpl);
@@ -39,8 +39,8 @@ describe('Span', function() {
   /**
    * Should return the Trace ID
    */
-  describe('get traceId()', function() {
-    it('should return the trace id', function() {
+  describe('get traceId()', () => {
+    it('should return the trace id', () => {
       const rootSpan = new RootSpanImpl(tracer);
       rootSpan.start();
       const span = new SpanImpl(rootSpan);
@@ -51,11 +51,11 @@ describe('Span', function() {
   /**
    * Should the trace context of span
    */
-  describe('get traceContext()', function() {
-    it('should the trace context of span', function() {
+  describe('get traceContext()', () => {
+    it('should the trace context of span', () => {
       const rootSpan = new RootSpanImpl(tracer);
       rootSpan.start();
-      
+
       const span = new SpanImpl(rootSpan);
       const context = span.traceContext;
 
@@ -68,11 +68,11 @@ describe('Span', function() {
   /**
    * Should start a span instance
    */
-  describe('start()', function() {
-    it('should start a span instance', function() {
+  describe('start()', () => {
+    it('should start a span instance', () => {
       const rootSpan = new RootSpanImpl(tracer);
       rootSpan.start();
-      
+
       const span = new SpanImpl(rootSpan);
       span.start();
 
@@ -83,11 +83,11 @@ describe('Span', function() {
   /**
    * Should end a span instance
    */
-  describe('end()', function() {
-    it('should end a span instance', function() {
+  describe('end()', () => {
+    it('should end a span instance', () => {
       const rootSpan = new RootSpanImpl(tracer);
       rootSpan.start();
-      
+
       const span = new SpanImpl(rootSpan);
       span.start();
       span.end();
@@ -99,11 +99,11 @@ describe('Span', function() {
   /**
    * Should not end a span instance
    */
-  describe('end() before start the span', function() {
-    it('should not end a span instance', function() {
+  describe('end() before start the span', () => {
+    it('should not end a span instance', () => {
       const rootSpan = new RootSpanImpl(tracer);
       rootSpan.start();
-      
+
       const span = new SpanImpl(rootSpan);
       span.end();