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 🚀