fix(combine-adapters): to not be stuck in configuring state (#1798)

* fix(combine-adapters): to not be stuck in configuring state

* Create ninety-nails-teach.md

.changeset/ninety-nails-teach.md

@@ -0,0 +1,5 @@
+---
+"@flopflip/combine-adapters": patch
+---
+
+fix(combine-adapters): to not be stuck in configuring state

packages/combine-adapters/src/adapter/adapter.spec.js

@@ -345,10 +345,10 @@ describe('when combining', () => {
       );
     });
 
-    it('should indicate that the adapter is not configured', () => {
+    it('should indicate that the adapter is configured regardless', () => {
       expect(
         adapter.getIsConfigurationStatus(AdapterConfigurationStatus.Configured)
-      ).toBe(false);
+      ).toBe(true);
     });
   });
 

packages/combine-adapters/src/adapter/adapter.ts

@@ -204,10 +204,22 @@ class CombineAdapters implements TCombinedAdapterInterface {
             initializationStatus === AdapterInitializationStatus.Succeeded
         );
 
+      // NOTE: We consider this adapter reconfigured if all adapters have been asked to do so
+      // and have reported to be initialized successfully.
+      this.setConfigurationStatus(AdapterConfigurationStatus.Configured);
+
+      this.#adapterState.emitter.emit(this.#__internalConfiguredStatusChange__);
+
       if (haveAllAdaptersInitializedSuccessfully) {
         return { initializationStatus: AdapterInitializationStatus.Succeeded };
       }
 
+      // NOTE: If not all adapters have initialized successfully we can not consider
+      // this adapter being configured fully.
+      this.setConfigurationStatus(AdapterConfigurationStatus.Unconfigured);
+
+      this.#adapterState.emitter.emit(this.#__internalConfiguredStatusChange__);
+
       return { initializationStatus: AdapterInitializationStatus.Failed };
     });
   }