Tracer web

karma.base.js

@@ -0,0 +1,26 @@
+/*!
+ * Copyright 2019, OpenTelemetry Authors
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+module.exports = {
+  listenAddress: 'localhost',
+  hostname: 'localhost',
+  browsers: ['ChromeHeadless'],
+  frameworks: ['mocha'],
+  reporters: ['spec'],
+  files: ['test/index-webpack.ts'],
+  preprocessors: { 'test/index-webpack.ts': ['webpack'] },
+  webpackMiddleware: { noInfo: true }
+};

packages/opentelemetry-basic-tracer/karma.conf.js

@@ -0,0 +1,24 @@
+/*!
+ * Copyright 2019, OpenTelemetry Authors
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+const webpackConfig = require('./webpack/test.config.js');
+const karmaBaseConfig = require('../../karma.base');
+
+module.exports = (config) => {
+  config.set(Object.assign({}, karmaBaseConfig, {
+    webpack: webpackConfig
+  }))
+};

packages/opentelemetry-basic-tracer/package.json

@@ -3,10 +3,15 @@
   "version": "0.0.1",
   "description": "OpenTelemetry Basic Tracer",
   "main": "build/src/index.js",
+  "browser": {
+    "./src/platform/index.ts": "./src/platform/browser/index.ts",
+    "./build/src/platform/index.js": "./build/src/platform/browser/index.js"
+  },
   "types": "build/src/index.d.ts",
   "repository": "open-telemetry/opentelemetry-js",
   "scripts": {
-    "test": "nyc ts-mocha -p tsconfig.json test/**/**/*.ts",
+    "test": "nyc ts-mocha -p tsconfig.json 'test/**/*.ts' --exclude 'test/index-webpack.ts'",
+    "test:browser": "karma start --single-run",
     "tdd": "yarn test -- --watch-extensions ts --watch",
     "codecov": "nyc report --reporter=json && codecov -f coverage/*.json -p ../../",
     "clean": "rimraf build/*",
@@ -40,16 +45,24 @@
   "devDependencies": {
     "@types/mocha": "^5.2.5",
     "@types/node": "^12.6.8",
+    "@types/webpack-env": "1.13.9",
     "codecov": "^3.1.0",
     "gts": "^1.0.0",
+    "karma": "^4.1.0",
+    "karma-chrome-launcher": "^2.2.0",
+    "karma-mocha": "^1.3.0",
+    "karma-spec-reporter": "^0.0.32",
+    "karma-webpack": "^4.0.2",
     "mocha": "^6.1.0",
     "nyc": "^14.1.1",
     "rimraf": "^3.0.0",
+    "tslint-consistent-codestyle":"^1.15.1",
     "tslint-microsoft-contrib": "^6.2.0",
-    "tslint-consistent-codestyle": "^1.15.1",
+    "ts-loader": "^6.0.4",
     "ts-mocha": "^6.0.0",
     "ts-node": "^8.0.0",
-    "typescript": "^3.4.5"
+    "typescript": "^3.4.5",
+    "webpack": "^4.35.2"
   },
   "dependencies": {
     "@opentelemetry/core": "^0.0.1",

packages/opentelemetry-basic-tracer/test/Span.test.ts

@@ -15,7 +15,6 @@
  */
 
 import * as assert from 'assert';
-import { performance } from 'perf_hooks';
 import { Span } from '../src/Span';
 import {
   SpanKind,
@@ -25,11 +24,14 @@ import {
 } from '@opentelemetry/types';
 import { BasicTracer } from '../src';
 import {
+  hrTime,
   hrTimeToNanoseconds,
   hrTimeToMilliseconds,
   NoopLogger,
 } from '@opentelemetry/core';
 
+const performanceTimeOrigin = hrTime();
+
 describe('Span', () => {
   const tracer = new BasicTracer({
     logger: new NoopLogger(),
@@ -50,7 +52,10 @@ describe('Span', () => {
 
   it('should have valid startTime', () => {
     const span = new Span(tracer, name, spanContext, SpanKind.SERVER);
-    assert.ok(hrTimeToMilliseconds(span.startTime) > performance.timeOrigin);
+    assert.ok(
+      hrTimeToMilliseconds(span.startTime) >
+        hrTimeToMilliseconds(performanceTimeOrigin)
+    );
   });
 
   it('should have valid endTime', () => {
@@ -62,7 +67,8 @@ describe('Span', () => {
     );
 
     assert.ok(
-      hrTimeToMilliseconds(span.endTime) > performance.timeOrigin,
+      hrTimeToMilliseconds(span.endTime) >
+        hrTimeToMilliseconds(performanceTimeOrigin),
       'end time must be bigger than time origin'
     );
   });
@@ -77,7 +83,8 @@ describe('Span', () => {
     const span = new Span(tracer, name, spanContext, SpanKind.SERVER);
     span.addEvent('my-event');
     assert.ok(
-      hrTimeToMilliseconds(span.events[0].time) > performance.timeOrigin
+      hrTimeToMilliseconds(span.events[0].time) >
+        hrTimeToMilliseconds(performanceTimeOrigin)
     );
   });
 

packages/opentelemetry-basic-tracer/test/index-webpack.ts

@@ -0,0 +1,20 @@
+/*!
+ * Copyright 2019, OpenTelemetry Authors
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      https://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+// This file is the webpack entry point for the browser Karma tests. It requires
+// all modules ending in "test" from the current folder and all its subfolders.
+const testsContext = require.context('.', true, /test$/);
+testsContext.keys().forEach(testsContext);

packages/opentelemetry-basic-tracer/webpack/test.config.js

@@ -0,0 +1,34 @@
+/*!
+ * Copyright 2019, OpenTelemetry Authors
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+const webpackNodePolyfills = require('../../../webpack.node-polyfills.js');
+
+// This is the webpack configuration for browser Karma tests.
+module.exports = {
+  mode: 'development',
+  target: 'web',
+  output: { filename: 'bundle.js' },
+  resolve: { extensions: ['.ts', '.js'] },
+  devtool: 'inline-source-map',
+  module: {
+    rules: [
+      { test: /\.ts$/, use: 'ts-loader' },
+      // This setting configures Node polyfills for the browser that will be
+      // added to the webpack bundle for Karma tests.
+      { parser: { node: webpackNodePolyfills } }
+    ]
+  }
+};

packages/opentelemetry-core/karma.conf.js

@@ -15,17 +15,10 @@
  */
 
 const webpackConfig = require('./webpack/test.config.js');
+const karmaBaseConfig = require('../../karma.base');
 
 module.exports = (config) => {
-  config.set({
-    listenAddress: 'localhost',
-    hostname: 'localhost',
-    browsers: ['ChromeHeadless'],
-    frameworks: ['mocha'],
-    reporters: ['spec'],
-    files: ['test/index-webpack.ts'],
-    preprocessors: {'test/index-webpack.ts': ['webpack']},
-    webpack: webpackConfig,
-    webpackMiddleware: {noInfo: true},
-  });
+  config.set(Object.assign({}, karmaBaseConfig, {
+    webpack: webpackConfig
+  }))
 };

packages/opentelemetry-core/package.json

@@ -4,7 +4,8 @@
   "description": "OpenTelemetry Core",
   "main": "build/src/index.js",
   "browser": {
-    "./src/platform/index.ts": "./src/platform/browser/index.ts"
+    "./src/platform/index.ts": "./src/platform/browser/index.ts",
+    "./build/src/platform/index.js": "./build/src/platform/browser/index.js"
   },
   "types": "build/src/index.d.ts",
   "repository": "open-telemetry/opentelemetry-js",

packages/opentelemetry-core/webpack/test.config.js

@@ -14,73 +14,21 @@
  * limitations under the License.
  */
 
-// This is the webpack configuration for browesr Karma tests.
+const webpackNodePolyfills = require('../../../webpack.node-polyfills.js');
+
+// This is the webpack configuration for browser Karma tests.
 module.exports = {
   mode: 'development',
-  output: {filename: 'bundle.js'},
-  resolve: {extensions: ['.ts', '.js']},
+  target: 'web',
+  output: { filename: 'bundle.js' },
+  resolve: { extensions: ['.ts', '.js'] },
   devtool: 'inline-source-map',
   module: {
     rules: [
-      {test: /\.ts$/, use: 'ts-loader'},
-      {
-        parser: {
-          // This setting configures Node polyfills for the browser that will be
-          // added to the webpack bundle for Karma tests.
-          node: {
-            // Enable the assert library polyfill because that is used in tests
-            assert: true,
-            // The assert polyfill from github.com/browserify/commonjs-assert
-            // also requires the `global` polyfill.
-            global: true,
-
-            // Turn off all other Node.js API polyfills for the browser tests to
-            // make sure that we are not attempting to use Node-specific APIs in
-            // the browser code. Instead, we will write browser specific
-            // implementations of platform functionality we need under the
-            // `./src/platform/browser` folder. This allows for writing browser
-            // optimized implementations of the specific needed functionality
-            // rather than bringing in (sometimes large) polyfills for the
-            // corresponding Node APIs.
-            Buffer: false,
-            __dirname: false,
-            __filename: false,
-            buffer: false,
-            child_process: false,
-            cluster: false,
-            console: false,
-            constants: false,
-            crypto: false,
-            dgram: false,
-            dns: false,
-            domain: false,
-            events: false,
-            fs: false,
-            http: false,
-            https: false,
-            module: false,
-            net: false,
-            os: false,
-            path: false,
-            process: false,
-            punycode: false,
-            querystring: false,
-            readline: false,
-            repl: false,
-            setImmediate: false,
-            stream: false,
-            string_decoder: false,
-            sys: false,
-            timers: false,
-            tls: false,
-            tty: false,
-            url: false,
-            util: false,
-            vm: false,
-            zlib: false,
-          },
-        },
-      },
-    ],
-  },
+      { test: /\.ts$/, use: 'ts-loader' },
+      // This setting configures Node polyfills for the browser that will be
+      // added to the webpack bundle for Karma tests.
+      { parser: { node: webpackNodePolyfills } }
+    ]
+  }
 };

packages/opentelemetry-tracer-web/.npmignore

@@ -0,0 +1,4 @@
+/bin
+/coverage
+/doc
+/test