Skip to content

Logs for debugging flaky test & rerun of failing container in specific order - #3948

Closed
jonbulz wants to merge 3 commits into
developfrom
debug-test-poi-category-form-view
Closed

Logs for debugging flaky test & rerun of failing container in specific order#3948
jonbulz wants to merge 3 commits into
developfrom
debug-test-poi-category-form-view

Conversation

@jonbulz

@jonbulz jonbulz commented Oct 15, 2025

Copy link
Copy Markdown
Contributor

Short description

  • Add logs in poi_category_form_view and test_poi_category_form_view to debug flaky test_no_changes_were_made_message. The intention is to remove the log statements before the next release
  • Set up the test job inside config.yml so that we can run a container that contains exactly the tests in the exact order of when this container failed before

Proposed changes

  • debug logs in poi_category_form_view to see if the message is created

  • debug logs in test_poi_category_form_view to see the rendered html

  • prettify response content to make the html readable in the logs

  • log handler for tests

  • WIP: find a way to have specific tests run in a specific order

Side effects

  • probably clutters our logs if this is ever deployed

Faithfulness to issue description and design

There are no intended deviations from the issue and design.

How to test

Resolved issues

Fixes: #


Pull Request Review Guidelines

@jonbulz
jonbulz requested a review from hannaseithe October 15, 2025 16:12

@JoeyStk JoeyStk left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you very much! LGTM! 🚀

@hannaseithe
hannaseithe force-pushed the debug-test-poi-category-form-view branch 3 times, most recently from ca4b687 to 1f19d77 Compare October 17, 2025 08:34
@hannaseithe
hannaseithe force-pushed the debug-test-poi-category-form-view branch from 1f19d77 to ca1b028 Compare October 17, 2025 09:36
@hannaseithe

Copy link
Copy Markdown
Contributor

I might have been able to fix the failing tests with #3950. But I want to use this branch still for figuring out a way to run specific tests inside a container with a specific order (so that we can force a fail that depends on the order of a container). Therefore I will adjst the description and keep the PR as draft

@hannaseithe hannaseithe changed the title logs for debugging test_poi_category_form_view logs for debugging test_poi_category_form_view & rerun of failing container in specific order Oct 17, 2025
@hannaseithe hannaseithe changed the title logs for debugging test_poi_category_form_view & rerun of failing container in specific order Logs for debugging flaky test & rerun of failing container in specific order Oct 17, 2025
@hannaseithe
hannaseithe marked this pull request as draft October 17, 2025 10:15
@hannaseithe hannaseithe self-assigned this Oct 17, 2025
@osmers osmers moved this to In review in Integreat 💛 Nov 21, 2025
@MizukiTemma MizukiTemma added the stale This PR has been stale for a while (~3 months). This is a first warning. label Jan 26, 2026
@MizukiTemma

Copy link
Copy Markdown
Member

@github-project-automation github-project-automation Bot moved this from In review to Done in Integreat 💛 Jan 26, 2026
@hannaseithe
hannaseithe deleted the debug-test-poi-category-form-view branch August 5, 2026 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stale This PR has been stale for a while (~3 months). This is a first warning.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants