Skip to content

Commit 59a4a2c

Browse files
committed
Allow multiples repos and GitHub Apps
Add log files and update repository name retrieval Signed-off-by: Rodrigo Nardi <rnardi@netdef.org>
1 parent 7666895 commit 59a4a2c

File tree

3 files changed

+7
-7
lines changed

3 files changed

+7
-7
lines changed

lib/models/pull_request.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,6 @@ def current_execution
3535
end
3636

3737
def self.unique_repository_names
38-
distinct.pluck(:repository_name)
38+
distinct.pluck(:repository)
3939
end
4040
end

spec/lib/github/re_run/comment_spec.rb

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -337,7 +337,7 @@
337337
'user' => { 'login' => 'John' }
338338
},
339339
'repository' => { 'full_name' => 'unit_test' },
340-
'issue' => { 'number' => '10' }
340+
'issue' => { 'number' => pull_request.github_pr_id }
341341
}
342342
end
343343

@@ -365,7 +365,7 @@
365365
]
366366
end
367367

368-
let(:fake_check_suite) { create(:check_suite) }
368+
let(:fake_check_suite) { create(:check_suite, pull_request: pull_request) }
369369
let(:check_suite_rerun) { CheckSuite.find_by(commit_sha_ref: commit_sha, re_run: true) }
370370

371371
it 'must returns success' do
@@ -384,7 +384,7 @@
384384
'body' => 'CI:rerun 000000'
385385
},
386386
'repository' => { 'full_name' => 'unit_test' },
387-
'issue' => { 'number' => '10' }
387+
'issue' => { 'number' => pull_request.github_pr_id }
388388
}
389389
end
390390

@@ -412,7 +412,7 @@
412412
]
413413
end
414414

415-
let(:fake_check_suite) { create(:check_suite) }
415+
let(:fake_check_suite) { create(:check_suite, pull_request: pull_request) }
416416

417417
before do
418418
create(:plan, github_repo_name: 'unit_test')

spec/lib/slack/subscribe_spec.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@
5252
end
5353

5454
context 'when update a subscription' do
55-
let(:create) do
55+
let(:create_request) do
5656
{
5757
'rule' => 'notify',
5858
'target' => 1,
@@ -73,7 +73,7 @@
7373
let(:sub) { PullRequestSubscription.find_by(slack_user_id: update['slack_user_id']) }
7474

7575
before do
76-
subscribe.call(create)
76+
subscribe.call(create_request)
7777
end
7878

7979
it 'must update a subscription' do

0 commit comments

Comments
 (0)