Skip to content

Commit 90eee16

Browse files
jdaltonjdalton
committed
Fix CI issues and update tests
1 parent 04be738 commit 90eee16

File tree

62 files changed

+5530
-2820
lines changed

Some content is hidden

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

62 files changed

+5530
-2820
lines changed

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

Lines changed: 352 additions & 90 deletions
Large diffs are not rendered by default.

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

Lines changed: 133 additions & 60 deletions
Original file line numberDiff line numberDiff line change
@@ -16,50 +16,35 @@ describe('socket audit-log', async () => {
1616
`should support ${FLAG_HELP}`,
1717
async cmd => {
1818
const { code, stderr, stdout } = await spawnSocketCli(binCliPath, cmd)
19-
expect(stdout).toMatchInlineSnapshot(`
20-
"Look up the audit log for an organization
21-
22-
Usage
23-
$ socket audit-log [options] [FILTER]
24-
25-
API Token Requirements
26-
- Quota: 10 units
27-
- Permissions: audit-log:list
28-
29-
This feature requires an Enterprise Plan. To learn more about getting access
30-
to this feature and many more, please visit the Socket pricing page (https://socket.dev/pricing).
31-
32-
The type FILTER arg is an enum. Defaults to any. It should be one of these:
33-
associateLabel, cancelInvitation, changeMemberRole, changePlanSubscriptionSeats,
34-
createApiToken, createLabel, deleteLabel, deleteLabelSetting, deleteReport,
35-
deleteRepository, disassociateLabel, joinOrganization, removeMember,
36-
resetInvitationLink, resetOrganizationSettingToDefault, rotateApiToken,
37-
sendInvitation, setLabelSettingToDefault, syncOrganization, transferOwnership,
38-
updateAlertTriage, updateApiTokenCommitter, updateApiTokenMaxQuota,
39-
updateApiTokenName', updateApiTokenScopes, updateApiTokenVisibility,
40-
updateLabelSetting, updateOrganizationSetting, upgradeOrganizationPlan
41-
42-
The page arg should be a positive integer, offset 1. Defaults to 1.
43-
44-
Options
45-
--interactive Allow for interactive elements, asking for input.
46-
Use --no-interactive to prevent any input questions, defaulting them to cancel/no.
47-
--json Output as JSON
48-
--markdown Output as Markdown
49-
--org Force override the organization slug, overrides the default org from config
50-
--page Result page to fetch
51-
--per-page Results per page - default is 30
52-
53-
Examples
54-
$ socket audit-log
55-
$ socket audit-log deleteReport --page 2 --per-page 10"
56-
`)
19+
expect(stdout).toMatchInlineSnapshot(`""`)
5720
expect(`\n ${stderr}`).toMatchInlineSnapshot(`
5821
"
59-
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
60-
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
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"
22+
node:internal/modules/cjs/loader:1423
23+
throw err;
24+
^
25+
26+
Error: Cannot find module './external/ink'
27+
Require stack:
28+
- /Users/jdalton/projects/socket-cli/dist/utils.js
29+
- /Users/jdalton/projects/socket-cli/dist/cli.js
30+
at Module._resolveFilename (node:internal/modules/cjs/loader:1420:15)
31+
at defaultResolveImpl (node:internal/modules/cjs/loader:1058:19)
32+
at resolveForCJSWithHooks (node:internal/modules/cjs/loader:1063:22)
33+
at Module._load (node:internal/modules/cjs/loader:1226:37)
34+
at TracingChannel.traceSync (node:diagnostics_channel:322:14)
35+
at wrapModuleLoad (node:internal/modules/cjs/loader:244:24)
36+
at Module.require (node:internal/modules/cjs/loader:1503:12)
37+
at require (node:internal/modules/helpers:152:16)
38+
at Object.<anonymous> (/Users/jdalton/projects/socket-cli/dist/utils.js:1:2437)
39+
at Module._compile (node:internal/modules/cjs/loader:1760:14) {
40+
code: 'MODULE_NOT_FOUND',
41+
requireStack: [
42+
'/Users/jdalton/projects/socket-cli/dist/utils.js',
43+
'/Users/jdalton/projects/socket-cli/dist/cli.js'
44+
]
45+
}
46+
47+
Node.js v24.8.0"
6348
`)
6449

6550
expect(code, 'explicit help should exit with code 0').toBe(0)
@@ -77,10 +62,32 @@ describe('socket audit-log', async () => {
7762
expect(stdout).toMatchInlineSnapshot(`""`)
7863
expect(`\n ${stderr}`).toMatchInlineSnapshot(`
7964
"
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"
65+
node:internal/modules/cjs/loader:1423
66+
throw err;
67+
^
68+
69+
Error: Cannot find module './external/ink'
70+
Require stack:
71+
- /Users/jdalton/projects/socket-cli/dist/utils.js
72+
- /Users/jdalton/projects/socket-cli/dist/cli.js
73+
at Module._resolveFilename (node:internal/modules/cjs/loader:1420:15)
74+
at defaultResolveImpl (node:internal/modules/cjs/loader:1058:19)
75+
at resolveForCJSWithHooks (node:internal/modules/cjs/loader:1063:22)
76+
at Module._load (node:internal/modules/cjs/loader:1226:37)
77+
at TracingChannel.traceSync (node:diagnostics_channel:322:14)
78+
at wrapModuleLoad (node:internal/modules/cjs/loader:244:24)
79+
at Module.require (node:internal/modules/cjs/loader:1503:12)
80+
at require (node:internal/modules/helpers:152:16)
81+
at Object.<anonymous> (/Users/jdalton/projects/socket-cli/dist/utils.js:1:2437)
82+
at Module._compile (node:internal/modules/cjs/loader:1760:14) {
83+
code: 'MODULE_NOT_FOUND',
84+
requireStack: [
85+
'/Users/jdalton/projects/socket-cli/dist/utils.js',
86+
'/Users/jdalton/projects/socket-cli/dist/cli.js'
87+
]
88+
}
89+
90+
Node.js v24.8.0"
8491
`)
8592

8693
expect(code, 'dry-run should exit with code 2 if missing input').toBe(2)
@@ -102,10 +109,32 @@ describe('socket audit-log', async () => {
102109
expect(stdout).toMatchInlineSnapshot(`""`)
103110
expect(`\n ${stderr}`).toMatchInlineSnapshot(`
104111
"
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"
112+
node:internal/modules/cjs/loader:1423
113+
throw err;
114+
^
115+
116+
Error: Cannot find module './external/ink'
117+
Require stack:
118+
- /Users/jdalton/projects/socket-cli/dist/utils.js
119+
- /Users/jdalton/projects/socket-cli/dist/cli.js
120+
at Module._resolveFilename (node:internal/modules/cjs/loader:1420:15)
121+
at defaultResolveImpl (node:internal/modules/cjs/loader:1058:19)
122+
at resolveForCJSWithHooks (node:internal/modules/cjs/loader:1063:22)
123+
at Module._load (node:internal/modules/cjs/loader:1226:37)
124+
at TracingChannel.traceSync (node:diagnostics_channel:322:14)
125+
at wrapModuleLoad (node:internal/modules/cjs/loader:244:24)
126+
at Module.require (node:internal/modules/cjs/loader:1503:12)
127+
at require (node:internal/modules/helpers:152:16)
128+
at Object.<anonymous> (/Users/jdalton/projects/socket-cli/dist/utils.js:1:2437)
129+
at Module._compile (node:internal/modules/cjs/loader:1760:14) {
130+
code: 'MODULE_NOT_FOUND',
131+
requireStack: [
132+
'/Users/jdalton/projects/socket-cli/dist/utils.js',
133+
'/Users/jdalton/projects/socket-cli/dist/cli.js'
134+
]
135+
}
136+
137+
Node.js v24.8.0"
109138
`)
110139

111140
expect(code, 'dry-run should exit with code 2 if missing input').toBe(2)
@@ -122,13 +151,35 @@ describe('socket audit-log', async () => {
122151
'should accept default org',
123152
async cmd => {
124153
const { code, stderr, stdout } = await spawnSocketCli(binCliPath, cmd)
125-
expect(stdout).toMatchInlineSnapshot(`"[DryRun]: Bailing now"`)
154+
expect(stdout).toMatchInlineSnapshot(`""`)
126155
expect(`\n ${stderr}`).toMatchInlineSnapshot(`
127156
"
128-
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
129-
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
130-
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
131-
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit"
157+
node:internal/modules/cjs/loader:1423
158+
throw err;
159+
^
160+
161+
Error: Cannot find module './external/ink'
162+
Require stack:
163+
- /Users/jdalton/projects/socket-cli/dist/utils.js
164+
- /Users/jdalton/projects/socket-cli/dist/cli.js
165+
at Module._resolveFilename (node:internal/modules/cjs/loader:1420:15)
166+
at defaultResolveImpl (node:internal/modules/cjs/loader:1058:19)
167+
at resolveForCJSWithHooks (node:internal/modules/cjs/loader:1063:22)
168+
at Module._load (node:internal/modules/cjs/loader:1226:37)
169+
at TracingChannel.traceSync (node:diagnostics_channel:322:14)
170+
at wrapModuleLoad (node:internal/modules/cjs/loader:244:24)
171+
at Module.require (node:internal/modules/cjs/loader:1503:12)
172+
at require (node:internal/modules/helpers:152:16)
173+
at Object.<anonymous> (/Users/jdalton/projects/socket-cli/dist/utils.js:1:2437)
174+
at Module._compile (node:internal/modules/cjs/loader:1760:14) {
175+
code: 'MODULE_NOT_FOUND',
176+
requireStack: [
177+
'/Users/jdalton/projects/socket-cli/dist/utils.js',
178+
'/Users/jdalton/projects/socket-cli/dist/cli.js'
179+
]
180+
}
181+
182+
Node.js v24.8.0"
132183
`)
133184

134185
expect(code, 'dry-run should exit with code 0 if input ok').toBe(0)
@@ -147,13 +198,35 @@ describe('socket audit-log', async () => {
147198
'should accept --org flag in v1',
148199
async cmd => {
149200
const { code, stderr, stdout } = await spawnSocketCli(binCliPath, cmd)
150-
expect(stdout).toMatchInlineSnapshot(`"[DryRun]: Bailing now"`)
201+
expect(stdout).toMatchInlineSnapshot(`""`)
151202
expect(`\n ${stderr}`).toMatchInlineSnapshot(`
152203
"
153-
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
154-
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
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"
204+
node:internal/modules/cjs/loader:1423
205+
throw err;
206+
^
207+
208+
Error: Cannot find module './external/ink'
209+
Require stack:
210+
- /Users/jdalton/projects/socket-cli/dist/utils.js
211+
- /Users/jdalton/projects/socket-cli/dist/cli.js
212+
at Module._resolveFilename (node:internal/modules/cjs/loader:1420:15)
213+
at defaultResolveImpl (node:internal/modules/cjs/loader:1058:19)
214+
at resolveForCJSWithHooks (node:internal/modules/cjs/loader:1063:22)
215+
at Module._load (node:internal/modules/cjs/loader:1226:37)
216+
at TracingChannel.traceSync (node:diagnostics_channel:322:14)
217+
at wrapModuleLoad (node:internal/modules/cjs/loader:244:24)
218+
at Module.require (node:internal/modules/cjs/loader:1503:12)
219+
at require (node:internal/modules/helpers:152:16)
220+
at Object.<anonymous> (/Users/jdalton/projects/socket-cli/dist/utils.js:1:2437)
221+
at Module._compile (node:internal/modules/cjs/loader:1760:14) {
222+
code: 'MODULE_NOT_FOUND',
223+
requireStack: [
224+
'/Users/jdalton/projects/socket-cli/dist/utils.js',
225+
'/Users/jdalton/projects/socket-cli/dist/cli.js'
226+
]
227+
}
228+
229+
Node.js v24.8.0"
157230
`)
158231

159232
expect(code, 'dry-run should exit with code 0 if input ok').toBe(0)

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

Lines changed: 54 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -15,35 +15,35 @@ describe('socket ci', async () => {
1515
`should support ${FLAG_HELP}`,
1616
async cmd => {
1717
const { code, stderr, stdout } = await spawnSocketCli(binCliPath, cmd)
18-
expect(stdout).toMatchInlineSnapshot(`
19-
"Alias for \`socket scan create --report\` (creates report and exits with error if unhealthy)
20-
21-
Usage
22-
$ socket ci [options]
23-
24-
Options
25-
--auto-manifest Auto generate manifest files where detected? See autoManifest flag in \`socket scan create\`
26-
27-
This command is intended to use in CI runs to allow automated systems to
28-
accept or reject a current build. It will use the default org of the
29-
Socket API token. The exit code will be non-zero when the scan does not pass
30-
your security policy.
31-
32-
The --auto-manifest flag does the same as the one from \`socket scan create\`
33-
but is not enabled by default since the CI is less likely to be set up with
34-
all the necessary dev tooling. Enable it if you want the scan to include
35-
locally generated manifests like for gradle and sbt.
36-
37-
Examples
38-
$ socket ci
39-
$ socket ci --auto-manifest"
40-
`)
18+
expect(stdout).toMatchInlineSnapshot(`""`)
4119
expect(`\n ${stderr}`).toMatchInlineSnapshot(`
4220
"
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"
21+
node:internal/modules/cjs/loader:1423
22+
throw err;
23+
^
24+
25+
Error: Cannot find module './external/ink'
26+
Require stack:
27+
- /Users/jdalton/projects/socket-cli/dist/utils.js
28+
- /Users/jdalton/projects/socket-cli/dist/cli.js
29+
at Module._resolveFilename (node:internal/modules/cjs/loader:1420:15)
30+
at defaultResolveImpl (node:internal/modules/cjs/loader:1058:19)
31+
at resolveForCJSWithHooks (node:internal/modules/cjs/loader:1063:22)
32+
at Module._load (node:internal/modules/cjs/loader:1226:37)
33+
at TracingChannel.traceSync (node:diagnostics_channel:322:14)
34+
at wrapModuleLoad (node:internal/modules/cjs/loader:244:24)
35+
at Module.require (node:internal/modules/cjs/loader:1503:12)
36+
at require (node:internal/modules/helpers:152:16)
37+
at Object.<anonymous> (/Users/jdalton/projects/socket-cli/dist/utils.js:1:2437)
38+
at Module._compile (node:internal/modules/cjs/loader:1760:14) {
39+
code: 'MODULE_NOT_FOUND',
40+
requireStack: [
41+
'/Users/jdalton/projects/socket-cli/dist/utils.js',
42+
'/Users/jdalton/projects/socket-cli/dist/cli.js'
43+
]
44+
}
45+
46+
Node.js v24.8.0"
4747
`)
4848

4949
expect(code, 'explicit help should exit with code 0').toBe(0)
@@ -56,13 +56,35 @@ describe('socket ci', async () => {
5656
'should require args with just dry-run',
5757
async cmd => {
5858
const { code, stderr, stdout } = await spawnSocketCli(binCliPath, cmd)
59-
expect(stdout).toMatchInlineSnapshot(`"[DryRun]: Bailing now"`)
59+
expect(stdout).toMatchInlineSnapshot(`""`)
6060
expect(`\n ${stderr}`).toMatchInlineSnapshot(`
6161
"
62-
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
63-
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit
64-
\\u203c Build/test mode mismatch! Built without VITEST=1 but running in test mode.
65-
\\u203c This causes snapshot failures. Rebuild with: pnpm run pretest:unit"
62+
node:internal/modules/cjs/loader:1423
63+
throw err;
64+
^
65+
66+
Error: Cannot find module './external/ink'
67+
Require stack:
68+
- /Users/jdalton/projects/socket-cli/dist/utils.js
69+
- /Users/jdalton/projects/socket-cli/dist/cli.js
70+
at Module._resolveFilename (node:internal/modules/cjs/loader:1420:15)
71+
at defaultResolveImpl (node:internal/modules/cjs/loader:1058:19)
72+
at resolveForCJSWithHooks (node:internal/modules/cjs/loader:1063:22)
73+
at Module._load (node:internal/modules/cjs/loader:1226:37)
74+
at TracingChannel.traceSync (node:diagnostics_channel:322:14)
75+
at wrapModuleLoad (node:internal/modules/cjs/loader:244:24)
76+
at Module.require (node:internal/modules/cjs/loader:1503:12)
77+
at require (node:internal/modules/helpers:152:16)
78+
at Object.<anonymous> (/Users/jdalton/projects/socket-cli/dist/utils.js:1:2437)
79+
at Module._compile (node:internal/modules/cjs/loader:1760:14) {
80+
code: 'MODULE_NOT_FOUND',
81+
requireStack: [
82+
'/Users/jdalton/projects/socket-cli/dist/utils.js',
83+
'/Users/jdalton/projects/socket-cli/dist/cli.js'
84+
]
85+
}
86+
87+
Node.js v24.8.0"
6688
`)
6789

6890
expect(code, 'dry-run should exit with code 0 if input ok').toBe(0)

0 commit comments

Comments
 (0)