Skip to content

Integration test fixes#6980

Open
ahoffer wants to merge 1 commit into2.29.xfrom
itest-edits
Open

Integration test fixes#6980
ahoffer wants to merge 1 commit into2.29.xfrom
itest-edits

Conversation

@ahoffer
Copy link
Contributor

@ahoffer ahoffer commented Mar 4, 2026

What does this PR do?

Patch up how itests are run. DDF integration tests pass locally.
Builds off of Zac's upgrade-karaf PR

Who is reviewing it?

@zkirksey
@clockard
@jrnorth
@jaymcnallie

Select relevant component teams:

@codice/test

Ask 2 committers to review/merge the PR and tag them here.

@clockard
@jrnorth

How should this be tested?

Clone, build, and run integration tests

Any background context you want to provide?

What are the relevant tickets?

N/A

Screenshots

None

Checklist:

  • [NO] Documentation Updated
  • [NO] Update / Add Threat Dragon models
  • [NO ] Update / Add Unit Tests
  • Update / Add Integration Tests

Notes on Review Process

No notes

Review Comment Legend:

  • ✏️ (Pencil) This comment is a nitpick or style suggestion, no action required for approval. This comment should provide a suggestion either as an in line code snippet or a gist.
  • ❓ (Question Mark) This comment is to gain a clearer understanding of design or code choices, clarification is required but action may not be necessary for approval.
  • ❗ (Exclamation Mark) This comment is critical and requires clarification or action before approval.

@ahoffer ahoffer requested a review from stustison as a code owner March 4, 2026 21:40
@zkirksey
Copy link
Collaborator

zkirksey commented Mar 4, 2026

Looks like we need to update the branches covered ratio in catalog-solr-solrclient:
image

Other than this error, I was able to get the full build to pass with tests (including itests).

@zkirksey zkirksey changed the base branch from upgrade-karaf-version to 2.29.x March 5, 2026 00:23
@zkirksey
Copy link
Collaborator

zkirksey commented Mar 5, 2026

Formatting error:

image

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.

2 participants