wovnrb 0.2.17 → 0.2.18

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 7348fe530bd3a1abe6f31aa1013a8262beca4153
-  data.tar.gz: facf8da674510feed09ef64b53ea50a769f9fb40
+  metadata.gz: 7bdcfd40914010805b291fbc68132434f9b95d20
+  data.tar.gz: 77a34e93cf0962516eab31bbe004ba2c4e4c8fd7
 SHA512:
-  metadata.gz: 388bf3245e438c050b83f28c5417a7b275a7892318319e048b1c7d3c21e302bbd57b393c650a1134bc85658c2a4216201830554e34404bb8a8005b28cefc247a
-  data.tar.gz: 1b6632a9627cc7ff68fee160e1c91e12389eaec80f606748355751111beed03984c122f0f4f40f8ebfcba5b411e76b285c7de2f2c4f93d4829e2b0cee57d0e47
+  metadata.gz: e91cb4c12e417853115fe9e6e4f175dcf6139218d388f96e1bedf6320749220f6ab16d18c323a79b5b305532bf095ef5896871861953f4012cab247883cdab33
+  data.tar.gz: 947e717e78b16d35597f68e91351444ac5d7cf62fa0bbf839fe02350e1699dc6335c9df4762639811f12386ff7612e13c6bb9b4489be82efa11b4d9925a4cdca

data/README.md

@@ -38,7 +38,7 @@ Insert the following into either config/application.rb or config/environments/.
 ...
 
 config.wovnrb = {
-  :user_token => '2Wle3'
+  :project_token => '2Wle3'
 }
 
 ...
@@ -54,7 +54,7 @@ Insert the following into either the Application File or config.ru.
 require 'wovnrb'
 
 use Wovnrb::Interceptor, {
-  :user_token => '2Wle3'
+  :project_token => '2Wle3'
 }
 
 ...
@@ -68,14 +68,14 @@ WOVN.io Ruby Library's valid parameters are as follows.
 
 Parameter Name | Required | Default Setting
 -------------- | -------- | ----------------
-user_token     | yes      | ''
+project_token  | yes      | ''
 url_pattern    | yes      | 'path'
 query          |          | []
 default_lang   | yes      | 'en'
 
-### 2.1. user_token
+### 2.1. project_token
 
-Set your WOVN.io Account's user token. This parameter is required.
+Set your WOVN.io Account's Project token. This parameter is required.
 
 ### 2.2. url_pattern
 

data/lib/wovnrb/api_data.rb

@@ -35,7 +35,7 @@ module Wovnrb
     end
 
     def build_api_uri
-      t = CGI::escape(@store.settings['user_token'])
+      t = CGI::escape(@store.settings['project_token'])
       u = CGI::escape(@access_url)
       URI.parse("#{@store.settings['api_url']}?token=#{t}&url=#{u}")
     end
@@ -56,4 +56,4 @@ module Wovnrb
       end
     end
   end
-end
+end

data/lib/wovnrb/html_replacers/script_replacer.rb

@@ -26,7 +26,7 @@ module Wovnrb
       insert_node['src'] = "//j.#{@store.wovn_host}/1"
       insert_node['async'] = true
       version = defined?(VERSION) ? VERSION : ''
-      insert_node['data-wovnio'] = "key=#{@store.settings['user_token']}&backend=true&currentLang=#{lang.lang_code}&defaultLang=#{@store.settings['default_lang']}&urlPattern=#{@store.settings['url_pattern']}&langCodeAliases=#{JSON.dump(@store.settings['custom_lang_aliases'])}&version=#{version}"
+      insert_node['data-wovnio'] = "key=#{@store.settings['project_token']}&backend=true&currentLang=#{lang.lang_code}&defaultLang=#{@store.settings['default_lang']}&urlPattern=#{@store.settings['url_pattern']}&langCodeAliases=#{JSON.dump(@store.settings['custom_lang_aliases'])}&version=#{version}"
       # do this so that there will be a closing tag (better compatibility with browsers)
       insert_node.content = ' '
       if parent_node.children.size > 0

data/lib/wovnrb/railtie.rb

@@ -11,7 +11,7 @@ module Wovnrb
 
       #if Rails.env.development? && config.respond_to?(:wovnrb)
       #  config.after_initialize do
-      #    config.wovnrb[:user_token] = User.first.short_token
+      #    config.wovnrb[:project_token] = User.first.short_token
       #  end
       #end
     end

data/lib/wovnrb/store.rb

@@ -13,7 +13,7 @@ module Wovnrb
 
     def self.default_settings
       {
-        'user_token' => '',
+        'project_token' => '',
         'log_path' => 'log/wovn_error.log',
         'ignore_paths' => [],
         'ignore_globs' => [],
@@ -55,9 +55,9 @@ module Wovnrb
     def valid_settings?
       valid = true
       errors = [];
-      if !settings.has_key?('user_token') || settings['user_token'].length < 5 || settings['user_token'].length > 6
+      if !settings.has_key?('project_token') || settings['project_token'].length < 5 || settings['project_token'].length > 6
         valid = false
-        errors.push("User token #{settings['user_token']} is not valid.")
+        errors.push("Project token #{settings['project_token']} is not valid.")
       end
       if settings.has_key?('ignore_paths') && !settings['ignore_paths'].kind_of?(Array)
         valid = false
@@ -128,6 +128,11 @@ module Wovnrb
         @settings['supported_langs'] = [@settings['default_lang']]
       end
 
+      if @settings.has_key?('user_token') && @settings['project_token'].empty?
+        @settings['project_token'] = @settings['user_token']
+      end
+      @settings.delete('user_token')
+
       if @settings['url_pattern'] == 'path'
         @settings['url_pattern_reg'] = "/(?<lang>[^/.?]+)"
       elsif @settings['url_pattern'] == 'query'

data/lib/wovnrb/version.rb

@@ -1,3 +1,3 @@
 module Wovnrb
-  VERSION = "0.2.17"
+  VERSION = "0.2.18"
 end

data/test/lib/api_data_test.rb

@@ -25,7 +25,7 @@ module Wovnrb
       stub_request(:get, "https://api.wovn.io/v0/values?token=#{token}&url=#{url}").
         to_return(:body => '{"test_body": "a"}')
       store = Wovnrb::Store.instance
-      store.settings['user_token'] = token
+      store.settings('user_token' => token)
       api_data = Wovnrb::ApiData.new(url, store)
 
       assert_equal({'test_body' => 'a'}, api_data.get_data)
@@ -37,7 +37,7 @@ module Wovnrb
       stub = stub_request(:get, "https://api.wovn.io/v0/values?token=#{token}&url=#{url}").
         to_return(:body => '{"test_body": "a"}')
       store = Wovnrb::Store.instance
-      store.settings['user_token'] = token
+      store.settings('user_token' => token)
       api_data = Wovnrb::ApiData.new(url, store)
 
       assert_equal({'test_body' => 'a'}, api_data.get_data)
@@ -51,7 +51,7 @@ module Wovnrb
       stub_request(:get, "https://api.wovn.io/v0/values?token=#{token}&url=#{url}").
         to_return(:status => [500, "Internal Server Error"])
       store = Wovnrb::Store.instance
-      store.settings['user_token'] = token
+      store.settings('user_token' => token)
       api_data = Wovnrb::ApiData.new(url, store)
       log_mock = Wovnrb::LogMock.mock_log
 

data/test/lib/html_replacers/script_replacer_test.rb

@@ -5,9 +5,11 @@ module Wovnrb
   class ScriptReplacerTest < WovnMiniTest
     def test_replace
       store = Store.instance
-      store.settings['user_token'] = 'test_token'
-      store.settings['default_lang'] = 'en'
-      store.settings['url_pattern'] = 'domain'
+      store.settings({
+        'user_token' => 'test_token',
+        'default_lang' => 'en',
+        'url_pattern' => 'domain'
+      })
 
       replacer = ScriptReplacer.new(store)
       dom = to_head_dom('<script src="test/test.js"></script>')
@@ -25,10 +27,12 @@ module Wovnrb
 
     def test_replace_with_wovn_dev_mode_on
       store = Store.instance
-      store.settings['user_token'] = 'test_token'
-      store.settings['default_lang'] = 'en'
-      store.settings['url_pattern'] = 'domain'
-      store.settings['wovn_dev_mode'] = true
+      store.settings({
+        'user_token' => 'test_token',
+        'default_lang' => 'en',
+        'url_pattern' => 'domain',
+        'wovn_dev_mode' => true
+      })
 
       replacer = ScriptReplacer.new(store)
       dom = to_head_dom('<script src="test/test.js"></script>')
@@ -46,9 +50,11 @@ module Wovnrb
 
     def test_with_embed_wovn
       store = Store.instance
-      store.settings['user_token'] = 'test_token'
-      store.settings['default_lang'] = 'en'
-      store.settings['url_pattern'] = 'domain'
+      store.settings({
+        'user_token' => 'test_token',
+        'default_lang' => 'en',
+        'url_pattern' => 'domain'
+      })
 
       replacer = ScriptReplacer.new(store)
       dom = to_head_dom('<script src="//j.wovn.io/aaaa" data-wovnio="key=test_token" async></script>')
@@ -62,9 +68,11 @@ module Wovnrb
 
     def test_with_multiple_embed_wovn
       store = Store.instance
-      store.settings['user_token'] = 'test_token'
-      store.settings['default_lang'] = 'en'
-      store.settings['url_pattern'] = 'domain'
+      store.settings({
+        'user_token' => 'test_token',
+        'default_lang' => 'en',
+        'url_pattern' => 'domain'
+      })
 
       replacer = ScriptReplacer.new(store)
       dom = to_head_dom('<script src="//j.wovn.io/aaaa" data-wovnio="key=test_token" async></script><script src="//j.wovn.io/bbb" data-wovnio="key=test_token" async></script>')
@@ -78,9 +86,11 @@ module Wovnrb
 
     def test_with_embed_wovn_at_body
       store = Store.instance
-      store.settings['user_token'] = 'test_token'
-      store.settings['default_lang'] = 'en'
-      store.settings['url_pattern'] = 'domain'
+      store.settings({
+        'user_token' => 'test_token',
+        'default_lang' => 'en',
+        'url_pattern' => 'domain'
+      })
 
       replacer = ScriptReplacer.new(store)
       dom = Wovnrb.get_dom('<script src="//j.wovn.io/aaaa" data-wovnio="key=test_token" async></script>')
@@ -94,10 +104,12 @@ module Wovnrb
 
     def test_contains_lang_code_aliases
       store = Store.instance
-      store.settings['user_token'] = 'test_token'
-      store.settings['default_lang'] = 'en'
-      store.settings['url_pattern'] = 'domain'
-      store.settings['custom_lang_aliases'] = {'ja' => 'staging-ja'}
+      store.settings({
+        'user_token' => 'test_token',
+        'default_lang' => 'en',
+        'url_pattern' => 'domain',
+        'custom_lang_aliases' => {'ja' => 'staging-ja'}
+      })
 
       replacer = ScriptReplacer.new(store)
       dom = to_head_dom('')
@@ -108,11 +120,11 @@ module Wovnrb
       check_wovn_script(scripts[0], 'test_token', 'ja', 'en', 'domain', '{"ja":"staging-ja"}')
     end
 
-    def check_wovn_script(node, user_token, current_lang, default_lang, url_pattern, custom_lang_aliases = {})
+    def check_wovn_script(node, project_token, current_lang, default_lang, url_pattern, custom_lang_aliases = {})
       wovn_data = [
         ['src', '//j.wovn.io/1'],
         ['async', 'true'],
-        ['data-wovnio', "key=#{user_token}&backend=true&currentLang=#{current_lang}&defaultLang=#{default_lang}&urlPattern=#{url_pattern}&langCodeAliases=#{custom_lang_aliases}&version=#{Wovnrb::VERSION}"]
+        ['data-wovnio', "key=#{project_token}&backend=true&currentLang=#{current_lang}&defaultLang=#{default_lang}&urlPattern=#{url_pattern}&langCodeAliases=#{custom_lang_aliases}&version=#{Wovnrb::VERSION}"]
       ]
       wovn_data.each do |data|
         assert_equal(data[1], node.get_attribute(data[0]))

data/test/lib/store_test.rb

@@ -14,6 +14,26 @@ module Wovnrb
       assert_equal('/(?<lang>[^/.?]+)', s.settings['url_pattern_reg'])
     end
 
+    def test_settings_user_token_retro_compatibility
+      s = Wovnrb::Store.instance
+      s.settings('user_token' => 'aaaaa')
+      assert_equal('aaaaa', s.settings['project_token'])
+      assert(!s.settings.has_key?('user_token'))
+    end
+
+    def test_settings_project_token_set
+      s = Wovnrb::Store.instance
+      s.settings('project_token' => 'bbbbbb')
+      assert_equal('bbbbbb', s.settings['project_token'])
+    end
+
+    def test_settings_project_token_set_without_retro_compatibility
+      s = Wovnrb::Store.instance
+      s.settings('project_token' => 'bbbbbb', 'user_token' => 'aaaaa')
+      assert_equal('bbbbbb', s.settings['project_token'])
+      assert(!s.settings.has_key?('user_token'))
+    end
+
     def test_settings_url_pattern_path
       s = Wovnrb::Store.instance
       s.settings({'url_pattern' => 'path'})
@@ -41,7 +61,7 @@ module Wovnrb
       valid = store.valid_settings?
 
       assert_equal(false, valid)
-      assert_equal(['User token  is not valid.'], mock.errors)
+      assert_equal(['Project token  is not valid.'], mock.errors)
     end
 
     def test_settings_ignore_paths
@@ -73,7 +93,7 @@ module Wovnrb
       store.settings({'ignore_paths' => 'aaaa'})
 
       assert_equal(false, store.valid_settings?)
-      assert_equal(['User token  is not valid.', 'Ignore Paths aaaa should be Array.'], mock.errors)
+      assert_equal(['Project token  is not valid.', 'Ignore Paths aaaa should be Array.'], mock.errors)
     end
 
     def test_settings_ignore_glob_injection

data/test/lib/wovnrb_test.rb

@@ -25,7 +25,7 @@ class WovnrbTest < Minitest::Test
 
   def test_api_call
     settings = Wovnrb.get_settings
-    token = settings['user_token']
+    token = settings['project_token']
     url = 'wovn.io/dashboard'
     stub = stub_request(:get, "#{settings['api_url']}?token=#{token}&url=#{url}").
       to_return(:body => '{"test_body": "a"}')
@@ -38,7 +38,7 @@ class WovnrbTest < Minitest::Test
 
   def test_api_call_with_cache
     settings = Wovnrb.get_settings
-    token = settings['user_token']
+    token = settings['project_token']
     url = 'wovn.io/dashboard'
     stub = stub_request(:get, "#{settings['api_url']}?token=#{token}&url=#{url}").
       to_return(:body => '{"test_body": "a"}')

data/test/test_helper.rb

@@ -55,7 +55,7 @@ module Wovnrb
 
   def get_settings(options={})
     settings = {}
-    settings['user_token'] = 'OHYx9'
+    settings['project_token'] = 'OHYx9'
     settings['url_pattern'] = 'path'
     settings['url_pattern_reg'] = "/(?<lang>[^/.?]+)"
     settings['query'] = []

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: wovnrb
 version: !ruby/object:Gem::Version
-  version: 0.2.17
+  version: 0.2.18
 platform: ruby
 authors:
 - Jeff Sandford
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-04-19 00:00:00.000000000 Z
+date: 2017-05-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: nokogumbo