play_time 0.0.3 → 0.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 52263d955063ada0381b1f92eb4f3eb888e9b09a
-  data.tar.gz: a4a5c66750eeba46e127b1a16813fb3935873559
+  metadata.gz: e23c57d7102a28cbe8b3554651891b0a5b6bff67
+  data.tar.gz: 813f6b865d38e05b6998991d307c8fa8c973a0e6
 SHA512:
-  metadata.gz: 416e3bbdbde16698f213f275b444181ca7e5ccbb448bb6f51de1fe026d87acc436da08573ce16df454d001ea2a6bc3ca405e5f4af51f6110f704363cfe28aa73
-  data.tar.gz: c8a5867c0c328306ee31d97a5d2f555e82bc85946d4ba10c3d72e79ebe6046ef13c7e2a125748a49a982af22242a33affd816da19bad15854cd51678b677f736
+  metadata.gz: 90cd9ec3380d972328fef4105a8729b57fd39aabc9c71269ecac47ed51938b3b18daba3b30adb24545a9fe5d7679dd633999f17ad8fcb848962aa2ac0325d7df
+  data.tar.gz: 841c881c775d74055937b3e3be0d96ce9dc68a883dae45440023ea01f51d9664a67297c5c383a23db913ab8df5af08383d83db93278a2346e8fb7ff696f825ac

data/.ruby-version

@@ -0,0 +1 @@
+2.1.3

data/README.md

@@ -22,6 +22,8 @@ Or install it yourself as:
 
 TODO: Write usage instructions here
 
+* add promotion syntax like `rake play_time:promote['alpha', 'beta']`
+
 ## Contributing
 
 1. Fork it ( https://github.com/[my-github-username]/play_time/fork )

data/lib/play_time/client.rb

@@ -5,6 +5,8 @@ require 'play_time/apk'
 
 module PlayTime
   class Client
+    class VersionCodeNotFound < StandardError; end
+
     TOKEN_URI   = 'https://accounts.google.com/o/oauth2/token'.freeze
     SCOPE       = 'https://www.googleapis.com/auth/androidpublisher'.freeze
     API         = 'androidpublisher'.freeze
@@ -17,27 +19,45 @@ module PlayTime
 
     def commit(track)
       create_insert
-      upload_apk
-      update_track(track)
+      version_code = upload_apk
+      update_track(track, version_code)
+      save
+    end
+
+    def update(track, version_code)
+      create_insert
+      old_track = get_old_track! version_code
+      update_track(old_track.track, *old_track.versionCodes - [version_code])
+      update_track(track, version_code)
       save
     end
 
     private
 
-    attr_reader :edit_id, :version_code
+    attr_reader :edit_id
 
     def create_insert
       @edit_id = run!(api_method: service.edits.insert, parameters: parameters).data.id
     end
 
+    def get_old_track!(version_code)
+      old_track = get_tracks.find { |track| track.versionCodes.include? version_code }
+
+      old_track || raise(VersionCodeNotFound, version_code)
+    end
+
+    def get_tracks
+      run!(api_method: service.edits.tracks.list, parameters: parameters.merge(editId: edit_id)).data.tracks
+    end
+
     def upload_apk
       upload_params = parameters.merge(editId: edit_id, uploadType: 'media')
-      @version_code = run!(api_method: service.edits.apks.upload, parameters: upload_params, media: Apk.load).data.versionCode
+      run!(api_method: service.edits.apks.upload, parameters: upload_params, media: Apk.load).data.versionCode
     end
 
-    def update_track(track)
+    def update_track(track, *version_codes)
       update_params = parameters.merge(editId: edit_id, track: track)
-      run! api_method: service.edits.tracks.update, parameters: update_params, body_object: { versionCodes: [version_code] }
+      run! api_method: service.edits.tracks.update, parameters: update_params, body_object: { versionCodes: version_codes }
     end
 
     def save

data/lib/play_time/configuration.rb

@@ -13,6 +13,18 @@ module PlayTime
       package_name
     ).freeze
 
+    def self.exists?
+      File.exist? PlayTime.config_path
+    end
+
+    def self.create_config(config_dir, config_path)
+      unless File.exist?(config_dir)
+        FileUtils.mkdir_p(config_dir)
+      end
+
+      FileUtils.cp("#{__dir__}/templates/play_time.yml", config_path)
+    end
+
     def initialize(config)
       @config = config
     end

data/lib/play_time/promote.rb

@@ -0,0 +1,22 @@
+require 'play_time/client'
+
+module PlayTime
+  class Promote
+    class << self
+      def promote(version_code, track)
+        new(Client.new).promote(version_code, track)
+      end
+    end
+
+    attr_reader :client
+
+    def initialize(client)
+      @client = client
+    end
+
+    def promote(version_code, track)
+      client.authorize!
+      client.update(track, version_code)
+    end
+  end
+end

data/lib/play_time/tasks.rb

@@ -0,0 +1,27 @@
+require 'rake'
+require 'play_time'
+
+namespace :play_time do
+  namespace :upload do
+    PlayTime::Track::TRACKS.each do |track|
+      desc "Uploads apk to #{track}"
+      task(track) do |task|
+        PlayTime.upload(track)
+      end
+    end
+  end
+
+  namespace :promote do
+    PlayTime::Track::TRACKS.each do |track|
+      desc "Promote apk version code to #{track}"
+      task(track, :version_code) do |task, args|
+        PlayTime.promote(track, args[:version_code].to_i)
+      end
+    end
+  end
+
+  desc 'Creates a config file in config/play_time.yml'
+  task :install do
+    PlayTime.install
+  end
+end

data/lib/play_time/templates/play_time.yml

@@ -0,0 +1,7 @@
+package_name: # package name
+secret_path: # path to secret.p12
+secret_passphrase: # secret.p12 passphrase
+apk_path: # path to apk
+client_name: # Google developer client application name
+client_version: # Google developer client application version
+issuer: # issuer for Google developer application

data/lib/play_time/track.rb

@@ -7,6 +7,8 @@ module PlayTime
     ROLLOUT = 'rollout'.freeze
     PRODUCTION = 'production'.freeze
 
+    TRACKS = [ALPHA, BETA, ROLLOUT, PRODUCTION].freeze
+
     def self.validate!(track)
       raise InvalidTrack.new("#{track} is an invalid track") unless valid?(track)
     end

data/lib/play_time/upload.rb

@@ -3,8 +3,7 @@ require 'play_time/client'
 module PlayTime
   class Upload
     def self.upload(track)
-      upload = Upload.new(Client.new)
-      upload.upload(track)
+      new(Client.new).upload(track)
     end
 
     attr_reader :client

data/lib/play_time/version.rb

@@ -1,3 +1,3 @@
 module PlayTime
-  VERSION = "0.0.3"
+  VERSION = "0.1.0"
 end

data/lib/play_time.rb

@@ -3,17 +3,33 @@ require 'play_time/version'
 require 'play_time/track'
 require 'play_time/configuration'
 require 'play_time/upload'
+require 'play_time/promote'
 
 module PlayTime
-  def self.config_path
-    ENV['PLAY_TIME_CONFIG_PATH'] || Configuration::DEFAULT_CONFIG
-  end
+  class << self
+    def config_path
+      ENV['PLAY_TIME_CONFIG_PATH'] || Configuration::DEFAULT_CONFIG
+    end
 
-  def self.configuration
-    @configuration ||= Configuration.new(YAML.load(open(config_path).read))
-  end
+    def configuration
+      @configuration ||= Configuration.new(YAML.load(open(config_path).read))
+    end
+
+    def upload(track)
+      Upload.upload(track)
+    end
+
+    def promote(version_code, track)
+      Promote.promote(version_code, track)
+    end
 
-  def self.upload(track)
-    Upload.upload(track)
+    def install
+      if Configuration.exists?
+        puts "You already have a config file in #{config_path}!"
+      else
+        puts "Generating a new config file: #{config_path}"
+        Configuration.create_config('config', config_path)
+      end
+    end
   end
 end

data/spec/{client_spec.rb → lib/client_spec.rb}

@@ -129,4 +129,113 @@ describe PlayTime::Client do
       )
     end
   end
+
+  describe '#update' do
+    let(:version_code) { 99 }
+    let(:track) { 'alpha' }
+    let(:service) { double(Google::APIClient::API) }
+    let(:data) { double(:data, id: 'id', tracks: tracks) }
+    let(:response) { double(:response, data: data) }
+    let(:tracks) { [double(:track, track: 'old track', versionCodes: [version_code])] }
+
+    subject { PlayTime::Client.new.update(track, version_code) }
+
+    before do
+      allow(PlayTime.configuration).to receive(:package_name).and_return('com.package.name')
+      allow(PlayTime::Runner).to receive(:run!).and_return(response)
+      allow(api_client).to receive(:discovered_api).and_return(service)
+      allow(service).to receive_message_chain('edits.insert').and_return('insert')
+      allow(service).to receive_message_chain('edits.tracks.update').and_return('track update')
+      allow(service).to receive_message_chain('edits.tracks.list').and_return('lists')
+      allow(service).to receive_message_chain('edits.commit').and_return('commit')
+    end
+
+
+    it 'creates a service' do
+      subject
+
+      expect(api_client).to have_received(:discovered_api).with('androidpublisher', 'v2')
+    end
+
+    it 'creates an edit' do
+      subject
+
+      expect(PlayTime::Runner).to have_received(:run!).with(
+        api_client, api_method: 'insert', parameters: { packageName: 'com.package.name' })
+    end
+
+    context 'when the version code is in no other track' do
+      let(:tracks) { [double(:track, track: 'old track', versionCodes: [-23])] }
+
+      it 'raises an error' do
+        expect {
+          subject
+        }.to raise_error PlayTime::Client::VersionCodeNotFound, version_code.to_s
+      end
+    end
+
+    context 'when the version code is in another track' do
+      let(:tracks) { [double(:track, track: 'old track', versionCodes: [version_code])] }
+
+      context 'when it is the only version in that track' do
+        it 'removes everything from the old track' do
+          subject
+
+          expect(PlayTime::Runner).to have_received(:run!).with(
+            api_client,
+            api_method: 'track update',
+            parameters: { packageName: 'com.package.name', editId: 'id', track: 'old track' },
+            body_object: { versionCodes: [] }
+          )
+        end
+      end
+
+      context 'when there are other versions in the track' do
+        let(:tracks) { [double(:track, track: 'old track', versionCodes: [version_code, 100])] }
+
+        it 'only removes itself from the track' do
+          subject
+
+          expect(PlayTime::Runner).to have_received(:run!).with(
+            api_client,
+            api_method: 'track update',
+            parameters: { packageName: 'com.package.name', editId: 'id', track: 'old track' },
+            body_object: { versionCodes: [100] }
+          )
+        end
+      end
+    end
+
+    it 'updates a track with the new version code' do
+      subject
+
+      expect(PlayTime::Runner).to have_received(:run!).with(
+        api_client,
+        api_method: 'track update',
+        parameters: { packageName: 'com.package.name', editId: 'id', track: track },
+        body_object: { versionCodes: [version_code] }
+      )
+    end
+
+    it 'updates a track with the new version code' do
+      subject
+
+      expect(PlayTime::Runner).to have_received(:run!).with(
+        api_client,
+        api_method: 'track update',
+        parameters: { packageName: 'com.package.name', editId: 'id', track: track },
+        body_object: { versionCodes: [version_code] }
+      )
+    end
+
+    it 'commits the changes' do
+      subject
+
+      expect(PlayTime::Runner).to have_received(:run!).with(
+        api_client,
+        api_method: 'commit',
+        parameters: { packageName: 'com.package.name', editId: 'id' }
+      )
+    end
+  end
 end

data/spec/lib/configuration_spec.rb

@@ -0,0 +1,69 @@
+describe PlayTime::Configuration do
+  shared_examples_for 'configuration option' do
+    let(:configuration) { PlayTime::Configuration.new({option.to_s => option, 'foo' => 'bar'}) }
+
+    subject { configuration.send(option) }
+
+    it "fetchings the option from the config" do
+      expect(subject).to eq option
+    end
+
+    context 'missing options' do
+      let(:configuration) { PlayTime::Configuration.new({'foo' => 'bar'}) }
+
+      it "raises an exception for each missing param" do
+        expect {
+          subject
+        }.to raise_error PlayTime::Configuration::MissingOption, "Missing #{option} in #{PlayTime.config_path}"
+      end
+    end
+  end
+
+  PlayTime::Configuration::OPTIONS.each do |option|
+    describe "##{option}" do
+      let(:option) { option }
+
+      it_behaves_like 'configuration option'
+    end
+  end
+
+  describe '.exists?' do
+    before do
+      allow(File).to receive(:exist?).and_return(true)
+      allow(PlayTime).to receive(:config_path).and_return('/path')
+    end
+
+    it 'delegates to file' do
+      PlayTime::Configuration.exists?
+
+      expect(File).to have_received(:exist?).with('/path')
+    end
+  end
+
+  describe '.create_config' do
+    let(:config_path) { '/config/path.yml' }
+    let(:config_dir) { '/config/dir' }
+
+    subject { PlayTime::Configuration.create_config(config_dir, config_path) }
+
+    before do
+      allow(FileUtils).to receive(:mkdir_p)
+    end
+
+    context 'when config dir exists' do
+      before do
+        allow(File).to receive(:exist?).and_return(true)
+      end
+
+      it 'does not create a config dir' do
+        expect(FileUtils).not_to have_received(:mkdir_p).with(config_dir)
+      end
+    end
+
+    context 'when config dir does not exist' do
+      before do
+        allow(File).to receive(:exist?).and_return(false)
+      end
+    end
+  end
+end

data/spec/lib/promote_spec.rb

@@ -0,0 +1,53 @@
+describe PlayTime::Promote do
+  let(:version_code) { 99 }
+  let(:track) { 'alpha' }
+
+  describe '.promote' do
+    let(:promote) { instance_double(PlayTime::Promote) }
+    let(:client) { instance_double(PlayTime::Client) }
+
+    subject { PlayTime::Promote.promote(version_code, track) }
+
+    before do
+      allow(promote).to receive(:promote)
+      allow(PlayTime::Client).to receive(:new).and_return(client)
+      allow(PlayTime::Promote).to receive(:new).and_return(promote)
+    end
+
+    it 'promotes the track' do
+      subject
+
+      expect(promote).to have_received(:promote).with(version_code, track)
+    end
+
+    it 'instantiates upload with new authorization and client' do
+      subject
+
+      expect(PlayTime::Promote).to have_received(:new).with(client)
+    end
+  end
+
+  describe '#promote' do
+    let(:client) { instance_double(PlayTime::Client) }
+    let(:promote) { PlayTime::Promote.new(client) }
+
+    subject { promote.promote(version_code, track) }
+
+    before do
+      allow(client).to receive(:authorize!)
+      allow(client).to receive(:update)
+    end
+
+    it 'authroizes the client' do
+      subject
+
+      expect(client).to have_received(:authorize!)
+    end
+
+    it 'updates the track with the version code' do
+      subject
+
+      expect(client).to have_received(:update).with(track, version_code)
+    end
+  end
+end

data/spec/lib/tasks_spec.rb

@@ -0,0 +1,76 @@
+require 'play_time/tasks'
+
+describe 'play_time tasks' do
+  describe 'upload' do
+    shared_examples_for 'upload task' do
+      it 'uploads an apk to production' do
+        allow(PlayTime).to receive(:upload)
+
+        Rake::Task["play_time:upload:#{track}"].invoke
+
+        expect(PlayTime).to have_received(:upload).with(track)
+      end
+    end
+
+    PlayTime::Track::TRACKS.each do |track|
+      it_behaves_like 'upload task' do
+        let(:track) { track }
+      end
+    end
+  end
+
+  describe 'promote' do
+    shared_examples_for 'promote task' do
+      it 'promotes the version number' do
+        allow(PlayTime).to receive(:promote)
+
+        Rake::Task["play_time:promote:#{track}"].invoke("232")
+
+        expect(PlayTime).to have_received(:promote).with(track, 232)
+      end
+    end
+
+    PlayTime::Track::TRACKS.each do |track|
+      it_behaves_like 'promote task' do
+        let(:track) { track }
+      end
+    end
+  end
+
+  describe 'install task' do
+    let(:config_file) { File.expand_path('config/play_time.yml') }
+
+    before do
+      allow(PlayTime).to receive(:config_path).and_return('config/play_time.yml')
+      Rake::Task['play_time:install'].reenable
+    end
+
+    after do
+      File.delete(config_file) if File.exist?(config_file)
+    end
+
+    context "when config file doesn't exist" do
+      it 'generates an exmaple config file' do
+        Rake::Task['play_time:install'].invoke
+
+        expect(File).to exist(config_file)
+        expect(open(File.expand_path("lib/play_time/templates/play_time.yml")).read). to eq(open(config_file).read)
+      end
+    end
+
+    context "when config file already exists" do
+      before do
+        Rake::Task['play_time:install'].invoke
+        Rake::Task['play_time:install'].reenable
+      end
+
+      it "doesn't do anything" do
+        sleep(0.5)
+
+        expect do
+          Rake::Task['play_time:install'].invoke
+        end.not_to change { File.open(PlayTime.config_path).mtime }
+      end
+    end
+  end
+end

data/spec/play_time_spec.rb

@@ -8,7 +8,6 @@ describe PlayTime, if: PlayTime.config_path.end_with?('default.yml') do
   end
 
   describe '.upload' do
-    let(:apk_path) { '/path/to/app.apk' }
     let(:track) { PlayTime::Track::ALPHA }
 
     subject { PlayTime.upload(track) }
@@ -21,4 +20,52 @@ describe PlayTime, if: PlayTime.config_path.end_with?('default.yml') do
       expect(PlayTime::Upload).to have_received(:upload).with(track)
     end
   end
+
+  describe '.promote' do
+    let(:version_code) { 99 }
+    let(:track) { PlayTime::Track::BETA }
+
+    subject { PlayTime.promote(version_code, track) }
+
+    it 'promotes the version code' do
+      allow(PlayTime::Promote).to receive(:promote)
+
+      subject
+
+      expect(PlayTime::Promote).to have_received(:promote).with(version_code, track)
+    end
+  end
+
+  describe '.install' do
+    subject { PlayTime.install }
+
+    context 'when configuration exists' do
+      it 'outputs message' do
+        allow(PlayTime::Configuration).to receive(:exists?).and_return(true)
+
+        expect{
+          subject
+        }.to output("You already have a config file in #{PlayTime.config_path}!\n").to_stdout
+      end
+    end
+
+    context 'when the configuration does not exist' do
+
+      before do
+        allow(PlayTime::Configuration).to receive(:create_config)
+        allow(PlayTime::Configuration).to receive(:exists?).and_return(false)
+      end
+
+      it 'outputs message' do
+        expect{
+          subject
+        }.to output("Generating a new config file: #{PlayTime.config_path}\n").to_stdout
+      end
+
+      it 'creates the config' do
+        subject
+        expect(PlayTime::Configuration).to have_received(:create_config).with('config', PlayTime.config_path)
+      end
+    end
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: play_time
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.1.0
 platform: ruby
 authors:
 - IS Dev team
@@ -75,6 +75,7 @@ extra_rdoc_files: []
 files:
 - ".gitignore"
 - ".rspec"
+- ".ruby-version"
 - Gemfile
 - LICENSE.txt
 - README.md
@@ -83,20 +84,25 @@ files:
 - lib/play_time/apk.rb
 - lib/play_time/client.rb
 - lib/play_time/configuration.rb
+- lib/play_time/promote.rb
 - lib/play_time/runner.rb
+- lib/play_time/tasks.rb
+- lib/play_time/templates/play_time.yml
 - lib/play_time/track.rb
 - lib/play_time/upload.rb
 - lib/play_time/version.rb
 - play_time.gemspec
-- spec/apk_spec.rb
-- spec/client_spec.rb
-- spec/configuration_spec.rb
+- spec/lib/apk_spec.rb
+- spec/lib/client_spec.rb
+- spec/lib/configuration_spec.rb
+- spec/lib/promote_spec.rb
+- spec/lib/runner_spec.rb
+- spec/lib/tasks_spec.rb
+- spec/lib/track_spec.rb
+- spec/lib/upload_spec.rb
 - spec/play_time_spec.rb
-- spec/runner_spec.rb
 - spec/spec_helper.rb
 - spec/support/config/default.yml
-- spec/track_spec.rb
-- spec/upload_spec.rb
 homepage: https://github.com/is-devteam/play_time
 licenses:
 - MIT
@@ -122,12 +128,14 @@ signing_key:
 specification_version: 4
 summary: PlayTime is a gem for uploading apks to the Google Play Store
 test_files:
-- spec/apk_spec.rb
-- spec/client_spec.rb
-- spec/configuration_spec.rb
+- spec/lib/apk_spec.rb
+- spec/lib/client_spec.rb
+- spec/lib/configuration_spec.rb
+- spec/lib/promote_spec.rb
+- spec/lib/runner_spec.rb
+- spec/lib/tasks_spec.rb
+- spec/lib/track_spec.rb
+- spec/lib/upload_spec.rb
 - spec/play_time_spec.rb
-- spec/runner_spec.rb
 - spec/spec_helper.rb
 - spec/support/config/default.yml
-- spec/track_spec.rb
-- spec/upload_spec.rb

data/spec/configuration_spec.rb

@@ -1,29 +0,0 @@
-describe PlayTime::Configuration do
-  shared_examples_for 'configuration option' do
-    let(:configuration) { PlayTime::Configuration.new({option.to_s => option, 'foo' => 'bar'}) }
-
-    subject { configuration.send(option) }
-
-    it "fetchings the option from the config" do
-      expect(subject).to eq option
-    end
-
-    context 'missing options' do
-      let(:configuration) { PlayTime::Configuration.new({'foo' => 'bar'}) }
-
-      it "raises an exception for each missing param" do
-        expect {
-          subject
-        }.to raise_error PlayTime::Configuration::MissingOption, "Missing #{option} in #{PlayTime.config_path}"
-      end
-    end
-  end
-
-  PlayTime::Configuration::OPTIONS.each do |option|
-    describe "##{option}" do
-      let(:option) { option }
-
-      it_behaves_like 'configuration option'
-    end
-  end
-end