Skip to content

Comments

Reduce logging for grid map cache population in case of errors.#1148

Merged
umeetiusbaar merged 1 commit intodevfrom
fix-excessive-test-logging
Oct 21, 2025
Merged

Reduce logging for grid map cache population in case of errors.#1148
umeetiusbaar merged 1 commit intodevfrom
fix-excessive-test-logging

Conversation

@umeetiusbaar
Copy link
Contributor

Description

Reduce logging for grid map cache population in case of errors. This created way too much logging in tests and failed the test result publishing in GitHub Actions.

Jira Issue:

Type of change

  • Bug fix
  • New feature
  • Other

Instructions for testing

Please describe tests how this change or new feature can be tested.

Checklist:

  • I have written new tests (if applicable)
  • I have ran the tests myself (if applicable)
  • I have made necessary changes to the documentation, link to confluence
    or other location:

Other relevant info

Please describe here if there is e.g. some requirements for this change or
other info that the tester/user needs to know.

…created way too much logging in tests and failed the test result publishing in GitHub Actions.
@github-actions
Copy link

Test Results

  422 files  +  259    422 suites  +260   2m 58s ⏱️ + 2m 32s
2 610 tests +1 434  2 610 ✅ +1 434  0 💤 ±0  0 ❌ ±0 
2 867 runs  +1 602  2 867 ✅ +1 602  0 💤 ±0  0 ❌ ±0 

Results for commit 9c59cda. ± Comparison against base commit f4b1481.

@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 65%)

See analysis details on SonarQube Cloud

@terovirtanen
Copy link

HAITATON-HANKE-SERVICE branch is deployed to platta: https://haitaton-hanke-service-pr1148.dev.hel.ninja 🚀🚀🚀

@umeetiusbaar umeetiusbaar merged commit f7d17ab into dev Oct 21, 2025
19 of 20 checks passed
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