diff --git a/module/gitlab/model.php b/module/gitlab/model.php index 321c72ad72..9188186bed 100644 --- a/module/gitlab/model.php +++ b/module/gitlab/model.php @@ -2539,7 +2539,8 @@ class gitlabModel extends model */ public function apiCreateBranchPriv($gitlabID, $projectID, $priv) { - if(empty($gitlabID)) return false; + if(empty($gitlabID)) return false; + if(empty($projectID)) return false; if(empty($priv->name)) return false; $url = sprintf($this->getApiRoot($gitlabID), "/projects/" . $projectID . '/protected_branches'); return json_decode(commonModel::http($url, $priv)); diff --git a/test/model/gitlab/apicreatebranchpriv.php b/test/model/gitlab/apicreatebranchpriv.php index 31bfc13038..5199077991 100755 --- a/test/model/gitlab/apicreatebranchpriv.php +++ b/test/model/gitlab/apicreatebranchpriv.php @@ -9,10 +9,10 @@ cid=1 pid=1 使用空的gitlabID、projectID、保护分支对象创建GitLab保护分支 >> return false -使用空的gitlabID、projectID,正确的保护分支对象创建GitLab保护分支 >> return null -使用正确的gitlabID、保护分支信息,错误的projectID创建保护分支 >> 404 Project Not Found -使用正确的gitlabID,projectID,保护分支对象创建GitLab保护分支 >> master -使用重复的保护分支信息创建保护分支 >> Protected branch 'master' already exists +使用空的gitlabID、projectID,正确的保护分支对象创建GitLab保护分支 >> return false +使用正确的gitlabID、保护分支信息,错误的projectID创建保护分支 >> return false +使用正确的gitlabID,projectID,保护分支对象创建GitLab保护分支 >> master +使用重复的保护分支信息创建保护分支 >> Protected branch 'master' already exists */ @@ -31,13 +31,15 @@ $branch->name = 'master'; $branch->merge_access_level = '40'; $branch->push_access_level = '40'; $result = $gitlab->apiCreateBranchPriv($gitlabID, $projectID, $branch); -if($result === null) $result = 'return null'; +if($result === false) $result = 'return false'; r($result) && p() && e('return false'); //使用空的gitlabID、projectID,正确的保护分支对象创建GitLab保护分支 $gitlabID = 1; -r($gitlab->apiCreateBranchPriv($gitlabID, $projectID, $branch)) && p('message') && e('404 Project Not Found'); //使用正确的gitlabID、保护分支信息,错误的projectID创建保护分支 +$result = $gitlab->apiCreateBranchPriv($gitlabID, $projectID, $branch); +if($result === false) $result = 'return false'; +r($result) && p() && e('return false'); //使用正确的gitlabID、保护分支信息,错误的projectID创建保护分支 $projectID = 966; -r($gitlab->apiCreateBranchPriv($gitlabID, $projectID, $branch)) && p('name') && e($branch->name); //通过gitlabID,projectID,保护分支对象正确创建GitLab保护分支 +r($gitlab->apiCreateBranchPriv($gitlabID, $projectID, $branch)) && p('name') && e($branch->name); //通过gitlabID,projectID,保护分支对象正确创建GitLab保护分支 r($gitlab->apiCreateBranchPriv($gitlabID, $projectID, $branch)) && p('message') && e("Protected branch 'master' already exists"); //使用重复的保护分支信息创建保护分支