`error_threshold_timeout_enabled` can open too aggressively

[AbdulRahmanAlHamali]

This option was introduced in #418, which is meant to open on consecutive errors, regardless of time window. However, its implementation does not really take "consecutiveness" into account, causing it to open incorrectly.

For example, with an error_threshold of 2, and error_threshold_timeout_enabled set to false, the following might cause the circuit to open:

error -> success -> success -> success -> success -> ....10000 more successes -> error

This is not the desired behaviour

[celso-shopify]

Is it correct to assume the success criteria here, for the example above, error_threshold_timeout_enabled = false and error_threshold = 2 should only open if there are 2 consecutive (nothing between) errors, regardless of the window between the 2 events?

[abishanan-shopify]

Is it correct to assume the success criteria here, for the example above, error_threshold_timeout_enabled = false and error_threshold = 2 should only open if there are 2 consecutive (nothing between) errors, regardless of the window between the 2 events?

Yes, I believe this is the documented behaviour, however at first glance this "consecutiveness" property does not seem to be implemented in the current Semian.

[abishanan-shopify]

Seems like our current test case verifies that this documented behaviour is not implemented in the current Semian:

814dac7