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

Rename TestCase to TestScene #2365

Merged
merged 12 commits into from
May 14, 2019
Merged

Rename TestCase to TestScene #2365

merged 12 commits into from
May 14, 2019

Conversation

UselessToucan
Copy link
Contributor

Current naming is not correct.
A Test Case is defined as a set of actions executed to verify a particular feature or functionality of the software application. It is not about running tests, creating scene content and so on.
I think it is more correct to call existing class a Scene.

@peppy
Copy link
Member

peppy commented Apr 23, 2019

Indifferent. May help distinguish from nUnit [TestCase]?

@UselessToucan
Copy link
Contributor Author

May help distinguish from nUnit [TestCase]?

yes, it is one of the reasons for this change.

@peppy peppy merged commit 293defe into ppy:master May 14, 2019
@UselessToucan UselessToucan deleted the rename_test_case branch May 14, 2019 09:02
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.

2 participants