chore: remove outdated comments.
This commit is contained in:
parent
56ee60d853
commit
58c92b63c6
@ -48,22 +48,14 @@ describe("discord multi-account logic", () => {
|
|||||||
discord: {
|
discord: {
|
||||||
token: "default_token",
|
token: "default_token",
|
||||||
accounts: {
|
accounts: {
|
||||||
bot_a: {}, // no token
|
bot_a: {},
|
||||||
},
|
},
|
||||||
},
|
},
|
||||||
},
|
},
|
||||||
} as unknown as MoltbotConfig;
|
} as unknown as MoltbotConfig;
|
||||||
|
|
||||||
const tokenA = resolveDiscordToken(config, { accountId: "bot_a" });
|
const tokenA = resolveDiscordToken(config, { accountId: "bot_a" });
|
||||||
// Current implementation might strictly require per-account token if it's not the default account.
|
|
||||||
// Let's verify behavior.
|
|
||||||
// Actually, looking at token.ts:
|
|
||||||
// const accountToken = ...
|
|
||||||
// if (accountToken) return ...
|
|
||||||
// const allowEnv = accountId === DEFAULT_ACCOUNT_ID;
|
|
||||||
// const configToken = allowEnv ? ... : undefined;
|
|
||||||
|
|
||||||
// So valid behavior: non-default accounts MUST have a specific token.
|
|
||||||
expect(tokenA.token).toBe("");
|
expect(tokenA.token).toBe("");
|
||||||
});
|
});
|
||||||
|
|
||||||
|
|||||||
@ -53,7 +53,6 @@ describe("resolveAgentRoute - Shared Channel", () => {
|
|||||||
const route = resolveAgentRoute(input);
|
const route = resolveAgentRoute(input);
|
||||||
console.log("Channel Default:", route.sessionKey);
|
console.log("Channel Default:", route.sessionKey);
|
||||||
|
|
||||||
// Should NOT have :default
|
|
||||||
expect(route.sessionKey).toBe("agent:main:discord:channel:channel_123");
|
expect(route.sessionKey).toBe("agent:main:discord:channel:channel_123");
|
||||||
});
|
});
|
||||||
});
|
});
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user