search

CARTAvis / carta-backend-ICD-test

Testing carta-backend via Interface Control Document
0 stars 0 forks source link

Remove the snapshot from FILEINFO #276

Closed acdo2002 closed 2 years ago

acdo2002 commented 3 years ago

Because we would like to monitor FILEINFO.test.ts in more manual handling way, I decide to remove the snapshot from FILEINFO.test.ts, then we may be clear to find the any change in header info.

acdo2002 commented 3 years ago

This branch will based on the FILEINFO returned by this backend branch

kswang1029 commented 3 years ago

using snapshot is still fine as long as the context is verified to be correct and the test framework can pick up differences between the snapshot and the generated file info from the backend.

acdo2002 commented 3 years ago

Yes, but it may easily skip the difference, I think back to the previous code is more easily to monitor any change in the files' header. I just checkout to previous commit and copy them.