Skip to content

Commit 9bc5b7f

Browse files
committed
Add test for try user delegation
1 parent e7c6e13 commit 9bc5b7f

File tree

1 file changed

+30
-0
lines changed

1 file changed

+30
-0
lines changed

src/bors/handlers/review.rs

Lines changed: 30 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -831,4 +831,34 @@ approve = ["+approved"]
831831
})
832832
.await;
833833
}
834+
835+
#[sqlx::test]
836+
async fn delegate_insufficient_permission_try_user(pool: sqlx::PgPool) {
837+
let world = World::default();
838+
let try_user = User::new(200, "try-user");
839+
world.default_repo().lock().permissions = Permissions::default();
840+
world
841+
.default_repo()
842+
.lock()
843+
.permissions
844+
.users
845+
.insert(try_user.clone(), vec![PermissionType::Try]);
846+
847+
BorsBuilder::new(pool)
848+
.world(world)
849+
.run_test(|mut tester| async {
850+
tester
851+
.post_comment(Comment::from("@bors delegate+").with_author(try_user))
852+
.await?;
853+
assert_eq!(
854+
tester.get_comment().await?,
855+
"@try-user: :key: Insufficient privileges: not in review users"
856+
);
857+
858+
let pr = tester.get_default_pr().await?;
859+
assert!(!pr.delegated);
860+
Ok(tester)
861+
})
862+
.await;
863+
}
834864
}

0 commit comments

Comments
 (0)