Re-enable the domain reload tests by BadSingleton · Pull Request #1404 · pythonnet/pythonnet

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@BadSingleton

Copy link

Contributor

@BadSingleton BadSingleton commented

Mar 1, 2021

edited

Loading

Fixes issue #1403

Checklist

Check all those that are applicable and complete.

  • Make sure to include one or more tests for your change
  • If an enhancement PR, please create docs and at best an example
  • Add yourself to AUTHORS
  • Updated the CHANGELOG

@BadSingleton BadSingleton changed the title Make the domain reload tests run again Re-enable the domain reload tests

Mar 1, 2021
Add `src/domain_tests` to `testpaths` in pyproject.toml

@filmor

Copy link

Member

filmor commented

Mar 1, 2021

Sorry about that. Could you please add these as separate actions, though, instead of adding them to the normal testing? They are only relevant on Mono and .NET Framework anyhow.

@BadSingleton

Copy link

Contributor Author

I think adding them as a separate action is not optimal, as devs will have to run pytest specifying explicitly the domain tests. I added pytest filtering instead

@BadSingleton

Copy link

Contributor Author

I think the timeouts are false positive. The domain tests takes about 1:15m to execute.

@lostmsu lostmsu merged commit 33d771c into pythonnet:master

May 3, 2021

@BadSingleton BadSingleton deleted the fix-1403 branch

May 4, 2021 13:20

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@BadSingleton @filmor @lostmsu