Skip to content

testsuite: add rust generator test - #1154

Merged
kiritofeng merged 1 commit into
DMOJ:masterfrom
Riolku:rust-generator-test
Dec 27, 2023
Merged

testsuite: add rust generator test#1154
kiritofeng merged 1 commit into
DMOJ:masterfrom
Riolku:rust-generator-test

Conversation

@Riolku

@Riolku Riolku commented Dec 27, 2023

Copy link
Copy Markdown
Contributor

This would have caught a failure on #1148.

@codecov-commenter

codecov-commenter commented Dec 27, 2023

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (1806ba4) 82.71% compared to head (a97e9d7) 82.77%.
Report is 1 commits behind head on master.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1154      +/-   ##
==========================================
+ Coverage   82.71%   82.77%   +0.05%     
==========================================
  Files         141      141              
  Lines        5310     5311       +1     
==========================================
+ Hits         4392     4396       +4     
+ Misses        918      915       -3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@kiritofeng kiritofeng left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Otherwise LGTM, thanks!

Comment thread testsuite/rust_generator/tests/AC/test.yml Outdated
This would have caught a failure on DMOJ#1148.
@Riolku
Riolku force-pushed the rust-generator-test branch from 87ad60e to a97e9d7 Compare December 27, 2023 04:35
@Riolku

Riolku commented Dec 27, 2023

Copy link
Copy Markdown
Contributor Author

Also, I did in fact test this against #1148.

@kiritofeng kiritofeng left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, thanks!

@kiritofeng
kiritofeng merged commit cc641b3 into DMOJ:master Dec 27, 2023
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