i18n_flow 0.1.0

data/spec/lib/i18n_flow/util_spec.rb

@@ -0,0 +1,194 @@
+require 'i18n_flow/util'
+
+describe I18nFlow::Util do
+  describe '.extract_args' do
+    it 'should extract translation arguments' do
+      {
+        ''                          => %w[],
+        'foo'                       => %w[],
+        '%{arg_1}'                  => %w[arg_1],
+        'foo %{arg_1}'              => %w[arg_1],
+        'foo %{arg_1} bar %{arg_2}' => %w[arg_1 arg_2],
+        'foo %{arg_2} bar %{arg_1}' => %w[arg_1 arg_2],
+        '%{arg_1}%{arg_2}'          => %w[arg_1 arg_2],
+        '%{arg_1}%%{arg_2}'         => %w[arg_1],
+        '%%{arg_1} bar %%%{arg_2}'  => %w[arg_2],
+      }.each do |input, output|
+        expect(I18nFlow::Util.extract_args(input)).to eq(output)
+      end
+    end
+  end
+
+  describe '.filepath_to_scope' do
+    it 'should parse file path into locale and scopes' do
+      {
+        'en.yml'                 => %w[en],
+        '.en.yml'                => %w[en],
+        'foo.en.yml'             => %w[en foo],
+        'foo/bar/en.yml'         => %w[en foo bar],
+        'foo/bar/baz.en.yml'     => %w[en foo bar baz],
+        'foo/bar/baz.bax.en.yml' => %w[en foo bar baz bax],
+      }.each do |input, output|
+        expect(I18nFlow::Util.filepath_to_scope(input)).to eq(output)
+      end
+    end
+  end
+
+  describe '.scope_to_filepath' do
+    it 'should build file path from scopes' do
+      {
+        [nil, 'en']            => 'en.yml',
+        %w[en]                 => 'en.yml',
+        %w[en foo]             => 'foo.en.yml',
+        %w[en foo bar]         => 'foo/bar.en.yml',
+        %w[en foo bar baz]     => 'foo/bar/baz.en.yml',
+        %w[en foo bar baz bax] => 'foo/bar/baz/bax.en.yml',
+      }.each do |input, output|
+        expect(I18nFlow::Util.scope_to_filepath(input)).to eq(output)
+      end
+    end
+  end
+
+  describe '.find_file_upward' do
+    let(:file)      { FakeFS::FakeFile.new }
+    let(:file_name) { 'file' }
+
+    it 'should return a file path if a file is exist in the current directory' do
+      pwd       = '/a/b/c/d'
+      file_path = "/a/b/c/d/#{file_name}"
+
+      FakeFS::FileSystem.add(pwd)
+      FakeFS::FileSystem.add(file_path, file)
+
+      Dir.chdir(pwd) do
+        expect(I18nFlow::Util.find_file_upward(file_name)).to eq(file_path)
+      end
+    end
+
+    it 'should return a file path if a file is exist at the parent directory' do
+      pwd       = '/a/b/c/d'
+      file_path = "/a/b/c/#{file_name}"
+
+      FakeFS::FileSystem.add(pwd)
+      FakeFS::FileSystem.add(file_path, file)
+
+      Dir.chdir(pwd) do
+        expect(I18nFlow::Util.find_file_upward(file_name)).to eq(file_path)
+      end
+    end
+
+    it 'should return a file path if a file is exist at somewhere of parent directories' do
+      pwd       = '/a/b/c/d'
+      file_path = "/a/#{file_name}"
+
+      FakeFS::FileSystem.add(pwd)
+      FakeFS::FileSystem.add(file_path, file)
+
+      Dir.chdir(pwd) do
+        expect(I18nFlow::Util.find_file_upward(file_name)).to eq(file_path)
+      end
+    end
+
+    it 'should return nil if a file is not exist in any parent directories' do
+      pwd       = '/a/b/c/d'
+      file_path = "/abc/#{file_name}"
+
+      FakeFS::FileSystem.add(pwd)
+      FakeFS::FileSystem.add(file_path, file)
+
+      Dir.chdir(pwd) do
+        expect(I18nFlow::Util.find_file_upward(file_name)).to be_nil
+      end
+    end
+
+    context 'Multiple file names' do
+      let(:file_2)      { FakeFS::FakeFile.new }
+      let(:file_name_2) { 'file' }
+
+      it 'should return a file path if one of files is exist at somewhere of parent directories' do
+        pwd       = '/a/b/c/d'
+        file_path = "/a/#{file_name}"
+
+        FakeFS::FileSystem.add(pwd)
+        FakeFS::FileSystem.add(file_path, file)
+
+        Dir.chdir(pwd) do
+          expect(I18nFlow::Util.find_file_upward(file_name, file_name_2)).to eq(file_path)
+        end
+      end
+
+      it 'should return the first match if one of files is exist at somewhere of parent directories' do
+        pwd         = '/a/b/c/d'
+        file_path   = "/a/#{file_name}"
+        file_path_2 = "/a/#{file_name_2}"
+
+        FakeFS::FileSystem.add(pwd)
+        FakeFS::FileSystem.add(file_path, file)
+        FakeFS::FileSystem.add(file_path_2, file)
+
+        Dir.chdir(pwd) do
+          expect(I18nFlow::Util.find_file_upward(file_name, file_name_2)).to eq(file_path)
+          expect(I18nFlow::Util.find_file_upward(file_name_2, file_name)).to eq(file_path_2)
+        end
+      end
+
+      it 'should return nil if non of files is not exist in any parent directories' do
+        pwd         = '/a/b/c/d'
+        file_path   = "/abc/#{file_name}"
+        file_path_2 = "/cba/#{file_name_2}"
+
+        FakeFS::FileSystem.add(pwd)
+        FakeFS::FileSystem.add(file_path, file)
+        FakeFS::FileSystem.add(file_path_2, file)
+
+        Dir.chdir(pwd) do
+          expect(I18nFlow::Util.find_file_upward(file_name, file_name_2)).to be_nil
+        end
+      end
+    end
+  end
+
+  describe '.parse_options' do
+    it 'should parse short options' do
+      args = ['-a', '-b']
+      options = I18nFlow::Util.parse_options(args)
+
+      expect(args).to be_empty
+      expect(options).to be_a(Hash)
+      expect(options['a']).to be(true)
+      expect(options['b']).to be(true)
+    end
+
+    it 'should parse long options' do
+      args = ['--long-a', '--long-b']
+      options = I18nFlow::Util.parse_options(args)
+
+      expect(args).to be_empty
+      expect(options).to be_a(Hash)
+      expect(options['long-a']).to be(true)
+      expect(options['long-b']).to be(true)
+    end
+
+    it 'should parse long options with value' do
+      args = ['--long-a=value-of-a', '--long-b=value-of-b']
+      options = I18nFlow::Util.parse_options(args)
+
+      expect(args).to be_empty
+      expect(options).to be_a(Hash)
+      expect(options['long-a']).to eq('value-of-a')
+      expect(options['long-b']).to eq('value-of-b')
+    end
+
+    it 'should not parse options after once non-option args appears' do
+      args = ['-a', '--long-a', 'non-option', '--long-b=value-of-b']
+      options = I18nFlow::Util.parse_options(args)
+
+      expect(args).to eq(['non-option', '--long-b=value-of-b'])
+      expect(options).to be_a(Hash)
+      expect(options['a']).to be(true)
+      expect(options['long-a']).to be(true)
+      expect(options['long-b']).to be_nil
+    end
+  end
+
+end

data/spec/lib/i18n_flow/validator/file_scope_spec.rb

@@ -0,0 +1,74 @@
+require 'i18n_flow/validator/file_scope'
+require 'i18n_flow/validator/errors'
+
+describe I18nFlow::Validator::FileScope do
+  let(:filepath) { 'foo/bar.en.yml' }
+  let(:validator) { I18nFlow::Validator::FileScope.new(nil, filepath: filepath) }
+
+  describe '#validate' do
+    it 'should pass if the filepath and the scope are perfectly matched' do
+      ast = parse_yaml(<<-YAML)
+      en:
+        foo:
+          bar:
+            key_1: text_1
+      YAML
+
+      allow(validator).to receive(:ast).and_return(ast)
+      validator.validate!
+
+      expect(validator.errors).to eq([])
+    end
+
+    it 'should fail if the scope is missing' do
+      ast = parse_yaml(<<-YAML)
+      en:
+        foo:
+          key_1: text_1
+      YAML
+
+      allow(validator).to receive(:ast).and_return(ast)
+      validator.validate!
+
+      expect(validator.errors).to eq([
+        I18nFlow::Validator::MissingKeyError.new('en.foo.bar'),
+      ])
+    end
+
+    it 'should fail if its structure is invalid' do
+      ast = parse_yaml(<<-YAML)
+      en:
+        foo:
+          bar: text_1
+      YAML
+
+      allow(validator).to receive(:ast).and_return(ast)
+      validator.validate!
+
+      expect(validator.errors).to eq([
+        I18nFlow::Validator::InvalidTypeError.new('en.foo.bar'),
+      ])
+    end
+
+    it 'should fail if it contains extra keys' do
+      ast = parse_yaml(<<-YAML)
+      en:
+        foo:
+          bar:
+            key_1: text_1
+          baz:
+            key_2: text_2
+          bax:
+            key_2: text_2
+      YAML
+
+      allow(validator).to receive(:ast).and_return(ast)
+      validator.validate!
+
+      expect(validator.errors).to eq([
+        I18nFlow::Validator::ExtraKeyError.new('en.foo.baz'),
+        I18nFlow::Validator::ExtraKeyError.new('en.foo.bax'),
+      ])
+    end
+  end
+end

data/spec/lib/i18n_flow/validator/multiplexer_spec.rb

@@ -0,0 +1,68 @@
+describe I18nFlow::Validator::Multiplexer do
+  include_examples :create_repository
+
+  let(:validator) do
+    I18nFlow::Validator::Multiplexer.new(
+      repository: repository,
+      valid_locales: %w[en ja fr],
+      locale_pairs: [
+        %w[en ja],
+        %w[ja fr],
+      ],
+    )
+  end
+
+  describe '#validate' do
+    it 'should pass' do
+      validator.validate!
+      expect(validator.errors).to eq({})
+    end
+
+    context 'violate single validator rules' do
+      let(:models_user_ja_yml) do
+        <<-YAML
+        ja:
+          modulo:
+            user:
+              key_1: text_1
+        YAML
+      end
+
+      it 'should fail' do
+        validator.validate!
+        expect(validator.errors).to eq({
+          'models/user.ja.yml' => {
+            'ja.models' => I18nFlow::Validator::MissingKeyError.new('ja.models'),
+            'ja.modulo' => I18nFlow::Validator::ExtraKeyError.new('ja.modulo'),
+          },
+        })
+      end
+    end
+
+    context 'violate symmetry validator rules' do
+      let(:views_profiles_show_ja_yml) do
+        <<-YAML
+        ja:
+          views:
+            profiles:
+              show:
+                key_2: text_2
+        YAML
+      end
+
+      it 'should fail' do
+        validator.validate!
+        expect(validator.errors).to eq({
+          'views/profiles/show.ja.yml' => {
+            'ja.views.profiles.show.key_1' => I18nFlow::Validator::MissingKeyError.new('ja.views.profiles.show.key_1'),
+            'ja.views.profiles.show.key_2' => I18nFlow::Validator::ExtraKeyError.new('ja.views.profiles.show.key_2'),
+          },
+          'views/profiles/show.fr.yml' => {
+            "fr.views.profiles.show.key_1"=> I18nFlow::Validator::ExtraKeyError.new('fr.views.profiles.show.key_1'),
+            'fr.views.profiles.show.key_2'=> I18nFlow::Validator::MissingKeyError.new('fr.views.profiles.show.key_2'),
+          },
+        })
+      end
+    end
+  end
+end