Fix #1438 - changeset dbms field serialization

[p1729]

Fixes #1438

---

name: Pull Request
about: Fix changeset dbms field serialisation
title: Fix changeset dbms field serialisation
labels: Status:Fixed
assignees: ''

---

Environment

Liquibase Version: 4.1.1-local-SNAPSHOT

Liquibase Integration & Version: CLI

Liquibase Extension(s) & Version:

Database Vendor & Version:

Operating System Type & Version: MacOS 10.15.6

Pull Request Type

• [x ] Bug fix (non-breaking change which fixes an issue.)
• Enhancement/New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Description

Serialisation of multiple dbms values in changeset is not happening correctly

Steps To Reproduce

Create a changeset with multiple dbms e.g 'dbms1, dbms2' and ask for serialisation of dbms field

Actual Behavior

Serialisation of multiple dbms adds ',xxx' between multiple dbms serialised value

Expected/Desired Behavior

Serialisation of multiple dbms add ',' between multiple dbms serialised value

Additional Context

Add any other context about the problem here.

Fast Track PR Acceptance Checklist:

• [ x] Build is successful and all new and existing tests pass
• [ x] Added Unit Test(s)
• Added Integration Test(s)
• Documentation Updated

Need Help?

Come chat with us on our discord channel

[codecov]

Codecov Report

Merging #1450 into master will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##             master    #1450   +/-   ##
=========================================
  Coverage     47.80%   47.80%           
+ Complexity     7844     7843    -1     
=========================================
  Files           792      792           
  Lines         38516    38516           
  Branches       6880     6880           
=========================================
  Hits          18413    18413           
  Misses        17674    17674           
  Partials       2429     2429           

Impacted Files	Coverage Δ	Complexity Δ
...e/src/main/java/liquibase/changelog/ChangeSet.java	64.16% <100.00%> (ø)	154.00 <0.00> (ø)
...re/src/main/java/liquibase/util/ISODateFormat.java	78.68% <0.00%> (-1.64%)	16.00% <0.00%> (-1.00%)
...e/src/main/java/liquibase/util/DependencyUtil.java	92.10% <0.00%> (+0.87%)	0.00% <0.00%> (ø%)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 85f71ff...27eb6e7. Read the comment docs.

[molivasdat]

Congrats @p1729 Hacktoberfest accepted. Thanks for your contribution!

[nvoxland]

Thanks!

Cherry picked into 4.2.x as well