Compile and run tests in Claude cloud environment #3644
Open
luis100 wants to merge 55 commits intodevelopmentfrom
Open
Compile and run tests in Claude cloud environment #3644luis100 wants to merge 55 commits intodevelopmentfrom
luis100 wants to merge 55 commits intodevelopmentfrom
Conversation
…rolledVocabulary dependency (#3592)
…le when the mime type is not defined
…k logic during plugin execution
…eleteAIPPermissionTest. (#3596)
…wt-jackson doesn't like it
Bumps [docker/build-push-action](https://github.com/docker/build-push-action) from 6 to 7. - [Release notes](https://github.com/docker/build-push-action/releases) - [Commits](docker/build-push-action@v6...v7) --- updated-dependencies: - dependency-name: docker/build-push-action dependency-version: '7' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [docker/metadata-action](https://github.com/docker/metadata-action) from 5 to 6. - [Release notes](https://github.com/docker/metadata-action/releases) - [Commits](docker/metadata-action@v5...v6) --- updated-dependencies: - dependency-name: docker/metadata-action dependency-version: '6' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [docker/login-action](https://github.com/docker/login-action) from 3 to 4. - [Release notes](https://github.com/docker/login-action/releases) - [Commits](docker/login-action@v3...v4) --- updated-dependencies: - dependency-name: docker/login-action dependency-version: '4' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Miguel Guimarães <mguimaraes@keep.pt>
…3590) Snyk has created this PR to upgrade org.apache.pekko:pekko-cluster_2.13 from 1.1.4 to 1.4.0. See this package in maven: org.apache.pekko:pekko-cluster_2.13 See this project in Snyk: https://app.snyk.io/org/luis100/project/d4f34799-92b4-4ac4-9760-afd88ae499a6?utm_source=github&utm_medium=referral&page=upgrade-pr Co-authored-by: snyk-bot <snyk-bot@snyk.io>
Documents codebase structure, build workflow, test setup, architecture, code style conventions, CI/CD, release process, and critical gotchas for AI coding assistants working in this repository. https://claude.ai/code/session_01DKVBL81YZps5hRHFjCPMG6
…RE4B Add httpcore dependency
Previously, tests required a running stack of Docker services (ZooKeeper, Solr, PostgreSQL, LDAP, MailHog) either via docker-compose or pre-started GitHub Actions service containers. LDAP was already managed via Testcontainers; this change extends the same pattern to the remaining services. Key changes: - TestContainersManager: singleton that starts ZooKeeper, Solr (Cloud mode), and PostgreSQL containers once per test suite run, then sets the system properties that ConfigurationManager and Spring Boot read for connection URLs. - RodaContainersLifecycleListener: TestNG ISuiteListener registered in testng.xml so containers start (and system properties are set) before any test class is instantiated or any Spring context is initialised. - ConfigurationManager.getConfigurationString: now checks System.getProperty as a fallback between the OS environment variable and the config file, allowing test helpers to override settings without forking the JVM. - CI.yml: removes the ZooKeeper, Solr, PostgreSQL, and MailHog service container definitions and their corresponding environment variable overrides; the test suite now self-provisions all required infrastructure via Testcontainers. https://claude.ai/code/session_01KuXDU1UWFJrTiZWK2hqPVj
Creates ~/.m2/settings.xml using GITHUB_MAVEN_USER and GITHUB_MAVEN_PASSWORD for GitHub Packages authentication, needed before running Maven builds. https://claude.ai/code/session_01C16yKTzMvrJbwSNTVAWLRK
…ZEa1 Add script to configure Maven settings from environment variables
…ries, and refactor DefaultModelService for hybrid persistence
) The following vulnerabilities are fixed with an upgrade: - https://snyk.io/vuln/SNYK-JAVA-ORGSPRINGFRAMEWORK-15701755 - https://snyk.io/vuln/SNYK-JAVA-ORGSPRINGFRAMEWORK-15701756 - https://snyk.io/vuln/SNYK-JAVA-ORGSPRINGFRAMEWORK-15701845 - https://snyk.io/vuln/SNYK-JAVA-COMNIMBUSDS-10691768 - https://snyk.io/vuln/SNYK-JAVA-ORGBOUNCYCASTLE-8731360 - https://snyk.io/vuln/SNYK-JAVA-ORGBOUNCYCASTLE-11777856 - https://snyk.io/vuln/SNYK-JAVA-ORGBOUNCYCASTLE-11789705 - https://snyk.io/vuln/SNYK-JAVA-ORGBOUNCYCASTLE-6084022 - https://snyk.io/vuln/SNYK-JAVA-ORGBOUNCYCASTLE-6613080 - https://snyk.io/vuln/SNYK-JAVA-ORGBOUNCYCASTLE-5771339 Co-authored-by: snyk-bot <snyk-bot@snyk.io>
…aven repository from pom list of repositories
The default zkConnectTimeout of 15s in SolrJ causes SolrZkClient to call ZooKeeper.close() if the ZK session isn't established in time. That close() sends a CLOSESESSION packet then waits indefinitely in Object.wait() with no timeout, because the ZK background threads are already gone — hanging the test JVM forever. Changes: - RodaCoreFactory.instantiateSolr(): add withZkConnectTimeout(300s) to the CloudSolrClient.Builder so SolrZkClient has enough time to establish the ZK session before panicking - roda-core.properties: raise connect timeout to 300s, document zk.client.timeout_ms - TestContainersManager: set zkConnectTimeout and RODA_CORE_SOLR_CLOUD_ZK_CONNECT_TIMEOUT_MS system properties as belt-and-suspenders fallback - roda-core-tests/pom.xml: add 172.*/192.* to http.nonProxyHosts in surefire argLine so Docker container IPs bypass the HTTPS proxy - testng-single.xml: single-class TestNG suite file for fast targeted test runs - CLAUDE.md: document cloud environment workflow (Docker startup, build commands, single-test shortcut, proxy quirks, ZK/Solr notes, pre-PR checklist) Verified: IndexServiceTest 17/17 tests pass with this fix. https://claude.ai/code/session_01XRhDcKRQtCBua46PGom5dx
dosubot
bot
added
the
size:XXL
luis100
changed the title
dosubot
bot
added
size:L
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
8 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.