Spark Tests

[sdreyer]

Description

Type of change

Does this correspond to an open issue?

Select type(s) of change

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Documentation update

Checklist:

• I have performed a self-review of my code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes
• I have fixed any merge conflicts

[codecov]

Codecov Report

Merging #958 (d98455e) into main (d0a114d) will increase coverage by 1.82%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #958      +/-   ##
==========================================
+ Coverage   57.22%   59.05%   +1.82%     
==========================================
  Files         107      107              
  Lines       19187    19189       +2     
==========================================
+ Hits        10980    11332     +352     
+ Misses       7396     7052     -344     
+ Partials      811      805       -6     

Files Changed	Coverage Δ
provider/filestore.go	41.81% <100.00%> (+2.32%)	⬆️
provider/spark.go	50.39% <100.00%> (+27.27%)	⬆️

... and 2 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[ahmadnazeri]

LGTM; just one question.