Skip to content

feat: Adapt client and integration tests to a KinD based CAPI - #1384

Draft
ZPascal wants to merge 3 commits into
cloudfoundry:mainfrom
ZPascal:integration-test-adaptions-main
Draft

ZPascal wants to merge 3 commits into
cloudfoundry:mainfrom
ZPascal:integration-test-adaptions-main

Conversation

@ZPascal

@ZPascal ZPascal commented Oct 10, 2026 •

Copy link
Copy Markdown

This is what it took to get the integration tests running against a CAPI deployed on KinD. It also bumps the supported CAPI version, which fixes #1370.

Client changes:

  • SUPPORTED_API_VERSION goes from 2.272.0 to 2.290.0. SUPPORTED_API_VERSION_V3 is already gone on main, so I didn't bring it back.
  • The v3 Space gets the suspended field.
  • v3 package bits uploads now always send a resources part, even an empty one. CAPI rejects the upload otherwise with CF-AppBitsUploadInvalid.
  • v2 app uploads without resources now also pass resources=[] as a query parameter. The nginx upload module can drop the form field, and the request then fails with "missing :resources". Only an empty list fits into a URL, so that is the only case covered.
  • The LDAP and SAML identity provider configurations get caCertificates.
  • The UAA authorization calls now send an Accept: application/x-www-form-urlencoded header and drop the authorization header where it doesn't belong (the earlier Fix UAA Authorizations accept headers #1379 commit is part of this branch). The openid-configuration lookup also no longer sends the authorization header.
  • Small one in pom.xml: the Reactor Javadoc link now points to javadoc.io, since the old one no longer resolves.

Integration tests:

  • At startup, the tests now reserve the TCP ports that NameFactory hands out on the default-tcp router group. The TCP route tests then no longer depend on test order or on the default port range of the deployment. This is skipped together with the other TCP routing tests (SKIP_TCP_ROUTING_TESTS). The new bean uses its own routing client, because the shared one depends on the cleaner, which in turn depends on this bean.

Testing

Connected PR

@ZPascal ZPascal changed the title Integration test adaptions main Adapt client and integration tests to KinD based CAPI Oct 10, 2026
Make the integration tests pass against a CAPI deployed on KinD.

Client:
- Send the "resources" field with every bits upload (v3 packages) and
  also as query parameter for v2 uploads without matched resources, as
  CAPI otherwise rejects them (CF-AppBitsUploadInvalid / missing
  :resources).
- Add the CA certificates of the LDAP and SAML identity provider
  configurations.

Integration tests:
- Reserve the TCP ports handed out by the NameFactory on the default-tcp
  router group at startup, so the TCP route tests don't depend on the
  test order or the deployment's default range. Skipped together with
  the TCP routing tests (SKIP_TCP_ROUTING_TESTS).

Javadoc
- Fix the javadoc link to Reactor Core API
@ZPascal
ZPascal force-pushed the integration-test-adaptions-main branch from efb4300 to 619ff37 Compare October 10, 2026 09:44
@ZPascal ZPascal changed the title Adapt client and integration tests to KinD based CAPI Adapt client and integration tests to a KinD based CAPI Oct 10, 2026
@ZPascal ZPascal changed the title Adapt client and integration tests to a KinD based CAPI feat: Adapt client and integration tests to a KinD based CAPI Oct 10, 2026

This branch has not been deployed

No deployments
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.

Update to latest CAPI Version 1.240.0

2 participants