Fixes #2352 by adding support for loading embedded jar files

[hayeskl]

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

Fixes #2352 so that embedded.jar files containing the changelog directory will once again work with includeAll. The issue was that the path passed into ZipResourceAccessor was stripping off the embedded path and need the extra path elements passed in to load the filesystem correctly.

Things to be aware of

Things to worry about

Additional Context

[MalloD12]

Thanks @hayeskl for submitting this PR. Only question I have is shouldn't these code changes be extended to DirectoryResourceAccessor and DirectoryPathHandler as well?

Other than that code changes look good to me. Build and tests have been successfully executed.

Thanks,
Daniel.

[hayeskl]

Thanks @hayeskl for submitting this PR. Only question I have is shouldn't these code changes be extended to DirectoryResourceAccessor and DirectoryPathHandler as well?

Other than that code changes look good to me. Build and tests have been successfully executed.

Thanks, Daniel.

I do not believe those have the same issue. In my testing I could load the migrations from the filesystem just fine, but when I tried loading the migrations from an embedded jar that is when it didn't work.

[MalloD12]

Approved.

Build and tests have been executed successfully, except for functional test which are failing due to an unknown issue trying to download artifacts.

[rberezen]

Tests are successful, with one exception unrelated to the code changes. Thank you, @hayeskl, for your contribution.