Skip to content

Rename trace.Fail to trace.Complete

Pedro Pombeiro (OOO from July 16th till Aug 7th) requested to merge minor-renames into master

What does this MR do?

Rename trace.Fail to trace.Complete, since trace.Fail is also used for successful cases. It also renames some variables to make them more consistent with the rest of the code.

Why was this MR needed?

When browsing the code, the existing names didn't feel right, and I assume the same will apply to people who are new to the codebase.

Are there points in the code the reviewer needs to double check?

Does this MR meet the acceptance criteria?

  • Documentation created/updated
  • Added tests for this feature/bug
  • In case of conflicts with master - branch was rebased

What are the relevant issue numbers?

Merge request reports