diff options
| author | Owen Jacobson <owen@grimoire.ca> | 2025-07-01 00:31:09 -0400 |
|---|---|---|
| committer | Owen Jacobson <owen@grimoire.ca> | 2025-07-01 02:00:17 -0400 |
| commit | b3ce81945621e9026e687b590e7aa541008575ac (patch) | |
| tree | c4dcc161b5323ad6e08f5d7c79f395649203dc95 /src/event/app.rs | |
| parent | c0c825477e476d6d7331bfc409bceff9c376b484 (diff) | |
Sending messages to a deleted channel should send to the deleted channel's ID, not to a fictitious ID.
The existing test scenario:
* Create a channel (with ID C1).
* Delete channel C1.
* Roll the dice to invent a channel ID C2.
* Send a message to channel C2.
* Observe that sending fails.
This was not verifying anything about the deleted channel C1 - it was basically reproducing the `nonexistent_channel` test scenario with the most marginal of garnishes on it. This is probably copy-paste damage from when this test was originally written. Sending did fail, so this test scenario passed, but it passed effectively by accident.
The new test scenario:
* Create a channel (with ID C1).
* Delete channel C1.
* Send a message to channel C1.
* Observe that sending fails.
Concerningly, sending does _not_ fail in this scenario (i.e., the test _does_ fail), so the broken test case was masking a real bug.
Diffstat (limited to 'src/event/app.rs')
0 files changed, 0 insertions, 0 deletions