If the bot can't connect to the database it should not run #153

Open
opened 2022-05-09 18:21:01 +01:00 by Vylpes · 1 comment
Vylpes commented 2022-05-09 18:21:01 +01:00 (Migrated from github.com)
No description provided.
Vylpes added this to the 3.5.0 milestone 2025-10-13 18:31:11 +01:00
Vylpes self-assigned this 2025-12-15 22:02:32 +00:00
Vylpes added this to the 3.3 Sprint 9 project 2025-12-15 22:02:33 +00:00
Vylpes stopped working 2025-12-23 15:53:53 +00:00
8 minutes 21 seconds
Vylpes removed their assignment 2026-09-16 13:25:19 +01:00
Smithy-bot removed their assignment 2026-09-22 12:05:43 +01:00
Member

QA results — #153 (step/testing → step/uat)

Branch: release/3.5.0 @ 31250de
Landed via: PR #542 (merged 2026-09-16) — src/client/client.ts now rethrows after logging when AppDataSource.initialize() fails, so start() rejects before super.login().

Verification

Check Result
throw err present in initialize .catch PASS
AppDataSource.initialize() awaited before super.login() PASS
yarn build (tsc) PASS
yarn lint PASS
yarn test PASS — 3 suites / 29 tests / 1 snapshot

Notes

  • Issue has no formal acceptance criteria / test-plan scripts; verified against PR #542 intent (“Throw an exception if database cannot be connected to”).
  • Entry point (src/vylbot.ts) calls client.start() without an explicit .catch/exit — failure surfaces as a rejected/unhandled promise and Discord login is skipped. Manual smoke (bad DB creds → process does not stay connected to Discord) left for human UAT.

Assigning to @Vylpes for UAT.

## QA results — #153 (step/testing → step/uat) **Branch:** `release/3.5.0` @ `31250de` **Landed via:** PR #542 (merged 2026-09-16) — `src/client/client.ts` now rethrows after logging when `AppDataSource.initialize()` fails, so `start()` rejects before `super.login()`. ### Verification | Check | Result | | --- | --- | | `throw err` present in initialize `.catch` | PASS | | `AppDataSource.initialize()` awaited before `super.login()` | PASS | | `yarn build` (`tsc`) | PASS | | `yarn lint` | PASS | | `yarn test` | PASS — 3 suites / 29 tests / 1 snapshot | ### Notes - Issue has no formal acceptance criteria / test-plan scripts; verified against PR #542 intent (“Throw an exception if database cannot be connected to”). - Entry point (`src/vylbot.ts`) calls `client.start()` without an explicit `.catch`/exit — failure surfaces as a rejected/unhandled promise and Discord login is skipped. Manual smoke (bad DB creds → process does not stay connected to Discord) left for human UAT. Assigning to @Vylpes for UAT.
Sign in to join this conversation.
No milestone
No project
No assignees
2 participants
Notifications
Total time spent: 8 minutes 21 seconds
Vylpes
8 minutes 21 seconds
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
RabbitLabs/vylbot-app#153
No description provided.