dodie / scott

Never debug a test again: Detailed failure reports and hassle free assertions for Java tests - Power Asserts for Java
https://advancedweb.hu/junit-and-cucumber-test-reports-based-on-source-code-and-behavior/
MIT License
137 stars 17 forks source link

Merge trackVariableName and trackLocalVariableState #65

Closed dodie closed 5 years ago

dodie commented 5 years ago

In the StateRegistry, variables are tracked by two methods: one that tracks their names, and one that tracks their values. Field tracking is done in one method, that tracks name and value with a single call.

Eliminate this asymmetry, and reduce the necessary calls required by eliminating the call to trackVariableName.

This is also required to reduce the amount of logic in the runtime part that needs to match names and values in a thread-safe manner.