Revert "test(node): Add pg-native integration tests" (#15464)

Reverts #15206

This PR requires `libpq` for `yarn install` to be complete, so going to
revert this. Let's put this requirement into the docker image instead.

Author: AbhiPrasad

.github/workflows/build.yml

@@ -713,10 +713,6 @@ jobs:
         with:
           dependency_cache_key: ${{ needs.job_build.outputs.dependency_cache_key }}
 
-      - name: Build `libpq`
-        run: yarn libpq:build
-        working-directory: dev-packages/node-integration-tests
-
       - name: Overwrite typescript version
         if: matrix.typescript == '3.8'
         run: node ./scripts/use-ts-3_8.js

dev-packages/node-integration-tests/package.json

@@ -16,7 +16,6 @@
     "build:types": "tsc -p tsconfig.types.json",
     "clean": "rimraf -g **/node_modules && run-p clean:script",
     "clean:script": "node scripts/clean.js",
-    "libpq:build": "npm rebuild libpq",
     "express-v5-install": "cd suites/express-v5 && yarn --no-lockfile",
     "lint": "eslint . --format stylish",
     "fix": "eslint . --format stylish --fix",
@@ -63,8 +62,7 @@
     "nock": "^13.5.5",
     "node-cron": "^3.0.3",
     "node-schedule": "^2.1.1",
-    "pg": "^8.13.1",
-    "pg-native": "3.2.0",
+    "pg": "^8.7.3",
     "proxy": "^2.1.1",
     "redis-4": "npm:redis@^4.6.14",
     "reflect-metadata": "0.2.1",

dev-packages/node-integration-tests/suites/tracing/postgres/scenario-native.js

@@ -53,54 +53,4 @@ describe('postgres auto instrumentation', () => {
       .expect({ transaction: EXPECTED_TRANSACTION })
       .start(done);
   });
-
-  test('should auto-instrument `pg-native` package', done => {
-    const EXPECTED_TRANSACTION = {
-      transaction: 'Test Transaction',
-      spans: expect.arrayContaining([
-        expect.objectContaining({
-          data: expect.objectContaining({
-            'db.system': 'postgresql',
-            'db.name': 'tests',
-            'sentry.origin': 'manual',
-            'sentry.op': 'db',
-          }),
-          description: 'pg.connect',
-          op: 'db',
-          status: 'ok',
-        }),
-        expect.objectContaining({
-          data: expect.objectContaining({
-            'db.system': 'postgresql',
-            'db.name': 'tests',
-            'db.statement': 'INSERT INTO "NativeUser" ("email", "name") VALUES ($1, $2)',
-            'sentry.origin': 'auto.db.otel.postgres',
-            'sentry.op': 'db',
-          }),
-          description: 'INSERT INTO "NativeUser" ("email", "name") VALUES ($1, $2)',
-          op: 'db',
-          status: 'ok',
-          origin: 'auto.db.otel.postgres',
-        }),
-        expect.objectContaining({
-          data: expect.objectContaining({
-            'db.system': 'postgresql',
-            'db.name': 'tests',
-            'db.statement': 'SELECT * FROM "NativeUser"',
-            'sentry.origin': 'auto.db.otel.postgres',
-            'sentry.op': 'db',
-          }),
-          description: 'SELECT * FROM "NativeUser"',
-          op: 'db',
-          status: 'ok',
-          origin: 'auto.db.otel.postgres',
-        }),
-      ]),
-    };
-
-    createRunner(__dirname, 'scenario-native.js')
-      .withDockerCompose({ workingDirectory: [__dirname], readyMatches: ['port 5432'] })
-      .expect({ transaction: EXPECTED_TRANSACTION })
-      .start(done);
-  });
 });