Add tests & rename getChainId.js

packages/fcl/src/default-config.js

@@ -1,5 +1,5 @@
 import {config} from "@onflow/config"
-import {setChainIdDefault} from "./utils/getChainId"
+import {setChainIdDefault, watchForChainIdChanges} from "./utils"
 
 const isServerSide = () => typeof window === "undefined"
 
@@ -15,13 +15,7 @@ config({
 })
 
 // Set chain id default on access node change
-config.subscribe(
-  function configSubscriber(config) {
-    const nextAccessNode = config?.["accessNode.api"]
-    if (prevAccessNode !== nextAccessNode) setChainIdDefault()
-    this.prevAccessNode = nextAccessNode
-  }.bind({})
-)
+watchForChainIdChanges()
 
 export async function configLens(regex) {
   return Object.fromEntries(

packages/fcl/src/utils/chain-id-watcher.js

@@ -0,0 +1,21 @@
+import {config} from "@onflow/config"
+import {setChainIdDefault} from "./get-chain-id"
+
+/**
+ * @description
+ * Watches the config for changes to access node and updates the chain id accordingly
+ *
+ * @returns {Function} A function that unsubscribes the listener
+ *
+ */
+export async function watchForChainIdChanges() {
+  return config.subscribe(
+    function configSubscriber(config) {
+      const nextAccessNode = config?.["accessNode.api"]
+      if (this.prevAccessNode !== nextAccessNode) {
+        setChainIdDefault()
+      }
+      this.prevAccessNode = nextAccessNode
+    }.bind({})
+  )
+}

packages/fcl/src/utils/chain-id-watcher.test.js

@@ -0,0 +1,42 @@
+import {watchForChainIdChanges} from "./chain-id-watcher"
+import {config} from "@onflow/config"
+import * as chainIdUtils from "./get-chain-id"
+
+describe("chain-id-watcher", () => {
+  afterEach(() => {
+    jest.restoreAllMocks()
+  })
+
+  test("flow.network.default is correctly set on first call", async () => {
+    config.overload({"accessNode.api": "https://example.com"}, async () => {
+      const spy = jest.spyOn(chainIdUtils, "setChainIdDefault")
+      spy.mockImplementation(() => {})
+
+      watchForChainIdChanges()
+
+      // Wait for microtask queue to flush
+      await new Promise(resolve => setTimeout(resolve, 0))
+
+      expect(chainIdUtils.setChainIdDefault).toHaveBeenCalled()
+    })
+  })
+
+  test("flow.network.default is correctly set on subsequent changes", async () => {
+    config.overload({"accessNode.api": "https://example.com"}, async () => {
+      watchForChainIdChanges()
+
+      // Wait for microtask queue to flush
+      await new Promise(resolve => setTimeout(resolve, 0))
+
+      const spy = jest.spyOn(chainIdUtils, "setChainIdDefault")
+      spy.mockImplementation(() => {})
+
+      config.put("accessNode.api", "https://example2.com")
+
+      // Wait for microtask queue to flush
+      await new Promise(resolve => setTimeout(resolve, 0))
+
+      expect(chainIdUtils.setChainIdDefault).toHaveBeenCalled()
+    })
+  })
+})

packages/fcl/src/utils/getChainId.js → packages/fcl/src/utils/get-chain-id.js

@@ -1,7 +1,7 @@
 import * as sdk from "@onflow/sdk"
 import {config} from "@onflow/config"
 import {log} from "@onflow/util-logger"
-import { invariant } from "@onflow/util-invariant"
+import {invariant} from "@onflow/util-invariant"
 
 async function getChainIdFromAccessNode() {
   const response = await sdk.send([sdk.getNetworkParameters()]).then(sdk.decode)
@@ -10,9 +10,9 @@ async function getChainIdFromAccessNode() {
 
 /**
  * Sets the default chain id to the chain id of the access node
- * 
+ *
  * @returns {string} The chain id of the access node
- * 
+ *
  * @example
  * // returns "testnet"
  * setChainIdDefault()
@@ -26,10 +26,10 @@ export async function setChainIdDefault() {
 /**
  * @description
  * Gets the chain ID if its set, otherwise gets the chain ID from the access node
- * 
+ *
  * @returns {string} The chain ID of the access node
  * @throws {Error} If the chain ID is not found
- * 
+ *
  * @example
  * // returns "testnet"
  * getChainId()

packages/fcl/src/utils/getChainId.test.js → packages/fcl/src/utils/get-chain-id.test.js

@@ -1,6 +1,6 @@
 import {config} from "@onflow/config"
 import assert from "assert"
-import {getChainId} from "./getChainId"
+import {getChainId} from "./get-chain-id"
 
 describe("getChainId", () => {
   it("getChainId assuming it's already in config", async () => {

packages/fcl/src/utils/index.js

@@ -1,4 +1,5 @@
-export {getChainId} from "./getChainId"
+export {getChainId} from "./get-chain-id"
+export {watchForChainIdChanges} from "./chain-id-watcher"
 
 export function isAndroid() {
   return (