Skip to content

add script path to failed tests#2316

Merged
zhangchiqing merged 1 commit intomasterfrom
leo/add-script-path-to-failed-tests
Apr 10, 2026
Merged

add script path to failed tests#2316
zhangchiqing merged 1 commit intomasterfrom
leo/add-script-path-to-failed-tests

Conversation

@zhangchiqing
Copy link
Copy Markdown
Member

Add the cdc file path in the error message when tests fails, make it easier to find out which test is failing.

Before:

❌ Command Error: Checking failed:
error: too few arguments
  --> 7465737400000000000000000000000000000000000000000000000000000000:81:4

  See documentation at: https://cadence-lang.org/docs/language/functions

error: mismatched types
  --> 7465737400000000000000000000000000000000000000000000000000000000:81:106

After:

❌ Command Error: error in test file "/Users/leozhang/FlowALP/cadence/tests/withdraw_and_pull_deposit_and_push_test.cdc": Checking failed:
error: too few arguments
  --> 7465737400000000000000000000000000000000000000000000000000000000:81:4

  See documentation at: https://cadence-lang.org/docs/language/functions

error: mismatched types
  --> 7465737400000000000000000000000000000000000000000000000000000000:81:106

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 9, 2026

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA ce3ae37.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

Scanned Files

None

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Copy link
Copy Markdown
Contributor

@janezpodhostnik janezpodhostnik left a comment

Choose a reason for hiding this comment

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

Nice change!

@zhangchiqing zhangchiqing merged commit a3170ba into master Apr 10, 2026
11 of 12 checks passed
@zhangchiqing zhangchiqing deleted the leo/add-script-path-to-failed-tests branch April 10, 2026 17:46
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.

3 participants