facebookincubator / TTPForge / 10452897637

64%

main: 67%

Pull #509

github

Use temp file to process powershell nested scripts as TTP steps (#509)

Summary:
Pull Request resolved: https://github.com/facebookincubator/TTPForge/pull/509

## Problem scope
Let's test with the following simple powershell script
```
% cat nested.ps
echo "start"
if ($true) {
  echo "nested"
}
echo "stop"
```

Here is results of running this script differently
```
% pwsh -File nested.ps
start
nested
stop
% pwsh nested.ps
start
nested
stop
% cat nested.ps | pwsh -File -
PS /data/users/nesusvet/fbsource/fbcode> echo "start"
start
PS /data/users/nesusvet/fbsource/fbcode> if ($true) {
>>   echo "nested"
>> }
>> echo "stop"
>> %
% cat nested.ps | pwsh -Command -
start
```

## Proposal
Instead of writing the TTP step body to power shell process stdin, we might rely on a new temp file to hold the body of TTP step to pass it to PowerShell as only argument.  This produces the best results.

## Limitations
Runtime environment where standard golang library call to `os.CreateTemp` fails or the file created is not writable.

Resolves https://github.com/facebookincubator/TTPForge/issues/481

Differential Revision: D61387125

36 of 72 new or added lines in 2 files covered. (50.0%)

1792 of 2830 relevant lines covered (63.32%)

13.5 hits per line