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.

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

Query/Test: add relationship test model which only contains references (no collections) #35707

Open
maumar opened this issue Mar 1, 2025 · 2 comments · May be fixed by #35711 or #35709
Open

Query/Test: add relationship test model which only contains references (no collections) #35707

maumar opened this issue Mar 1, 2025 · 2 comments · May be fixed by #35711 or #35709
Assignees
@maumar
Labels
area-query area-test closed-fixed The issue has been fixed and is/will be included in the release indicated by the issue milestone. type-enhancement
Milestone
10.0.0

Comments

@maumar
Copy link
Contributor

maumar commented Mar 1, 2025

So we can test scenarios which don't support collections (e.g. non-json complex types)
@maumar maumar self-assigned this Mar 1, 2025
@maumar maumar changed the title Test: add relationship test model which only contains references (no collections) Query/Test: add relationship test model which only contains references (no collections) Mar 1, 2025
@roji
Copy link
Member

roji commented Mar 1, 2025

Noting that this doesn't have to be a fully separate model - just move the reference-only tests to a different suite and ignore the collection references, or similar (whatever's simplest basically).

@maumar
Copy link
Contributor Author

maumar commented Mar 1, 2025

yeah that was my thinking as well

maumar added a commit that referenced this issue Mar 2, 2025
@maumar
Navigation test refactoring - improving structure for scenarios that …
Loading
Loading status checks…
d268786 
…don't support collections

Fixes #35707
@maumar maumar linked a pull request Mar 2, 2025 that will close this issue
Draft
maumar added a commit that referenced this issue Mar 2, 2025
@maumar
Navigation test refactoring - improving structure for scenarios that …
Loading
Loading status checks…
e829e59 
…don't support collections

Plus minor cleanup

Fixes #35707
@maumar maumar linked a pull request Mar 2, 2025 that will close this issue
Open
maumar added a commit that referenced this issue Mar 2, 2025
@maumar
Navigation test refactoring - improving structure for scenarios that …
Loading
Loading status checks…
6dda505 
…don't support collections

Plus minor cleanup

Fixes #35707
maumar added a commit that referenced this issue Mar 2, 2025
@maumar
Navigation test refactoring - improving structure for scenarios that …
Loading
Loading status checks…
754bfe8 
…don't support collections

Plus minor cleanup

Fixes #35707
maumar added a commit that referenced this issue Mar 2, 2025
@maumar
Navigation test refactoring - improving structure for scenarios that …
Loading
Loading status checks…
fdaacb0 
…don't support collections

Plus minor cleanup

Fixes #35707
@maumar maumar added the closed-fixed The issue has been fixed and is/will be included in the release indicated by the issue milestone. label Mar 3, 2025
@maumar maumar added this to the 10.0.0 milestone Mar 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@maumar maumar

Labels
area-query area-test closed-fixed The issue has been fixed and is/will be included in the release indicated by the issue milestone. type-enhancement
Projects
None yet
Milestone
10.0.0
2 participants
@maumar @roji