edfcore 0.4.486 → 0.4.487
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/dist/constants.d.ts +1 -1
- package/dist/constants.js +1 -1
- package/docs/CHANGELOG.md +19 -0
- package/package.json +1 -1
- package/src/constants.ts +1 -1
package/dist/constants.d.ts
CHANGED
|
@@ -109,5 +109,5 @@ export declare const SIGNAL_FIELD_BLOCK_OFFSETS: {
|
|
|
109
109
|
readonly reserved: 224;
|
|
110
110
|
};
|
|
111
111
|
/** Published package version. Kept in sync with package.json by a test. */
|
|
112
|
-
export declare const VERSION = "0.4.
|
|
112
|
+
export declare const VERSION = "0.4.487";
|
|
113
113
|
//# sourceMappingURL=constants.d.ts.map
|
package/dist/constants.js
CHANGED
package/docs/CHANGELOG.md
CHANGED
|
@@ -6,6 +6,25 @@ alone does not tell you whether you were affected.
|
|
|
6
6
|
edfcore is pre-1.0. Patch releases have carried behaviour changes where the old behaviour was a
|
|
7
7
|
defect; those are called out below.
|
|
8
8
|
|
|
9
|
+
## 0.4.487
|
|
10
|
+
|
|
11
|
+
- **Set** the suite's test timeout explicitly, at thirty seconds, with the reasoning in the config.
|
|
12
|
+
vitest's default is five, which nobody here chose, and no test in this suite uses the clock as an
|
|
13
|
+
assertion — the cost tests count READS and BYTES, which is what makes them stable. The only job
|
|
14
|
+
a timeout has here is stopping an infinite loop from taking the run with it.
|
|
15
|
+
- Five seconds is too tight for that job. Several tests build multi-megabyte fixtures and the
|
|
16
|
+
heaviest ordinary case runs in about a second, so the default left under six times its own cost
|
|
17
|
+
as headroom — and a run under `--coverage` is several times slower, which is where the failures
|
|
18
|
+
appeared: two in `read-pattern.test.ts` and one in `envelope.test.ts`, all three passing on the
|
|
19
|
+
same commit without instrumentation. A full coverage run of the suite now passes where it failed
|
|
20
|
+
before this change.
|
|
21
|
+
- A timeout that fires on correct work teaches the reader to rerun rather than to read, which is
|
|
22
|
+
the opposite of what a red test is for.
|
|
23
|
+
- Thirty seconds is thirty times the slowest ordinary test and still fails a genuine hang well
|
|
24
|
+
inside a minute. The files needing longer still say so themselves, and 0.4.484's `beforeAll` is
|
|
25
|
+
unaffected: moving a file-level cost out of an arbitrary test body is right whatever the budget
|
|
26
|
+
is.
|
|
27
|
+
|
|
9
28
|
## 0.4.486
|
|
10
29
|
|
|
11
30
|
- **Added** tests for the sentence the repeat counter turns into. `parseTalRegion` collapses many
|
package/package.json
CHANGED
package/src/constants.ts
CHANGED