fix: minor tslint issues

Author: silva-fabio

packages/opencensus-core/src/exporters/console-exporter.ts

@@ -22,13 +22,13 @@ import {Logger} from '../common/types';
 
 /** Do not send span data */
 export class NoopExporter implements Exporter {
-  logger: Logger
+  logger: Logger;
   onEndSpan(root: RootSpan) {}
   publish(rootSpans: RootSpan[]) {}
 }
 
 /** Format and sends span data to the console. */
-export class ConsoleLogExporter implements Exporter {
+export class ConsoleExporter implements Exporter {
   /** Buffer object to store the spans. */
   private buffer: Buffer;
   logger: Logger;

packages/opencensus-core/src/exporters/types.ts

@@ -17,12 +17,12 @@
 
 import {OnEndSpanEventListener, RootSpan} from '../trace/model/types';
 import {BufferConfig} from '../trace/config/types';
-import {Logger} from '../common/types'
+import {Logger} from '../common/types';
 
 /** Defines an exporter interface. */
 export interface Exporter extends OnEndSpanEventListener {
   /** An object to log information to */
-  logger: Logger
+  logger: Logger;
 
   /**
    * Sends a list of root spans to the service.

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

@@ -15,7 +15,7 @@
  */
 
 import {Config, Sampler, TracerConfig} from '../config/types';
-import {Logger} from '../../common/types'
+import {Logger} from '../../common/types';
 
 
 /** Default type for functions */

packages/opencensus-core/test/test-console-exporter.ts

@@ -18,7 +18,7 @@ import * as assert from 'assert';
 import * as mocha from 'mocha';
 
 import {Buffer} from '../src/exporters/buffer';
-import {ConsoleLogExporter, NoopExporter} from '../src/exporters/consolelog-exporter';
+import {ConsoleExporter, NoopExporter} from '../src/exporters/consolelog-exporter';
 import {RootSpanImpl} from '../src/trace/model/rootspan';
 import {TracerImpl} from '../src/trace/model/tracer';
 import {RootSpan} from '../src/trace/model/types';
@@ -60,7 +60,7 @@ describe('ConsoleLogExporter', () => {
   /** Should end a span */
   describe('onEndSpan()', () => {
     it('should end a span', () => {
-      const exporter = new ConsoleLogExporter(defaultBufferConfig);
+      const exporter = new ConsoleExporter(defaultBufferConfig);
       const rootSpan = new RootSpanImpl(tracer);
       exporter.onEndSpan(rootSpan);
       assert.ok(true);
@@ -70,7 +70,7 @@ describe('ConsoleLogExporter', () => {
   /** Should publish the rootspan in queue */
   describe('publish()', () => {
     it('should publish the rootspans in queue', () => {
-      const exporter = new ConsoleLogExporter(defaultBufferConfig);
+      const exporter = new ConsoleExporter(defaultBufferConfig);
       const rootSpan = new RootSpanImpl(tracer);
       rootSpan.startSpan('name', 'type', rootSpan.traceId);
       const queue: RootSpan[] = [];

packages/opencensus-core/test/test-console-logger.ts

@@ -16,15 +16,15 @@
 
 import * as assert from 'assert';
 import * as mocha from 'mocha';
-import * as logger from '../src/common/consolelogger';
 
-import {Logger} from '../src/common/types'
-import {RootSpanImpl} from '../src/trace/model/rootspan'
+import * as logger from '../src/common/consolelogger';
+import {Logger} from '../src/common/types';
+import {Buffer} from '../src/exporters/buffer';
+import {ConsoleExporter} from '../src/exporters/consolelog-exporter';
+import {BufferConfig, TracerConfig} from '../src/trace/config/types';
+import {RootSpanImpl} from '../src/trace/model/rootspan';
 import {TracerImpl} from '../src/trace/model/tracer';
-import {TracerConfig, BufferConfig} from '../src/trace/config/types';
 import {TraceOptions} from '../src/trace/model/types';
-import {Buffer} from '../src/exporters/buffer';
-import {ConsoleLogExporter} from '../src/exporters/consolelog-exporter'
 
 const LEVELS = ['error', 'warn', 'info', 'debug', 'silly'];
 let consoleTxt = '';
@@ -36,7 +36,7 @@ describe('ConsoleLogger', () => {
     return txt;
   });
 
-/** Should create a new ConsoleLogger */
+  /** Should create a new ConsoleLogger */
   describe('new ConsoleLogger()', () => {
     it('should consoleLogger with default levels', () => {
       const consoleLogger = logger();
@@ -100,10 +100,9 @@ describe('ConsoleLogger', () => {
       assert.equal(validateString, -1);
     });
   });
-  
+
   /** Should logger error, warn and info log */
-   describe('info logger', () => {
-     
+  describe('info logger', () => {
     const consoleLogger = logger(LEVELS[2]);
 
     it('should logger error', () => {
@@ -112,7 +111,7 @@ describe('ConsoleLogger', () => {
         consoleTxt = txt;
         return txt;
       });
-  
+
       consoleTxt = '';
       consoleLogger.error('error test logger');
       unhookIntercept();
@@ -127,7 +126,7 @@ describe('ConsoleLogger', () => {
         consoleTxt = txt;
         return txt;
       });
-  
+
       consoleTxt = '';
       consoleLogger.warn('warn test logger');
       unhookIntercept();
@@ -142,7 +141,7 @@ describe('ConsoleLogger', () => {
         consoleTxt = txt;
         return txt;
       });
-  
+
       consoleTxt = '';
       consoleLogger.info('info test logger');
       unhookIntercept();
@@ -157,7 +156,7 @@ describe('ConsoleLogger', () => {
         consoleTxt = txt;
         return txt;
       });
-  
+
       consoleTxt = '';
       consoleLogger.debug('debug test logger');
       unhookIntercept();
@@ -172,7 +171,7 @@ describe('ConsoleLogger', () => {
         consoleTxt = txt;
         return txt;
       });
-  
+
       consoleTxt = '';
       consoleLogger.silly('silly test logger');
       unhookIntercept();
@@ -185,13 +184,10 @@ describe('ConsoleLogger', () => {
   describe('Model classes has a logger', () => {
     // tslint:disable:no-any
     function instanceOfLogger(object: any): object is Logger {
-      return 'error' in object
-      && 'warn' in object
-      && 'info' in object
-      && 'debug' in object
-      && 'silly' in object;
+      return 'error' in object && 'warn' in object && 'info' in object &&
+          'debug' in object && 'silly' in object;
     }
-    
+
     const consoleLogger = logger('debug');
 
     const tracer = new TracerImpl();
@@ -204,23 +200,22 @@ describe('ConsoleLogger', () => {
     it('checks if RootSpanImpl and SpanImpl has a logger', () => {
       tracer.startRootSpan({name: 'rootSpanTest'} as TraceOptions, (root) => {
         assert.ok(instanceOfLogger(root.logger));
-        
-        let span = tracer.startSpan('spanTest')
+
+        const span = tracer.startSpan('spanTest');
         assert.ok(instanceOfLogger(span.logger));
       });
     });
-    
-    let exporterConfig = {logger: consoleLogger};
-    let exporter = new ConsoleLogExporter(exporterConfig);
-    
+
+    const exporterConfig = {logger: consoleLogger};
+    const exporter = new ConsoleExporter(exporterConfig);
+
     it('checks if exporter has a logger', () => {
       assert.ok(instanceOfLogger(exporter.logger));
     });
 
     it('checks if buffer has a logger', () => {
-      let buffer = new Buffer(exporter, exporterConfig)
+      const buffer = new Buffer(exporter, exporterConfig);
       assert.ok(instanceOfLogger(buffer.logger));
     });
   });
-  
 });