Commit 01eebed
committed
chore: drop Node 18 from CI matrix, fix useObservable test for Vitest 4
Vite 8 (via rolldown) requires Node 20+ for styleText from node:util.
Update CI test matrix to Node 20/22 and build/publish jobs to Node 20.
Node 18 reached EOL in April 2025.
Also fix the "throws an error if no observableId is provided" test which
was failing under Vitest 4. React 18 dispatches a window error event
when a component throws during render, even when the throw is caught by
expect().toThrow(). Vitest 4 surfaces these as uncaught errors. Fix by
adding a window error listener that calls preventDefault(), and match on
the error message via expect.objectContaining to account for React
setting _suppressLogging on the error object.
Also widen the firebase peerDependency to accept v10/v11/v12.1 parent f3beddc commit 01eebed
3 files changed
Lines changed: 14 additions & 5 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
23 | 23 | | |
24 | 24 | | |
25 | 25 | | |
26 | | - | |
| 26 | + | |
27 | 27 | | |
28 | 28 | | |
29 | 29 | | |
| |||
44 | 44 | | |
45 | 45 | | |
46 | 46 | | |
47 | | - | |
| 47 | + | |
48 | 48 | | |
49 | 49 | | |
50 | 50 | | |
| |||
83 | 83 | | |
84 | 84 | | |
85 | 85 | | |
86 | | - | |
| 86 | + | |
87 | 87 | | |
88 | 88 | | |
89 | 89 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
36 | 36 | | |
37 | 37 | | |
38 | 38 | | |
39 | | - | |
| 39 | + | |
40 | 40 | | |
41 | 41 | | |
42 | 42 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
137 | 137 | | |
138 | 138 | | |
139 | 139 | | |
| 140 | + | |
| 141 | + | |
| 142 | + | |
| 143 | + | |
| 144 | + | |
| 145 | + | |
| 146 | + | |
| 147 | + | |
140 | 148 | | |
141 | 149 | | |
142 | 150 | | |
143 | 151 | | |
144 | | - | |
| 152 | + | |
145 | 153 | | |
146 | 154 | | |
147 | 155 | | |
| 156 | + | |
148 | 157 | | |
149 | 158 | | |
150 | 159 | | |
| |||
0 commit comments