Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

278: Enhance the /solves command #556

Closed
wants to merge 4 commits into from
Closed
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
@@ -196,15 +196,15 @@ void multiple(TestInfo testInfo) throws IOException {
TestBotRunner.runPeriodicItems(prBot);

// The bot should add both
assertLastCommentContains(pr, "Adding additional issue to solves list");
assertLastCommentContains(pr, "Adding additional issue to issue list");
assertLastCommentContains(pr, ": First");
assertLastCommentContains(pr, ": Second");

// Remove one
pr.addComment("/issue remove " + issue1.id());
TestBotRunner.runPeriodicItems(prBot);

assertLastCommentContains(pr, "Removing additional issue from solves list: `" + issue1Number + "`");
assertLastCommentContains(pr, "Removing additional issue from issue list: `" + issue1Number + "`");

// Approve it as another user
var approvalPr = integrator.pullRequest(pr.id());