bcgov / cas-cif

CleanBC Industry Fund project
Apache License 2.0
6 stars 2 forks source link

fix: 1759 multiple attachment diffs issue #1845

Closed JoshLarouche closed 1 year ago

JoshLarouche commented 1 year ago

zenhub 1759

JoshLarouche commented 1 year ago

Does a test need to be or should be added to ProjectAttachmentsFormSummary.test.tsx? I'm not sure.

JoshLarouche commented 1 year ago

Looks great, bug is fixed! Yes, it would be good to add a test. You could do something like Only displays the data fields that have changed from ProjectAnnualReportFormSummary.test.tsx, but instead of checking for label names, just check the correct filenames show up/don't show up.

Test added!