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

Add getRelationship[s] coverage to apiComplianceSuite #914

Closed
foundrytom opened this issue May 3, 2023 · 0 comments · Fixed by #952
Closed

Add getRelationship[s] coverage to apiComplianceSuite #914

foundrytom opened this issue May 3, 2023 · 0 comments · Fixed by #952
Assignees
@foundrytom
Milestone
v1.0.0-beta.1.0

Comments

@foundrytom
Copy link
Collaborator

No description provided.

@foundrytom foundrytom added this to the Core stable milestone May 3, 2023
@foundrytom foundrytom self-assigned this May 22, 2023
foundrytom added a commit to foundrytom/OpenAssetIO that referenced this issue Jun 5, 2023
@foundrytom
[Core] Add apiComplianceSuite coverage for getWithRelationship[s]
1caa67b 
Adds basic coverage for the relationship query methods. Note: this has
deliberately been left in it's somewhat long-winded form, as it hints
that we may well want to look at a bit of a refactor of the harness, now
its being used at scale.

Closes OpenAssetIO#914

Signed-off-by: Tom Cowland <tom@foundry.com>
foundrytom added a commit to foundrytom/OpenAssetIO that referenced this issue Jun 5, 2023
@foundrytom
[Core] Add apiComplianceSuite coverage for getWithRelationship[s]
181d3ca 
Adds basic coverage for the relationship query methods. Note: this has
deliberately been left in it's somewhat long-winded form, as it hints
that we may well want to look at a bit of a refactor of the harness, now
its being used at scale.

Closes OpenAssetIO#914

Signed-off-by: Tom Cowland <tom@foundry.com>
@foundrytom foundrytom mentioned this issue Jun 5, 2023
Merged
foundrytom added a commit to foundrytom/OpenAssetIO that referenced this issue Jun 20, 2023
@foundrytom
[Core] Add apiComplianceSuite coverage for getWithRelationship[s]
3f0dc96 
Adds basic coverage for the relationship query methods. Note: this has
deliberately been left in it's somewhat long-winded form, as it hints
that we may well want to look at a bit of a refactor of the harness, now
its being used at scale.

Closes OpenAssetIO#914

Signed-off-by: Tom Cowland <tom@foundry.com>
feltech added a commit to foundrytom/OpenAssetIO that referenced this issue Jun 20, 2023
@feltech
[CI] Re-enable BAL e2e tests
8c180f2 
Closes OpenAssetIO#914. Possible now that OpenAssetIO/OpenAssetIO-Manager-BAL#47
has updated BAL to use the new signatures for `getWithRelationship[s]`

Signed-off-by: David Feltell <david.feltell@foundry.com>
@feltech feltech mentioned this issue Jun 20, 2023
Merged
feltech added a commit to foundrytom/OpenAssetIO that referenced this issue Jun 22, 2023
@feltech
[Core][Python] apiComplianceSuite entity ref utils
d748612 
In support of OpenAssetIO#914 (and beyond), add a couple of useful utility methods
for test cases in the `apiComplianceSuite` test harness.

Specifically, `requireEntityReferenceFixture` and
`requireEntityReferencesFixture` convert a string fixture into an
`EntityReference` object, or a list-of-strings fixture into a list of
`EntityReference` objects, respectively.

Signed-off-by: David Feltell <david.feltell@foundry.com>
feltech pushed a commit to foundrytom/OpenAssetIO that referenced this issue Jun 22, 2023
@foundrytom @feltech
[Core] Add apiComplianceSuite coverage for getWithRelationship[s]
43f339d 
Closes OpenAssetIO#914

Adds basic coverage for the relationship query methods.

Rather than getting tripped up by coming up with good unambiguous names
for fixtures, re-use similar names by namespacing the fixtures under
their test name, i.e. rather than using `"shared"` fixtures.

Signed-off-by: David Feltell <david.feltell@foundry.com>
feltech added a commit to foundrytom/OpenAssetIO that referenced this issue Jun 22, 2023
@feltech
[Core][Python] apiComplianceSuite entity ref utils
529df33 
In support of OpenAssetIO#914 (and beyond), add a couple of useful utility methods
for test cases in the `apiComplianceSuite` test harness.

Specifically, `requireEntityReferenceFixture` and
`requireEntityReferencesFixture` convert a string fixture into an
`EntityReference` object, or a list-of-strings fixture into a list of
`EntityReference` objects, respectively.

Signed-off-by: David Feltell <david.feltell@foundry.com>
feltech pushed a commit to foundrytom/OpenAssetIO that referenced this issue Jun 22, 2023
@foundrytom @feltech
[Core] Add apiComplianceSuite coverage for getWithRelationship[s]
d789472 
Closes OpenAssetIO#914

Adds basic coverage for the relationship query methods.

Rather than getting tripped up by coming up with good unambiguous names
for fixtures, re-use similar names by namespacing the fixtures under
their test name, i.e. rather than using `"shared"` fixtures.

Signed-off-by: David Feltell <david.feltell@foundry.com>
feltech pushed a commit to foundrytom/OpenAssetIO that referenced this issue Jun 26, 2023
@foundrytom @feltech
[Core] Add apiComplianceSuite coverage for getWithRelationship[s]
80ec3ec 
Closes OpenAssetIO#914

Adds basic coverage for the relationship query methods.

Rather than getting tripped up by coming up with good unambiguous names
for fixtures, re-use similar names by namespacing the fixtures under
their test name, i.e. rather than using `"shared"` fixtures.

Signed-off-by: David Feltell <david.feltell@foundry.com>
feltech added a commit to foundrytom/OpenAssetIO that referenced this issue Jun 26, 2023
@feltech
[Core][Python] apiComplianceSuite entity ref utils
f53f647 
In support of OpenAssetIO#914 (and beyond), add a couple of useful utility methods
for test cases in the `apiComplianceSuite` test harness.

Specifically, `requireEntityReferenceFixture` and
`requireEntityReferencesFixture` convert a string fixture into an
`EntityReference` object, or a list-of-strings fixture into a list of
`EntityReference` objects, respectively.

Signed-off-by: David Feltell <david.feltell@foundry.com>
feltech pushed a commit to foundrytom/OpenAssetIO that referenced this issue Jun 26, 2023
@foundrytom @feltech
[Core] Add apiComplianceSuite coverage for getWithRelationship[s]
20f2302 
Closes OpenAssetIO#914

Adds basic coverage for the relationship query methods.

Rather than getting tripped up by coming up with good unambiguous names
for fixtures, re-use similar names by namespacing the fixtures under
their test name, i.e. rather than using `"shared"` fixtures.

Signed-off-by: David Feltell <david.feltell@foundry.com>
feltech pushed a commit to foundrytom/OpenAssetIO that referenced this issue Jun 26, 2023
@foundrytom @feltech
[Core] Add apiComplianceSuite coverage for getWithRelationship[s]
073220f 
Closes OpenAssetIO#914

Adds basic coverage for the relationship query methods.

Rather than getting tripped up by coming up with good unambiguous names
for fixtures, re-use similar names by namespacing the fixtures under
their test name, i.e. rather than using `"shared"` fixtures.

Signed-off-by: David Feltell <david.feltell@foundry.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@foundrytom foundrytom

Labels
None yet
Projects
OpenAssetIO Development
Status: Done
Milestone
v1.0.0-beta.1.0
Development

Successfully merging a pull request may close this issue.

[Core] Add apiComplianceSuite coverage for getWithRelationship[s] foundrytom/OpenAssetIO
1 participant
@foundrytom