fix: disabled breaker emits fire events

To revert to previous behaviour up to v8.1.4.

nodeshift#894

lib/circuit.js

@@ -619,6 +619,13 @@ class CircuitBreaker extends EventEmitter {
 
     const args = rest.slice();
 
+    /**
+     * Emitted when the circuit breaker action is executed
+     * @event CircuitBreaker#fire
+     * @type {any} the arguments passed to the fired function
+     */
+    this.emit('fire', args);
+
     // Protection, caches and coalesce disabled.
     if (!this[ENABLED]) {
       const result = this.action.apply(context, args);
@@ -630,13 +637,6 @@ class CircuitBreaker extends EventEmitter {
     // Need to create variable here to prevent extra calls if cache is disabled
     let cacheKey = '';
 
-    /**
-     * Emitted when the circuit breaker action is executed
-     * @event CircuitBreaker#fire
-     * @type {any} the arguments passed to the fired function
-     */
-    this.emit('fire', args);
-
     // If cache is enabled, check if we have a cached value
     if (this.options.cache) {
       cacheKey = this.options.cacheGetKey.apply(this, rest);