Testing Checklist: Difference between revisions
Jump to navigation
Jump to search
Hhyyrylainen (talk | contribs) (→Release Testing: added some specific release testing items) |
Hhyyrylainen (talk | contribs) (link the general testing page here) |
||
| (One intermediate revision by the same user not shown) | |||
| Line 2: | Line 2: | ||
[[Category:Testing]] | [[Category:Testing]] | ||
This page contains checklists to aid in testing Thrive. | This page contains checklists to aid in testing Thrive. General info on how to perform testing is on [[Testing|this page]]. | ||
[[Category:Testing]] | [[Category:Testing]] | ||
| Line 12: | Line 12: | ||
* The PR works correctly for the main feature / bug it addresses | * The PR works correctly for the main feature / bug it addresses | ||
* The PR properly fixes all the related issues it is marked as fixing | * The PR properly fixes all the related issues it is marked as fixing | ||
* If the PR changes or adds any new GUI parts, those work with a controller / keyboard (unless the change is to a prototype part of the game) and the [https://github.com/Revolutionary-Games/Thrive/blob/master/doc/making_guis.md working with GUIs] documentation recommendations are followed | |||
* Saving and loading does not break on the PR branch | * Saving and loading does not break on the PR branch | ||
* Saving while using the new feature / fixed feature works correctly (for example a PR changing the organelle move function should be tested so that a save is made and loaded while moving an organelle as that is a likely hotspot for issues regarding saving) | * Saving while using the new feature / fixed feature works correctly (for example a PR changing the organelle move function should be tested so that a save is made and loaded while moving an organelle as that is a likely hotspot for issues regarding saving) | ||
Latest revision as of 18:25, 12 April 2023
This page contains checklists to aid in testing Thrive. General info on how to perform testing is on this page.
Pull Request (PR) Testing
The following items should be (in the optimal case, often we don't have enough testing resources) checked for each pull request before merging:
- The PR works correctly for the main feature / bug it addresses
- The PR properly fixes all the related issues it is marked as fixing
- If the PR changes or adds any new GUI parts, those work with a controller / keyboard (unless the change is to a prototype part of the game) and the working with GUIs documentation recommendations are followed
- Saving and loading does not break on the PR branch
- Saving while using the new feature / fixed feature works correctly (for example a PR changing the organelle move function should be tested so that a save is made and loaded while moving an organelle as that is a likely hotspot for issues regarding saving)
- Loading saves made with a previous version should work correctly on the PR branch. If they don't and it is intentional the PR needs to at least add a save incompatibility point. Preferably the PR would include code for a save upgrader to upgrade old saves to be compatible.
- Different window resolutions display correctly (for PRs with GUI related changes). At least 720p and 1080p at 16:9 aspect ratio should be tested.
Release Testing
The following should be tested before releasing a new version of Thrive:
- Saving and loading works
- Loading a game made with previous version works, or a save incompatibility point is added (or a save version converter is implemented)
- Tutorials work
- Playing a short play session starting a new game and going to the editor a few times and making changes while playing without cheats, doesn't show up any issues
- In an optimal world: someone would have enough time to play for over an hour and complete the game to check that there aren't any issues regarding that
Some specific things to check that works:
- Flagellum animation
- Freebuild microbe editor for multiple editor cycles