Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Testcase #4630

Draft
wants to merge 88 commits into
base: master
Choose a base branch
from
Draft

Testcase #4630

wants to merge 88 commits into from

Conversation

Gouvernathor
Copy link
Member

@Gouvernathor Gouvernathor commented May 13, 2023

To do :

  • document drag, scroll, move, type, jump and call
  • don't let the launcher freeze during test execution
  • show a better error message when the test fails (open it like Lint maybe, in the renpy-set editor instead of the system-default text editor)

Possible improvements (various ideas, sometimes contradictory, may be left to future PRs) :

  • list all tests in the Navigate Script menu (along defines, todos, labels, transforms...), with a button on each to launch it
  • make some testcases "publics" (meant to be triggered directly from the launcher) and others not
  • have a config or something else (all the public testcases in alphabetical order for example) store a list of testcases, defaulting to ("default",), to execute in sequence when clicking the "launch tests"
    • when the game quits at the end of one testcase, the next is launched by the launcher - it's not a jump at the end of the testcase, there's a cold reboot.

Currently, the "Launch tests" button on the launcher, just below Lint, launches the default testcase and that's it.

@Gouvernathor Gouvernathor added the enhancement A proposed or requested enhancement or improvement to renpy's features. label May 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement A proposed or requested enhancement or improvement to renpy's features.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant