percy-client 0.8.0 → 0.9.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 5df2807d5bef5b5b77ca00292bb84ebe7f7fd83d
-  data.tar.gz: 783f788102ad2f7c36e3118b3a41e67e97d21fd5
+  metadata.gz: bc578e0917f5911a1aa66074c6b3df2cfd29f13b
+  data.tar.gz: e8018bcffe5f1023331a360f6e3c1137409e15e9
 SHA512:
-  metadata.gz: 26e5228491098661a2338a110da7f2f2fad75273d12a6649221be1ec489eee7f25e2a52b4f1814f1032326ca3eadffb0e2d0998e672416c8c2626d45d11e84a0
-  data.tar.gz: dbc6d60778ace333d3378f539617f1922c550c55618466c62f5ba4a43964fdd682369186916900591557d33d56d234e53e1ef30ddb3ea952725d3b096ec326e9
+  metadata.gz: f2fd7787234164b43c0ba7b9cc65a5414f374b7515f7ae397cbcb2de889020f1bb55678876433ab166c4be29418b00b4260ed8e55141deff7da6383c2ec820a6
+  data.tar.gz: e6b211360bd8650475918f344f9f810349407e0f1a961d97fdd5f56e9214062de4e95390128d9d60aaf2350a518ad511ee591687a01b4d95d74b35e975ced155

data/lib/percy/client/builds.rb

@@ -6,6 +6,13 @@ module Percy
           Percy::Client::Environment.pull_request_number
         commit_data = options[:commit_data] || Percy::Client::Environment.commit
         resources = options[:resources]
+        parallel_nonce = options[:parallel_nonce] || Percy::Client::Environment.parallel_nonce
+        parallel_total_shards = options[:parallel_total_shards] \
+          || Percy::Client::Environment.parallel_total_shards
+
+        if (parallel_nonce && !parallel_total_shards) || (!parallel_nonce && parallel_total_shards)
+          raise ArgumentError.new('If parallel_nonce is given, parallel_total_shards is required.')
+        end
 
         data = {
           'data' => {
@@ -20,6 +27,8 @@ module Percy
               'commit-committer-email' => commit_data[:committer_email],
               'commit-message' => commit_data[:message],
               'pull-request-number' => pull_request_number,
+              'parallel-nonce' => parallel_nonce,
+              'parallel-total-shards' => parallel_total_shards,
             },
           }
         }
@@ -41,6 +50,12 @@ module Percy
 
         build_data = post("#{config.api_url}/repos/#{repo}/builds/", data)
         Percy.logger.debug { "Build #{build_data['data']['id']} created" }
+        parallelism_msg = if parallel_total_shards
+          "#{parallel_total_shards} shards detected (nonce: #{parallel_nonce.inspect})"
+        else
+          'not detected'
+        end
+        Percy.logger.debug { "Parallel test environment: #{parallelism_msg}" }
         build_data
       end
 

data/lib/percy/client/environment.rb

@@ -156,6 +156,14 @@ module Percy
         end
       end
 
+      def self.parallel_nonce
+        return ENV['PERCY_PARALLEL_NONCE'] if ENV['PERCY_PARALLEL_NONCE']
+      end
+
+      def self.parallel_total_shards
+        return Integer(ENV['PERCY_PARALLEL_TOTAL']) if ENV['PERCY_PARALLEL_TOTAL']
+      end
+
       # @private
       def self._get_origin_url
         `git config --get remote.origin.url`

data/lib/percy/client/version.rb

@@ -1,5 +1,5 @@
 module Percy
   class Client
-    VERSION = '0.8.0'
+    VERSION = '0.9.0'
   end
 end

data/spec/lib/percy/client/builds_spec.rb

@@ -33,6 +33,16 @@ RSpec.describe Percy::Client::Builds, :vcr do
       expect(build['data']['relationships']['missing-resources']['data']).to be
       expect(build['data']['relationships']['missing-resources']['data'].length).to eq(1)
     end
+    context 'parallel test environment' do
+      it 'raises an error if either parallel_nonce or parallel_total_shards is given alone' do
+        expect do
+          Percy.create_build('fotinakis/percy-examples', parallel_nonce: 123)
+        end.to raise_error(ArgumentError)
+        expect do
+          Percy.create_build('fotinakis/percy-examples', parallel_total_shards: 2)
+        end.to raise_error(ArgumentError)
+      end
+    end
   end
   describe '#finalize_build' do
     it 'finalizes a build' do

data/spec/lib/percy/client/environment_spec.rb

@@ -5,6 +5,8 @@ RSpec.describe Percy::Client::Environment do
     ENV['PERCY_BRANCH'] = nil
     ENV['PERCY_PULL_REQUEST'] = nil
     ENV['PERCY_REPO_SLUG'] = nil
+    ENV['PERCY_PARALLEL_NONCE'] = nil
+    ENV['PERCY_PARALLEL_TOTAL'] = nil
 
     # Unset Travis vars.
     ENV['TRAVIS_BUILD_ID'] = nil
@@ -139,6 +141,24 @@ RSpec.describe Percy::Client::Environment do
           Percy::Client::Environment::RepoNotFoundError)
       end
     end
+    describe '#parallel_nonce' do
+      it 'returns nil' do
+        expect(Percy::Client::Environment.parallel_nonce).to be_nil
+      end
+      it 'can be set with environment var' do
+        ENV['PERCY_PARALLEL_NONCE'] = 'nonce'
+        expect(Percy::Client::Environment.parallel_nonce).to eq('nonce')
+      end
+    end
+    describe '#parallel_total_shards' do
+      it 'returns nil' do
+        expect(Percy::Client::Environment.parallel_nonce).to be_nil
+      end
+      it 'can be set with environment var' do
+        ENV['PERCY_PARALLEL_TOTAL'] = '3'
+        expect(Percy::Client::Environment.parallel_total_shards).to eq(3)
+      end
+    end
   end
   context 'in Jenkins CI' do
     before(:each) do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: percy-client
 version: !ruby/object:Gem::Version
-  version: 0.8.0
+  version: 0.9.0
 platform: ruby
 authors:
 - Perceptual Inc.
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-10-01 00:00:00.000000000 Z
+date: 2015-10-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday