cucumber-ci-environment 8.0.1 → 9.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 05b1fbe1d9ad4999ac7141330120cead66848d06b4897ecba69744073ffacdb7
-  data.tar.gz: 8730be1238d194789a7e8637498d4560880448b48894b0db9eca66e6ee8537a4
+  metadata.gz: 8b113a16cc3df5fa4c163edac4bed8dc9cc479b03d7bc873d0e35e551b17e133
+  data.tar.gz: b350572faa0a9f959d12dfbda6b3b323cf3d18e9a46635d813af29a808f6dd27
 SHA512:
-  metadata.gz: 8b8c1277549217539a97e45138b0a7fa94d37d025f74bafe476b51cf9ca0c8aab5d26be211497f7f0bf07b08b46aa874583f55893fab0096653e92660419d6b6
-  data.tar.gz: f5153a06f6e2fde67fa16e1fa2f755494925395492efe59fba54b258ccffdfa9125d0bc52b6b424b21fd607d3e380399f344cc5a9cc8fdc17caca3d147b7e739
+  metadata.gz: b094712321f9251fab4120c75ed7fd261e187cced6aea4ed0e5fffaf8703b9dc679f5b0a7eb9979c6ff66fe7d40427fface9b156940d2f6c120b9d2785948475
+  data.tar.gz: 71a3c6542c46d9064cc4f276378ca4c9296cbff2be31a7a6073bf78d36f74c88ef23affc4f941eedcd35ca25cc4bb1e155f27c83b87f86dd5e344010a9e8873c

data/lib/cucumber/ci_environment/CiEnvironments.json

@@ -17,8 +17,7 @@
     "git": {
       "remote": "${bamboo_planRepository_repositoryUrl}",
       "revision": "${bamboo_planRepository_revision}",
-      "branch": "${bamboo_planRepository_branch}",
-      "tag": null
+      "branch": "${bamboo_planRepository_branch}"
     }
   },
   {
@@ -61,8 +60,7 @@
     "git": {
       "remote": "${CF_COMMIT_URL/(.*)\\/commit.+$/\\1}.git",
       "revision": "${CF_REVISION}",
-      "branch": "${CF_BRANCH}",
-      "tag": null
+      "branch": "${CF_BRANCH}"
     }
   },
   {
@@ -72,8 +70,7 @@
     "git": {
       "remote": "${CI_PULL_REQUEST/(.*)\\/pull\\/\\d+/\\1.git}",
       "revision": "${CI_COMMIT_ID}",
-      "branch": "${CI_BRANCH}",
-      "tag": null
+      "branch": "${CI_BRANCH}"
     }
   },
   {
@@ -105,8 +102,7 @@
     "git": {
       "remote": "${GO_SCM_*_PR_URL/(.*)\\/pull\\/\\d+/\\1.git}",
       "revision": "${GO_REVISION}",
-      "branch": "${GO_SCM_*_PR_BRANCH/.*:(.*)/\\1}",
-      "tag": null
+      "branch": "${GO_SCM_*_PR_BRANCH/.*:(.*)/\\1}"
     }
   },
   {
@@ -116,8 +112,7 @@
     "git": {
       "remote": "${GIT_URL}",
       "revision": "${GIT_COMMIT}",
-      "branch": "${GIT_LOCAL_BRANCH}",
-      "tag": null
+      "branch": "${GIT_LOCAL_BRANCH}"
     }
   },
   {
@@ -149,8 +144,7 @@
     "git": {
       "remote": "https://${WERCKER_GIT_DOMAIN}/${WERCKER_GIT_OWNER}/${WERCKER_GIT_REPOSITORY}.git",
       "revision": "${WERCKER_GIT_COMMIT}",
-      "branch": "${WERCKER_GIT_BRANCH}",
-      "tag": null
+      "branch": "${WERCKER_GIT_BRANCH}"
     }
   }
 ]

data/lib/cucumber/ci_environment.rb

@@ -7,17 +7,17 @@ module Cucumber
     extend VariableExpression
     CI_ENVIRONMENTS_PATH = File.join(File.dirname(__FILE__), 'ci_environment/CiEnvironments.json')
 
-    def detect_ci_environment(env)
-      ci_environments = JSON.parse(IO.read(CI_ENVIRONMENTS_PATH))
+    def detect_ci_environment(env, file_reader = IO.method(:read))
+      ci_environments = JSON.parse(file_reader.call(CI_ENVIRONMENTS_PATH))
       ci_environments.each do |ci_environment|
-        detected = detect(ci_environment, env)
+        detected = detect(ci_environment, env, file_reader)
         return detected unless detected.nil?
       end
 
       nil
     end
 
-    def detect(ci_environment, env)
+    def detect(ci_environment, env, file_reader)
       url = evaluate(ci_environment['url'], env)
       return nil if url.nil?
 
@@ -27,13 +27,13 @@ module Cucumber
         buildNumber: evaluate(ci_environment['buildNumber'], env),
       }
 
-      detected_git = detect_git(ci_environment, env)
+      detected_git = detect_git(ci_environment, env, file_reader)
       result[:git] = detected_git if detected_git
       result
     end
 
-    def detect_git(ci_environment, env)
-      revision = evaluate(ci_environment['git']['revision'], env)
+    def detect_git(ci_environment, env, file_reader)
+      revision = detect_revision(ci_environment, env, file_reader)
       return nil if revision.nil?
 
       remote = evaluate(ci_environment['git']['remote'], env)
@@ -51,6 +51,17 @@ module Cucumber
       git_info
     end
 
+    def detect_revision(ci_environment, env, file_reader)
+      if env['GITHUB_EVENT_NAME'] == 'pull_request'
+        raise StandardError('GITHUB_EVENT_PATH not set') unless env['GITHUB_EVENT_PATH']
+        event = JSON.parse(file_reader.call(env['GITHUB_EVENT_PATH']))
+        raise StandardError('GITHUB_EVENT_PATH not set') unless event['before']
+        return event['before']
+      end
+
+      return evaluate(ci_environment['git']['revision'], env)
+    end
+
     def remove_userinfo_from_url(value)
       return nil if value.nil?
 
@@ -63,6 +74,6 @@ module Cucumber
       end
     end
 
-    module_function :detect_ci_environment, :detect, :detect_git, :remove_userinfo_from_url
+    module_function :detect_ci_environment, :detect, :detect_git, :detect_revision, :remove_userinfo_from_url
   end
 end

data/spec/cucumber/ci_environment/ci_environment_spec.rb

@@ -8,7 +8,12 @@ describe 'detect_ci_environment' do
     context "with #{File.basename(test_data_file, '.txt')}" do
       subject { JSON.parse(ci_environment.to_json) }
 
-      let(:ci_environment) { Cucumber::CiEnvironment.detect_ci_environment(env) }
+      def mock_reader(path)
+        return '{"before": "2436f28fad432a895bfc595bce16e907144b0dc3"}' if path.end_with?('_github_workflow/event.json')
+        IO.read(path)
+      end
+
+      let(:ci_environment) { Cucumber::CiEnvironment.detect_ci_environment(env, method(:mock_reader)) }
       let(:env) { Hash[entries] }
       let(:entries) { env_data.split(/\n/).map { |line| line.split(/=/) } }
       let(:env_data) { IO.read(test_data_file) }

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: cucumber-ci-environment
 version: !ruby/object:Gem::Version
-  version: 8.0.1
+  version: 9.0.1
 platform: ruby
 authors:
 - Vincent Prêtre
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-12-29 00:00:00.000000000 Z
+date: 2022-03-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -36,20 +36,20 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.10'
+        version: '3.11'
     - - ">="
       - !ruby/object:Gem::Version
-        version: 3.10.0
+        version: 3.11.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.10'
+        version: '3.11'
     - - ">="
       - !ruby/object:Gem::Version
-        version: 3.10.0
+        version: 3.11.0
 description: Detect CI Environment from environment variables
 email: cukes@googlegroups.com
 executables: []
@@ -92,7 +92,7 @@ requirements: []
 rubygems_version: 3.2.32
 signing_key: 
 specification_version: 4
-summary: cucumber-ci-environment-8.0.1
+summary: cucumber-ci-environment-9.0.1
 test_files:
 - spec/capture_warnings.rb
 - spec/cucumber/ci_environment/ci_environment_spec.rb