fix: Issue/644

[karel-rehor]

Closes #644

Proposed Changes

When FluxCSVParser parses a value, and the stringValue is empty and the default value for the column is empty, now checks whether the data type for the column is "string" and if this is the case returns the empty default value instead of null.

Note that headerless csv tables will still return null and that null will be returned if the default value is null.

Two tests are added to FluxCsvParserTest.

Checklist

• CHANGELOG.md updated
• Rebased/mergeable
• A test has been added if appropriate
• mvn test completes successfully
• Commit messages are conventional
• Sign CLA (if not already signed)

[bednar]

Thank you for your PR! 👍 Could you please update the CHANGELOG.md file to reflect your changes?

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (19bde3e) 88.36% compared to head (776c67e) 88.36%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@ Coverage Diff @@ ## master #648 +/- ## ========================================= Coverage 88.36% 88.36% Complexity 779 779 ========================================= Files 172 172 Lines 7022 7024 +2 Branches 382 383 +1 ========================================= + Hits 6205 6207 +2  Misses 698 698 Partials 119 119 

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[bednar]

LGTM 🚀