copycat 0.2.6 → 0.2.7

data/spec/dummy/tmp/cache/assets/test/sprockets/v3.0/5pjjE7UQ1zT-W3Rm7l8onDEWZV9b-lyRmeW9nFAbLoU.cache

@@ -0,0 +1,2 @@
+[o:Set:
+@hash{I"environment-version:ETTI"environment-paths;TTI"0processors:type=text/css&file_type=text/css;TTI"cfile-digest:///Users/andrewross/code/copycat/spec/dummy/app/assets/stylesheets/application.css;TTI">processors:type=text/css&file_type=text/css&pipeline=self;TTI"Sfile-digest:///Users/andrewross/code/copycat/spec/dummy/app/assets/stylesheets;TT

data/spec/dummy/tmp/cache/assets/test/sprockets/v3.0/9ZnyF5L7jalLFn0sk_WNwUCeeXBylJBp2yRgFPQE3ho.cache

@@ -0,0 +1 @@
+I"
�file:///Users/andrewross/code/copycat/spec/dummy/app/assets/stylesheets/application.css?type=text/css&id=44a81ee07e41d721880da38bef491b2aae63a506c3b4b40084eea75e9fc78e98:ET

data/spec/dummy/tmp/cache/assets/test/sprockets/v3.0/YQfOnucF3bHs51e6OOpX6Gh8ih98sfqcSc88X90cxYg.cache

@@ -0,0 +1 @@
+I"
�file:///Users/andrewross/code/copycat/spec/dummy/app/assets/stylesheets/application.css?type=text/css&pipeline=self&id=4bfd9c65970fe06b4964470bbf20c1c68255b2420fa04d5b4a39a9999e0057c5:ET

data/spec/dummy/tmp/cache/assets/test/sprockets/v3.0/bUaVNTYJb9Cp-33TI2vIHrdmO8U0fOh3szyB6M4K7BY.cache

@@ -0,0 +1,3 @@
+[o:Set:
+@hash{
+I"environment-version:ETTI"environment-paths;TTI">processors:type=text/css&file_type=text/css&pipeline=self;TTI"cfile-digest:///Users/andrewross/code/copycat/spec/dummy/app/assets/stylesheets/application.css;TTI"Sfile-digest:///Users/andrewross/code/copycat/spec/dummy/app/assets/stylesheets;TT

data/spec/dummy/tmp/cache/assets/test/sprockets/v3.0/qGrLWZnG1ElHmKVImQl1l5olVVtVeDSZ13_9Ll1qeYU.cache

@@ -0,0 +1 @@
+"%9�#�U�-���Jnݶ�W��Gx5>T�u�\x��:�

data/spec/dummy/tmp/cache/assets/test/sprockets/v3.0/zQEVaz1UlfTQQY8e-XT2PRLny8dybtFJeuDtPTwwS2E.cache

@@ -0,0 +1 @@
+"%��Rm�v�S3.��O�L���ڹc�ƕq�=_j�

data/spec/factories/copycat_translations.rb

@@ -2,10 +2,10 @@ FactoryGirl.define do
   sequence :string do |n|
     "string%09d" % n
   end
-  
+
   factory :copycat_translation do
-    key { FactoryGirl.generate(:string) }
-    value { FactoryGirl.generate(:string) }
+    key { generate(:string) }
+    value { generate(:string) }
     locale I18n.default_locale
   end
 end

data/spec/{integration → features}/copycat_spec.rb

@@ -4,9 +4,9 @@ require 'spec_helper'
 
 
 feature "copycat index" do
- 
+
   before do
-    FactoryGirl.create(:copycat_translation, :key => "foo", :value => "bar")
+    create(:copycat_translation, :key => "foo", :value => "bar")
     page.driver.browser.basic_authorize Copycat.username, Copycat.password
     visit copycat_translations_path
   end
@@ -18,59 +18,59 @@ feature "copycat index" do
   end
 
   it "doesn't show any tokens by default" do
-    page.should_not have_content 'foo'
-    page.should_not have_content 'bar'
+    expect(page).not_to have_content 'foo'
+    expect(page).not_to have_content 'bar'
   end
 
   it "allows search by key" do
     fill_in 'search', :with => 'foo'
     click_button 'Search'
-    page.should have_content 'foo'
-    page.should have_content 'bar'
+    expect(page).to have_content 'foo'
+    expect(page).to have_content 'bar'
   end
 
   it "allows search by key" do
     fill_in 'search', :with => 'xfoo'
     click_button 'Search'
-    page.should_not have_content 'foo'
-    page.should_not have_content 'bar'
+    expect(page).not_to have_content 'foo'
+    expect(page).not_to have_content 'bar'
   end
 
   it "allows search by value" do
     fill_in 'search', :with => 'bar'
     click_button 'Search'
-    page.should have_content 'foo'
-    page.should have_content 'bar'
+    expect(page).to have_content 'foo'
+    expect(page).to have_content 'bar'
   end
 
   it "allows search by value" do
     fill_in 'search', :with => 'xbar'
     click_button 'Search'
-    page.should_not have_content 'foo'
-    page.should_not have_content 'bar'
+    expect(page).not_to have_content 'foo'
+    expect(page).not_to have_content 'bar'
   end
-  
+
   it "searches in the middles of strings" do
-    FactoryGirl.create(:copycat_translation, :key => "site.index.something")
+    create(:copycat_translation, :key => "site.index.something")
     fill_in 'search', :with => 'index'
     click_button 'Search'
-    page.should have_content 'site.index.something'
+    expect(page).to have_content 'site.index.something'
   end
 
   it "can show all" do
-    FactoryGirl.create(:copycat_translation, :key => "foe", :value => "beer")
+    create(:copycat_translation, :key => "foe", :value => "beer")
     click_button 'Search'
-    page.should have_content 'foo'
-    page.should have_content 'foe'
+    expect(page).to have_content 'foo'
+    expect(page).to have_content 'foe'
   end
 
   context "more than one locale" do
 
     before do
       CopycatTranslation.destroy_all
-      FactoryGirl.create(:copycat_translation, key: "foo", value: "bar1", locale: "en")
-      FactoryGirl.create(:copycat_translation, key: "foo", value: "bar2", locale: "fa")
-      FactoryGirl.create(:copycat_translation, key: "foo", value: "bar3", locale: "it")
+      create(:copycat_translation, key: "foo", value: "bar1", locale: "en")
+      create(:copycat_translation, key: "foo", value: "bar2", locale: "fa")
+      create(:copycat_translation, key: "foo", value: "bar3", locale: "it")
     end
 
     #locale
@@ -95,100 +95,100 @@ feature "copycat index" do
 
     it "nil locale, nil search" do
       visit copycat_translations_path
-      page.should_not have_content 'foo'
+      expect(page).not_to have_content 'foo'
     end
 
     it "nil locale, blank search" do
       # impossible for user to replicate this case
       visit copycat_translations_path('search' => '', 'commit' => 'Search')
-      page.should have_content 'bar1'
-      page.should_not have_content 'bar2'
-      page.should_not have_content 'bar3'
+      expect(page).to have_content 'bar1'
+      expect(page).not_to have_content 'bar2'
+      expect(page).not_to have_content 'bar3'
     end
 
     it "nil locale, present search" do
       # impossible for user to replicate this case
       visit copycat_translations_path('search' => 'foo', 'commit' => 'Search')
-      page.should have_content 'bar1'
-      page.should_not have_content 'bar2'
-      page.should_not have_content 'bar3'
+      expect(page).to have_content 'bar1'
+      expect(page).not_to have_content 'bar2'
+      expect(page).not_to have_content 'bar3'
       visit copycat_translations_path('search' => 'fuu', 'commit' => 'Search')
-      page.should_not have_content 'foo'
+      expect(page).not_to have_content 'foo'
     end
 
     it "blank locale, nil search" do
       # impossible for user to replicate this case
       visit copycat_translations_path('locale' => '', 'commit' => 'Search')
-      page.should_not have_content 'foo'
+      expect(page).not_to have_content 'foo'
     end
 
     it "blank locale, blank search" do
       select '', :from => 'locale'
       click_button 'Search'
-      page.should have_content 'bar1'
-      page.should have_content 'bar2'
-      page.should have_content 'bar3'
+      expect(page).to have_content 'bar1'
+      expect(page).to have_content 'bar2'
+      expect(page).to have_content 'bar3'
     end
 
     it "blank locale, present search" do
       select '', :from => 'locale'
       fill_in 'search', :with => 'foo'
       click_button 'Search'
-      page.should have_content 'bar1'
-      page.should have_content 'bar2'
-      page.should have_content 'bar3'
+      expect(page).to have_content 'bar1'
+      expect(page).to have_content 'bar2'
+      expect(page).to have_content 'bar3'
       fill_in 'search', :with => 'fuu'
       click_button 'Search'
-      page.should_not have_content 'foo'
+      expect(page).not_to have_content 'foo'
     end
 
     it "present locale, nil search" do
       # impossible for user to replicate this case
       visit copycat_translations_path('locale' => 'en', 'commit' => 'Search')
-      page.should_not have_content 'foo'
+      expect(page).not_to have_content 'foo'
     end
 
     it "present locale, blank search" do
       select 'en', :from => 'locale'
       click_button 'Search'
-      page.should have_content 'bar1'
-      page.should_not have_content 'bar2'
-      page.should_not have_content 'bar3'
+      expect(page).to have_content 'bar1'
+      expect(page).not_to have_content 'bar2'
+      expect(page).not_to have_content 'bar3'
       select 'fa', :from => 'locale'
       click_button 'Search'
-      page.should_not have_content 'bar1'
-      page.should have_content 'bar2'
-      page.should_not have_content 'bar3'
+      expect(page).not_to have_content 'bar1'
+      expect(page).to have_content 'bar2'
+      expect(page).not_to have_content 'bar3'
       select 'it', :from => 'locale'
       click_button 'Search'
-      page.should_not have_content 'bar1'
-      page.should_not have_content 'bar2'
-      page.should have_content 'bar3'
+      expect(page).not_to have_content 'bar1'
+      expect(page).not_to have_content 'bar2'
+      expect(page).to have_content 'bar3'
     end
 
     it "present locale, present search" do
       select 'en', :from => 'locale'
       fill_in 'search', :with => 'foo'
       click_button 'Search'
-      page.should have_content 'bar1'
-      page.should_not have_content 'bar2'
-      page.should_not have_content 'bar3'
+      expect(page).to have_content 'bar1'
+      expect(page).not_to have_content 'bar2'
+      expect(page).not_to have_content 'bar3'
       select 'fa', :from => 'locale'
       fill_in 'search', :with => 'foo'
       click_button 'Search'
-      page.should_not have_content 'bar1'
-      page.should have_content 'bar2'
-      page.should_not have_content 'bar3'
+      expect(page).not_to have_content 'bar1'
+      expect(page).to have_content 'bar2'
+      expect(page).not_to have_content 'bar3'
       select 'it', :from => 'locale'
       fill_in 'search', :with => 'foo'
       click_button 'Search'
-      page.should_not have_content 'bar1'
-      page.should_not have_content 'bar2'
-      page.should have_content 'bar3'
+      expect(page).not_to have_content 'bar1'
+      expect(page).not_to have_content 'bar2'
+      expect(page).to have_content 'bar3'
       select 'en', :from => 'locale'
       fill_in 'search', :with => 'fuu'
       click_button 'Search'
-      page.should_not have_content 'foo'
+      expect(page).not_to have_content 'foo'
     end
 
   end
@@ -197,7 +197,7 @@ end
 
 feature "copycat edit" do
   before do
-    FactoryGirl.create(:copycat_translation, :key => "foo", :value => "bar")
+    create(:copycat_translation, :key => "foo", :value => "bar")
     page.driver.browser.basic_authorize Copycat.username, Copycat.password
     visit copycat_translations_path
   end
@@ -212,7 +212,7 @@ end
 
 feature "copycat update, delete" do
   before do
-    FactoryGirl.create(:copycat_translation, :key => "foo", :value => "bar")
+    create(:copycat_translation, :key => "foo", :value => "bar")
     page.driver.browser.basic_authorize Copycat.username, Copycat.password
     visit copycat_translations_path
     fill_in 'search', :with => 'foo'
@@ -223,16 +223,16 @@ feature "copycat update, delete" do
   scenario "update" do
     fill_in "copycat_translation[value]", :with => 'baz'
     click_button "Update"
-    current_path.should == copycat_translations_path
-    CopycatTranslation.find_by_key("foo").value.should == 'baz'
-    page.should have_content "foo updated!"
+    expect(current_path).to eq(copycat_translations_path)
+    expect(CopycatTranslation.find_by_key("foo").value).to eq('baz')
+    expect(page).to have_content "foo updated!"
   end
 
   scenario "delete" do
     click_button "Delete this item"
-    current_path.should == copycat_translations_path
-    CopycatTranslation.find_by_key("foo").should be_nil
-    page.should have_content "foo deleted!"
+    expect(current_path).to eq(copycat_translations_path)
+    expect(CopycatTranslation.find_by_key("foo")).to be_nil
+    expect(page).to have_content "foo deleted!"
   end
 end
 
@@ -242,11 +242,11 @@ feature "downloading and uploading yaml files" do
   end
 
   it "round-trips the YAML" do
-    FactoryGirl.create(:copycat_translation, :key => "a.foo1", :value => "bar1")
-    FactoryGirl.create(:copycat_translation, :key => "a.foo2:", :value => "bar2")
-    FactoryGirl.create(:copycat_translation, :key => "a.b.foo3", :value => "bar3")
-    FactoryGirl.create(:copycat_translation, :key => "c.foo4", :value => "bar4")
-    FactoryGirl.create(:copycat_translation, :key => 2, :value => "bar5")
+    create(:copycat_translation, :key => "a.foo1", :value => "bar1")
+    create(:copycat_translation, :key => "a.foo2:", :value => "bar2")
+    create(:copycat_translation, :key => "a.b.foo3", :value => "bar3")
+    create(:copycat_translation, :key => "c.foo4", :value => "bar4")
+    create(:copycat_translation, :key => 2, :value => "bar5")
     assert CopycatTranslation.count == 5
 
     visit import_export_copycat_translations_path
@@ -254,7 +254,7 @@ feature "downloading and uploading yaml files" do
     CopycatTranslation.destroy_all
     assert CopycatTranslation.count == 0
 
-    yaml = page.text
+    yaml = page.body
     file = Tempfile.new 'copycat'
     file.write yaml
     file.close
@@ -274,13 +274,13 @@ feature "downloading and uploading yaml files" do
 
   it "round-trips the yaml with complicated text" do
     value = "“hello world“ üokåa®fgsdf;::fs;kdf"
-    FactoryGirl.create(:copycat_translation, :key => "a.foo", :value => value)
+    create(:copycat_translation, :key => "a.foo", :value => value)
 
     visit import_export_copycat_translations_path
     click_link 'Download as YAML'
     CopycatTranslation.destroy_all
 
-    yaml = page.text
+    yaml = page.body
     file = Tempfile.new 'copycat'
     file.write yaml
     file.close
@@ -301,8 +301,8 @@ feature "downloading and uploading yaml files" do
     attach_file "file", file.path
     click_button "Upload"
     file.unlink
-    page.status_code.should == 400
-    page.should have_content("There was an error processing your upload!")
+    expect(page.status_code).to eq(400)
+    expect(page).to have_content("There was an error processing your upload!")
     assert CopycatTranslation.count == 0
   end
 
@@ -339,13 +339,13 @@ feature "locales" do
   end
 
   it "exports yaml containing multiple locales" do
-    FactoryGirl.create(:copycat_translation, locale: 'en', key: 'hello', value: 'world')
-    FactoryGirl.create(:copycat_translation, locale: 'es', key: 'hello', value: 'mundo')
+    create(:copycat_translation, locale: 'en', key: 'hello', value: 'world')
+    create(:copycat_translation, locale: 'es', key: 'hello', value: 'mundo')
 
     visit download_copycat_translations_path
     yaml = page.text
     assert yaml =~ /en:\s*hello: world/
-    assert yaml =~ /es:\s*hello: mundo/  
+    assert yaml =~ /es:\s*hello: mundo/
   end
 
 end

data/spec/features/dummy_spec.rb

@@ -0,0 +1,92 @@
+#encoding: utf-8
+
+require 'spec_helper'
+
+feature "use #t" do
+
+  it "the dummy app has a translation for site.index.header but not site.index.intro" do
+    expect(I18n.t('site.index.header')).to eq('The Header')
+    expect(I18n.t('site.index.intro')).to eq("translation missing: en.site.index.intro")
+  end
+
+  it "uses i18n.t" do
+    visit root_path
+    expect(page).to have_content 'The Header'
+    expect(page).to have_content 'Intro' #ActionView::Helpers::TranslationHelper#translate wrapper
+  end
+
+  it "creates a copycat_translation if the yaml has an entry" do
+    expect(CopycatTranslation.find_by_key('site.index.header')).to be_nil
+    visit root_path
+    expect(CopycatTranslation.find_by_key('site.index.header')).not_to be_nil
+  end
+
+  it "creates a copycat_translation if the yaml does not have an entry" do
+    expect(CopycatTranslation.find_by_key('site.index.intro')).to be_nil
+    visit root_path
+    expect(CopycatTranslation.find_by_key('site.index.intro')).not_to be_nil
+  end
+
+  it "shows the copycat_translation instead of the yaml" do
+    create(:copycat_translation, key: 'site.index.header', value: 'A different header')
+    visit root_path
+    expect(page).not_to have_content 'The Header'
+    expect(page).to have_content 'A different header'
+  end
+
+end
+
+feature "locales" do
+
+  it "displays different text based on users' locale" do
+    create(:copycat_translation, locale: 'en', key: 'site.index.intro', value: 'world')
+    create(:copycat_translation, locale: 'es', key: 'site.index.intro', value: 'mundo')
+
+    I18n.locale = :en
+    visit root_path
+    expect(page).to have_content 'world'
+    expect(page).not_to have_content 'mundo'
+
+    I18n.locale = :es
+    visit root_path
+    expect(page).to have_content 'mundo'
+    expect(page).not_to have_content 'world'
+
+    I18n.locale = :fa
+    visit root_path
+    expect(page).not_to have_content 'world'
+    expect(page).not_to have_content 'mundo'
+
+    I18n.locale = :en  # reset
+  end
+
+end
+
+feature "yaml" do
+
+  it "round-trips both translations correctly (and doesn't export nils)" do
+    visit root_path
+    expect(CopycatTranslation.find_by_key('site.index.intro').value).to be_nil
+    expect(CopycatTranslation.find_by_key('site.index.header').value).to eq('The Header')
+    expect(CopycatTranslation.count).to eq(2)
+
+    page.driver.browser.basic_authorize Copycat.username, Copycat.password
+    visit import_export_copycat_translations_path
+    click_link 'Download as YAML'
+    CopycatTranslation.destroy_all
+    expect(CopycatTranslation.count).to eq(0)
+    yaml = page.body
+    file = Tempfile.new 'copycat'
+    file.write yaml
+    file.close
+    visit import_export_copycat_translations_path
+    attach_file "file", file.path
+    click_button "Upload"
+    file.unlink
+
+    expect(CopycatTranslation.count).to eq(1)
+    expect(CopycatTranslation.find_by_key('site.index.intro')).to be_nil
+    expect(CopycatTranslation.find_by_key('site.index.header').value).to eq('The Header')
+  end
+
+end