Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added tests for InitCommand #11678

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

jakobvibe
Copy link
Contributor

Added some test for InitCommand as requested in #10796

Simple testing, but added a test approach for testing protected functions through dummyclass extension

@Seldaek Seldaek changed the title Added tests Added tests for InitCommand Oct 6, 2023
@greew greew mentioned this pull request Oct 6, 2023
@Seldaek
Copy link
Member

Seldaek commented Oct 27, 2023

I have nothing against the cleanup with the dummy class, but did I miss something or did you get rid of a bunch of tests?

@Seldaek Seldaek added this to the 2.7 milestone Oct 27, 2023
@Seldaek Seldaek modified the milestones: 2.7, 2.8 Feb 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants