lokalise_manager 2.2.0 → 3.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fa7eb812b614ec768710c4f085879dfaf6cf89f72e25b1a777f34af3e9cd8be5
-  data.tar.gz: afdd4c08bf9672302c279cd88b4c4caf63317b1aca98036a5e8c1dad361289f0
+  metadata.gz: f68d621f5413a53e386f1d1f1539e8f824f6f3e90e81c0fb93c71e9e1c271c9a
+  data.tar.gz: ecaf5166531cb24f7fa6162ea11d86f885ead2e911f40543cdd08e717e9731e4
 SHA512:
-  metadata.gz: 5857c35bde158e85bdec5c2e4618ebf1ddc7768f5822d5492109347fa93907434869d23c5053f6c7b98da8016e4e22c40fa8fa4b513c347630aeb18288f8d18d
-  data.tar.gz: 02bbc800a21396c6f106b2c18df5eee2f7247d43fac80da656496d4a8ec6a50c8d9438bb3514d13b57647f29aebfb1caf1e633a79d97eb37e30ae6bbb2634ecd
+  metadata.gz: 659795e4573dab1751fa00b7d23ed43d6c9b276561fc763341b9a2d72dabadb9e8eeb638a7888d4f736b42e49efcb75a627d12c3b5bc63c9a46f369077dacb2c
+  data.tar.gz: f3681cca401d48512786731538697a2dcac4a7f47ab8d7037a6ce9444308dc4ab71ee34220218654dc71823c1d1a0b3945494015f6f822ac089ca87627a92d1d

data/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## 3.0.0 (11-Mar-22)
+
+* **Breaking change**: Require Ruby 2.7 or above
+* **Breaking change (potentially)**: Use ruby-lokalise-api v6. In general, this transition should not affect you if you employ `export!` and `import!` methods only. However, please be aware that ruby-lokalise-api has a few breaking changes [listed in its own changelog](https://lokalise.github.io/ruby-lokalise-api/additional_info/changelog)
+* Use Zeitwerk loader
+* Prettify and update source code
+
 ## 2.2.0 (23-Feb-22)
 
 * Use ruby-lokalise-api v5

data/README.md

@@ -70,7 +70,7 @@ The uploading process is multi-threaded.
 `processes` will contain an array of objects responding to the following methods:
 
 * `#success` — usually returns `true` (to learn more, check documentation for the `:raise_on_export_fail` option below)
-* `#process` — returns an object (an instance of the `Lokalise::Resources::QueuedProcess`) representing a [queued background process](https://lokalise.github.io/ruby-lokalise-api/api/queued-processes) as uploading is done in the background on Lokalise.
+* `#process` — returns an object (an instance of the `RubyLokaliseApi::Resources::QueuedProcess`) representing a [queued background process](https://lokalise.github.io/ruby-lokalise-api/api/queued-processes) as uploading is done in the background on Lokalise.
 * `#path` — returns an instance of the `Pathname` class which represent the file being uploaded.
 
 You can perform periodic checks to read the status of the process. Here's a very simple example:
@@ -139,7 +139,7 @@ importer = LokaliseManager.importer api_token: '1234abc',
 ```
 
 * `import_safe_mode` (`boolean`) — default to `false`. When this option is enabled, the import task will check whether the directory set with `locales_path` is empty or not. If it is not empty, you will be prompted to continue.
-* `max_retries_import` (`integer`) — this option is introduced to properly handle Lokalise API rate limiting. If the HTTP status code 429 (too many requests) has been received, this gem will apply an exponential backoff mechanism with a very simple formula: `2 ** retries`. If the maximum number of retries has been reached, a `Lokalise::Error::TooManyRequests` exception will be raised and the operation will be halted.
+* `max_retries_import` (`integer`) — this option is introduced to properly handle Lokalise API rate limiting. If the HTTP status code 429 (too many requests) has been received, this gem will apply an exponential backoff mechanism with a very simple formula: `2 ** retries`. If the maximum number of retries has been reached, a `RubyLokaliseApi::Error::TooManyRequests` exception will be raised and the operation will be halted.
 
 ### Export config
 
@@ -171,7 +171,7 @@ In this case the `export_opts` will have `detect_icu_plurals` set to `true` and
 c.skip_file_export = ->(file) { f.split[1].to_s.include?('fr') }
 ```
 
-* `max_retries_export` (`integer`) — this option is introduced to properly handle Lokalise API rate limiting. If the HTTP status code 429 (too many requests) has been received, LokaliseManager will apply an exponential backoff mechanism with a very simple formula: `2 ** retries` (initially `retries` is `0`). If the maximum number of retries has been reached, a `Lokalise::Error::TooManyRequests` exception will be raised and the export operation will be halted. By default, LokaliseManager will make up to `5` retries which potentially means `1 + 2 + 4 + 8 + 16 + 32 = 63` seconds of waiting time. If the `max_retries_export` is less than `1`, LokaliseManager will not perform any retries and give up immediately after receiving error 429.
+* `max_retries_export` (`integer`) — this option is introduced to properly handle Lokalise API rate limiting. If the HTTP status code 429 (too many requests) has been received, LokaliseManager will apply an exponential backoff mechanism with a very simple formula: `2 ** retries` (initially `retries` is `0`). If the maximum number of retries has been reached, a `RubyLokaliseApi::Error::TooManyRequests` exception will be raised and the export operation will be halted. By default, LokaliseManager will make up to `5` retries which potentially means `1 + 2 + 4 + 8 + 16 + 32 = 63` seconds of waiting time. If the `max_retries_export` is less than `1`, LokaliseManager will not perform any retries and give up immediately after receiving error 429.
 * `raise_on_export_fail` (`boolean`) — default is `true`. When this option is enabled, LokaliseManager will re-raise any exceptions that happened during the file uploading. In other words, if any uploading thread raised an exception, your exporting process will exit with an exception. Suppose, you are uploading 12 translation files; these files will be split in 2 groups with 6 files each, and each group will be uploaded in parallel (using threads). However, suppose some exception happens when uploading the first group. By default this exception will be re-raised for the whole process and the script will never try to upload the second group. If you would like to continue uploading even if an exception happened, set the `raise_on_export_fail` to `false`. In this case the `export!` method will return an array with scheduled processes and with information about processes that were not successfully scheduled. This information is represented as an object with three methods: `path` (contains an instance of the `Pathname` class which says which file could not be uploaded), `error` (the actual exception), and `success` (returns `false`). So, you can use the following snippet to check your processes:
 
 ```ruby

data/lib/lokalise_manager/error.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 module LokaliseManager
+  # LokaliseManager error class, subclass of StandardError
   class Error < StandardError
     # Initializes a new Error object
     def initialize(message = '')

data/lib/lokalise_manager/global_config.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 module LokaliseManager
+  # Global configuration options available for LokaliseManager
   class GlobalConfig
     class << self
       attr_accessor :api_token, :project_id

data/lib/lokalise_manager/task_definitions/base.rb

@@ -1,10 +1,11 @@
 # frozen_string_literal: true
 
-require 'ruby-lokalise-api'
+require 'ruby_lokalise_api'
 require 'pathname'
 
 module LokaliseManager
   module TaskDefinitions
+    # Base class for LokaliseManager task definitions that includes common methods and logic
     class Base
       using LokaliseManager::Utils::HashUtils
 
@@ -33,18 +34,18 @@ module LokaliseManager
 
       # Creates a Lokalise API client
       #
-      # @return [Lokalise::Client]
+      # @return [RubyLokaliseApi::Client]
       def api_client
         client_opts = [config.api_token, config.timeouts]
         client_method = config.use_oauth2_token ? :oauth2_client : :client
 
-        @api_client = ::Lokalise.send(client_method, *client_opts)
+        @api_client = ::RubyLokaliseApi.send(client_method, *client_opts)
       end
 
       # Resets API client
       def reset_api_client!
-        ::Lokalise.reset_client!
-        ::Lokalise.reset_oauth2_client!
+        ::RubyLokaliseApi.reset_client!
+        ::RubyLokaliseApi.reset_oauth2_client!
         @api_client = nil
       end
 
@@ -94,7 +95,7 @@ module LokaliseManager
         retries = 0
         begin
           yield
-        rescue Lokalise::Error::TooManyRequests => e
+        rescue RubyLokaliseApi::Error::TooManyRequests => e
           raise(e.class, "Gave up after #{retries} retries") if retries >= max_retries
 
           sleep 2**retries

data/lib/lokalise_manager/task_definitions/exporter.rb

@@ -4,6 +4,7 @@ require 'base64'
 
 module LokaliseManager
   module TaskDefinitions
+    # Exporter class is used when you want to upload translation files from your project to Lokalise
     class Exporter < Base
       # Lokalise allows no more than 6 requests per second
       MAX_THREADS = 6
@@ -38,7 +39,9 @@ module LokaliseManager
       end
 
       def raise_on_fail(thread)
-        raise(thread.error.class, "Error while trying to upload #{thread.path}: #{thread.error.message}") unless thread.success
+        return if thread.success
+
+        raise(thread.error.class, "Error while trying to upload #{thread.path}: #{thread.error.message}")
       end
 
       # Performs the actual file uploading to Lokalise. If the API rate limit is exceeed,
@@ -59,7 +62,7 @@ module LokaliseManager
       # Gets translation files from the specified directory
       def all_files
         loc_path = config.locales_path
-        Dir["#{loc_path}/**/*"].map do |f|
+        Dir["#{loc_path}/**/*"].filter_map do |f|
           full_path = Pathname.new f
 
           next unless file_matches_criteria? full_path
@@ -67,7 +70,7 @@ module LokaliseManager
           relative_path = full_path.relative_path_from Pathname.new(loc_path)
 
           [full_path, relative_path]
-        end.compact
+        end
       end
 
       # Generates export options

data/lib/lokalise_manager/task_definitions/importer.rb

@@ -6,6 +6,7 @@ require 'fileutils'
 
 module LokaliseManager
   module TaskDefinitions
+    # Importer class is used when you want to download translation files from Lokalise into your project
     class Importer < Base
       # Performs translation files import from Lokalise
       #

data/lib/lokalise_manager/utils/hash_utils.rb

@@ -1,11 +1,12 @@
 # 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
+    # Common helper methods for hashes
     module HashUtils
       refine Hash do
+        # Deeply merges two hashes
+        # Taken from https://github.com/rails/rails/blob/83217025a171593547d1268651b446d3533e2019/activesupport/lib/active_support/core_ext/hash/deep_merge.rb
         def deep_merge(other_hash, &block)
           dup.deep_merge!(other_hash, &block)
         end

data/lib/lokalise_manager/version.rb

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

data/lib/lokalise_manager.rb

@@ -1,16 +1,21 @@
 # frozen_string_literal: true
 
+require 'zeitwerk'
 require 'yaml'
 
-require 'lokalise_manager/utils/hash_utils'
-
-require 'lokalise_manager/version'
-require 'lokalise_manager/error'
-require 'lokalise_manager/global_config'
-require 'lokalise_manager/task_definitions/base'
-require 'lokalise_manager/task_definitions/importer'
-require 'lokalise_manager/task_definitions/exporter'
+loader = Zeitwerk::Loader.for_gem
+loader.setup
 
+# LokaliseManager main module that exposes helper methods:
+#
+#   importer = LokaliseManager.importer api_token: '1234abc', project_id: '123.abc'
+#   exporter = LokaliseManager.exporter api_token: '1234abc', project_id: '123.abc'
+#
+# Use the instantiated objects to import or export your translation files:
+#
+#   importer.import!
+#   exporter.export!
+#
 module LokaliseManager
   class << self
     # Initializes a new importer client which is used to download

data/lokalise_manager.gemspec

@@ -12,7 +12,7 @@ Gem::Specification.new do |spec|
   spec.homepage              = 'https://github.com/bodrovis/lokalise_manager'
   spec.license               = 'MIT'
   spec.platform              = Gem::Platform::RUBY
-  spec.required_ruby_version = '>= 2.5.0'
+  spec.required_ruby_version = '>= 2.7'
 
   spec.files = Dir['README.md', 'LICENSE',
                    'CHANGELOG.md', 'lib/**/*.rb',
@@ -23,8 +23,9 @@ Gem::Specification.new do |spec|
   spec.extra_rdoc_files = ['README.md']
   spec.require_paths    = ['lib']
 
-  spec.add_dependency 'ruby-lokalise-api', '~> 5.0'
+  spec.add_dependency 'ruby-lokalise-api', '~> 6.0'
   spec.add_dependency 'rubyzip', '~> 2.3'
+  spec.add_dependency 'zeitwerk', '~> 2.4'
 
   spec.add_development_dependency 'codecov', '~> 0.2'
   spec.add_development_dependency 'dotenv', '~> 2.5'

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

@@ -22,7 +22,7 @@ describe LokaliseManager::TaskDefinitions::Base do
   end
 
   specify '.reset_client!' do
-    expect(described_object.api_client).to be_an_instance_of(Lokalise::Client)
+    expect(described_object.api_client).to be_an_instance_of(RubyLokaliseApi::Client)
     described_object.reset_api_client!
     current_client = described_object.instance_variable_get :@api_client
     expect(current_client).to be_nil
@@ -63,14 +63,18 @@ describe LokaliseManager::TaskDefinitions::Base do
     it 'raises an error when the API key is not set' do
       allow(LokaliseManager::GlobalConfig).to receive(:api_token).and_return(nil)
 
-      expect { described_object.send(:check_options_errors!) }.to raise_error(LokaliseManager::Error, /API token is not set/i)
+      expect do
+        described_object.send(:check_options_errors!)
+      end.to raise_error(LokaliseManager::Error, /API token is not set/i)
 
       expect(LokaliseManager::GlobalConfig).to have_received(:api_token)
     end
 
     it 'returns an error when the project_id is not set' do
       allow_project_id described_object, nil do
-        expect { described_object.send(:check_options_errors!) }.to raise_error(LokaliseManager::Error, /ID is not set/i)
+        expect do
+          described_object.send(:check_options_errors!)
+        end.to raise_error(LokaliseManager::Error, /ID is not set/i)
       end
     end
   end
@@ -95,8 +99,8 @@ describe LokaliseManager::TaskDefinitions::Base do
                                                                       })
 
       client = described_object.api_client
-      expect(client).to be_an_instance_of(Lokalise::Client)
-      expect(client).not_to be_an_instance_of(Lokalise::OAuth2Client)
+      expect(client).to be_an_instance_of(RubyLokaliseApi::Client)
+      expect(client).not_to be_an_instance_of(RubyLokaliseApi::OAuth2Client)
       expect(client.open_timeout).to eq(100)
       expect(client.timeout).to eq(500)
     end
@@ -106,8 +110,8 @@ describe LokaliseManager::TaskDefinitions::Base do
 
       client = described_object.api_client
 
-      expect(client).to be_an_instance_of(Lokalise::OAuth2Client)
-      expect(client).not_to be_an_instance_of(Lokalise::Client)
+      expect(client).to be_an_instance_of(RubyLokaliseApi::OAuth2Client)
+      expect(client).not_to be_an_instance_of(RubyLokaliseApi::Client)
     end
   end
 end

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

@@ -14,129 +14,155 @@ describe LokaliseManager::TaskDefinitions::Exporter do
   end
 
   context 'with many translation files' do
-    before :all do
-      add_translation_files! with_ru: true, additional: 5
-    end
-
-    after :all do
-      rm_translation_files
-    end
-
     describe '.export!' do
-      it 'sends a proper API request and handles rate limiting' do
-        process = nil
-
-        VCR.use_cassette('upload_files_multiple') do
-          expect { process = described_object.export!.first.process }.to output(/complete!/).to_stdout
+      context 'with no errors' do
+        before do
+          add_translation_files! with_ru: true, additional: 5
         end
 
-        expect(process.project_id).to eq(project_id)
-        expect(process.status).to eq('queued')
-      end
+        after do
+          rm_translation_files
+        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)
+        it 'sends a proper API request and handles rate limiting' do
+          process = nil
 
-        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)
+          VCR.use_cassette('upload_files_multiple') do
+            expect { process = described_object.export!.first.process }.to output(/complete!/).to_stdout
+          end
 
-        expect { described_object.export! }.to raise_error(Lokalise::Error::TooManyRequests, /Gave up after 1 retries/i)
+          expect(process.project_id).to eq(project_id)
+          expect(process.status).to eq('queued')
+        end
 
-        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
+        it 'handles too many requests but does not re-raise anything when raise_on_export_fail is false' do
+          allow(described_object.config).to receive(:max_retries_export).and_return(1)
+          allow(described_object.config).to receive(:raise_on_export_fail).and_return(false)
+          allow(described_object).to receive(:sleep).and_return(0)
+
+          fake_client = instance_double('RubyLokaliseApi::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(RubyLokaliseApi::Error::TooManyRequests)
+          allow(described_object).to receive(:api_client).and_return(fake_client)
+          processes = []
+          expect { processes = described_object.export! }.not_to raise_error
+
+          expect(processes[0].success).to be false
+          expect(processes[1].error.class).to eq(RubyLokaliseApi::Error::TooManyRequests)
+          expect(processes.count).to eq(7)
+
+          expect(described_object).to have_received(:sleep).exactly(7).times
+          expect(described_object).to have_received(:api_client).at_least(14).times
+          expect(fake_client).to have_received(:upload_file).exactly(14).times
+        end
       end
 
-      it 'handles too many requests but does not re-raise anything when raise_on_export_fail is false' do
-        allow(described_object.config).to receive(:max_retries_export).and_return(1)
-        allow(described_object.config).to receive(:raise_on_export_fail).and_return(false)
-        allow(described_object).to receive(:sleep).and_return(0)
+      context 'with errors' do
+        before do
+          add_translation_files! with_ru: true
+        end
+
+        after do
+          rm_translation_files
+        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)
-        processes = []
-        expect { processes = described_object.export! }.not_to raise_error
+          fake_client = instance_double('RubyLokaliseApi::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(RubyLokaliseApi::Error::TooManyRequests)
+          allow(described_object).to receive(:api_client).and_return(fake_client)
 
-        expect(processes[0].success).to be false
-        expect(processes[1].error.class).to eq(Lokalise::Error::TooManyRequests)
-        expect(processes.count).to eq(7)
+          expect do
+            described_object.export!
+          end.to raise_error(RubyLokaliseApi::Error::TooManyRequests, /Gave up after 1 retries/i)
 
-        expect(described_object).to have_received(:sleep).exactly(7).times
-        expect(described_object).to have_received(:api_client).at_least(14).times
-        expect(fake_client).to have_received(:upload_file).exactly(14).times
+          expect(described_object).to have_received(:sleep).exactly(2).times
+          expect(described_object).to have_received(:api_client).at_least(4).times
+          expect(fake_client).to have_received(:upload_file).exactly(4).times
+        end
       end
     end
   end
 
   context 'with one translation file' do
-    before :all do
-      add_translation_files!
-    end
-
-    after :all do
-      rm_translation_files
-    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)
+    context 'without files' do
+      it 'halts when the API key is not set' do
+        allow(described_object.config).to receive(:api_token).and_return(nil)
 
-        process = nil
+        expect { described_object.export! }.to raise_error(LokaliseManager::Error, /API token is not set/i)
+        expect(described_object.config).to have_received(:api_token)
+      end
 
-        VCR.use_cassette('upload_files') do
-          expect { process = described_object.export!.first.process }.not_to output(/complete!/).to_stdout
+      it 'halts when the project_id is not set' do
+        allow_project_id described_object, nil do
+          expect { described_object.export! }.to raise_error(LokaliseManager::Error, /ID is not set/i)
         end
-
-        expect(process.status).to eq('queued')
-        expect(described_object.config).to have_received(:silent_mode).at_most(1).times
       end
+    end
 
-      it 'sends a proper API request' do
-        process = VCR.use_cassette('upload_files') do
-          described_object.export!
-        end.first.process
+    context 'with files' do
+      before do
+        add_translation_files!
+      end
 
-        expect(process.project_id).to eq(project_id)
-        expect(process.status).to eq('queued')
+      after do
+        rm_translation_files
       end
 
-      it 'sends a proper API request when a different branch is provided' do
-        allow(described_object.config).to receive(:branch).and_return('develop')
+      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_data = VCR.use_cassette('upload_files_branch') do
-          described_object.export!
-        end.first
+          process = nil
 
-        expect(described_object.config).to have_received(:branch).at_most(2).times
-        expect(process_data.success).to be true
-        expect(process_data.path.to_s).to include('en.yml')
+          VCR.use_cassette('upload_files') do
+            expect { process = described_object.export!.first.process }.not_to output(/complete!/).to_stdout
+          end
 
-        process = process_data.process
-        expect(process).to be_an_instance_of(Lokalise::Resources::QueuedProcess)
-        expect(process.project_id).to eq(project_id)
-        expect(process.status).to eq('queued')
-      end
+          expect(process.status).to eq('queued')
+          expect(described_object.config).to have_received(:silent_mode).at_most(1).times
+        end
 
-      it 'halts when the API key is not set' do
-        allow(described_object.config).to receive(:api_token).and_return(nil)
+        it 'sends a proper API request' do
+          process = VCR.use_cassette('upload_files') do
+            described_object.export!
+          end.first.process
 
-        expect { described_object.export! }.to raise_error(LokaliseManager::Error, /API token is not set/i)
-        expect(described_object.config).to have_received(:api_token)
-      end
+          expect(process.project_id).to eq(project_id)
+          expect(process.status).to eq('queued')
+        end
 
-      it 'halts when the project_id is not set' do
-        allow_project_id described_object, nil do
-          expect { described_object.export! }.to raise_error(LokaliseManager::Error, /ID is not set/i)
+        it 'sends a proper API request when a different branch is provided' do
+          allow(described_object.config).to receive(:branch).and_return('develop')
+
+          process_data = VCR.use_cassette('upload_files_branch') do
+            described_object.export!
+          end.first
+
+          expect(described_object.config).to have_received(:branch).at_most(2).times
+          expect(process_data.success).to be true
+          expect(process_data.path.to_s).to include('en.yml')
+
+          process = process_data.process
+          expect(process).to be_an_instance_of(RubyLokaliseApi::Resources::QueuedProcess)
+          expect(process.project_id).to eq(project_id)
+          expect(process.status).to eq('queued')
         end
       end
     end
 
     describe '#all_files' do
+      before do
+        add_translation_files!
+      end
+
+      after do
+        rm_translation_files
+      end
+
       it 'yield proper arguments' do
         expect(described_object.send(:all_files).flatten).to include(
           Pathname.new(path),
@@ -146,6 +172,14 @@ describe LokaliseManager::TaskDefinitions::Exporter do
     end
 
     describe '.opts' do
+      before do
+        add_translation_files!
+      end
+
+      after do
+        rm_translation_files
+      end
+
       let(:base64content) { Base64.strict_encode64(File.read(path).strip) }
 
       it 'generates proper options' do
@@ -178,11 +212,11 @@ describe LokaliseManager::TaskDefinitions::Exporter do
     let(:filename_ru) { 'ru.yml' }
     let(:path_ru) { "#{Dir.getwd}/locales/#{filename_ru}" }
 
-    before :all do
+    before do
       add_translation_files! with_ru: true
     end
 
-    after :all do
+    after do
       rm_translation_files
     end
 
@@ -191,7 +225,7 @@ describe LokaliseManager::TaskDefinitions::Exporter do
         allow_project_id described_object, '542886116159f798720dc4.94769464'
 
         VCR.use_cassette('upload_files_error') do
-          expect { described_object.export! }.to raise_error(Lokalise::Error::BadRequest, /Unknown `lang_iso`/)
+          expect { described_object.export! }.to raise_error(RubyLokaliseApi::Error::BadRequest, /Unknown `lang_iso`/)
         end
       end
     end

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

@@ -42,7 +42,7 @@ describe LokaliseManager::TaskDefinitions::Importer do
 
       VCR.use_cassette('download_files_error') do
         expect { described_object.send :download_files }.
-          to raise_error(Lokalise::Error::BadRequest, /Invalid `project_id` parameter/)
+          to raise_error(RubyLokaliseApi::Error::BadRequest, /Invalid `project_id` parameter/)
       end
     end
   end
@@ -52,11 +52,11 @@ describe LokaliseManager::TaskDefinitions::Importer do
       it 'handles too many requests' do
         allow(described_object).to receive(:sleep).and_return(0)
 
-        fake_client = instance_double('Lokalise::Client')
-        allow(fake_client).to receive(:download_files).and_raise(Lokalise::Error::TooManyRequests)
+        fake_client = instance_double('RubyLokaliseApi::Client')
+        allow(fake_client).to receive(:download_files).and_raise(RubyLokaliseApi::Error::TooManyRequests)
         allow(described_object).to receive(:api_client).and_return(fake_client)
 
-        expect { described_object.import! }.to raise_error(Lokalise::Error::TooManyRequests, /Gave up after 2 retries/i)
+        expect { described_object.import! }.to raise_error(RubyLokaliseApi::Error::TooManyRequests, /Gave up after 2 retries/i)
 
         expect(described_object).to have_received(:sleep).exactly(2).times
         expect(described_object).to have_received(:api_client).exactly(3).times
@@ -81,10 +81,9 @@ describe LokaliseManager::TaskDefinitions::Importer do
     context 'when directory is empty' do
       before do
         mkdir_locales
-        rm_translation_files
       end
 
-      after :all do
+      after do
         rm_translation_files
       end
 
@@ -141,16 +140,13 @@ describe LokaliseManager::TaskDefinitions::Importer do
                             import_safe_mode: true
       end
 
-      before :all do
-        mkdir_locales
-      end
-
       before do
+        mkdir_locales
         rm_translation_files
         add_translation_files!
       end
 
-      after :all do
+      after do
         rm_translation_files
       end
 
@@ -183,7 +179,7 @@ describe LokaliseManager::TaskDefinitions::Importer do
         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
+      it 'import halts when a user chooses to halt 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')

data/spec/support/file_manager.rb

@@ -4,7 +4,9 @@ require 'fileutils'
 
 module FileManager
   def mkdir_locales
-    FileUtils.mkdir_p(LokaliseManager::GlobalConfig.locales_path) unless File.directory?(LokaliseManager::GlobalConfig.locales_path)
+    return if File.directory?(LokaliseManager::GlobalConfig.locales_path)
+
+    FileUtils.mkdir_p(LokaliseManager::GlobalConfig.locales_path)
   end
 
   def rm_translation_files

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: lokalise_manager
 version: !ruby/object:Gem::Version
-  version: 2.2.0
+  version: 3.0.0
 platform: ruby
 authors:
 - Ilya Bodrov
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-02-23 00:00:00.000000000 Z
+date: 2022-03-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ruby-lokalise-api
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '5.0'
+        version: '6.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '5.0'
+        version: '6.0'
 - !ruby/object:Gem::Dependency
   name: rubyzip
   requirement: !ruby/object:Gem::Requirement
@@ -38,6 +38,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '2.3'
+- !ruby/object:Gem::Dependency
+  name: zeitwerk
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.4'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.4'
 - !ruby/object:Gem::Dependency
   name: codecov
   requirement: !ruby/object:Gem::Requirement
@@ -194,8 +208,8 @@ files:
 - spec/lib/lokalise_manager/task_definitions/base_spec.rb
 - spec/lib/lokalise_manager/task_definitions/exporter_spec.rb
 - spec/lib/lokalise_manager/task_definitions/importer_spec.rb
+- spec/lib/lokalise_manager/utils/hash_utils_spec.rb
 - spec/lib/lokalise_manager_spec.rb
-- spec/lib/utils/hash_utils_spec.rb
 - spec/spec_helper.rb
 - spec/support/file_manager.rb
 - spec/support/spec_addons.rb
@@ -213,14 +227,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.5.0
+      version: '2.7'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.7
+rubygems_version: 3.3.8
 signing_key:
 specification_version: 4
 summary: Lokalise integration for Ruby
@@ -229,8 +243,8 @@ test_files:
 - spec/lib/lokalise_manager/task_definitions/base_spec.rb
 - spec/lib/lokalise_manager/task_definitions/exporter_spec.rb
 - spec/lib/lokalise_manager/task_definitions/importer_spec.rb
+- spec/lib/lokalise_manager/utils/hash_utils_spec.rb
 - spec/lib/lokalise_manager_spec.rb
-- spec/lib/utils/hash_utils_spec.rb
 - spec/spec_helper.rb
 - spec/support/file_manager.rb
 - spec/support/spec_addons.rb