Skip to content

[fix] Fix updating fixed_at time#4621

Merged
bruntib merged 1 commit intoEricsson:masterfrom
gulyasgergely902:fix-changing-fixed-date
Jul 16, 2025
Merged

[fix] Fix updating fixed_at time#4621
bruntib merged 1 commit intoEricsson:masterfrom
gulyasgergely902:fix-changing-fixed-date

Conversation

@gulyasgergely902
Copy link
Copy Markdown
Collaborator

Fix updating fixed_at time on every store command. Now it should only update if there is no fixed_at timestamp present, yet.

@gulyasgergely902 gulyasgergely902 added this to the release 6.27.0 milestone Jul 8, 2025
@gulyasgergely902 gulyasgergely902 force-pushed the fix-changing-fixed-date branch 6 times, most recently from 608677a to 818651c Compare July 15, 2025 06:51
self.assertEqual(report.detectionStatus, DetectionStatus.REOPENED)
self.assertEqual(report.reviewData.status, ReviewStatus.FALSE_POSITIVE)
self.assertNotEqual(report.fixedAt, fixed_at)
self.assertEqual(report.fixedAt, fixed_at)
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.

Please, modify the explanatory comment above this section accordingly.

@gulyasgergely902 gulyasgergely902 force-pushed the fix-changing-fixed-date branch from 818651c to d33b254 Compare July 16, 2025 09:09
@gulyasgergely902 gulyasgergely902 requested a review from bruntib July 16, 2025 09:09
… tests

Fix deviating fixed_at date for issue suppression which might caused issues in statistics graph. Add option to run analyzer and web functional tests separately.
@gulyasgergely902 gulyasgergely902 force-pushed the fix-changing-fixed-date branch from d33b254 to c7c1c9f Compare July 16, 2025 13:14
@bruntib bruntib merged commit c896c3b into Ericsson:master Jul 16, 2025
8 of 9 checks passed
@gulyasgergely902 gulyasgergely902 deleted the fix-changing-fixed-date branch July 18, 2025 08:08
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