lokalise_manager 1.0.0 → 2.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f0640f0599d5b4967f79d2ced69f63d7cb7edde3e2a3a49336d08d17e5fb4c0f
-  data.tar.gz: 0766d8cb7f6b478e427f8ef7dd1409d86072a74a4606b026eaa718f6fe4761b5
+  metadata.gz: 0f9ca0fe8caf151e84dcc09f29368bc61b5bc3cdb4341ef5c1b21040b764a17a
+  data.tar.gz: 8c9194a836a1261638f2b22d8d2d7553bfbe605325911cb0c0ecef9332e9084c
 SHA512:
-  metadata.gz: 38a3adfcc0bf4a78f595547cdc3ba2962a4122d8bbeaa7e132d47dc4e76992711e653710a802e395fafcb0c316743607876a10f0914c2633472994891fd4850e
-  data.tar.gz: 98b2edd83de044615db56ee71d090afa03fcfc7793c55854e322ab466996e0eb46c74f3b22dbdbd44e570f31746d40394912ba39c29dd17337b17e08d0417761
+  metadata.gz: 7c4b7131ea9d6b60ff4094d0e34c92484818eb3c608a016932426c83bbcab4a06f4c820399f2fa70cc2a78d9a60a73dcf442ec9ba082bdc095ab539b467e1677
+  data.tar.gz: b98c062b4ecc030c37fa1f1ad95126209749c99ec7aba72b616fad380819d55552dc6d4b2a611b94471f79b3748855eefbaab3c9397ab49e9f8e19a6a6afb1ab

data/CHANGELOG.md

@@ -1,5 +1,30 @@
 # Changelog
 
+## 2.0.0 (27-Jan-22)
+
+* `export!` method is now taking advantage of multi-threading (as Lokalise API allows to send requests in parallel since January 2022)
+* Test with Ruby 3.1.0
+* Other minor fixes
+
+## 1.2.1 (26-Nov-21)
+
+* Use refinements instead of monkey patching to add hash methods
+* Don't use `OpenStruct` anymore to store config opts
+* Minor fixes
+
+## 1.2.0 (26-Oct-21)
+
+* Add a new option `:silent_mode` which is `false` by default. When silent mode is enabled, no debug info will be printed out to `$stdout`. The only exception are the "safe mode" messages — you'll still be prompted to continue if the target directory is not empty.
+* Use `#deep_merge` instead of a simple merge when processing options.
+
+## 1.1.0 (25-Oct-21)
+
+* Add a new option `:use_oauth2_token` which is `false` by default. When enabled, you'll be able to provide a token obtained via [OAuth 2 flow](https://docs.lokalise.com/en/articles/5574713-oauth-2) rather than generated via Lokalise profile. The token should still be provided via the `:api_token` option:
+
+```ruby
+importer = LokaliseManager.importer api_token: 'TOKEN_VIA_OAUTH2', project_id: '123.abc', use_oauth2_token: true
+```
+
 ## 1.0.0 (14-Oct-21)
 
 * Initial release

data/README.md

@@ -1,7 +1,7 @@
 # LokaliseManager
 
 ![Gem](https://img.shields.io/gem/v/lokalise_manager)
-[![Build Status](https://travis-ci.com/bodrovis/lokalise_manager.svg?branch=master)](https://travis-ci.com/github/bodrovis/lokalise_manager)
+![CI](https://github.com/bodrovis/lokalise_manager/actions/workflows/ci.yml/badge.svg)
 [![Test Coverage](https://codecov.io/gh/bodrovis/lokalise_manager/graph/badge.svg)](https://codecov.io/gh/bodrovis/lokalise_manager)
 ![Downloads total](https://img.shields.io/gem/dt/lokalise_manager)
 
@@ -15,6 +15,8 @@ If you are looking for a Rails integration, please check [lokalise_rails](https:
 
 This gem requires Ruby 2.5+. You will also need to [setup a Lokalise account](https://app.lokalise.com/signup) and create a [translation project](https://docs.lokalise.com/en/articles/1400460-projects). Finally, you will need to generate a [read/write API token](https://docs.lokalise.com/en/articles/1929556-api-tokens) at your Lokalise profile.
 
+Alternatively, you can utilize a token obtained via OAuth 2 flow. When using such a token, you'll have to set `:use_oauth2_token` option to `true` (see below).
+
 ### Installation
 
 Add the gem to your `Gemfile`:
@@ -87,10 +89,12 @@ Please don't forget that Lokalise API has rate limiting and you cannot send more
 ### Common config
 
 * `api_token` (`string`, required) — Lokalise API token with read/write permissions.
+* `use_oauth2_token` (`boolean`) — whether you would like to use a token obtained via [OAuth 2 flow](https://docs.lokalise.com/en/articles/5574713-oauth-2). Defaults to `false`.
 * `project_id` (`string`, required) — Lokalise project ID. You must have import/export permissions in the specified project.
 * `locales_path` (`string`) — path to the directory with your translation files. Defaults to `"#{Dir.getwd}/locales"`.
 * `branch` (`string`) — Lokalise project branch to use. Defaults to `""` (no branch is provided).
 * `timeouts` (`hash`) — set [request timeouts for the Lokalise API client](https://lokalise.github.io/ruby-lokalise-api/additional_info/customization#setting-timeouts). By default, requests have no timeouts: `{open_timeout: nil, timeout: nil}`. Both values are in seconds.
+* `silent_mode` (`boolean`) — whether you would like to output debugging information to `$stdout`. By default, after a task is performed, a short notification message will be printed out to the terminal. When set to `false`, notifications won't be printed. Please note that currently `import_safe_mode` has higher priority. Even if you enable `silent_mode`, and the `import_safe_mode` is enabled as well, you will be prompted to confirm the import operation if the target directory is not empty.
 
 ### Import config
 
@@ -169,6 +173,10 @@ If your translation files are not in YAML format, you will need to adjust the fo
 * `translations_converter` (`lambda` or `proc`) — converts translations data to a proper format before saving them to a translation file. Defaults to `->(raw_data) { raw_data.to_yaml }`. In the simplest case you may just return the data back, for example `-> (raw_data) { raw_data }`.
 * `lang_iso_inferer` (`lambda` or `proc`) — infers language ISO code based on the translation file data before uploading it to Lokalise. Defaults to `->(data) { YAML.safe_load(data)&.keys&.first }`.
 
+### Customizing JSON parser and network adapter
+
+JSON parser and network adapter utilized by lokalise_manager can be customized as well. Please check [ruby-lokalise-api doc](https://lokalise.github.io/ruby-lokalise-api/additional_info/customization) to learn more.
+
 ## Providing config options
 
 ### Per-client
@@ -263,4 +271,4 @@ importer.import!
 
 ## License
 
-Copyright (c) [Lokalise team](http://lokalise.com), [Ilya Bodrov](http://bodrovis.tech). License type is [MIT](https://github.com/bodrovis/lokalise_manager/blob/master/LICENSE).
+Copyright (c) [Lokalise team](http://lokalise.com), [Ilya Bodrov](http://bodrovis.tech). License type is [MIT](https://github.com/bodrovis/lokalise_manager/blob/master/LICENSE).

data/lib/lokalise_manager/global_config.rb

@@ -7,13 +7,24 @@ module LokaliseManager
       attr_writer :import_opts, :import_safe_mode, :export_opts, :locales_path,
                   :file_ext_regexp, :skip_file_export, :branch, :timeouts,
                   :translations_loader, :translations_converter, :lang_iso_inferer,
-                  :max_retries_export, :max_retries_import
+                  :max_retries_export, :max_retries_import, :use_oauth2_token, :silent_mode
 
       # Main interface to provide configuration options
       def config
         yield self
       end
 
+      # When enabled, won't print any debugging info to $stdout
+      def silent_mode
+        @silent_mode || false
+      end
+
+      # When enabled, will use OAuth 2 Lokalise client and will require to provide a token obtained via OAuth 2 flow
+      # rather than via Lokalise profile
+      def use_oauth2_token
+        @use_oauth2_token || false
+      end
+
       # Full path to directory with translation files
       def locales_path
         @locales_path || "#{Dir.getwd}/locales"

data/lib/lokalise_manager/task_definitions/base.rb

@@ -2,11 +2,12 @@
 
 require 'ruby-lokalise-api'
 require 'pathname'
-require 'ostruct'
 
 module LokaliseManager
   module TaskDefinitions
     class Base
+      using LokaliseManager::Utils::HashUtils
+
       attr_accessor :config
 
       # Creates a new importer or exporter. It accepts custom config and merges it
@@ -23,19 +24,27 @@ module LokaliseManager
           opts[reader.to_sym] = global_config.send(reader)
         end
 
-        @config = OpenStruct.new primary_opts.merge(custom_opts)
+        all_opts = primary_opts.deep_merge(custom_opts)
+
+        config_klass = Struct.new(*all_opts.keys, keyword_init: true)
+
+        @config = config_klass.new all_opts
       end
 
       # Creates a Lokalise API client
       #
       # @return [Lokalise::Client]
       def api_client
-        @api_client ||= ::Lokalise.client config.api_token, {enable_compression: true}.merge(config.timeouts)
+        client_opts = [config.api_token, {enable_compression: true}.merge(config.timeouts)]
+        client_method = config.use_oauth2_token ? :oauth_client : :client
+
+        @api_client = ::Lokalise.send(client_method, *client_opts)
       end
 
       # Resets API client
       def reset_api_client!
-        Lokalise.reset_client!
+        ::Lokalise.reset_client!
+        ::Lokalise.reset_oauth_client!
         @api_client = nil
       end
 

data/lib/lokalise_manager/task_definitions/exporter.rb

@@ -5,6 +5,7 @@ require 'base64'
 module LokaliseManager
   module TaskDefinitions
     class Exporter < Base
+      using LokaliseManager::Utils::ArrayUtils
       # Performs translation file export to Lokalise and returns an array of queued processes
       #
       # @return [Array]
@@ -12,31 +13,48 @@ module LokaliseManager
         check_options_errors!
 
         queued_processes = []
-        each_file do |full_path, relative_path|
-          queued_processes << do_upload(full_path, relative_path)
-        rescue StandardError => e
-          raise e.class, "Error while trying to upload #{full_path}: #{e.message}"
+
+        all_files.in_groups_of(6) do |files_group|
+          parallel_upload(files_group).each do |thr|
+            raise_on_fail thr
+
+            queued_processes.push thr[:process]
+          end
         end
 
-        $stdout.print 'Task complete!'
+        $stdout.print('Task complete!') unless config.silent_mode
 
         queued_processes
       end
 
       private
 
+      def parallel_upload(files_group)
+        files_group.compact.map do |file_data|
+          do_upload(*file_data)
+        end.map(&:value)
+      end
+
+      def raise_on_fail(thread)
+        raise thread[:error].class, "Error while trying to upload #{thread[:path]}: #{thread[:error].message}" if thread[:status] == :fail
+      end
+
       # Performs the actual file uploading to Lokalise. If the API rate limit is exceeed,
       # applies exponential backoff
       def do_upload(f_path, r_path)
-        with_exp_backoff(config.max_retries_export) do
-          api_client.upload_file project_id_with_branch, opts(f_path, r_path)
+        Thread.new do
+          process = with_exp_backoff(config.max_retries_export) do
+            api_client.upload_file project_id_with_branch, opts(f_path, r_path)
+          end
+          {status: :ok, process: process}
+        rescue StandardError => e
+          {status: :fail, path: f_path, error: e}
         end
       end
 
-      # Processes each translation file in the specified directory
-      def each_file
-        return unless block_given?
-
+      # Gets translation files from the specified directory
+      def all_files
+        files = []
         loc_path = config.locales_path
         Dir["#{loc_path}/**/*"].sort.each do |f|
           full_path = Pathname.new f
@@ -45,8 +63,9 @@ module LokaliseManager
 
           relative_path = full_path.relative_path_from Pathname.new(loc_path)
 
-          yield full_path, relative_path
+          files << [full_path, relative_path]
         end
+        files
       end
 
       # Generates export options

data/lib/lokalise_manager/task_definitions/importer.rb

@@ -14,13 +14,13 @@ module LokaliseManager
         check_options_errors!
 
         unless proceed_when_safe_mode?
-          $stdout.print 'Task cancelled!'
+          $stdout.print('Task cancelled!') unless config.silent_mode
           return false
         end
 
         open_and_process_zip download_files['bundle_url']
 
-        $stdout.print 'Task complete!'
+        $stdout.print('Task complete!') unless config.silent_mode
         true
       end
 

data/lib/lokalise_manager/utils/array_utils.rb

@@ -0,0 +1,27 @@
+# frozen_string_literal: true
+
+# Taken from https://github.com/rails/rails/blob/6bfc637659248df5d6719a86d2981b52662d9b50/activesupport/lib/active_support/core_ext/array/grouping.rb
+
+module LokaliseManager
+  module Utils
+    module ArrayUtils
+      refine Array do
+        def in_groups_of(number, fill_with = nil, &block)
+          if number.to_i <= 0
+            raise ArgumentError,
+                  "Group size must be a positive integer, was #{number.inspect}"
+          end
+
+          if fill_with == false
+            collection = self
+          else
+            padding = (number - (size % number)) % number
+            collection = dup.concat(Array.new(padding, fill_with))
+          end
+
+          collection.each_slice(number, &block)
+        end
+      end
+    end
+  end
+end

data/lib/lokalise_manager/utils/hash_utils.rb

@@ -0,0 +1,28 @@
+# frozen_string_literal: true
+
+# Taken from https://github.com/rails/rails/blob/83217025a171593547d1268651b446d3533e2019/activesupport/lib/active_support/core_ext/hash/deep_merge.rb
+
+module LokaliseManager
+  module Utils
+    module HashUtils
+      refine Hash do
+        def deep_merge(other_hash, &block)
+          dup.deep_merge!(other_hash, &block)
+        end
+
+        # Same as +deep_merge+, but modifies +self+.
+        def deep_merge!(other_hash, &block)
+          merge!(other_hash) do |key, this_val, other_val|
+            if this_val.is_a?(Hash) && other_val.is_a?(Hash)
+              this_val.deep_merge(other_val, &block)
+            elsif block
+              yield(key, this_val, other_val)
+            else
+              other_val
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/lib/lokalise_manager/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module LokaliseManager
-  VERSION = '1.0.0'
+  VERSION = '2.0.0'
 end

data/lib/lokalise_manager.rb

@@ -2,6 +2,9 @@
 
 require 'yaml'
 
+require 'lokalise_manager/utils/hash_utils'
+require 'lokalise_manager/utils/array_utils'
+
 require 'lokalise_manager/version'
 require 'lokalise_manager/error'
 require 'lokalise_manager/global_config'

data/lokalise_manager.gemspec

@@ -32,7 +32,10 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency 'rspec', '~> 3.6'
   spec.add_development_dependency 'rubocop', '~> 1.0'
   spec.add_development_dependency 'rubocop-performance', '~> 1.5'
-  spec.add_development_dependency 'rubocop-rspec', '~> 2.5.0'
+  spec.add_development_dependency 'rubocop-rspec', '~> 2.6'
   spec.add_development_dependency 'simplecov', '~> 0.16'
   spec.add_development_dependency 'vcr', '~> 6.0'
+  spec.metadata = {
+    'rubygems_mfa_required' => 'true'
+  }
 end

data/spec/lib/lokalise_manager/global_config_spec.rb

@@ -14,6 +14,16 @@ describe LokaliseManager::GlobalConfig do
     fake_class.project_id = '123.abc'
   end
 
+  it 'is possible to set silent_mode' do
+    allow(fake_class).to receive(:silent_mode=).with(true)
+    fake_class.silent_mode = true
+  end
+
+  it 'is possible to set use_oauth2_token' do
+    allow(fake_class).to receive(:use_oauth2_token=).with(true)
+    fake_class.use_oauth2_token = true
+  end
+
   it 'is possible to set file_ext_regexp' do
     allow(fake_class).to receive(:file_ext_regexp=).with(Regexp.new('.*'))
     fake_class.file_ext_regexp = Regexp.new('.*')

data/spec/lib/lokalise_manager/task_definitions/base_spec.rb

@@ -24,7 +24,7 @@ describe LokaliseManager::TaskDefinitions::Base do
   specify '.reset_client!' do
     expect(described_object.api_client).to be_an_instance_of(Lokalise::Client)
     described_object.reset_api_client!
-    current_client = described_object.instance_variable_get '@api_client'
+    current_client = described_object.instance_variable_get :@api_client
     expect(current_client).to be_nil
   end
 
@@ -94,9 +94,20 @@ describe LokaliseManager::TaskDefinitions::Base do
                                                                         timeout: 500
                                                                       })
 
-      expect(described_object.api_client).to be_an_instance_of(Lokalise::Client)
-      expect(described_object.api_client.open_timeout).to eq(100)
-      expect(described_object.api_client.timeout).to eq(500)
+      client = described_object.api_client
+      expect(client).to be_an_instance_of(Lokalise::Client)
+      expect(client).not_to be_an_instance_of(Lokalise::OAuthClient)
+      expect(client.open_timeout).to eq(100)
+      expect(client.timeout).to eq(500)
+    end
+
+    it 'uses .oauth_client when the use_oauth2_token is true' do
+      allow(described_object.config).to receive(:use_oauth2_token).and_return(true)
+
+      client = described_object.api_client
+
+      expect(client).to be_an_instance_of(Lokalise::OAuthClient)
+      expect(client).not_to be_an_instance_of(Lokalise::Client)
     end
   end
 end

data/spec/lib/lokalise_manager/task_definitions/exporter_spec.rb

@@ -24,26 +24,30 @@ describe LokaliseManager::TaskDefinitions::Exporter do
 
     describe '.export!' do
       it 'sends a proper API request and handles rate limiting' do
-        process = VCR.use_cassette('upload_files_multiple') do
-          described_object.export!
-        end.first
+        process = nil
+
+        VCR.use_cassette('upload_files_multiple') do
+          expect(-> { process = described_object.export!.first }).to output(/complete!/).to_stdout
+        end
 
         expect(process.project_id).to eq(project_id)
         expect(process.status).to eq('queued')
       end
 
       it 'handles too many requests' do
+        allow(described_object.config).to receive(:max_retries_export).and_return(1)
         allow(described_object).to receive(:sleep).and_return(0)
 
         fake_client = instance_double('Lokalise::Client')
+        allow(fake_client).to receive(:token).with(any_args).and_return('fake_token')
         allow(fake_client).to receive(:upload_file).with(any_args).and_raise(Lokalise::Error::TooManyRequests)
         allow(described_object).to receive(:api_client).and_return(fake_client)
 
-        expect(-> { described_object.export! }).to raise_error(Lokalise::Error::TooManyRequests, /Gave up after 2 retries/i)
+        expect(-> { described_object.export! }).to raise_error(Lokalise::Error::TooManyRequests, /Gave up after 1 retries/i)
 
-        expect(described_object).to have_received(:sleep).exactly(2).times
-        expect(described_object).to have_received(:api_client).exactly(3).times
-        expect(fake_client).to have_received(:upload_file).exactly(3).times
+        expect(described_object).to have_received(:sleep).exactly(6).times
+        expect(described_object).to have_received(:api_client).at_least(12).times
+        expect(fake_client).to have_received(:upload_file).exactly(12).times
       end
     end
   end
@@ -58,6 +62,19 @@ describe LokaliseManager::TaskDefinitions::Exporter do
     end
 
     describe '.export!' do
+      it 'sends a proper API request but does not output anything when silent_mode is enabled' do
+        allow(described_object.config).to receive(:silent_mode).and_return(true)
+
+        process = nil
+
+        VCR.use_cassette('upload_files') do
+          expect(-> { process = described_object.export!.first }).not_to output(/complete!/).to_stdout
+        end
+
+        expect(process.status).to eq('queued')
+        expect(described_object.config).to have_received(:silent_mode).at_most(1).times
+      end
+
       it 'sends a proper API request' do
         process = VCR.use_cassette('upload_files') do
           described_object.export!
@@ -93,9 +110,9 @@ describe LokaliseManager::TaskDefinitions::Exporter do
       end
     end
 
-    describe '.each_file' do
+    describe '#all_files' do
       it 'yield proper arguments' do
-        expect { |b| described_object.send(:each_file, &b) }.to yield_with_args(
+        expect(described_object.send(:all_files).first).to include(
           Pathname.new(path),
           Pathname.new(relative_name)
         )
@@ -165,30 +182,29 @@ describe LokaliseManager::TaskDefinitions::Exporter do
       end
     end
 
-    describe '.each_file' do
-      it 'yields every translation file' do
-        expect { |b| described_object.send(:each_file, &b) }.to yield_successive_args(
-          [
-            Pathname.new(path),
-            Pathname.new(relative_name)
-          ],
-          [
-            Pathname.new(path_ru),
-            Pathname.new(filename_ru)
-          ]
+    describe '#each_file' do
+      it 'returns all files' do
+        files = described_object.send(:all_files)
+        expect(files[0]).to include(
+          Pathname.new(path),
+          Pathname.new(relative_name)
+        )
+        expect(files[1]).to include(
+          Pathname.new(path_ru),
+          Pathname.new(filename_ru)
         )
       end
 
-      it 'does not yield files that have to be skipped' do
+      it 'does not return files that have to be skipped' do
         allow(described_object.config).to receive(:skip_file_export).twice.and_return(
           ->(f) { f.split[1].to_s.include?('ru') }
         )
-        expect { |b| described_object.send(:each_file, &b) }.to yield_successive_args(
-          [
-            Pathname.new(path),
-            Pathname.new(relative_name)
-          ]
+        files = described_object.send(:all_files)
+        expect(files[0]).to include(
+          Pathname.new(path),
+          Pathname.new(relative_name)
         )
+        expect(files.count).to eq(1)
 
         expect(described_object.config).to have_received(:skip_file_export).twice
       end

data/spec/lib/lokalise_manager/task_definitions/importer_spec.rb

@@ -9,12 +9,16 @@ describe LokaliseManager::TaskDefinitions::Importer do
   let(:loc_path) { described_object.config.locales_path }
   let(:project_id) { ENV['LOKALISE_PROJECT_ID'] }
   let(:local_trans) { "#{Dir.getwd}/spec/fixtures/trans.zip" }
-  let(:faulty_trans) { "#{Dir.getwd}/spec/fixtures/faulty_trans.zip" }
 
-  describe '.open_and_process_zip' do
+  describe '#open_and_process_zip' do
     it 're-raises errors during file processing' do
-      expect(-> { described_object.send(:open_and_process_zip, faulty_trans) }).
-        to raise_error(Psych::DisallowedClass, /Error when trying to process fail\.yml/)
+      entry = double
+      allow(entry).to receive(:name).and_return('fail.yml')
+      allow(described_object).to receive(:data_from).with(entry).and_raise(EncodingError)
+      expect(-> { described_object.send(:process!, entry) }).
+        to raise_error(EncodingError, /Error when trying to process fail\.yml/)
+
+      expect(described_object).to have_received(:data_from)
     end
 
     it 're-raises errors during file opening' do
@@ -23,7 +27,7 @@ describe LokaliseManager::TaskDefinitions::Importer do
     end
   end
 
-  describe '.download_files' do
+  describe '#download_files' do
     it 'returns a proper download URL' do
       response = VCR.use_cassette('download_files') do
         described_object.send :download_files
@@ -103,15 +107,31 @@ describe LokaliseManager::TaskDefinitions::Importer do
       end
 
       it 'runs import successfully' do
-        result = VCR.use_cassette('import') do
-          described_object.import!
+        result = nil
+
+        VCR.use_cassette('import') do
+          expect(-> { result = described_object.import! }).to output(/complete!/).to_stdout
         end
 
         expect(result).to be true
 
-        expect(count_translations).to eq(8)
-        expect_file_exist loc_path, 'en.yml'
-        expect_file_exist loc_path, 'ru.yml'
+        expect(count_translations).to eq(24)
+        expect_file_exist loc_path, 'en_1.yml'
+        expect_file_exist loc_path, 'ru_2.yml'
+      end
+
+      it 'runs import successfully but does not provide any output when silent_mode is enabled' do
+        allow(described_object.config).to receive(:silent_mode).and_return(true)
+        result = nil
+
+        VCR.use_cassette('import') do
+          expect(-> { result = described_object.import! }).not_to output(/complete!/).to_stdout
+        end
+
+        expect(result).to be true
+        expect_file_exist loc_path, 'en_1.yml'
+        expect_file_exist loc_path, 'ru_2.yml'
+        expect(described_object.config).to have_received(:silent_mode).at_most(1).times
       end
     end
 
@@ -157,9 +177,21 @@ describe LokaliseManager::TaskDefinitions::Importer do
       it 'import halts when a user chooses not to proceed' do
         allow(safe_mode_obj).to receive(:download_files).at_most(0).times
         allow($stdin).to receive(:gets).and_return('N')
-        expect(-> { safe_mode_obj.import! }).to output(/is not empty/).to_stdout
+        expect(-> { safe_mode_obj.import! }).to output(/cancelled/).to_stdout
+
+        expect(safe_mode_obj).not_to have_received(:download_files)
+        expect($stdin).to have_received(:gets)
+        expect(count_translations).to eq(1)
+      end
+
+      it 'import halts when a user chooses not to proceed and debug info is not printed out when silent_mode is enabled' do
+        allow(safe_mode_obj.config).to receive(:silent_mode).and_return(true)
+        allow(safe_mode_obj).to receive(:download_files).at_most(0).times
+        allow($stdin).to receive(:gets).and_return('N')
+        expect(-> { safe_mode_obj.import! }).not_to output(/cancelled/).to_stdout
 
         expect(safe_mode_obj).not_to have_received(:download_files)
+        expect(safe_mode_obj.config).to have_received(:silent_mode)
         expect($stdin).to have_received(:gets)
         expect(count_translations).to eq(1)
       end

data/spec/lib/utils/array_utils_spec.rb

@@ -0,0 +1,18 @@
+# frozen_string_literal: true
+
+describe LokaliseManager::Utils::ArrayUtils do
+  using described_class
+  let(:arr) { (1..8).to_a }
+
+  describe '#in_groups_of' do
+    it 'raises an exception when the number is less than 1' do
+      expect(-> { arr.in_groups_of(-1) }).to raise_error(ArgumentError)
+    end
+
+    it 'uses collection itself if fill_with is false' do
+      enum = arr.in_groups_of(5, false)
+      enum.next
+      expect(enum.next.count).to eq(3)
+    end
+  end
+end

data/spec/lib/utils/hash_utils_spec.rb

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+
+describe LokaliseManager::Utils::HashUtils do
+  using described_class
+  let(:h1) { {a: 100, b: 200, c: {c1: 100}} }
+  let(:h2) { {b: 250, c: {c1: 200}} }
+
+  specify '#deep_merge' do
+    result = h1.deep_merge(h2) { |_key, this_val, other_val| this_val + other_val }
+    expect(result[:b]).to eq(450)
+    expect(result[:c][:c1]).to eq(300)
+  end
+end

data/spec/support/vcr.rb

@@ -6,6 +6,8 @@ VCR.configure do |c|
   c.ignore_hosts 'codeclimate.com'
   c.hook_into :faraday
   c.cassette_library_dir = File.join(File.dirname(__FILE__), '..', 'fixtures', 'vcr_cassettes')
-  c.filter_sensitive_data('<LOKALISE_TOKEN>') { ENV.fetch('LOKALISE_API_TOKEN') }
+  c.filter_sensitive_data('<LOKALISE_TOKEN>') do |_i|
+    ENV.fetch('LOKALISE_API_TOKEN')
+  end
   c.configure_rspec_metadata!
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: lokalise_manager
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 2.0.0
 platform: ruby
 authors:
 - Ilya Bodrov
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-10-14 00:00:00.000000000 Z
+date: 2022-01-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ruby-lokalise-api
@@ -128,14 +128,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.5.0
+        version: '2.6'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.5.0
+        version: '2.6'
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
@@ -187,6 +187,8 @@ files:
 - lib/lokalise_manager/task_definitions/base.rb
 - lib/lokalise_manager/task_definitions/exporter.rb
 - lib/lokalise_manager/task_definitions/importer.rb
+- lib/lokalise_manager/utils/array_utils.rb
+- lib/lokalise_manager/utils/hash_utils.rb
 - lib/lokalise_manager/version.rb
 - lokalise_manager.gemspec
 - spec/lib/lokalise_manager/global_config_spec.rb
@@ -194,6 +196,8 @@ files:
 - spec/lib/lokalise_manager/task_definitions/exporter_spec.rb
 - spec/lib/lokalise_manager/task_definitions/importer_spec.rb
 - spec/lib/lokalise_manager_spec.rb
+- spec/lib/utils/array_utils_spec.rb
+- spec/lib/utils/hash_utils_spec.rb
 - spec/spec_helper.rb
 - spec/support/file_manager.rb
 - spec/support/spec_addons.rb
@@ -201,7 +205,8 @@ files:
 homepage: https://github.com/bodrovis/lokalise_manager
 licenses:
 - MIT
-metadata: {}
+metadata:
+  rubygems_mfa_required: 'true'
 post_install_message:
 rdoc_options: []
 require_paths:
@@ -217,7 +222,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.26
+rubygems_version: 3.3.5
 signing_key:
 specification_version: 4
 summary: Lokalise integration for Ruby
@@ -227,6 +232,8 @@ test_files:
 - spec/lib/lokalise_manager/task_definitions/exporter_spec.rb
 - spec/lib/lokalise_manager/task_definitions/importer_spec.rb
 - spec/lib/lokalise_manager_spec.rb
+- spec/lib/utils/array_utils_spec.rb
+- spec/lib/utils/hash_utils_spec.rb
 - spec/spec_helper.rb
 - spec/support/file_manager.rb
 - spec/support/spec_addons.rb