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

[Bug] Unable to run tests in the pulsar-broker-auth-sasl module #22664

Closed
3 tasks done
BewareMyPower opened this issue May 7, 2024 · 0 comments · Fixed by #22665
Closed
3 tasks done

[Bug] Unable to run tests in the pulsar-broker-auth-sasl module #22664

BewareMyPower opened this issue May 7, 2024 · 0 comments · Fixed by #22665
Labels
type/bug The PR fixed a bug or issue reported a bug

Comments

@BewareMyPower
Copy link
Contributor

Search before asking

  • I searched in the issues and found nothing similar.

Read release policy

  • I understand that unsupported versions don't get bug fixes. I will attempt to reproduce the issue on a supported version of Pulsar client and Pulsar broker.

Version

  • OS: macOS Ventura 13.6.3
  • Java: 17.0.7
  • Pulsar: master (a7e1fcd)

Minimal reproduce step

Run ProxySaslAuthenticationTest in Intellij Idea or just trivial maven command:

mvn test -pl pulsar-broker-auth-sasl -Dtest='ProxySaslAuthenticationTest'

What did you expect to see?

The test should run.

What did you see instead?

When running tests in Intellij, it said unable to resolve class reference io/opentelemetry/sdk/testing/assertj/LongSumAssert.

2024-05-07T19:45:04,135 - INFO  - [main:Logger@112] - [[TestNGClassFinder]] Unable to read methods on class org.apache.pulsar.broker.authentication.ProxySaslAuthenticationTest - unable to resolve class reference io/opentelemetry/sdk/testing/assertj/LongSumAssert
2024-05-07T19:45:04,151 - INFO  - [main:Logger@112] - [TestNG] Running:
  /Users/xuyunze/Library/Caches/JetBrains/IdeaIC2024.1/temp-testng-customsuite.xml

When running the mvn test command, no tests are run.

[INFO] Running org.apache.pulsar.broker.authentication.ProxySaslAuthenticationTest
[INFO] Tests run: 0, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.487 s - in org.apache.pulsar.broker.authentication.ProxySaslAuthenticationTest

Anything else?

It seems that this module missed the opentelemetry-sdk-testing dependency.

Are you willing to submit a PR?

  • I'm willing to submit a PR!
@BewareMyPower BewareMyPower added the type/bug The PR fixed a bug or issue reported a bug label May 7, 2024
BewareMyPower added a commit to BewareMyPower/pulsar that referenced this issue May 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/bug The PR fixed a bug or issue reported a bug
Projects
None yet
1 participant