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

test: allow fetching CRD from a local replace statement #387

Merged
merged 1 commit into from
Nov 14, 2023

Conversation

dciabrin
Copy link
Contributor

@dciabrin dciabrin commented Nov 9, 2023

Ginkgo test suites require fetching the CRDs of the custom resources used in the tests. Those are fetched from the go.mod and any replace statement is parsed and honoured.
Add the ability to process replace statements that point to a local directory as well, to ease the developement process when a project depends on a feature from another project that has not been merged yet.

@dciabrin dciabrin requested review from stuggi and gibizer November 9, 2023 17:28
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you please extend the test coverage of this in https://github.com/openstack-k8s-operators/lib-common/blob/main/modules/test/crd_test.go with the new path?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done now.

Ginkgo test suites require fetching the CRDs of the custom resources
used in the tests. Those are fetched from the go.mod and any replace
statement is parsed and honoured.
Add the ability to process replace statements that point to a local
directory as well, to ease the developement process when a project
depends on a feature from another project that has not been merged yet.
Copy link
Contributor

@gibizer gibizer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@gibizer gibizer merged commit 65eb1b1 into openstack-k8s-operators:main Nov 14, 2023
2 checks passed
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.

3 participants