Skip to content

Clean up test config for clarity#124

Merged
mixonic merged 1 commit intomasterfrom
mixonic/test-cleanup
Jun 4, 2021
Merged

Clean up test config for clarity#124
mixonic merged 1 commit intomasterfrom
mixonic/test-cleanup

Conversation

@mixonic
Copy link
Member

@mixonic mixonic commented Jun 4, 2021

Remove a test for "modern" arguments which are now just used through the whole codebase. Cause the test suite to throw when an unhandled exception is encountered.

Remove all use RAISE_ON_DEPRECATION stuff, as this addon has its own configuration for throwing on unhandled deprecation which should be under test here.

Extracted from #118

@mixonic mixonic force-pushed the mixonic/test-cleanup branch from 48f65bf to 86065d5 Compare June 4, 2021 01:33
Remove a test for "modern" arguments which are now just used through the
whole codebase. Cause the test suite to throw when an unhandled
exception is encountered.

Remove all use RAISE_ON_DEPRECATION stuff, as this addon has its own
configuration for throwing on unhandled deprecation which should be
under test here.
@mixonic mixonic force-pushed the mixonic/test-cleanup branch from 86065d5 to b8e6d4d Compare June 4, 2021 01:46
@mixonic mixonic merged commit 2dfaf4d into master Jun 4, 2021
@mixonic mixonic deleted the mixonic/test-cleanup branch June 4, 2021 01:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant