Skip to content

Add new gameplay tests - #1033

Merged
4ian merged 7 commits into
mainfrom
claude/gameplay-tests-starters-6jb8r0
Aug 9, 2026
Merged

Add new gameplay tests#1033
4ian merged 7 commits into
mainfrom
claude/gameplay-tests-starters-6jb8r0

Conversation

@4ian

@4ian 4ian commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions

github-actions Bot commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

Preview the game(s) changed or added in this Pull Request


This is an automatic message displaying links to the games in this PR - double check the JSON in case of doubt.

@4ian
4ian force-pushed the claude/gameplay-tests-starters-6jb8r0 branch from 440ab12 to b1d40a3 Compare August 9, 2026 15:35
GDevelop master d1b3797 fixes the inverted camera-measured pitch, the last
thing that stopped the aiming helper being usable, so the two tests that aim
now use it instead of doing the trigonometry themselves:

- starting-first-person-shooter drops its hand-rolled pitch controller (which
  computed the eye height and knew the game pitches around Y). The shot now
  lands 5 units from the target.
- starting-first-person-farming drops the sweep that looked down a step at a
  time and acted along the way. It now aims straight at the seed patch, and
  ends at 34 degrees - the true angle to it.

Both need `toleranceDegrees`: the default 3 degrees misses a target at this
range.

The shooting test now waits for the impact effects of the click that takes
the pointer lock to expire before counting them. They are short lived, and
the helper is fast enough that an old one dying as the new one is born kept
the count at 1, making a good shot look like it did nothing.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01M2jc7PVAvmMmirAQude2v1
@4ian
4ian merged commit db96575 into main Aug 9, 2026
5 checks passed
@4ian
4ian deleted the claude/gameplay-tests-starters-6jb8r0 branch August 9, 2026 20:34
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.

2 participants