Use separate environment for every executed command

[andywaltlova]

HMS-2240

• Change the way we set the env var from globally to local to the command
• Write unit test to verify that the env var is not present globally
• Look for a way to verify if the env var is set for the command executed only
  • the test verifies that same variables can be set for two different commands with different values

[codecov-commenter]

Codecov Report

Patch coverage: 100.00% and project coverage change: +1.41% 🎉

Comparison is base (1050628) 66.56% compared to head (0421466) 67.97%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main      #59      +/-   ##
==========================================
+ Coverage   66.56%   67.97%   +1.41%     
==========================================
  Files           5        5              
  Lines         326      331       +5     
==========================================
+ Hits          217      225       +8     
+ Misses         93       92       -1     
+ Partials       16       14       -2     

Flag	Coverage Δ
go-1.16	67.97% <100.00%> (+1.41%)	⬆️
go-1.20	67.97% <100.00%> (+1.41%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed	Coverage Δ
src/runner.go	82.65% <100.00%> (+4.15%)	⬆️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[andywaltlova]

something is not right, every other run of tests fails

SOLVED: Alright, it's not deterministic to use os.Environ() in test, the environment can change during test execution. I changed the test function to test that the environment is subset rather than exactly same.

[r0x0d]

We need to remember to remove those two logs later

[andywaltlova]

I just removed all of the debug logs now, I never used it, can be manually added anytime during local development setup anyway

[r0x0d]

Just a few comments, but otherwise it looks good!