RubyGems - worth_saving - Versions diffs - 0.0.9 - Mend

worth_saving 0.0.9

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

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