Description

Story: https://breedinginsight.atlassian.net/browse/BI-1842

Loosened the validation of observations to only throw an error if a new upload will change the value of an existing observation. If the incoming value is blank, then no validation error will be thrown.

Dependencies

None

Testing

1. Upload an experiment with observations

2. Upload a file to the same experiment with new observations to a different phenotype/trait. Ensure that the observations from the first upload are blank in the second file

3. Ensure the file uploads successfully

4. Upload an experiment with observations

5. Upload a file to the same experiment with new observations to a different phenotype/trait. Ensure that the observations from the first upload are copied from the first file to the second file

6. Ensure the file uploads successfully

7. Upload an experiment with observations

8. Upload a file to the same experiment with new observations to a different phenotype/trait. Also put new values for the existing observations

9. Ensure the file upload throws a validation error for the existing observations attempting to be overwritten

Checklist:

• I have performed a self-review of my own code
• I have tested my code and ensured it meets the acceptance criteria of the story
• I have tested that my code works with both the brapi-java-server and BreedBase
• I have create/modified unit tests to cover this change
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to documentation
• I have run TAF: <please include a link to TAF run>