Include inherited labels in databasechangelog table

[nvoxland]

Impact

• Bug fix (non-breaking change which fixes expected existing functionality)
• Enhancement/New feature (adds functionality without impacting existing logic)
• Breaking change (fix or feature that would cause existing functionality to change)

Description

Changes MarkChangeSetRanGenerator so that if a changeSet was included with an include/includeAll containing a labels attribute, those labels are included in the labels column of databasechangelog. Makes the behavior match what the contexts attribute was generally doing before.

Fixes #2824
Fixes #2883 (please test)

However, when looking at the current behavior, I found the following wrong logic and made the following changes:

• The contexts from the include are only logged if changeSet has contexts as well
  • Now the include's context is always tracked
  • The tracked context is always null vs. sometimes an empty string
  • New logic is only applied to new changests going forward, we don't update existing values in the databasechangelog table
• The include/includeAll functions on DatabaseChangeLog took a LabelExpression and not a Labels like it should. In the changelog file, we define just comma separated lists of labels, not a label expression.
  • The java objects now store Labels not LabelExpression objects
  • I added deprecated versions of the DatabaseChangeLog's include(), includeAll(), and setIncludeLabels() functions to match the old versions that took a LabelExpression and converted them into Labels.
  • I did NOT add deprecated version of DatabaseChangeLog.getIncludedLabels() or ChangeSet.getInheriteableLabels() since java cannot overload return values. I don't think those should be methods generally used by extensions and so it is not worth switching to less useful names to preserve the API compatibility.

Additional Context

I'm adding the ReleaseMajor label due to the changes in logic and APIs listed in the description. They should not be serious breaks and will likely not impact anyone, but something for users to be aware of.

[github-actions]

Unit Test Results

  4 548 files  ±  0    4 548 suites  ±0   31m 38s ⏱️ - 5m 9s
  4 508 tests +  2    4 294 ✔️ +  2     214 💤 ±0  0 ❌ ±0 
53 376 runs  +24  48 368 ✔️ +24  5 008 💤 ±0  0 ❌ ±0 

Results for commit 184019f. ± Comparison against base commit 5048d9b.

♻️ This comment has been updated with latest results.

[ecarneiro01]

I tested the issues linked to this PR and both of them are fixed in this build. I also checked that both of them could be applied simultaneously:

• Check label values are propagating correctly to changelog files included in a main changelog.
  • include tag. PASS
• Check context values are propagating correctly to changelog files included in a main changelog.
  • includeAll tag. PASS
  • include tag. PASS

2870.zip