Skip to content

chore(migrator): add new properties to AuditLogEntity#1026

Merged
yanavasileva merged 1 commit intomainfrom
1014-auditLog-new-properties
Feb 26, 2026
Merged

chore(migrator): add new properties to AuditLogEntity#1026
yanavasileva merged 1 commit intomainfrom
1014-auditLog-new-properties

Conversation

@yanavasileva
Copy link
Member

Pull Request Template

Description

#1014

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Refactoring (no functional changes)
  • Test-only changes (no production code changes)

Testing Checklist

Black-Box Testing Requirements

  • Tests follow black-box testing approach: verify behavior through observable outputs (logs, C8 API queries, real-world skip scenarios)
  • Tests DO NOT access implementation details (DbClient, IdKeyMapper, ..impl.. packages except logging constants)
  • Architecture tests pass (ArchitectureTest validates these rules)

Test Coverage

  • Added tests for new functionality
  • Updated tests for modified functionality
  • All tests pass locally

Architecture Compliance

Run architecture tests to ensure compliance:

mvn test -Dtest=ArchitectureTest

If architecture tests fail, refactor your tests to use:

  • LogCapturer for log assertions
  • camundaClient.new*SearchRequest() for C8 queries
  • Real-World skip scenarios (e.g., migrate children without parents)

Documentation

  • Updated TESTING_GUIDELINES.md if adding new test patterns
  • Added Javadoc comments for public APIs
  • Updated README if user-facing changes

Checklist

  • Code follows project style guidelines
  • Self-reviewed the code
  • Added comments for complex logic
  • No new compiler warnings
  • Dependent changes have been merged

Related Issues

#1014

@yanavasileva yanavasileva self-assigned this Feb 20, 2026
@yanavasileva yanavasileva changed the base branch from main to adopt-rdbms-mapper-changes February 20, 2026 18:52
@yanavasileva yanavasileva force-pushed the 1014-auditLog-new-properties branch from 8032850 to a10ace7 Compare February 20, 2026 18:53
@yanavasileva
Copy link
Member Author

Base automatically changed from adopt-rdbms-mapper-changes to main February 23, 2026 07:37
@yanavasileva yanavasileva force-pushed the 1014-auditLog-new-properties branch 2 times, most recently from a8f9d7b to 204fddf Compare February 24, 2026 15:17
Copy link
Member

@venetrius venetrius left a comment

Choose a reason for hiding this comment

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

👍 Looks good to me.

@yanavasileva yanavasileva force-pushed the 1014-auditLog-new-properties branch from 204fddf to 67121b3 Compare February 25, 2026 13:46
@yanavasileva yanavasileva force-pushed the 1014-auditLog-new-properties branch from 67121b3 to adf39cc Compare February 25, 2026 19:29
@yanavasileva yanavasileva merged commit 18d1bd0 into main Feb 26, 2026
26 of 27 checks passed
@yanavasileva yanavasileva deleted the 1014-auditLog-new-properties branch February 26, 2026 06:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants