Skip to content

Expressions: Add CHECK-DIAGS command to slang-lit; Move expression test - #1926

Open
AndrewNolte wants to merge 2 commits into
MikePopoloski:masterfrom
AndrewNolte:AndrewNolte/stack/26
Open

Expressions: Add CHECK-DIAGS command to slang-lit; Move expression test#1926
AndrewNolte wants to merge 2 commits into
MikePopoloski:masterfrom
AndrewNolte:AndrewNolte/stack/26

Conversation

@AndrewNolte

@AndrewNolte AndrewNolte commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Stacked PRs:


View individual changes

Expressions: Add CHECK-DIAGS command to slang-lit; Move expression test

This moves 'Assignment pattern errors' to the be a slang-lit test to show the diag ranges, and make updating expectations easier.
It also adds to the test to show some suboptimal diag ranges.

@codecov

codecov Bot commented Aug 5, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 98.79518% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 96.06%. Comparing base (e3291f8) to head (310fdb9).

Files with missing lines Patch % Lines
source/diagnostics/DiagnosticEngine.cpp 97.22% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1926      +/-   ##
==========================================
+ Coverage   96.03%   96.06%   +0.03%     
==========================================
  Files         247      247              
  Lines       56855    56913      +58     
==========================================
+ Hits        54599    54675      +76     
+ Misses       2256     2238      -18     
Files with missing lines Coverage Δ
include/slang/diagnostics/DiagnosticEngine.h 100.00% <100.00%> (ø)
include/slang/diagnostics/JsonDiagnosticClient.h 100.00% <ø> (ø)
include/slang/diagnostics/TextDiagnosticClient.h 100.00% <ø> (ø)
source/diagnostics/JsonDiagnosticClient.cpp 91.56% <100.00%> (+37.39%) ⬆️
source/diagnostics/TextDiagnosticClient.cpp 93.89% <100.00%> (+0.59%) ⬆️
source/diagnostics/DiagnosticEngine.cpp 91.46% <97.22%> (-0.19%) ⬇️

... and 2 files with indirect coverage changes


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e3291f8...310fdb9. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@AndrewNolte
AndrewNolte marked this pull request as draft August 6, 2026 04:13
@AndrewNolte
AndrewNolte force-pushed the AndrewNolte/stack/26 branch from 07adf65 to fa4a4f2 Compare August 6, 2026 04:13
@AndrewNolte
AndrewNolte marked this pull request as ready for review August 6, 2026 04:13
@AndrewNolte
AndrewNolte marked this pull request as draft August 6, 2026 13:57
@AndrewNolte
AndrewNolte force-pushed the AndrewNolte/stack/26 branch from fa4a4f2 to 58562f7 Compare August 6, 2026 13:57
@AndrewNolte
AndrewNolte marked this pull request as ready for review August 6, 2026 13:57
@MikePopoloski

Copy link
Copy Markdown
Owner

I don't really want all of these tests moved over in one giant chunk. There's no way I can review these to make sure nothing got dropped or changed, and debugging the regression tests is slightly more annoying than with the in-process unit tests. If you want to put new tests and/or move a few specific ones over that's fine.

@AndrewNolte

Copy link
Copy Markdown
Contributor Author

Will amend to just the one on the following pr

@AndrewNolte
AndrewNolte marked this pull request as draft August 7, 2026 16:05
@AndrewNolte
AndrewNolte force-pushed the AndrewNolte/stack/26 branch from 58562f7 to 83ae1f1 Compare August 7, 2026 16:06
@AndrewNolte
AndrewNolte marked this pull request as ready for review August 7, 2026 16:06
@AndrewNolte
AndrewNolte marked this pull request as draft August 7, 2026 17:14
@AndrewNolte
AndrewNolte force-pushed the AndrewNolte/stack/26 branch from 83ae1f1 to a2a5fb6 Compare August 7, 2026 17:14
@AndrewNolte AndrewNolte changed the title Expressions: Move diagnostic tests to slang-lit; Add CHECK-DIAGS command to slang-lit Expressions: Add CHECK-DIAGS command to slang-lit; Move expression test Aug 7, 2026
@AndrewNolte
AndrewNolte marked this pull request as ready for review August 7, 2026 17:14
This puts some more restrictions on notes, many that were already being followed:
- Notes cannot have subnotes
- Notes cannot be waived, only diags
- Note severities can only be remapped to ignored, not errors

stack-info: PR: MikePopoloski#1925, branch: AndrewNolte/stack/25
This moves 'Assignment pattern errors' to the be a slang-lit test to show the diag ranges, and make updating expectations easier.
It also adds to the test to show some suboptimal diag ranges.

stack-info: PR: MikePopoloski#1926, branch: AndrewNolte/stack/26
@AndrewNolte
AndrewNolte marked this pull request as draft August 13, 2026 14:25
@AndrewNolte
AndrewNolte force-pushed the AndrewNolte/stack/26 branch from a2a5fb6 to 310fdb9 Compare August 13, 2026 14:25
@AndrewNolte
AndrewNolte marked this pull request as ready for review August 13, 2026 14:26
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