ci: gts fix for ci build

Signed-off-by: Olivier Albertini <[email protected]>
open-telemetry · Aug 6, 2019 · bec4e2c · bec4e2c
1 parent 284aaeb
commit bec4e2c
Show file tree

Hide file tree

Showing 13 changed files with 248 additions and 85 deletions.
diff --git a/packages/opentelemetry-basic-tracer/src/BasicTracer.ts b/packages/opentelemetry-basic-tracer/src/BasicTracer.ts
@@ -15,7 +15,12 @@
  */
 
 import * as types from '@opentelemetry/types';
-import { ALWAYS_SAMPLER, BinaryTraceContext, HttpTraceContext, NOOP_SPAN } from '@opentelemetry/core';
+import {
+  ALWAYS_SAMPLER,
+  BinaryTraceContext,
+  HttpTraceContext,
+  NOOP_SPAN,
+} from '@opentelemetry/core';
 import { BinaryFormat, HttpTextFormat } from '@opentelemetry/types';
 import { BasicTracerConfig } from '../src/types';
 import { ScopeManager } from '@opentelemetry/scope-base';
@@ -29,7 +34,6 @@ export class BasicTracer implements types.Tracer {
   private readonly _binaryFormat: types.BinaryFormat;
   private readonly _httpTextFormat: types.HttpTextFormat;
   private readonly _sampler: types.Sampler;
-
   protected readonly _scopeManager: ScopeManager;
 
   /**
@@ -57,7 +61,7 @@ export class BasicTracer implements types.Tracer {
     }
 
     const spanOptions = Object.assign({}, options, {
-      parent: parentSpanContext
+      parent: parentSpanContext,
     });
     const span = new Span(this, name, spanOptions);
 
@@ -77,7 +81,10 @@ export class BasicTracer implements types.Tracer {
   /**
    * Enters the scope of code where the given Span is in the current context.
    */
-  withSpan<T extends (...args: unknown[]) => ReturnType<T>>(span: types.Span, fn: T): ReturnType<T> {
+  withSpan<T extends (...args: unknown[]) => ReturnType<T>>(
+    span: types.Span,
+    fn: T
+  ): ReturnType<T> {
     // Set given span to context.
     return this._scopeManager.with(span, fn);
   }
@@ -103,7 +110,9 @@ export class BasicTracer implements types.Tracer {
     return this._httpTextFormat;
   }
 
-  private _getParentSpanContext(parent: types.Span | types.SpanContext | undefined): types.SpanContext | undefined {
+  private _getParentSpanContext(
+    parent: types.Span | types.SpanContext | undefined
+  ): types.SpanContext | undefined {
     if (!parent) return undefined;
 
     // parent is a SpanContext

diff --git a/packages/opentelemetry-node-tracer/src/NodeTracer.ts b/packages/opentelemetry-node-tracer/src/NodeTracer.ts
@@ -25,7 +25,9 @@ export class NodeTracer extends BasicTracer {
    * Constructs a new Tracer instance.
    */
   constructor(config: BasicTracerConfig) {
-    super(Object.assign({}, { scopeManager: new AsyncHooksScopeManager() }, config));
+    super(
+      Object.assign({}, { scopeManager: new AsyncHooksScopeManager() }, config)
+    );
 
     // @todo: Integrate Plugin Loader (pull/126).
   }

diff --git a/packages/opentelemetry-node-tracer/src/index.ts b/packages/opentelemetry-node-tracer/src/index.ts
@@ -14,4 +14,4 @@
  * limitations under the License.
  */
 
- export * from './NodeTracer';
+export * from './NodeTracer';
diff --git a/packages/opentelemetry-node-tracer/test/NodeTracer.test.ts b/packages/opentelemetry-node-tracer/test/NodeTracer.test.ts
@@ -21,7 +21,7 @@ import {
   HttpTraceContext,
   NEVER_SAMPLER,
   NoopLogger,
-  NOOP_SPAN
+  NOOP_SPAN,
 } from '@opentelemetry/core';
 import { AsyncHooksScopeManager } from '@opentelemetry/scope-async-hooks';
 import { NodeTracer } from '../src/NodeTracer';
@@ -31,39 +31,39 @@ describe('NodeTracer', () => {
   describe('constructor', () => {
     it('should construct an instance with required only options', () => {
       const tracer = new NodeTracer({
-        scopeManager: new AsyncHooksScopeManager()
+        scopeManager: new AsyncHooksScopeManager(),
       });
       assert.ok(tracer instanceof NodeTracer);
     });
 
     it('should construct an instance with binary format', () => {
       const tracer = new NodeTracer({
         binaryFormat: new BinaryTraceContext(),
-        scopeManager: new AsyncHooksScopeManager()
+        scopeManager: new AsyncHooksScopeManager(),
       });
       assert.ok(tracer instanceof NodeTracer);
     });
 
     it('should construct an instance with http text format', () => {
       const tracer = new NodeTracer({
         httpTextFormat: new HttpTraceContext(),
-        scopeManager: new AsyncHooksScopeManager()
+        scopeManager: new AsyncHooksScopeManager(),
       });
       assert.ok(tracer instanceof NodeTracer);
     });
 
     it('should construct an instance with logger', () => {
       const tracer = new NodeTracer({
         logger: new NoopLogger(),
-        scopeManager: new AsyncHooksScopeManager()
+        scopeManager: new AsyncHooksScopeManager(),
       });
       assert.ok(tracer instanceof NodeTracer);
     });
 
     it('should construct an instance with sampler', () => {
       const tracer = new NodeTracer({
         scopeManager: new AsyncHooksScopeManager(),
-        sampler: ALWAYS_SAMPLER
+        sampler: ALWAYS_SAMPLER,
       });
       assert.ok(tracer instanceof NodeTracer);
     });
@@ -72,9 +72,9 @@ describe('NodeTracer', () => {
       const tracer = new NodeTracer({
         defaultAttributes: {
           region: 'eu-west',
-          asg: 'my-asg'
+          asg: 'my-asg',
         },
-        scopeManager: new AsyncHooksScopeManager()
+        scopeManager: new AsyncHooksScopeManager(),
       });
       assert.ok(tracer instanceof NodeTracer);
     });
@@ -83,15 +83,15 @@ describe('NodeTracer', () => {
   describe('.startSpan()', () => {
     it('should start a span with name only', () => {
       const tracer = new NodeTracer({
-        scopeManager: new AsyncHooksScopeManager()
+        scopeManager: new AsyncHooksScopeManager(),
       });
       const span = tracer.startSpan('my-span');
       assert.ok(span);
     });
 
     it('should start a span with name and options', () => {
       const tracer = new NodeTracer({
-        scopeManager: new AsyncHooksScopeManager()
+        scopeManager: new AsyncHooksScopeManager(),
       });
       const span = tracer.startSpan('my-span', {});
       assert.ok(span);
@@ -100,7 +100,7 @@ describe('NodeTracer', () => {
     it('should return a default span with no sampling', () => {
       const tracer = new NodeTracer({
         sampler: NEVER_SAMPLER,
-        scopeManager: new AsyncHooksScopeManager()
+        scopeManager: new AsyncHooksScopeManager(),
       });
       const span = tracer.startSpan('my-span');
       assert.deepStrictEqual(span, NOOP_SPAN);
@@ -116,7 +116,7 @@ describe('NodeTracer', () => {
   describe('.getCurrentSpan()', () => {
     it('should return null with AsyncHooksScopeManager when no span started', () => {
       const tracer = new NodeTracer({
-        scopeManager: new AsyncHooksScopeManager()
+        scopeManager: new AsyncHooksScopeManager(),
       });
       assert.deepStrictEqual(tracer.getCurrentSpan(), null);
     });
@@ -125,7 +125,7 @@ describe('NodeTracer', () => {
   describe('.withSpan()', () => {
     it('should run scope with AsyncHooksScopeManager scope manager', done => {
       const tracer = new NodeTracer({
-        scopeManager: new AsyncHooksScopeManager()
+        scopeManager: new AsyncHooksScopeManager(),
       });
       const span = tracer.startSpan('my-span');
       tracer.withSpan(span, () => {
@@ -137,7 +137,7 @@ describe('NodeTracer', () => {
 
     it('should run scope with AsyncHooksScopeManager scope manager with multiple spans', done => {
       const tracer = new NodeTracer({
-        scopeManager: new AsyncHooksScopeManager()
+        scopeManager: new AsyncHooksScopeManager(),
       });
       const span = tracer.startSpan('my-span');
       tracer.withSpan(span, () => {
@@ -146,7 +146,10 @@ describe('NodeTracer', () => {
         const span1 = tracer.startSpan('my-span1', { parent: span });
         tracer.withSpan(span1, () => {
           assert.deepStrictEqual(tracer.getCurrentSpan(), span1);
-          assert.deepStrictEqual(span1.context().traceId, span.context().traceId);
+          assert.deepStrictEqual(
+            span1.context().traceId,
+            span.context().traceId
+          );
           return done();
         });
       });
@@ -164,7 +167,7 @@ describe('NodeTracer', () => {
   describe('getBinaryFormat', () => {
     it('should get default binary formatter', () => {
       const tracer = new NodeTracer({
-        scopeManager: new AsyncHooksScopeManager()
+        scopeManager: new AsyncHooksScopeManager(),
       });
       assert.ok(tracer.getBinaryFormat() instanceof BinaryTraceContext);
     });
@@ -173,7 +176,7 @@ describe('NodeTracer', () => {
   describe('.getHttpTextFormat()', () => {
     it('should get default HTTP text formatter', () => {
       const tracer = new NodeTracer({
-        scopeManager: new AsyncHooksScopeManager()
+        scopeManager: new AsyncHooksScopeManager(),
       });
       assert.ok(tracer.getHttpTextFormat() instanceof HttpTraceContext);
     });

diff --git a/packages/opentelemetry-plugin-http/package.json b/packages/opentelemetry-plugin-http/package.json
@@ -38,20 +38,19 @@
     "access": "public"
   },
   "devDependencies": {
-    "@opentelemetry/scope-async-hooks": "^0.0.1",
     "@types/mocha": "^5.2.7",
     "@types/nock": "^10.0.3",
     "@types/node": "^12.6.9",
     "@types/semver": "^6.0.1",
     "@types/shimmer": "^1.0.1",
     "@types/sinon":"^7.0.13",
+    "@opentelemetry/scope-async-hooks": "^0.0.1",
     "codecov": "^3.5.0",
-    "gts": "^1.1.0",
+    "gts": "^1.0.0",
     "mocha": "^6.2.0",
+    "nock": "^10.0.6",
     "nyc": "^14.1.1",
     "sinon": "^7.3.2",
-    "c8": "^5.0.1",
-    "nock": "^10.0.6",
     "ts-mocha": "^6.0.0",
     "ts-node": "^8.3.0",
     "typescript": "^3.5.3"

diff --git a/packages/opentelemetry-plugin-http/src/enums/attributes.ts b/packages/opentelemetry-plugin-http/src/enums/attributes.ts
@@ -13,5 +13,5 @@ export enum Attributes {
   ATTRIBUTE_HTTP_STATUS_CODE = 'http.status_code',
   // NOT ON OFFICIAL SPEC
   ATTRIBUTE_HTTP_ERROR_NAME = 'http.error_name',
-  ATTRIBUTE_HTTP_ERROR_MESSAGE = 'http.error_message'
+  ATTRIBUTE_HTTP_ERROR_MESSAGE = 'http.error_message',
 }
diff --git a/packages/opentelemetry-plugin-http/src/enums/format.ts b/packages/opentelemetry-plugin-http/src/enums/format.ts
@@ -1,4 +1,4 @@
 // Should we export this to the types package in order to expose all values ?
 export enum Format {
-  HTTP = 'HttpTraceContext'
+  HTTP = 'HttpTraceContext',
 }