idaes-pse icon indicating copy to clipboard operation
idaes-pse copied to clipboard

Prevent test suite from modifying data/config.yaml file

Open lbianchi-lbl opened this issue 2 years ago • 4 comments

See: https://github.com/IDAES/idaes-pse/pull/948#discussion_r967367224

lbianchi-lbl avatar Sep 14 '22 00:09 lbianchi-lbl

Possible fixes:

  • get the test to write this file into a tmp dir
  • get the test to "clean up after itself" and undo the changes it made to the file
  • not have this file in the repo, add it to .gitignore, and have the file generated by the test (then thrown away)

ksbeattie avatar Sep 15 '22 18:09 ksbeattie

Short fix would be to remove the test(s) that cause the data file to be modified.

dangunter avatar Feb 15 '24 19:02 dangunter

This will be taken care of by the forthcoming retirement of the DMF.

ksbeattie avatar Feb 22 '24 19:02 ksbeattie

Related to #1368

ksbeattie avatar Mar 21 '24 18:03 ksbeattie