RubyGems - worth_saving - Versions diffs - 0.0.9 - Mend

worth_saving 0.0.9

Files changed (115) hide show

data/spec/dummy/public/404.html ADDED Viewed

@@ -0,0 +1,26 @@
+<!DOCTYPE html>
+<html>
+<head>
+  <title>The page you were looking for doesn't exist (404)</title>
+  <style type="text/css">
+    body { background-color: #fff; color: #666; text-align: center; font-family: arial, sans-serif; }
+    div.dialog {
+      width: 25em;
+      padding: 0 4em;
+      margin: 4em auto 0 auto;
+      border: 1px solid #ccc;
+      border-right-color: #999;
+      border-bottom-color: #999;
+    }
+    h1 { font-size: 100%; color: #f00; line-height: 1.5em; }
+  </style>
+</head>
+<body>
+  <!-- This file lives in public/404.html -->
+  <div class="dialog">
+    <h1>The page you were looking for doesn't exist.</h1>
+    <p>You may have mistyped the address or the page may have moved.</p>
+  </div>
+</body>
+</html>

data/spec/dummy/public/422.html ADDED Viewed

@@ -0,0 +1,26 @@
+<!DOCTYPE html>
+<html>
+<head>
+  <title>The change you wanted was rejected (422)</title>
+  <style type="text/css">
+    body { background-color: #fff; color: #666; text-align: center; font-family: arial, sans-serif; }
+    div.dialog {
+      width: 25em;
+      padding: 0 4em;
+      margin: 4em auto 0 auto;
+      border: 1px solid #ccc;
+      border-right-color: #999;
+      border-bottom-color: #999;
+    }
+    h1 { font-size: 100%; color: #f00; line-height: 1.5em; }
+  </style>
+</head>
+<body>
+  <!-- This file lives in public/422.html -->
+  <div class="dialog">
+    <h1>The change you wanted was rejected.</h1>
+    <p>Maybe you tried to change something you didn't have access to.</p>
+  </div>
+</body>
+</html>

data/spec/dummy/public/500.html ADDED Viewed

@@ -0,0 +1,25 @@
+<!DOCTYPE html>
+<html>
+<head>
+  <title>We're sorry, but something went wrong (500)</title>
+  <style type="text/css">
+    body { background-color: #fff; color: #666; text-align: center; font-family: arial, sans-serif; }
+    div.dialog {
+      width: 25em;
+      padding: 0 4em;
+      margin: 4em auto 0 auto;
+      border: 1px solid #ccc;
+      border-right-color: #999;
+      border-bottom-color: #999;
+    }
+    h1 { font-size: 100%; color: #f00; line-height: 1.5em; }
+  </style>
+</head>
+<body>
+  <!-- This file lives in public/500.html -->
+  <div class="dialog">
+    <h1>We're sorry, but something went wrong.</h1>
+  </div>
+</body>
+</html>

data/spec/dummy/public/favicon.ico ADDED Viewed

File without changes

data/spec/dummy/script/rails ADDED Viewed

@@ -0,0 +1,6 @@
+#!/usr/bin/env ruby
+# This command will automatically be run when you run "rails" with Rails 3 gems installed from the root of your application.
+APP_PATH = File.expand_path('../../config/application',  __FILE__)
+require File.expand_path('../../config/boot',  __FILE__)
+require 'rails/commands'

data/spec/dummy/tmp/cache/assets/CA6/7A0/sprockets%2Ff6723fb0918cb134f62480f155a0b364 ADDED Viewed

Binary file

data/spec/dummy/tmp/cache/assets/CD8/370/sprockets%2F357970feca3ac29060c1e3861e2c0953 ADDED Viewed

Binary file

data/spec/dummy/tmp/cache/assets/CF8/AF0/sprockets%2F228f63c526665cc53d670799eece3b98 ADDED Viewed

Binary file

data/spec/dummy/tmp/cache/assets/D09/190/sprockets%2F993fae97313ed7d42322c85e60b02bb1 ADDED Viewed

Binary file

data/spec/dummy/tmp/cache/assets/D1C/CE0/sprockets%2F9a5f4484900982bd33f792ca06e7f5ea ADDED Viewed

Binary file

data/spec/dummy/tmp/cache/assets/D31/620/sprockets%2F01f101ae68348ed1bb4e46d7f452a73a ADDED Viewed

Binary file

data/spec/dummy/tmp/cache/assets/D32/A10/sprockets%2F13fe41fee1fe35b49d145bcc06610705 ADDED Viewed

Binary file

data/spec/dummy/tmp/cache/assets/D3C/300/sprockets%2Ffb893496ed88af814fa985858f60ee66 ADDED Viewed

Binary file

data/spec/dummy/tmp/cache/assets/D43/EB0/sprockets%2F43d74a38cb4f14de5087ea7f871ac166 ADDED Viewed

Binary file

data/spec/dummy/tmp/cache/assets/D4E/1B0/sprockets%2Ff7cbd26ba1d28d48de824f0e94586655 ADDED Viewed

Binary file

data/spec/dummy/tmp/cache/assets/D5A/EA0/sprockets%2Fd771ace226fc8215a3572e0aa35bb0d6 ADDED Viewed

Binary file

data/spec/dummy/tmp/cache/assets/D97/C50/sprockets%2F74b5c193fc06fb3cb3c4ac12ac7589a7 ADDED Viewed

Binary file

data/spec/dummy/tmp/cache/assets/D9E/470/sprockets%2F36bde6f05aa955c51ab2cd97b59e742a ADDED Viewed

Binary file

data/spec/dummy/tmp/cache/assets/DCF/000/sprockets%2Fcfbeb9f4e8b3b7d1ac752d93c64017d3 ADDED Viewed

Binary file

data/spec/dummy/tmp/cache/assets/DDC/400/sprockets%2Fcffd775d018f68ce5dba1ee0d951a994 ADDED Viewed

Binary file

data/spec/dummy/tmp/cache/assets/E04/890/sprockets%2F2f5173deea6c795b8fdde723bb4b63af ADDED Viewed

Binary file

data/spec/dummy/tmp/cache/assets/test/sprockets/01f101ae68348ed1bb4e46d7f452a73a ADDED Viewed

Binary file

data/spec/dummy/tmp/cache/assets/test/sprockets/13fe41fee1fe35b49d145bcc06610705 ADDED Viewed

Binary file

data/spec/dummy/tmp/cache/assets/test/sprockets/228f63c526665cc53d670799eece3b98 ADDED Viewed

Binary file

data/spec/dummy/tmp/cache/assets/test/sprockets/2f5173deea6c795b8fdde723bb4b63af ADDED Viewed

Binary file

data/spec/dummy/tmp/cache/assets/test/sprockets/357970feca3ac29060c1e3861e2c0953 ADDED Viewed

Binary file

data/spec/dummy/tmp/cache/assets/test/sprockets/36bde6f05aa955c51ab2cd97b59e742a ADDED Viewed

Binary file

data/spec/dummy/tmp/cache/assets/test/sprockets/43d74a38cb4f14de5087ea7f871ac166 ADDED Viewed

Binary file

data/spec/dummy/tmp/cache/assets/test/sprockets/74b5c193fc06fb3cb3c4ac12ac7589a7 ADDED Viewed

Binary file

data/spec/dummy/tmp/cache/assets/test/sprockets/993fae97313ed7d42322c85e60b02bb1 ADDED Viewed

Binary file

data/spec/dummy/tmp/cache/assets/test/sprockets/9a5f4484900982bd33f792ca06e7f5ea ADDED Viewed

Binary file

data/spec/dummy/tmp/cache/assets/test/sprockets/cfbeb9f4e8b3b7d1ac752d93c64017d3 ADDED Viewed

Binary file

data/spec/dummy/tmp/cache/assets/test/sprockets/cffd775d018f68ce5dba1ee0d951a994 ADDED Viewed

Binary file

data/spec/dummy/tmp/cache/assets/test/sprockets/d771ace226fc8215a3572e0aa35bb0d6 ADDED Viewed

Binary file

data/spec/dummy/tmp/cache/assets/test/sprockets/f6723fb0918cb134f62480f155a0b364 ADDED Viewed

Binary file

data/spec/dummy/tmp/cache/assets/test/sprockets/f7cbd26ba1d28d48de824f0e94586655 ADDED Viewed

Binary file

data/spec/dummy/tmp/cache/assets/test/sprockets/fb893496ed88af814fa985858f60ee66 ADDED Viewed

Binary file

data/spec/factories.rb ADDED Viewed

@@ -0,0 +1,28 @@
+require 'spec_helper'
+FactoryGirl.define do
+  factory :user do
+    name 'John Doe'
+  end
+  factory :page do
+    user
+    title    'My Big Fat Greek Page'
+    content  <<-EOS
+Lorem ipsum dolor sit amet, consectetur adipisicing elit, sed do eiusmod tempor incididunt
+ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco
+laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in
+voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat
+cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.
+EOS
+    approved    false
+    page_type   'Front'
+    position    'Left'
+  end
+  factory :subpage do
+    page
+    subtitle  'My Not-so-big Subtitle'
+    content   'This is some content that goes in here.  Not a lot.'
+  end
+end

data/spec/features/worth_saving_features_spec.rb ADDED Viewed

@@ -0,0 +1,60 @@
+require 'spec_helper'
+feature 'WorthSaving' do
+  let(:fred) { given_user 'Fred Flintstone' }
+  let(:barney) { given_user 'Barney Rubble' }
+  scenario "A user begins work on a new page and sees recovery option when refreshing the page", js: true do
+    login fred
+    visit new_page_path
+    within '.worth-saving-standard-form-container .worth-saving-form-container' do
+      fill_in_form
+    end
+    chill_out_long_enough_to_draft
+    visit current_path
+    within '.worth-saving-header-message' do
+      expect(page).to have_content "It appears you were working on a draft that didn't get saved"
+    end
+    within '.worth-saving-recovery-form-container #new_page' do
+      verify_blank_form
+    end
+    within '.worth-saving-recovery-form-container #recovered_new_page' do
+      verify_form({}, true)
+    end
+  end
+  scenario "As a user I start work on a new page, refresh and see draft, but another user can't see it", js: true do
+    login fred
+    visit new_page_path
+    within '.worth-saving-standard-form-container .worth-saving-form-container' do
+      fill_in_form
+    end
+    chill_out_long_enough_to_draft
+    visit current_path
+    within '.worth-saving-header-message' do
+      expect(page).to have_content "It appears you were working on a draft that didn't get saved"
+    end
+    login barney
+    visit new_page_path
+    page.should_not have_css '.worth-saving-header-message'
+  end
+  scenario "One user creates a page.  Another edits that page, and drafts do not get saved because of methods in ApplicationController", js: true do
+    login fred
+    visit new_page_path
+    within '.worth-saving-standard-form-container .worth-saving-form-container' do
+      fill_in_form
+    end
+    click_on 'Create Page'
+    path = current_path
+    login barney
+    visit path
+    within '.worth-saving-standard-form-container .worth-saving-form-container' do
+      fill_in 'Content', with: "Here is some new content that I will not expect to see"
+    end
+    chill_out_long_enough_to_draft
+    within '.worth-saving-form-message.error' do
+      page.should have_content "Not authorized to draft this Page"
+    end
+  end
+end

data/spec/lib/active_record_ext_spec.rb ADDED Viewed

@@ -0,0 +1,291 @@
+require 'spec_helper'
+describe WorthSaving::ActiveRecordExt do
+  before do
+    Rails.application.config.stub(:filter_parameters).and_return [:redacted_field]
+  end
+  with_model :WorthSavingDraft do
+    table do |t|
+      t.integer   :recordable_id
+      t.string    :recordable_type
+      t.integer   :scopeable_id
+      t.string    :scopeable_type
+      t.text      :form_data
+    end
+    model do
+      belongs_to :recordable, polymorphic: true
+      belongs_to :scopeable, polymorphic: true
+    end
+  end
+  with_model :ImportantThing do
+    table do |t|
+      t.string :stuff
+    end
+    model do
+      is_worth_saving
+      if Rails.version.match /^3/
+        attr_accessible :stuff
+      end
+    end
+  end
+  with_model :AnyThing do
+  end
+  before do
+    if Rails.version.match /^3/
+      class WorthSaving::Draft
+        attr_accessible :recordable_id, :recordable_type, :scopeable_id, :scopeable_type, :form_data, :scopeable
+      end
+    end
+  end
+  describe 'subclass that is worth_saving' do
+    describe 'associations' do
+      let(:record) { ImportantThing.new }
+      let(:draft) { record.build_worth_saving_draft }
+      context 'when record is persisted' do
+        it 'has a worth_saving draft' do
+          record.save
+          draft.save
+          record.worth_saving_draft.should eq draft
+        end
+      end
+      context 'when record is not persisted' do
+        it 'has a worth_saving draft' do
+          draft.save
+          record.worth_saving_draft.should eq draft
+        end
+      end
+    end
+    describe 'knowing it is worth_saving' do
+      subject { ImportantThing.worth_saving? }
+      it { should be_true }
+    end
+    describe 'instance knowing it is worth_saving' do
+      subject { ImportantThing.new.worth_saving? }
+      it { should eq true }
+    end
+  end
+  describe 'normal(not-worth_saving) subclass' do
+    describe 'knowing it is NOT worth_saving' do
+      subject { AnyThing.worth_saving? }
+      it { should eq false }
+    end
+    describe 'instance knowing it is NOT worth_saving' do
+      subject { AnyThing.new.worth_saving? }
+      it { should eq false }
+    end
+  end
+  context 'worth_saving subclass with excepted fields' do
+    with_model :ImportantThingButUnimportantAuthor do
+      table do |t|
+        t.string  :title
+        t.string  :author
+      end
+      model do
+        is_worth_saving except: :author
+      end
+    end
+    context 'when field is NOT in the exceptions list' do
+      describe 'knowing the field is worth_saving' do
+        subject { ImportantThingButUnimportantAuthor.worth_saving? :title }
+        it { should eq true }
+      end
+      describe 'having an instance that knows the field is worth_saving' do
+        subject { ImportantThingButUnimportantAuthor.new.worth_saving? :title }
+        it { should eq true }
+      end
+    end
+    context 'when a field IS in the exceptions list' do
+      describe 'knowing the field is NOT worth_saving' do
+        subject { ImportantThingButUnimportantAuthor.worth_saving? :author }
+        it { should eq false }
+      end
+      describe 'having an instance that knows the field is NOT worth_saving' do
+        subject { ImportantThingButUnimportantAuthor.new.worth_saving? :author }
+        it { should eq false }
+      end
+    end
+    context 'when multiple fields are in the exceptions list (array argument)' do
+      with_model :ImportantThingWithUnimportantFields do
+        table do |t|
+          t.string  :title
+          t.string  :author
+          t.string  :content
+        end
+        model do
+          is_worth_saving except: [:author, :title]
+        end
+      end
+      describe 'knowing which fields are and are NOT worth_saving' do
+        it 'should be able to tell the difference' do
+          ImportantThingWithUnimportantFields.worth_saving?(:author).should eq false
+          ImportantThingWithUnimportantFields.worth_saving?(:title).should eq false
+          ImportantThingWithUnimportantFields.worth_saving?(:content).should eq true
+        end
+        context 'Rails.application.filter_params' do
+          it 'should not consider these fields worth_saving' do
+            ImportantThingWithUnimportantFields.worth_saving?(:redacted_field).should eq false
+          end
+        end
+      end
+    end
+  end
+  describe 'scope option' do
+    with_model :User do
+    end
+    with_model :ScopedThing do
+      table do |t|
+        t.belongs_to :user
+      end
+      model do
+        if Rails.version.match /^3/
+          attr_accessible :user
+        end
+        belongs_to :user
+        is_worth_saving scope: :user
+      end
+    end
+    describe '.worth_saving_scope' do
+      it 'returns scope given in options' do
+        ScopedThing.worth_saving_info.scope.should eq :user
+      end
+    end
+    describe '.worth_saving_info.scope' do
+      it 'returns scope given in options' do
+        ScopedThing.worth_saving_info.scope.should eq :user
+      end
+    end
+    describe '#worth_saving_draft' do
+      let(:user) { User.create }
+      let(:user2) { User.create }
+      let(:thing) { ScopedThing.new user: user }
+      context 'an unsaved ScopedThing' do
+        it 'finds the draft by scope' do
+          draft = WorthSaving::Draft.create recordable_type: 'ScopedThing', scopeable_id: user.id, scopeable_type: 'User'
+          thing.worth_saving_draft.should eq draft
+        end
+        it 'does not find the draft belonging to the wrong scoped object' do
+          draft = WorthSaving::Draft.create recordable_type: 'ScopedThing', scopeable_id: user.id, scopeable_type: 'User'
+          thing = ScopedThing.new user: user2
+          thing.worth_saving_draft.should be_nil
+        end
+        it 'does not find the draft belonging to the wrong recordable' do
+          draft = WorthSaving::Draft.create recordable_type: 'ImportantThing', scopeable_id: user.id, scopeable_type: 'User'
+          thing = ScopedThing.new user: user
+          thing.worth_saving_draft.should be_nil
+        end
+      end
+      context 'a saved Scopedthing' do
+        it 'finds the draft through the has_one association' do
+          thing.save
+          draft = WorthSaving::Draft.create recordable_id: thing.id, recordable_type: 'ScopedThing'
+          thing.reload
+          thing.worth_saving_draft.should eq draft
+        end
+      end
+      context "an unsaved unscoped thing" do
+        subject { ImportantThing.new }
+        let(:thing) { ImportantThing.new }
+        let(:draft) { thing.build_worth_saving_draft }
+        before do
+          draft.save
+        end
+        its(:worth_saving_draft) { should eq draft }
+      end
+    end
+    describe '#build_worth_saving_draft' do
+      let(:user) { User.create }
+      let(:thing) { ScopedThing.new user: user }
+      context 'an unsaved ScopedThing' do
+        subject { thing.build_worth_saving_draft }
+        its(:recordable_type) { should eq 'ScopedThing' }
+        its(:recordable_id) { should be_nil }
+        its(:scopeable) { should eq user }
+        describe 'disallowed attribute override' do
+          subject { thing.build_worth_saving_draft recordable_type: 'WrongClass', scopeable_id: 4444 }
+          its(:recordable_type) { should eq 'ScopedThing' }
+          its(:scopeable) { should eq user }
+        end
+      end
+      context 'an unsaved ScopedThing' do
+        let(:thing) { ScopedThing.create user: user }
+        subject { thing.build_worth_saving_draft form_data: 'some info' }
+        its(:recordable) { should eq thing }
+        its(:form_data) { should eq 'some info' }
+      end
+    end
+    describe 'destruction of worth_saving_draft during save' do
+      let(:thing) { ImportantThing.new stuff: 'Something' }
+      context 'new record' do
+        it 'happens after but not before' do
+          draft = thing.create_worth_saving_draft
+          WorthSaving::Draft.find(draft.id).should eq draft
+          thing.save
+          -> { WorthSaving::Draft.find(draft.id).should }.should raise_error ActiveRecord::RecordNotFound
+        end
+      end
+      context 'persisted record' do
+        it 'happens after but not before' do
+          thing.save
+          draft = thing.create_worth_saving_draft
+          WorthSaving::Draft.find(draft.id).should eq draft
+          thing.update_attributes stuff: 'Changed something'
+          -> { WorthSaving::Draft.find(draft.id).should }.should raise_error ActiveRecord::RecordNotFound
+        end
+      end
+    end
+  end
+end