Commit 36332eb
readline: use Date.now() and move test to parallel
The readline module wants a truthy time while using Timer.now() doesn't
necessarily guarantee that early on in the process' life. It also
doesn't actually resolve the timing issues experienced in an earlier
issue. Instead, this PR fixes the related tests and moves them back
to parallel.
Refs: #14674
PR-URL: #18563
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>1 parent 0277993 commit 36332ebCopy full SHA for 36332eb
File tree
Expand file treeCollapse file tree
3 files changed
+81
-114
lines changedOpen diff view settings
Filter options
- lib
- test
- parallel
- sequential
Expand file treeCollapse file tree
3 files changed
+81
-114
lines changedOpen diff view settings
Collapse file
+4-6Lines changed: 4 additions & 6 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
48 | 48 | |
49 | 49 | |
50 | 50 | |
51 | | - |
52 | | - |
53 | 51 | |
54 | 52 | |
55 | 53 | |
| ||
411 | 409 | |
412 | 410 | |
413 | 411 | |
414 | | - |
| 412 | + |
415 | 413 | |
416 | 414 | |
417 | 415 | |
| ||
424 | 422 | |
425 | 423 | |
426 | 424 | |
427 | | - |
| 425 | + |
428 | 426 | |
429 | 427 | |
430 | 428 | |
| ||
917 | 915 | |
918 | 916 | |
919 | 917 | |
920 | | - |
| 918 | + |
921 | 919 | |
922 | 920 | |
923 | 921 | |
924 | 922 | |
925 | 923 | |
926 | 924 | |
927 | | - |
| 925 | + |
928 | 926 | |
929 | 927 | |
930 | 928 | |
|
Collapse file
test/parallel/test-readline-interface.js
Copy file name to clipboardExpand all lines: test/parallel/test-readline-interface.js+77Lines changed: 77 additions & 0 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
869 | 869 | |
870 | 870 | |
871 | 871 | |
| 872 | + |
| 873 | + |
| 874 | + |
| 875 | + |
| 876 | + |
| 877 | + |
| 878 | + |
| 879 | + |
| 880 | + |
| 881 | + |
| 882 | + |
| 883 | + |
| 884 | + |
| 885 | + |
| 886 | + |
| 887 | + |
| 888 | + |
| 889 | + |
| 890 | + |
| 891 | + |
| 892 | + |
| 893 | + |
| 894 | + |
| 895 | + |
| 896 | + |
| 897 | + |
| 898 | + |
| 899 | + |
| 900 | + |
| 901 | + |
| 902 | + |
| 903 | + |
| 904 | + |
| 905 | + |
| 906 | + |
| 907 | + |
| 908 | + |
| 909 | + |
| 910 | + |
| 911 | + |
| 912 | + |
| 913 | + |
| 914 | + |
| 915 | + |
| 916 | + |
| 917 | + |
| 918 | + |
| 919 | + |
| 920 | + |
| 921 | + |
| 922 | + |
| 923 | + |
| 924 | + |
| 925 | + |
| 926 | + |
| 927 | + |
| 928 | + |
| 929 | + |
| 930 | + |
| 931 | + |
| 932 | + |
| 933 | + |
| 934 | + |
| 935 | + |
| 936 | + |
| 937 | + |
| 938 | + |
| 939 | + |
| 940 | + |
| 941 | + |
| 942 | + |
| 943 | + |
| 944 | + |
| 945 | + |
| 946 | + |
| 947 | + |
| 948 | + |
Collapse file
test/sequential/test-readline-interface.js
Copy file name to clipboardExpand all lines: test/sequential/test-readline-interface.js-108Lines changed: 0 additions & 108 deletions
This file was deleted.
0 commit comments