Skip to content

Commit c3c6dc6

Browse files
Merge pull request #32 from siddharthkp/github-actions-text
fix test for Github actions
2 parents 8920b47 + c155d99 commit c3c6dc6

File tree

1 file changed

+8
-6
lines changed

1 file changed

+8
-6
lines changed

test.js

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -57,14 +57,16 @@ if (ci) {
5757
});
5858

5959
test("pull_request_number is set", t => {
60-
let circlePullRequestNumber;
60+
let pullRequestNumber;
6161
if (process.env.CI_PULL_REQUEST)
62-
circlePullRequestNumber = process.env.CI_PULL_REQUEST.split("/").pop();
63-
62+
pullRequestNumber = process.env.CI_PULL_REQUEST.split("/").pop();
63+
if(process.env.GITHUB_ACTION)
64+
pullRequestNumber = process.env.GITHUB_REF.split('/')[2];
65+
6466
const real_pull_request_number =
6567
process.env.TRAVIS_PULL_REQUEST ||
6668
process.env.DRONE_PULL_REQUEST ||
67-
circlePullRequestNumber ||
69+
pullRequestNumber ||
6870
""; // wercker does not expose pull request number
6971

7072
t.is(pull_request_number, real_pull_request_number);
@@ -102,7 +104,7 @@ if (ci) {
102104
if (event === "pull_request")
103105
t.is(
104106
branch,
105-
process.env.TRAVIS_PULL_REQUEST_BRANCH || process.env.GITHUB_REF
107+
process.env.TRAVIS_PULL_REQUEST_BRANCH || process.env.GITHUB_HEAD_REF
106108
);
107109
else {
108110
const real_branch =
@@ -111,7 +113,7 @@ if (ci) {
111113
process.env.WERCKER_GIT_BRANCH ||
112114
process.env.DRONE_BRANCH ||
113115
process.env.CI_BRANCH || // codeship
114-
process.env.GITHUB_REF;
116+
process.env.GITHUB_REF.split('/')[2];
115117

116118
t.is(branch, real_branch);
117119
}

0 commit comments

Comments
 (0)