bisu 1.8.0 → 1.10.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2db1130663e6afdca1e4d1b22d34c5863db2f120cee81c1c802d69999b1c094b
-  data.tar.gz: 6f0cfd04fd34bb489055eaa0211aa74691f50f32c5fb13d8b7a1559c68a0f25a
+  metadata.gz: fd090e1e936ebc979392e5ba1cb44657bf0c716fdadbc1566f3aafa8252ff3dd
+  data.tar.gz: 89b8897c02e0ef9eba1945c125b43e4dff1fc38db73017d2eeca8b3ebb16674c
 SHA512:
-  metadata.gz: b2d8325f651a93ca9bc2edb7108af31ebc5d1e3d5e954c8a8890a1a7985d2afababe15166ca4d909984a0be453d6f6ad8965eb44b4e1588bd970bc0e3d9669af
-  data.tar.gz: 67699cebd837fc150ae54b433fb3386d79617211ff988b64bca73d681cd8daaeb6202c1e84280d1cfcf39a906c0930272c9f5f237f8d1eb187d856b0494570af
+  metadata.gz: 521148f8d23cb2bc308eab074f90c2b964cee468a3fa65594504153d834b4878fd4b1dc9fcc96c7a04064e909b534bc6cafec49b199164d3b5d8310f215f8e36
+  data.tar.gz: a011abe84e8c72ef2ad755f64595141f03acbd7367b70c321fb867f059f3891debd4878545acfaa8bb7f59b5ea1562aae3683430d7613d05244249af6ce67315

data/.rspec

@@ -1,5 +1,4 @@
 --color
 --format documentation
 --profile
---drb
 --require spec_helper

data/.ruby-version

@@ -1 +1 @@
-2.5.0
+2.7.3

data/CHANGELOG.md

@@ -2,6 +2,32 @@
 All notable changes to this project will be documented in this file.
 `Bisu` adheres to [Semantic Versioning](http://semver.org/).
 
+## [1.10.2](https://github.com/hole19/bisu/releases/tag/v1.10.2)
+Released on 2021/08/30
+
+#### Fixed
+- Fixes bug where we were assuming "key" would always be the translations-key column title
+
+## [1.10.1](https://github.com/hole19/bisu/releases/tag/v1.10.1)
+Released on 2021/08/30
+
+#### Fixed
+- Crash caused by an unexpected redirect
+
+## [1.10.0](https://github.com/hole19/bisu/releases/tag/v1.10.0)
+Released on 2021/08/30
+
+#### Fixed
+- Google Spreadsheet option stopped working due to a change in the API. This version fixes that.
+
+Note: You'll need a new setup. View README.
+
+## [1.9.0](https://github.com/hole19/bisu/releases/tag/v1.9.0)
+Released on 2021/05/21
+
+#### Added
+- Adds a new option to surpress missing params when that's intentional: add "//ignore-params" to the end of the translation key
+
 ## [1.8.0](https://github.com/hole19/bisu/releases/tag/v1.8.0)
 Released on 2020/04/30
 

data/Gemfile

@@ -1,9 +1,8 @@
 source 'https://rubygems.org'
-ruby '2.5.0'
+ruby '2.7.3'
 
 gem 'safe_yaml',  '~> 1.0'
 gem 'colorize',   '~> 0.7'
-gem 'xml-simple', '~> 1.1'
 
 group :test do
   gem 'rake'

data/Gemfile.lock

@@ -1,37 +1,37 @@
 GEM
   remote: https://rubygems.org/
   specs:
-    addressable (2.7.0)
+    addressable (2.8.0)
       public_suffix (>= 2.0.2, < 5.0)
     colorize (0.8.1)
-    crack (0.4.3)
-      safe_yaml (~> 1.0.0)
-    diff-lcs (1.3)
+    crack (0.4.5)
+      rexml
+    diff-lcs (1.4.4)
     hashdiff (1.0.1)
-    public_suffix (4.0.3)
-    rake (13.0.1)
-    rspec (3.9.0)
-      rspec-core (~> 3.9.0)
-      rspec-expectations (~> 3.9.0)
-      rspec-mocks (~> 3.9.0)
-    rspec-core (3.9.1)
-      rspec-support (~> 3.9.1)
-    rspec-expectations (3.9.0)
+    public_suffix (4.0.6)
+    rake (13.0.3)
+    rexml (3.2.5)
+    rspec (3.10.0)
+      rspec-core (~> 3.10.0)
+      rspec-expectations (~> 3.10.0)
+      rspec-mocks (~> 3.10.0)
+    rspec-core (3.10.1)
+      rspec-support (~> 3.10.0)
+    rspec-expectations (3.10.1)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.9.0)
+      rspec-support (~> 3.10.0)
     rspec-its (1.3.0)
       rspec-core (>= 3.0.0)
       rspec-expectations (>= 3.0.0)
-    rspec-mocks (3.9.1)
+    rspec-mocks (3.10.2)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.9.0)
-    rspec-support (3.9.2)
+      rspec-support (~> 3.10.0)
+    rspec-support (3.10.2)
     safe_yaml (1.0.5)
     webmock (1.24.6)
       addressable (>= 2.3.6)
       crack (>= 0.3.2)
       hashdiff
-    xml-simple (1.1.5)
 
 PLATFORMS
   ruby
@@ -43,10 +43,9 @@ DEPENDENCIES
   rspec-its
   safe_yaml (~> 1.0)
   webmock (~> 1.20)
-  xml-simple (~> 1.1)
 
 RUBY VERSION
-   ruby 2.5.0p0
+   ruby 2.7.3p183
 
 BUNDLED WITH
-   1.17.2
+   2.2.26

data/README.md

@@ -24,7 +24,7 @@ Usage
 1. Run: `bisu`
 1. That's it!*
 
-*given that someone already configured Bisu
+\*_given that someone already configured Bisu_
 
 Setup your configuration file
 -----
@@ -66,13 +66,14 @@ Setup your configuration file
 
 1. First ["Publish to the web"](https://www.google.com/search?q=google+sheets+publish+to+web) your Google Sheet
 1. Share only the sheet that contains the translations
+1. Make sure you CSV format
 1. First column should contain the translation keys
 1. First row should contain the locale for each language
 
   ```
   dictionary:
     type: google_sheet
-    sheet_id: <GOOGLE-DRIVE-SHEET-ID>
+    url: <GOOGLE-DRIVE-SHEET-CSV-URL>
     keys_column: <GOOGLE-DRIVE-KEY-COLUMN-TITLE>
   ```
 

data/bisu.gemspec

@@ -18,7 +18,4 @@ Gem::Specification.new do |s|
 
   s.add_runtime_dependency 'safe_yaml',  '~> 1.0'
   s.add_runtime_dependency 'colorize',   '~> 0.7'
-  s.add_runtime_dependency 'xml-simple', '~> 1.1'
-
-  s.add_development_dependency 'webmock', '~> 1.20'
 end

data/generic.translatable.yml

@@ -3,7 +3,7 @@ type: iOS
 
 dictionary:
   type: google_sheet
-  sheet_id: <GOOGLE-DRIVE-SHEET-ID>
+  url: <GOOGLE-DRIVE-SHEET-CSV-URL>
   keys_column: <GOOGLE-DRIVE-KEY-COLUMN-TITLE>
 
 translate:

data/lib/bisu/config.rb

@@ -61,9 +61,9 @@ module Bisu
       type: Hash,
       elements: {
         type: { type: String },
-        sheet_id: { type: String },
-        keys_column: { type: String }
-      }
+        url: { type: String },
+        keys_column: { type: String },
+      },
     }
 
     ONE_SKY_STRUCT = {
@@ -72,15 +72,15 @@ module Bisu
         api_key: { type: String },
         api_secret: { type: String },
         project_id: { type: Integer },
-        file_name: { type: String }
-      }
+        file_name: { type: String },
+      },
     }
 
     URL_STRUCT = {
       type: Hash,
       elements: {
-        url: { type: String }
-      }
+        url: { type: String },
+      },
     }
 
     DICTIONARY_STRUCT = {

data/lib/bisu/localizer.rb

@@ -32,15 +32,15 @@ module Bisu
           unless t.gsub!(l[:match], localized)
             Logger.warn("Could not find translation for #{l[:match]} in #{language}")
           end
+
+          unless @type.eql?(:ror) || l[:ignore_param_warn] == true
+            localized.scan(/%{[^}]+}/) { |match| Logger.error("Could not find translation param for #{match} in #{language}") }
+          end
         else
           Logger.warn("Could not find translation for #{l[:match]} in #{language}")
         end
       end
 
-      unless @type.eql?(:ror)
-        t.scan(/%{[^}]+}/) { |match| Logger.error("Could not find translation param for #{match} in #{language}") }
-      end
-
       t
     end
 
@@ -57,7 +57,7 @@ module Bisu
     private
 
     def to_localize(text)
-      all_matches = text.to_enum(:scan, /\$([^\$\{]+)(?:\{(.+)\})?\$/).map { Regexp.last_match }
+      all_matches = text.to_enum(:scan, /\$([^\$\{\/]+)(?:\{(.+)\})?(\/\/ignore-params)?\$/).map { Regexp.last_match }
       all_matches.map do |match|
         params = if match[2]
           params = match[2].split(",").map(&:strip).map do |param|
@@ -67,9 +67,12 @@ module Bisu
           Hash[params]
         end
 
-        { match:  match[0],
+        {
+          match:  match[0],
           key:    match[1],
-          params: params || {} }
+          params: params || {},
+          ignore_param_warn: text.include?("//ignore-params")
+        }
       end
     end
 

data/lib/bisu/source/google_sheet.rb

@@ -1,64 +1,65 @@
 require "net/https"
-require "xmlsimple"
+require "csv"
 
 module Bisu
   module Source
     class GoogleSheet
-      def initialize(sheet_id, keys_column)
-        @sheet_id = sheet_id
-        @key_column = keys_column
+      def initialize(url, keys_column)
+        @url = url
+        @keys_column = keys_column
       end
 
       def to_i18
-        raw = raw_data(@sheet_id)
+        Logger.info("Downloading Google Sheet from #{@url}...")
 
-        Logger.info("Downloading dictionary from Google Sheet...")
+        csv = get_csv(@url)
 
-        non_language_columns = ["id", "updated", "category", "title", "content", "link", @key_column]
+        hash = {}
 
-        kb = {}
-        raw["entry"].each do |entry|
-          unless (key = entry[@key_column]) && key = key.first
-            raise "Bisu::Source::GoogleSheet: Cannot find key column '#{@key_column}'"
-          end
+        languages = csv.headers[1..]
+        languages.each { |lang| hash[lang] = {} }
 
-          entry.select { |c| !non_language_columns.include?(c) }.each do |lang, texts|
-            kb[lang] ||= {}
-            kb[lang][key] = texts.first unless texts.first == {}
+        csv.each do |row|
+          languages.each do |lang|
+            hash[lang][row[@keys_column]] = row[lang] unless row[lang].nil?
           end
         end
 
         Logger.info("Google Sheet parsed successfully!")
-        Logger.info("Found #{kb.count} languages.")
+        Logger.info("Found #{languages.count} languages.")
 
-        kb
+        hash
       end
 
       private
 
-      def xml_data(uri, headers=nil)
-        uri = URI.parse(uri)
+      def get_csv(url)
+        data = get(url)
+
+        begin
+          CSV.parse(data, headers: true)
+        rescue StandardError => e
+          raise "Bisu::Source::GoogleSheet: Cannot parse. Expected CSV at #{url}: #{e}"
+        end
+      end
+
+      def get(url, max_redirects = 1)
+        raise "Bisu::Source::GoogleSheet: Too may redirects" if max_redirects == -1
+
+        uri = URI(url)
+
         http = Net::HTTP.new(uri.host, uri.port)
         http.use_ssl = true
         http.verify_mode = OpenSSL::SSL::VERIFY_NONE
-        data = http.get(uri.path, headers)
 
-        unless data.code.to_i == 200
-          raise "Bisu::Source::GoogleSheet: Cannot access sheet at #{uri} - HTTP #{data.code}"
-        end
+        request = Net::HTTP::Get.new(uri.request_uri)
+        response = http.request(request)
 
-        begin
-          XmlSimple.xml_in(data.body, 'KeyAttr' => 'name')
-        rescue
-          raise "Bisu::Source::GoogleSheet: Cannot parse. Expected XML at #{uri}"
-        end
-      end
+        return get(response['location'], max_redirects - 1) if response.is_a? Net::HTTPRedirection
+
+        raise "Bisu::Source::GoogleSheet: Http Error #{response.body}" if response.code.to_i >= 400
 
-      def raw_data(sheet_id)
-        Logger.info("Downloading Google Sheet...")
-        sheet = xml_data("https://spreadsheets.google.com/feeds/worksheets/#{sheet_id}/public/full")
-        url   = sheet["entry"][0]["link"][0]["href"]
-        xml_data(url)
+        response.body
       end
     end
   end

data/lib/bisu/version.rb

@@ -1,4 +1,4 @@
 module Bisu
-  VERSION = '1.8.0'
-  VERSION_UPDATED_AT = '2020-04-30'
+  VERSION = '1.10.2'
+  VERSION_UPDATED_AT = '2021-08-30'
 end

data/lib/bisu.rb

@@ -59,7 +59,7 @@ module Bisu
     source =
       case config[:type]
       when "google_sheet"
-        Bisu::Source::GoogleSheet.new(config[:sheet_id], config[:keys_column])
+        Bisu::Source::GoogleSheet.new(config[:url], config[:keys_column])
       when "one_sky"
         Bisu::Source::OneSky.new(config[:api_key], config[:api_secret], config[:project_id], config[:file_name])
       when "url"
@@ -70,7 +70,7 @@ module Bisu
 
     save_to_path = options[:dictionary_save_path]
     if save_to_path && file = open_file(save_to_path, "w", false)
-      file.write(source.to_json)
+      file.write(deep_force_encoding(source).to_json)
       file.flush
       file.close
     end
@@ -149,4 +149,15 @@ module Bisu
 
     true
   end
+
+  def deep_force_encoding(hash)
+    hash.each do |_, value|
+      case value
+      when String
+        value.force_encoding(Encoding::UTF_8)
+      when Hash
+        deep_force_encoding(value)
+      end
+    end
+  end
 end

data/spec/fixtures/sample.translatable.yml

@@ -2,8 +2,8 @@ type: iOS
 
 dictionary:
   type: google_sheet
-  sheet_id: sheet-id
-  keys_column: keys-column
+  url: sheet-public-url
+  keys_column: "key name"
 
 translate:
   - in: hole19/Localizable.strings.translatable

data/spec/fixtures/sample_google_response.csv

@@ -0,0 +1,4 @@
+key name,en,ja,fr,de,ko
+kConnectFacebook,Connect with Facebook,フェイスブックへ接続,Connexion par Facebook,Mit Facebook verbinden,페이스북으로 접속
+kNoNoNoMr,"No, no, no. Mr %{name} not here",,,,
+kTwitterServer,,,,,트위터 서버연결 실패. \n잠시 후 재시도.

data/spec/lib/bisu/config_spec.rb

@@ -4,24 +4,24 @@ describe Bisu::Config do
   let(:hash) { {
     type: "BisuOS",
     dictionary: {
-      type:         "google_sheet",
-      sheet_id:     "abc1234567890",
-      keys_column:  "key_name"
+      type: "google_sheet",
+      url: "https://abc1234567890",
+      keys_column: "key name",
     },
     translate: [
-      { in:        "path/to/file/to/1.ext.translatable",
-        out:       "path/to/final-%{locale}/1.ext",
+      { in: "path/to/file/to/1.ext.translatable",
+        out: "path/to/final-%{locale}/1.ext",
         out_en_us: "path/to/default/1.ext"
       },
-      { in:        "path/to/file/to/2.ext.translatable",
-        out:       "path/to/final-%{locale}/2.ext",
+      { in: "path/to/file/to/2.ext.translatable",
+        out: "path/to/final-%{locale}/2.ext",
         out_en_us: "path/to/default/2.ext"
       },
     ],
     languages: [
-      { locale: "en-US",      language: "english"    },
-      { locale: "pt",         language: "portuguese" },
-      { locale: "pt-PT",      language: "portuguese" },
+      { locale: "en-US", language: "english"    },
+      { locale: "pt", language: "portuguese" },
+      { locale: "pt-PT", language: "portuguese" },
       { locale: "pt-Batatas", language: "portuguese-bt", fallback_language: "portuguese" }
     ]
   } }
@@ -37,16 +37,16 @@ describe Bisu::Config do
   describe "#dictionary" do
     subject(:dictionary) { config.dictionary }
 
-    it { should eq({ type: "google_sheet", sheet_id: "abc1234567890", keys_column: "key_name" }) }
+    it { should eq({ type: "google_sheet", url: "https://abc1234567890", keys_column: "key name" }) }
 
     context "when given a OneSky type dictionary" do
       before do
         hash[:dictionary] = {
-          type:       "one_sky",
-          api_key:    "as387oavh48",
+          type: "one_sky",
+          api_key: "as387oavh48",
           api_secret: "bp0s5avo8a59",
           project_id: 328742,
-          file_name:  "file.json"
+          file_name: "file.json"
         }
       end
 
@@ -57,7 +57,7 @@ describe Bisu::Config do
       before do
         hash[:dictionary] = {
           type: "url",
-          url:  "a_url"
+          url: "a_url"
         }
       end
 

data/spec/lib/bisu/localizer_spec.rb

@@ -1,4 +1,9 @@
 describe Bisu::Localizer do
+  def translates(text, fallbacks: [], to:, lang: nil)
+    translation = localizer.localize(text, lang || language, locale, fallbacks)
+    expect(translation).to eq to
+  end
+
   let(:language) { "Portuguese" }
   let(:locale)   { "pt-PT" }
 
@@ -33,11 +38,6 @@ describe Bisu::Localizer do
   shared_examples_for "a localizer" do
     it { expect { localizer }.not_to raise_error }
 
-    def translates(text, fallbacks: [], to:, lang: nil)
-      translation = localizer.localize(text, lang || language, locale, fallbacks)
-      expect(translation).to eq to
-    end
-
     it { translates("a line with no key", to: "a line with no key") }
 
     it { translates("this special key: $specialKComment1$", to: "this special key: This file was automatically generated based on a translation template.") }
@@ -87,7 +87,7 @@ describe Bisu::Localizer do
        .and not_change { Bisu::Logger.summary[:error] }
     end
 
-    it "throws an error when missing key parameters (expect on ruby on rails)" do
+    it "throws a error when missing key parameters (expect on ruby on rails)" do
       if type == :ror
         expect {
           localizer.localize("$k1ParameterKey$", language, locale)
@@ -114,6 +114,17 @@ describe Bisu::Localizer do
       }.to not_change { Bisu::Logger.summary[:warn] }
        .and change { Bisu::Logger.summary[:error] }.by(1)
     end
+
+    # non localizable text with params
+
+    it "does not try to translate params outside $$" do
+      translates("%{some_text}", to: "%{some_text}")
+
+      expect {
+        localizer.localize("%{some_text}", language, locale)
+      }.to not_change { Bisu::Logger.summary[:warn] }
+       .and not_change { Bisu::Logger.summary[:error] }
+    end
   end
 
   let(:type_dependent_defaults) { {
@@ -134,6 +145,33 @@ describe Bisu::Localizer do
     ) }
 
     it_behaves_like "a localizer"
+
+    context "when a parameter replacemnt was not defined" do
+      let(:line) { "1 parameter: $k1ParameterKey$" }
+      let(:expected) { "1 parameter: Não sabes nada %{name}" }
+
+      it { translates(line, to: expected) }
+
+      it "returns an error" do
+        expect {
+          localizer.localize(line, language, locale)
+        }.to not_change { Bisu::Logger.summary[:warn] }
+         .and change { Bisu::Logger.summary[:error] }.by(1)
+      end
+
+      context "when passing //ignore-params" do
+        let(:line) { "1 parameter: $k1ParameterKey//ignore-params$" }
+
+        it { translates(line, to: expected) }
+
+        it "does not return an error" do
+          expect {
+            localizer.localize(line, language, locale)
+          }.to not_change { Bisu::Logger.summary[:warn] }
+           .and not_change { Bisu::Logger.summary[:error] }
+        end
+      end
+    end
   end
 
   describe "of type Android" do

data/spec/lib/bisu/source/google_sheet_spec.rb

@@ -1,68 +1,43 @@
 describe Bisu::Source::GoogleSheet do
-  subject(:to_i18) { Bisu::Source::GoogleSheet.new(sheet_id, key_column).to_i18 }
+  subject(:to_i18) { Bisu::Source::GoogleSheet.new(url, "key name").to_i18 }
 
-  let(:sheet_id) { "abc1234567890" }
-  let(:url_info)  { "https://spreadsheets.google.com/feeds/worksheets/#{sheet_id}/public/full" }
-  let(:url_sheet) { "https://spreadsheets.google.com/feeds/list/#{sheet_id}/od6/public/full" }
+  let(:url) { "https://docs.google.com/spreadsheets/d/e/2PACX-1vTm6yu_zfbxKizC-PvUE1HVFCsplmiyz0s0qLSIGeokA7KtS3BgtqaA79CsfYfPsXH6xzUaP8HDTcj8/pub?gid=0&single=true&output=csv" }
+  let(:response) { File.read("spec/fixtures/sample_google_response.csv") }
 
-  let(:key_column) { "key_column" }
-
-  context "when given a valid sheet" do
-    let(:file_info)  { File.read("spec/fixtures/sample_kb_public_info.html") }
-    let(:file_sheet) { File.read("spec/fixtures/sample_kb_public_sheet.html") }
-
-    before do
-      stub_request(:get, url_info).to_return(:status => 200, :body => file_info, :headers => {})
-      stub_request(:get, url_sheet).to_return(:status => 200, :body => file_sheet, :headers => {})
-    end
-
-    it do
-      expect { to_i18 }.not_to raise_error
-    end
-
-    it "returns an hash in i18 format" do
-      expect(to_i18).to eq({
-        "english"    => { "kConnectFacebook" => "Connect with Facebook",  "kConnectEmail" => "Connect with Email" },
-        "german"     => { "kConnectFacebook" => "Mit Facebook verbinden", "kConnectEmail" => "Mit E-Mail verbinden" },
-        "portuguese" => { "kConnectFacebook" => "Registar com Facebook",  "kConnectEmail" => "Registar com Email" },
-        "spanish"    => { "kConnectFacebook" => "Conéctate con Facebook", "kConnectEmail" => "Conéctate con Email" },
-        "french"     => { "kConnectFacebook" => "Connecter Facebook" },
-        "dutch"      => { "kConnectFacebook" => "Facebook Verbinden",     "kConnectEmail" => "Email Verbinden" },
-        "korean"     => { "kConnectFacebook" => "페이스북으로 접속",           "kConnectEmail" => "이메일로 접속" },
-        "japanese"   => { "kConnectFacebook" => "フェイスブックへ接続",      "kConnectEmail" => "電子メールアカウントに接続" }
-     })
-    end
+  def stub_url(status:, response:)
+    stub_request(:get, url).
+      to_return(:status => status, :body => response, :headers => {})
+  end
 
-    context "but the key column is not present in the first sheet" do
-      let(:key_column) { "expecting_another_key_column" }
+  before { stub_url(status: 200, response: response) }
 
-      it do
-        expect { to_i18 }.to raise_error /Cannot find key column/
-      end
-    end
+  it do
+    expect { to_i18 }.not_to raise_error
   end
 
-  context "when given an inexistent sheet" do
-    before { stub_request(:get, url_info).to_return(:status => 400, :body => "Not Found", :headers => {}) }
-
-    it do
-      expect { to_i18 }.to raise_error /Cannot access sheet/
-    end
+  it "returns an hash in i18 format" do
+    expect(to_i18).to eq({
+      "en" => { "kConnectFacebook" => "Connect with Facebook", "kNoNoNoMr" => "No, no, no. Mr %{name} not here" },
+      "ja" => { "kConnectFacebook" => "フェイスブックへ接続" },
+      "fr" => { "kConnectFacebook" => "Connexion par Facebook" },
+      "de" => { "kConnectFacebook" => "Mit Facebook verbinden" },
+      "ko" => { "kConnectFacebook" => "페이스북으로 접속", "kTwitterServer" => "트위터 서버연결 실패. \\n잠시 후 재시도." }
+   })
   end
 
-  context "when given a private sheet" do
-    before { stub_request(:get, url_info).to_return(:status => 302, :body => "<HTML></HTML>", :headers => {}) }
+  context "when given an inexistent sheet" do
+    before { stub_url(status: 400, response: "Not Found") }
 
     it do
-      expect { to_i18 }.to raise_error /Cannot access sheet/
+      expect { to_i18 }.to raise_error /Http Error/
     end
   end
 
-  context "when url content is not XML" do
-    before { stub_request(:get, url_info).to_return(:status => 200, :body => "This is not XML; { this: \"is json\" }", :headers => {}) }
+  context "when url content is not CSV" do
+    before { stub_url(status: 200, response: "{\"asdsa\": \"This is a json\"}") }
 
     it do
-      expect { to_i18 }.to raise_error /Cannot parse. Expected XML/
+      expect { to_i18 }.to raise_error /Cannot parse. Expected CSV/
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bisu
 version: !ruby/object:Gem::Version
-  version: 1.8.0
+  version: 1.10.2
 platform: ruby
 authors:
 - joaoffcosta
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-04-30 00:00:00.000000000 Z
+date: 2021-08-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: safe_yaml
@@ -38,34 +38,6 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.7'
-- !ruby/object:Gem::Dependency
-  name: xml-simple
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '1.1'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '1.1'
-- !ruby/object:Gem::Dependency
-  name: webmock
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '1.20'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '1.20'
 description: Bisu manages your app iOS and Android localization files for you. No
   more copy+paste induced errors!
 email: joaostacosta@gmail.com
@@ -97,9 +69,8 @@ files:
 - lib/bisu/source/url.rb
 - lib/bisu/version.rb
 - spec/fixtures/sample.translatable.yml
+- spec/fixtures/sample_google_response.csv
 - spec/fixtures/sample_json_response.json
-- spec/fixtures/sample_kb_public_info.html
-- spec/fixtures/sample_kb_public_sheet.html
 - spec/fixtures/sample_one_sky_response.json
 - spec/fixtures/sample_one_sky_response_with_bug.json
 - spec/lib/bisu/config_spec.rb
@@ -131,7 +102,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.2
+rubygems_version: 3.1.6
 signing_key: 
 specification_version: 4
 summary: A localization automation service

data/spec/fixtures/sample_kb_public_info.html

@@ -1 +0,0 @@
-<?xml version='1.0' encoding='UTF-8'?><feed xmlns='http://www.w3.org/2005/Atom' xmlns:openSearch='http://a9.com/-/spec/opensearchrss/1.0/' xmlns:gs='http://schemas.google.com/spreadsheets/2006'><id>https://spreadsheets.google.com/feeds/worksheets/abc1234567890/public/full</id><updated>2015-02-09T18:59:53.874Z</updated><category scheme='http://schemas.google.com/spreadsheets/2006' term='http://schemas.google.com/spreadsheets/2006#worksheet'/><title type='text'>H19: Translation Knowledge Base</title><link rel='alternate' type='application/atom+xml' href='https://docs.google.com/spreadsheets/d/abc1234567890/pubhtml'/><link rel='http://schemas.google.com/g/2005#feed' type='application/atom+xml' href='https://spreadsheets.google.com/feeds/worksheets/abc1234567890/public/full'/><link rel='http://schemas.google.com/g/2005#post' type='application/atom+xml' href='https://spreadsheets.google.com/feeds/worksheets/abc1234567890/public/full'/><link rel='self' type='application/atom+xml' href='https://spreadsheets.google.com/feeds/worksheets/abc1234567890/public/full'/><author><name>anthony.douglas</name><email>anthony.douglas@hole19golf.com</email></author><openSearch:totalResults>1</openSearch:totalResults><openSearch:startIndex>1</openSearch:startIndex><entry><id>https://spreadsheets.google.com/feeds/worksheets/abc1234567890/public/full/od6</id><updated>2015-02-09T18:59:53.874Z</updated><category scheme='http://schemas.google.com/spreadsheets/2006' term='http://schemas.google.com/spreadsheets/2006#worksheet'/><title type='text'>Translations</title><content type='text'>Translations</content><link rel='http://schemas.google.com/spreadsheets/2006#listfeed' type='application/atom+xml' href='https://spreadsheets.google.com/feeds/list/abc1234567890/od6/public/full'/><link rel='http://schemas.google.com/spreadsheets/2006#cellsfeed' type='application/atom+xml' href='https://spreadsheets.google.com/feeds/cells/abc1234567890/od6/public/full'/><link rel='http://schemas.google.com/visualization/2008#visualizationApi' type='application/atom+xml' href='https://docs.google.com/spreadsheets/d/abc1234567890/gviz/tq?gid=0&amp;pub=1'/><link rel='http://schemas.google.com/spreadsheets/2006#exportcsv' type='text/csv' href='https://docs.google.com/spreadsheets/d/abc1234567890/export?gid=0&amp;format=csv'/><link rel='self' type='application/atom+xml' href='https://spreadsheets.google.com/feeds/worksheets/abc1234567890/public/full/od6'/><gs:colCount>25</gs:colCount><gs:rowCount>665</gs:rowCount></entry></feed>

data/spec/fixtures/sample_kb_public_sheet.html

@@ -1 +0,0 @@
-<?xml version='1.0' encoding='UTF-8'?><feed xmlns='http://www.w3.org/2005/Atom' xmlns:openSearch='http://a9.com/-/spec/opensearchrss/1.0/' xmlns:gsx='http://schemas.google.com/spreadsheets/2006/extended'><id>https://spreadsheets.google.com/feeds/list/abc1234567890/od6/public/full</id><updated>2015-02-09T18:59:53.874Z</updated><category scheme='http://schemas.google.com/spreadsheets/2006' term='http://schemas.google.com/spreadsheets/2006#list'/><title type='text'>Translations</title><link rel='alternate' type='application/atom+xml' href='https://docs.google.com/spreadsheets/d/abc1234567890/pubhtml'/><link rel='http://schemas.google.com/g/2005#feed' type='application/atom+xml' href='https://spreadsheets.google.com/feeds/list/abc1234567890/od6/public/full'/><link rel='http://schemas.google.com/g/2005#post' type='application/atom+xml' href='https://spreadsheets.google.com/feeds/list/abc1234567890/od6/public/full'/><link rel='self' type='application/atom+xml' href='https://spreadsheets.google.com/feeds/list/abc1234567890/od6/public/full'/><author><name>anthony.douglas</name><email>anthony.douglas@hole19golf.com</email></author><openSearch:totalResults>664</openSearch:totalResults><openSearch:startIndex>1</openSearch:startIndex><entry><id>https://spreadsheets.google.com/feeds/list/abc1234567890/od6/public/full/cokwr</id><updated>2015-02-09T18:59:53.874Z</updated><category scheme='http://schemas.google.com/spreadsheets/2006' term='http://schemas.google.com/spreadsheets/2006#list'/><title type='text'>kConnectFacebook</title><content type='text'>english: Connect with Facebook, german: Mit Facebook verbinden, portuguese: Registar com Facebook, spanish: Conéctate con Facebook, french: Connecter Facebook, dutch: Facebook Verbinden, korean: 페이스북으로 접속, japanese: フェイスブックへ接続, key_column_2: kConnectFacebook</content><link rel='self' type='application/atom+xml' href='https://spreadsheets.google.com/feeds/list/abc1234567890/od6/public/full/cokwr'/><gsx:key_column>kConnectFacebook</gsx:key_column><gsx:english>Connect with Facebook</gsx:english><gsx:german>Mit Facebook verbinden</gsx:german><gsx:portuguese>Registar com Facebook</gsx:portuguese><gsx:spanish>Conéctate con Facebook</gsx:spanish><gsx:french>Connecter Facebook</gsx:french><gsx:dutch>Facebook Verbinden</gsx:dutch><gsx:korean>페이스북으로 접속</gsx:korean><gsx:japanese>フェイスブックへ接続</gsx:japanese></entry><entry><id>https://spreadsheets.google.com/feeds/list/abc1234567890/od6/public/full/cpzh4</id><updated>2015-02-09T18:59:53.874Z</updated><category scheme='http://schemas.google.com/spreadsheets/2006' term='http://schemas.google.com/spreadsheets/2006#list'/><title type='text'>kConnectEmail</title><content type='text'>english: Connect with Email, german: Mit E-Mail verbinden, portuguese: Registar com Email, spanish: Conéctate con Email, dutch: Email Verbinden, korean: 이메일로 접속, japanese: 電子メールアカウントに接続, key_column_2: kConnectEmail</content><link rel='self' type='application/atom+xml' href='https://spreadsheets.google.com/feeds/list/abc1234567890/od6/public/full/cpzh4'/><gsx:key_column>kConnectEmail</gsx:key_column><gsx:english>Connect with Email</gsx:english><gsx:german>Mit E-Mail verbinden</gsx:german><gsx:portuguese>Registar com Email</gsx:portuguese><gsx:spanish>Conéctate con Email</gsx:spanish><gsx:french></gsx:french><gsx:dutch>Email Verbinden</gsx:dutch><gsx:korean>이메일로 접속</gsx:korean><gsx:japanese>電子メールアカウントに接続</gsx:japanese></entry></feed>