Add Unittesting #60

sithlord48 · 2021-10-15T01:21:05Z

Create unittests dir to house our testing structure
Create in unittests/subdir a simple macro MAKE_FF7TK_<subdir>_TEST to create a unittest for that library
- Takes a NAME
- Source File
Make tests for data objects
- FF7Text Test to check all chars. (Ignores all spaces being 0x00)
- FF7Save Full Coverage!
Make tests for utils objects
- PsColor
- LZS
- GZIPPS
- GZIP
Fix Any defects found by tests.

.github/workflows/build.yml

src/data/FF7Save.cpp

unittests/data/CMakeLists.txt

remove use of goto

3ee19ee

sithlord48 force-pushed the addTests branch 9 times, most recently from e807595 to 57f39d1 Compare October 17, 2021 13:54

sithlord48 changed the title ~~Add tests~~ Add Unittesting Oct 17, 2021

sithlord48 force-pushed the addTests branch 2 times, most recently from c50ccd6 to 683c7e4 Compare October 18, 2021 12:08

Fix ExportSlot, fix setPSXDesc

f74ae88

sithlord48 force-pushed the addTests branch 16 times, most recently from 6e74bf0 to 6363d9c Compare October 20, 2021 18:26

sithlord48 force-pushed the addTests branch from 791654e to 84e94aa Compare October 22, 2021 13:59

myst6re reviewed Oct 22, 2021

View reviewed changes

.github/workflows/build.yml Show resolved Hide resolved

myst6re reviewed Oct 22, 2021

View reviewed changes

src/data/FF7Save.cpp Show resolved Hide resolved

myst6re reviewed Oct 22, 2021

View reviewed changes

unittests/data/CMakeLists.txt Show resolved Hide resolved

myst6re reviewed Oct 22, 2021

View reviewed changes

unittests/data/CMakeLists.txt Outdated Show resolved Hide resolved

myst6re self-requested a review October 22, 2021 14:11

sithlord48 force-pushed the addTests branch 14 times, most recently from 0f8c51a to 13c7ea3 Compare October 23, 2021 12:38

sithlord48 added 5 commits October 23, 2021 14:36

Add Correct Camera Strings

f2db76f

Fix Switch Detect

400b27c

Implement FF7Save::lenSlot

8d2e5f9

adjust maxZ

97f01ce

Add Graphviz, Fix macOS stdLib

610519a

sithlord48 force-pushed the addTests branch from 13c7ea3 to f5ac91c Compare October 23, 2021 18:36

Enable Tests

785cd30

sithlord48 force-pushed the addTests branch from f5ac91c to 785cd30 Compare October 23, 2021 19:19

sithlord48 merged commit 96d38e8 into master Oct 23, 2021

sithlord48 deleted the addTests branch October 23, 2021 21:12

Provide feedback