[22.01] Try to unit test more of DictParser in Viz component.

[jmchilton]

Changed a little during 22.01 release cycle - making sure did not regress.

How to test the changes?

(Select all options that apply)

• I've included appropriate automated tests.

License

• I agree to license these contributions under Galaxy's current license.
• I agree to allow the Galaxy committers to license these and all my past contributions to the core galaxy codebase under the MIT license. If this condition is an issue, uncheck and just let us know why with an e-mail to galaxy-committers@lists.galaxyproject.org.

[mvdbeek]

For what it's worth, #13373 is fixing #13384. There's also a unit test we did change, so while we had coverage, we shouldn't have changed it ...

[jmchilton]

But we never had unit test coverage of the odd logic related to nested duplicate tags in DictParser.

                asJson: Any
                if element.tag == element[0].tag:
                    asJson = ListParser(element)
                else:
                    aDict = DictParser(element)
                    if element.items():
                        aDict.update(dict(element.items()))
                    asJson = aDict
                self.update({element.tag: asJson})

I'm glad the source bug was tracked down but I was just verifying this isn't a problem.

[mvdbeek]

Thank you!