spree-faq 2.0.0

data/spec/controllers/admin/question_categories_controller_spec.rb

@@ -0,0 +1,67 @@
+require 'spec_helper'
+
+describe Spree::Admin::QuestionCategoriesController do
+  stub_authorization!
+
+  let!(:user) { create(:user) }
+  let!(:question_category) { create(:question_category) }
+  let!(:question) { create(:question, question_category: question_category) }
+  let (:attributes) { attributes_for(:question_category) }
+
+  before { controller.stub spree_current_user: user }
+
+  context 'controller instance' do
+    it 'use Admin::QuestionCategoriesController' do
+      expect(controller).to be_an_instance_of Spree::Admin::QuestionCategoriesController
+    end
+  end
+
+  context '#index' do
+    specify do
+      spree_get :index
+      expect(response).to render_template :index
+    end
+  end
+
+  context '#new' do
+    before { spree_get :new }
+
+    it { expect(assigns(:question_category)).to be_a_new Spree::QuestionCategory }
+    it { expect(response).to render_template :new }
+  end
+
+  context '#edit' do
+    before { spree_get :edit, id: question_category }
+
+    it { expect(assigns(:question_category)).to eq question_category }
+    it { expect(response).to render_template :edit }
+  end
+
+  context '#create' do
+    context 'with valid params' do
+      it 'creates a new Spree::QuestionCategory' do
+        expect {
+          spree_post :create, question_category: attributes
+        }.to change(Spree::QuestionCategory, :count).by(1)
+      end
+
+      it 'assigns a newly created question_category as @question_category' do
+        spree_post :create, question_category: attributes
+        expect(assigns(:question_category)).to be_a Spree::QuestionCategory
+        expect(assigns(:question_category)).to be_persisted
+      end
+    end
+  end
+
+  context '#destroy' do
+    it 'destroys the requested question_category' do
+      expect {
+        spree_delete :destroy, id: question_category
+      }.to change(Spree::QuestionCategory, :count).by(-1)
+    end
+
+    it 'requires the :id parameter' do
+      expect { spree_delete :destroy }.to raise_error
+    end
+  end
+end

data/spec/controllers/faqs_controller_spec.rb

@@ -0,0 +1,29 @@
+require 'spec_helper'
+
+describe Spree::FaqsController do
+
+  let!(:question_category) { create(:question_category) }
+  let!(:question) { create(:question, question_category: question_category) }
+
+  before { controller.stub spree_current_user: nil }
+
+  context '#index' do
+    specify do
+      categories = [
+        question_category,
+        create(:question_category),
+        create(:question_category)
+      ]
+
+      spree_get :index
+      expect(assigns(:categories)).to eq categories
+      expect(response).to be_success
+    end
+  end
+
+  context '#default_title' do
+    it 'return default title' do
+      expect(subject.default_title).to eq Spree.t(:frequently_asked_questions, scope: :spree_faq)
+    end
+  end
+end

data/spec/factories/question_category_factory.rb

@@ -0,0 +1,5 @@
+FactoryGirl.define do
+  factory :question_category, class: Spree::QuestionCategory do
+    sequence(:name) { |n| "category#{n}" }
+  end
+end

data/spec/factories/question_factory.rb

@@ -0,0 +1,7 @@
+FactoryGirl.define do
+  factory :question, class: Spree::Question do
+    question_category
+    sequence(:question) { |n| "question#{n}" }
+    answer { generate(:random_description) }
+  end
+end

data/spec/features/admin/faq_spec.rb

@@ -0,0 +1,67 @@
+require 'spec_helper'
+
+feature 'Admin FAQ', :js do
+  stub_authorization!
+
+  given!(:admin_user) { create(:admin_user) }
+
+  background do
+    visit spree.admin_path
+    click_link 'Configuration'
+  end
+
+  scenario 'have faq tab in configuration sidebar' do
+    expect(page).to have_text 'FAQ'
+  end
+
+  context 'create new category' do
+
+    background do
+      click_link 'FAQ'
+      click_link 'New Category'
+    end
+
+    scenario 'save with valid name' do
+      fill_in 'Name', with: 'Shopping'
+      click_button 'Create'
+      within_table('listing_faq') do
+        expect(page).to have_text 'Shopping'
+      end
+    end
+
+    scenario 'error when empty name' do
+      click_button 'Create'
+      expect(page).to have_text 'Name can\'t be blank'
+    end
+  end
+
+  context 'modify' do
+
+    given!(:question_category) { create(:question_category, name: 'Shopping') }
+    given!(:question) do
+      create(:question,
+        question_category: question_category,
+        question: 'Do you sell stuff?',
+        answer: 'Think so..')
+    end
+
+    background do
+      click_link 'FAQ'
+    end
+
+    scenario 'update category' do
+      within_table('listing_faq') do
+        expect(page).to have_text 'Shopping'
+        click_icon :edit
+      end
+      expect(page).to have_text 'Editing Category Questions'
+
+      fill_in 'Category Name', with: 'Selling'
+      click_button 'Update'
+
+      within_table 'listing_faq' do
+        expect(page).to have_text 'Selling'
+      end
+    end
+  end
+end

data/spec/features/faq_spec.rb

@@ -0,0 +1,30 @@
+require 'spec_helper'
+
+feature 'FAQ', :js do
+
+  given!(:user) { create(:user) }
+  given!(:question_category) { create(:question_category, name: 'Shopping') }
+  given!(:question) do
+    create(:question,
+      question_category: question_category,
+      question: 'Do you sell stuff?',
+      answer: 'Think so..')
+  end
+
+  background do
+    visit spree.faq_path
+  end
+
+  scenario 'have all elements' do
+    expect(page).to have_text Spree.t(:frequently_asked_questions, scope: :spree_faq)
+    expect(page).to have_text 'Shopping'
+    expect(page).to have_text 'Do you sell stuff?'
+  end
+
+  context 'when click on question' do
+    scenario 'show answer' do
+      click_link 'Do you sell stuff?'
+      expect(page).to have_text 'Think so..'
+    end
+  end
+end

data/spec/models/question_category_spec.rb

@@ -0,0 +1,50 @@
+require 'spec_helper'
+
+describe Spree::QuestionCategory do
+
+  context 'instance attributes' do
+    it 'create a new instance given valid attributes' do
+      described_class.create!(name: 'Question 1')
+    end
+  end
+
+  context 'factory' do
+    it 'is valid' do
+      expect(build(:question_category)).to be_valid
+    end
+  end
+
+  context 'relation' do
+    it { should have_many(:questions) }
+
+    it 'have questions' do
+      expect(subject.questions).not_to be_nil
+    end
+  end
+
+  context 'validation' do
+    it { should validate_presence_of(:name) }
+    it { should validate_uniqueness_of(:name) }
+    it { should accept_nested_attributes_for(:questions) }
+  end
+
+  context 'mass assignment' do
+    %w(name questions_attributes question answer).each do |column|
+      it { should allow_mass_assignment_of(column.to_sym) }
+    end
+  end
+
+  context 'acts as list' do
+
+    subject { create(:question_category) }
+
+    before do
+      2.times { create(:question_category) }
+    end
+
+    it 'can have its position changed' do
+      subject.move_to_bottom
+      expect(subject.position).to eq(3)
+    end
+  end
+end

data/spec/models/question_spec.rb

@@ -0,0 +1,70 @@
+require 'spec_helper'
+
+describe Spree::Question do
+
+  let!(:question_category) { create(:question_category) }
+  let(:question) { create(:question, question_category_id: question_category.id) }
+
+  subject { question }
+
+  context 'factory' do
+    it 'is valid' do
+      expect(build(:question)).to be_valid
+    end
+  end
+
+  context 'instance attributes' do
+    it 'create a new instance given valid attributes' do
+      described_class.create!(question: 'Question 1',
+                              answer: 'Answer 1',
+                              question_category: create(:question_category))
+    end
+  end
+
+  context 'relation' do
+    it { should belong_to(:question_category) }
+
+    it 'belong to a category' do
+      expect(subject.question_category).not_to be_nil
+    end
+  end
+
+  context 'validation' do
+    it { should validate_presence_of(:question_category_id) }
+    it { should validate_presence_of(:question) }
+    it { should validate_presence_of(:answer) }
+
+    it 'require a category' do
+      invalid_question = build(:question, question_category: nil)
+      expect(invalid_question).to have(1).error_on(:question_category_id)
+    end
+
+    it 'require a question' do
+      invalid_question = build(:question, question: nil)
+      expect(invalid_question).to have(1).error_on(:question)
+    end
+
+    it 'require a answer' do
+      invalid_question = build(:question, answer: nil)
+      expect(invalid_question).to have(1).error_on(:answer)
+    end
+  end
+
+  context 'mass assignment' do
+    %w(question answer question_category_id question_category).each do |column|
+      it { should allow_mass_assignment_of(column.to_sym) }
+    end
+  end
+
+  context 'acts as list' do
+
+    before do
+      2.times { create(:question) }
+    end
+
+    it 'can have its position changed' do
+      subject.move_to_bottom
+      expect(subject.position).to eq(3)
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -0,0 +1,30 @@
+require 'simplecov'
+require 'coveralls'
+SimpleCov.formatter = SimpleCov::Formatter::MultiFormatter[
+  SimpleCov::Formatter::HTMLFormatter,
+  Coveralls::SimpleCov::Formatter
+]
+SimpleCov.start do
+  add_filter 'spec'
+  add_filter 'lib/spree_faq/engine'
+  add_group  'Controllers', 'app/controllers'
+  add_group  'Overrides', 'app/overrides'
+  add_group  'Models', 'app/models'
+  add_group  'Libraries', 'lib'
+end
+
+ENV['RAILS_ENV'] = 'test'
+
+require File.expand_path('../dummy/config/environment.rb', __FILE__)
+
+require 'rspec/rails'
+require 'shoulda-matchers'
+require 'ffaker'
+
+RSpec.configure do |config|
+  config.mock_with :rspec
+  config.use_transactional_fixtures = false
+  config.treat_symbols_as_metadata_keys_with_true_values = true
+end
+
+Dir[File.join(File.dirname(__FILE__), 'support/**/*.rb')].each { |f| require f }

data/spec/support/capybara.rb

@@ -0,0 +1,17 @@
+require 'capybara/rspec'
+require 'capybara/rails'
+require 'capybara/poltergeist'
+
+module CapybaraHelpers; end
+
+RSpec.configure do |config|
+  config.include CapybaraHelpers, type: :feature
+
+  Capybara.javascript_driver = :poltergeist
+
+  config.before(:each, :js) do
+    if Capybara.javascript_driver == :selenium
+      page.driver.browser.manage.window.maximize
+    end
+  end
+end

data/spec/support/database_cleaner.rb

@@ -0,0 +1,18 @@
+require 'database_cleaner'
+
+RSpec.configure do |config|
+
+  config.before :suite do
+    DatabaseCleaner.strategy = :transaction
+    DatabaseCleaner.clean_with :truncation
+  end
+
+  config.before do
+    DatabaseCleaner.strategy = example.metadata[:js] ? :truncation : :transaction
+    DatabaseCleaner.start
+  end
+
+  config.after do
+    DatabaseCleaner.clean
+  end
+end

data/spec/support/factory_girl.rb

@@ -0,0 +1,7 @@
+require 'factory_girl'
+
+FactoryGirl.find_definitions
+
+RSpec.configure do |config|
+  config.include FactoryGirl::Syntax::Methods
+end

data/spec/support/i18n.rb

@@ -0,0 +1,13 @@
+require 'rspec'
+require 'i18n-spec'
+
+RSpec.configure do |config|
+
+  config.after(type: :feature) do
+    missing_translations = page.body.scan(/translation missing: #{I18n.locale}\.(.*?)[\s<\"&]/)
+    if missing_translations.any?
+      puts "\e[1m\e[35mFound missing translations: #{missing_translations.inspect}\e[0m"
+      puts "\e[1m\e[35mIn spec: #{example.location}\e[0m"
+    end
+  end
+end

data/spec/support/spree.rb

@@ -0,0 +1,10 @@
+require 'spree/testing_support/factories'
+require 'spree/testing_support/controller_requests'
+require 'spree/testing_support/authorization_helpers'
+require 'spree/testing_support/url_helpers'
+require 'spree/testing_support/capybara_ext'
+
+RSpec.configure do |config|
+  config.include Spree::TestingSupport::ControllerRequests
+  config.include Spree::TestingSupport::UrlHelpers
+end

data/spec/translations/locale_spec.rb

@@ -0,0 +1,15 @@
+require 'spec_helper'
+
+describe 'locale' do
+  Dir.glob('config/locales/*.yml') do |locale_file|
+    context locale_file do
+      it { should be_parseable }
+      it { should have_valid_pluralization_keys }
+      it { should_not have_missing_pluralization_keys }
+      it { should have_one_top_level_namespace }
+      it { should_not have_legacy_interpolations }
+      it { should have_a_valid_locale }
+      it { should be_a_complete_translation_of 'config/locales/en.yml' }
+    end
+  end
+end

data/spree_faq.gemspec

@@ -0,0 +1,46 @@
+lib = File.expand_path('../lib/', __FILE__)
+$:.unshift lib unless $:.include?(lib)
+
+require 'spree_faq/version'
+
+Gem::Specification.new do |s|
+  s.platform     = Gem::Platform::RUBY
+  s.name          = 'spree-faq'
+  s.version       = SpreeFaq.version
+  s.summary       = %q{Adds an easy FAQ page for Spree Commerce}
+  s.description   = %q{With this gem you get an faq page and the management tools to make it very easy to update your faq and reduce the demand on your sites customer service}
+  s.required_ruby_version = '>= 1.9.3'
+
+  s.authors      = ['Josh Nussbaum', 'Tobias Bohwalli']
+  s.email        = 'hi@futhr.io'
+  s.homepage     = 'https://github.com/futhr/spree-faq'
+  s.license      = %q{BSD-3}
+
+  s.files        = `git ls-files`.split("\n")
+  s.test_files   = `git ls-files -- spec/*`.split("\n")
+  s.require_path = 'lib'
+  s.requirements << 'none'
+
+  s.has_rdoc = false
+
+  s.add_runtime_dependency 'spree_core', '~> 2.0.0'
+
+  s.add_development_dependency 'rspec-rails', '~> 2.14'
+  s.add_development_dependency 'capybara', '~> 2.2.1'
+  s.add_development_dependency 'selenium-webdriver', '>= 2.40.0'
+  s.add_development_dependency 'poltergeist', '~> 1.5.0'
+  s.add_development_dependency 'factory_girl', '~> 4.4'
+  s.add_development_dependency 'shoulda-matchers', '~> 2.5'
+  s.add_development_dependency 'sqlite3', '~> 1.3.8'
+  s.add_development_dependency 'simplecov', '~> 0.7.1'
+  s.add_development_dependency 'database_cleaner', '~> 1.2.0'
+  s.add_development_dependency 'i18n-spec', '~> 0.4.1'
+  s.add_development_dependency 'coffee-rails', '~> 3.2.2'
+  s.add_development_dependency 'sass-rails', '~> 3.2.6'
+  s.add_development_dependency 'ffaker', '>= 1.24.0'
+  s.add_development_dependency 'guard-rspec', '>= 4.2.0'
+  s.add_development_dependency 'launchy', '>= 2.4.0'
+  s.add_development_dependency 'pry-rails', '>= 0.3.0'
+  s.add_development_dependency 'coveralls', '>= 0.7.0'
+  s.add_development_dependency 'localeapp', '>= 0.7.2'
+end

data/vendor/assets/javascripts/jquery.scrollTo-min.js

@@ -0,0 +1,11 @@
+/**
+ * jQuery.ScrollTo - Easy element scrolling using jQuery.
+ * Copyright (c) 2007-2009 Ariel Flesler - aflesler(at)gmail(dot)com | http://flesler.blogspot.com
+ * Dual licensed under MIT and GPL.
+ * Date: 5/25/2009
+ * @author Ariel Flesler
+ * @version 1.4.2
+ *
+ * http://flesler.blogspot.com/2007/10/jqueryscrollto.html
+ */
+;(function(d){var k=d.scrollTo=function(a,i,e){d(window).scrollTo(a,i,e)};k.defaults={axis:'xy',duration:parseFloat(d.fn.jquery)>=1.3?0:1};k.window=function(a){return d(window)._scrollable()};d.fn._scrollable=function(){return this.map(function(){var a=this,i=!a.nodeName||d.inArray(a.nodeName.toLowerCase(),['iframe','#document','html','body'])!=-1;if(!i)return a;var e=(a.contentWindow||a).document||a.ownerDocument||a;return d.browser.safari||e.compatMode=='BackCompat'?e.body:e.documentElement})};d.fn.scrollTo=function(n,j,b){if(typeof j=='object'){b=j;j=0}if(typeof b=='function')b={onAfter:b};if(n=='max')n=9e9;b=d.extend({},k.defaults,b);j=j||b.speed||b.duration;b.queue=b.queue&&b.axis.length>1;if(b.queue)j/=2;b.offset=p(b.offset);b.over=p(b.over);return this._scrollable().each(function(){var q=this,r=d(q),f=n,s,g={},u=r.is('html,body');switch(typeof f){case'number':case'string':if(/^([+-]=)?\d+(\.\d+)?(px|%)?$/.test(f)){f=p(f);break}f=d(f,this);case'object':if(f.is||f.style)s=(f=d(f)).offset()}d.each(b.axis.split(''),function(a,i){var e=i=='x'?'Left':'Top',h=e.toLowerCase(),c='scroll'+e,l=q[c],m=k.max(q,i);if(s){g[c]=s[h]+(u?0:l-r.offset()[h]);if(b.margin){g[c]-=parseInt(f.css('margin'+e))||0;g[c]-=parseInt(f.css('border'+e+'Width'))||0}g[c]+=b.offset[h]||0;if(b.over[h])g[c]+=f[i=='x'?'width':'height']()*b.over[h]}else{var o=f[h];g[c]=o.slice&&o.slice(-1)=='%'?parseFloat(o)/100*m:o}if(/^\d+$/.test(g[c]))g[c]=g[c]<=0?0:Math.min(g[c],m);if(!a&&b.queue){if(l!=g[c])t(b.onAfterFirst);delete g[c]}});t(b.onAfter);function t(a){r.animate(g,j,b.easing,a&&function(){a.call(this,n,b)})}}).end()};k.max=function(a,i){var e=i=='x'?'Width':'Height',h='scroll'+e;if(!d(a).is('html,body'))return a[h]-d(a)[e.toLowerCase()]();var c='client'+e,l=a.ownerDocument.documentElement,m=a.ownerDocument.body;return Math.max(l[h],m[h])-Math.min(l[c],m[c])};function p(a){return typeof a=='object'?a:{top:a,left:a}}})(jQuery);