@@ -397,10 +397,18 @@ func TestPullView_CodeOwner(t *testing.T) {
397397 require .NoError (t , err )
398398
399399 session := loginUser (t , "user5" )
400- testPullCreate (t , session , "user5" , "test_codeowner_fork" , false , forkedRepo .DefaultBranch , "codeowner-basebranch-forked" , "Test Pull Request2" )
401400
402- pr := unittest .AssertExistsAndLoadBean (t , & issues_model.PullRequest {BaseRepoID : repo .ID , HeadBranch : "codeowner-basebranch-forked" })
401+ // create a pull request on the forked repository, code reviewers should not be mentioned
402+ testPullCreateDirectly (t , session , "user5" , "test_codeowner_fork" , forkedRepo .DefaultBranch , "" , "" , "codeowner-basebranch-forked" , "Test Pull Request on Forked Repository" )
403+
404+ pr := unittest .AssertExistsAndLoadBean (t , & issues_model.PullRequest {BaseRepoID : forkedRepo .ID , HeadBranch : "codeowner-basebranch-forked" })
403405 unittest .AssertExistsIf (t , false , & issues_model.Review {IssueID : pr .IssueID , Type : issues_model .ReviewTypeRequest , ReviewerID : 8 })
406+
407+ // create a pull request to base repository, code reviewers should be mentioned
408+ testPullCreateDirectly (t , session , repo .OwnerName , repo .Name , repo .DefaultBranch , forkedRepo .OwnerName , forkedRepo .Name , "codeowner-basebranch-forked" , "Test Pull Request3" )
409+
410+ pr = unittest .AssertExistsAndLoadBean (t , & issues_model.PullRequest {BaseRepoID : repo .ID , HeadRepoID : forkedRepo .ID , HeadBranch : "codeowner-basebranch-forked" })
411+ unittest .AssertExistsIf (t , true , & issues_model.Review {IssueID : pr .IssueID , Type : issues_model .ReviewTypeRequest , ReviewerID : 8 })
404412 })
405413 })
406414}
0 commit comments