paper_trail_scrapbook 0.1.9 → 0.1.10

data/spec/paper_trail_scrapbook/secondary_chapter_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 module PaperTrailScrapbook

data/spec/paper_trail_scrapbook/user_journal_spec.rb

@@ -1,45 +1,47 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 module PaperTrailScrapbook
   RSpec.describe UserJournal do
     before do
       PaperTrailScrapbook.config.whodunnit_class = Person
-      PaperTrail.request.whodunnit = person.id
+      PaperTrail.request.whodunnit               = person.id
 
       a_ship
     end
 
-    let(:name) {'The Tim Man'}
-    let(:title) {'How the Grinch stole Xmas'}
-    let(:a_name) {'Dr. Seuss'}
-    let(:changes) {'made the following changes:'}
-    let(:b_changes) {'made the following Book changes:'}
+    let(:name) { 'The Tim Man' }
+    let(:title) { 'How the Grinch stole Xmas' }
+    let(:a_name) { 'Dr. Seuss' }
+    let(:changes) { 'made the following changes:' }
+    let(:b_changes) { 'made the following Book changes:' }
 
-    let(:book) {Book.create!(title: title)}
-    let(:author) {Person.create!(name: a_name)}
-    let(:a_ship) {Authorship.create!(book: book, author: author)}
+    let(:book) { Book.create!(title: title) }
+    let(:author) { Person.create!(name: a_name) }
+    let(:a_ship) { Authorship.create!(book: book, author: author) }
     let(:person) do
-      who = PaperTrail.request.whodunnit
+      who                          = PaperTrail.request.whodunnit
       PaperTrail.request.whodunnit = nil
-      p = Person.new(name: name)
+      p                            = Person.new(name: name)
       p.save!
       PaperTrail.request.whodunnit = who
       p
     end
-    let(:format) {PaperTrailScrapbook.config.time_format}
-    let(:object) {described_class.new(person)}
-    let(:subject) {object.story}
-    let(:f_starts) {starts.strftime(format).squeeze(' ')}
-    let(:f_ends) {ends.strftime(format).squeeze(' ')}
+    let(:format) { PaperTrailScrapbook.config.time_format }
+    let(:object) { described_class.new(person) }
+    let(:subject) { object.story }
+    let(:f_starts) { starts.strftime(format).squeeze(' ') }
+    let(:f_ends) { ends.strftime(format).squeeze(' ') }
 
     describe '#story' do
       context 'with a provided user' do
         it 'provides a whole story' do
-          #expect(object.starts).to eql 'Thursday, 01 Jan 1970 at 12:00 AM'
-          #expect(object.ends).to eql Time.current.in_time_zone
+          # expect(object.starts).to eql 'Thursday, 01 Jan 1970 at 12:00 AM'
+          # expect(object.ends).to eql Time.current.in_time_zone
 
           expect(subject)
-              .to match(/Between .* and .*, #{name} #{changes}/)
+            .to match(/Between .* and .*, #{name} #{changes}/)
           expect(subject).to match(/On .*, created Book\[#{book.id}\]:/)
           expect(subject).to match(/ • title: #{title}/)
           expect(subject).to match(/On .*, created Person\[#{author.id}\]:/)
@@ -74,43 +76,43 @@ module PaperTrailScrapbook
       end
 
       context 'with a provided class' do
-        let(:object) {described_class.new(person, klass: Book)}
+        let(:object) { described_class.new(person, klass: Book) }
 
         it 'provides a story for provided class' do
           expect(subject)
-              .to match(/Between .* and .*, #{name} #{b_changes}/)
+            .to match(/Between .* and .*, #{name} #{b_changes}/)
           expect(subject).to match(/On .*, created Book\[#{book.id}\]:/)
           expect(subject).to match(/ • title: #{title}/)
           expect(subject).not_to match(/On .*, created Person\[#{author.id}\]:/)
           expect(subject).not_to match(/ • name: #{a_name}/)
           expect(subject)
-              .not_to match(/On .*, created Authorship\[#{a_ship.id}\]:/)
+            .not_to match(/On .*, created Authorship\[#{a_ship.id}\]:/)
         end
       end
 
       context 'with start time and class but no end time' do
-        let(:object) {described_class.new(person, klass: Book, start: starts)}
-        let(:starts) {Time.current.advance(minutes: -5)}
+        let(:object) { described_class.new(person, klass: Book, start: starts) }
+        let(:starts) { Time.current.advance(minutes: -5) }
 
         it 'provides a story with start time' do
           expect(subject)
-              .to match(/Between #{f_starts} and .*, #{name} #{b_changes}/)
+            .to match(/Between #{f_starts} and .*, #{name} #{b_changes}/)
           expect(subject).to match(/On .*, created Book\[#{book.id}\]:/)
           expect(subject).to match(/ • title: #{title}/)
           expect(subject).not_to match(/On .*, created Person\[#{author.id}\]:/)
           expect(subject).not_to match(/ • name: #{a_name}/)
           expect(subject)
-              .not_to match(/On .*, created Authorship\[#{a_ship.id}\]:/)
+            .not_to match(/On .*, created Authorship\[#{a_ship.id}\]:/)
         end
       end
 
       context 'with end time but no start time and no class' do
-        let(:ends) {Time.current.advance(minutes: 5)}
-        let(:object) {described_class.new(person, end: ends)}
+        let(:ends) { Time.current.advance(minutes: 5) }
+        let(:object) { described_class.new(person, end: ends) }
 
         it 'provides a story with end time in the future' do
           expect(subject)
-              .to match(/Between .* and #{f_ends}, #{name} #{changes}/)
+            .to match(/Between .* and #{f_ends}, #{name} #{changes}/)
           expect(subject).to match(/On .*, created Book\[#{book.id}\]:/)
           expect(subject).to match(/ • title: #{title}/)
           expect(subject).to match(/On .*, created Person\[#{author.id}\]:/)
@@ -119,7 +121,7 @@ module PaperTrailScrapbook
         end
 
         describe 'with end time in past' do
-          let(:ends) {Time.current.advance(minutes: -5)}
+          let(:ends) { Time.current.advance(minutes: -5) }
 
           it 'provides a story with end time in the past' do
             expect(subject).not_to match(/#{name} created the following Person/)
@@ -131,13 +133,13 @@ module PaperTrailScrapbook
       end
 
       context 'with start and end times but no class' do
-        let(:starts) {Time.current.advance(minutes: -4)}
-        let(:ends) {starts.advance(hours: 1)}
-        let(:object) {described_class.new(person, start: starts, end: ends)}
+        let(:starts) { Time.current.advance(minutes: -4) }
+        let(:ends) { starts.advance(hours: 1) }
+        let(:object) { described_class.new(person, start: starts, end: ends) }
 
         it 'provides a story with start and end times' do
           expect(subject)
-              .to match(/Between .* and .*, #{name} #{changes}/)
+            .to match(/Between .* and .*, #{name} #{changes}/)
           expect(subject).to match(/On .*, created Book\[#{book.id}\]:/)
           expect(subject).to match(/ • title: #{title}/)
           expect(subject).to match(/On .*, created Person\[#{author.id}\]:/)
@@ -148,8 +150,8 @@ module PaperTrailScrapbook
       end
 
       context 'with start time, end time, and a class' do
-        let(:starts) {Time.current.advance(minutes: -4)}
-        let(:ends) {starts.advance(hours: 1)}
+        let(:starts) { Time.current.advance(minutes: -4) }
+        let(:ends) { starts.advance(hours: 1) }
         let(:object) do
           described_class.new(person, klass: Book, start: starts, end: ends)
         end
@@ -157,19 +159,19 @@ module PaperTrailScrapbook
         describe 'when time range covers current time' do
           it 'provides a story for provided class with start and end times' do
             expect(subject)
-                .to match(/Between #{f_starts} and #{f_ends}, #{name} #{b_changes}/)
+              .to match(/Between #{f_starts} and #{f_ends}, #{name} #{b_changes}/)
             expect(subject).to match(/On .*, created Book\[#{book.id}\]:/)
             expect(subject).to match(/ • title: #{title}/)
             expect(subject)
-                .not_to match(/On .*, created Person\[#{author.id}\]:/)
+              .not_to match(/On .*, created Person\[#{author.id}\]:/)
             expect(subject).not_to match(/ • name: #{a_name}/)
             expect(subject)
-                .not_to match(/On .*, created Authorship\[#{a_ship.id}\]:/)
+              .not_to match(/On .*, created Authorship\[#{a_ship.id}\]:/)
           end
         end
 
         describe 'when time range is completely in the future' do
-          let(:starts) {Time.current.advance(minutes: 4)}
+          let(:starts) { Time.current.advance(minutes: 4) }
 
           it 'provides a story' do
             expect(subject).to eql("Between #{f_starts} and #{f_ends}, #{name}"\
@@ -178,10 +180,12 @@ module PaperTrailScrapbook
         end
 
         describe 'when no time range' do
-          let(:starts) {nil}
-          let(:ends) {nil}
+          let(:starts) { nil }
+          let(:ends) { nil }
           it 'provides a story' do
-            expect(subject).to match("Between Thursday, 01 Jan 1970 at 12:00 AM and #{Time.current.in_time_zone.strftime(format).squeeze(' ')}, #{name}"\
+            expect(subject)
+              .to match('Between Thursday, 01 Jan 1970 at 12:00 AM ' \
+                          "and #{Time.current.in_time_zone.strftime(format).squeeze(' ')}, #{name}"\
                                      " #{b_changes}\n\n".squeeze(' '))
           end
         end

data/spec/paper_trail_scrapbook/version_helpers_spec.rb

@@ -1,10 +1,12 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 require 'ostruct'
 
 module PaperTrailScrapbook
   RSpec.describe VersionHelpers do
-    let(:person) {Person.create!(name: 'The Tim Man')}
-    let(:book) {Book.create!(title: 'How the Grinch stole Xmas')}
+    let(:person) { Person.create!(name: 'The Tim Man') }
+    let(:book) { Book.create!(title: 'How the Grinch stole Xmas') }
     let(:version) do
       OpenStruct.new(event: 'create',
                      item_type: 'Book',
@@ -12,8 +14,8 @@ module PaperTrailScrapbook
                      created_at: Time.current,
                      version_author: person.id)
     end
-    let(:config) {PaperTrailScrapbook.config}
-    let(:subject) {JournalEntry.new(version)}
+    let(:config) { PaperTrailScrapbook.config }
+    let(:subject) { JournalEntry.new(version) }
 
     before do
       PaperTrailScrapbook.config.whodunnit_class = Person
@@ -59,7 +61,7 @@ module PaperTrailScrapbook
     describe '#whenn' do
       it 'returns version created_at' do
         expect(subject.whenn)
-            .to eql(version.created_at.strftime(config.time_format))
+          .to eql(version.created_at.strftime(config.time_format))
       end
     end
 
@@ -69,12 +71,12 @@ module PaperTrailScrapbook
       end
 
       context 'garbage' do
-        let(:subject) {JournalEntry.new(version)}
+        let(:subject) { JournalEntry.new(version) }
         it 'returns expected value' do
           x = PaperTrailScrapbook.config.events
           PaperTrailScrapbook.config.events = {}
 
-          expect {subject.kind}.to raise_error(ArgumentError, 'incorrect event:create')
+          expect { subject.kind }.to raise_error(ArgumentError, 'incorrect event:create')
 
           PaperTrailScrapbook.config.events = x
         end

data/spec/paper_trail_scrapbook/version_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 module PaperTrailScrapbook

data/spec/paper_trail_scrapbook_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 module PaperTrailScrapbook

data/spec/spec_helper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # This file is copied to spec/ when you run 'rails generate rspec:install'
 ENV['RAILS_ENV'] ||= 'test'
 ENV['DB'] ||= 'sqlite'
@@ -54,7 +56,7 @@ def params_wrapper(args)
   end
 end
 
-require File.expand_path('../dummy_app/config/environment', __FILE__)
+require File.expand_path('dummy_app/config/environment', __dir__)
 require 'rspec/rails'
 require 'ffaker'
 require 'timecop'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: paper_trail_scrapbook
 version: !ruby/object:Gem::Version
-  version: 0.1.9
+  version: 0.1.10
 platform: ruby
 authors:
 - Timothy Chambers
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-07-11 00:00:00.000000000 Z
+date: 2018-07-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -218,6 +218,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '3.6'
+- !ruby/object:Gem::Dependency
+  name: rspec_junit_formatter
+  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: rubocop
   requirement: !ruby/object:Gem::Requirement
@@ -280,11 +294,13 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".circleci/config.yml"
 - ".codeclimate.yml"
 - ".gitignore"
 - ".rspec"
 - ".rubocop.yml"
 - ".ruby-gemset"
+- ".ruby-version"
 - CODE_OF_CONDUCT.md
 - Changelog.md
 - Gemfile
@@ -292,7 +308,6 @@ files:
 - LICENSE.md
 - README.md
 - Rakefile
-- circle.yml
 - lib/paper_trail_scrapbook.rb
 - lib/paper_trail_scrapbook/changes.rb
 - lib/paper_trail_scrapbook/chapter.rb
@@ -401,7 +416,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.2.0
+      version: 2.3.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
@@ -409,7 +424,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: 1.3.6
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.7.6
+rubygems_version: 2.5.2.3
 signing_key: 
 specification_version: 4
 summary: Paper Trail Scrapbook

data/circle.yml

@@ -1,22 +0,0 @@
----
-machine:
-  ruby:
-    version: '2.3.0'
-  environment:
-    CC_TEST_REPORTER_ID: e0db6fd6beb8208980b2908947735982fb5a4e8a9a3cb742dc194e00df79f388
-
-dependencies:
-  pre:
-    - gem install bundler
-
-database:
-  override:
-    - rake -T -f spec/dummy_app/Rakefile
-    - bundle exec rake -f spec/dummy_app/Rakefile db:setup
-    
-test:
-  override:
-    - bundle exec rake spec
-    - CODECLIMATE_REPO_TOKEN=e0db6fd6beb8208980b2908947735982fb5a4e8a9a3cb742dc194e00df79f388 bundle exec codeclimate-test-reporter
-
-