phone_gap-build 0.5.1 → 0.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 2e0c5f823e46d6457be86cb5297cd24b461aa5fc
-  data.tar.gz: 792c13d4c77551350b495716467a691c6850aabb
+  metadata.gz: c6a3bfa7b5f905953982154c7819b908426a7c20
+  data.tar.gz: eed8c96e1abdbe2fd72271ee5dd1f748563b22ae
 SHA512:
-  metadata.gz: 619c405c7655213e187d098711b087b272981d3896a9741fbdef2a99db832cefe4124ec6d17e65023da43c4b358f08bf783f3587d70726ffaed8da079d13c1d3
-  data.tar.gz: 6c6661f1685aa071b3858fda80a482c74525496662e11fca7cdd68ea35142108b6c4fbc50581698c58897f6af0c23632a82ca1ce73a900153d871373debfcade
+  metadata.gz: f42ab955ad35a889ad6d04dab15d986cc52ad4ef228d397c3e606bd443c06c7769e16eb8bbed2f4e7fd14cd604bfbd2f6b229847886c6a2d678eb01b2b57f679
+  data.tar.gz: dbaadd0913db4e23f8ae394251770620ef7c436873209d874d57fc446d5914bd29bbf4eb23d04314a4142272edf4b659dbec07457a3a383aa6281311460f83e3

data/lib/phone_gap/build/app.rb

@@ -8,7 +8,7 @@ module PhoneGap
       PATH = '/apps'
 
       attr_accessor :title, :create_method, :package, :version, :description, :debug, :keys, :private,
-                    :phonegap_version, :hydrates, :file
+                    :phonegap_version, :hydrates, :file, :status
 
       attr_creatable :title, :create_method, :package, :version, :description, :debug, :keys, :private,
                      :phonegap_version, :hydrates, :file
@@ -45,8 +45,36 @@ module PhoneGap
         end
         raise BuildError.new('An error occurred building at least one of the apps.') if error
         raise BuildError.new('Builds did not complete within the allotted time.') if !error && !complete
+        populate_from_json(json_object)
         complete
       end
+
+      #@TODO another hacky method. Come on Seb :-)
+      def download(params = {})
+        platforms_to_download = params[:platforms] ? params[:platforms] : built_platforms
+        platforms_to_download.each do |platform|
+          response =  ApiRequest.new.get("#{PATH}/#{id}/#{platform}")
+          if response.success?
+            file_name = file_name_from_uri(response.request.instance_variable_get(:@last_uri).request_uri)
+            dir = File.join((params[:save_to] ? params[:save_to] : '/tmp'), platform.to_s)
+            file_path =  File.join(dir, file_name)
+            FileUtils.mkdir_p(dir)
+            puts "Saving to #{file_path}"
+            File.open(file_path, 'w+') { |f| f.write(response.body) }
+            puts 'Download complete'
+          end
+        end
+      end
+
+      private
+
+      def built_platforms
+        status.delete_if { |package, build_status| build_status != 'complete' }.keys
+      end
+
+      def file_name_from_uri(uri)
+        uri.match(/\/([^\/]*)$/)[0]
+      end
     end
 
     class BuildError < Exception ; end

data/lib/phone_gap/build/version.rb

@@ -1,5 +1,5 @@
 module PhoneGap
   module Build
-    VERSION = '0.5.1'
+    VERSION = '0.6.0'
   end
 end

data/spec/phone_gap/build/app_spec.rb

@@ -143,6 +143,7 @@ describe PhoneGap::Build::App do
           end
 
         end
+
         context 'when the build is complete' do
 
           let(:http_response) { double('http response', success?: true, body: response_body_for('get-app') ) }
@@ -170,10 +171,86 @@ describe PhoneGap::Build::App do
           end
         end
       end
+    end
+
+    describe '#download' do
+
+      let(:http_response) { double('http response', success?: false) }
+
+      before do
+        api_request.stub(:get).and_return http_response
+      end
+
+      context 'when there are builds that are complete' do
+
+        before do
+          subject.instance_variable_set('@status', { android: 'complete', ios: 'complete', winphone: 'complete' })
+        end
+
+        context 'when no specific platforms are specified' do
+
+          let(:platforms) { %w(android ios winphone) }
+
+          it 'downloads each platform\'s package' do
+            platforms.each do |platform|
+              expect(api_request).to receive(:get).with("/apps/#{id}/#{platform}")
+            end
+            subject.download
+          end
+        end
+
+        context 'when specific packages are specified' do
+
+          let(:platforms) { %w(ios android) }
 
-      it 'sends a POST request to build the app' do
-        expect(api_request).to receive(:post).with("/apps/#{subject.id}/build")
-        subject.build
+          it 'downloads the specific packages' do
+            platforms.each do |platform|
+              expect(api_request).to receive(:get).with("/apps/#{id}/#{platform}")
+            end
+            subject.download(platforms: platforms)
+          end
+        end
+
+        context 'when the download response is successful' do
+
+          let(:file_name) { 'Something.ipa' }
+          let(:uri) { double('uri', request_uri: "/ios.phonegap/slicehost-production/apps/894786/#{file_name}" ) }
+          let(:request) { double('request') }
+          let(:http_response) { double('http response', success?: true, request: request, body: 'file content') }
+
+          before do
+            api_request.stub(:get).and_return http_response
+            request.stub(:instance_variable_get).with(:@last_uri).and_return uri
+            FileUtils.stub(:mkdir_p)
+          end
+
+          it 'creates a directory to store the file' do
+            expect(FileUtils).to receive(:mkdir_p)
+            subject.download
+          end
+
+          context 'and given a directory to save to' do
+
+            let(:target_dir) { '/save/me/here' }
+
+            it 'saves each package to the given folder' do
+              expect(File).to receive(:open).with("#{target_dir}/ios/#{file_name}", anything)
+              expect(File).to receive(:open).with("#{target_dir}/android/#{file_name}", anything)
+              expect(File).to receive(:open).with("#{target_dir}/winphone/#{file_name}", anything)
+              subject.download(save_to: target_dir)
+            end
+          end
+
+          context 'and not given a directory to save to' do
+
+            it 'saves the package to the tmp folder' do
+              expect(File).to receive(:open).with("/tmp/ios/#{file_name}", anything)
+              expect(File).to receive(:open).with("/tmp/android/#{file_name}", anything)
+              expect(File).to receive(:open).with("/tmp/winphone/#{file_name}", anything)
+              subject.download
+            end
+          end
+        end
       end
     end
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: phone_gap-build
 version: !ruby/object:Gem::Version
-  version: 0.5.1
+  version: 0.6.0
 platform: ruby
 authors:
 - Seb Glazebrook
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-04-28 00:00:00.000000000 Z
+date: 2014-04-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: httmultiparty
@@ -101,6 +101,7 @@ files:
 - lib/phone_gap/build/creatable.rb
 - lib/phone_gap/build/credentials.rb
 - lib/phone_gap/build/error.rb
+- lib/phone_gap/build/package_downloader.rb
 - lib/phone_gap/build/rest_resource.rb
 - lib/phone_gap/build/version.rb
 - phone_gap-build.gemspec