[tests] suppress debug logs in CI environment + improve calendar symbol test stability #3941
Conversation
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
Set LOG_LEVEL=ERROR in vitest-setup.js when running in CI to reduce verbose console output while keeping full logging for local development.
|
The failing test looks like a Timing-Flake 🤔 I'm looking into it. |
Replace external calendarlabs.com URL with local mock file to improve test stability and CI reliability. External URLs can cause timeouts and introduce non-deterministic failures unrelated to the tested functionality. The test still validates the same symbol array feature but now runs faster and more reliably without external dependencies.
KristjanESPERANTO
changed the title
|
Test works now -> PR description updated -> still a lot of logging -> I'm investigating |
|
Hold on. I'll take a look at the expected error messages so that we might get a clean output after all 🤓 |
|
We now have clean output - the expected error messages are no longer displayed in test mode 🚀 |
khassel
approved these changes
Nov 3, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 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.
CI Log Suppression
Two-level approach for clean test output:
logger.setLogLevel("ERROR")in CI to hide verbose loggingmmTestModeflag and check it before logging errors that are part of test assertions (e.g., testing error handling ingit_helper.jsandserver_functions.js)This keeps CI output clean and makes genuine failures immediately visible, while preserving full logging for local development.
Before: 1348 log lines with verbose debug/info output
After: 168 log clean lines with only test results
Calendar Symbol Test Stability
Convert the calendar symbol test from external URL (
calendarlabs.com) to existing local mock file (12_events.ics). This eliminates CI timeouts caused by external dependencies and improves test reliability.The test still validates the same symbol array feature but now runs faster and deterministically without network dependencies.