Skip to content

chore: fix typo in n-queens test description#2167

Merged
trekhleb merged 1 commit into
trekhleb:masterfrom
saixiaoxi:chore/typo-nqueens-test-have
May 28, 2026
Merged

chore: fix typo in n-queens test description#2167
trekhleb merged 1 commit into
trekhleb:masterfrom
saixiaoxi:chore/typo-nqueens-test-have

Conversation

@saixiaoxi
Copy link
Copy Markdown
Contributor

Summary

Fixes # — __
Source issue:

Spec

Problem

The n-queens Jest test description contains a typo: hae should be have.

Acceptance criteria

  • The test description reads naturally: should not have solution for 3 queens.
  • No runtime behavior changes are introduced.

Approach

Fix the typo in the n-queens test name only.

Files likely touched

  • src/algorithms/uncategorized/n-queens/__test__/nQueens.test.js

Risk / blast radius

Very low. This is a test description string only and does not affect algorithm behavior.

Test plan

  • Run the focused n-queens Jest test.
  • Run git diff --check.

Work breakdown (TDD)

TODO

  • Fix the typo in the n-queens test description.
  • Verify the focused n-queens test still passes.
  • Run whitespace diff checks.

Visual verification

No screenshots captured (stub or non-UI change).

Blockers / open questions

Test plan

  • pnpm lint
  • pnpm typecheck
  • pnpm test
  • pnpm build
  • Manual smoke on the affected surface (see screenshots)

Generated by the auto-github-contributor skill. The agent followed a red-green-refactor loop and captured visual artefacts under .auto-pr/screenshots/.

@saixiaoxi saixiaoxi force-pushed the chore/typo-nqueens-test-have branch from b8ea25e to c42b84a Compare May 28, 2026 07:59
@saixiaoxi saixiaoxi force-pushed the chore/typo-nqueens-test-have branch from c42b84a to c8f4396 Compare May 28, 2026 07:59
@trekhleb trekhleb merged commit 1c21085 into trekhleb:master May 28, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants