Skip to content

test: Fix broken test cases from #18194 - #18295

Closed
guybedford wants to merge 1 commit into
nodejs:masterfrom
guybedford:esm-inspect-brk
Closed

test: Fix broken test cases from #18194#18295
guybedford wants to merge 1 commit into
nodejs:masterfrom
guybedford:esm-inspect-brk

Conversation

@guybedford

Copy link
Copy Markdown
Contributor

It seems I messed up the sync during the commit process, this should fix the CI issues in #18293.

@nodejs-github-botnodejs-github-bot added the test Issues and PRs related to the tests. label Jan 22, 2018
@apapirovski

Copy link
Copy Markdown
Contributor

I've already updated mine with a fix, FWIW.

@guybedfordguybedford mentioned this pull request Jan 22, 2018
2 tasks
@guybedford

Copy link
Copy Markdown
ContributorAuthor

@apapirovski including the test cases?

@apapirovski

Copy link
Copy Markdown
Contributor

@guybedford Yea, the second commit.

@guybedford

Copy link
Copy Markdown
ContributorAuthor

Ahh nice thanks, let's check the CI on that then first.

@guybedford

Copy link
Copy Markdown
ContributorAuthor

Superseded by fixes in #18293.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testIssues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@guybedford@apapirovski@nodejs-github-bot