diff options
| author | DanConwayDev <DanConwayDev@protonmail.com> | 2024-11-26 07:48:30 +0000 |
|---|---|---|
| committer | DanConwayDev <DanConwayDev@protonmail.com> | 2024-11-26 12:26:47 +0000 |
| commit | a0593e3aa9b19b9ca3c3881cbe0d9d207fe46d2c (patch) | |
| tree | 3482e7c558eff09cc91604ad6ddcd02cf2c94699 /tests/ngit_push.rs | |
| parent | 1332c625b75fce616e06b415c99a068bc45c8210 (diff) | |
refactor: err msgs 'cannot' > 'failed to'
in nearly all cases 'cannot' was used when an action was tried and
failed. 'failed to' is strictly better because:
* just because the action didn't work that time doesnt mean it
cannot work
* it is better at drawing the users attention to a problem
Diffstat (limited to 'tests/ngit_push.rs')
| -rw-r--r-- | tests/ngit_push.rs | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/ngit_push.rs b/tests/ngit_push.rs index f37e027..d4b3894 100644 --- a/tests/ngit_push.rs +++ b/tests/ngit_push.rs | |||
| @@ -58,7 +58,7 @@ mod when_proposal_isnt_associated_with_branch_name { | |||
| 58 | p.expect("fetching updates...\r\n")?; | 58 | p.expect("fetching updates...\r\n")?; |
| 59 | p.expect_eventually("\r\n")?; // some updates listed here | 59 | p.expect_eventually("\r\n")?; // some updates listed here |
| 60 | p.expect_end_with( | 60 | p.expect_end_with( |
| 61 | "Error: cannot find proposal that matches the current branch name\r\n", | 61 | "Error: failed to find proposal that matches the current branch name\r\n", |
| 62 | )?; | 62 | )?; |
| 63 | for p in [51, 52, 53, 55, 56] { | 63 | for p in [51, 52, 53, 55, 56] { |
| 64 | relay::shutdown_relay(8000 + p)?; | 64 | relay::shutdown_relay(8000 + p)?; |