diff --git a/src/commands/network/setNetwork.ts b/src/commands/network/setNetwork.ts index de771cb8..d6c50a5c 100644 --- a/src/commands/network/setNetwork.ts +++ b/src/commands/network/setNetwork.ts @@ -98,7 +98,7 @@ export class NetworkActions extends BaseAction { async setNetwork(networkName?: string): Promise { const entries = this.getNetworkEntries(); - if (networkName || networkName === "") { + if (networkName) { const selectedNetwork = entries.find(n => n.alias === networkName || (n.type === "built-in" && n.name === networkName), ); diff --git a/tests/actions/setNetwork.test.ts b/tests/actions/setNetwork.test.ts index a7698150..901487c7 100644 --- a/tests/actions/setNetwork.test.ts +++ b/tests/actions/setNetwork.test.ts @@ -85,12 +85,19 @@ describe("NetworkActions", () => { expect(networkActions["succeedSpinner"]).not.toHaveBeenCalled(); }); - test("setNetwork method fails for empty network name", async () => { + test("setNetwork method prompts user for an empty network name", async () => { + vi.mocked(inquirer.prompt).mockResolvedValue({ + selectedNetwork: "localnet", + }); + await networkActions.setNetwork(""); - expect(networkActions["failSpinner"]).toHaveBeenCalledWith("Network not found"); - expect(networkActions["writeConfig"]).not.toHaveBeenCalled(); - expect(networkActions["succeedSpinner"]).not.toHaveBeenCalled(); + expect(inquirer.prompt).toHaveBeenCalled(); + expect(networkActions["writeConfig"]).toHaveBeenCalledWith("network", "localnet"); + expect(networkActions["succeedSpinner"]).toHaveBeenCalledWith( + `Network successfully set to ${localnet.name}`, + ); + expect(networkActions["failSpinner"]).not.toHaveBeenCalled(); }); test("setNetwork method prompts user when no network name provided", async () => {