fix test array values

[robbtraister]

GEMD Python PR

Description

Fix test values. Without this comma, adjacent string values are concatenated into a single string element in the array.

PR Type:

• Breaking change (fix or feature that would cause existing functionality to change)
• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Maintenance (non-breaking change to assist developers)

Adherence to team decisions

• I have added tests for 100% coverage
• I have written Numpy-style docstrings for every method and class.
• I have communicated the downstream consequences of the PR to others.
• I have bumped the version in __version__.py

[robbtraister]

@kroenlein do you have any advice on how to handle this PR? It feels odd to do a version bump for a test syntax bugfix.

[kroenlein]

Good catch.

[kroenlein]

Whoever thought auto-concatenate across lines was a good idea should not be allowed to touch a keyboard.

[kroenlein]

@robbtraister says

@kroenlein do you have any advice on how to handle this PR? It feels odd to do a version bump for a test syntax bugfix.

Formally, you could do a force merge. Since this is a testing issue not a code issue, the fact that someone might have the "wrong" commit installed shouldn't be a problem and a developer should always be working off main regardless. This isn't something we'd release for. I don't see harm in a patch bump, but won't be bothered if it doesn't move.

[robbtraister]

I have an unrelated feature proposal on my local that justifies a minor version bump. I could roll it into that.