[Chapel Merge] Replace chplenv variables in ReadFileWithComments

Branch: refs/heads/master
Revision: e318a2b
Author: ronawho
Log Message:

Merge pull request #16812 from ronawho/chplenv-subtest-read-file

Replace chplenv variables in ReadFileWithComments

[reviewed by @ben-albrecht and @lydia-duncan]

ReadFileWithComments and runSkipIf are responsible for reading most
testing infrastructure files in sub_test. Both can either just read in a
non-executable file or run an executable file and read in the output.
runSkipIf will inject the chplenv for executable and non-executable
files, but ReadFileWithComments was only doing so for executable

For non-executable files ReadFileWithComments was only injecting
env vars. Update it to inject chplenv vars as well.

Related prior work:

See https://github.com/chapel-lang/chapel/pull/16805 for motivation

Modified Files:
M test/library/packages/ProtobufProtocolSupport/EXECENV
M test/library/packages/UnitTest/EXECENV
M test/mason/EXECENV
M test/mason/mason-offline/EXECENV
M test/mason/search/EXECENV
M test/mason/search/badFileName/EXECENV
M test/mason/search/cacheTest/EXECENV
M test/patterns/cltools/fileCompile/chplCompile.execenv
M util/test/sub_test.py

Compare: https://github.com/chapel-lang/chapel/compare/2e5f9504121e...e318a2b9a142