Skip to content

Conversation

@ReimarBauer
Copy link
Member

Purpose of PR?:

Fixes #659

Does this PR introduce a breaking change?

If the changes in this PR are manually verified, list down the scenarios covered::

Additional information for reviewer? :
Mention if this PR is part of any design or a continuation of previous PRs

Does this PR results in some Documentation changes?
If yes, include the list of Documentation changes

Checklist:

  • Bug fix. Fixes #
  • New feature (Non-API breaking changes that adds functionality)
  • PR Title follows the convention of <type>: <subject>
  • Commit has unit tests

@ReimarBauer ReimarBauer linked an issue May 14, 2025 that may be closed by this pull request
@ReimarBauer ReimarBauer changed the title tests for the msui.main function tests for the mss main() function May 14, 2025
@ReimarBauer ReimarBauer requested a review from matrss May 14, 2025 15:47
@ReimarBauer
Copy link
Member Author

@matrss Looks like my changes are causing the timeout.

with pytest.raises(SystemExit):
mswms.main()
captured = capsys.readouterr()
assert "Debug mode: on" in captured.out
Copy link
Member Author

Choose a reason for hiding this comment

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

this fails currently, because it writes off here.

@ReimarBauer ReimarBauer marked this pull request as draft May 16, 2025 17:04
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.

write tests for several main() functions

2 participants