Skip to content

474 add test of EnabledIf behavior with resources

Alexander Nørskov Larsen requested to merge 474-parameterized-enabledif-resources into master

Steps which parameterize a resource of a child step should not depend on those resources.

Currently, if a resource is parameterized on a top-level step, it will be open for the entirety of the test plan run, when it is probably only used for a small portion of it. This MR fixes that behavior.

The behavior change mentioned above also happens to fix the particular instance of the problem that this issue refers to, but we are still not checking the EnabledIf property on parameterized steps. Doing this would make the complexity of EnabledIf checking quadratic in the depth of the testplan.

e.g. the degenerate case below would be O(n2)

-- Sequence
-- -- Sequence
-- -- -- Sequence
-- -- -- -- Sequence
-- -- -- -- -- Sequence

Since EnabledIf can be a performance bottleneck, I also added some performance tests along with the tests to verify the new resource behavior.

Closes #474 (closed)

Edited by Alexander Nørskov Larsen

Merge request reports