Merge pull request #4 from Dharin-shah/add-appium-info

add appium info

Author: Dharin-shah

packages/opentelemetry/lib/plugin.js

@@ -27,7 +27,7 @@ export default class OpentelemetryPlugin extends BasePlugin {
   static handleSetOpentelemetryConfig (req, res) {
     const opentelemetryBlob = req.body;
     const exporterConfig = opentelemetryBlob.exporter;
-    if (!exporterConfig) {
+    if (typeof exporterConfig == 'undefined' || exporterConfig == null) {
       const response = JSON.stringify({
         status: STATUS_MESSAGE.FAILURE, message: STATUS_MESSAGE.INVALID_EXPORTER_ERROR
       });

packages/opentelemetry/lib/tracing/exporterFactory.js

@@ -6,7 +6,7 @@ import { ConsoleSpanExporter, SpanExporter } from '@opentelemetry/tracing'; // e
 const DEFAULT_SERVICE_NAME = 'appium';
 
 const AVAILABLE_EXPORTERS = {
-  JAGGER: 'jagger',
+  JAEGER: 'jaeger',
   ZIPKIN: 'zipkin',
   PROMETHEUS: 'prometheus',
   CONSOLE: 'console'
@@ -21,7 +21,7 @@ const AVAILABLE_EXPORTERS = {
 */
 function buildExporter (exporterType, config = null) {
   switch (exporterType) {
-    case AVAILABLE_EXPORTERS.JAGGER:
+    case AVAILABLE_EXPORTERS.JAEGER:
       return new JaegerExporter(config || { serviceName: DEFAULT_SERVICE_NAME });
     case AVAILABLE_EXPORTERS.ZIPKIN:
       return new ZipkinExporter(config || { serviceName: DEFAULT_SERVICE_NAME });

packages/opentelemetry/package.json

@@ -5,6 +5,13 @@
   "scripts": {
     "test:unit": "mocha ./build/test/unit/*-specs.js"
   },
+  "tags": [
+    "appium"
+  ],
+  "appium": {
+    "pluginName": "opentelemetry",
+    "mainClass": "OpentelemetryPlugin"
+  },
   "main": "./build/index.js",
   "directories": {
     "lib": "./lib"

packages/opentelemetry/test/unit/exporter-specs.js

@@ -21,8 +21,8 @@ describe('ExporterFactory', function () {
     exporterObject.should.be.instanceOf(ConsoleSpanExporter);
     await exporterObject.shutdown();
   });
-  it(`should return exporter object for ${AVAILABLE_EXPORTERS.JAGGER} exporter`, async function () {
-    const exporterObject = buildExporter('jagger');
+  it(`should return exporter object for ${AVAILABLE_EXPORTERS.JAEGER} exporter`, async function () {
+    const exporterObject = buildExporter('jaeger');
     exporterObject.should.be.instanceOf(JaegerExporter);
     await exporterObject.shutdown();
   });