remove error-reports from unit tests

[zargot]

@yulric, I think we should remove the error-report assets (used in unit-tests) since they make it very difficult to change the error-messages, etc. I don't think we need this redundancy? The unit-tests can still check that the correct errors are being reported by accessing key information like row-number, error-type and invalid-value.

[yulric]

Do you mean files like this one https://github.com/Big-Life-Lab/PHES-ODM-Validation/blob/main/assets/validation-rules/duplicate-entries-found/error-report-1.json?

[zargot]

Do you mean files like this one https://github.com/Big-Life-Lab/PHES-ODM-Validation/blob/main/assets/validation-rules/duplicate-entries-found/error-report-1.json?

yes, exactly