Skip to content

Fix 'noSuchProperty' test case. #2418

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.

Sign up for GitHub

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

Jump to bottom
Closed
gregturn opened this issue Jan 25, 2022 · 0 comments
Closed

Fix 'noSuchProperty' test case. #2418

gregturn opened this issue Jan 25, 2022 · 0 comments
Assignees
@gregturn
Labels
in: repository Repositories abstraction type: regression A regression from a previous release
Milestone
2.7 M3 (2021.2.0)

Comments

@gregturn
Copy link
Contributor

The error message for nonexistent properties inside query derivation now wraps the named property with single quotes. This breaks an existing test case, which was looking for spaces before and after.
@gregturn gregturn added type: regression A regression from a previous release in: repository Repositories abstraction labels Jan 25, 2022
@gregturn gregturn added this to the 2.7 M3 (2021.2.0) milestone Jan 25, 2022
@gregturn gregturn self-assigned this Jan 25, 2022
gregturn added a commit that referenced this issue Jan 25, 2022
@gregturn
GH-2418 - Prepare branch
0f42719
gregturn added a commit that referenced this issue Jan 25, 2022
@gregturn
Fix 'noSuchProperty' test case.
fc93a6a 
The error message for nonexistent properties inside query derivation now wraps the named property with single quotes. This breaks a test case that was looking for spaces before and after.

Closes #2418.
gregturn added a commit that referenced this issue Jan 25, 2022
@gregturn
Fix 'noSuchProperty' test case.
47f36f6 
The error message for nonexistent properties inside query derivation now wraps the named property with single quotes. This breaks a test case that was looking for spaces before and after.

Backported to 2.6.x.

See #2418.
gregturn added a commit that referenced this issue Jan 25, 2022
@gregturn
Fix 'noSuchProperty' test case.
881bfbd 
The error message for nonexistent properties inside query derivation now wraps the named property with single quotes. This breaks a test case that was looking for spaces before and after.

Forward ported to 3.0.x

See #2418.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gregturn gregturn

Labels
in: repository Repositories abstraction type: regression A regression from a previous release
Projects
None yet
Milestone
2.7 M3 (2021.2.0)
Development

No branches or pull requests
1 participant
@gregturn