[Test] Fix monitor-opentelemetry-exporter live tests

[jeremymeng]

Migrate the non-standard tests

[jeremymeng]

/azp run js - monitor-opentelemetry-exporter - tests

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[JacksonWeber]

LGTM

[azure-sdk]

API change check

API changes are not detected in this pull request.

[jeremymeng]

/azp run js - monitor-opentelemetry-exporter - tests

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[jeremymeng]

Looks that the tests are being executed now. However, the tests are now failing. It looks like a test issue to me. Previously they were passing because the test catches error and calls done(). Going to merge this PR as it makes things better, not worse.

-    it("should work", (done) => {
-      scenario
-        .run()
-        .then(() => {
-          // promisify doesn't work on this, so use callbacks/done for now
-          // eslint-disable-next-line promise/always-return
-          return scenario.flush().then(() => {
-            assertMetricExpectation(ingest, scenario.expectation);
-            assertCount(ingest, scenario.expectation);
-            done();
-          });
-        })
-        .catch((e) => {
-          done(e);
-        });
+    it("should work", async () => {
+      await scenario.run();
+      // promisify doesn't work on this, so use callbacks/done for now
+      // eslint-disable-next-line promise/always-return
+      await scenario.flush();
+      assertMetricExpectation(ingest, scenario.expectation);
+      assertCount(ingest, scenario.expectation);
     });

[jeremymeng]

/check-enforcer override