test(cli): assert dry run exit through its spy #2814
jdaltonAnnotations
12 errors and 10 warnings
|
socket(import-block-is-hoisted):
packages/cli/test/unit/commands/manifest/bazel/extract_bazel_to_maven-lockfile-gate.test.mts#L98
packages/cli/test/unit/commands/manifest/bazel/extract_bazel_to_maven-lockfile-gate.test.mts:98:1: Import is below executable code. Move it into the top import region while preserving dependency order. Vitest hoists vi.mock; vi.doMock requires a subsequent dynamic import.
|
|
socket(no-prose-jargon):
packages/cli/src/commands/manifest/bazel/bazel-maven-workspace.mts#L43
packages/cli/src/commands/manifest/bazel/bazel-maven-workspace.mts:43:1: Comment uses shop-metaphor jargon `wedged`; name the mechanism instead.
|
|
eslint(complexity):
packages/cli/src/commands/manifest/bazel/bazel-maven-workspace.mts#L63
packages/cli/src/commands/manifest/bazel/bazel-maven-workspace.mts:63:8: async function `processWorkspaceForMaven` has a complexity of 34. Maximum allowed is 15.
|
|
socket(import-block-is-hoisted):
packages/cli/test/unit/commands/manifest/bazel/extract_bazel_to_maven-completeness.test.mts#L101
packages/cli/test/unit/commands/manifest/bazel/extract_bazel_to_maven-completeness.test.mts:101:1: Import is below executable code. Move it into the top import region while preserving dependency order. Vitest hoists vi.mock; vi.doMock requires a subsequent dynamic import.
|
|
socket(import-block-is-hoisted):
packages/cli/test/unit/commands/manifest/bazel/extract_bazel_to_maven-completeness.test.mts#L100
packages/cli/test/unit/commands/manifest/bazel/extract_bazel_to_maven-completeness.test.mts:100:1: Import is below executable code. Move it into the top import region while preserving dependency order. Vitest hoists vi.mock; vi.doMock requires a subsequent dynamic import.
|
|
socket(import-block-is-hoisted):
packages/cli/test/unit/commands/manifest/bazel/extract_bazel_to_maven-completeness.test.mts#L99
packages/cli/test/unit/commands/manifest/bazel/extract_bazel_to_maven-completeness.test.mts:99:1: Import is below executable code. Move it into the top import region while preserving dependency order. Vitest hoists vi.mock; vi.doMock requires a subsequent dynamic import.
|
|
socket(import-block-is-hoisted):
packages/cli/test/unit/commands/manifest/bazel/extract_bazel_to_maven-completeness.test.mts#L98
packages/cli/test/unit/commands/manifest/bazel/extract_bazel_to_maven-completeness.test.mts:98:1: Import is below executable code. Move it into the top import region while preserving dependency order. Vitest hoists vi.mock; vi.doMock requires a subsequent dynamic import.
|
|
socket(import-block-is-hoisted):
packages/cli/test/unit/commands/manifest/bazel/extract_bazel_to_maven-completeness.test.mts#L94
packages/cli/test/unit/commands/manifest/bazel/extract_bazel_to_maven-completeness.test.mts:94:1: Import is below executable code. Move it into the top import region while preserving dependency order. Vitest hoists vi.mock; vi.doMock requires a subsequent dynamic import.
|
|
socket(import-block-is-hoisted):
packages/cli/test/unit/commands/manifest/bazel/extract_bazel_to_maven-completeness.test.mts#L93
packages/cli/test/unit/commands/manifest/bazel/extract_bazel_to_maven-completeness.test.mts:93:1: Import is below executable code. Move it into the top import region while preserving dependency order. Vitest hoists vi.mock; vi.doMock requires a subsequent dynamic import.
|
|
eslint(complexity):
packages/cli/src/util/socket/sdk.mts#L186
packages/cli/src/util/socket/sdk.mts:186:8: async function `setupSdk` has a complexity of 18. Maximum allowed is 15.
|
|
🧪 Test (ubuntu-latest)
Process completed with exit code 1.
|
|
packages/cli/test/integration/cli/cmd-fix.test.mts > socket fix > environment variable handling > should not show env var names when all CI env vars are present: `fix --dry-run --config {"apiToken":"fake-token","defaultOrg":"fake-org"}`:
packages/cli/test/integration/cli/cmd-fix.test.mts#L125
AssertionError: expected 1 to be +0 // Object.is equality
- Expected
+ Received
- 0
+ 1
❯ packages/cli/test/integration/cli/cmd-fix.test.mts:125:22
|
|
socket(no-required-in-options-bag):
packages/cli/src/commands/manifest/bazel/bazel-query-runner.mts#L11
packages/cli/src/commands/manifest/bazel/bazel-query-runner.mts:11:3: A required member is not an option; hoist it to a positional param (`fn(required, options?)`).
|
|
socket(no-required-in-options-bag):
packages/cli/src/commands/manifest/bazel/bazel-query-runner.mts#L10
packages/cli/src/commands/manifest/bazel/bazel-query-runner.mts:10:3: A required member is not an option; hoist it to a positional param (`fn(required, options?)`).
|
|
socket(no-optional-param-before-options-bag):
packages/cli/src/util/telemetry/integration.mts#L328
packages/cli/src/util/telemetry/integration.mts:328:3: Optional param `metadata` before the options bag; move it into the bag as `metadata?`.
|
|
socket(no-required-in-options-bag):
packages/cli/src/commands/manifest/bazel/extract_bazel_to_pypi.mts#L389
packages/cli/src/commands/manifest/bazel/extract_bazel_to_pypi.mts:389:3: A required member is not an option; hoist it to a positional param (`fn(required, options?)`).
|
|
socket(no-required-in-options-bag):
packages/cli/src/commands/manifest/bazel/extract_bazel_to_pypi.mts#L387
packages/cli/src/commands/manifest/bazel/extract_bazel_to_pypi.mts:387:3: A required member is not an option; hoist it to a positional param (`fn(required, options?)`).
|
|
socket(no-required-in-options-bag):
packages/cli/src/commands/manifest/bazel/extract_bazel_to_pypi.mts#L385
packages/cli/src/commands/manifest/bazel/extract_bazel_to_pypi.mts:385:3: A required member is not an option; hoist it to a positional param (`fn(required, options?)`).
|
|
socket(no-required-in-options-bag):
packages/cli/src/commands/manifest/bazel/extract_bazel_to_pypi.mts#L384
packages/cli/src/commands/manifest/bazel/extract_bazel_to_pypi.mts:384:3: A required member is not an option; hoist it to a positional param (`fn(required, options?)`).
|
|
socket(no-required-in-options-bag):
packages/cli/src/commands/manifest/bazel/extract_bazel_to_pypi.mts#L383
packages/cli/src/commands/manifest/bazel/extract_bazel_to_pypi.mts:383:3: A required member is not an option; hoist it to a positional param (`fn(required, options?)`).
|
|
socket(no-required-in-options-bag):
packages/cli/src/commands/manifest/bazel/extract_bazel_to_pypi.mts#L382
packages/cli/src/commands/manifest/bazel/extract_bazel_to_pypi.mts:382:3: A required member is not an option; hoist it to a positional param (`fn(required, options?)`).
|
|
socket(no-required-in-options-bag):
packages/cli/src/commands/manifest/bazel/extract_bazel_to_pypi.mts#L381
packages/cli/src/commands/manifest/bazel/extract_bazel_to_pypi.mts:381:3: A required member is not an option; hoist it to a positional param (`fn(required, options?)`).
|