Skip to content

Tests: re-enable dependency source file test - #434

Merged
LesterEvSe merged 1 commit into
BlockstreamResearch:masterfrom
LesterEvSe:tests/dep-re-enable
Oct 7, 2026
Merged

LesterEvSe merged 1 commit into
BlockstreamResearch:masterfrom
LesterEvSe:tests/dep-re-enable

Conversation

@LesterEvSe

@LesterEvSe LesterEvSe commented Oct 7, 2026 •

Copy link
Copy Markdown
Collaborator

Re-enables the test ignored in #290. The bug was already fixed in #370, but the test checked Display, which has no file paths, so this PR checked it with render_to_string function.

Checklist

  • If this PR adds or changes a language feature, it is used in
    examples/kitchen_sink.simf, and test-data/kitchen_sink.json is
    regenerated.

@LesterEvSe
LesterEvSe requested a review from KyrylR October 7, 2026 13:34
@LesterEvSe LesterEvSe self-assigned this Oct 7, 2026
@LesterEvSe
LesterEvSe requested a review from delta1 as a code owner October 7, 2026 13:34
@LesterEvSe LesterEvSe added the bug Something isn't working label Oct 7, 2026
@KyrylR

KyrylR commented Oct 7, 2026

Copy link
Copy Markdown
Collaborator

ACK cff30a3; successfully ran local tests

@LesterEvSe
LesterEvSe merged commit 1a41b2d into BlockstreamResearch:master Oct 7, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ErrorHandler: Fails to report correct source file path for dependency AST errors

2 participants