Merge pull request #37 from LucasThompson/bugfix/disabled-tests-tokenisation

drowaudio · web-flow · commit 13dabcc68a2c · 2020-07-17T10:27:23.000+01:00
Fix handling of `--disabled-tests` options
diff --git a/Source/Validator.cpp b/Source/Validator.cpp
@@ -474,7 +474,7 @@ class ValidatorSlaveProcess : public ChildProcessSlave,
             options.dataFile = File (v[IDs::dataFile].toString());
             options.outputDir = File (v[IDs::outputDir].toString());
             options.withGUI = v.getProperty (IDs::withGUI, true);
-            options.disabledTests = StringArray::fromTokens (v.getProperty (IDs::disabledTests).toString(), false);
+            options.disabledTests = StringArray::fromLines (v.getProperty (IDs::disabledTests).toString());
 
             for (auto c : v)
             {

Original file line number	Diff line number	Diff line change
`@@ -474,7 +474,7 @@ class ValidatorSlaveProcess : public ChildProcessSlave,`
`474`	`474`	`options.dataFile = File (v[IDs::dataFile].toString());`
`475`	`475`	`options.outputDir = File (v[IDs::outputDir].toString());`
`476`	`476`	`options.withGUI = v.getProperty (IDs::withGUI, true);`
`477`		`- options.disabledTests = StringArray::fromTokens (v.getProperty (IDs::disabledTests).toString(), false);`
	`477`	`+ options.disabledTests = StringArray::fromLines (v.getProperty (IDs::disabledTests).toString());`
`478`	`478`
`479`	`479`	`for (auto c : v)`
`480`	`480`	`{`