dandify 0.0.1

data/config/routes.rb

@@ -0,0 +1,9 @@
+Spree::Core::Engine.add_routes do
+  namespace :admin do
+    resource :stylesheets do
+      post 'restore', to: 'stylesheets#restore'
+    end
+  end
+
+  match 'dandystyles', to: 'stylesheets#show', via: :all
+end

data/config/spring.rb

@@ -0,0 +1 @@
+Spring.application_root = './spec/dummy'

data/db/migrate/20140616202617_create_spree_stylesheets.rb

@@ -0,0 +1,10 @@
+class CreateSpreeStylesheets < ActiveRecord::Migration
+  def change
+    create_table :spree_stylesheets do |t|
+      t.text :style_raw
+      t.text :style_compressed
+
+      t.timestamps
+    end
+  end
+end

data/db/migrate/20140617221725_add_paper_trail_versions.rb

@@ -0,0 +1,13 @@
+class AddPaperTrailVersions < ActiveRecord::Migration
+  def change
+    create_table :versions do |t|
+      t.string   :item_type, :null => false
+      t.integer  :item_id,   :null => false
+      t.string   :event,     :null => false
+      t.string   :whodunnit
+      t.text     :object
+      t.datetime :created_at
+    end
+    add_index :versions, [:item_type, :item_id]
+  end
+end

data/lib/dandify.rb

@@ -0,0 +1,7 @@
+require 'spree_core'
+require 'sass'
+require 'paper_trail'
+require 'dandify/engine'
+
+module Dandify
+end

data/lib/dandify/engine.rb

@@ -0,0 +1,18 @@
+module Dandify
+  class Engine < ::Rails::Engine
+    require 'spree/core'
+    isolate_namespace Spree
+    engine_name 'dandify'
+
+    config.autoload_paths += Dir["#{config.root}/lib"]
+    config.generators { |gen| gen.test_framework :rspec }
+
+    def self.activate
+      Dir[File.join(__dir__, '../../app/**/*_decorator*.rb')].each do |klass|
+        Rails.application.config.cache_classes ? require(klass) : load(klass)
+      end
+    end
+
+    config.to_prepare(&method(:activate).to_proc)
+  end
+end

data/lib/dandify/version.rb

@@ -0,0 +1,3 @@
+module Dandify
+  VERSION = '0.0.1'
+end

data/lib/generators/dandify/install/install_generator.rb

@@ -0,0 +1,24 @@
+module Dandify
+  module Generators
+    class InstallGenerator < Rails::Generators::Base
+      class_option :auto_run_migrations, type: :boolean, default: true
+
+      def add_migrations
+        run 'bundle exec rake railties:install:migrations FROM=dandify'
+      end
+
+      def add_stylesheets
+        inject_into_file 'vendor/assets/stylesheets/spree/backend/all.css', "*= require spree/backend/dandify\n", before: /\*\//, verbose: true
+      end
+
+      def run_migrations
+        res = ask('Would you like to run the migrations now? [Y/n]').downcase
+        if ['', 'y'].include?(res)
+          run 'bundle exec rake db:migrate'
+        else
+          logger 'Skipping rake db:migrate, don\'t forget to run it!'
+        end
+      end
+    end
+  end
+end

data/lib/tasks/dandify_tasks.rake

@@ -0,0 +1,4 @@
+# desc "Explaining what the task does"
+# task :dandify do
+#   # Task goes here
+# end

data/spec/controllers/spree/admin/stylesheets_controller_spec.rb

@@ -0,0 +1,71 @@
+require 'spec_helper'
+
+describe Spree::Admin::StylesheetsController do
+  stub_authorization!
+
+  let(:show_path) { spree.admin_stylesheets_path }
+  let(:good_style) { 'body {color:#ffff00;}' }
+  let(:bad_style) { 'body {color:#ffff00;' }
+
+  before do
+    user = create :admin_user
+    controller.stub(try_spree_current_user: user)
+  end
+
+  context '#show' do
+    it 'assigns new instance when none is found' do
+       spree_get :show
+       expect(assigns[:style]).to be_new_record
+    end
+
+    it 'assigns existing' do
+      style = create :stylesheet
+      spree_get :show
+      expect(assigns[:style]).to eq style
+    end
+  end
+
+  context '#new' do
+    it 'can create first a revision' do
+      style = create :stylesheet
+      spree_get :show
+      expect(assigns[:style]).to eq style
+      expect(response).to render_template(:show)
+    end
+
+    it 'can not create first revision because error' do
+      spree_post :create, stylesheet: { style_raw: bad_style }
+      expect(response).to render_template(:new)
+    end
+  end
+
+  context '#edit' do
+    it 'can create a revision' do
+       create :stylesheet
+       spree_get :show
+       expect(response).to render_template(:show)
+    end
+
+    it 'can not create a revision because error' do
+      spree_post :update, stylesheet: { style_raw: bad_style }
+      expect(response).to render_template(:edit)
+    end
+  end
+
+  context '#restore', versioning: true do
+    before { create :stylesheet }
+
+    it 'fails to reify when there is only one revision' do
+      spree_post :restore
+      expect(flash[:error]).to eq Spree.t('dandify.restore.error')
+      expect(response).to redirect_to show_path
+    end
+
+    it 'able to reify when there is more than one revision' do
+      spree_post :update, stylesheet: { style_raw: good_style }
+      spree_post :restore
+      expect(flash[:success]).to eq Spree.t('dandify.restore.success')
+      expect(response).to redirect_to show_path
+    end
+  end
+end

data/spec/controllers/spree/stylesheets_controller_spec.rb

@@ -0,0 +1,36 @@
+require 'spec_helper'
+
+describe Spree::StylesheetsController, '#show' do
+  context 'with a bad format' do
+    it 'responds with blank sheet' do
+      spree_get :show, format: :xml
+
+      expect(response.header['Content-Type']).to include 'text/css'
+    end
+
+    it 'responds with found sheet' do
+      css = create(:stylesheet).style_compressed
+      spree_get :show, format: :atom
+
+      expect(response.header['Content-Type']).to include 'text/css'
+      expect(assigns[:style].style_compressed).to eq css
+    end
+  end
+
+  context 'when there is no stylesheet' do
+    it 'displays a blank stylesheet' do
+      spree_get :show, format: :css
+
+      expect(assigns[:style]).to be_a Spree::BlankStylesheet
+    end
+  end
+
+  context 'when there is a stylesheet' do
+    it 'assigns the stored sheet' do
+      create :stylesheet
+      spree_get :show, format: :css
+
+      expect(assigns[:style]).to be_a Spree::Stylesheet
+    end
+  end
+end

data/spec/factories/stylesheet_factory.rb

@@ -0,0 +1,5 @@
+FactoryGirl.define do
+  factory :stylesheet, class: Spree::Stylesheet do
+    style_raw '#main { display: none; }'
+  end
+end

data/spec/features/dandy_style_spec.rb

@@ -0,0 +1,10 @@
+require 'spec_helper'
+
+feature 'Dandy stylesheet' do
+  scenario 'store contains dandystylesheet' do
+    visit spree.root_path
+    expect(page).to(
+      have_xpath('//link[contains(@href, "/dandystyles.css")]', visible: false)
+    )
+  end
+end

data/spec/features/managing_stylsheet_spec.rb

@@ -0,0 +1,80 @@
+require 'spec_helper'
+
+feature 'Managing stylesheets' do
+  stub_authorization!
+
+  let(:show_path) { spree.admin_stylesheets_path }
+  let(:new_path) { spree.new_admin_stylesheets_path }
+  let(:edit_path) { spree.edit_admin_stylesheets_path }
+  let(:restore_path) { spree.restore_admin_stylesheets_path }
+  let(:style) { 'body {display: none;}' }
+
+  context 'without existing style' do
+    scenario 'user can create styles' do
+      visit new_path
+      fill_in Spree.t('dandify.form.label'), with: style
+      click_button Spree.t(:create)
+
+      expect(page.status_code).to be(200)
+      expect(page).to have_text Spree.t('dandify.show.title')
+    end
+  end
+
+  context 'with existing style' do
+    before { create :stylesheet }
+
+    scenario 'user sees first audit trail', versioning: true do
+      visit show_path
+
+      within_row(1) { expect(page).to have_content('Create') }
+    end
+
+    scenario 'user can edit exiting styles' do
+      visit edit_path
+
+      fill_in Spree.t('dandify.form.label'), with: style
+      click_button Spree.t(:update)
+
+      expect(page.status_code).to be(200)
+      expect(page).to have_text(Spree.t('dandify.show.title'))
+    end
+
+    scenario 'user creates new audit trail on edit', versioning: true do
+      visit edit_path
+
+      fill_in Spree.t('dandify.form.label'), with: style
+      click_button Spree.t(:update)
+
+      visit show_path
+
+      within_row(1) { page.should have_content('Update') }
+      within_row(2) { page.should have_content('Create') }
+    end
+  end
+
+  context 'restoring stylesheet' do
+    before { create :stylesheet }
+
+    let(:css_path) { 'table#listing_versions tbody tr' }
+
+    scenario 'not possible when there is only one version', versioning: true do
+      visit show_path
+
+      page.should have_css(css_path, count: 1)
+      within_row(1) { expect(page).to have_link('', href: edit_path) }
+    end
+
+    scenario 'user can restore to previous version', versioning: true do
+      visit edit_path
+
+      fill_in Spree.t('dandify.form.label'), with: style
+      click_button Spree.t(:update)
+
+      visit show_path
+
+      page.should have_css(css_path, count: 2)
+      within_row(1) { expect(page).to have_link('', href: edit_path) }
+      within_row(2) { expect(page).to have_link('', href: restore_path) }
+    end
+  end
+end

data/spec/features/sidebar_spec.rb

@@ -0,0 +1,41 @@
+require 'spec_helper'
+
+describe 'In the admin' do
+  stub_authorization!
+
+  let(:configuration_path) { spree.edit_admin_general_settings_path }
+  let(:show_path) { spree.admin_stylesheets_path }
+  let(:new_path) { spree.new_admin_stylesheets_path }
+  let(:edit_path) { spree.edit_admin_stylesheets_path }
+
+  context 'on the configuration page' do
+    it 'the user views Dandify link in sidebar' do
+      visit configuration_path
+      expect(page).to have_link(Spree.t('dandify.sidebar'), href: show_path)
+    end
+
+    it 'the user views non-highlighted Dandify link in sidebar' do
+      visit configuration_path
+      expect(page).to_not(
+        have_selector(:css, "li.active a[href='#{show_path}']")
+      )
+    end
+  end
+
+  context 'the user views highlighted sidebar link' do
+    it 'on Dandify show page' do
+      visit show_path
+      expect(page).to have_selector(:css, "li.active a[href='#{show_path}']")
+    end
+
+    it 'on Dandify edit page' do
+      visit edit_path
+      expect(page).to have_selector(:css, "li.active a[href='#{show_path}']")
+    end
+
+    it 'on Dandify new page' do
+      visit new_path
+      expect(page).to have_selector(:css, "li.active a[href='#{show_path}']")
+    end
+  end
+end

data/spec/features/viewing_admin_spec.rb

@@ -0,0 +1,49 @@
+require 'spec_helper'
+
+feature 'Viewing stylesheet administration page' do
+  stub_authorization!
+
+  let(:show_path) { spree.admin_stylesheets_path }
+
+  context 'without existing style' do
+    scenario 'user views page with no stylesheet' do
+      visit show_path
+
+      expect(page).to have_text Spree.t('dandify.show.none')
+    end
+
+    scenario 'user has access to create link' do
+      visit show_path
+
+      expect(page).to have_text Spree.t('dandify.show.buttons.new')
+    end
+
+    scenario 'user does not have access to edit link' do
+      visit show_path
+
+      expect(page).to_not have_text Spree.t('dandify.show.buttons.edit')
+    end
+  end
+
+  context 'with existing style' do
+    before { create :stylesheet }
+
+    scenario 'user views page with an active stylesheet' do
+      visit show_path
+
+      expect(page).to_not have_text Spree.t('dandify.show.none')
+    end
+
+    scenario 'user does not have access to create link' do
+      visit show_path
+
+      expect(page).to_not have_text Spree.t('dandify.show.buttons.new')
+    end
+
+    scenario 'user has access to edit link' do
+      visit show_path
+
+      expect(page).to have_text Spree.t('dandify.show.buttons.edit')
+    end
+  end
+end

data/spec/helpers/admin/navigation_helper_spec.rb

@@ -0,0 +1,14 @@
+require 'spec_helper'
+
+describe Spree::Admin::NavigationHelper do
+  describe '#link_to_restore_url' do
+    let(:restore_path) { spree.edit_admin_stylesheets_path }
+
+    context 'link to restore URL' do
+      it 'should build circular restore button' do
+        restore_button = helper.link_to_restore_url(restore_path)
+        restore_button.should include(Spree.t('dandify.confirm.restore'))
+      end
+    end
+  end
+end

data/spec/models/spree/blank_stylesheet_spec.rb

@@ -0,0 +1,7 @@
+require 'spec_helper'
+
+describe Spree::BlankStylesheet, '#style_compressed' do
+  it 'returns a blank string' do
+    expect(Spree::BlankStylesheet.new.style_compressed).to eq ''
+  end
+end

data/spec/models/spree/stylesheet_spec.rb

@@ -0,0 +1,102 @@
+require 'spec_helper'
+
+describe Spree::Stylesheet do
+  context 'versions', versioning: true do
+    it { should be_versioned }
+
+    it 'restores previous versions' do
+      style = create :stylesheet
+      style.update style_raw: 'h1{display:none; color: blue;}'
+
+      expect(style.versions.size).to eq(2)
+    end
+  end
+
+  context 'validations' do
+    let(:valid_css) { '#main {background-color: #0000ff}' }
+    let(:invalid_css) { 'main {background-color #0000ff}' }
+
+    it { should validate_presence_of(:style_raw) }
+
+    it 'pass with proper vanilla syntax' do
+      expect(style_with valid_css).to be_valid
+    end
+
+    it 'fail with bad vanilla syntax' do
+      expect(style_with invalid_css).to_not be_valid
+    end
+  end
+
+  context 'processing raw' do
+    it 'allows both css and scss' do
+      expect(style_with(<<-CSS)).to be_valid
+
+$font-stack: Helvetica, sans-serif;
+
+body {
+  font: 100% $font-stack
+}
+
+h1 { display: inline ;}
+
+      CSS
+    end
+
+    context 'compresses' do
+      it 'vanilla css' do
+        compressed = "#main{display:block}.foo{width:300px;height:200px}\n"
+        expect(create_style_with(<<-CSS).style_compressed).to eq compressed
+
+#main {
+   display: block;
+}
+
+.foo {
+  width: 300px;
+  height: 200px;
+}
+
+        CSS
+      end
+
+      it 'compresses scss' do
+        compressed = "nav ul{margin:0;padding:0}a{display:block}\n"
+        expect(create_style_with(<<-CSS).style_compressed).to eq compressed
+
+nav {
+  ul {
+    margin: 0;
+    padding: 0;
+  }
+}
+
+a {
+  display: block;
+}
+
+        CSS
+      end
+    end
+  end
+
+  def create_style_with(css)
+    style = style_with(css)
+    style.save
+    style
+  end
+
+  def style_with(css)
+    Spree::Stylesheet.new(style_raw: css)
+  end
+end
+
+describe Spree::Stylesheet, '#load_style' do
+  it 'returns Blanksheet when no sheet is found' do
+    expect(Spree::Stylesheet.load_style).to be_a Spree::BlankStylesheet
+  end
+
+  it 'returns a found instance of itself' do
+    styles = create :stylesheet
+    expect(Spree::Stylesheet.load_style).to eq styles
+  end
+end