translator_with_localeapp 1.0.0 → 1.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: ec29e837e47f36f1b8ec77af4a95107519578e30
-  data.tar.gz: fd296837f7d3a2df0f4db78768e27dd151363ab4
+  metadata.gz: 787b142325f9c37a78279d612dcbf9c2907869da
+  data.tar.gz: c02c9a17a1da150c276c8e8443d8fdb523d65cf2
 SHA512:
-  metadata.gz: 06f6c3fef7926e466b7525d544deefaed3cbc23535c3070a4ef087e866e33ba383c6b453fcdda84fcd6ef78270ad57c7e2e4184a9709d4b28163253b20717b71
-  data.tar.gz: c3781148eebedb327930d636c9d310e56d3f567bf7ceb9ecdb869a3dc861eb0ece7925da54be290567884792bb7839316c4903a43587c5b7d1d9db4a7d698a59
+  metadata.gz: 50f25f941e07140ebe6b68c4acbe3e3d0d0fe8e2a689f1ea54e95df83c162a42a63bc2faa2cd9e66b461e40455bd1997c84e55e98d1dabb6542fc396280c8c84
+  data.tar.gz: eb32d162808540c111d63453d6b5dab8fee822bfd62fa532c530f24f3837a0ff687f562e5e5241739b594e6d747c4ec1f985c30fc3facf8c920f77f6862790c7

data/.gitignore

@@ -2,3 +2,5 @@
 locales/
 .localeapp/
 Gemfile.lock
+*.gem
+/coverage

data/.rspec

@@ -0,0 +1,2 @@
+--format documentation
+--color

data/.travis.yml

@@ -0,0 +1,9 @@
+language: ruby
+services:
+  - redis
+rvm:
+  - 2.1.6
+  - 2.2.0
+script:
+  - bundle
+  - bundle exec rspec .

data/Gemfile

@@ -2,7 +2,11 @@ source 'https://rubygems.org'
 
 gemspec
 
-group :development do
+group :development, :test do
   gem 'redis'
   gem 'byebug'
 end
+
+group :test do
+  gem 'simplecov'
+end

data/Rakefile

@@ -0,0 +1 @@
+require 'bundler/gem_tasks'

data/lib/translator/synchronizer.rb

@@ -1,3 +1,5 @@
+require 'localeapp'
+
 module Translator
   class Synchronizer
     attr_reader :paths, :output_stream
@@ -12,14 +14,25 @@ module Translator
     end
 
     private
+      def pusher
+        @pusher ||= ::Localeapp::CLI::Push.new(output: output_stream)
+      end
+
+      def puller
+        @puller ||= Localeapp::CLI::Pull.new(output: output_stream)
+      end
+
+      def updator
+        @updator ||= Localeapp::CLI::Update.new(output: output_stream)
+      end
+
       def push
-        pusher = ::Localeapp::CLI::Push.new(output: output_stream)
         paths.each { |path| pusher.execute(path)  }
       end
 
       def update
         pull unless pulled?
-        Localeapp::CLI::Update.new(output: output_stream).execute
+        updator.execute
       end
 
       def pull
@@ -27,7 +40,7 @@ module Translator
         FileUtils.mkdir_p(Translator.data_directory)
         FileUtils.mkdir_p(Pathname.new(Translator.synchronization_data_file).parent)
         File.open(Translator.synchronization_data_file, 'w') { }
-        Localeapp::CLI::Pull.new(output: output_stream).execute
+        puller.execute
       end
 
       def pulled?
@@ -37,6 +50,5 @@ module Translator
       def resolve_path(path)
         File.expand_path(path, __FILE__)
       end
-
   end
 end

data/lib/translator/version.rb

@@ -1,5 +1,5 @@
 module Translator
   module Version
-    VERSION = '1.0.0'.freeze
+    VERSION = '1.0.1'.freeze
   end
 end

data/spec/helpers/test_data/file1.yml

@@ -0,0 +1,2 @@
+en:
+  first_name: 'Kuldeep'

data/spec/helpers/test_data/file2.yml

@@ -0,0 +1,2 @@
+en:
+  last_name: 'Aggarwal'

data/spec/spec_helper.rb

@@ -0,0 +1,17 @@
+$LOAD_PATH.unshift File.expand_path('../../lib', __FILE__)
+require 'byebug'
+require 'simplecov'
+require 'translator'
+SimpleCov.start
+
+Translator.setup do |config|
+  config.synchronization_data_file = File.expand_path('../.localeapp/log.yml', __FILE__)
+  config.data_directory            = File.expand_path('../locales', __FILE__)
+end
+
+RSpec.configure do |config|
+  config.after(:all) do
+    FileUtils.rm_rf(Translator.data_directory)
+    FileUtils.rm_rf(Pathname.new(Translator.synchronization_data_file).parent)
+  end
+end

data/spec/translator/null_stream_spec.rb

@@ -0,0 +1,18 @@
+require 'spec_helper'
+
+describe Translator::NullStream do
+  let(:subject) { Translator::NullStream.new }
+
+  describe '#puts' do
+    it 'does not print' do
+      expect(subject.puts("Test")).to eq(nil)
+    end
+  end
+
+  describe 'Aliasing' do
+    it 'write and << are aliases of puts' do
+      expect(subject.method(:write)).to eq(subject.method(:puts))
+      expect(subject.method(:<<)).to eq(subject.method(:puts))
+    end
+  end
+end

data/spec/translator/store_spec.rb

@@ -0,0 +1,60 @@
+require 'spec_helper'
+
+describe Translator::Store do
+  let(:store) { Translator::Store.instance }
+
+  before { store.clear }
+
+  describe '#write' do
+    before { store.write('first_name', nil) }
+
+    it 'writes to the storage' do
+      store.write('first_name', 'Kuldeep')
+      expect(store.read('first_name')).to eq('Kuldeep')
+    end
+  end
+
+  describe '#read' do
+    context 'storage has nothing as data' do
+      it 'returns nil' do
+        expect(store.read('first_name')).to eq(nil)
+      end
+    end
+
+    context 'storage has something as data' do
+      before do
+        store.write('first_name', 'Kuldeep')
+        store.write('lastname', nil)
+      end
+
+      it 'lastname is nil' do
+        expect(store.read('lastname')).to eq(nil)
+      end
+
+      it 'first_name == Kuldeep' do
+        expect(store.read('first_name')).to eq('Kuldeep')
+      end
+    end
+  end
+
+  describe '#load_files' do
+    let(:yaml_data) {
+      {
+        en: { first_name: 'Kuldeep' },
+        pa: { first_name: 'ਕੁਲਦੀਪ'   }
+      }
+    }
+    before do
+      allow_any_instance_of(Translator::YamlFileFlattener).to receive(:process).and_return(yaml_data)
+    end
+    after do
+      store.clear
+    end
+
+    it 'loads all yml files in the storage' do
+      store.load_files(nil)
+      expect(store[:en]).to eq(first_name: 'Kuldeep')
+      expect(store[:pa]).to eq(first_name: 'ਕੁਲਦੀਪ')
+    end
+  end
+end

data/spec/translator/synchronizer_spec.rb

@@ -0,0 +1,90 @@
+require 'spec_helper'
+
+class TestSynchronizer
+  def execute(*args); end
+end
+
+class TestPusher < TestSynchronizer; end
+class TestPuller < TestSynchronizer; end
+class TestUpdator < TestSynchronizer; end
+
+describe Translator::Synchronizer do
+  let(:file_paths) { Dir[File.expand_path('../../helpers/test_data/*.yml', __FILE__)] }
+  let(:synchronizer) do
+    syn = Translator::Synchronizer.new(file_paths)
+    syn.instance_eval do
+      def puller
+        @puller ||= TestPuller.new
+      end
+
+      def pusher
+        @pusher ||= TestPusher.new
+      end
+
+      def updator
+        @updator ||= TestUpdator.new
+      end
+    end
+    syn
+  end
+
+  describe '#process' do
+    it 'pushes and updates all files for processing' do
+      expect(synchronizer).to receive(:push)
+      expect(synchronizer).to receive(:update)
+      synchronizer.process
+    end
+  end
+
+  describe '#push' do
+    it 'pushes all file by Localeapp' do
+      expect(synchronizer.pusher).to receive(:execute).exactly(file_paths.count).times
+      synchronizer.send(:push)
+    end
+  end
+
+  describe '#update' do
+    context 'when ever pulled' do
+      before { allow(synchronizer).to receive(:pulled?).and_return(true) }
+
+      it 'does not pull the files' do
+        expect(synchronizer).to_not receive(:pull)
+        synchronizer.send(:update)
+      end
+    end
+
+    context 'when pulled neever' do
+      before { allow(synchronizer).to receive(:pulled?).and_return(false) }
+      after { FileUtils.rm_rf(Pathname.new(Translator.synchronization_data_file).parent) }
+
+      it 'pulls the file' do
+        expect(synchronizer.puller).to receive(:execute)
+        synchronizer.send(:update)
+        expect(File.exist?(Translator.synchronization_data_file)).to eq(true)
+      end
+    end
+
+    it 'updates the files' do
+      expect(synchronizer.updator).to receive(:execute)
+      synchronizer.send(:update)
+    end
+  end
+
+  describe '#puller' do
+    it 'is an instance of Localeapp::CLI::Pull' do
+      expect(Translator::Synchronizer.new([]).send(:puller)).to be_an_instance_of(Localeapp::CLI::Pull)
+    end
+  end
+
+  describe '#pusher' do
+    it 'is an instance of Localeapp::CLI::Push' do
+      expect(Translator::Synchronizer.new([]).send(:pusher)).to be_an_instance_of(Localeapp::CLI::Push)
+    end
+  end
+
+  describe '#updator' do
+    it 'is an instance of Localeapp::CLI::Update' do
+      expect(Translator::Synchronizer.new([]).send(:updator)).to be_an_instance_of(Localeapp::CLI::Update)
+    end
+  end
+end

data/spec/translator/yaml_file_flattener_spec.rb

@@ -0,0 +1,17 @@
+require 'spec_helper'
+
+describe Translator::YamlFileFlattener do
+  describe '#process' do
+    let(:file_paths) { Dir[File.expand_path('../../helpers/test_data/*.yml', __FILE__)] }
+    let(:flattener) { Translator::YamlFileFlattener.new(file_paths) }
+
+    it 'merges all yaml files data' do
+      expect(flattener.process).to eq({
+        'en' => {
+          'first_name' => 'Kuldeep',
+          'last_name'  => 'Aggarwal'
+        }
+      })
+    end
+  end
+end

data/spec/translator_spec.rb

@@ -0,0 +1,64 @@
+require 'spec_helper'
+
+describe Translator do
+  it 'has default Redis Storage' do
+    expect(Translator.storage).to be :Redis
+  end
+
+  it 'has default storage options' do
+    expect(Translator.storage_options).to eq({ host: 'localhost', port: 6379, db: 1 })
+  end
+
+  it 'has output stream' do
+    expect(Translator).to respond_to(:output_stream)
+  end
+
+  it 'has default Null output stream' do
+    expect(Translator.default_output_stream).to be_an_instance_of(Translator::NullStream)
+  end
+
+  it 'has a log file in the .localapp folder' do
+    expect(Translator.synchronization_data_file).to_not be_nil
+  end
+
+  it 'syncs the data in data directory' do
+    expect(Translator.data_directory).to eq(File.expand_path('../../spec/locales', __FILE__))
+  end
+
+  it 'does not have localeapp api key' do
+    expect(Translator.localeapp_api_key).to be_empty
+  end
+
+  describe 'Class Methods' do
+    before do
+      FileUtils.mkdir_p(Translator.data_directory)
+    end
+
+    after { FileUtils.rm_rf(Translator.data_directory) }
+
+    describe '.default_file_paths' do
+      let(:file_path) { "#{ Translator.data_directory }/_test.yml" }
+
+      before do
+        File.open(file_path, 'w') {}
+      end
+
+      it 'returns all .yml file paths in the data directory' do
+        expect(Translator.default_file_paths).to eq([file_path])
+      end
+    end
+
+    describe '.load!' do
+      after { Translator.load!(nil) }
+
+      it 'synchronizes locale files from the server' do
+        expect_any_instance_of(Translator::Synchronizer).to receive(:process).and_return(true)
+      end
+
+      it 'loads the locale in the store' do
+        allow_any_instance_of(Translator::Synchronizer).to receive(:process).and_return(true)
+        expect(Translator::Store.instance).to receive(:load_files).with(nil)
+      end
+    end
+  end
+end

data/translator.gemspec

@@ -22,7 +22,11 @@ Gem::Specification.new do |s|
 
   s.required_ruby_version = '>= 1.9.3'
 
-  s.add_dependency('localeapp', '~> 0.9')
-  s.add_dependency('moneta', '~> 0.8')
-  s.add_dependency('activesupport', '>= 3.2', '< 5')
+  s.add_dependency 'localeapp',     '~> 0.9'
+  s.add_dependency 'moneta',        '~> 0.8'
+  s.add_dependency 'activesupport', '>= 3.2', '< 5'
+
+  s.add_development_dependency 'bundler', '~> 1.6'
+  s.add_development_dependency 'rake',    '~> 10.0'
+  s.add_development_dependency 'rspec',   '~> 3.2.0', '>= 3.1.0'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: translator_with_localeapp
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.0.1
 platform: ruby
 authors:
 - Kuldeep Aggarwal
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-05-22 00:00:00.000000000 Z
+date: 2015-05-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: localeapp
@@ -58,6 +58,54 @@ dependencies:
     - - <
       - !ruby/object:Gem::Version
         version: '5'
+- !ruby/object:Gem::Dependency
+  name: bundler
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '1.6'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '1.6'
+- !ruby/object:Gem::Dependency
+  name: rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '10.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '10.0'
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 3.2.0
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: 3.1.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 3.2.0
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: 3.1.0
 description: Flexible syncronization solution of locales with LocaleApp.
 email:
 - kd.engineer@yahoo.co.in
@@ -66,15 +114,26 @@ extensions: []
 extra_rdoc_files: []
 files:
 - .gitignore
+- .rspec
+- .travis.yml
 - Gemfile
 - LICENSE.txt
 - README.md
+- Rakefile
 - lib/translator.rb
 - lib/translator/null_stream.rb
 - lib/translator/store.rb
 - lib/translator/synchronizer.rb
 - lib/translator/version.rb
 - lib/translator/yaml_file_flattener.rb
+- spec/helpers/test_data/file1.yml
+- spec/helpers/test_data/file2.yml
+- spec/spec_helper.rb
+- spec/translator/null_stream_spec.rb
+- spec/translator/store_spec.rb
+- spec/translator/synchronizer_spec.rb
+- spec/translator/yaml_file_flattener_spec.rb
+- spec/translator_spec.rb
 - translator.gemspec
 homepage: https://github.com/kuldeepaggarwal/translator
 licenses: