Skip to content

Commit

Permalink
Per #1319, setting PATH as an envvar might cause problems. All variab…
Browse files Browse the repository at this point in the history
…les set prior to the test are unset afterwards! So we'd run all the rest of the tests after unit_python.xml with an empty path. That would likely cause any subsequent call to Rscript to fail. Recommend tightening up this logic when we move these tests to GHA.
  • Loading branch information
MET Tools Test Account committed Apr 5, 2021
1 parent 69f5b5c commit dd22521
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/xml/unit_python.xml
Original file line number Diff line number Diff line change
Expand Up @@ -392,10 +392,10 @@

<!-- Invokes Python pickle logic to read NUMPY text data -->
<test name="python_numpy_plot_data_plane_pickle">
<exec>&MET_BIN;/plot_data_plane</exec>
<exec>export PATH=&ANACONDA_BIN;:${PATH}; \
&MET_BIN;/plot_data_plane</exec>
<env>
<pair><name>MET_PYTHON_EXE</name> <value>&MET_PYTHON_EXE;</value></pair>
<pair><name>PATH</name> <value>&ANACONDA_BIN;</value></pair>
</env>
<param> \
PYTHON_NUMPY \
Expand Down

0 comments on commit dd22521

Please sign in to comment.