kms 0.9.0 → 1.0.0

data/config/routes.rb

@@ -5,19 +5,20 @@ Kms::Engine.routes.draw do
     module: :devise,
     controllers: { registrations: "users/registrations" }
   constraints(format: "json") do
-    resources :templates, format: true
+    resources :templates, except: [:new, :edit], format: true
     resources :snippets, except: [:new, :edit], format: true
-    resources :pages, format: true do
+    resources :pages, except: [:new, :edit], format: true do
       collection do
         post 'sorting'
       end
     end
-    resources :assets, format: true, as: :kms_assets
-    resources :users, only: [:index, :destroy], format: true
+    resources :assets, except: [:new, :edit], format: true, as: :kms_assets
+    resources :users, only: [:index, :destroy], format: true do
+      post :from_kms, on: :collection, action: :create
+      get :kms_user, on: :collection
+    end
     resources :resources, only: :index, format: true
     resource :settings, only: [:show, :update], format: true
-    get '/users/kms_user' => "users#kms_user", format: true
-    post "/users/from_kms" => "users#create", format: true
   end
   post "/assets/ckeditor" => "assets#ckeditor"
   constraints(format: "html") do

data/lib/kms/dependencies.rb

@@ -1,9 +1,7 @@
-require 'bower-rails'
 require "slim-rails"
-require "less-rails"
 require 'russian'
 require 'angular-rails-templates'
-require 'liquor'#, github: 'webgradus/liquor'
+require 'liquor'
 require 'fog/aws'
 require 'carrierwave'
 require 'carrierwave/orm/activerecord'
@@ -12,7 +10,6 @@ require 'active_model_serializers'
 require 'ancestry'
 require 'devise'
 require 'devise/orm/active_record'
-#require 'evil-front'
 require 'cancancan'
 require 'dragonfly'
 require 'textacular'

data/lib/kms/engine.rb

@@ -27,10 +27,9 @@ module Kms
     end
 
     initializer "kms.assets" do |app|
-      app.config.less.paths << File.join(Rails.root,'vendor', 'assets', 'bower_components')
       app.config.assets.paths << Rails.root.join('vendor', 'assets', 'bower_components')
       app.config.assets.precompile += %w(kms/application.js kms/application.css)
-      app.config.assets.precompile += %w( Responsive-Dashboard/src/img/** )
+      app.config.assets.precompile += %w( avatar.jpg )
       app.config.assets.precompile += %w( **/*.svg **/*.eot **/*.woff **/*.ttf )
       app.config.assets.precompile += %w( ng-ckeditor/libs/ckeditor/** )
     end

data/lib/kms/search_item.rb

@@ -1,8 +1,13 @@
 module Kms
   class SearchItem
+    extend ActionView::Helpers::TextHelper
     include Liquor::Dropable
     include Liquor::External
+
     attr_reader :title, :content, :link
+
+    HIGHLIGHTER = '<mark class="highlight">\1</mark>'.freeze
+
     def initialize opts
       @title = opts[:title]
       @link = opts[:link]
@@ -24,5 +29,18 @@ module Kms
     def content
       @content
     end
+
+    # attributes_map for Kms::Page is:
+    # {title: :title, link: :fullpath, content: :content}
+    def self.item_for_searchable(searchable, keywords, attributes_map)
+      sanitized_title = ActionController::Base.helpers.sanitize(searchable[attributes_map[:title]])
+      sanitized_content = ActionController::Base.helpers.sanitize(searchable[attributes_map[:content]])
+      Kms::SearchItem.new(
+        title: highlight(sanitized_title, keywords, highlighter: HIGHLIGHTER, sanitize: false),
+        link: searchable[attributes_map[:link]],
+        content: highlight(sanitized_content, keywords, highlighter: HIGHLIGHTER, sanitize: false)
+      )
+    end
+
   end
 end

data/lib/kms/version.rb

@@ -1,3 +1,3 @@
 module Kms
-  VERSION = "0.9.0"
+  VERSION = "1.0.0"
 end

data/spec/controllers/kms/assets_controller_spec.rb

@@ -0,0 +1,51 @@
+require 'spec_helper'
+
+describe Kms::AssetsController, type: :controller do
+  login_user
+  describe 'GET index' do
+    it 'returns assets json representation' do
+      FactoryGirl.create(:asset)
+      get :index, format: :json
+
+      expect(response).to be_success
+      expect(json.length).to eq(1)
+    end
+  end
+
+  describe 'POST create' do
+    it 'returns 200 status and creates asset' do
+      asset_params = FactoryGirl.attributes_for(:asset)
+      post :create, params: { asset: asset_params }, format: :json
+      expect(response).to be_success
+      expect(Kms::Asset.count).to eq(1)
+    end
+  end
+
+  describe 'PUT update' do
+    it 'returns 200 status and updates asset' do
+      css_asset = FactoryGirl.create(:css_asset)
+      put :update, params: { id: css_asset.id, asset: {performing_plain_text: true, text: 'h1 {}'} }, format: :json
+      expect(response).to be_success
+      expect(Kms::Asset.find(css_asset.id).text).to be_eql('h1 {}')
+    end
+  end
+
+  describe 'GET show' do
+    it 'returns asset json representation' do
+      css_asset = FactoryGirl.create(:css_asset)
+      get :index, params: { id: css_asset.id }, format: :json
+
+      expect(response).to be_success
+      expect(json[0]['text']).to eq(css_asset.text)
+    end
+  end
+
+  describe 'DELETE destroy' do
+    it "deletes asset" do
+      asset = FactoryGirl.create(:asset)
+      delete :destroy, params: { id: asset.id }, format: :json
+      expect(response).to have_http_status(204)
+      expect { Kms::Asset.find(asset.id) }.to raise_error(ActiveRecord::RecordNotFound)
+    end
+  end
+end

data/spec/controllers/kms/pages_controller_spec.rb

@@ -0,0 +1,82 @@
+require 'spec_helper'
+
+describe Kms::PagesController, type: :controller do
+    login_user
+
+    describe 'GET index' do
+      it 'returns pages json representation' do
+        FactoryGirl.create(:page)
+        get :index, format: :json
+
+        expect(response).to be_success
+        expect(json.length).to eq(1)
+      end
+    end
+
+    describe 'POST create' do
+      context 'when save failed' do
+        it "returns errors" do
+          post :create, params: { page: FactoryGirl.attributes_for(:invalid_page) }, format: :json
+          expect(response.unprocessable?).to be true
+          expect(json['errors']).to_not be nil
+        end
+      end
+      context 'when save successful' do
+        let(:page_params) { FactoryGirl.attributes_for(:page) }
+        it "returns 204 status" do
+          post :create, params: { page: page_params }, format: :json
+          expect(response).to have_http_status(204)
+        end
+      end
+    end
+
+    describe 'PUT update' do
+      let(:page) { FactoryGirl.create(:page) }
+      context 'when update failed' do
+        it "returns errors" do
+          put :update, params: { id: page.id, page: FactoryGirl.attributes_for(:invalid_page) }, format: :json
+          expect(response.unprocessable?).to be true
+          expect(json['errors']).to_not be nil
+        end
+      end
+      context 'when update successful' do
+        let(:page_params) { FactoryGirl.attributes_for(:page) }
+        it "returns 204 status" do
+          put :update, params: { id: page.id, page: page_params }, format: :json
+          expect(response).to have_http_status(204)
+        end
+      end
+    end
+
+    describe 'GET show' do
+      let(:page) { FactoryGirl.create(:page) }
+      it "returns page json representation" do
+        get :show, params: { id: page.id }, format: :json
+        expect(response).to be_success
+        expect(json['title']).to eq(page.title)
+      end
+    end
+
+    describe 'DELETE destroy' do
+      let(:page) { FactoryGirl.create(:page) }
+      it "deletes page" do
+        delete :destroy, params: { id: page.id }, format: :json
+        expect(response).to have_http_status(204)
+      end
+    end
+
+    describe 'POST sorting' do
+      it "sorts pages according to updated order" do
+        page1 = FactoryGirl.create(:page)
+        page2 = FactoryGirl.create(:page, parent: Kms::Page.where(slug: 'index').first)
+        pages_tree = Kms::Page.arrange_serializable(order: :position)
+        # reorder here
+        pages_tree[0]['children'].sort! {|a,b| b['position'] <=> a['position'] }
+        post :sorting, params: { '_json' => [pages_tree]}, format: :json
+        expect(response).to be_success
+        expect(json[0]['children'][0]['slug']).to eq(page2.slug)
+        expect(json[0]['children'][-1]['slug']).to eq(page1.slug)
+      end
+    end
+
+end

data/spec/controllers/kms/public/pages_controller_spec.rb

@@ -0,0 +1,30 @@
+require 'spec_helper'
+
+describe Kms::Public::PagesController, type: :controller do
+
+  describe 'GET show' do
+    context 'when page was found' do
+
+    end
+
+    context 'when page not found' do
+      context 'when special 404 page exists' do
+        it 'returns 404 page content' do
+          page_404 = FactoryGirl.create(:page_404)
+          get :show, params: { path: 'blabla' }
+          expect(response.status).to eq(404)
+          expect(response.body).to include('Not Found')
+        end
+      end
+      context 'when no special 404 page' do
+        it 'returns 404.html from public folder' do
+          get :show, params: { path: '/blabla' }
+          expect(response.status).to eq(404)
+          expect(response.body).to eq(File.read(File.join(Rails.root, 'public/404.html')))
+        end
+      end
+
+    end
+  end
+
+end

data/spec/controllers/kms/snippets_controller_spec.rb

@@ -1,69 +1,83 @@
 require 'spec_helper'
 
 describe Kms::SnippetsController, type: :controller do
-  login_user
-  describe 'GET index' do
-    it 'returns snippets json representation' do
-      FactoryGirl.create(:snippet)
-      get :index, format: :json
+  context 'when user/content manager' do
+    login_user
 
-      expect(response).to be_success
-      expect(json.length).to eq(1)
+    describe 'GET index' do
+      it 'raises AccessDenied exception' do
+        FactoryGirl.create(:snippet)
+        expect { get :index, format: :json }.to raise_error(CanCan::AccessDenied)
+      end
     end
   end
 
-  describe 'POST create' do
-    context 'when save failed' do
-      it "returns errors" do
-        post :create, params: { snippet: FactoryGirl.attributes_for(:invalid_snippet) }, format: :json
-        expect(response.unprocessable?).to be true
-        expect(json['errors']).to_not be nil
+  context 'when admin' do
+    login_admin
+
+    describe 'GET index' do
+      it 'returns snippets json representation' do
+        FactoryGirl.create(:snippet)
+        get :index, format: :json
+
+        expect(response).to be_success
+        expect(json.length).to eq(1)
       end
     end
-    context 'when save successful' do
-      it "returns 204 status" do
-        snippet_params = FactoryGirl.attributes_for(:snippet)
-        post :create, params: { snippet: snippet_params }, format: :json
-        expect(response).to have_http_status(204)
+
+    describe 'POST create' do
+      context 'when save failed' do
+        it "returns errors" do
+          post :create, params: { snippet: FactoryGirl.attributes_for(:invalid_snippet) }, format: :json
+          expect(response.unprocessable?).to be true
+          expect(json['errors']).to_not be nil
+        end
       end
+      context 'when save successful' do
+        it "returns 204 status" do
+          snippet_params = FactoryGirl.attributes_for(:snippet)
+          post :create, params: { snippet: snippet_params }, format: :json
+          expect(response).to have_http_status(204)
+        end
+      end
+
     end
 
-  end
+    describe 'PUT update' do
+      context 'when update failed' do
+        it "returns errors" do
+          snippet = FactoryGirl.create(:snippet)
+          put :update, params: { id: snippet.id, snippet: {slug: nil} }, format: :json
+          expect(response.unprocessable?).to be true
+          expect(json['errors']).to_not be nil
+        end
+      end
+      context 'when update successful' do
+        it "returns 204 status" do
+          snippet = FactoryGirl.create(:snippet)
+          put :update, params: { id: snippet.id, snippet: {slug: 'new-slug'} }, format: :json
+          expect(response).to have_http_status(204)
+        end
+      end
+    end
 
-  describe 'PUT update' do
-    context 'when update failed' do
-      it "returns errors" do
+    describe 'GET snippet' do
+      it "returns snippet object" do
         snippet = FactoryGirl.create(:snippet)
-        put :update, params: { id: snippet.id, snippet: {slug: nil} }, format: :json
-        expect(response.unprocessable?).to be true
-        expect(json['errors']).to_not be nil
+        get :show, params: { id: snippet.id }, format: :json
+        expect(response).to be_success
+        expect(json['name']).to eq(snippet.name)
+        expect(json['slug']).to eq(snippet.slug)
+        expect(json['content']).to eq(snippet.content)
       end
     end
-    context 'when update successful' do
-      it "returns 204 status" do
+
+    describe 'DELETE snippet' do
+      it 'deletes snippet and returns no content' do
         snippet = FactoryGirl.create(:snippet)
-        put :update, params: { id: snippet.id, snippet: {slug: 'new-slug'} }, format: :json
+        delete :destroy, params: { id: snippet.id }, format: :json
         expect(response).to have_http_status(204)
       end
     end
   end
-
-  describe 'GET snippet' do
-    it "returns snippet object" do
-      snippet = FactoryGirl.create(:snippet)
-      get :show, params: { id: snippet.id }, format: :json
-      expect(response).to be_success
-      expect(json['name']).to eq(snippet.name)
-      expect(json['slug']).to eq(snippet.slug)
-      expect(json['content']).to eq(snippet.content)
-    end
-  end
-
-  describe 'DELETE snippet' do
-    it 'deletes snippet and returns no content' do
-      snippet = FactoryGirl.create(:snippet)
-      delete :destroy, params: { id: snippet.id }, format: :json
-      expect(response).to have_http_status(204)
-    end
-  end
 end

data/spec/controllers/kms/templates_controller_spec.rb

@@ -0,0 +1,51 @@
+require 'spec_helper'
+
+describe Kms::TemplatesController, type: :controller do
+  context 'when user/content manager' do
+    login_user
+
+    describe 'GET index' do
+      it 'returns templates json representation' do
+        FactoryGirl.create(:template)
+        get :index, format: :json
+
+        expect(response).to be_success
+        expect(json.length).to eq(1)
+      end
+    end
+
+  end
+
+  context 'when admin' do
+    login_admin
+
+    describe 'POST create' do
+      context 'when save failed' do
+        it "returns errors" do
+          post :create, params: { template: FactoryGirl.attributes_for(:invalid_template) }, format: :json
+          expect(response.unprocessable?).to be true
+          expect(json['errors']).to_not be nil
+        end
+      end
+      context 'when save successful' do
+        let(:template_params) { FactoryGirl.attributes_for(:template) }
+        it "returns 204 status" do
+          post :create, params: { template: template_params }, format: :json
+          expect(response).to have_http_status(204)
+        end
+        it "substitutes <script> and <link> tags with asset_tag helper in template content" do
+          post :create, params: { template: template_params }, format: :json
+          template = Kms::Template.find_by(name: template_params[:name])
+          expect(template.content).to include("{{ 'compressed.js' | asset_tag }}")
+          expect(template.content).to include("{{ 'fonts.css' | asset_tag }}")
+        end
+        it "substitutes <img src..> with <img src='{{ ... | asset_path }}' helper in template content" do
+          post :create, params: { template: template_params }, format: :json
+          template = Kms::Template.find_by(name: template_params[:name])
+          expect(template.content).to include("{{ 'slide.jpg' | asset_path }}")
+        end
+      end
+    end
+  end
+
+end