Move from tap to the node core test runner

As titled

This could be part of the next semver major that drops old nodes.

Will give a shot at this

@mcollina

@mcollina There are a lot of test files using tap (around 150).

Are we looking to migrate all of them in one PR or migrating part by part is fine?

Hey!

Given the large number of tests and the need to meticulously avoid false positives, would it be more effective to split the test suites into separate PRs?

For instance, test:cookies appears to be easily portable independently of other suites.

How about we create a list of these suites to distribute the workload and more effectively monitor the outcomes?

Is there something like t.plan in node:test?

@Uzlopak

@Uzlopak There isn't, we are using @matteo.collina/tspl package instead of t.plan

Example usage here.

I am on this now.

I opened a PR #2719 for an example transformation.
Also I will split the necessary work into smaller PRs to reduce the review work.

Opened #2720 and #2721 to incrementally prepare the following PR.

LOL. I created basically 6 batches and lost the following stuff, which I stashed in git.

So please review and merge the open PRs. Then I can continue to migrate it.

Nice. Its done.

Maybe we can do the same in fastify

Navigation Menu

Repository navigation

Move from tap to the node core test runner #2267

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Move from tap to the node core test runner #2267

Description

Activity

ashutoshdhande commented on Dec 27, 2023

anurag-roy commented on Dec 27, 2023

pmarchini commented on Dec 27, 2023

Uzlopak commented on Jan 27, 2024

anurag-roy commented on Jan 27, 2024

Uzlopak commented on Feb 8, 2024

Uzlopak commented on Feb 9, 2024

Uzlopak commented on Feb 12, 2024

Uzlopak commented on Feb 13, 2024

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions