Made label optional in the parse strategy

[jesper-friis]

Description:

Made label optional in the parse strategy. Added unused metadata field.

Type of change:

• Bug fix.
• New feature.
• Documentation update.

Checklist for the reviewer:

This checklist should be used as a help for the reviewer.

• Is the change limited to one issue?
• Does this PR close the issue?
• Is the code easy to read and understand, including clearly named variables?
• Do all new feature have an accompanying new test?
• Has the documentation been updated as necessary?

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.04%. Comparing base (2d1330d) to head (d92949c).

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #252      +/-   ##
==========================================
+ Coverage   90.00%   90.04%   +0.04%     
==========================================
  Files          15       15              
  Lines         450      452       +2     
==========================================
+ Hits          405      407       +2     
  Misses         45       45              

Flag	Coverage Δ
linux	90.04% <100.00%> (+0.04%)	⬆️
windows	90.04% <100.00%> (+0.04%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

[francescalb]

I orignially approved, but it does not work. I have added a fix.

However, there was not test for this core strategy, and this is something to be remedied immediately. I have started on a test, but please complete it.

NB! when adding the test, it became clear that the 'driver' in 'configuration' cannot be optional, so I hgave removed that.

[francescalb]

good