fixups post review

src/CircuitBreaker.ts

@@ -24,14 +24,14 @@ export class CircuitBreaker extends EventEmitter {
     // a promise that unit tests can await on, as jest's fake timers
     // get confused with async flows
     _evaluatingPromiseHook: Promise<unknown> | null;
-    _cbError: boolean;
+    _failedProbes: boolean;
 
     constructor(config: unknown) {
         super();
 
         this._config = validate(config);
         this._probes = (this._config.probes || []).map(buildProbe);
-        this._cbError = false;
+        this._failedProbes = false;
 
         this._aggregateState = BreakerState.Nominal;
         this._stabilizingCounter = 0;
@@ -51,8 +51,8 @@ export class CircuitBreaker extends EventEmitter {
         return this._aggregateState;
     }
 
-    get probesFailed(): boolean {
-        return this._cbError;
+    get failedProbes(): boolean {
+        return this._failedProbes;
     }
 
     start() {
@@ -68,14 +68,13 @@ export class CircuitBreaker extends EventEmitter {
     }
 
     async _evaluate() {
+        this._failedProbes = false;
         this._evaluatingPromiseHook = Promise.allSettled(this._probes.map(p => p.check().catch(() => {
-            this._cbError = true;
+            this._failedProbes = true;
         })));
         await this._evaluatingPromiseHook;
         this._evaluatingPromiseHook = null;
-        if (!this._cbError) {
-            this._cbError = false;
-        }
+
         const allOk = this._probes.every(probe => probe.value);
         const initialState = this._aggregateState;