Run the assertion in test_add_extras_only_accepts_one_package

[dylanpulver]

The assertion in test_add_extras_only_accepts_one_package sits inside the with pytest.raises(...) block, after the call that raises:

with pytest.raises(ValueError) as e:
    tester.execute(command)
    assert (
        str(e.value)
        == "You can only specify one package when using the --extras option"
    )

tester.execute(command) raises, so control leaves the block and the assertion below it never runs. The test still passes, but only because pytest.raises caught the ValueError; the message it names is unchecked.

Moving it out of the block is the whole change. It passes there, and it now fails if the expected message is wrong (changing the expected string gives assert 'You can only...extras option' == 'NOT THE REAL MESSAGE').

tests/console/commands/test_add.py: 84 passed. ruff check and ruff format clean.

Found with an AST scan for assertions positioned after the raising statement inside a pytest.raises block. It was the only one in the repository.

This PR was written with AI assistance (Claude Code), including the scan and the verification above.

[sourcery-ai]

Hey - I've reviewed your changes and they look great!

Sourcery assessment

Approved.

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook