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.

Sign up for GitHub

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

Jump to bottom

Rework Level #14

Open
TheJoeSmo opened this issue Jan 3, 2021 · 0 comments
Open

Rework Level #14

TheJoeSmo opened this issue Jan 3, 2021 · 0 comments
Labels
enhancement New feature or request testing Need a test

Comments

@TheJoeSmo
Copy link
Owner

Is your feature request related to a problem? Please describe.
Level is unable to be tested and ultimately is a God class that handles way to many things.

Describe the solution you'd like
We need to abstract this class into multiple smaller parts and treat the Level class as a container to talk to the outside world. This would require a separation between the GUI and the business logic and additional abstraction for testing and DI.
@TheJoeSmo TheJoeSmo added enhancement New feature or request testing Need a test labels Jan 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request testing Need a test
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@TheJoeSmo