Note Only change in docs/dev_docs/merger.md that actually occured is that ReadCsvData() calls scanFile() instead of duplicationRemoval(), and that we describe scanFile() which is now the one calling duplicationRemoval().
This is a unit-test that was missing.
Refactoring the scanner-lines into scanFile() simplified the testing of duplicationRemoval(), and will hopefully make it easier to do majority vote later down the line.
Note Only change in
docs/dev_docs/merger.mdthat actually occured is that ReadCsvData() calls scanFile() instead of duplicationRemoval(), and that we describe scanFile() which is now the one calling duplicationRemoval().This is a unit-test that was missing.
Refactoring the scanner-lines into scanFile() simplified the testing of duplicationRemoval(), and will hopefully make it easier to do majority vote later down the line.
Resolves #124