Skip to content

TICKET-602: Fix marks_earned marker ignored when earned equals total or zero#716

Merged
david-yz-liu merged 3 commits intomasterfrom
TICKET-602_prevent_pytest_mark_overwrites
Apr 24, 2026
Merged

TICKET-602: Fix marks_earned marker ignored when earned equals total or zero#716
david-yz-liu merged 3 commits intomasterfrom
TICKET-602_prevent_pytest_mark_overwrites

Conversation

@Naragod
Copy link
Copy Markdown
Contributor

@Naragod Naragod commented Apr 10, 2026

@Naragod Naragod added this to the v2.9.7 milestone Apr 10, 2026
@Naragod Naragod requested a review from donny-wong April 10, 2026 15:59
Copy link
Copy Markdown
Contributor

@donny-wong donny-wong left a comment

Choose a reason for hiding this comment

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

Thank you @Naragod, looks good! Please add this PR to the changelog.

@Naragod
Copy link
Copy Markdown
Contributor Author

Naragod commented Apr 17, 2026

Thank you @Naragod, looks good! Please add this PR to the changelog.

Done.

@donny-wong donny-wong requested a review from david-yz-liu April 22, 2026 13:28
Copy link
Copy Markdown
Contributor

@david-yz-liu david-yz-liu left a comment

Choose a reason for hiding this comment

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

@Naragod the change is good, please just note my comment about code style

Comment thread server/autotest_server/testers/py/py_tester.py Outdated
@Naragod Naragod requested a review from david-yz-liu April 23, 2026 19:34
Copy link
Copy Markdown
Contributor

@david-yz-liu david-yz-liu left a comment

Choose a reason for hiding this comment

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

Thanks, @Naragod!

@david-yz-liu david-yz-liu merged commit dacf807 into master Apr 24, 2026
7 checks passed
@david-yz-liu david-yz-liu deleted the TICKET-602_prevent_pytest_mark_overwrites branch April 24, 2026 01:27
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.

markus_marks_earned gives full marks when 0 is entered markus_marks_earned overridden when earned marks == total marks and a test fails

3 participants