solidus_static_content 1.0.0

data/spec/controllers/static_content_controller_spec.rb

@@ -0,0 +1,37 @@
+require 'spec_helper'
+
+describe Spree::StaticContentController, type: :controller do
+  before do
+    controller.stub spree_current_user: nil
+  end
+
+  let!(:store) { create(:store, default: true) }
+
+  context '#show' do
+    it 'accepts path as root' do
+      page = create(:page, slug: '/', stores: [store])
+      request.path = page.slug
+      spree_get :show, path: page.slug
+      expect(response).to be_success
+    end
+
+    it 'accepts path as string' do
+      page = create(:page, slug: 'hello', stores: [store])
+      request.path = page.slug
+      spree_get :show, path: page.slug
+      expect(response).to be_success
+    end
+
+    it 'accepts path as nested' do
+      page = create(:page, slug: 'aa/bb/cc', stores: [store])
+      request.path = page.slug
+      spree_get :show, path: page.slug
+      expect(response).to be_success
+    end
+
+    it 'respond with a 404 when no page exists' do
+      spree_get :show
+      expect(response.response_code).to eq(404)
+    end
+  end
+end

data/spec/factories/page_factory.rb

@@ -0,0 +1,18 @@
+FactoryGirl.define do
+  factory :page, class: Spree::Page do
+    title { generate(:random_string) }
+    body  { generate(:random_description) }
+
+    sequence(:slug) { |n| "/page#{n}" }
+
+    trait :with_foreign_link do
+      foreign_link do
+        if defined?(FFaker)
+          FFaker::Internet.http_url
+        else
+          Faker::Internet.http_url
+        end
+      end
+    end
+  end
+end

data/spec/features/admin/pages_spec.rb

@@ -0,0 +1,67 @@
+require 'spec_helper'
+
+RSpec.feature 'Admin Static Content', js: true do
+  stub_authorization!
+
+  context 'when no page exists' do
+    background do
+      visit spree.admin_path
+      click_link 'Pages'
+    end
+
+    scenario 'can create new page' do
+      expect(page).to have_text /No Pages found, Add One!/i
+
+      click_link 'New page'
+      expect(page).to have_text 'Static pages'
+
+      fill_in 'page_title', with: 'Contact us'
+      fill_in 'page_slug', with: 'contact-us'
+      fill_in 'page_body', with: 'hello..'
+
+      fill_in 'page_meta_title', with: 'Contact us'
+      fill_in 'page_meta_keywords', with: 'spree, contact'
+      fill_in 'page_meta_description', with: 'Short description for contact page.'
+
+      check 'page_show_in_sidebar'
+      check 'page_show_in_header'
+      check 'page_show_in_footer'
+
+      expect(find('#page_visible')).to be_checked
+
+      click_button 'Create'
+      expect(page).to have_text 'successfully created!'
+    end
+  end
+
+  context 'when page exists' do
+    given!(:static_page) { create(:page) }
+
+    background do
+      visit spree.admin_path
+      click_link 'Pages'
+    end
+
+    scenario 'can be updated' do
+      within_row(1) do
+        click_icon :edit
+      end
+
+      fill_in 'page_title', with: 'Contact'
+      fill_in 'page_slug', with: 'contact'
+
+      click_button 'Update'
+      expect(page).to have_text 'successfully updated!'
+    end
+
+    scenario 'can be deleted' do
+      within_row(1) do
+        click_icon :trash
+      end
+
+      page.driver.browser.switch_to.alert.accept unless Capybara.javascript_driver == :poltergeist
+
+      expect(page).not_to have_text static_page.title
+    end
+  end
+end

data/spec/features/static_content_spec.rb

@@ -0,0 +1,53 @@
+require 'spec_helper'
+
+feature 'Static Content Page', js: true do
+
+  let!(:store) { create(:store, default: true) }
+
+  context 'render page' do
+    scenario 'is a query string' do
+      create(:page, slug: '/page', title: 'Query Test', stores: [store])
+      visit '/page?test'
+      expect(page).to have_text 'Query Test'
+    end
+
+    scenario 'can have slug not starting by /' do
+      create(:page, slug: 'page2', title: 'No Slash Prefix Test', stores: [store])
+      visit '/page2'
+      expect(page).to have_text 'No Slash Prefix Test'
+    end
+
+    scenario 'can have slug with multiple /' do
+      create(:page, slug: '/hello/shoppers/page3', title: 'Multiple Slash Test', stores: [store])
+      visit '/hello/shoppers/page3'
+      expect(page).to have_text 'Multiple Slash Test'
+    end
+
+    scenario 'can be a custom root page' do
+      create(:page, slug: '/', title: 'Root Page Test', stores: [store])
+      visit '/'
+      expect(page).to have_text 'Root Page Test'
+    end
+
+    scenario 'is limited within its own constraints' do
+      create(:page, slug: '/t/categories/page3', title: 'Constraint Test', stores: [store])
+      visit '/t/categories/page3'
+      expect(page).not_to have_text 'Constraint Test'
+    end
+
+    scenario 'fetch correct page' do
+      create(:page, slug: '/', stores: [store])
+      create(:page, slug: 'hello', title: 'Hello', stores: [store])
+      create(:page, slug: 'somwhere', stores: [store])
+      create(:page, :with_foreign_link, slug: 'whatever', stores: [store])
+      visit '/hello'
+      expect(page).to have_text 'Hello'
+    end
+
+    scenario 'do not effect the rendering of the rest of the site' do
+      product = create(:product)
+      visit spree.product_path(product)
+      expect(page).to have_text product.name
+    end
+  end
+end

data/spec/helpers/page_helper_spec.rb

@@ -0,0 +1,14 @@
+require 'spec_helper'
+
+RSpec.describe Spree::PagesHelper, type: :helper do
+  context '.render_snippet' do
+    it 'return body when page exists' do
+      page = create(:page)
+      expect(render_snippet(page.slug)).to eq page.body
+    end
+
+    it 'return nil when no page exist' do
+      expect(render_snippet('hello')).to be_nil
+    end
+  end
+end

data/spec/lib/spree_static.content_spec.rb

@@ -0,0 +1,34 @@
+require 'spec_helper'
+
+describe StaticPage do
+  context '.remove_spree_mount_point' do
+    specify do
+      path = '/hello'
+      expect(subject.remove_spree_mount_point(path)).to eq 'hello'
+    end
+  end
+end
+
+describe Spree::StaticPage do
+  subject { described_class }
+
+  context '.matches?' do
+    it 'is true when valid page' do
+      page = create(:page, slug: 'hello', visible: true)
+      request = double('request', path: page.slug)
+      expect(subject.matches?(request)).to be true
+    end
+
+    it 'is false when using reserved slug name' do
+      page = create(:page, slug: 'login', visible: true)
+      request = double('request', path: page.slug)
+      expect(subject.matches?(request)).to be false
+    end
+
+    it 'is false when page is not accessible' do
+      page = create(:page, slug: 'hello', visible: false)
+      request = double('request', path: page.slug)
+      expect(subject.matches?(request)).to be false
+    end
+  end
+end

data/spec/models/page_spec.rb

@@ -0,0 +1,46 @@
+require 'spec_helper'
+
+describe Spree::Page do
+  let!(:page) { create(:page) }
+
+  context 'factory' do
+    it 'is valid' do
+      expect(page).to be_valid
+    end
+  end
+
+  it 'always add / prefix to slug' do
+    page = create(:page, slug: 'hello')
+    expect(page.slug).to eq '/hello'
+  end
+
+  context '.link' do
+    it 'return slug if foreign_link blank' do
+      page = create(:page, slug: 'hello')
+      expect(page.link).to eq page.slug
+    end
+
+    it 'return foreign_link if set' do
+      page = create(:page, :with_foreign_link, slug: 'hello')
+      expect(page.link).to eq page.foreign_link
+    end
+  end
+
+
+  context "pages in stores" do
+
+    before(:each) do
+      @store = create(:store)
+      @page = create(:page, :stores => [@store])
+      @page2 = create(:page)
+    end
+
+    it 'should correctly find pages by store' do
+      pages_by_store = Spree::Page.by_store(@store)
+      expect(pages_by_store).to include(@page)
+      expect(pages_by_store).to_not include(@page2)
+    end
+
+  end
+
+end

data/spec/spec_helper.rb

@@ -0,0 +1,50 @@
+require 'simplecov'
+SimpleCov.start 'rails'
+
+ENV["RAILS_ENV"] = "test"
+
+require File.expand_path("../dummy/config/environment.rb",  __FILE__)
+
+require 'ffaker'
+require 'rspec/rails'
+require 'database_cleaner'
+require 'capybara/rspec'
+require 'capybara/rails'
+require 'capybara/poltergeist'
+
+Dir[Rails.root.join("spec/support/**/*.rb")].each {|f| require f}
+
+require 'spree/testing_support/factories'
+require 'spree/testing_support/url_helpers'
+require 'spree/testing_support/controller_requests'
+require 'spree/testing_support/authorization_helpers'
+require 'spree/testing_support/capybara_ext'
+
+FactoryGirl.find_definitions
+
+RSpec.configure do |config|
+  config.mock_with :rspec
+  config.use_transactional_fixtures = false
+
+  config.include Spree::TestingSupport::ControllerRequests, type: :controller
+  config.include Spree::TestingSupport::UrlHelpers
+  config.include FactoryGirl::Syntax::Methods
+
+  config.extend Spree::TestingSupport::AuthorizationHelpers::Request, type: :feature
+
+  config.before(:suite) do
+    DatabaseCleaner.strategy = :transaction
+    DatabaseCleaner.clean_with(:truncation)
+  end
+
+  config.before(:each) do
+    DatabaseCleaner.strategy = RSpec.current_example.metadata[:js] ? :truncation : :transaction
+    DatabaseCleaner.start
+  end
+
+  config.after(:each) do
+    DatabaseCleaner.clean
+  end
+
+  Capybara.javascript_driver = :poltergeist
+end

metadata

@@ -0,0 +1,294 @@
+--- !ruby/object:Gem::Specification
+name: solidus_static_content
+version: !ruby/object:Gem::Version
+  version: 1.0.0
+platform: ruby
+authors:
+- Peter Berkenbosch
+- Roman Smirnov
+autorequire: 
+bindir: bin
+cert_chain: []
+date: 2016-09-22 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: solidus
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '1.1'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '3'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '1.1'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '3'
+- !ruby/object:Gem::Dependency
+  name: deface
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.0.2
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.0.2
+- !ruby/object:Gem::Dependency
+  name: capybara
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.7'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.7'
+- !ruby/object:Gem::Dependency
+  name: factory_girl
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '4.7'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '4.7'
+- !ruby/object:Gem::Dependency
+  name: ffaker
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: database_cleaner
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.5'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.5'
+- !ruby/object:Gem::Dependency
+  name: rspec-rails
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.5'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.5'
+- !ruby/object:Gem::Dependency
+  name: sqlite3
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.3'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.3'
+- !ruby/object:Gem::Dependency
+  name: poltergeist
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.10'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.10'
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.12'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.12'
+- !ruby/object:Gem::Dependency
+  name: pry-rails
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: test-unit
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+description: Extention to manage the static pages for your Spree shop.
+email: peter@pero-ict.nl
+executables: []
+extensions: []
+extra_rdoc_files: []
+files:
+- ".gitignore"
+- ".rspec"
+- ".travis.yml"
+- Gemfile
+- Guardfile
+- LICENSE.md
+- README.md
+- Rakefile
+- app/assets/javascripts/spree/backend/spree_static_content.js
+- app/assets/javascripts/spree/frontend/spree_static_content.js
+- app/assets/stylesheets/spree/backend/spree_static_content.css
+- app/assets/stylesheets/spree/frontend/spree_static_content.css
+- app/controllers/spree/admin/pages_controller.rb
+- app/controllers/spree/static_content_controller.rb
+- app/helpers/spree/pages_helper.rb
+- app/models/spree/page.rb
+- app/overrides/pages_in_footer.rb
+- app/overrides/pages_in_header.rb
+- app/overrides/pages_in_menu.rb
+- app/overrides/pages_in_sidebar.rb
+- app/views/spree/admin/pages/_form.html.erb
+- app/views/spree/admin/pages/edit.html.erb
+- app/views/spree/admin/pages/index.html.erb
+- app/views/spree/admin/pages/new.html.erb
+- app/views/spree/static_content/_static_content_footer.html.erb
+- app/views/spree/static_content/_static_content_header.html.erb
+- app/views/spree/static_content/_static_content_list.html.erb
+- app/views/spree/static_content/_static_content_sidebar.html.erb
+- app/views/spree/static_content/show.html.erb
+- bin/rails
+- config/locales/de.yml
+- config/locales/en.yml
+- config/locales/es.yml
+- config/locales/et.yml
+- config/locales/fr.yml
+- config/locales/it.yml
+- config/locales/nl.yml
+- config/locales/pl.yml
+- config/locales/pt-BR.yml
+- config/locales/pt.yml
+- config/locales/ro.yml
+- config/locales/ru.yml
+- config/locales/sv.yml
+- config/routes.rb
+- db/migrate/20081216193152_create_pages.rb
+- db/migrate/20090625125735_extend_pages.rb
+- db/migrate/20090814113100_add_visible_to_pages.rb
+- db/migrate/20090814142845_add_default_true_to_visible_for_page.rb
+- db/migrate/20090829000527_add_index_for_page.rb
+- db/migrate/20091219021134_add_meta_fields_to_pages.rb
+- db/migrate/20100204105222_add_layout_to_pages.rb
+- db/migrate/20100323085528_add_show_in_sidebar_option_to_pages.rb
+- db/migrate/20110717103112_add_meta_title_to_page.rb
+- db/migrate/20120723144115_add_render_as_partial_for_layout_for_spree_pages.rb
+- db/migrate/20140926121757_add_pages_stores.rb
+- lib/generators/solidus_static_content/install/install_generator.rb
+- lib/solidus_static_content.rb
+- lib/spree_static_content/engine.rb
+- solidus_static_content.gemspec
+- spec/controllers/static_content_controller_spec.rb
+- spec/factories/page_factory.rb
+- spec/features/admin/pages_spec.rb
+- spec/features/static_content_spec.rb
+- spec/helpers/page_helper_spec.rb
+- spec/lib/spree_static.content_spec.rb
+- spec/models/page_spec.rb
+- spec/spec_helper.rb
+homepage: http://spreecommerce.com/extensions/139-static-content
+licenses:
+- BSD-3
+metadata: {}
+post_install_message: 
+rdoc_options: []
+require_paths:
+- lib
+required_ruby_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - ">="
+    - !ruby/object:Gem::Version
+      version: 1.9.3
+required_rubygems_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - ">="
+    - !ruby/object:Gem::Version
+      version: '0'
+requirements:
+- none
+rubyforge_project: 
+rubygems_version: 2.5.1
+signing_key: 
+specification_version: 4
+summary: Extention to manage the static pages for your Spree shop.
+test_files:
+- spec/controllers/static_content_controller_spec.rb
+- spec/factories/page_factory.rb
+- spec/features/admin/pages_spec.rb
+- spec/features/static_content_spec.rb
+- spec/helpers/page_helper_spec.rb
+- spec/lib/spree_static.content_spec.rb
+- spec/models/page_spec.rb
+- spec/spec_helper.rb