lokalise_rails 1.1.0 → 1.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 951a2d142a56078d2d9509fa36c13f87f7069b8f2defa9bd158a90a67aba2b9a
-  data.tar.gz: ac2360ad608a4251c9061454c0fea986f868facb90567a5bcd94ae01d33bd8e3
+  metadata.gz: 6448a3718185d09a6712428f671e02121e59c967199ae6e24f27b8623f12b280
+  data.tar.gz: c0f2501d9c937b26088ee23a6603a842121cb6a88516ae19d09d40abe1968161
 SHA512:
-  metadata.gz: 4f41072fa0c0e138fc3f6a838b672c50e0f3af799e151b0cbc2b66aa1792446c6cdde4a1426ed1a5243871c48cc55bc9e12439ecc7eb00be090629ad0a77bd4c
-  data.tar.gz: ae1acd8d95ce227c8c16a362d2fb0d0bc371a986e5108043a7d9f3037bf292e1da2a48efa4c84720e7a4f1f1e66c13e757ba59ababe1ab081d77960516340803
+  metadata.gz: be178f379eb4abbb4d50c4f3817422c665bed0c8a75f256c083b0229a57594956ca8c42648fe98be40ed1c4183f0e00fd51fcaf153a87ed408279c1d2ac152ec
+  data.tar.gz: db7dd0efdb0d9a7b72116adba959a04b899e41d5ccd89181f48e2ec2e1d097ebfebb732e6592c725b831cebf89134e77e0639deb8591876267306583064efa96

data/CHANGELOG.md

@@ -1,5 +1,26 @@
 # Changelog
 
+## 1.4.0 (29-Jun-21)
+
+* Re-worked exception handling. Now when something goes wrong during the import or export process, this gem will re-raise all such errors (previously it just printed out some errors to the `$stdout`). If you run a Rake task, it will exit with a status code `1` and the actual error message. If you run a task programattically, you'll get an exception.
+* Dropped support for Ruby 2.5
+* Test against Ruby 3
+
+## 1.3.1 (01-Apr-21)
+
+* A bit better exception handling
+* Update dependencies
+
+## 1.3.0 (02-Feb-21)
+
+* Use ruby-lokalise-api v3
+
+## 1.2.0 (11-Nov-20)
+
+* New option `translations_loader`
+* New option `translations_converter`
+* New option `lang_iso_inferer`
+
 ## 1.1.0 (23-Oct-20)
 
 * New option `branch`

data/README.md

@@ -1,8 +1,9 @@
 # LokaliseRails
 
-[![Gem Version](https://badge.fury.io/rb/lokalise_rails.svg)](https://badge.fury.io/rb/lokalise_rails)
-[![Build Status](https://travis-ci.org/bodrovis/lokalise_rails.svg?branch=master)](https://travis-ci.org/bodrovis/lokalise_rails)
+![Gem](https://img.shields.io/gem/v/lokalise_rails)
+[![Build Status](https://travis-ci.com/bodrovis/lokalise_rails.svg?branch=master)](https://travis-ci.com/github/bodrovis/lokalise_rails)
 [![Test Coverage](https://codecov.io/gh/bodrovis/lokalise_rails/graph/badge.svg)](https://codecov.io/gh/bodrovis/lokalise_rails)
+![Downloads total](https://img.shields.io/gem/dt/lokalise_rails)
 
 This gem provides [Lokalise](http://lokalise.com) integration for Ruby on Rails and allows to exchange translation files easily. It relies on [ruby-lokalise-api](https://lokalise.github.io/ruby-lokalise-api) to send APIv2 requests.
 
@@ -92,7 +93,6 @@ Options are specified in the `config/lokalise_rails.rb` file.
 * `api_token` (`string`, required) - Lokalise API token with read/write permissions.
 * `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 `"#{Rails.root}/config/locales"`.
-* `file_ext_regexp` (`regexp`) - regular expression applied to file extensions to determine which files should be imported and exported. Defaults to `/\.ya?ml\z/i` (YAML files).
 * `branch` (`string`) - Lokalise project branch to use. Defaults to `"master"`.
 * `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.
 
@@ -134,6 +134,15 @@ en_US:
 c.skip_file_export = ->(file) { f.split[1].to_s.include?('fr') }
 ```
 
+### Settings to work with formats other than YAML
+
+If your translation files are not in YAML format, you will need to adjust the following options:
+
+* `file_ext_regexp` (`regexp`) - regular expression applied to file extensions to determine which files should be imported and exported. Defaults to `/\.ya?ml\z/i` (YAML files).
+* `translations_loader` (`lambda` or `proc`) - loads translations data and makes sure they are valid before saving them to a translation file. Defaults to `->(raw_data) { YAML.safe_load raw_data }`. In the simplest case you may just return the data back, for example `-> (raw_data) { raw_data }`.
+* `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 }`.
+
 ## Running tests
 
 1. Copypaste `.env.example` file as `.env`. Put your Lokalise API token and project ID inside. The `.env` file is excluded from version control so your data is safe. All in all, we use pre-recorded VCR cassettes, so the actual API requests won’t be sent. However, providing at least some values is required.

data/lib/generators/templates/lokalise_rails_config.rb

@@ -35,6 +35,16 @@ LokaliseRails.config do |c|
   # Provide additional file exclusion criteria for exports (by default, any file with the proper extension will be exported)
   # c.skip_file_export = ->(file) { file.split[1].to_s.include?('fr') }
 
-  # Regular expression to use when choosing the files to extract from the downloaded archive and upload to Lokalise
-  # c.file_ext_regexp = /\.ya?ml\z/i
+  # Set the options below if you would like to work with format other than YAML
+  ## Regular expression to use when choosing the files to extract from the downloaded archive and upload to Lokalise
+  ## c.file_ext_regexp = /\.ya?ml\z/i
+
+  ## Load translations data and make sure they are valid:
+  ## c.translations_loader = ->(raw_data) { YAML.safe_load raw_data }
+
+  ## Convert translations data to a proper format:
+  ## c.translations_converter = ->(raw_data) { raw_data.to_yaml }
+
+  ## Infer language ISO code for the translation file:
+  ## c.lang_iso_inferer = ->(data) { YAML.safe_load(data)&.keys&.first }
 end

data/lib/lokalise_rails.rb

@@ -1,5 +1,6 @@
 # frozen_string_literal: true
 
+require 'lokalise_rails/error'
 require 'lokalise_rails/task_definition/base'
 require 'lokalise_rails/task_definition/importer'
 require 'lokalise_rails/task_definition/exporter'
@@ -8,7 +9,8 @@ module LokaliseRails
   class << self
     attr_accessor :api_token, :project_id
     attr_writer :import_opts, :import_safe_mode, :export_opts, :locales_path,
-                :file_ext_regexp, :skip_file_export, :branch, :timeouts
+                :file_ext_regexp, :skip_file_export, :branch, :timeouts,
+                :translations_loader, :translations_converter, :lang_iso_inferer
 
     # Main interface to provide configuration options for rake tasks
     def config
@@ -61,6 +63,20 @@ module LokaliseRails
     def skip_file_export
       @skip_file_export || ->(_) { false }
     end
+
+    def translations_loader
+      @translations_loader || ->(raw_data) { YAML.safe_load raw_data }
+    end
+
+    # Converts translations data to the proper format
+    def translations_converter
+      @translations_converter || ->(raw_data) { raw_data.to_yaml }
+    end
+
+    # Infers lang ISO for the given translation file
+    def lang_iso_inferer
+      @lang_iso_inferer || ->(data) { YAML.safe_load(data)&.keys&.first }
+    end
   end
 end
 

data/lib/lokalise_rails/error.rb

@@ -0,0 +1,10 @@
+# frozen_string_literal: true
+
+module LokaliseRails
+  class Error < StandardError
+    # Initializes a new Error object
+    def initialize(message = '')
+      super(message)
+    end
+  end
+end

data/lib/lokalise_rails/task_definition/base.rb

@@ -25,11 +25,12 @@ module LokaliseRails
         # Checks task options
         #
         # @return Array
-        def opt_errors
+        def check_options_errors!
           errors = []
-          errors << 'Project ID is not set! Aborting...' if LokaliseRails.project_id.nil? || LokaliseRails.project_id.empty?
-          errors << 'Lokalise API token is not set! Aborting...' if LokaliseRails.api_token.nil? || LokaliseRails.api_token.empty?
-          errors
+          errors << 'Project ID is not set!' if LokaliseRails.project_id.nil? || LokaliseRails.project_id.empty?
+          errors << 'Lokalise API token is not set!' if LokaliseRails.api_token.nil? || LokaliseRails.api_token.empty?
+
+          raise(LokaliseRails::Error, errors.join(' ')) if errors.any?
         end
 
         private

data/lib/lokalise_rails/task_definition/exporter.rb

@@ -10,12 +10,7 @@ module LokaliseRails
         #
         # @return [Array]
         def export!
-          errors = opt_errors
-
-          if errors.any?
-            errors.each { |e| $stdout.puts e }
-            return errors
-          end
+          check_options_errors!
 
           queued_processes = []
           each_file do |full_path, relative_path|
@@ -23,7 +18,7 @@ module LokaliseRails
               project_id_with_branch, opts(full_path, relative_path)
             )
           rescue StandardError => e
-            $stdout.puts "Error while trying to upload #{full_path}: #{e.inspect}"
+            raise e.class, "Error while trying to upload #{full_path}: #{e.message}"
           end
 
           $stdout.print 'Task complete!'
@@ -55,12 +50,10 @@ module LokaliseRails
         def opts(full_p, relative_p)
           content = File.read full_p
 
-          lang_iso = YAML.safe_load(content)&.keys&.first
-
           initial_opts = {
             data: Base64.strict_encode64(content.strip),
             filename: relative_p,
-            lang_iso: lang_iso
+            lang_iso: LokaliseRails.lang_iso_inferer.call(content)
           }
 
           initial_opts.merge LokaliseRails.export_opts

data/lib/lokalise_rails/task_definition/importer.rb

@@ -13,12 +13,7 @@ module LokaliseRails
         #
         # @return [Boolean]
         def import!
-          errors = opt_errors
-
-          if errors.any?
-            errors.each { |e| $stdout.puts e }
-            return false
-          end
+          check_options_errors!
 
           unless proceed_when_safe_mode?
             $stdout.print 'Task cancelled!'
@@ -39,16 +34,18 @@ module LokaliseRails
 
           api_client.download_files project_id_with_branch, opts
         rescue StandardError => e
-          $stdout.puts "There was an error when trying to download files: #{e.inspect}"
+          raise e.class, "There was an error when trying to download files: #{e.message}"
         end
 
         # Opens ZIP archive (local or remote) with translations and processes its entries
         #
         # @param path [String]
         def open_and_process_zip(path)
-          Zip::File.open_buffer(URI.open(path)) do |zip|
+          Zip::File.open_buffer(open_file_or_remote(path)) do |zip|
             fetch_zip_entries(zip) { |entry| process!(entry) }
           end
+        rescue StandardError => e
+          raise e.class, "There was an error when trying to process the downloaded files: #{e.message}"
         end
 
         # Iterates over ZIP entries. Each entry may be a file or folder.
@@ -64,16 +61,16 @@ module LokaliseRails
 
         # Processes ZIP entry by reading its contents and creating the corresponding translation file
         def process!(zip_entry)
-          data = YAML.safe_load zip_entry.get_input_stream.read
+          data = data_from zip_entry
           subdir, filename = subdir_and_filename_for zip_entry.name
           full_path = "#{LokaliseRails.locales_path}/#{subdir}"
           FileUtils.mkdir_p full_path
 
           File.open(File.join(full_path, filename), 'w+:UTF-8') do |f|
-            f.write data.to_yaml
+            f.write LokaliseRails.translations_converter.call(data)
           end
         rescue StandardError => e
-          $stdout.puts "Error when trying to process #{zip_entry&.name}: #{e.inspect}"
+          raise e.class, "Error when trying to process #{zip_entry&.name}: #{e.message}"
         end
 
         # Checks whether the user wishes to proceed when safe mode is enabled and the target directory is not empty
@@ -87,6 +84,24 @@ module LokaliseRails
           answer = $stdin.gets
           answer.to_s.strip == 'Y'
         end
+
+        # Opens a local file or a remote URL using the provided patf
+        #
+        # @return [String]
+        def open_file_or_remote(path)
+          parsed_path = URI.parse(path)
+          if parsed_path&.scheme&.include?('http')
+            parsed_path.open
+          else
+            File.open path
+          end
+        end
+
+        private
+
+        def data_from(zip_entry)
+          LokaliseRails.translations_loader.call zip_entry.get_input_stream.read
+        end
       end
     end
   end

data/lib/lokalise_rails/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module LokaliseRails
-  VERSION = '1.1.0'
+  VERSION = '1.4.0'
 end

data/lib/tasks/lokalise_rails_tasks.rake

@@ -6,9 +6,13 @@ require "#{Rails.root}/config/lokalise_rails"
 namespace :lokalise_rails do
   task :import do
     LokaliseRails::TaskDefinition::Importer.import!
+  rescue StandardError => e
+    abort e.inspect
   end
 
   task :export do
     LokaliseRails::TaskDefinition::Exporter.export!
+  rescue StandardError => e
+    abort e.inspect
   end
 end

data/lokalise_rails.gemspec

@@ -23,21 +23,21 @@ Gem::Specification.new do |spec|
   spec.extra_rdoc_files = ['README.md']
   spec.require_paths    = ['lib']
 
-  spec.add_dependency 'ruby-lokalise-api', '~> 3.1'
+  spec.add_dependency 'ruby-lokalise-api', '~> 4.0'
   spec.add_dependency 'rubyzip', '~> 2.3'
 
-  spec.add_development_dependency 'codecov', '~> 0.1'
+  spec.add_development_dependency 'codecov', '~> 0.2'
   spec.add_development_dependency 'dotenv', '~> 2.5'
   if ENV['TEST_RAILS_VERSION'].nil?
-    spec.add_development_dependency 'rails', '~> 6.0.3'
+    spec.add_development_dependency 'rails', '~> 6.1.0'
   else
     spec.add_development_dependency 'rails', ENV['TEST_RAILS_VERSION'].to_s
   end
   spec.add_development_dependency 'rake', '~> 13.0'
   spec.add_development_dependency 'rspec', '~> 3.6'
-  spec.add_development_dependency 'rubocop', '~> 0.60'
+  spec.add_development_dependency 'rubocop', '~> 1.0'
   spec.add_development_dependency 'rubocop-performance', '~> 1.5'
-  spec.add_development_dependency 'rubocop-rspec', '~> 1.37'
+  spec.add_development_dependency 'rubocop-rspec', '~> 2.4.0'
   spec.add_development_dependency 'simplecov', '~> 0.16'
   spec.add_development_dependency 'vcr', '~> 6.0'
 end

data/spec/dummy/config/environments/test.rb

@@ -17,9 +17,7 @@ Rails.application.configure do
 
   # Configure public file server for tests with Cache-Control for performance.
   config.public_file_server.enabled = true
-  config.public_file_server.headers = {
-    'Cache-Control' => "public, max-age=#{1.hour.to_i}"
-  }
+  config.public_file_server.headers = {'Cache-Control' => 'public, max-age=3600'}
 
   # Show full error reports and disable caching.
   config.consider_all_requests_local       = true

data/spec/dummy/config/lokalise_rails.rb

@@ -1,5 +1,3 @@
-# frozen_string_literal: true
-
 require 'lokalise_rails'
 LokaliseRails.config do |c|
   c.api_token = ENV['LOKALISE_API_TOKEN']

data/spec/lib/lokalise_rails/task_definition/base_spec.rb

@@ -34,23 +34,19 @@ describe LokaliseRails::TaskDefinition::Base do
     end
   end
 
-  describe '.opt_errors' do
-    it 'returns an error when the API key is not set' do
+  describe '.check_options_errors!' do
+    it 'raises an error when the API key is not set' do
       allow(LokaliseRails).to receive(:api_token).and_return(nil)
-      errors = described_class.opt_errors
+
+      expect(-> { described_class.check_options_errors! }).to raise_error(LokaliseRails::Error, /API token is not set/i)
 
       expect(LokaliseRails).to have_received(:api_token)
-      expect(errors.length).to eq(1)
-      expect(errors.first).to include('API token is not set')
     end
 
     it 'returns an error when the project_id is not set' do
-      allow(LokaliseRails).to receive(:project_id).and_return(nil)
-      errors = described_class.opt_errors
-
-      expect(LokaliseRails).to have_received(:project_id)
-      expect(errors.length).to eq(1)
-      expect(errors.first).to include('Project ID is not set')
+      allow_project_id nil do
+        expect(-> { described_class.check_options_errors! }).to raise_error(LokaliseRails::Error, /ID is not set/i)
+      end
     end
   end
 

data/spec/lib/lokalise_rails/task_definition/exporter_spec.rb

@@ -44,15 +44,14 @@ describe LokaliseRails::TaskDefinition::Exporter do
       it 'halts when the API key is not set' do
         allow(LokaliseRails).to receive(:api_token).and_return(nil)
 
-        expect(-> { described_class.export! }).to output(/API token is not set/).to_stdout
+        expect(-> { described_class.export! }).to raise_error(LokaliseRails::Error, /API token is not set/i)
         expect(LokaliseRails).to have_received(:api_token)
       end
 
       it 'halts when the project_id is not set' do
-        allow(LokaliseRails).to receive(:project_id).and_return(nil)
-
-        expect(-> { described_class.export! }).to output(/Project ID is not set/).to_stdout
-        expect(LokaliseRails).to have_received(:project_id)
+        allow_project_id nil do
+          expect(-> { described_class.export! }).to raise_error(LokaliseRails::Error, /ID is not set/i)
+        end
       end
     end
 
@@ -108,17 +107,12 @@ describe LokaliseRails::TaskDefinition::Exporter do
     end
 
     describe '.export!' do
-      it 'rescues from export errors' do
+      it 're-raises export errors' do
         allow_project_id
 
-        processes = VCR.use_cassette('upload_files_error') do
-          described_class.export!
+        VCR.use_cassette('upload_files_error') do
+          expect { described_class.export! }.to raise_error(Lokalise::Error::BadRequest, /Unknown `lang_iso`/)
         end
-
-        expect(processes.length).to eq(1)
-        process = processes.first
-        expect(process.project_id).to eq(LokaliseRails.project_id)
-        expect(process.status).to eq('queued')
       end
     end
 

data/spec/lib/lokalise_rails/task_definition/importer_spec.rb

@@ -4,29 +4,34 @@ describe LokaliseRails::TaskDefinition::Importer do
   describe '.open_and_process_zip' do
     let(:faulty_trans) { "#{Rails.root}/public/faulty_trans.zip" }
 
-    it 'rescues from errors during file processing' do
+    it 're-raises errors during file processing' do
       expect(-> { described_class.open_and_process_zip(faulty_trans) }).
-        to output(/Psych::DisallowedClass/).to_stdout
+        to raise_error(Psych::DisallowedClass, /Error when trying to process fail\.yml/)
+    end
+
+    it 're-raises errors during file opening' do
+      expect(-> { described_class.open_and_process_zip('http://fake.url/wrong/path.zip') }).
+        to raise_error(SocketError, /Failed to open TCP connection/)
     end
   end
 
   describe '.download_files' do
     it 'returns a proper download URL' do
-      allow(LokaliseRails).to receive(:project_id).and_return('189934715f57a162257d74.88352370')
-      response = VCR.use_cassette('download_files') do
-        described_class.download_files
-      end
+      allow_project_id '189934715f57a162257d74.88352370' do
+        response = VCR.use_cassette('download_files') do
+          described_class.download_files
+        end
 
-      expect(LokaliseRails).to have_received(:project_id)
-      expect(response['project_id']).to eq('189934715f57a162257d74.88352370')
-      expect(response['bundle_url']).to include('s3-eu-west-1.amazonaws.com')
+        expect(response['project_id']).to eq('189934715f57a162257d74.88352370')
+        expect(response['bundle_url']).to include('s3-eu-west-1.amazonaws.com')
+      end
     end
 
-    it 'rescues from errors during file download' do
+    it 're-raises errors during file download' do
       allow_project_id 'invalid'
       VCR.use_cassette('download_files_error') do
         expect(-> { described_class.download_files }).
-          to output(/Lokalise::Error::BadRequest/).to_stdout
+          to raise_error(Lokalise::Error::BadRequest, /Invalid `project_id` parameter/)
       end
     end
   end
@@ -34,16 +39,16 @@ describe LokaliseRails::TaskDefinition::Importer do
   describe '.import!' do
     it 'halts when the API key is not set' do
       allow(LokaliseRails).to receive(:api_token).and_return(nil)
-      expect(-> { described_class.import! }).to output(/API token is not set/).to_stdout
+      expect(-> { described_class.import! }).to raise_error(LokaliseRails::Error, /API token is not set/i)
       expect(LokaliseRails).to have_received(:api_token)
       expect(count_translations).to eq(0)
     end
 
     it 'halts when the project_id is not set' do
-      allow(LokaliseRails).to receive(:project_id).and_return(nil)
-      expect(-> { described_class.import! }).to output(/Project ID is not set/).to_stdout
-      expect(LokaliseRails).to have_received(:project_id)
-      expect(count_translations).to eq(0)
+      allow_project_id nil do
+        expect(-> { described_class.import! }).to raise_error(LokaliseRails::Error, /ID is not set/i)
+        expect(count_translations).to eq(0)
+      end
     end
   end
 end

data/spec/lib/lokalise_rails_spec.rb

@@ -73,5 +73,23 @@ describe LokaliseRails do
       allow(fake_class).to receive(:skip_file_export=).with(cond)
       fake_class.skip_file_export = cond
     end
+
+    it 'is possible to set translations_loader' do
+      runner = ->(f) { f.to_json }
+      allow(fake_class).to receive(:translations_loader=).with(runner)
+      fake_class.translations_loader = runner
+    end
+
+    it 'is possible to set translations_converter' do
+      runner = ->(f) { f.to_json }
+      allow(fake_class).to receive(:translations_converter=).with(runner)
+      fake_class.translations_converter = runner
+    end
+
+    it 'is possible to set lang_iso_inferer' do
+      runner = ->(f) { f.to_json }
+      allow(fake_class).to receive(:lang_iso_inferer=).with(runner)
+      fake_class.lang_iso_inferer = runner
+    end
   end
 end

data/spec/lib/tasks/export_task_spec.rb

@@ -1,7 +1,44 @@
 # frozen_string_literal: true
 
 RSpec.describe LokaliseRails do
+  it 'halts when the API key is not set' do
+    allow(described_class).to receive(:api_token).and_return(nil)
+
+    expect(export_executor).to raise_error(SystemExit, /API token is not set/i)
+    expect(described_class).to have_received(:api_token)
+  end
+
+  it 'halts when the project ID is not set' do
+    allow_project_id nil do
+      expect(export_executor).to raise_error(SystemExit, /ID is not set/i)
+    end
+  end
+
   it 'runs export rake task properly' do
     expect(export_executor).to output(/complete!/).to_stdout
   end
+
+  context 'with two translation files' do
+    let(:filename_ru) { 'ru.yml' }
+    let(:path_ru) { "#{Rails.root}/config/locales/#{filename_ru}" }
+    let(:relative_name_ru) { filename_ru }
+
+    before :all do
+      add_translation_files! with_ru: true
+    end
+
+    after :all do
+      rm_translation_files
+    end
+
+    describe '.export!' do
+      it 're-raises export errors' do
+        allow_project_id
+
+        VCR.use_cassette('upload_files_error') do
+          expect(export_executor).to raise_error(SystemExit, /Unknown `lang_iso`/)
+        end
+      end
+    end
+  end
 end

data/spec/lib/tasks/import_task_spec.rb

@@ -7,6 +7,19 @@ RSpec.describe LokaliseRails do
   let(:local_trans) { "#{Rails.root}/public/trans.zip" }
   let(:remote_trans) { 'https://github.com/bodrovis/lokalise_rails/blob/master/spec/dummy/public/trans.zip?raw=true' }
 
+  it 'halts when the API key is not set' do
+    allow(described_class).to receive(:api_token).and_return(nil)
+
+    expect(import_executor).to raise_error(SystemExit, /API token is not set/i)
+    expect(described_class).to have_received(:api_token)
+  end
+
+  it 'halts when the project ID is not set' do
+    allow_project_id nil do
+      expect(import_executor).to raise_error(SystemExit, /ID is not set/i)
+    end
+  end
+
   context 'when directory is empty' do
     before do
       mkdir_locales

data/spec/spec_helper.rb

@@ -20,7 +20,6 @@ Dir["#{File.dirname(__FILE__)}/support/**/*.rb"].sort.each { |f| require f }
 
 # Configure Rails Environment
 ENV['RAILS_ENV'] = 'test'
-
 require_relative '../spec/dummy/config/environment'
 ENV['RAILS_ROOT'] ||= "#{File.dirname(__FILE__)}../../../spec/dummy"
 

data/spec/support/spec_addons.rb

@@ -1,8 +1,11 @@
 # frozen_string_literal: true
 
 module SpecAddons
-  def allow_project_id(value = '189934715f57a162257d74.88352370')
-    allow(LokaliseRails).to receive(:project_id).
-      and_return(value)
+  def allow_project_id(value = ENV['LOKALISE_PROJECT_ID'])
+    allow(LokaliseRails).to receive(:project_id).and_return(value)
+    return unless block_given?
+
+    yield
+    expect(LokaliseRails).to have_received(:project_id)
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: lokalise_rails
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.4.0
 platform: ruby
 authors:
 - Ilya Bodrov
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-10-23 00:00:00.000000000 Z
+date: 2021-07-29 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: '3.1'
+        version: '4.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.1'
+        version: '4.0'
 - !ruby/object:Gem::Dependency
   name: rubyzip
   requirement: !ruby/object:Gem::Requirement
@@ -44,14 +44,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.1'
+        version: '0.2'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.1'
+        version: '0.2'
 - !ruby/object:Gem::Dependency
   name: dotenv
   requirement: !ruby/object:Gem::Requirement
@@ -72,14 +72,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 6.0.3
+        version: 6.1.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 6.0.3
+        version: 6.1.0
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -114,14 +114,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.60'
+        version: '1.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.60'
+        version: '1.0'
 - !ruby/object:Gem::Dependency
   name: rubocop-performance
   requirement: !ruby/object:Gem::Requirement
@@ -142,14 +142,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.37'
+        version: 2.4.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.37'
+        version: 2.4.0
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
@@ -198,6 +198,7 @@ files:
 - lib/generators/lokalise_rails/install_generator.rb
 - lib/generators/templates/lokalise_rails_config.rb
 - lib/lokalise_rails.rb
+- lib/lokalise_rails/error.rb
 - lib/lokalise_rails/railtie.rb
 - lib/lokalise_rails/task_definition/base.rb
 - lib/lokalise_rails/task_definition/exporter.rb
@@ -243,7 +244,7 @@ homepage: https://github.com/bodrovis/lokalise_rails
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -258,8 +259,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.4
-signing_key: 
+rubygems_version: 3.2.24
+signing_key:
 specification_version: 4
 summary: Lokalise integration for Ruby on Rails
 test_files: