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

Code Smells: components/cutscenes #652

Closed
28 tasks done
Tracked by #455
Caleb-J-Murphy opened this issue Oct 17, 2024 · 0 comments · Fixed by #658
Closed
28 tasks done
Tracked by #455

Code Smells: components/cutscenes #652

Caleb-J-Murphy opened this issue Oct 17, 2024 · 0 comments · Fixed by #658

Comments

@Caleb-J-Murphy
Copy link
Contributor

Caleb-J-Murphy commented Oct 17, 2024

Description

Task: Code Smell: components/cutscenes.
Feature: Code Smells
Fix all 22 code smells located in the main game folder, specifically the /game/components/cutscenes/ folder.
Fix all 49 code smells located in the test game folder, specifically the /game/components/cutscenes/ folder.

Dependencies

N/A

Milestones

The game files that need to fixed:

  • BackstoryCutscene.java
  • Cutscene.java
  • CutsceneActions.java
  • CutsceneArea.java
  • CutsceneFactory.java
  • CutsceneScreenDisplay.java
  • CutsceneTextDisplay.java
  • Day2Cutscene.java
  • Day3Cutscene.java
  • IntroCutscene.java

The test files that need to be fixed:

  • AnimatedSceneTest.java
  • BackgroundCutsceneTest.java
  • BackstoryCutsceneDisplayTest.java
  • BackstoryCutsceneTest.java
  • BadEndCutsceneTest.java
  • CutsceneScreenDisplayTest.java
  • CutsceneTest.java
  • Day2CutsceneTest.java
  • Day3CutsceneTest.java
  • Day4CutsceneTest.java
  • GoodEndCutsceneTest.java
  • GoodEndTest.java
  • IntroCutsceneTest.java
  • LoseEndTest.java
  • MoralDay1CutsceneTest.java
  • MoralDay2CutsceneTest.java
  • MoralDay3CutsceneTest.java
  • MoralDay4CutsceneTest.java

Completion Deadline: Oct. 17

Documentation

  • ...

Member

@Caleb-J-Murphy Caleb-J-Murphy mentioned this issue Oct 17, 2024
17 tasks
Caleb-J-Murphy pushed a commit that referenced this issue Oct 17, 2024
Caleb-J-Murphy pushed a commit that referenced this issue Oct 17, 2024
@Caleb-J-Murphy Caleb-J-Murphy linked a pull request Oct 17, 2024 that will close this issue
27 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant