Skip to content

test: fix unreliable test-fs-write-file.js - #36102

Merged
Trott merged 1 commit into
nodejs:masterfrom
Trott:fix-broken-test
Nov 12, 2020
Merged

test: fix unreliable test-fs-write-file.js#36102
Trott merged 1 commit into
nodejs:masterfrom
Trott:fix-broken-test

Conversation

@Trott

Copy link
Copy Markdown
Member

The test uses a file name twice, causing unreliability in CI. In
particular, it's failing a lot on the Raspberry Pi devices.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • commit message follows commit guidelines

@nodejs-github-botnodejs-github-bot added the test Issues and PRs related to the tests. label Nov 12, 2020
@Trott

Copy link
Copy Markdown
MemberAuthor

Fast track?

@TrottTrott added the request-ci Add this label to start a Jenkins CI on a PR. label Nov 12, 2020
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Nov 12, 2020
@nodejs-github-bot

This comment has been minimized.

@TrottTrott added the flaky-test Issues and PRs related to the tests with unstable failures on the CI. label Nov 12, 2020
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@TrottTrott added the fast-track PRs that do not need to wait for 48 hours to land. label Nov 12, 2020
The test uses a file name twice, causing unreliability in CI. In
particular, it's failing a lot on the Raspberry Pi devices.
Fixes: nodejs#36090
PR-URL: nodejs#36102
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
@Trott
Trott merged commit f03f7cb into nodejs:masterNov 12, 2020
@Trott

Copy link
Copy Markdown
MemberAuthor

Landed in f03f7cb

@Trott
Trott deleted the fix-broken-test branch November 12, 2020 23:08
targos pushed a commit to targos/node that referenced this pull request Apr 30, 2021
The test uses a file name twice, causing unreliability in CI. In
particular, it's failing a lot on the Raspberry Pi devices.
Fixes: nodejs#36090
PR-URL: nodejs#36102
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
targos pushed a commit that referenced this pull request Apr 30, 2021
The test uses a file name twice, causing unreliability in CI. In
particular, it's failing a lot on the Raspberry Pi devices.
Fixes: #36090
PR-URL: #36102
Backport-PR-URL: #38386
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
@danielleadamsdanielleadams mentioned this pull request May 3, 2021
Pranay180420 pushed a commit to Pranay180420/Node.jsforme that referenced this pull request Feb 17, 2025
The test uses a file name twice, causing unreliability in CI. In
particular, it's failing a lot on the Raspberry Pi devices.
Fixes: nodejs/node#36090
PR-URL: nodejs/node#36102
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fast-trackPRs that do not need to wait for 48 hours to land.flaky-testIssues and PRs related to the tests with unstable failures on the CI.testIssues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@Trott@nodejs-github-bot@lpinca@richardlau@targos@BethGriggs