test: comments

Author: aryamohanan

packages/collector/test/tracing/protocols/http/server/test.js

@@ -471,7 +471,7 @@ function registerTests(agentControls, appUsesHttps, useHttp2CompatApi) {
         )
       ));
 
-  // Skip this test because native fetch does not allow URLs with embedded credentials
+  // Skipped this test because native fetch does not allow URLs with embedded credentials
   // Error: "Request cannot be constructed from a URL that includes credentials"
   // This is a security feature of the Fetch API standard
   it.skip(`must not collect credentials embedded in URLs (HTTPS: ${appUsesHttps})`, () =>
@@ -530,7 +530,7 @@ function registerTests(agentControls, appUsesHttps, useHttp2CompatApi) {
       })
     );
 
-    // The request will eventually complete after 4 seconds (as defined in app.js)
+    // The request will eventually complete after 4 seconds
     return requestPromise.catch(() => {});
   });
 

packages/opentelemetry-exporter/test/test.js

@@ -212,7 +212,7 @@ function verifySpans(spans, appControls) {
   // NOTE: Native fetch with OpenTelemetry does not create an EXIT span for the HTTP client call
   // in older versions. Starting with @opentelemetry/auto-instrumentations-node v0.46+,
   // @opentelemetry/instrumentation-undici is included and enabled by default, which instruments
-  // native fetch. However, this test may use an older version without undici instrumentation.
+  // native fetch. However, this test use an older version without the instrumentation.
   // NOTE: middleware spans are not collected when migrating to express v5 because:
   //       middleware initialization was removed in
   //         https://github.com/expressjs/express/commit/78e50547f16e2adb5763a953586d05308d8aba4c.

packages/opentelemetry-sampler/test/test.js

@@ -130,7 +130,7 @@ mochaSuiteFn('Instana OpenTelemetry Sampler', function () {
       // NOTE: Native fetch with OpenTelemetry does not create an EXIT span for the HTTP client call
       // in older versions. Starting with @opentelemetry/auto-instrumentations-node v0.46+,
       // @opentelemetry/instrumentation-undici is included and enabled by default, which instruments
-      // native fetch. However, this test may use an older version without undici instrumentation.
+      // native fetch. However, this test use an older version without undici instrumentation.
       const spanNames = ['tcp.connect', 'tls.connect', 'GET'];
       expect(spanNames).to.eql(resp.spans.map(s => s.name));
       expect(resp.spans.length).to.be.gte(3);