rahix/forgejo-api
1
0
Fork 0
Code Activity

use wrap_err instead of map_err

Browse source
This commit is contained in:
Cyborus 2023-11-30 15:14:51 -05:00
parent bab332d520
commit f05936cbd1
No known key found for this signature in database
1 changed files with 8 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
tests/ci_test.rs
View file

@ -1,5 +1,5 @@
use forgejo_api::Forgejo; use forgejo_api::Forgejo;
use eyre::{eyre, ensure}; use eyre::{eyre, ensure, WrapErr};
#[tokio::test] #[tokio::test]
async fn ci() -> eyre::Result<()> { async fn ci() -> eyre::Result<()> {
@ -9,8 +9,8 @@ async fn ci() -> eyre::Result<()> {
let mut results = Vec::new(); let mut results = Vec::new();
results.push(user(&api).await.map_err(|e| eyre!("user error: {e}"))); results.push(user(&api).await.wrap_err("user error"));
results.push(repo(&api).await.map_err(|e| eyre!("repo error: {e}"))); results.push(repo(&api).await.wrap_err("repo error"));
let mut errors = 0; let mut errors = 0;
for res in results.into_iter().filter_map(Result::err) { for res in results.into_iter().filter_map(Result::err) {
@ -102,7 +102,7 @@ async fn repo(api: &forgejo_api::Forgejo) -> eyre::Result<()> {
}); });
let mut push_options = git2::PushOptions::new(); let mut push_options = git2::PushOptions::new();
push_options.remote_callbacks(callbacks); push_options.remote_callbacks(callbacks);
origin.push(&[branch_ref_name], Some(&mut push_options)).map_err(|e| eyre!("failed to push branch, {e}"))?; origin.push(&[branch_ref_name], Some(&mut push_options)).wrap_err("failed to push branch")?;
let pr_opt = forgejo_api::CreatePullRequestOption { let pr_opt = forgejo_api::CreatePullRequestOption {
assignee: None, assignee: None,
@ -115,8 +115,8 @@ async fn repo(api: &forgejo_api::Forgejo) -> eyre::Result<()> {
milestone: None, milestone: None,
title: "test pr".into(), title: "test pr".into(),
}; };
let pr = api.create_pr("TestingAdmin", "test", pr_opt).await.map_err(|e| eyre!("couldn't create pr, {e}"))?; let pr = api.create_pr("TestingAdmin", "test", pr_opt).await.wrap_err("couldn't create pr")?;
let is_merged = api.is_merged("TestingAdmin", "test", pr.number).await.map_err(|e| eyre!("couldn't find unmerged pr {}, {}", pr.number, e))?; let is_merged = api.is_merged("TestingAdmin", "test", pr.number).await.wrap_err_with(|| eyre!("couldn't find unmerged pr {}", pr.number))?;
ensure!(!is_merged, "pr should not yet be merged"); ensure!(!is_merged, "pr should not yet be merged");
let merge_opt = forgejo_api::MergePullRequestOption { let merge_opt = forgejo_api::MergePullRequestOption {
act: forgejo_api::MergePrAction::Merge, act: forgejo_api::MergePrAction::Merge,
@ -128,8 +128,8 @@ async fn repo(api: &forgejo_api::Forgejo) -> eyre::Result<()> {
head_commit_id: None, head_commit_id: None,
merge_when_checks_succeed: None, merge_when_checks_succeed: None,
}; };
api.merge_pr("TestingAdmin", "test", pr.number, merge_opt).await.map_err(|e| eyre!("couldn't merge pr {}, {}", pr.number, e))?; api.merge_pr("TestingAdmin", "test", pr.number, merge_opt).await.wrap_err_with(|| eyre!("couldn't merge pr {}", pr.number))?;
let is_merged = api.is_merged("TestingAdmin", "test", pr.number).await.map_err(|e| eyre!("couldn't find merged pr {}, {}", pr.number, e))?; let is_merged = api.is_merged("TestingAdmin", "test", pr.number).await.wrap_err_with(|| eyre!("couldn't find merged pr {}", pr.number))?;
ensure!(is_merged, "pr should be merged"); ensure!(is_merged, "pr should be merged");
Ok(()) Ok(())