Skip to content

Add Overworld test plans#112

Merged
Gr33ndev merged 29 commits intomainfrom
dokumentation/overworld-test-plans
Aug 12, 2022
Merged

Add Overworld test plans#112
Gr33ndev merged 29 commits intomainfrom
dokumentation/overworld-test-plans

Conversation

@WuestFlorian
Copy link
Contributor

@WuestFlorian WuestFlorian commented Aug 10, 2022

@cdingm
Copy link
Collaborator

cdingm commented Aug 10, 2022

i commited the HUD tests, so i changed the comment again. But we should be sure that there are no more test plans needed before merging

Gr33ndev
Gr33ndev previously approved these changes Aug 10, 2022
Copy link
Contributor

@Gr33ndev Gr33ndev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All test plans lgtm, I also tested them on my laptop.

Tested on: MacBook Pro 2020 m1 13" - macOS Monterey 12.5 - Firefox 103.0.2 (64-Bit) - Testdeploy Server

U.area.1 -> works
U.area.2 -> not checked

U.barrier.1 -> works
U.barrier.2 -> you can walk to area 2 even when the barrier is set because of the updated hitboxes
U.barrier.3 -> works but same as U.barrier.2

U.dungeon.1 -> works

U.hud.1 -> works
U.hud.2 -> works
U.hud.3 -> minimap zoom works as intended - overworld zoom level 1 and 2 is identical for me, level 3 zooms far out
U.hud.4 -> works
U.hud.5 -> works

U.minigame.1 -> minigamespot isn't blue - everything else works
U.minigame.2 -> works
U.minigame.3 -> works
U.minigame.4 -> works

U.object.1 -> not checked

Copy link
Contributor

@delvh delvh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll leave the formalities of complying with the template again up to you.

@@ -0,0 +1,35 @@
# Name: Area colliders
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should probably be split up into separate testplans per area,
maybe even with this testplan being refactored into a summary of the other testplans.

Comment on lines 27 to 28
8. The tester zooms out the game with the keybind and checks if the minimap also zoomed out
9. The tester zooms in the game with the keybind and checks if the minimap also zoomed in
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Isn't that already in another test plan?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes, but rather be safe then sorry no?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why test it twice?
Doesn't that only waste time?

Copy link
Contributor

@delvh delvh Aug 10, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Especially if we test for multiple OSs?

Comment on lines 20 to 33
1. The tester checks if the Player head is always in the middle of the minimap by walking around
2. The tester zooms in the minimap with the keybind
3. The tester zooms out the minimap with the keybind
4. The tester zooms in the minimap with the button
5. The tester zooms out the minimap with the button
6. The tester checks if there are always 4 zoom levels
7. The tester walks around and checks if the minimap displays npc heads and area symbols
8. The tester zooms out the game with the keybind and checks if the minimap also zoomed out
9. The tester zooms in the game with the keybind and checks if the minimap also zoomed in
10. The tester walks around with different zoom levels and checks the minimap if everything is displayed correctly

## Alternative procedures

N/A
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
1. The tester checks if the Player head is always in the middle of the minimap by walking around
2. The tester zooms in the minimap with the keybind
3. The tester zooms out the minimap with the keybind
4. The tester zooms in the minimap with the button
5. The tester zooms out the minimap with the button
6. The tester checks if there are always 4 zoom levels
7. The tester walks around and checks if the minimap displays npc heads and area symbols
8. The tester zooms out the game with the keybind and checks if the minimap also zoomed out
9. The tester zooms in the game with the keybind and checks if the minimap also zoomed in
10. The tester walks around with different zoom levels and checks the minimap if everything is displayed correctly
## Alternative procedures
N/A
1. Check that the Player head is always in the middle of the minimap when walking around
2. Zoom in the minimap with the shortcut
3. Zoom out the minimap with the shortcut
4. Check that there is a finite and constant amount of zoom levels
5. Walk around and check if the minimap displays NPC heads and area symbols
6. Walk around with different zoom levels and check on the minimap if everything is displayed correctly and no world borders are visible
## Alternative procedures
2.1. zoom in using the button \
3.1. zoom out using the button

@@ -0,0 +1,39 @@
# Name: Minigame spot configured and not done
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can be deleted then.

@delvh
Copy link
Contributor

delvh commented Aug 10, 2022

U.area.1 -> works U.area.2 -> not checked

U.barrier.1 -> works U.barrier.2 -> you can walk to area 2 even when the barrier is set because of the updated hitboxes U.barrier.3 -> works but same as U.barrier.2

U.dungeon.1 -> works

U.hud.1 -> works U.hud.2 -> works U.hud.3 -> minimap zoom works as intended - overworld zoom level 1 and 2 is identical for me, level 3 zooms far out U.hud.4 -> works U.hud.5 -> works

U.minigame.1 -> minigamespot isn't blue - everything else works U.minigame.2 -> works U.minigame.3 -> works U.minigame.4 -> works

U.object.1 -> not checked

Copied your test results into the execution log. Next time, please do it yourself.

@Gr33ndev
Copy link
Contributor

Copied your test results into the execution log. Next time, please do it yourself.

Thank you, I had tried that, but unfortunately the sheet is locked...
Bildschirmfoto 2022-08-10 um 17 02 49

@delvh
Copy link
Contributor

delvh commented Aug 10, 2022

(Google only displays it like that. I've explicitly excluded the columns A-G.)

@Gr33ndev
Copy link
Contributor

Oh ok, sorry... I didn't know that....

Co-authored-by: delvh <dev.lh@web.de>
WuestFlorian and others added 2 commits August 11, 2022 14:19
Apply suggestions from delvh

Co-authored-by: delvh <dev.lh@web.de>
@@ -0,0 +1,35 @@
# Overworld: Barrier block locked worlds (`u.overworld-3`)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
# Overworld: Barrier block locked worlds (`u.overworld-3`)
# Overworld: Barriers block locked worlds (`u.overworld-03`)

Also, the file name should be adapted to the action as well...

@delvh
Copy link
Contributor

delvh commented Aug 11, 2022

Preferably, these paddings can even be added to the linkages, but that is absolutely optional, as sorting should work correctly as the filenames are padded.

Co-authored-by: delvh <dev.lh@web.de>
Co-authored-by: Gr33nbl00d <st172032@stud.uni-stuttgart.de>
@delvh
Copy link
Contributor

delvh commented Aug 11, 2022

When do we merge?

@delvh
Copy link
Contributor

delvh commented Aug 11, 2022

Deadline for further comments: Today@3pm

@Gr33ndev Gr33ndev merged commit 82829ef into main Aug 12, 2022
@Gr33ndev Gr33ndev deleted the dokumentation/overworld-test-plans branch August 12, 2022 10:54
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 this pull request may close these issues.

Testing: Testplans for the Overworld

5 participants