Skip to content

Conversation

@maxmynter
Copy link
Contributor

@maxmynter maxmynter commented May 1, 2025

Re: #17526

Summary

Add integration tests for Python Semantic Syntax for InvalidStarExpression, DuplicateMatchKey, and DuplicateMatchClassAttribute.

Note

  • Red knot integration tests for DuplicateMatchKey exist already in line 89-101.

Test Plan

This is a test.

@github-actions
Copy link
Contributor

github-actions bot commented May 1, 2025

mypy_primer results

No ecosystem changes detected ✅

@maxmynter maxmynter changed the title (tests) Add integration tests for Python Semantic Syntax [semantic-syntax-tests] Add integration tests for Python Semantic Syntax May 1, 2025
@maxmynter maxmynter changed the title [semantic-syntax-tests] Add integration tests for Python Semantic Syntax [semantic-syntax-tests] for for InvalidStarExpression, DuplicateMatchKey, and DuplicateMatchClassAttribute May 1, 2025
@github-actions
Copy link
Contributor

github-actions bot commented May 1, 2025

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

@MichaReiser MichaReiser added the testing Related to testing Ruff itself label May 1, 2025
Copy link
Contributor

@carljm carljm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

red-knot tests look good

@MichaReiser MichaReiser requested a review from ntBre May 3, 2025 20:30
Copy link
Contributor

@ntBre ntBre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

for `InvalidStarExpression`, `DuplicateMatchKey`, and
`DuplicateMatchClassAttribue`
@ntBre ntBre force-pushed the semantic-coverage-v2 branch from c54410d to ea62fc9 Compare May 5, 2025 16:14
@ntBre
Copy link
Contributor

ntBre commented May 5, 2025

I rebased to pick up the red-knot -> ty name change. Will merge once CI passes!

@ntBre ntBre enabled auto-merge (squash) May 5, 2025 16:15
@ntBre
Copy link
Contributor

ntBre commented May 5, 2025

Ah, I forgot we changed the error message in #17772 too. I'll push one more update.

@ntBre ntBre merged commit 101e1a5 into astral-sh:main May 5, 2025
34 checks passed
AlexWaygood pushed a commit that referenced this pull request May 5, 2025
…tchKey`, and `DuplicateMatchClassAttribute` (#17754)

Re: #17526 

## Summary
Add integration tests for Python Semantic Syntax for
`InvalidStarExpression`, `DuplicateMatchKey`, and
`DuplicateMatchClassAttribute`.

## Note
- Red knot integration tests for `DuplicateMatchKey` exist already in
line 89-101.
<!-- What's the purpose of the change? What does it do, and why? -->

## Test Plan
This is a test.
<!-- How was it tested? -->
Glyphack pushed a commit to Glyphack/ruff that referenced this pull request May 6, 2025
…tchKey`, and `DuplicateMatchClassAttribute` (astral-sh#17754)

Re: astral-sh#17526 

## Summary
Add integration tests for Python Semantic Syntax for
`InvalidStarExpression`, `DuplicateMatchKey`, and
`DuplicateMatchClassAttribute`.

## Note
- Red knot integration tests for `DuplicateMatchKey` exist already in
line 89-101.
<!-- What's the purpose of the change? What does it do, and why? -->

## Test Plan
This is a test.
<!-- How was it tested? -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testing Related to testing Ruff itself

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants