Skip to content

Commit fe74f52

Browse files
jdaltonjdalton
committed
Update test snapshots and improve test coverage
1 parent b64d51f commit fe74f52

File tree

76 files changed

+3067
-35
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

76 files changed

+3067
-35
lines changed

‎src/commands/analytics/cmd-analytics.test.mts‎

Lines changed: 52 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,10 @@ describe('socket analytics', async () => {
5151
if (!skipOnWin32Node24) {
5252
expect(`\n ${stderr}`).toMatchInlineSnapshot(`
5353
"
54+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
55+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
56+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
57+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
5458
_____ _ _ /---------------
5559
| __|___ ___| |_ ___| |_ | CLI: <redacted>
5660
|__ | * | _| '_| -_| _| | token: <redacted>, org: <redacted>
@@ -73,6 +77,10 @@ describe('socket analytics', async () => {
7377
expect(stdout).toMatchInlineSnapshot(`"[DryRun]: Bailing now"`)
7478
expect(`\n ${stderr}`).toMatchInlineSnapshot(`
7579
"
80+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
81+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
82+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
83+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
7684
_____ _ _ /---------------
7785
| __|___ ___| |_ ___| |_ | CLI: <redacted>
7886
|__ | * | _| '_| -_| _| | token: <redacted>, org: <redacted>
@@ -100,6 +108,10 @@ describe('socket analytics', async () => {
100108
expect(stdout).toMatchInlineSnapshot(`""`)
101109
expect(`\n ${stderr}`).toMatchInlineSnapshot(`
102110
"
111+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
112+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
113+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
114+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
103115
_____ _ _ /---------------
104116
| __|___ ___| |_ ___| |_ | CLI: <redacted>
105117
|__ | * | _| '_| -_| _| | token: <redacted>, org: <redacted>
@@ -123,6 +135,10 @@ describe('socket analytics', async () => {
123135
expect(stdout).toMatchInlineSnapshot(`"[DryRun]: Bailing now"`)
124136
expect(`\n ${stderr}`).toMatchInlineSnapshot(`
125137
"
138+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
139+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
140+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
141+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
126142
_____ _ _ /---------------
127143
| __|___ ___| |_ ___| |_ | CLI: <redacted>
128144
|__ | * | _| '_| -_| _| | token: <redacted>, org: <redacted>
@@ -141,6 +157,10 @@ describe('socket analytics', async () => {
141157
expect(stdout).toMatchInlineSnapshot(`"[DryRun]: Bailing now"`)
142158
expect(`\n ${stderr}`).toMatchInlineSnapshot(`
143159
"
160+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
161+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
162+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
163+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
144164
_____ _ _ /---------------
145165
| __|___ ___| |_ ___| |_ | CLI: <redacted>
146166
|__ | * | _| '_| -_| _| | token: <redacted>, org: <redacted>
@@ -165,6 +185,10 @@ describe('socket analytics', async () => {
165185
expect(stdout).toMatchInlineSnapshot(`""`)
166186
expect(`\n ${stderr}`).toMatchInlineSnapshot(`
167187
"
188+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
189+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
190+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
191+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
168192
_____ _ _ /---------------
169193
| __|___ ___| |_ ___| |_ | CLI: <redacted>
170194
|__ | * | _| '_| -_| _| | token: <redacted>, org: <redacted>
@@ -195,6 +219,10 @@ describe('socket analytics', async () => {
195219
expect(stdout).toMatchInlineSnapshot(`"[DryRun]: Bailing now"`)
196220
expect(`\n ${stderr}`).toMatchInlineSnapshot(`
197221
"
222+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
223+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
224+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
225+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
198226
_____ _ _ /---------------
199227
| __|___ ___| |_ ___| |_ | CLI: <redacted>
200228
|__ | * | _| '_| -_| _| | token: <redacted>, org: <redacted>
@@ -213,6 +241,10 @@ describe('socket analytics', async () => {
213241
expect(stdout).toMatchInlineSnapshot(`"[DryRun]: Bailing now"`)
214242
expect(`\n ${stderr}`).toMatchInlineSnapshot(`
215243
"
244+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
245+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
246+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
247+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
216248
_____ _ _ /---------------
217249
| __|___ ___| |_ ___| |_ | CLI: <redacted>
218250
|__ | * | _| '_| -_| _| | token: <redacted>, org: <redacted>
@@ -231,6 +263,10 @@ describe('socket analytics', async () => {
231263
expect(stdout).toMatchInlineSnapshot(`"[DryRun]: Bailing now"`)
232264
expect(`\n ${stderr}`).toMatchInlineSnapshot(`
233265
"
266+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
267+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
268+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
269+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
234270
_____ _ _ /---------------
235271
| __|___ ___| |_ ___| |_ | CLI: <redacted>
236272
|__ | * | _| '_| -_| _| | token: <redacted>, org: <redacted>
@@ -249,6 +285,10 @@ describe('socket analytics', async () => {
249285
expect(stdout).toMatchInlineSnapshot(`"[DryRun]: Bailing now"`)
250286
expect(`\n ${stderr}`).toMatchInlineSnapshot(`
251287
"
288+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
289+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
290+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
291+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
252292
_____ _ _ /---------------
253293
| __|___ ___| |_ ___| |_ | CLI: <redacted>
254294
|__ | * | _| '_| -_| _| | token: <redacted>, org: <redacted>
@@ -275,6 +315,10 @@ describe('socket analytics', async () => {
275315
expect(stdout).toMatchInlineSnapshot(`""`)
276316
expect(`\n ${stderr}`).toMatchInlineSnapshot(`
277317
"
318+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
319+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
320+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
321+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
278322
_____ _ _ /---------------
279323
| __|___ ___| |_ ___| |_ | CLI: <redacted>
280324
|__ | * | _| '_| -_| _| | token: <redacted>, org: <redacted>
@@ -305,6 +349,10 @@ describe('socket analytics', async () => {
305349
expect(stdout).toMatchInlineSnapshot(`"[DryRun]: Bailing now"`)
306350
expect(`\n ${stderr}`).toMatchInlineSnapshot(`
307351
"
352+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
353+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
354+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
355+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
308356
_____ _ _ /---------------
309357
| __|___ ___| |_ ___| |_ | CLI: <redacted>
310358
|__ | * | _| '_| -_| _| | token: <redacted>, org: <redacted>
@@ -331,6 +379,10 @@ describe('socket analytics', async () => {
331379
expect(stdout).toMatchInlineSnapshot(`"[DryRun]: Bailing now"`)
332380
expect(`\n ${stderr}`).toMatchInlineSnapshot(`
333381
"
382+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
383+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
384+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
385+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
334386
_____ _ _ /---------------
335387
| __|___ ___| |_ ___| |_ | CLI: <redacted>
336388
|__ | * | _| '_| -_| _| | token: <redacted>, org: <redacted>

‎src/commands/audit-log/cmd-audit-log.test.mts‎

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -58,6 +58,10 @@ describe('socket audit-log', async () => {
5858
)
5959
expect(`\n ${stderr}`).toMatchInlineSnapshot(`
6060
"
61+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
62+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
63+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
64+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
6165
_____ _ _ /---------------
6266
| __|___ ___| |_ ___| |_ | CLI: <redacted>
6367
|__ | * | _| '_| -_| _| | token: <redacted>, org: <redacted>
@@ -79,6 +83,10 @@ describe('socket audit-log', async () => {
7983
expect(stdout).toMatchInlineSnapshot(`""`)
8084
expect(`\n ${stderr}`).toMatchInlineSnapshot(`
8185
"
86+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
87+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
88+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
89+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
8290
_____ _ _ /---------------
8391
| __|___ ___| |_ ___| |_ | CLI: <redacted>
8492
|__ | * | _| '_| -_| _| | token: <redacted>, org: <redacted>
@@ -113,6 +121,10 @@ describe('socket audit-log', async () => {
113121
expect(stdout).toMatchInlineSnapshot(`""`)
114122
expect(`\n ${stderr}`).toMatchInlineSnapshot(`
115123
"
124+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
125+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
126+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
127+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
116128
_____ _ _ /---------------
117129
| __|___ ___| |_ ___| |_ | CLI: <redacted>
118130
|__ | * | _| '_| -_| _| | token: <redacted>, org: <redacted>
@@ -140,6 +152,10 @@ describe('socket audit-log', async () => {
140152
expect(stdout).toMatchInlineSnapshot(`"[DryRun]: Bailing now"`)
141153
expect(`\n ${stderr}`).toMatchInlineSnapshot(`
142154
"
155+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
156+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
157+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
158+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
143159
_____ _ _ /---------------
144160
| __|___ ___| |_ ___| |_ | CLI: <redacted>
145161
|__ | * | _| '_| -_| _| | token: <redacted>, org: <redacted>
@@ -165,6 +181,10 @@ describe('socket audit-log', async () => {
165181
expect(stdout).toMatchInlineSnapshot(`"[DryRun]: Bailing now"`)
166182
expect(`\n ${stderr}`).toMatchInlineSnapshot(`
167183
"
184+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
185+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
186+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
187+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
168188
_____ _ _ /---------------
169189
| __|___ ___| |_ ___| |_ | CLI: <redacted>
170190
|__ | * | _| '_| -_| _| | token: <redacted>, org: <redacted>

‎src/commands/ci/cmd-ci.test.mts‎

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,10 @@ describe('socket ci', async () => {
4040
`)
4141
expect(`\n ${stderr}`).toMatchInlineSnapshot(`
4242
"
43+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
44+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
45+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
46+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
4347
_____ _ _ /---------------
4448
| __|___ ___| |_ ___| |_ | CLI: <redacted>
4549
|__ | * | _| '_| -_| _| | token: <redacted>, org: <redacted>
@@ -59,6 +63,10 @@ describe('socket ci', async () => {
5963
expect(stdout).toMatchInlineSnapshot(`"[DryRun]: Bailing now"`)
6064
expect(`\n ${stderr}`).toMatchInlineSnapshot(`
6165
"
66+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
67+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
68+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
69+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
6270
_____ _ _ /---------------
6371
| __|___ ___| |_ ___| |_ | CLI: <redacted>
6472
|__ | * | _| '_| -_| _| | token: <redacted>, org: <redacted>

‎src/commands/cli.test.mts‎

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -77,6 +77,10 @@ describe('socket root command', async () => {
7777
`)
7878
expect(`\n ${stderr}`).toMatchInlineSnapshot(`
7979
"
80+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
81+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
82+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
83+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
8084
_____ _ _ /---------------
8185
| __|___ ___| |_ ___| |_ | CLI: <redacted>
8286
|__ | * | _| '_| -_| _| | token: <redacted>, org: <redacted>
@@ -98,7 +102,11 @@ describe('socket root command', async () => {
98102
)
99103
expect(`\n ${stderr}`).toMatchInlineSnapshot(`
100104
"
101-
usage: socketcli [-h] [--api-token <token>] [--repo <owner/repo>]
105+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
106+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
107+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
108+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
109+
usage: socketcli [-h] [--api-token <token>] [--repo <owner/repo>]
102110
[--repo-is-public] [--branch <name>] [--integration <type>]
103111
[--owner <name>] [--pr-number <number>]
104112
[--commit-message <message>] [--commit-sha <sha>]

‎src/commands/config/cmd-config-auto.test.mts‎

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,10 @@ describe('socket config auto', async () => {
4141
`)
4242
expect(`\n ${stderr}`).toMatchInlineSnapshot(`
4343
"
44+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
45+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
46+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
47+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
4448
_____ _ _ /---------------
4549
| __|___ ___| |_ ___| |_ | CLI: <redacted>
4650
|__ | * | _| '_| -_| _| | token: <redacted>, org: <redacted>
@@ -69,6 +73,10 @@ describe('socket config auto', async () => {
6973
expect(stdout).toMatchInlineSnapshot(`"[DryRun]: Bailing now"`)
7074
expect(`\n ${stderr}`).toMatchInlineSnapshot(`
7175
"
76+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
77+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
78+
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
79+
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
7280
_____ _ _ /---------------
7381
| __|___ ___| |_ ___| |_ | CLI: <redacted>
7482
|__ | * | _| '_| -_| _| | token: <redacted>, org: <redacted>

0 commit comments

Comments
 (0)