Merge pull request #607 from appsignal/script-test-files

Ignore script test files in published package

packages/nodejs/.changesets/do-not-ship-test-files.md

@@ -0,0 +1,6 @@
+---
+bump: "patch"
+type: "remove"
+---
+
+Do not ship the extension install script test files in the published package. Reduces the package size a tiny bit.

packages/nodejs/.npmignore

@@ -1,6 +1,7 @@
 *
 !/dist/**/*
 !/scripts/**/*
+scripts/**/*.test.js
 !/ext/appsignal_extension.cpp
 !/cert/**/*
 !binding.gyp

packages/nodejs/scripts/extension/extension.failure.test.js → packages/nodejs/scripts/extension/__tests__/extension.failure.test.js

@@ -1,6 +1,6 @@
 const fs = require("fs")
 const path = require("path")
-const { reportPath } = require("./report")
+const { reportPath } = require("../report")
 
 function hasExtensionFailure() {
   if (process.env._TEST_APPSIGNAL_EXTENSION_FAILURE !== "true") {

packages/nodejs/scripts/extension/extension.test.js → packages/nodejs/scripts/extension/__tests__/extension.test.js

@@ -1,9 +1,9 @@
 const fs = require("fs")
-const { reportPath } = require("./report")
-const { downloadFromMirror } = require("./extension")
+const { reportPath } = require("../report")
+const { downloadFromMirror } = require("../extension")
 const nock = require("nock")
 const { Writable, EventEmitter } = require("stream")
-const { AGENT_VERSION } = require("./support/constants")
+const { AGENT_VERSION } = require("../support/constants")
 
 describe("Extension install success", () => {
   test("writes success result to diagnose installation report", () => {

packages/nodejs/scripts/extension/report.test.js → packages/nodejs/scripts/extension/__tests__/report.test.js

@@ -2,9 +2,9 @@ const originalProcess = process
 const originalProcessEnv = process.env
 const mockHelpers = { hasMusl: jest.fn() }
 
-jest.doMock("./support/helpers", () => mockHelpers)
+jest.doMock("../support/helpers", () => mockHelpers)
 
-const { createBuildReport } = require("./report")
+const { createBuildReport } = require("../report")
 
 describe("muslOverride", () => {
   beforeEach(() => {