adityamparikh opened a new pull request, #198:
URL: https://github.com/apache/solr-mcp/pull/198
### Summary
Removes the class-level `@Disabled` from `OtlpExportIntegrationTest`, which
had made every build report 7 skipped tests since observability landed in #41.
### Why it was disabled, and why that no longer applies
The annotation cited a `ClassNotFoundException` for
`org.eclipse.jetty.client.transport.HttpClientTransportOverHTTP` when starting
the Grafana LGTM Testcontainer. The Jetty pin has moved since (SolrJ 10 bump);
`jetty-client` 12.0.34 is on the test runtime classpath and contains that
class. With the annotation gone the container starts and the OTLP wiring works.
### The one real failure, and the fix
Three methods then failed with `AuthenticationCredentialsNotFoundException`.
The test calls `@PreAuthorize("isAuthenticated()")` service methods directly
under the `http` profile, so there is no MCP request and no principal. Instead
of setting `http.security.enabled=false` (as `DistributedTracingTest` does),
the test authenticates its own thread with a `TestingAuthenticationToken` in
`@BeforeEach` and clears it in `@AfterEach`. Secure-by-default and method
security stay on; OTLP export is verified against the real LGTM stack.
### Verification
`./gradlew build` on Java 25: 407 tests, 0 failures, **0 skipped**.
`OtlpExportIntegrationTest` 7/7 against `grafana/otel-lgtm:latest`.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
https://claude.ai/code/session_019zJ9WA8nNyA5Yxb8ueM7vV
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
