copycat 0.0.4 → 0.0.5

data/spec/dummy/tmp/pids/server.pid

@@ -0,0 +1 @@
+81725

data/spec/factories/copycat_translations.rb

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

data/spec/integration/copycat_spec.rb

@@ -2,41 +2,198 @@
 
 require 'spec_helper'
 
-feature "use #t" do
 
-  it "the dummy app has a translation for site.index.header but not site.index.intro" do
-    I18n.t('site.index.header').should == 'The Header'
-    I18n.t('site.index.intro').should == "translation missing: en.site.index.intro"
+feature "copycat index" do
+ 
+  before do
+    Factory(:copycat_translation, :key => "foo", :value => "bar")
+    page.driver.browser.basic_authorize COPYCAT_USERNAME, COPYCAT_PASSWORD
+    visit copycat_translations_path
   end
 
-  it "uses i18n.t" do
-    visit root_path
-    page.should have_content 'The Header'
-    page.should have_content 'Intro' #ActionView::Helpers::TranslationHelper#translate wrapper 
+  it "has a nav bar" do
+    click_link 'Import / Export'
+    click_link 'Help'
+    click_link 'Search'
   end
 
-  it "creates a copycat_translation if the yaml has an entry" do
-    CopycatTranslation.find_by_key('site.index.header').should be_nil
-    visit root_path
-    CopycatTranslation.find_by_key('site.index.header').should_not be_nil
+  it "doesn't show any tokens by default" do
+    page.should_not have_content 'foo'
+    page.should_not have_content 'bar'
   end
 
-  it "creates a copycat_translation if the yaml does not have an entry" do
-    CopycatTranslation.find_by_key('site.index.intro').should be_nil
-    visit root_path
-    CopycatTranslation.find_by_key('site.index.intro').should_not be_nil
+  it "allows search by key" do
+    fill_in 'search', :with => 'foo'
+    click_button 'Search'
+    page.should have_content 'foo'
+    page.should have_content 'bar'
   end
 
-  it "shows the copycat_translation instead of the yaml" do
-    FactoryGirl.create(:copycat_translation, key: 'site.index.header', value: 'A different header')
-    visit root_path
-    page.should_not have_content 'The Header'
-    page.should have_content 'A different header'
+  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'
   end
 
-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'
+  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'
+  end
+  
+  it "searches in the middles of strings" do
+    Factory(:copycat_translation, :key => "site.index.something")
+    fill_in 'search', :with => 'index'
+    click_button 'Search'
+    page.should have_content 'site.index.something'
+  end
+
+  it "can show all" do
+    Factory(:copycat_translation, :key => "foe", :value => "beer")
+    click_button 'Search'
+    page.should have_content 'foo'
+    page.should have_content 'foe'
+  end
+
+  context "more than one locale" do
+
+    before do
+      CopycatTranslation.destroy_all
+      Factory(:copycat_translation, key: "foo", value: "bar1", locale: "en")
+      Factory(:copycat_translation, key: "foo", value: "bar2", locale: "fa")
+      Factory(:copycat_translation, key: "foo", value: "bar3", locale: "it")
+    end
+
+    #locale
+    # 1. not on URL at all
+    #   - set to default_locale
+    # 2. present but blank
+    # 3. present with value
+
+    #search
+    # 1. not on URL at all
+    #  - show nothing
+    # 2. present but blank
+    #  - show everything
+    #    - L1 got set to default locale
+    #    - L2 show for all locales
+    #    - L3 scope to one locale
+    # 3. present with value
+    #  - show matching
+    #    - L1 got set to default locale
+    #    - L2 show for all locales
+    #    - L3 scope to one locale
+
+    it "nil locale, nil search" do
+      visit copycat_translations_path
+      page.should_not have_content 'foo'
+    end
+
+    it "nil locale, blank search" do
+      # impossible for user to replicate this case
+      visit "/copycat_translations?search=&commit=Search"
+      page.should have_content 'bar1'
+      page.should_not have_content 'bar2'
+      page.should_not have_content 'bar3'
+    end
+
+    it "nil locale, present search" do
+      # impossible for user to replicate this case
+      visit "/copycat_translations?search=foo&commit=Search"
+      page.should have_content 'bar1'
+      page.should_not have_content 'bar2'
+      page.should_not have_content 'bar3'
+      visit "/copycat_translations?search=fuu&commit=Search"
+      page.should_not have_content 'foo'
+    end
+
+    it "blank locale, nil search" do
+      # impossible for user to replicate this case
+      visit "/copycat_translations?locale=&commit=Search"
+      page.should_not have_content 'foo'
+    end
 
-require 'tempfile'
+    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'
+    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'
+      fill_in 'search', :with => 'fuu'
+      click_button 'Search'
+      page.should_not have_content 'foo'
+    end
+
+    it "present locale, nil search" do
+      # impossible for user to replicate this case
+      visit "/copycat_translations?locale=en&commit=Search"
+      page.should_not 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'
+      select 'fa', :from => 'locale'
+      click_button 'Search'
+      page.should_not have_content 'bar1'
+      page.should have_content 'bar2'
+      page.should_not 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'
+    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'
+      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'
+      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'
+      select 'en', :from => 'locale'
+      fill_in 'search', :with => 'fuu'
+      click_button 'Search'
+      page.should_not have_content 'foo'
+    end
+
+  end
+
+end
 
 feature "downloading and uploading yaml files" do
   before do
@@ -51,7 +208,8 @@ feature "downloading and uploading yaml files" do
     Factory(:copycat_translation, :key => 2, :value => "bar5")
     assert CopycatTranslation.count == 5
 
-    visit "/copycat_translations.yaml"
+    visit import_export_copycat_translations_path
+    click_link 'Download as YAML'
     CopycatTranslation.destroy_all
     assert CopycatTranslation.count == 0
 
@@ -60,7 +218,7 @@ feature "downloading and uploading yaml files" do
     file.write yaml
     file.close
 
-    visit upload_copycat_translations_path
+    visit import_export_copycat_translations_path
     attach_file "file", file.path
     click_button "Upload"
     file.unlink
@@ -74,18 +232,23 @@ feature "downloading and uploading yaml files" do
   end
 
   it "round-trips the yaml with complicated text" do
-    Factory(:copycat_translation, :key => "a.foo", :value => "“hello world“ üokåa®fgsdf;::fs;kdf")
-    visit "/copycat_translations.yaml"
+    value = "“hello world“ üokåa®fgsdf;::fs;kdf"
+    Factory(:copycat_translation, :key => "a.foo", :value => value)
+
+    visit import_export_copycat_translations_path
+    click_link 'Download as YAML'
     CopycatTranslation.destroy_all
+
     yaml = page.text
     file = Tempfile.new 'copycat'
     file.write yaml
     file.close
-    visit upload_copycat_translations_path
+
+    visit import_export_copycat_translations_path
     attach_file "file", file.path
     click_button "Upload"
     file.unlink
-    assert CopycatTranslation.find_by_key("a.foo").value == "“hello world“ üokåa®fgsdf;::fs;kdf"
+    assert CopycatTranslation.find_by_key("a.foo").value == value
   end
 
   it "gives 400 on bad upload" do
@@ -93,7 +256,7 @@ feature "downloading and uploading yaml files" do
     file.write "<<<%%#$W%s"
     file.close
 
-    visit upload_copycat_translations_path
+    visit import_export_copycat_translations_path
     attach_file "file", file.path
     click_button "Upload"
     file.unlink
@@ -105,27 +268,9 @@ feature "downloading and uploading yaml files" do
 end
 
 feature "locales" do
+  before 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
+    page.driver.browser.basic_authorize COPYCAT_USERNAME, COPYCAT_PASSWORD
   end
 
   it "imports yaml containing multiple locales" do
@@ -138,8 +283,7 @@ feature "locales" do
     YAML
     file.close
 
-    page.driver.browser.basic_authorize COPYCAT_USERNAME, COPYCAT_PASSWORD
-    visit upload_copycat_translations_path
+    visit import_export_copycat_translations_path
     attach_file "file", file.path
     click_button "Upload"
     file.unlink
@@ -157,8 +301,7 @@ feature "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"
+    visit download_copycat_translations_path
     yaml = page.text
     assert yaml =~ /en:\s*hello: world/
     assert yaml =~ /es:\s*hello: mundo/  

data/spec/integration/dummy_spec.rb

@@ -0,0 +1,63 @@
+#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
+    I18n.t('site.index.header').should == 'The Header'
+    I18n.t('site.index.intro').should == "translation missing: en.site.index.intro"
+  end
+
+  it "uses i18n.t" do
+    visit root_path
+    page.should have_content 'The Header'
+    page.should have_content 'Intro' #ActionView::Helpers::TranslationHelper#translate wrapper 
+  end
+
+  it "creates a copycat_translation if the yaml has an entry" do
+    CopycatTranslation.find_by_key('site.index.header').should be_nil
+    visit root_path
+    CopycatTranslation.find_by_key('site.index.header').should_not be_nil
+  end
+
+  it "creates a copycat_translation if the yaml does not have an entry" do
+    CopycatTranslation.find_by_key('site.index.intro').should be_nil
+    visit root_path
+    CopycatTranslation.find_by_key('site.index.intro').should_not be_nil
+  end
+
+  it "shows the copycat_translation instead of the yaml" do
+    FactoryGirl.create(:copycat_translation, key: 'site.index.header', value: 'A different header')
+    visit root_path
+    page.should_not have_content 'The Header'
+    page.should have_content 'A different header'
+  end
+
+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
+
+end

data/spec/spec_helper.rb

@@ -2,6 +2,7 @@ ENV["RAILS_ENV"] = 'test'
 require File.expand_path("../dummy/config/environment.rb", __FILE__)
 require 'rspec/rails'
 require 'capybara/rspec'
+require 'tempfile'
 
 Dir["#{File.dirname(__FILE__)}/factories/**/*.rb"].each { |f| require f }
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: copycat
 version: !ruby/object:Gem::Version
-  version: 0.0.4
+  version: 0.0.5
   prerelease: 
 platform: ruby
 authors:
@@ -10,11 +10,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-03-20 00:00:00.000000000 Z
+date: 2012-03-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
-  requirement: &2151799480 !ruby/object:Gem::Requirement
+  requirement: &2151799300 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -22,7 +22,7 @@ dependencies:
         version: 3.0.0
   type: :runtime
   prerelease: false
-  version_requirements: *2151799480
+  version_requirements: *2151799300
 description: Edit live website copy.
 email:
 - info@vermonster.com
@@ -33,11 +33,10 @@ files:
 - app/assets/stylesheets/copycat_engine.css
 - app/controllers/copycat_translations_controller.rb
 - app/models/copycat_translation.rb
-- app/views/copycat_translations/_copycat_header.html.erb
 - app/views/copycat_translations/edit.html.erb
+- app/views/copycat_translations/help.html.erb
+- app/views/copycat_translations/import_export.html.erb
 - app/views/copycat_translations/index.html.erb
-- app/views/copycat_translations/readme.html.erb
-- app/views/copycat_translations/upload.html.erb
 - app/views/layouts/copycat.html.erb
 - config/routes.rb
 - db/migrate/20120313191745_create_copycat_translations.rb
@@ -73,7 +72,7 @@ files:
 - spec/dummy/config/routes.rb
 - spec/dummy/config.ru
 - spec/dummy/db/development.sqlite3
-- spec/dummy/db/migrate/20120320143433_create_copycat_translations.copycat_engine.rb
+- spec/dummy/db/migrate/20120320194234_create_copycat_translations.copycat_engine.rb
 - spec/dummy/db/schema.rb
 - spec/dummy/db/test.sqlite3
 - spec/dummy/log/development.log
@@ -84,16 +83,21 @@ files:
 - spec/dummy/public/favicon.ico
 - spec/dummy/Rakefile
 - spec/dummy/script/rails
+- spec/dummy/tmp/cache/assets/C19/060/sprockets%2F125436f53217b7f564aa5016d1168709
+- spec/dummy/tmp/cache/assets/CB3/6E0/sprockets%2F0f4c96b04436bf7ae900561753947d4c
 - spec/dummy/tmp/cache/assets/CD8/370/sprockets%2F357970feca3ac29060c1e3861e2c0953
 - spec/dummy/tmp/cache/assets/D09/190/sprockets%2Fb2bddd007999a56d8b20c92301ea1983
 - spec/dummy/tmp/cache/assets/D1A/DB0/sprockets%2F26d700197b8d25f954d56d8f31f2c8bc
 - spec/dummy/tmp/cache/assets/D32/A10/sprockets%2F13fe41fee1fe35b49d145bcc06610705
 - spec/dummy/tmp/cache/assets/D5A/EA0/sprockets%2Fd771ace226fc8215a3572e0aa35bb0d6
+- spec/dummy/tmp/cache/assets/DA4/EE0/sprockets%2Fcd39e3d56788faea331e0f5bac67329b
 - spec/dummy/tmp/cache/assets/DBD/220/sprockets%2F0cbff5c1a349439c1c444cbb4a57acc4
 - spec/dummy/tmp/cache/assets/DDD/780/sprockets%2Fe7880b84c3ecb507fabc99b9ef6b70e3
 - spec/dummy/tmp/cache/assets/DF8/310/sprockets%2Fccb12c4009c2f876d51dee8bae4c64ec
+- spec/dummy/tmp/pids/server.pid
 - spec/factories/copycat_translations.rb
 - spec/integration/copycat_spec.rb
+- spec/integration/dummy_spec.rb
 - spec/lib/copycat_spec.rb
 - spec/models/copycat_translation_spec.rb
 - spec/spec_helper.rb
@@ -147,7 +151,7 @@ test_files:
 - spec/dummy/config/routes.rb
 - spec/dummy/config.ru
 - spec/dummy/db/development.sqlite3
-- spec/dummy/db/migrate/20120320143433_create_copycat_translations.copycat_engine.rb
+- spec/dummy/db/migrate/20120320194234_create_copycat_translations.copycat_engine.rb
 - spec/dummy/db/schema.rb
 - spec/dummy/db/test.sqlite3
 - spec/dummy/log/development.log
@@ -158,16 +162,21 @@ test_files:
 - spec/dummy/public/favicon.ico
 - spec/dummy/Rakefile
 - spec/dummy/script/rails
+- spec/dummy/tmp/cache/assets/C19/060/sprockets%2F125436f53217b7f564aa5016d1168709
+- spec/dummy/tmp/cache/assets/CB3/6E0/sprockets%2F0f4c96b04436bf7ae900561753947d4c
 - spec/dummy/tmp/cache/assets/CD8/370/sprockets%2F357970feca3ac29060c1e3861e2c0953
 - spec/dummy/tmp/cache/assets/D09/190/sprockets%2Fb2bddd007999a56d8b20c92301ea1983
 - spec/dummy/tmp/cache/assets/D1A/DB0/sprockets%2F26d700197b8d25f954d56d8f31f2c8bc
 - spec/dummy/tmp/cache/assets/D32/A10/sprockets%2F13fe41fee1fe35b49d145bcc06610705
 - spec/dummy/tmp/cache/assets/D5A/EA0/sprockets%2Fd771ace226fc8215a3572e0aa35bb0d6
+- spec/dummy/tmp/cache/assets/DA4/EE0/sprockets%2Fcd39e3d56788faea331e0f5bac67329b
 - spec/dummy/tmp/cache/assets/DBD/220/sprockets%2F0cbff5c1a349439c1c444cbb4a57acc4
 - spec/dummy/tmp/cache/assets/DDD/780/sprockets%2Fe7880b84c3ecb507fabc99b9ef6b70e3
 - spec/dummy/tmp/cache/assets/DF8/310/sprockets%2Fccb12c4009c2f876d51dee8bae4c64ec
+- spec/dummy/tmp/pids/server.pid
 - spec/factories/copycat_translations.rb
 - spec/integration/copycat_spec.rb
+- spec/integration/dummy_spec.rb
 - spec/lib/copycat_spec.rb
 - spec/models/copycat_translation_spec.rb
 - spec/spec_helper.rb

data/app/views/copycat_translations/_copycat_header.html.erb

@@ -1,9 +0,0 @@
-<div class="copycat-header">
-<ul>
-  <li><a href=<%=copycat_translations_path%>><h2>Copycat</h2></a></li>
-  <% href = copycat_translations_path + ".yaml" %>
-  <li><a href=<%=href%>>Download</a></li>
-  <li><a href=<%= upload_copycat_translations_path %>>Upload</a></li>
-  <li><a href=<%= readme_copycat_translations_path %>>Readme</a></li>
-</ul>
-</div>

data/app/views/copycat_translations/readme.html.erb

@@ -1,27 +0,0 @@
-<div id="readme">
-
-  <div class="section">
-    <h2> Copycat </h2>
-    <p>Copycat is a Ruby on Rails engine that allows you to edit live website copy.</p>
-  </div>
-
-  <div class="section">
-    <h2> How to use </h2>
-    <p>The values you see here correspond to copy text in different parts of the parent website. When you edit them, the text on the website will change. Use ctrl/command-f to search for text, then click the left hand side links to edit. </p> 
-
-    <p>Values with keys ending in "_html" will be parsed as literal html, so you can insert tags. All other values are parsed as plain text.</p>
-  </div>
-
-  <div class="section">
-    <h2> Importing and Exporting </h2>
-    <p> Copycat text is data--not code. By default, Copycat values are stored in your server's database, and not the codebase that powers your website. Therefore, you may have to download/upload Copycat data to transfer your copy edits between servers, or to safeguard them between database wipes. Copycat exports (and is able to re-import) all of its data in a format called "YAML."</p> 
-    <h4> Downloading </h4>
-    <p>You can get a YAML file of all your copy by clicking "Download" on the main Copycat page.</p>
-    <h4> Uploading </h4>
-    <p>You can upload copy to Copycat by clicking "Upload" on the main page and uploading a properly formatted YAML file. Uploading copy won't delete any entries from Copycat, but it may update them or create new ones. Use this feature to mass-update your copy from between servers -- e.g., make changes to copy on a test server, and then when you're satisfied, download the YAML file and upload it to your production site. Downloading your copy as YAML and then immediately re-uploading it will leave your copy unchanged.</p>
-    <h4> Commiting copy to your codebase </h4>
-    <p>If you're pretty happy with your copy and don't expect to need to update it live anymore, or just want a bit more security, you can download your copy as YAML and include it as part of the Ruby on Rails application underneath your main website. You can then remove Copycat from the application entirely, and the website will still run exactly as before--it interfaces with Ruby on Rails's built-in translation mechanisms in a very unobtrusive way.
-  </div>
-
-</div>
-