copycat 0.0.3 → 0.0.4

data/spec/factories/copycat_translations.rb

@@ -6,6 +6,7 @@ FactoryGirl.define do
   factory :copycat_translation do
     key { Factory.next(:string) }
     value { Factory.next(:string) }
+    locale { "en" }
   end
 end
 

data/spec/integration/copycat_spec.rb

@@ -89,8 +89,6 @@ feature "downloading and uploading yaml files" do
   end
 
   it "gives 400 on bad upload" do
-    visit "/copycat_translations.yaml"
-
     file = Tempfile.new 'copycat'
     file.write "<<<%%#$W%s"
     file.close
@@ -106,3 +104,64 @@ feature "downloading and uploading yaml files" do
 
 end
 
+feature "locales" do
+
+  it "displays different text based on users' locale" do
+    Factory(:copycat_translation, locale: 'en', key: 'site.index.intro', value: 'world')
+    Factory(:copycat_translation, locale: 'es', key: 'site.index.intro', value: 'mundo')
+
+    I18n.locale = :en
+    visit root_path
+    page.should have_content 'world'
+    page.should_not have_content 'mundo'
+    
+    I18n.locale = :es
+    visit root_path
+    page.should have_content 'mundo'
+    page.should_not have_content 'world'
+    
+    I18n.locale = :fa
+    visit root_path
+    page.should_not have_content 'world'
+    page.should_not have_content 'mundo'
+
+    I18n.locale = :en  # reset
+  end
+
+  it "imports yaml containing multiple locales" do
+    file = Tempfile.new 'copycat'
+    file.write <<-YAML
+      en:
+        hello: world
+      es:
+        hello: mundo
+    YAML
+    file.close
+
+    page.driver.browser.basic_authorize COPYCAT_USERNAME, COPYCAT_PASSWORD
+    visit upload_copycat_translations_path
+    attach_file "file", file.path
+    click_button "Upload"
+    file.unlink
+
+    assert CopycatTranslation.count == 2
+    a = CopycatTranslation.where(locale: 'en').first
+    assert a.key == 'hello'
+    assert a.value == 'world'
+    b = CopycatTranslation.where(locale: 'es').first
+    assert b.key == 'hello'
+    assert b.value == 'mundo'
+  end
+
+  it "exports yaml containing multiple locales" do
+    Factory(:copycat_translation, locale: 'en', key: 'hello', value: 'world')
+    Factory(:copycat_translation, locale: 'es', key: 'hello', value: 'mundo')
+
+    page.driver.browser.basic_authorize COPYCAT_USERNAME, COPYCAT_PASSWORD
+    visit "/copycat_translations.yaml"
+    yaml = page.text
+    assert yaml =~ /en:\s*hello: world/
+    assert yaml =~ /es:\s*hello: mundo/  
+  end
+
+end

data/spec/lib/copycat_spec.rb

@@ -31,30 +31,12 @@ describe Copycat do
     end
     it "returns copycat_translation if present" do
       cct = FactoryGirl.create(:copycat_translation)
-      base.lookup(nil, cct.key).should == cct.value
+      base.lookup(cct.locale, cct.key).should == cct.value
     end
     it "creates copycat_translation if one is missing" do
-      CopycatTranslation.find_by_key('foo').should be_nil
-      base.lookup(nil, 'foo')
-      CopycatTranslation.find_by_key('foo').should_not be_nil
-    end
-  end
-
-  describe "#hash_flatten" do
-    it "flattens hashes" do
-      before = {"a" => {"b" => "c", "d" => "e"}, "f" => {"g" => {"h" => "i", "j" => "k"}, "l" => "m"}}
-      after = Copycat.hash_flatten(before)
-      assert after == {"a.b" => "c", "a.d" => "e", "f.g.h" => "i", "f.g.j" => "k", "f.l" => "m"}
-    end
-  end
-
-  describe "#hash_fatten" do
-    it "fattens hashes" do
-      before = {"a" => {"b" => "c", "d" => "e"}, "f" => {"g" => {"h" => "i"}, "l" => "m"}}
-      keys = "f.g.j".split(".")
-      value = "k"
-      after = Copycat.hash_fatten(before, keys, value)
-      assert after == {"a" => {"b" => "c", "d" => "e"}, "f" => {"g" => {"h" => "i", "j" => "k"}, "l" => "m"}}
+      CopycatTranslation.where(locale: 'en', key: 'foo').should be_empty
+      base.lookup('en', 'foo')
+      CopycatTranslation.where(locale: 'en', key: 'foo').should_not be_empty
     end
   end
 

data/spec/models/copycat_translation_spec.rb

@@ -3,6 +3,32 @@ require 'spec_helper'
 
 describe CopycatTranslation do
 
+  describe "database constraints" do
+    it "validates uniqueness of key & locale" do
+      CopycatTranslation.new(key: "foo", locale: "en", value: "bar").save
+      a = CopycatTranslation.new(key: "foo", locale: "en", value: "bar2")
+      expect { a.save }.should raise_error
+      b = CopycatTranslation.new(key: "foo", locale: "fa", value: "bar")
+      expect { b.save }.should_not raise_error
+    end
+  end
+  
+  describe "helper methods" do
+    it "flattens hashes" do
+      before = {"a" => {"b" => "c", "d" => "e"}, "f" => {"g" => {"h" => "i", "j" => "k"}, "l" => "m"}}
+      after = CopycatTranslation.hash_flatten(before)
+      assert after == {"a.b" => "c", "a.d" => "e", "f.g.h" => "i", "f.g.j" => "k", "f.l" => "m"}
+    end
+
+    it "fattens hashes" do
+      hash = {"a" => {"b" => "c", "d" => "e"}, "f" => {"g" => {"h" => "i"}, "l" => "m"}}
+      keys = "f.g.j".split(".")
+      value = "k"
+      CopycatTranslation.hash_fatten!(hash, keys, value)
+      assert hash == {"a" => {"b" => "c", "d" => "e"}, "f" => {"g" => {"h" => "i", "j" => "k"}, "l" => "m"}}
+    end
+  end
+
   it "imports YAML" do
     Factory(:copycat_translation, :key => "sample_copy", :value => "copyfoo")
     Factory(:copycat_translation, :key => "sample_copy2", :value => "copybaz")
@@ -27,7 +53,7 @@ describe CopycatTranslation do
     it "can be consumed by i18N" do
       I18n.t('site.title').should_not == 'My Blog'
       CopycatTranslation.destroy_all
-      CopycatTranslation.create(key: 'site.title', value: 'My Blog')
+      CopycatTranslation.create(key: 'site.title', value: 'My Blog', locale: 'en')
       data = YAML.load(CopycatTranslation.export_yaml)
       CopycatTranslation.destroy_all
       data.each { |locale, d| I18n.backend.store_translations(locale, d || {}) } #i18n/backend/base.rb:159

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: copycat
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.0.4
   prerelease: 
 platform: ruby
 authors:
@@ -10,11 +10,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-03-19 00:00:00.000000000 Z
+date: 2012-03-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
-  requirement: &2151799440 !ruby/object:Gem::Requirement
+  requirement: &2151799480 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -22,7 +22,7 @@ dependencies:
         version: 3.0.0
   type: :runtime
   prerelease: false
-  version_requirements: *2151799440
+  version_requirements: *2151799480
 description: Edit live website copy.
 email:
 - info@vermonster.com
@@ -73,7 +73,7 @@ files:
 - spec/dummy/config/routes.rb
 - spec/dummy/config.ru
 - spec/dummy/db/development.sqlite3
-- spec/dummy/db/migrate/20120316153248_create_copycat_translations.copycat_engine.rb
+- spec/dummy/db/migrate/20120320143433_create_copycat_translations.copycat_engine.rb
 - spec/dummy/db/schema.rb
 - spec/dummy/db/test.sqlite3
 - spec/dummy/log/development.log
@@ -147,7 +147,7 @@ test_files:
 - spec/dummy/config/routes.rb
 - spec/dummy/config.ru
 - spec/dummy/db/development.sqlite3
-- spec/dummy/db/migrate/20120316153248_create_copycat_translations.copycat_engine.rb
+- spec/dummy/db/migrate/20120320143433_create_copycat_translations.copycat_engine.rb
 - spec/dummy/db/schema.rb
 - spec/dummy/db/test.sqlite3
 - spec/dummy/log/development.log