rahix/forgejo-api
1
0
Fork 0
Code Activity

add context to pr errors

Browse source
This commit is contained in:
Cyborus 2023-11-29 16:57:42 -05:00
parent 1291c95545
commit 3737588e28
No known key found for this signature in database
1 changed files with 14 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
tests/ci_test.rs
View file

@ -96,6 +96,14 @@ async fn repo(api: &forgejo_api::Forgejo) -> eyre::Result<()> {
let branch_ref_name = branch_ref.name().ok_or_else(|| eyre!("branch name not found"))?; let branch_ref_name = branch_ref.name().ok_or_else(|| eyre!("branch name not found"))?;
local_repo.set_head(branch_ref_name)?; local_repo.set_head(branch_ref_name)?;
let mut callbacks = git2::RemoteCallbacks::new();
callbacks.credentials(|_url, _username_from_url, _allowed_types| {
git2::Cred::userpass_plaintext("TestingAdmin", "password")
});
let mut push_options = git2::PushOptions::new();
push_options.remote_callbacks(callbacks);
origin.push(&[branch_ref_name], Some(&mut push_options)).map_err(|e| eyre!("failed to push branch, {e}"))?;
let pr_opt = forgejo_api::CreatePullRequestOption { let pr_opt = forgejo_api::CreatePullRequestOption {
assignee: None, assignee: None,
assignees: vec!["TestingAdmin".into()], assignees: vec!["TestingAdmin".into()],
@ -107,8 +115,9 @@ 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?; let pr = api.create_pr("TestingAdmin", "test", pr_opt).await.map_err(|e| eyre!("couldn't create pr, {e}"))?;
ensure!(!api.is_merged("TestingAdmin", "test", pr.number).await?, "pr should not yet be merged"); let is_merged = api.is_merged("TestingAdmin", "test", pr.number).await.map_err(|e| eyre!("couldn't find unmerged pr {}, {}", pr.number, e))?;
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,
merge_commit_id: None, merge_commit_id: None,
@ -119,8 +128,9 @@ 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?; api.merge_pr("TestingAdmin", "test", pr.number, merge_opt).await.map_err(|e| eyre!("couldn't merge pr {}, {}", pr.number, e))?;
ensure!(api.is_merged("TestingAdmin", "test", pr.number).await?, "pr should be merged"); let is_merged = api.is_merged("TestingAdmin", "test", pr.number).await.map_err(|e| eyre!("couldn't find merged pr {}, {}", pr.number, e))?;
ensure!(is_merged, "pr should be merged");
Ok(()) Ok(())
} }