Skip to content

Add @pytest.mark.lalr1 marker to test_lalr in test_lalr1_but_not_slr.py#13

Merged
alejandroklever merged 2 commits intofeature/implement-testsfrom
copilot/sub-pr-9
Mar 3, 2026
Merged

Add @pytest.mark.lalr1 marker to test_lalr in test_lalr1_but_not_slr.py#13
alejandroklever merged 2 commits intofeature/implement-testsfrom
copilot/sub-pr-9

Conversation

Copy link

Copilot AI commented Feb 28, 2026

test_lalr1_but_not_slr.py was missing the @pytest.mark.lalr1 decorator on test_lalr(), making it impossible to selectively run LALR1 tests via pytest markers (e.g. -m lalr1).

Changes

  • Added import pytest to tests/test_lalr1_but_not_slr.py
  • Added @pytest.mark.lalr1 to test_lalr(), consistent with the pattern used in test_arithmetic_grammar.py
@pytest.mark.lalr1
def test_lalr():
    g = grammar()
    parser = g.get_parser("lalr1")
    assert not parser.has_conflicts

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

…not_slr.py

Co-authored-by: alejandroklever <45394625+alejandroklever@users.noreply.github.com>
Copilot AI changed the title [WIP] Update grammar tests based on feedback in PR #9 Add @pytest.mark.lalr1 marker to test_lalr in test_lalr1_but_not_slr.py Feb 28, 2026
@alejandroklever alejandroklever marked this pull request as ready for review March 3, 2026 17:03
@alejandroklever alejandroklever merged commit d8b05e0 into feature/implement-tests Mar 3, 2026
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