Skip to content

Conversation

@keturn
Copy link
Contributor

@keturn keturn commented Sep 4, 2021

Separating some test clean-up out from #64.

It claims to test the deprecated interface, which we're getting close to removing.
But it doesn't even do that! It uses the new MTEExtension! Hypocrisy!
@keturn keturn added Size: S Small effort likely only affecting a single area and requiring little to no research Category: Test/QA Requests, Issues and Changes targeting tests and quality assurance labels Sep 4, 2021
Cervator
Cervator previously approved these changes Sep 5, 2021
Copy link
Member

@Cervator Cervator left a comment

Choose a reason for hiding this comment

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

Neat! Definitely clear what the test does, and it seems really well done!

Haven't tested or been particularly deep in this area, but I like what I see :-)

@keturn keturn merged commit 7928a2e into develop Sep 5, 2021
@keturn keturn deleted the test/clearing branch September 5, 2021 00:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Category: Test/QA Requests, Issues and Changes targeting tests and quality assurance Size: S Small effort likely only affecting a single area and requiring little to no research

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants