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

GD-75: Fix test parameter parsing of parameterized tests with String arguments #104

Merged
merged 1 commit into from
Jan 30, 2023

Conversation

MikeSchulze
Copy link
Owner

Why

Using Strings containing spaces or tabs was failing on parameterized tests. The valus was distorted.

What

  • Fix the test argument parsing by do not clean up the function signature to pare the arguments
  • Fix also failing GdUnitExecutor test based on changed gdunit settings during test run

cherry-picked from GdUnit3

…arguments (#359)

# Why
Using Strings containing spaces or tabs was failing on parameterized tests.
The valus was distorted.

# What
- Fix the test argument parsing by do not clean up the function signature to pare the arguments
- Fix also failing GdUnitExecutor test based on changed gdunit settings during test run


cherry-picked from Gdunit3
# Conflicts:
#	addons/gdUnit3/test/core/TestSuiteScannerTest.gd
#	addons/gdUnit4/src/core/parse/GdScriptParser.gd
#	addons/gdUnit4/test/core/GdUnitExecutorTest.gd
#	addons/gdUnit4/test/core/parse/GdScriptParserTest.gd
@MikeSchulze MikeSchulze self-assigned this Jan 30, 2023
@MikeSchulze MikeSchulze linked an issue Jan 30, 2023 that may be closed by this pull request
@MikeSchulze MikeSchulze merged commit b3693eb into master Jan 30, 2023
@MikeSchulze MikeSchulze deleted the GD-75 branch January 30, 2023 19:38
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.

GD-75: Using strings as paramterized test arguments are distorted
1 participant