Skip to content

speedup tests and reduce log spam #89

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.

Sign up for GitHub

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

Jump to bottom
Merged
ddanielr merged 3 commits into from
Aug 22, 2025
Merged

speedup tests and reduce log spam #89

ddanielr merged 3 commits into from
Aug 22, 2025

Conversation

@ddanielr
Copy link
Contributor

@ddanielr ddanielr commented Aug 22, 2025

Redirects test output to log files.

Collapses the invalid auth tests into a single test method which reduced the test runtime by ~30s for each Protocol IT test.

Before:

[INFO] -------------------------------------------------------
[INFO]  T E S T S
[INFO] -------------------------------------------------------
[INFO] Running org.apache.accumulo.proxy.its.TBinaryProxyIT
[INFO] Tests run: 123, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 162.0 s -- in org.apache.accumulo.proxy.its.TBinaryProxyIT
[INFO] Running org.apache.accumulo.proxy.its.ProxyDurabilityIT
[INFO] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 114.1 s -- in org.apache.accumulo.proxy.its.ProxyDurabilityIT
[INFO] Running org.apache.accumulo.proxy.its.TCompactProxyIT
[INFO] Tests run: 123, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 161.1 s -- in org.apache.accumulo.proxy.its.TCompactProxyIT
[INFO] Running org.apache.accumulo.proxy.its.TJsonProtocolProxyIT
[INFO] Tests run: 123, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 172.4 s -- in org.apache.accumulo.proxy.its.TJsonProtocolProxyIT
[INFO] Running org.apache.accumulo.proxy.its.TTupleProxyIT
[INFO] Tests run: 123, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 165.2 s -- in org.apache.accumulo.proxy.its.TTupleProxyIT
[INFO] 
[INFO] Results:
[INFO] 
[INFO] Tests run: 493, Failures: 0, Errors: 0, Skipped: 0

After:

INFO] -------------------------------------------------------
[INFO]  T E S T S
[INFO] -------------------------------------------------------
[INFO] Running org.apache.accumulo.proxy.its.TBinaryProxyIT
[INFO] Tests run: 38, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 135.8 s -- in org.apache.accumulo.proxy.its.TBinaryProxyIT
[INFO] Running org.apache.accumulo.proxy.its.ProxyDurabilityIT
[INFO] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 112.9 s -- in org.apache.accumulo.proxy.its.ProxyDurabilityIT
[INFO] Running org.apache.accumulo.proxy.its.TCompactProxyIT
[INFO] Tests run: 38, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 136.5 s -- in org.apache.accumulo.proxy.its.TCompactProxyIT
[INFO] Running org.apache.accumulo.proxy.its.TJsonProtocolProxyIT
[INFO] Tests run: 38, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 131.6 s -- in org.apache.accumulo.proxy.its.TJsonProtocolProxyIT
[INFO] Running org.apache.accumulo.proxy.its.TTupleProxyIT
[INFO] Tests run: 38, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 133.1 s -- in org.apache.accumulo.proxy.its.TTupleProxyIT

@ddanielr
Redirect cluster logs to files to reduce output
223c4b5 
When the tests are run, the exception messages from the proxy server
are sent to STDOUT. This pollutes the output and makes it harder to
read the test output.

This change redirects the server messages to a file and keeps the output
clean.

Also replaced a deprecated property with its newer version to remove a
build warning message.
@ddanielr
Collapsed tests to speed up runtime.
fb97355 
These single tests all trigger the BeforeEach method which
creates new clients, tables, and namespaces. They also trigger the
AfterEach method which deletes the table and namespace and also closes
the client.

This commit collapses the invalid auth tests into a single test case so
the table creation and deletion is not triggered for each auth attempt.
@ddanielr ddanielr changed the title Feature/speedup tests reduce log spam speedup tests and reduce log spam Aug 22, 2025
@ddanielr ddanielr merged commit 721a752 into main Aug 22, 2025
2 checks passed
@ddanielr ddanielr deleted the feature/speedup-tests-reduce-log-spam branch August 22, 2025 17:02
@ddanielr ddanielr linked an issue Sep 17, 2025 that may be closed by this pull request
2.0.1 Update causes Security Exceptions #31
Closed
@ddanielr ddanielr mentioned this pull request Sep 17, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@keith-turner keith-turner keith-turner approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2.0.1 Update causes Security Exceptions

2 participants

@ddanielr @keith-turner