lokalise_manager 2.1.0 → 2.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f3a7da094cb7939fd6826c89afbbbc525dff59b553a23fba7b49ed38046e68b9
-  data.tar.gz: c0a323464c864d12042f126d765a483adc128a4397604d012cdeadee31003fcf
+  metadata.gz: fa7eb812b614ec768710c4f085879dfaf6cf89f72e25b1a777f34af3e9cd8be5
+  data.tar.gz: afdd4c08bf9672302c279cd88b4c4caf63317b1aca98036a5e8c1dad361289f0
 SHA512:
-  metadata.gz: 79c0b5a93767b5ffa8cd55dbf6591a90cafbb586043e8e75ca040761b8fa882882971783358e6d342afab96e298d34c67ca5d414d376c909fa564e8ad03d2466
-  data.tar.gz: ed2ad8e59fb6e15db8ba7c60de48e808c2eab0ff94b8f287c54bb2e618e1c930d5d3244a2483a6e61e69b7094943a05db59ea54ef32fc3596ef344f5956553ee
+  metadata.gz: 5857c35bde158e85bdec5c2e4618ebf1ddc7768f5822d5492109347fa93907434869d23c5053f6c7b98da8016e4e22c40fa8fa4b513c347630aeb18288f8d18d
+  data.tar.gz: 02bbc800a21396c6f106b2c18df5eee2f7247d43fac80da656496d4a8ec6a50c8d9438bb3514d13b57647f29aebfb1caf1e633a79d97eb37e30ae6bbb2634ecd

data/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## 2.2.0 (23-Feb-22)
+
+* Use ruby-lokalise-api v5
+* Don't use any compression options (compression is now enabled by default)
+* Update tests
+
 ## 2.1.0 (27-Jan-22)
 
 * **Breaking change**: `export!` will now return an array of objects responding to the following methods:

data/README.md

@@ -43,7 +43,7 @@ importer = LokaliseManager.importer api_token: '1234abc', project_id: '123.abc'
 exporter = LokaliseManager.exporter api_token: '1234abc', project_id: '123.abc'
 ```
 
-You *must* provide an API token and a project ID (your project ID can be found under Lokalise project settings). [Other options can be customized as well (see below)](https://github.com/bodrovis/lokalise_manager#configuration) but they have sensible defaults.
+You *must* provide an API token and a project ID (your project ID can be found under Lokalise project settings). [Other options can be customized as well (see below)](#configuration) but they have sensible defaults.
 
 ### Importing files from Lokalise into your project
 
@@ -55,7 +55,7 @@ result = importer.import!
 
 The `result` will contain a boolean value which says whether the operation was successfull or not.
 
-Please note that upon importing translations any duplicating files inside the `locales` directory (or any other directory that you've specified in the options) will be overwritten! You can enable [safe mode](https://github.com/bodrovis/lokalise_manager#import-config) to check whether the folder is empty or not.
+Please note that upon importing translations any duplicating files inside the `locales` directory (or any other directory that you've specified in the options) **will be overwritten**! You can enable [safe mode](#import-config) to check whether the folder is empty or not.
 
 ### Exporting files from your project to Lokalise
 
@@ -65,19 +65,21 @@ To upload your translation files from a local directory (defaults to `locales/`)
 processes = exporter.export!
 ```
 
+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.
-* `path` — returns an instance of the `Pathname` class which represent the file being uploaded.
+* `#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.
+* `#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:
 
 ```ruby
 def uploaded?(process)
   5.times do # try to check the status 5 times
-    process = process.reload_data # load new data
-    return(true) if process.status == 'finished' # return true is the upload has finished
+    process = process.reload_data # load new info about this process
+    return(true) if process.status == 'finished' # return true if the upload has finished
     sleep 1 # wait for 1 second, adjust this number with regards to the upload size
   end
 
@@ -169,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, LokaliseRails 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`, LokaliseRails 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 `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.
 * `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
@@ -222,7 +224,7 @@ importer = LokaliseManager.importer api_token: '1234abc',
 
 These options will be merged with the default ones. Please note that per-client config has the highest priority.
 
-You can also adjust individual options later using the `#config` instance variable (it contains an OpenStruct object):
+You can also adjust individual options later using the `#config` instance variable (it contains a Struct object):
 
 ```
 importer.config.project_id = '678xyz'

data/lib/lokalise_manager/task_definitions/base.rb

@@ -35,8 +35,8 @@ module LokaliseManager
       #
       # @return [Lokalise::Client]
       def api_client
-        client_opts = [config.api_token, {enable_compression: true}.merge(config.timeouts)]
-        client_method = config.use_oauth2_token ? :oauth_client : :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)
       end
@@ -44,7 +44,7 @@ module LokaliseManager
       # Resets API client
       def reset_api_client!
         ::Lokalise.reset_client!
-        ::Lokalise.reset_oauth_client!
+        ::Lokalise.reset_oauth2_client!
         @api_client = nil
       end
 

data/lib/lokalise_manager/task_definitions/exporter.rb

@@ -5,8 +5,7 @@ require 'base64'
 module LokaliseManager
   module TaskDefinitions
     class Exporter < Base
-      using LokaliseManager::Utils::ArrayUtils
-
+      # Lokalise allows no more than 6 requests per second
       MAX_THREADS = 6
 
       # Performs translation file export to Lokalise and returns an array of queued processes
@@ -17,7 +16,7 @@ module LokaliseManager
 
         queued_processes = []
 
-        all_files.in_groups_of(MAX_THREADS) do |files_group|
+        all_files.each_slice(MAX_THREADS) do |files_group|
           parallel_upload(files_group).each do |thr|
             raise_on_fail(thr) if config.raise_on_export_fail
 
@@ -33,7 +32,7 @@ module LokaliseManager
       private
 
       def parallel_upload(files_group)
-        files_group.compact.map do |file_data|
+        files_group.map do |file_data|
           do_upload(*file_data)
         end.map(&:value)
       end
@@ -59,18 +58,16 @@ module LokaliseManager
 
       # Gets translation files from the specified directory
       def all_files
-        files = []
         loc_path = config.locales_path
-        Dir["#{loc_path}/**/*"].sort.each do |f|
+        Dir["#{loc_path}/**/*"].map do |f|
           full_path = Pathname.new f
 
           next unless file_matches_criteria? full_path
 
           relative_path = full_path.relative_path_from Pathname.new(loc_path)
 
-          files << [full_path, relative_path]
-        end
-        files
+          [full_path, relative_path]
+        end.compact
       end
 
       # Generates export options

data/lib/lokalise_manager/version.rb

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

data/lib/lokalise_manager.rb

@@ -3,7 +3,6 @@
 require 'yaml'
 
 require 'lokalise_manager/utils/hash_utils'
-require 'lokalise_manager/utils/array_utils'
 
 require 'lokalise_manager/version'
 require 'lokalise_manager/error'

data/lokalise_manager.gemspec

@@ -23,7 +23,7 @@ Gem::Specification.new do |spec|
   spec.extra_rdoc_files = ['README.md']
   spec.require_paths    = ['lib']
 
-  spec.add_dependency 'ruby-lokalise-api', '~> 4.0'
+  spec.add_dependency 'ruby-lokalise-api', '~> 5.0'
   spec.add_dependency 'rubyzip', '~> 2.3'
 
   spec.add_development_dependency 'codecov', '~> 0.2'

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

@@ -63,14 +63,14 @@ 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 { described_object.send(:check_options_errors!) }.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 { described_object.send(:check_options_errors!) }.to raise_error(LokaliseManager::Error, /ID is not set/i)
       end
     end
   end
@@ -96,7 +96,7 @@ 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::OAuthClient)
+      expect(client).not_to be_an_instance_of(Lokalise::OAuth2Client)
       expect(client.open_timeout).to eq(100)
       expect(client.timeout).to eq(500)
     end
@@ -106,7 +106,7 @@ describe LokaliseManager::TaskDefinitions::Base do
 
       client = described_object.api_client
 
-      expect(client).to be_an_instance_of(Lokalise::OAuthClient)
+      expect(client).to be_an_instance_of(Lokalise::OAuth2Client)
       expect(client).not_to be_an_instance_of(Lokalise::Client)
     end
   end

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

@@ -27,7 +27,7 @@ describe LokaliseManager::TaskDefinitions::Exporter do
         process = nil
 
         VCR.use_cassette('upload_files_multiple') do
-          expect(-> { process = described_object.export!.first.process }).to output(/complete!/).to_stdout
+          expect { process = described_object.export!.first.process }.to output(/complete!/).to_stdout
         end
 
         expect(process.project_id).to eq(project_id)
@@ -43,7 +43,7 @@ describe LokaliseManager::TaskDefinitions::Exporter do
         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 1 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(6).times
         expect(described_object).to have_received(:api_client).at_least(12).times
@@ -60,9 +60,8 @@ describe LokaliseManager::TaskDefinitions::Exporter do
         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
+        expect { processes = described_object.export! }.not_to raise_error
 
-        expect(processes[0].path.to_s).to include('en_0')
         expect(processes[0].success).to be false
         expect(processes[1].error.class).to eq(Lokalise::Error::TooManyRequests)
         expect(processes.count).to eq(7)
@@ -90,7 +89,7 @@ describe LokaliseManager::TaskDefinitions::Exporter do
         process = nil
 
         VCR.use_cassette('upload_files') do
-          expect(-> { process = described_object.export!.first.process }).not_to output(/complete!/).to_stdout
+          expect { process = described_object.export!.first.process }.not_to output(/complete!/).to_stdout
         end
 
         expect(process.status).to eq('queued')
@@ -126,20 +125,20 @@ describe LokaliseManager::TaskDefinitions::Exporter do
       it 'halts when the API key is not set' do
         allow(described_object.config).to receive(:api_token).and_return(nil)
 
-        expect(-> { described_object.export! }).to raise_error(LokaliseManager::Error, /API token is not set/i)
+        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
 
       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)
+          expect { described_object.export! }.to raise_error(LokaliseManager::Error, /ID is not set/i)
         end
       end
     end
 
     describe '#all_files' do
       it 'yield proper arguments' do
-        expect(described_object.send(:all_files).first).to include(
+        expect(described_object.send(:all_files).flatten).to include(
           Pathname.new(path),
           Pathname.new(relative_name)
         )
@@ -209,14 +208,14 @@ describe LokaliseManager::TaskDefinitions::Exporter do
       end
     end
 
-    describe '#each_file' do
+    describe '#all_files' do
       it 'returns all files' do
-        files = described_object.send(:all_files)
-        expect(files[0]).to include(
+        files = described_object.send(:all_files).flatten
+        expect(files).to include(
           Pathname.new(path),
           Pathname.new(relative_name)
         )
-        expect(files[1]).to include(
+        expect(files).to include(
           Pathname.new(path_ru),
           Pathname.new(filename_ru)
         )
@@ -226,7 +225,7 @@ describe LokaliseManager::TaskDefinitions::Exporter do
         allow(described_object.config).to receive(:skip_file_export).twice.and_return(
           ->(f) { f.split[1].to_s.include?('ru') }
         )
-        files = described_object.send(:all_files)
+        files = described_object.send(:all_files).sort
         expect(files[0]).to include(
           Pathname.new(path),
           Pathname.new(relative_name)

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

@@ -15,14 +15,14 @@ describe LokaliseManager::TaskDefinitions::Importer do
       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) }).
+      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
-      expect(-> { described_object.send(:open_and_process_zip, 'http://fake.url/wrong/path.zip') }).
+      expect { described_object.send(:open_and_process_zip, 'http://fake.url/wrong/path.zip') }.
         to raise_error(SocketError, /Failed to open TCP connection/)
     end
   end
@@ -35,14 +35,13 @@ describe LokaliseManager::TaskDefinitions::Importer do
 
       expect(response['project_id']).to eq('672198945b7d72fc048021.15940510')
       expect(response['bundle_url']).to include('s3-eu-west-1.amazonaws.com')
-      expect(described_object.api_client.enable_compression).to eq(true)
     end
 
     it 're-raises errors during file download' do
       allow_project_id described_object, 'invalid'
 
       VCR.use_cassette('download_files_error') do
-        expect(-> { described_object.send :download_files }).
+        expect { described_object.send :download_files }.
           to raise_error(Lokalise::Error::BadRequest, /Invalid `project_id` parameter/)
       end
     end
@@ -57,7 +56,7 @@ describe LokaliseManager::TaskDefinitions::Importer do
         allow(fake_client).to receive(:download_files).and_raise(Lokalise::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(Lokalise::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
@@ -66,14 +65,14 @@ describe LokaliseManager::TaskDefinitions::Importer do
 
       it 'halts when the API key is not set' do
         allow(described_object.config).to receive(:api_token).and_return(nil)
-        expect(-> { described_object.import! }).to raise_error(LokaliseManager::Error, /API token is not set/i)
+        expect { described_object.import! }.to raise_error(LokaliseManager::Error, /API token is not set/i)
         expect(described_object.config).to have_received(:api_token)
         expect(count_translations).to eq(0)
       end
 
       it 'halts when the project_id is not set' do
         allow_project_id described_object, nil do
-          expect(-> { described_object.import! }).to raise_error(LokaliseManager::Error, /ID is not set/i)
+          expect { described_object.import! }.to raise_error(LokaliseManager::Error, /ID is not set/i)
           expect(count_translations).to eq(0)
         end
       end
@@ -110,7 +109,7 @@ describe LokaliseManager::TaskDefinitions::Importer do
         result = nil
 
         VCR.use_cassette('import') do
-          expect(-> { result = described_object.import! }).to output(/complete!/).to_stdout
+          expect { result = described_object.import! }.to output(/complete!/).to_stdout
         end
 
         expect(result).to be true
@@ -125,7 +124,7 @@ describe LokaliseManager::TaskDefinitions::Importer do
         result = nil
 
         VCR.use_cassette('import') do
-          expect(-> { result = described_object.import! }).not_to output(/complete!/).to_stdout
+          expect { result = described_object.import! }.not_to output(/complete!/).to_stdout
         end
 
         expect(result).to be true
@@ -164,7 +163,7 @@ describe LokaliseManager::TaskDefinitions::Importer do
         )
 
         allow($stdin).to receive(:gets).and_return('Y')
-        expect(-> { safe_mode_obj.import! }).to output(/is not empty/).to_stdout
+        expect { safe_mode_obj.import! }.to output(/is not empty/).to_stdout
 
         expect(count_translations).to eq(5)
         expect($stdin).to have_received(:gets)
@@ -177,7 +176,7 @@ 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(/cancelled/).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)
@@ -188,7 +187,7 @@ describe LokaliseManager::TaskDefinitions::Importer 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.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)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: lokalise_manager
 version: !ruby/object:Gem::Version
-  version: 2.1.0
+  version: 2.2.0
 platform: ruby
 authors:
 - Ilya Bodrov
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-01-27 00:00:00.000000000 Z
+date: 2022-02-23 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: '4.0'
+        version: '5.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '5.0'
 - !ruby/object:Gem::Dependency
   name: rubyzip
   requirement: !ruby/object:Gem::Requirement
@@ -187,7 +187,6 @@ 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
@@ -196,7 +195,6 @@ 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
@@ -222,7 +220,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.5
+rubygems_version: 3.3.7
 signing_key:
 specification_version: 4
 summary: Lokalise integration for Ruby
@@ -232,7 +230,6 @@ 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

data/lib/lokalise_manager/utils/array_utils.rb

@@ -1,27 +0,0 @@
-# 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/spec/lib/utils/array_utils_spec.rb

@@ -1,18 +0,0 @@
-# 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