Skip to content

Unit Tests#6

Merged
math0898 merged 6 commits intov1.2from
5-unit-tests
Mar 24, 2022
Merged

Unit Tests#6
math0898 merged 6 commits intov1.2from
5-unit-tests

Conversation

@math0898
Copy link
Copy Markdown
Owner

@math0898 math0898 commented Mar 17, 2022 •
edited
Loading

  • Added Mockito to mock objects during unit testing.
  • Implemented setThread() unit test, and BeforeEach setup.
  • Added unit test for getThread().
  • Made the build test action no longer depend on UnitTests
  • UnitTest action still depends on Build but not the standalone build.
  • Mocking awt.swing objects to prevent headless exceptions.
  • Finished implementing the unit tests of BasicGame.
  • Added volatile keyword to GraphicsPanel.drawingListeners to hopefully make thread safe.

@math0898 math0898 added the enhancement New feature or request label Mar 17, 2022
@math0898 math0898 self-assigned this Mar 17, 2022
@math0898 math0898 linked an issue Mar 17, 2022 that may be closed by this pull request
Unit Tests #5
Closed
@math0898 math0898 added the patch Fix of a bug or unintended behavior. label Mar 17, 2022
@math0898 math0898 added this to the v1.2.* milestone Mar 17, 2022
math0898 added 6 commits March 19, 2022 23:42
@math0898
Dependency Inversion of Game
496ffc5 
- Completed dependency inversion of Game. If you wish to maintain the new functionality use BasicGame.
@math0898
Moved around Game stuffs
5b86f71
@math0898
Dependency Inversion, Unit Test
c779e36
@math0898
Continued Implementing BasicGame tests.
73a1396
@math0898
Continued Implementing BasicGame tests.
d3d5b04
@math0898
Finished BasicGame Unit tests
acdb941 
- Finished implementing the unit tests of BasicGame.
- Added volatile keyword to GraphicsPanel.drawingListeners to hopefully make thread safe.
@math0898 math0898 changed the base branch from main to v1.2 March 24, 2022 08:05
@math0898 math0898 merged commit ed2b01d into v1.2 Mar 24, 2022
@math0898 math0898 deleted the 5-unit-tests branch March 24, 2022 08:08
@math0898 math0898 mentioned this pull request Mar 24, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@math0898 math0898

Labels

enhancement New feature or request patch Fix of a bug or unintended behavior.

Projects

None yet

Milestone

v1.2.*

Development

Successfully merging this pull request may close these issues.

Unit Tests

1 participant

@math0898