Skip to content

Enhancing test suite logging for FIX messages#1238

Merged
chrjohn merged 1 commit into
masterfrom
copilot/add-logging-for-failure-cases
May 22, 2026
Merged

Enhancing test suite logging for FIX messages#1238
chrjohn merged 1 commit into
masterfrom
copilot/add-logging-for-failure-cases

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 22, 2026

Required to see messages on acceptance test failures.

@chrjohn
Copy link
Copy Markdown
Member

chrjohn commented May 22, 2026

@copilot wouldn't it be sufficient to just set quickfix.test.acceptance.level=FINE and leave the general logging at INFO level?

Copy link
Copy Markdown
Contributor Author

Copilot AI commented May 22, 2026

@copilot wouldn't it be sufficient to just set quickfix.test.acceptance.level=FINE and leave the general logging at INFO level?

Not quite with java.util.logging: if java.util.logging.ConsoleHandler.level stays at INFO, it will still drop FINE records even when quickfix.test.acceptance.level=FINE. The global logger level is still INFO; I only lowered the handler threshold so acceptance FINE logs can actually be emitted (commit 07842c9).

Copilot AI requested a review from chrjohn May 22, 2026 17:33
@chrjohn chrjohn added this to the QFJ 3.0.2 milestone May 22, 2026
@chrjohn chrjohn marked this pull request as ready for review May 22, 2026 17:35
@chrjohn chrjohn merged commit 700b798 into master May 22, 2026
22 checks passed
@chrjohn chrjohn deleted the copilot/add-logging-for-failure-cases branch May 22, 2026 17:35
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