wovnrb 1.1.0 → 2.0.0

data/lib/wovnrb/services/html_replace_marker.rb

@@ -0,0 +1,38 @@
+module Wovnrb
+  class HtmlReplaceMarker
+    KEY_PREFIX = '__wovn-backend-ignored-key-'.freeze
+
+    def initialize
+      @current_key_number = 0
+      @mapped_values = []
+    end
+
+    # Add argument's value to mapping information with comment style key
+    def add_comment_value(value)
+      key = "<!-- #{generate_key} -->"
+      @mapped_values << [key, value]
+
+      key
+    end
+
+    def revert(marked_html)
+      i = @mapped_values.size
+      while i > 0
+        i -= 1
+        key, value = @mapped_values[i]
+        marked_html = marked_html.sub(key, value)
+      end
+
+      marked_html
+    end
+
+    private
+
+    def generate_key
+      next_key = "#{KEY_PREFIX}#{@current_key_number}"
+      @current_key_number += 1
+
+      next_key
+    end
+  end
+end

data/lib/wovnrb/services/wovn_logger.rb

@@ -5,6 +5,12 @@ module Wovnrb
   class WovnLogger
     include Singleton
 
+    class << self
+      def error(message)
+        instance.error(message)
+      end
+    end
+
     def initialize
       path = Store.instance.settings['log_path']
       if path
@@ -26,9 +32,7 @@ module Wovnrb
 
     def set_logger(logger)
       [:error].each do |method|
-        unless logger.respond_to? method
-          raise 'not suite for logger'
-        end
+        raise 'not suite for logger' unless logger.respond_to? method
       end
 
       @logger = logger
@@ -42,4 +46,4 @@ module Wovnrb
       end
     end
   end
-end
+end

data/lib/wovnrb/settings.rb

@@ -8,12 +8,14 @@ module Wovnrb
     def [](key)
       return @dynamic_settings[key] if @dynamic_settings.key?(key)
       return ignore_globs if key == 'ignore_globs'
+
       super(key)
     end
 
     def ignore_globs
       ignore_paths = self['ignore_paths']
-      return [] unless ignore_paths.kind_of?(Array)
+      return [] unless ignore_paths.is_a?(Array)
+
       ignore_paths.map { |pattern| Glob.new(pattern) }
     end
 
@@ -31,7 +33,7 @@ module Wovnrb
 
     DYNAMIC_KEYS = {
       'wovn_token' => 'project_token',
-      'wovn_ignore_paths' => 'ignore_paths',
-    }
+      'wovn_ignore_paths' => 'ignore_paths'
+    }.freeze
   end
 end

data/lib/wovnrb/store.rb

@@ -13,7 +13,7 @@ module Wovnrb
     include Singleton
 
     def self.default_settings
-      Settings.new.merge({
+      Settings.new.merge(
         'project_token' => '',
         'log_path' => 'log/wovn_error.log',
         'ignore_paths' => [],
@@ -22,7 +22,7 @@ module Wovnrb
         'url_pattern_reg' => "/(?<lang>[^/.?]+)",
         'query' => [],
         'ignore_class' => [],
-        'api_url' => 'https://api.wovn.io/v0/values',
+        'api_url' => 'https://wovn.global.ssl.fastly.net/v0/',
         'api_timeout_seconds' => 0.5,
         'default_lang' => 'en',
         'supported_langs' => ['en'],
@@ -30,11 +30,11 @@ module Wovnrb
         'test_url' => '',
         'cache_megabytes' => nil,
         'ttl_seconds' => nil,
-        'use_proxy' => false,  # use env['HTTP_X_FORWARDED_HOST'] instead of env['HTTP_HOST'] and env['SERVER_NAME'] when this setting is true.
+        'use_proxy' => false, # use env['HTTP_X_FORWARDED_HOST'] instead of env['HTTP_HOST'] and env['SERVER_NAME'] when this setting is true.
         'custom_lang_aliases' => {},
-        'wovn_dev_mode' => false,
-        'translate_fragment' => true
-      })
+        'translate_fragment' => true,
+        'wovn_dev_mode' => false
+      )
     end
 
     def initialize
@@ -54,7 +54,7 @@ module Wovnrb
     #
     # @return [Boolean] Returns true if the token is valid, and false if it is not
     def valid_token?(token)
-      return !token.nil? && (token.length == 5 || token.length == 6)
+      !token.nil? && (token.length == 5 || token.length == 6)
     end
 
     # Returns true or false based on whether the settings are valid or not, logs any invalid settings to ../error.log
@@ -62,8 +62,8 @@ module Wovnrb
     # @return [Boolean] Returns true if the settings are valid, and false if they are not
     def valid_settings?
       valid = true
-      errors = [];
-      #if valid_token?(!settings.has_key?('project_token') || settings['project_token'].length < 5 || settings['project_token'].length > 6
+      errors = []
+      # if valid_token?(!settings.has_key?('project_token') || settings['project_token'].length < 5 || settings['project_token'].length > 6
       if !valid_token?(settings['project_token'])
         errors.push("Project token #{settings['project_token']} is not valid.")
       end
@@ -98,7 +98,7 @@ module Wovnrb
           WovnLogger.instance.error(e)
         end
       end
-      return valid
+      valid
     end
 
     # Returns the settings object, pulling from Rails config the first time this is called
@@ -148,34 +148,43 @@ module Wovnrb
         @settings['url_pattern_reg'] = "^(?<lang>[^.]+)\."
       end
 
-      if @settings['test_mode'] != true || @settings['test_mode'] != 'on'
-        @settings['test_mode'] = false
-      else
-        @settings['test_mode'] = true
-      end
+      @settings['test_mode'] = if @settings['test_mode'] != true || @settings['test_mode'] != 'on'
+                                 false
+                               else
+                                 true
+                               end
+    end
 
-      if wovn_dev_mode? && @settings['api_url'] == Store.default_settings['api_url']
-        @settings['api_url'] = "#{wovn_protocol}://api.#{wovn_host}/v0/values"
-      end
+    def custom_lang_aliases
+      @setttings['custom_lang_aliases'] || {}
     end
 
-    def wovn_dev_mode?
-      @settings['wovn_dev_mode']
+    def default_lang
+      @settings['default_lang']
     end
 
-    def wovn_protocol
-      wovn_dev_mode? ? 'http' : 'https'
+    def default_lang_alias
+      custom_alias = custom_lang_aliases[default_lang]
+      custom_alias || default_lang
+    end
+
+    def supported_langs
+      @settings['supported_langs'] || []
     end
 
     def wovn_host
-      wovn_dev_mode? ? 'dev-wovn.io:3000' : 'wovn.io'
+      if @settings['wovn_dev_mode']
+        'dev-wovn.io:3000'
+      else
+        'wovn.io'
+      end
     end
 
     private
 
-    def stringify_keys!(h)
-      h.keys.each do |k|
-        h[k.to_s] = h.delete(k)
+    def stringify_keys!(hash)
+      hash.keys.each do |k|
+        hash[k.to_s] = hash.delete(k)
       end
     end
   end

data/lib/wovnrb/text_caches/cache_base.rb

@@ -13,11 +13,12 @@ module Wovnrb
     @@singleton_cache = nil
     def self.get_single
       raise 'cache is not initialized' unless @@singleton_cache
+
       @@singleton_cache
     end
 
     def self.set_single(config)
-      @@singleton_cache = self.build(config)
+      @@singleton_cache = build(config)
     end
 
     def self.reset_cache
@@ -49,4 +50,4 @@ module Wovnrb
       raise NotImplementedError.new('put is not defined')
     end
   end
-end
+end

data/lib/wovnrb/text_caches/memory_cache.rb

@@ -41,11 +41,11 @@ module Wovnrb
     end
 
     def compress(value)
-      LZ4::compress(value)
+      LZ4.compress(value)
     end
 
     def decompress(value)
-      LZ4::decompress(value, value.bytesize, 'UTF-8')
+      LZ4.decompress(value, value.bytesize, 'UTF-8')
     end
   end
 end

data/lib/wovnrb/version.rb

@@ -1,3 +1,3 @@
 module Wovnrb
-  VERSION = "1.1.0"
+  VERSION = '2.0.0'.freeze
 end

data/test/fixtures/html/test.html

@@ -0,0 +1,8 @@
+<html>
+  <head>
+    <title>Original Content</title>
+  </head>
+  <body>
+    <p>This is a test.</p>
+  </body>
+</html>

data/test/fixtures/html/test_translated.html

@@ -0,0 +1,8 @@
+<html>
+  <head>
+    <title>Translated Content</title>
+  </head>
+  <body>
+    <p>This is a test.</p>
+  </body>
+</html>

data/test/lib/api_translator_test.rb

@@ -0,0 +1,109 @@
+require 'test_helper'
+
+module Wovnrb
+  class ApiTranslatorTest < WovnMiniTest
+    def test_translate
+      assert_translation('test.html', 'test_translated.html', true)
+    end
+
+    def test_translate_falls_back_to_original_body_if_exception
+      Net::HTTP.any_instance.expects(:request).raises
+      assert_translation('test.html', 'test_translated.html', false, nil)
+    end
+
+    def test_translate_falls_back_to_original_body_if_api_error
+      assert_translation('test.html', 'test_translated.html', false, status_code: 500)
+    end
+
+    def test_translate_falls_back_to_original_body_if_api_response_is_not_compressed
+      assert_translation('test.html', 'test_translated.html', false, encoding: 'unknown')
+    end
+
+    private
+
+    def assert_translation(original_html_fixture, translated_html_fixture, success_expected, response = { encoding: 'gzip', status_code: 200 })
+      original_html = File.read("test/fixtures/html/#{original_html_fixture}")
+      translated_html = File.read("test/fixtures/html/#{translated_html_fixture}")
+      actual_translated_html = translate(original_html, translated_html, response)
+
+      if success_expected
+        assert_equal(actual_translated_html, translated_html)
+      else
+        assert_equal(actual_translated_html, original_html)
+      end
+    end
+
+    def translate(original_html, translated_html, response)
+      settings = {
+        'project_token' => '123456',
+        'custom_lang_aliases' => { 'ja' => 'Japanese' },
+        'default_lang' => 'en',
+        'url_pattern' => 'subdomain',
+        'url_pattern_reg' => '^(?<lang>[^.]+).'
+      }
+      store = Wovnrb::Store.instance
+      store.update_settings(settings)
+      headers = Wovnrb::Headers.new(
+        Wovnrb.get_env('url' => 'http://fr.wovn.io/test'),
+        Wovnrb.get_settings(settings)
+      )
+      api_translator = ApiTranslator.new(store, headers)
+      translation_request_stub = stub_translation_api_request(store, headers, original_html, translated_html, response)
+
+      actual_translated_html = api_translator.translate(original_html)
+      assert_requested(translation_request_stub, times: 1) if translation_request_stub
+      actual_translated_html
+    end
+
+    def stub_translation_api_request(store, headers, original_html, translated_html, response)
+      if response
+        cache_key = generate_cache_key(store, original_html)
+        api_url = "wovn.global.ssl.fastly.net/v0/translation?cache_key=#{cache_key}"
+        compressed_data = compress(generate_data(original_html))
+        headers = {
+          'Accept' => '*/*',
+          'Accept-Encoding' => 'gzip',
+          'Content-Length' => compressed_data.bytesize,
+          'Content-Type' => 'application/octet-stream',
+          'User-Agent' => 'Ruby'
+        }
+        compressed_response = compress("{\"body\":\"#{translated_html.gsub("\n", '\n')}\"}")
+        response_headers = { 'Content-Encoding' => response[:encoding] || 'gzip' }
+        stub = stub_request(:post, api_url)
+               .with(body: compressed_data, headers: headers)
+               .to_return(status: response[:status_code] || 200, body: compressed_response, headers: response_headers)
+
+        stub
+      end
+    end
+
+    def generate_cache_key(store, original_html)
+      settings_hash = Digest::MD5.hexdigest(JSON.dump(store.settings))
+      body_hash = Digest::MD5.hexdigest(original_html)
+      escaped_key = CGI.escape("token=123456&settings_hash=#{settings_hash}&body_hash=#{body_hash}&path=/test&lang=fr")
+
+      "(#{escaped_key})"
+    end
+
+    def generate_data(original_html)
+      data = {
+        'url' => 'http://wovn.io/test',
+        'token' => '123456',
+        'lang_code' => 'fr',
+        'url_pattern' => 'subdomain',
+        'product' => 'WOVN.rb',
+        'version' => VERSION,
+        'body' => original_html,
+        'custom_lang_aliases' => '{"ja":"Japanese"}'
+      }
+
+      data.map { |key, value| "#{key}=#{CGI.escape(value)}" }.join('&')
+    end
+
+    def compress(string)
+      gzip = Zlib::GzipWriter.new(StringIO.new)
+      gzip << string
+      gzip.close.string
+    end
+  end
+end

data/test/lib/headers_test.rb

@@ -1,12 +1,7 @@
 require 'test_helper'
-require 'wovnrb/headers'
-require 'wovnrb/lang'
-require 'minitest/autorun'
-require 'pry'
 
 module Wovnrb
   class LangTest < WovnMiniTest
-
     #########################
     # INITIALIZE
     #########################
@@ -16,13 +11,6 @@ module Wovnrb
       refute_nil(h)
     end
 
-    # def test_initialize_env
-    #   env = Wovnrb.get_env
-    #   h = Wovnrb::Headers.new(env, {})
-    #   binding.pry
-    #   #assert_equal(''
-    # end
-
     def test_initialize_with_simple_url
       h = Wovnrb::Headers.new(Wovnrb.get_env('url' => 'https://wovn.io'), Wovnrb.get_settings)
       assert_equal('wovn.io/', h.url)
@@ -110,6 +98,47 @@ module Wovnrb
       assert_equal('https', h.protocol)
     end
 
+    def test_pathname_with_trailing_slash_if_present_when_trailing_slash_is_not_present
+      env = Wovnrb.get_env('REQUEST_URI' => 'http://page.com/test')
+      headers = Wovnrb::Headers.new(env, Wovnrb.get_settings)
+
+      assert_equal('/test', headers.pathname_with_trailing_slash_if_present)
+    end
+
+    def test_pathname_with_trailing_slash_if_present_with_default_lang_when_trailing_slash_is_present
+      env = Wovnrb.get_env('REQUEST_URI' => 'http://page.com/test/')
+      headers = Wovnrb::Headers.new(env, Wovnrb.get_settings)
+
+      assert_equal('/test/', headers.pathname_with_trailing_slash_if_present)
+    end
+
+    def test_pathname_with_trailing_slash_if_present_with_subdomain_lang_when_trailing_slash_is_present
+      headers = Wovnrb::Headers.new(
+        Wovnrb.get_env('REQUEST_URI' => 'http://ja.page.com/test/'),
+        Wovnrb.get_settings('url_pattern' => 'subdomain', 'url_pattern_reg' => '^(?<lang>[^.]+).')
+      )
+
+      assert_equal('/test/', headers.pathname_with_trailing_slash_if_present)
+    end
+
+    def test_pathname_with_trailing_slash_if_present_with_path_lang_when_trailing_slash_is_present
+      headers = Wovnrb::Headers.new(
+        Wovnrb.get_env('REQUEST_URI' => 'http://page.com/ja/test/'),
+        Wovnrb.get_settings('url_pattern' => 'path', 'url_pattern_reg' => '/(?<lang>[^/.?]+)')
+      )
+
+      assert_equal('/test/', headers.pathname_with_trailing_slash_if_present)
+    end
+
+    def test_pathname_with_trailing_slash_if_present_with_query_lang_when_trailing_slash_is_present
+      headers = Wovnrb::Headers.new(
+        Wovnrb.get_env('REQUEST_URI' => 'http://page.com/test/?wovn=ja'),
+        Wovnrb.get_settings('url_pattern' => 'query', 'url_pattern_reg' => '((\\?.*&)|\\?)wovn=(?<lang>[^&]+)(&|$)')
+      )
+
+      assert_equal('/test/', headers.pathname_with_trailing_slash_if_present)
+    end
+
     #########################
     # REDIRECT_LOCATION
     #########################
@@ -117,16 +146,16 @@ module Wovnrb
     def test_redirect_location_without_custom_lang_code
       h = Wovnrb::Headers.new(
         Wovnrb.get_env('url' => 'http://wovn.io/contact', 'HTTP_X_FORWARDED_HOST' => 'wovn.io'),
-        Wovnrb.get_settings('url_pattern' => 'subdomain', 'url_pattern_reg' => '^(?<lang>[^.]+).'),
+        Wovnrb.get_settings('url_pattern' => 'subdomain', 'url_pattern_reg' => '^(?<lang>[^.]+).')
       )
       assert_equal('http://ja.wovn.io/contact', h.redirect_location('ja'))
     end
 
     def test_redirect_location_without_custom_lang_code
-      Store.instance.update_settings({'custom_lang_aliases' => {'ja' => 'staging-ja'}})
+      Store.instance.update_settings('custom_lang_aliases' => { 'ja' => 'staging-ja' })
       h = Wovnrb::Headers.new(
         Wovnrb.get_env('url' => 'http://wovn.io/contact', 'HTTP_X_FORWARDED_HOST' => 'wovn.io'),
-        Wovnrb.get_settings('url_pattern' => 'subdomain', 'url_pattern_reg' => '^(?<lang>[^.]+).'),
+        Wovnrb.get_settings('url_pattern' => 'subdomain', 'url_pattern_reg' => '^(?<lang>[^.]+).')
       )
       assert_equal('http://staging-ja.wovn.io/contact', h.redirect_location('ja'))
     end
@@ -140,12 +169,12 @@ module Wovnrb
         Wovnrb.get_env(
           'SERVER_NAME' => 'ja.wovn.io',
           'REQUEST_URI' => '/test',
-          'HTTP_REFERER' => 'http://ja.wovn.io/test',
+          'HTTP_REFERER' => 'http://ja.wovn.io/test'
         ),
         Wovnrb.get_settings(
           'url_pattern' => 'subdomain',
-          'url_pattern_reg' => '^(?<lang>[^.]+).',
-        ),
+          'url_pattern_reg' => '^(?<lang>[^.]+).'
+        )
       )
       env = h.request_out('ja')
       assert_equal('ja', env['wovnrb.target_lang'])
@@ -154,7 +183,7 @@ module Wovnrb
     def test_request_out_with_wovn_target_lang_header_using_path
       h = Wovnrb::Headers.new(
         Wovnrb.get_env('REQUEST_URI' => '/ja/test', 'HTTP_REFERER' => 'http://wovn.io/ja/test'),
-        Wovnrb.get_settings,
+        Wovnrb.get_settings
       )
       env = h.request_out('ja')
       assert_equal('ja', env['wovnrb.target_lang'])
@@ -165,8 +194,8 @@ module Wovnrb
         Wovnrb.get_env('REQUEST_URI' => 'test?wovn=ja', 'HTTP_REFERER' => 'http://wovn.io/test'),
         Wovnrb.get_settings(
           'url_pattern' => 'query',
-          'url_pattern_reg' => "((\\?.*&)|\\?)wovn=(?<lang>[^&]+)(&|$)",
-        ),
+          'url_pattern_reg' => '((\\?.*&)|\\?)wovn=(?<lang>[^&]+)(&|$)'
+        )
       )
       env = h.request_out('ja')
       assert_equal('ja', env['wovnrb.target_lang'])
@@ -175,7 +204,7 @@ module Wovnrb
     def test_request_out_with_use_proxy_false
       h = Wovnrb::Headers.new(
         Wovnrb.get_env('url' => 'http://localhost/contact', 'HTTP_X_FORWARDED_HOST' => 'ja.wovn.io'),
-        Wovnrb.get_settings('url_pattern' => 'subdomain', 'url_pattern_reg' => '^(?<lang>[^.]+).'),
+        Wovnrb.get_settings('url_pattern' => 'subdomain', 'url_pattern_reg' => '^(?<lang>[^.]+).')
       )
       env = h.request_out('ja')
       assert_equal('ja.wovn.io', env['HTTP_X_FORWARDED_HOST'])
@@ -184,7 +213,7 @@ module Wovnrb
     def test_request_out_with_use_proxy_true
       h = Wovnrb::Headers.new(
         Wovnrb.get_env('url' => 'http://localhost/contact', 'HTTP_X_FORWARDED_HOST' => 'ja.wovn.io'),
-        Wovnrb.get_settings('url_pattern' => 'subdomain', 'url_pattern_reg' => '^(?<lang>[^.]+).', 'use_proxy' => true),
+        Wovnrb.get_settings('url_pattern' => 'subdomain', 'url_pattern_reg' => '^(?<lang>[^.]+).', 'use_proxy' => true)
       )
       env = h.request_out('ja')
       assert_equal('wovn.io', env['HTTP_X_FORWARDED_HOST'])
@@ -195,12 +224,12 @@ module Wovnrb
         Wovnrb.get_env(
           'SERVER_NAME' => 'ja.wovn.io',
           'REQUEST_URI' => '/test',
-          'HTTP_REFERER' => 'http://ja.wovn.io/test',
+          'HTTP_REFERER' => 'http://ja.wovn.io/test'
         ),
         Wovnrb.get_settings(
           'url_pattern' => 'subdomain',
-          'url_pattern_reg' => '^(?<lang>[^.]+).',
-        ),
+          'url_pattern_reg' => '^(?<lang>[^.]+).'
+        )
       )
       env = h.request_out('ja')
       assert_equal('http://wovn.io/test', env['HTTP_REFERER'])
@@ -209,41 +238,41 @@ module Wovnrb
     def test_request_out_http_referer_path
       h = Wovnrb::Headers.new(
         Wovnrb.get_env('REQUEST_URI' => '/ja/test', 'HTTP_REFERER' => 'http://wovn.io/ja/test'),
-        Wovnrb.get_settings,
+        Wovnrb.get_settings
       )
       env = h.request_out('ja')
       assert_equal('http://wovn.io/test', env['HTTP_REFERER'])
     end
 
     def test_request_out_http_referer_subdomain_with_custom_lang_code
-      Store.instance.update_settings({'custom_lang_aliases' => {'ja' => 'staging-ja'}})
+      Store.instance.update_settings('custom_lang_aliases' => { 'ja' => 'staging-ja' })
       h = Wovnrb::Headers.new(
         Wovnrb.get_env(
           'SERVER_NAME' => 'staging-ja.wovn.io',
           'REQUEST_URI' => '/test',
-          'HTTP_REFERER' => 'http://staging-ja.wovn.io/test',
+          'HTTP_REFERER' => 'http://staging-ja.wovn.io/test'
         ),
         Wovnrb.get_settings(
           'url_pattern' => 'subdomain',
-          'url_pattern_reg' => '^(?<lang>[^.]+).',
-        ),
+          'url_pattern_reg' => '^(?<lang>[^.]+).'
+        )
       )
       env = h.request_out('ja')
       assert_equal('http://wovn.io/test', env['HTTP_REFERER'])
     end
 
     def test_out_http_referer_subdomain_with_custom_lang_code
-      Store.instance.update_settings({'custom_lang_aliases' => {'ja' => 'staging-ja'}})
+      Store.instance.update_settings('custom_lang_aliases' => { 'ja' => 'staging-ja' })
       h = Wovnrb::Headers.new(
         Wovnrb.get_env(
           'SERVER_NAME' => 'staging-ja.wovn.io',
           'REQUEST_URI' => '/test',
-          'HTTP_REFERER' => 'http://staging-ja.wovn.io/test',
+          'HTTP_REFERER' => 'http://staging-ja.wovn.io/test'
         ),
         Wovnrb.get_settings(
           'url_pattern' => 'subdomain',
-          'url_pattern_reg' => '^(?<lang>[^.]+).',
-        ),
+          'url_pattern_reg' => '^(?<lang>[^.]+).'
+        )
       )
       headers = h.request_out('ja')
       assert_equal('http://wovn.io/test', headers['HTTP_REFERER'])
@@ -256,12 +285,12 @@ module Wovnrb
         Wovnrb.get_env(
           'SERVER_NAME' => 'wovn.io',
           'REQUEST_URI' => '/test',
-          'HTTP_REFERER' => 'http://wovn.io/test',
+          'HTTP_REFERER' => 'http://wovn.io/test'
         ),
         Wovnrb.get_settings(
           'url_pattern' => 'subdomain',
-          'url_pattern_reg' => '^(?<lang>[^.]+).',
-        ),
+          'url_pattern_reg' => '^(?<lang>[^.]+).'
+        )
       )
       headers = h.request_out(h.lang_code)
       assert_equal('http://wovn.io/test', headers['HTTP_REFERER'])
@@ -274,12 +303,12 @@ module Wovnrb
         Wovnrb.get_env(
           'SERVER_NAME' => 'wovn.io',
           'REQUEST_URI' => '/test',
-          'HTTP_REFERER' => 'http://wovn.io/test',
+          'HTTP_REFERER' => 'http://wovn.io/test'
         ),
         Wovnrb.get_settings(
           'url_pattern' => 'path',
-          'url_pattern_reg' => '/(?<lang>[^/.?]+)',
-        ),
+          'url_pattern_reg' => '/(?<lang>[^/.?]+)'
+        )
       )
       headers = h.request_out(h.lang_code)
       assert_equal('http://wovn.io/test', headers['HTTP_REFERER'])
@@ -292,12 +321,12 @@ module Wovnrb
         Wovnrb.get_env(
           'SERVER_NAME' => 'wovn.io',
           'REQUEST_URI' => '/test',
-          'HTTP_REFERER' => 'http://wovn.io/test',
+          'HTTP_REFERER' => 'http://wovn.io/test'
         ),
         Wovnrb.get_settings(
           'url_pattern' => 'query',
-          'url_pattern_reg' => '((\\?.*&)|\\?)wovn=(?<lang>[^&]+)(&|$)',
-        ),
+          'url_pattern_reg' => '((\\?.*&)|\\?)wovn=(?<lang>[^&]+)(&|$)'
+        )
       )
       headers = h.request_out(h.lang_code)
       assert_equal('http://wovn.io/test', headers['HTTP_REFERER'])
@@ -310,12 +339,12 @@ module Wovnrb
         Wovnrb.get_env(
           'SERVER_NAME' => 'ja.wovn.io',
           'REQUEST_URI' => '/test',
-          'HTTP_REFERER' => 'http://ja.wovn.io/test',
+          'HTTP_REFERER' => 'http://ja.wovn.io/test'
         ),
         Wovnrb.get_settings(
           'url_pattern' => 'subdomain',
-          'url_pattern_reg' => '^(?<lang>[^.]+).',
-        ),
+          'url_pattern_reg' => '^(?<lang>[^.]+).'
+        )
       )
       headers = h.out(h.request_out('ja'))
       assert_equal('ja', headers['wovnrb.target_lang'])
@@ -324,7 +353,7 @@ module Wovnrb
     def test_out_with_wovn_target_lang_header_using_path
       h = Wovnrb::Headers.new(
         Wovnrb.get_env('REQUEST_URI' => '/ja/test', 'HTTP_REFERER' => 'http://wovn.io/ja/test'),
-        Wovnrb.get_settings,
+        Wovnrb.get_settings
       )
       headers = h.out(h.request_out('ja'))
       assert_equal('ja', headers['wovnrb.target_lang'])
@@ -335,8 +364,8 @@ module Wovnrb
         Wovnrb.get_env('REQUEST_URI' => 'test?wovn=ja', 'HTTP_REFERER' => 'http://wovn.io/test'),
         Wovnrb.get_settings(
           'url_pattern' => 'query',
-          'url_pattern_reg' => "((\\?.*&)|\\?)wovn=(?<lang>[^&]+)(&|$)",
-        ),
+          'url_pattern_reg' => '((\\?.*&)|\\?)wovn=(?<lang>[^&]+)(&|$)'
+        )
       )
       headers = h.out(h.request_out('ja'))
       assert_equal('ja', headers['wovnrb.target_lang'])
@@ -2643,7 +2672,7 @@ module Wovnrb
     def test_path_lang_sudomain_with_use_proxy_false
       h = Wovnrb::Headers.new(
         Wovnrb.get_env('url' => 'http://localhost:1234/test', 'HTTP_X_FORWARDED_HOST' => 'zh-cht.wovn.io'),
-        Wovnrb.get_settings('url_pattern' => 'subdomain', 'url_pattern_reg' => '^(?<lang>[^.]+).'),
+        Wovnrb.get_settings('url_pattern' => 'subdomain', 'url_pattern_reg' => '^(?<lang>[^.]+).')
       )
       assert_equal('', h.path_lang)
     end
@@ -2652,7 +2681,7 @@ module Wovnrb
       env = Wovnrb.get_env('url' => 'http://localhost:1234/test', 'HTTP_X_FORWARDED_HOST' => 'zh-cht.wovn.io')
       h = Wovnrb::Headers.new(
         env,
-        Wovnrb.get_settings('url_pattern' => 'subdomain', 'url_pattern_reg' => '^(?<lang>[^.]+).', 'use_proxy' => true),
+        Wovnrb.get_settings('url_pattern' => 'subdomain', 'url_pattern_reg' => '^(?<lang>[^.]+).', 'use_proxy' => true)
       )
       assert_equal('zh-CHT', h.path_lang)
     end
@@ -6217,13 +6246,13 @@ module Wovnrb
     end
 
     def test_remove_lang_subdomain_with_custom_lang_alias
-      Store.instance.update_settings({'custom_lang_aliases' => {'fr' => 'staging-fr'}})
+      Store.instance.update_settings('custom_lang_aliases' => { 'fr' => 'staging-fr' })
       h = Wovnrb::Headers.new(Wovnrb.get_env, Wovnrb.get_settings('url_pattern' => 'subdomain'))
 
-      uri_without_scheme = h.remove_lang("staging-fr.wovn.io/", 'fr')
+      uri_without_scheme = h.remove_lang('staging-fr.wovn.io/', 'fr')
       assert_equal('wovn.io/', uri_without_scheme)
 
-      uri_with_scheme = h.remove_lang("https://staging-fr.wovn.io", 'fr')
+      uri_with_scheme = h.remove_lang('https://staging-fr.wovn.io', 'fr')
       assert_equal('https://wovn.io', uri_with_scheme)
     end
   end