refactor: migrate test framework in build module

[reidn3r]

These change are related to the issue #5555

• Migrated from tap framework to node:test in fastify build module
• Ran the tests that i changed and confirmed they passed successfully

[mcollina]

Please revert this. That section of the package.json is left for the collaborators (also listed in the README).

[reidn3r]

Of course! I misunderstood. I apologize.

[reidn3r]

@gurgunday could you please review this PR?

[gurgunday]

please uninstall tap

Oh I see, I think we’d prefer a complete removal of tap in one PR rather than these incremental changes

[jsumners]

please uninstall tap

Oh I see, I think we’d prefer a complete removal of tap in one PR rather than these incremental changes

I think that would be way too much. We should go for incremental updates like this PR is proposing. But we cannot do it with the current way tests are run:

fastify/package.json

Lines 21 to 30 in 7ddced6

	"test": "npm run lint && npm run unit && npm run test:typescript",
	"test:ci": "npm run unit -- --coverage-report=lcovonly && npm run test:typescript",
	"test:report": "npm run lint && npm run unit:report && npm run test:typescript",
	"test:validator:integrity": "npm run build:validation && git diff --quiet --ignore-all-space --ignore-blank-lines --ignore-cr-at-eol lib/error-serializer.js && git diff --quiet --ignore-all-space --ignore-blank-lines --ignore-cr-at-eol lib/configValidator.js",
	"test:typescript": "tsc test/types/import.ts --noEmit && tsd",
	"test:watch": "npm run unit -- --watch --coverage-report=none --reporter=terse",
	"unit": "tap",
	"unit:junit": "tap-mocha-reporter xunit < out.tap > test/junit-testresults.xml",
	"unit:report": "tap --coverage-report=html --coverage-report=cobertura | tee out.tap",
	"citgm": "tap --jobs=1 --timeout=120"

I don't think there's a guarantee that using tap as a runner that it will recognize the output and result of node:test based suites. I think that before we can start on gradually migrating tests in this repo, we need to see a PR merged that looks a lot like newrelic/node-newrelic#2387

[jsumners]

Why did you close this? #5720 should be merged soon.