fix(tests): delete flaky test

.github/workflows/ci.yaml

@@ -43,11 +43,6 @@ jobs:
       - run: node --version
       - run: npm install
       - run: npm test
-      - name: coverage
-        uses: codecov/codecov-action@v3
-        with:
-          name: actions ${{ matrix.node }}
-          fail_ci_if_error: true
   windows:
     runs-on: windows-latest
     env:
@@ -61,11 +56,6 @@ jobs:
           node-version: 14
       - run: npm install
       - run: npm test
-      - name: coverage
-        uses: codecov/codecov-action@v3
-        with:
-          name: actions windows
-          fail_ci_if_error: true
   lint:
     runs-on: ubuntu-latest
     steps:

scripts/index.ts

@@ -87,6 +87,6 @@ async function run(steps: string[]) {
 
 run(steps).catch(err => {
   console.error(err);
-  // eslint-disable-next-line no-process-exit
+  // eslint-disable-next-line n/no-process-exit
   process.exit(1);
 });

src/plugins/plugin-connect.ts

@@ -13,7 +13,7 @@
 // limitations under the License.
 
 import {IncomingMessage, ServerResponse} from 'http';
-// eslint-disable-next-line node/no-deprecated-api
+// eslint-disable-next-line n/no-deprecated-api
 import {parse as urlParse} from 'url';
 
 import {PluginTypes} from '..';

src/plugins/plugin-hapi.ts

@@ -14,7 +14,7 @@
 
 import {ServerResponse} from 'http';
 import * as shimmer from 'shimmer';
-// eslint-disable-next-line node/no-deprecated-api
+// eslint-disable-next-line n/no-deprecated-api
 import {parse as urlParse} from 'url';
 
 import {PluginTypes} from '..';

src/plugins/plugin-http2.ts

@@ -15,7 +15,7 @@
 
 // This is imported only for types. Generated .js file should NOT load 'http2'.
 // `http2` must be used only in type annotations, not in expressions.
-// eslint-disable-next-line node/no-unsupported-features/node-builtins
+// eslint-disable-next-line n/no-unsupported-features/node-builtins
 import * as http2 from 'http2';
 import * as shimmer from 'shimmer';
 import {URL} from 'url';

src/plugins/plugin-koa.ts

@@ -14,7 +14,7 @@
 
 import {ServerResponse} from 'http';
 import * as shimmer from 'shimmer';
-// eslint-disable-next-line node/no-deprecated-api
+// eslint-disable-next-line n/no-deprecated-api
 import {parse as urlParse} from 'url';
 
 import {PluginTypes} from '..';

src/trace-writer.ts

@@ -152,7 +152,7 @@ export class TraceWriter extends Service {
         this.flushBuffer();
         if (onUncaughtException === 'flushAndExit') {
           setTimeout(() => {
-            // eslint-disable-next-line no-process-exit
+            // eslint-disable-next-line n/no-process-exit
             process.exit(1);
           }, 2000);
         }

test/non-interference/http-e2e.js

@@ -44,7 +44,7 @@ const testCommonPath = [
 });
 if (!testCommonPath) {
   console.error('No common.js or common/index.js found in test directory');
-  // eslint-disable-next-line no-process-exit
+  // eslint-disable-next-line n/no-process-exit
   process.exit(1);
 }
 cp.execFileSync('sed', [

test/non-interference/start-agent.js

@@ -15,7 +15,7 @@
 'use strict';
 
 const shimmer = require('shimmer');
-// eslint-disable-next-line node/no-missing-require
+// eslint-disable-next-line n/no-missing-require
 const util = require('../../build/src/util.js');
 // Stub generateTraceContext so that it always returns the same thing.
 // This is because web framework unit tests check that similar/identical
@@ -29,7 +29,7 @@ shimmer.wrap(util, 'generateTraceContext', () => {
     return 'ffeeddccbbaa99887766554433221100/0?o=1';
   };
 });
-// eslint-disable-next-line node/no-missing-require
+// eslint-disable-next-line n/no-missing-require
 require('../..').start({
   projectId: '0',
   logLevel: 1,

test/plugins/test-trace-http2.ts

@@ -16,7 +16,7 @@
 import {describe, it, after, afterEach, before} from 'mocha';
 // This is imported only for types. Generated .js file should NOT load 'http2'
 // in this place. It is dynamically loaded later from each test suite below.
-// eslint-disable-next-line node/no-unsupported-features/node-builtins
+// eslint-disable-next-line n/no-unsupported-features/node-builtins
 import * as http2Types from 'http2';
 import * as semver from 'semver';
 import * as stream from 'stream';
@@ -46,7 +46,7 @@
     traceTestModule.setPluginLoaderForTest();
     traceTestModule.setCLSForTest();
     traceTestModule.start();
-    // eslint-disable-next-line node/no-unsupported-features/node-builtins
+    // eslint-disable-next-line n/no-unsupported-features/node-builtins
     http2 = require('http2');
   });
 
@@ -81,7 +81,7 @@
     });
 
     it('should patch the necessary functions', () => {
-      assert.strictEqual(http2.connect['__wrapped'], true);
+      assert.strictEqual((http2 as any).connect['__wrapped'], true);
     });
 
     it('should accurately measure request time', done => {

test/plugins/test-trace-node-fetch.ts

@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-// eslint-disable-next-line node/no-extraneous-import
+// eslint-disable-next-line n/no-extraneous-import
 import * as fetchTypes from 'node-fetch'; // For types only.
 import * as testTraceModule from '../trace';
 import * as assert from 'assert';

test/trace.ts

@@ -120,7 +120,7 @@
 }
 
 export function setLoggerForTest(impl?: typeof logger.Logger) {
-  if (logger.Logger.__wrapped) {
+  if ((logger.Logger as any).__wrapped) {
     shimmer.unwrap(logger, 'Logger');
   }
   if (impl) {