Import Test change detection variable fixes.

Hyperfoil / Horreum

Benchmark results repository service

https://horreum.hyperfoil.io/

Apache License 2.0

34 stars 30 forks source link

Import Test change detection variable fixes. #1744

Closed whitingjr closed 3 weeks ago

whitingjr commented 1 month ago

Added testcase to flush out defect when importing an existing Test with new definitions of Change Detection variables. Added fixes to add new variables.

Fixes Issue

fixes #1740

Changes proposed

[x] - Allow more situation when a Test fields are updated

Check List (Check all the applicable boxes)

[X] My code follows the code style of this project.
[X] All new and existing tests passed.

whitingjr commented 1 month ago

can you update testImportTestWithChangeDetectionVariable() to verify the ExperimentProfile import as well, and change the test name to reflect.

Sure, this has been updated.

whitingjr commented 4 weeks ago

@johnaohara Changes done.