RH-FMK / skt / 1309

65%

master: 66%

Pull #358

travis-ci

Remove a default value from Popen() call

The shell=False option is the default value. Having it explicitely
stated in the call makes people scratch their heads - what am I missing
in the code? Where was the True value passed that we need to override it
here?

We should avoid confusing code like this.

Signed-off-by: Veronika Kabatova <vkabatov@redhat.com>

307 of 540 branches covered (56.85%)

Branch coverage included in aggregate %.

982 of 1428 relevant lines covered (68.77%)

0.69 hits per line