weeler 0.0.4 → 1.0.0

data/spec/factories/dummy_posts.rb

@@ -0,0 +1,6 @@
+FactoryGirl.define do
+ factory :dummy_post, class: Post do
+    title "Foo bar"
+    body "Foo baar bazaar"
+  end
+end

data/spec/routing/routes_spec.rb

@@ -28,14 +28,14 @@ describe Weeler::Engine.routes do
     before do
       routes.draw do
         mount_weeler_at '/admin' do
-          weeler_resources :mini_posts, include_in_weeler_menu: true 
+          weeler_resources :mini_posts, include_in_weeler_menu: true
         end
       end
     end
 
     it "adds resource to weeler menu item" do
-      expect(Weeler.menu_items.size).to eq(1)
-      expect(Weeler.menu_items[0]).to eq(:mini_posts)
+      expect(Weeler.content_menu_items.size).to eq(1)
+      expect(Weeler.content_menu_items[0][:name]).to eq("Mini_posts")
     end
   end
 
@@ -137,4 +137,4 @@ describe Weeler::Engine.routes do
       end
     end
   end
-end
+end

data/spec/spec_helper.rb

@@ -1,5 +1,12 @@
+require 'simplecov'
 require 'coveralls'
-Coveralls.wear!
+
+# SimpleCov.formatters = [SimpleCov::Formatter::HTMLFormatter, Coveralls::SimpleCov::Formatter]
+
+SimpleCov.start do
+  add_filter '/spec/'
+  minimum_coverage(90)
+end
 
 ENV['RAILS_ENV'] ||= 'test'
 
@@ -24,12 +31,12 @@ RSpec.configure do |config|
   config.infer_base_class_for_anonymous_controllers = false
   config.order = "random"
 
-  config.color_enabled = true
+  # config.color_enabled = true
 
   config.add_formatter(:progress)
   config.add_formatter(:html, 'rspec.html')
 
-  # config.include Rails.application.routes.url_helpers
+  config.include Rails.application.routes.url_helpers
 
   # FactoryGirl
   config.include FactoryGirl::Syntax::Methods
@@ -44,6 +51,10 @@ RSpec.configure do |config|
     DatabaseCleaner.start
   end
 
+  config.before(:all) do
+    I18n.available_locales = [:en, :lv]
+  end
+
   config.after do
     # Timecop.return
     DatabaseCleaner.clean

data/spec/weeler/action_controller/acts/restful_spec.rb

@@ -0,0 +1,155 @@
+require 'spec_helper'
+
+describe Weeler::ActionController::Acts::Restful, :type => :controller do
+
+  before(:each) do
+    FactoryGirl.create_list(:dummy_post, 2)
+  end
+
+  before(:all) do
+    Dummy::Application.reload_routes!
+  end
+
+  describe "acts_as_restful", :type => :controller do
+
+    context "permited title with array" do
+      controller(Weeler::ContentController) do
+        acts_as_restful Post, permit_params: [:title]
+      end
+
+      describe "model" do
+        it "returns current controller model" do
+          expect(controller.model).to eq(Post)
+        end
+      end
+
+      describe "actions" do
+        describe "GET #index" do
+          it "returns success" do
+            get "index"
+            response.should be_success
+          end
+
+          it "assigns posts" do
+            get "index"
+            expect(assigns(:items).size).to eq(2)
+          end
+        end
+
+        describe "POST #create" do
+          it "redirects to edit path" do
+            post "create", post: attributes_for(:dummy_post)
+            post = Post.last
+            expect(response).to redirect_to("http://test.host/anonymous/#{post.id}/edit")
+          end
+
+          it "sets only permited params" do
+            post "create", post: attributes_for(:dummy_post, body: "Heila")
+            post = Post.last
+            expect(post.title).to eq("Foo bar")
+            expect(post.body).to eq(nil)
+          end
+        end
+
+        describe "GET #edit" do
+          it "returns success status" do
+            get "edit", id: Post.last.id
+            response.should be_success
+          end
+        end
+
+        describe "GET #new" do
+          it "returns success status" do
+            get "new"
+            response.should be_success
+          end
+        end
+
+        describe "PUT #update" do
+          it "sets only permited params" do
+            post = Post.last
+            put "update", id: post.id, post: {title: "Shivauva", body: "Another world!"}
+            post.reload
+            expect(post.title).to eq("Shivauva")
+            expect(post.body).to eq("Foo baar bazaar")
+          end
+        end
+
+        describe "DELETE #destroy" do
+          it "destroys item" do
+            post = Post.last
+            delete "destroy", id: post.id
+            expect { post.reload }.to raise_error(ActiveRecord::RecordNotFound)
+          end
+        end
+
+        specify "POST #order" do
+          routes.draw { post "order" => "anonymous#order" }
+
+          post "order", orders: "order[]=2&order[]=1"
+          post1 = Post.order(sequence: :asc).first
+          post2 = Post.order(sequence: :asc).last
+          expect(post1.sequence).to eq(0)
+          expect(post2.sequence).to eq(1)
+          expect(response.body).to eq("all ok")
+        end
+      end
+    end
+
+    context "no permited params" do
+      controller(Weeler::ContentController) do
+        acts_as_restful Post
+      end
+
+      describe "actions" do
+        describe "POST #create" do
+          it "sets only permited params" do
+            post "create", post: attributes_for(:dummy_post, title: "Foo lala", body: "Heila")
+            post = Post.last
+            expect(post.title).to eq(nil)
+            expect(post.body).to eq(nil)
+          end
+
+          it "warns developer" do
+            controller.should_receive(:warn).with("[UNPERMITED PARAMS] To permiting {\"title\"=>\"Foo lala\", \"body\"=>\"Heila\"} params, add 'permit_params: [:title, :body]' option to 'acts_as_restful'")
+            post "create", post: attributes_for(:dummy_post, title: "Foo lala", body: "Heila")
+          end
+        end
+      end
+    end
+
+    context "permited all with block" do
+      controller(Weeler::ContentController) do
+        acts_as_restful Post, permit_params: -> (params) { params.require(:post).permit! }
+      end
+
+      describe "actions" do
+        describe "POST #create" do
+          it "sets only permited params" do
+            post "create", post: attributes_for(:dummy_post, title: "Foo lala", body: "Heila")
+            post = Post.last
+            expect(post.title).to eq("Foo lala")
+            expect(post.body).to eq("Heila")
+          end
+        end
+      end
+
+    end
+
+    context "permited all with block" do
+      controller(Weeler::ContentController) do
+        acts_as_restful Post, order_by: {id: :desc}, permit_params: [:title]
+      end
+
+      describe "GET #index" do
+        it "users order by for ordering" do
+          get "index"
+          expect(assigns(:items).first.id).to eq(2)
+          expect(assigns(:items).last.id).to eq(1)
+        end
+      end
+    end
+
+
+  end
+end

data/spec/weeler/action_view/helpers/form_helper_spec.rb

@@ -0,0 +1,26 @@
+require 'spec_helper'
+
+describe Weeler::ActionView::Helpers::FormHelper, :type => :helper do
+
+  let(:resource)  { FactoryGirl.build :dummy_post }
+  let(:helper)    { ActionView::Helpers::FormBuilder.new(:post, resource, self, {})}
+
+  describe :image_upload_field do
+    let(:output)    {
+      helper.image_upload_field :image
+    }
+
+    it 'create file field' do
+      expect(output).to include '<input class="form-control" id="post_image" name="post[image]" type="file" />'
+    end
+
+    it 'creates a label' do
+      expect(output).to include '<label'
+    end
+
+    it "creates img preview tag" do
+      expect(output).to include '<img alt="Original" src="/images/original.jpg" style="height: 80px;" />'
+    end
+  end
+
+end

data/spec/weeler/action_view/helpers/translation_helper_spec.rb

@@ -0,0 +1,16 @@
+require 'spec_helper'
+
+describe Weeler::ActionView::Helpers::TranslationHelper, :type => :helper do
+
+  describe :translate do
+
+    let(:output)    {
+      helper.translate "not.existing.key"
+    }
+
+    it 'dont renders key' do
+      expect(output).to eq('<span class="translation_missing" title="translation missing: en.not.existing.key">Key</span>')
+    end
+  end
+
+end

data/spec/weeler/i18n/backend/weeler/dedupe_spec.rb

@@ -0,0 +1,21 @@
+require "spec_helper"
+
+describe I18n::Backend::Weeler::Dedupe do
+
+  describe "#dedupe" do
+
+    context "empty db" do
+      before(:each) do
+        3.times { I18n::Backend::Weeler::Translation.create(locale: :en, key: "weeler.dublicate") }
+        I18n::Backend::Weeler::Translation.dedupe
+      end
+
+      it "translation stores all translation in each locale" do
+        expect(I18n::Backend::Weeler::Translation.locale(:en).where(key: "weeler.dublicate").size).to eq(1)
+      end
+
+    end
+
+  end
+
+end

data/spec/weeler/i18n/backend/weeler/exporter_spec.rb

@@ -44,10 +44,8 @@ describe I18n::Backend::Weeler::Exporter do
         I18n.backend.store_translations(:en, foo: {two: nil })
         I18n.backend.store_translations(:lv, foo: {two: "some"})
 
-        translations = I18n::Backend::Weeler::Translation.where(key: "foo.one").load
-        translations.size
+        translations = I18n::Backend::Weeler::Translation.where(key: "foo.one")
         @other_rows = translations.as_xlsx_package.workbook.worksheets[0].rows
-       
       end
 
       it "should have 2 rows" do
@@ -69,4 +67,4 @@ describe I18n::Backend::Weeler::Exporter do
 
   end
 
-end
+end

data/spec/weeler/i18n/backend/weeler/importer_spec.rb

@@ -3,7 +3,7 @@ require "spec_helper"
 describe I18n::Backend::Weeler::Importer do
 
   # This is file with 10 translations and 3 locales.
-  let(:file_path) {File.dirname(__FILE__) + '/../../../../fixtures/test.xlsx'}
+  let(:file_path) { Rack::Test::UploadedFile.new(File.dirname(__FILE__) + '/../../../../fixtures/test.xlsx') }
 
   describe :import do
 
@@ -31,12 +31,15 @@ describe I18n::Backend::Weeler::Importer do
     context "full db" do
       before(:each) do
         I18n::Backend::Weeler::Translation.delete_all
+        I18n.backend.backends[0].reload_cache
         I18n.backend.store_translations(:en, welcome: {title: 'fooo'})
       end
 
       it "file import overrides value" do
         expect(I18n.t(:"welcome.title", :locale => :en)).to eq("fooo")
         I18n::Backend::Weeler::Translation.import file_path
+        Settings.i18n_updated_at = Time.now
+
         expect(I18n.t(:"welcome.title", :locale => :en)).to eq("EN welcome")
       end
 
@@ -44,4 +47,4 @@ describe I18n::Backend::Weeler::Importer do
 
   end
 
-end
+end

data/spec/weeler/i18n/backend/weeler_spec.rb

@@ -13,17 +13,38 @@ describe I18n::Backend::Weeler do
 
   end
 
-  describe "#store_translations" do
+  before do
+    # isolate each test cases
+    Weeler.i18n_cache.clear
+  end
 
-    it "finds one locale" do
-      expect(I18n.backend.available_locales.count).to be(1) 
+  describe "#reload_cache" do
+    it "clears translations cache" do
+      Weeler.i18n_cache.should_receive(:clear)
+      I18n.backend.backends[0].reload_cache
     end
 
-    it "finds all existing locales if theres is translations in other locale" do
-      translation = I18n::Backend::Weeler::Translation.new(:key => 'foo', :value => 'bar', :locale => :lv)
-      translation.interpolations = %w(count name)
-      translation.save
-      expect(I18n.backend.available_locales.count).to be(2)
+    it "writes last translations update timestamp to cache" do
+      Settings.i18n_updated_at = Time.now
+      I18n.backend.backends[0].reload_cache
+      expect(Weeler.i18n_cache.read('UPDATED_AT')).to eq(Settings.i18n_updated_at)
+    end
+
+    it "loads all translated data to cache" do
+      I18n::Backend::Weeler::Translation.create(:key => 'weeler.test.value', :value => "testējam", :locale => :lv)
+      I18n::Backend::Weeler::Translation.create(:key => 'weeler.test.value', :value => "testing", :locale => :en)
+
+      I18n.backend.backends[0].reload_cache
+
+      expect(Weeler.i18n_cache.read(["lv", "weeler.test.value"]).value).to eq("testējam")
+      expect(Weeler.i18n_cache.read(["en", "weeler.test.value"]).value).to eq("testing")
+    end
+  end
+
+  describe "#available_locales" do
+
+    it "finds one locale" do
+      expect(I18n.available_locales.count).to be(2)
     end
 
   end
@@ -46,11 +67,31 @@ describe I18n::Backend::Weeler do
 
   end
 
-  describe "#lookup" do
-    it "show warning" do
-      I18n::Backend::Weeler::Translation.should_receive(:warn).with("[DEPRECATION] Giving a separator to Translation.lookup is deprecated. You can change the internal separator by overwriting FLATTEN_SEPARATOR.")
-      I18n::Backend::Weeler::Translation.lookup("foo", "|")
+  describe "#html?" do
+    before(:all) do
+      @html_key_translation = I18n::Backend::Weeler::Translation.create(:key => 'methods.body_html', :value => "Super duper", :locale => :en)
+      @html_value_translation = I18n::Backend::Weeler::Translation.create(:key => 'methods.body', :value => "Super <b>duper</b>", :locale => :en)
+      @html_fake_value_translation = I18n::Backend::Weeler::Translation.create(:key => 'methods.body_fake', :value => "Super < b", :locale => :en)
+    end
+
+    it "is true if key is html" do
+      expect(@html_key_translation.html?).to be(true)
     end
+
+    it "is true if value contains html" do
+      expect(@html_value_translation.html?).to be(true)
+    end
+
+    it "is false if value not contains html" do
+
+      expect(@html_fake_value_translation.html?).to be(false)
+    end
+
+  end
+
+  describe "#lookup" do
+
+
   end
 
   describe "groups" do
@@ -75,14 +116,17 @@ describe I18n::Backend::Weeler do
   describe "#store_translations" do
     it "store_translations does not allow ambiguous keys (1)" do
       I18n::Backend::Weeler::Translation.delete_all
-      I18n.backend.store_translations(:en, :foo => 'foo')
-      I18n.backend.store_translations(:en, :foo => { :bar => 'bar' })
-      I18n.backend.store_translations(:en, :foo => { :baz => 'baz' })
+
+      I18n.backend.store_translations(:en, foo: 'foo')
+      I18n.backend.store_translations(:en, foo: { bar: 'bar' })
+      I18n.backend.store_translations(:en, foo: { baz: 'baz' })
+
+      I18n.backend.backends[0].reload_cache
 
       translations = I18n::Backend::Weeler::Translation.locale(:en).lookup('foo')
       expect(translations.map(&:value)).to eq(%w(bar baz))
 
-      expect(I18n.t(:foo)).to eq({ :bar => 'bar', :baz => 'baz' })
+      expect(I18n.t(:foo)).to eq("Foo") # no translation
     end
 
     it "store_translations does not allow ambiguous keys (2)" do
@@ -92,29 +136,81 @@ describe I18n::Backend::Weeler do
       I18n.backend.store_translations(:en, :foo => 'foo')
 
       translations = I18n::Backend::Weeler::Translation.locale(:en).lookup('foo')
-      expect(translations.map(&:value)).to eq(%w(foo)) 
+      expect(translations.map(&:value)).to eq(%w(foo))
 
-      expect(I18n.t(:foo)).to eq('foo') 
+      expect(I18n.t(:foo)).to eq('foo')
     end
 
     it "can store translations with keys that are translations containing special chars" do
-      I18n.backend.store_translations(:es, :"Pagina's" => "Pagina's" )
-      expect(I18n.t(:"Pagina's", :locale => :es)).to eq("Pagina's")
+      I18n.backend.store_translations(:en, :"Pagina's" => "Pagina's" )
+      expect(I18n.t(:"Pagina's", :locale => :en)).to eq("Pagina's")
     end
   end
 
   describe "#lookup" do
 
+    it "show warning" do
+      I18n::Backend::Weeler::Translation.should_receive(:warn).with("[DEPRECATION] Giving a separator to Translation.lookup is deprecated. You can change the internal separator by overwriting FLATTEN_SEPARATOR.")
+      I18n::Backend::Weeler::Translation.lookup("foo", "|")
+    end
+
+    context "cache" do
+      context "differs from settings timestamp" do
+        before do
+          Settings.i18n_updated_at = Time.now
+        end
+
+        it "reloads cache" do
+          I18n.backend.backends[0].should_receive(:reload_cache)
+          I18n.t("cancel", scope: "admin.content")
+        end
+      end
+
+      context "is same as updates timestamp" do
+        before do
+          Weeler.i18n_cache.write('UPDATED_AT', Settings.i18n_updated_at)
+        end
+
+        it "does not reload cache" do
+          I18n.backend.backends[0].should_not_receive(:reload_cache)
+          I18n.t("cancel", scope: "admin.content")
+        end
+      end
+    end
+
     it "returns translation" do
       FactoryGirl.create(:translation)
+      I18n.backend.backends[0].reload_cache
       expect(I18n.t("title")).to eq("This is weeler")
     end
 
     context "missing translations" do
 
-      it "persists the key" do        
+      context "exist in yml" do
+
+        context "new translation" do
+
+          it "persist it" do
+            I18n.t('hello')
+            expect(I18n::Backend::Weeler::Translation.locale(:en).find_by_key('hello').value).to eq(I18n.t('hello'))
+          end
+
+        end
+
+        context "already stored" do
+          before(:all) do
+            FactoryGirl.create(:translation, key: "weeler.test.cms_title", locale: "en", value: nil)
+          end
+
+          it "saves the fallback backend value" do
+            I18n.t('weeler.test.cms_title')
+            expect(I18n::Backend::Weeler::Translation.locale(:en).find_by_key('weeler.test.cms_title').value).to eq("Weeler is cool")
+          end
+        end
+      end
+
+      it "persists the key" do
         I18n.t('hello')
-        
         expect(I18n::Backend::Weeler::Translation.count).to eq(1)
         expect(I18n::Backend::Weeler::Translation.locale(:en).find_by_key('hello')).to_not be(nil)
       end
@@ -146,6 +242,7 @@ describe I18n::Backend::Weeler do
       it "creates a stub when a custom separator is used" do
         I18n.t('foo|baz', :separator => '|')
         I18n::Backend::Weeler::Translation.locale(:en).lookup("foo.baz").first.update_attributes!(:value => 'baz!')
+        I18n.backend.backends[0].reload_cache
         expect(I18n.t('foo|baz', :separator => '|')).to eq('baz!')
       end
 
@@ -159,6 +256,7 @@ describe I18n::Backend::Weeler do
         key = 'foo|baz.zab'
         I18n.t(key, :separator => '|')
         I18n::Backend::Weeler::Translation.locale(:en).lookup("foo.baz\001zab").first.update_attributes!(:value => 'baz!')
+        I18n.backend.backends[0].reload_cache
         expect(I18n.t(key, :separator => '|')).to eq('baz!')
       end
 
@@ -166,4 +264,4 @@ describe I18n::Backend::Weeler do
 
   end
 
-end
+end