nasa / mmt

NASA's Metadata Management Tool.

Apache License 2.0

87 stars 36 forks source link

MMT-3868: Fixed test that failed do to upgrade of rexml #1292

Closed cgokey closed 2 months ago

cgokey commented 2 months ago

Overview

What is the feature?

rexml library was updated in the last gem update. This resulted in a broken test.

What is the Solution?

Fixed the test by properly giving it a valid xml string.

What areas of the application does this impact?

Bamboo, tests should pass now.

Testing

Verify we get a green build in bamboo. Brief regression testing of MMT in SIT would be helpful too.

Checklist

[X] I have added automated tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation
[X] My changes generate no new warnings