motion-sparkle-sandbox 2.0.1 → 2.2

data/lib/motion/project/sparkle.rb

@@ -15,29 +15,22 @@ module Motion
         # verify_installation
       end
 
+      def after_initialize
+        self.feed_url = appcast.feed_url
+      end
+
       def appcast
-        @appcast ||= Appcast.new
+        @appcast ||= Appcast.new(self)
       end
 
       def publish(key, value)
+        return if appcast.process_option(key, value)
+
         case key
         when :public_key
-          self.public_EdDSA_key = value
-        when :base_url
-          appcast.base_url = value
-          self.feed_url = appcast.feed_url
-        when :feed_base_url
-          appcast.feed_base_url = value
-          self.feed_url = appcast.feed_url
-        when :feed_filename
-          appcast.feed_filename = value
-          self.feed_url = appcast.feed_url
+          self.public_ed_dsa_key = value
         when :version
-          version value
-        when :package_base_url, :package_filename, :notes_base_url, :notes_filename, :use_exported_private_key
-          appcast.send "#{key}=", value
-        when :archive_folder
-          appcast.archive_folder = value
+          version(value)
         else
           raise "Unknown Sparkle config option #{key}"
         end
@@ -61,15 +54,13 @@ module Motion
         @config.info_plist['SUFeedURL'] = url
       end
 
-      # rubocop:disable Naming/MethodName
-      def public_EdDSA_key
+      def public_ed_dsa_key
         @config.info_plist['SUPublicEDKey']
       end
 
-      def public_EdDSA_key=(key)
+      def public_ed_dsa_key=(key)
         @config.info_plist['SUPublicEDKey'] = key
       end
-      # rubocop:enable Naming/MethodName
 
       # File manipulation and certificates
 
@@ -109,7 +100,7 @@ module Motion
 
         if appcast.use_exported_private_key && File.exist?(private_key_path)
           App.info 'Sparkle', "Private key already exported at `#{private_key_path}` and will be used."
-          if public_EdDSA_key.present?
+          if public_ed_dsa_key.present?
             App.info '', <<~EXISTS
               SUPublicEDKey already set
 
@@ -139,12 +130,12 @@ module Motion
           return
         end
 
-        results, status = Open3.capture2e(generate_keys_app, '-p')
+        results, status = Open3.capture2e(generate_keys_app, '-p', '--account', appcast.cli_options[:account])
 
         if status.success?
-          App.info 'Sparkle', 'Public/private keys found in the keychain'
+          App.info 'Sparkle', "Public/private keys found in the keychain for account #{appcast.cli_options[:account]}"
 
-          if results.strip == public_EdDSA_key
+          if results.strip == public_ed_dsa_key
             App.info 'Sparkle', 'Keychain public key matches `SUPublicEDKey`'
 
             if appcast.use_exported_private_key && !File.exist?(private_key_path)
@@ -155,7 +146,7 @@ module Motion
               Keychain public key DOES NOT match `SUPublicEDKey`
 
                   Keychain public key:      #{results.strip}
-                  SUPublicEDKey public key: #{public_EdDSA_key}
+                  SUPublicEDKey public key: #{public_ed_dsa_key}
 
             NOT_MATCHED
               .indent(11, skip_first_line: true)
@@ -173,7 +164,7 @@ module Motion
       def create_private_key
         App.info 'Sparkle',
                  'Generating a new signing key into the Keychain. This may take a moment, depending on your machine.'
-        results, status = Open3.capture2e(generate_keys_app)
+        results, status = Open3.capture2e(generate_keys_app, '--account', appcast.cli_options[:account])
 
         App.fail 'Sparkle could not generate keys' unless status.success?
 
@@ -181,7 +172,7 @@ module Motion
         puts results.lines[1..].join.indent(11)
 
         # Extract the public key so we can use it in message
-        results, status = Open3.capture2e(generate_keys_app, '-p')
+        results, status = Open3.capture2e(generate_keys_app, '-p', '--account', appcast.cli_options[:account])
 
         App.fail 'Unable to read public key' unless status.success?
 
@@ -199,7 +190,7 @@ module Motion
 
       # Export the private key from the keychain
       def export_private_key
-        _results, status = Open3.capture2e(generate_keys_app, '-x', private_key_path.to_s)
+        _results, status = Open3.capture2e(generate_keys_app, '-x', private_key_path.to_s, '--account', appcast.cli_options[:account])
 
         App.fail 'Unable to export private key' unless status.success?
 
@@ -216,6 +207,92 @@ module Motion
           .indent(11)
       end
 
+      # copy the release notes and zip archive into the releases_folder,
+      # where the appcast will get built
+      def copy_to_release
+        destination_path = (project_path + releases_folder).realpath
+        zip_file_path = (sparkle_release_path + zip_file)
+
+        [release_notes_path, zip_file_path].each do |file|
+          FileUtils.cp(file, "#{destination_path}/")
+          App.info 'Copied', "#{destination_path}/#{file}"
+        end
+      end
+
+      # Generate the appcast.
+      # Note: We do not support the old DSA keys, only the newer EdDSA keys.
+      #       See https://sparkle-project.org/documentation/eddsa-migration
+      def generate_appcast
+        generate_appcast_app = "#{vendored_sparkle_path}/bin/generate_appcast"
+        path = (project_path + releases_folder).realpath
+        appcast_filename = (path + appcast.feed_filename)
+        appcast.cli_options[:output_path] = appcast_filename
+
+        FileUtils.mkdir_p(path) unless File.exist?(path)
+
+        App.info('Sparkle', "Generating appcast using `#{generate_appcast_app}`")
+        puts "from files in `#{path}`...".indent(11)
+
+        args = appcast.prepare_args
+
+        App.info 'Executing', [generate_appcast_app, *args, path.to_s].join(' ')
+
+        results, status = Open3.capture2e(generate_appcast_app, *args, path.to_s)
+
+        App.info('Sparkle', "Saved appcast to `#{appcast_filename}`") if status.success?
+        puts results.indent(11)
+
+        return unless status.success?
+
+        puts
+        puts "SUFeedURL     : #{feed_url}".indent(11)
+        puts "SUPublicEDKey : #{public_ed_dsa_key}".indent(11)
+      end
+
+      def generate_appcast_help
+        generate_appcast_app = "#{vendored_sparkle_path}/bin/generate_appcast"
+        results, _status = Open3.capture2e(generate_appcast_app, '--help')
+
+        puts results
+      end
+
+      def create_release_notes
+        App.fail "Release notes template not found as expected at ./#{release_notes_template_path}" unless File.exist?(release_notes_template_path)
+
+        create_release_folder
+
+        File.open(release_notes_path.to_s, 'w') do |f|
+          template = File.read(release_notes_template_path)
+          f << ERB.new(template).result(binding)
+        end
+
+        App.info 'Create', "./#{release_notes_path}"
+      end
+
+      def release_notes_template_path
+        sparkle_config_path.join('release_notes.template.erb')
+      end
+
+      def release_notes_content_path
+        sparkle_config_path.join('release_notes.content.html')
+      end
+
+      def release_notes_path
+        sparkle_release_path + (appcast.notes_filename || "#{app_name}.#{@config.short_version}.html")
+      end
+
+      def release_notes_content
+        if File.exist?(release_notes_content_path)
+          File.read(release_notes_content_path)
+        else
+          App.fail "Missing #{release_notes_content_path}"
+        end
+      end
+
+      def release_notes_html
+        release_notes_content
+      end
+
       # A few helpers
 
       def project_path
@@ -242,10 +319,6 @@ module Motion
         sparkle_config_path.join(EDDSA_PRIV_KEY)
       end
 
-      def legacy_private_key_path
-        sparkle_config_path.join(DSA_PRIV_KEY)
-      end
-
       def app_bundle_path
         Pathname.new(@config.app_bundle_raw('MacOSX'))
       end
@@ -262,8 +335,8 @@ module Motion
         appcast.package_filename || "#{app_name}.#{@config.short_version}.zip"
       end
 
-      def archive_folder
-        appcast.archive_folder
+      def releases_folder
+        appcast.releases_folder
       end
 
       def app_file

data/lib/motion-sparkle-sandbox/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module MotionSparkleSandbox
-  VERSION = '2.0.1'
+  VERSION = '2.2'
 end

data/lib/motion-sparkle-sandbox.rb

@@ -16,7 +16,7 @@ require 'motion/project/indent_string'
 
 lib_dir_path = File.dirname(File.expand_path(__FILE__))
 
-POD_VERSION = '~> 2.0.0'
+POD_VERSION = '~> 2.2.0'
 
 unless @running_specs
   Motion::Project::App.setup do |app|

data/sample-app/.gitignore

@@ -7,3 +7,4 @@ vendor
 
 sparkle/release
 sparkle/config
+sparkle/config/eddsa_priv.key

data/sample-app/Rakefile

@@ -25,8 +25,8 @@ Motion::Project::App.setup do |app|
 
   app.sparkle do
     publish :base_url, 'http://example.com/your_app_folder/releases/'
-    publish :public_key, '06Iq4HGJFToJrA8lqWWG0SCqqtGXLuEN1Wgy/CVJBnI='
-    publish :archive_folder, '../published/releases/v2.2/'
+    publish :public_key, '1/eJupmw4JH+iFQIlh99nS2qvwYealJsNzFN3LL6FFE=x'
+    publish :releases_folder, 'tmp/releases/v2.2/'
 
     # Appcast Feed
     # publish :feed_base_url, 'http://downloads.example.com/releases' # defaults to base_url
@@ -40,5 +40,8 @@ Motion::Project::App.setup do |app|
     # publish :package_base_url, 'http://downloads.example.com/releases/' # defaults to base_url
 
     # publish :use_exported_private_key, true
+
+    publish :account, 'motion-sparkle-sandbox.sample-app'
+    publish :full_release_notes_url, 'http://example.com/full_release_notes_changed_again.html'
   end
 end

data/spec/appcast_spec.rb

@@ -0,0 +1,60 @@
+# frozen_string_literal: true
+
+require File.expand_path('spec_utils', __dir__)
+
+describe 'Appcast' do
+  describe 'arguments' do
+    before do
+      @appcast = Motion::Project::Sparkle::Appcast.new(nil)
+    end
+
+    it 'defaults to empty' do
+      args = @appcast.prepare_args
+
+      expect(args).to eq ['--account=ed25519']
+    end
+
+    it 'sets all command line options' do
+      expected_results = [
+        '--account=test1',
+        '--ed-key-file=test2',
+        '--download-url-prefix=test3',
+        '--release-notes-url-prefix=test4',
+        '--full-release-notes-url=test5',
+        '--link=test6',
+        '--versions=test7',
+        '--maximum-deltas=test8',
+        '--delta-compression=test9',
+        '--delta-compression-level=test10',
+        '--channel=test11',
+        '--major-version=test12',
+        '--ignore-skipped-upgrades-below-version=test13',
+        '--phased-rollout-interval=test14',
+        '--critical-update-version=test15',
+        '--informational-update-versions=test16',
+        '-o=test17'
+      ]
+
+      @appcast.process_option(:account, 'test1')
+      @appcast.process_option(:private_ed_key_file, 'test2')
+      @appcast.process_option(:download_url_prefix, 'test3')
+      @appcast.process_option(:release_notes_url_prefix, 'test4')
+      @appcast.process_option(:full_release_notes_url, 'test5')
+      @appcast.process_option(:link, 'test6')
+      @appcast.process_option(:versions, 'test7')
+      @appcast.process_option(:maximum_deltas, 'test8')
+      @appcast.process_option(:delta_compression, 'test9')
+      @appcast.process_option(:delta_compression_level, 'test10')
+      @appcast.process_option(:channel, 'test11')
+      @appcast.process_option(:major_version, 'test12')
+      @appcast.process_option(:ignore_skipped_upgrades_below_version, 'test13')
+      @appcast.process_option(:phased_rollout_interval, 'test14')
+      @appcast.process_option(:critical_update_version, 'test15')
+      @appcast.process_option(:informational_update_versions, 'test16')
+      @appcast.process_option(:output_path, 'test17')
+      args = @appcast.prepare_args
+
+      expect(args).to match_array expected_results
+    end
+  end
+end

data/spec/setup_spec.rb

@@ -2,66 +2,41 @@
 
 require File.expand_path('spec_utils', __dir__)
 
-module Motion
-  module Project
-    class Config
-      attr_writer :project_dir
-    end
-  end
-end
-
 describe 'Sparkle setup' do
   before(:all) do
-    SpecUtils::TemporaryDirectory.teardown
-    SpecUtils::TemporaryDirectory.setup
-
-    FileUtils.mkdir_p("#{SpecUtils::TemporaryDirectory.directory}resources")
-    FileUtils.mkdir_p("#{SpecUtils::TemporaryDirectory.directory}vendor")
-    FileUtils.touch("#{SpecUtils::TemporaryDirectory.directory}.gitignore")
-  end
-
-  context 'something' do
-    before do
-      @config = App.config
-      @config.project_dir = SpecUtils::TemporaryDirectory.directory.to_s
-      @config.instance_eval do
-        pods do
-          pod 'Sparkle', POD_VERSION
-        end
-
-        sparkle do
-          release :base_url, 'http://example.com/'
-          # release :public_key, 'public_key.pem'
-          publish :public_key, '<YOUR-EDDSA-PUBLIC-KEY>'
-          release :version, '1.0'
-
-          # Optional config options
-          release :feed_base_url, 'http://rss.example.com/'
-          release :feed_filename, 'example.xml'
-          release :notes_base_url, 'http://www.example.com/'
-          release :notes_filename, 'example.html'
-          release :package_base_url, 'http://download.example.com/'
-          release :package_filename, 'example.zip'
-          # publish :use_exported_private_key, true
-        end
+    @config = App.config
+    @config.project_dir = SpecUtils::TemporaryDirectory.directory.to_s
+    @config.instance_eval do
+      sparkle do
+        release :base_url, 'http://example.com/'
+        # release :public_key, 'public_key.pem'
+        publish :public_key, '<YOUR-EDDSA-PUBLIC-KEY>'
+        release :version, '1.0'
+
+        # Optional config options
+        release :feed_base_url, 'http://rss.example.com/'
+        release :feed_filename, 'example.xml'
+        release :notes_base_url, 'http://www.example.com/'
+        release :notes_filename, 'example.html'
+        release :package_base_url, 'http://download.example.com/'
+        release :package_filename, 'example.zip'
+        # publish :use_exported_private_key, true
       end
-
-      Rake::Task['pod:install'].invoke
-      Rake::Task['sparkle:setup'].invoke
-      # Rake::Task['sparkle:setup_certificates'].invoke
-    end
-
-    it 'should create private certificate' do
-      expect(File.exist?(@config.sparkle.private_key_path.to_s)).to be_truthy
     end
 
-    it 'should create public certificate' do
-      expect(File.exist?(@config.sparkle.public_key_path.to_s)).to be_truthy
-    end
+    Rake::Task['sparkle:setup'].invoke
+  end
 
-    it 'should add files to gitignore' do
-      a = `cat .gitignore`
-      expect(a.strip).not_to eq ''
-    end
+  # it 'should create private certificate' do
+  #   expect(File.exist?(@config.sparkle.private_key_path.to_s)).to be_truthy
+  # end
+  #
+  # it 'should create public certificate' do
+  #   expect(File.exist?(@config.sparkle.public_key_path.to_s)).to be_truthy
+  # end
+
+  it 'should add files to gitignore' do
+    a = `cat .gitignore`
+    expect(a.strip).not_to eq ''
   end
 end

data/spec/sparkle_spec.rb

@@ -2,37 +2,20 @@
 
 require File.expand_path('spec_utils', __dir__)
 
-module Motion
-  module Project
-    class Config
-      attr_writer :project_dir
-    end
-  end
-end
-
 # rubocop:disable Metrics/BlockLength
 describe 'motion-sparkle-sandbox' do
   before(:all) do
-    SpecUtils::TemporaryDirectory.teardown
-    SpecUtils::TemporaryDirectory.setup
-
-    FileUtils.mkdir_p("#{SpecUtils::TemporaryDirectory.directory}resources")
-    FileUtils.mkdir_p("#{SpecUtils::TemporaryDirectory.directory}vendor")
-    FileUtils.touch("#{SpecUtils::TemporaryDirectory.directory}.gitignore")
-  end
-
-  context 'configuration' do
-    before do
-      @config = App.config
-      @config.sparkle = nil
-      @config.project_dir = SpecUtils::TemporaryDirectory.directory.to_s
-      @config.instance_eval do
-        sparkle do
-          release :base_url, 'http://example.com/'
-        end
+    @config = App.config
+    @config.sparkle = nil
+    @config.project_dir = SpecUtils::TemporaryDirectory.directory.to_s
+    @config.instance_eval do
+      sparkle do
+        release :base_url, 'http://example.com/'
       end
     end
+  end
 
+  context 'configuration' do
     describe 'base url' do
       it 'base url should be set correctly' do
         expect(@config.sparkle.appcast.base_url).to eq 'http://example.com/'
@@ -46,6 +29,7 @@ describe 'motion-sparkle-sandbox' do
 
       it 'uses feed_base_url' do
         @config.sparkle.publish(:feed_base_url, 'http://rss.example.com/')
+        @config.sparkle.after_initialize
 
         expect(@config.info_plist['SUFeedURL']).to eq 'http://rss.example.com/releases.xml'
       end
@@ -53,6 +37,7 @@ describe 'motion-sparkle-sandbox' do
       it 'uses feed_filename' do
         @config.sparkle.publish(:feed_base_url, 'http://rss.example.com/')
         @config.sparkle.publish(:feed_filename, 'example.xml')
+        @config.sparkle.after_initialize
 
         expect(@config.info_plist['SUFeedURL']).to eq 'http://rss.example.com/example.xml'
       end
@@ -115,22 +100,10 @@ describe 'motion-sparkle-sandbox' do
   end
 
   context 'cocoapod' do
-    before do
-      @config = App.config
-      @config.project_dir = SpecUtils::TemporaryDirectory.directory.to_s
-      @config.instance_eval do
-        pods do
-          pod 'Sparkle', POD_VERSION
-        end
-      end
-
-      Rake::Task['pod:install'].invoke
-    end
-
     it 'Sparkle framework pod should be embedded' do
       sparkle_framework_path = 'vendor/Pods/Sparkle/Sparkle.framework'
-      @config.pods.pods_libraries
 
+      @config.pods.pods_libraries
       expect(@config.embedded_frameworks.first.end_with?(sparkle_framework_path)).to be_truthy
     end
   end

data/spec/spec_helper.rb

@@ -97,4 +97,12 @@ RSpec.configure do |config|
   #   # test failures related to randomization by passing the same `--seed` value
   #   # as the one that triggered the failure.
   #   Kernel.srand config.seed
+
+  config.before(:suite) do
+    SpecUtils::SparkleSetup.initial_install
+  end
+
+  config.after(:suite) do
+    SpecUtils::SparkleSetup.final_deinstall
+  end
 end

data/spec/spec_utils.rb

@@ -13,7 +13,43 @@ $:.unshift("#{ROOT}lib".to_s)
 require 'motion/project/template/osx'
 require 'motion-sparkle-sandbox'
 
+# necessary for us to be able to overwrite the `project_dir`
+module Motion
+  module Project
+    class Config
+      attr_writer :project_dir
+    end
+  end
+end
+
 module SpecUtils
+  module SparkleSetup
+    # run from a before(:suite)
+    def self.initial_install
+      SpecUtils::TemporaryDirectory.setup
+
+      FileUtils.mkdir_p("#{SpecUtils::TemporaryDirectory.directory}/resources")
+      FileUtils.mkdir_p("#{SpecUtils::TemporaryDirectory.directory}/vendor")
+      FileUtils.touch("#{SpecUtils::TemporaryDirectory.directory}/.gitignore")
+
+      @config = App.config
+      @config.sparkle = nil
+      @config.project_dir = SpecUtils::TemporaryDirectory.directory.to_s
+      @config.instance_eval do
+        pods do
+          pod 'Sparkle', POD_VERSION
+        end
+      end
+
+      Rake::Task['pod:install'].invoke
+    end
+
+    # run from an after(:suite)
+    def self.final_deinstall
+      SpecUtils::TemporaryDirectory.teardown
+    end
+  end
+
   module TemporaryDirectory
     TEMPORARY_DIRECTORY = ROOT + 'tmp' # rubocop:disable Style/StringConcatenation
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: motion-sparkle-sandbox
 version: !ruby/object:Gem::Version
-  version: 2.0.1
+  version: '2.2'
 platform: ruby
 authors:
 - Brett Walker
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-05-27 00:00:00.000000000 Z
+date: 2022-07-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: motion-cocoapods
@@ -95,6 +95,7 @@ files:
 - sample-app/resources/Assets.xcassets/Contents.json
 - sample-app/resources/Credits.rtf
 - sample-app/spec/main_spec.rb
+- spec/appcast_spec.rb
 - spec/setup_spec.rb
 - spec/sparkle_spec.rb
 - spec/spec_helper.rb
@@ -124,6 +125,7 @@ signing_key:
 specification_version: 4
 summary: Sparkle (sandboxed) integration for Rubymotion projects
 test_files:
+- spec/appcast_spec.rb
 - spec/spec_helper.rb
 - spec/sparkle_spec.rb
 - spec/setup_spec.rb