Skip to content

Commit 7296973

Browse files
committed
test: remove postmortem test from flaky list
test-postmortem-metadata does not ever seem to fail on master. According to Michaël Zasso, when it fails locally for him during a V8 upgrade, he pings Colin Ihrig who fixes it almost immediately. There was some discussion about marking it flaky when it first landed but it's not clear to me that it is necessary. (We can always mark it flaky again if it turns out that it is necessary.) The reason I want this test removed from the status file for flaky tests is that I believe that file can and should be used as a list of tests to fix. But this test doesn't need any fixing. Refs: nodejs#17685
1 parent f86e5fc commit 7296973

File tree

1 file changed

+0
-2
lines changed

1 file changed

+0
-2
lines changed

test/parallel/parallel.status

-2
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,6 @@ prefix parallel
55
# sample-test : PASS,FLAKY
66

77
[true] # This section applies to all platforms
8-
# Postmortem debugging data is prone to accidental removal during V8 updates.
9-
test-postmortem-metadata: PASS,FLAKY
108

119
[$system==win32]
1210
test-child-process-fork-net: PASS,FLAKY

0 commit comments

Comments
 (0)