historiographer 2.0.1 → 3.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cde507fc3738878242d8f3f9c6fe7c86e6c5346077da3d63882a04fc92f0ba94
-  data.tar.gz: 0d8afa8b8d24dd870a4e4c26e322c3d64cd2c7a58e04651382a20e7da0d7b1c5
+  metadata.gz: 193d6326a6d665c589bcf2c67c16434b3cbf6e4bacd1b1d2f28f80d6ce3b229c
+  data.tar.gz: 1e09d0ecf55bf816d444376fc11cbf00a38a8c9f2a2c6ebbbf62f6a7d173282d
 SHA512:
-  metadata.gz: 0d139a03f5f2ff866de63981475804033dffb3ec5b67e4af7ed24241fc177bca89af8cc48806913094091478b22c2f64b88e85ad93c2692e7afec02c261e7977
-  data.tar.gz: d398de437246001483d25630aac20dfcf0652fec518b2345d22eedb2f3956a7b17125f8a8803b1337075e0097e72057e8619329d93a3a2c144121d7bca180347
+  metadata.gz: 32854e876a7a557c8c0c7008b93cfaf33bd5e06748c97f86890cdcfc269b5baef958555ada0c49c551f035b9b655c41a7036debb908347e028554ec1c55ea5fb
+  data.tar.gz: 3a27716e6159f01260bbb682309d9a82b1bf261cb3ba3eae13f38d8265894a99db938c1473b24a465c63ae24dbf27d0a4d7d66c9a823bf5e16ed0ce6fd8cab6a

data/Gemfile

@@ -1,31 +1,33 @@
-source "https://rubygems.org"
-ruby "2.6.3"
+# frozen_string_literal: true
 
-gem "activerecord", ">= 5"
-gem "activesupport"
-gem "rollbar"
-gem "activerecord-import"
+source 'https://rubygems.org'
+ruby '2.6.3'
+
+gem 'activerecord', '>= 6'
+gem 'activerecord-import'
+gem 'activesupport'
+gem 'rollbar'
 
 group :development, :test do
-  gem "pg"
-  gem "pry"
-  gem "mysql2", "0.5"
-  gem "standalone_migrations"
-  gem "timecop"
-  gem "paranoia"
+  gem 'mysql2', '0.5'
+  gem 'paranoia'
+  gem 'pg'
+  gem 'pry'
+  gem 'standalone_migrations'
+  gem 'timecop'
 end
 
 group :development do
-  gem "rdoc", "~> 3.12"
-  gem "bundler", "~> 1.0"
-  gem "jeweler", git: "https://github.com/technicalpickles/jeweler", branch: "master"
-  gem "simplecov", ">= 0"
+  gem 'bundler', '~> 1.0'
+  gem 'jeweler', git: 'https://github.com/technicalpickles/jeweler', branch: 'master'
+  gem 'rdoc', '~> 3.12'
+  gem 'simplecov', '>= 0'
 end
 
 group :test do
-  gem "rspec"
-  gem "guard"
-  gem "guard-rspec"
-  gem "database_cleaner"
-  gem "factory_bot_rails"
+  gem 'database_cleaner'
+  gem 'factory_bot_rails'
+  gem 'guard'
+  gem 'guard-rspec'
+  gem 'rspec'
 end

data/Gemfile.lock

@@ -47,7 +47,7 @@ GEM
     addressable (2.4.0)
     builder (3.2.4)
     coderay (1.1.3)
-    concurrent-ruby (1.1.9)
+    concurrent-ruby (1.1.10)
     crass (1.0.6)
     database_cleaner (2.0.1)
       database_cleaner-active_record (~> 2.0.0)
@@ -69,7 +69,7 @@ GEM
       multipart-post (>= 1.2, < 3)
     ffi (1.15.5)
     formatador (1.1.0)
-    git (1.10.2)
+    git (1.11.0)
       rchardet (~> 1.8)
     github_api (0.16.0)
       addressable (~> 2.4.0)
@@ -94,10 +94,10 @@ GEM
       rspec (>= 2.99.0, < 4.0)
     hashie (5.0.0)
     highline (2.0.3)
-    i18n (1.9.1)
+    i18n (1.12.0)
       concurrent-ruby (~> 1.0)
     json (1.8.6)
-    jwt (2.3.0)
+    jwt (2.5.0)
     listen (3.7.1)
       rb-fsevent (~> 0.10, >= 0.10.3)
       rb-inotify (~> 0.9, >= 0.9.10)
@@ -107,21 +107,21 @@ GEM
     lumberjack (1.2.8)
     method_source (1.0.0)
     mime-types (2.99.3)
-    mini_portile2 (2.7.1)
-    minitest (5.15.0)
+    mini_portile2 (2.8.0)
+    minitest (5.16.3)
     multi_json (1.15.0)
     multi_xml (0.6.0)
-    multipart-post (2.1.1)
+    multipart-post (2.2.3)
     mysql2 (0.5.0)
     nenv (0.3.0)
-    nokogiri (1.13.1)
-      mini_portile2 (~> 2.7.0)
+    nokogiri (1.13.8)
+      mini_portile2 (~> 2.8.0)
       racc (~> 1.4)
     notiffany (0.1.3)
       nenv (~> 0.1)
       shellany (~> 0.0)
-    oauth2 (1.4.7)
-      faraday (>= 0.8, < 2.0)
+    oauth2 (1.4.8)
+      faraday (>= 0.8, < 3.0)
       jwt (>= 1.0, < 3.0)
       multi_json (~> 1.3)
       multi_xml (~> 0.5)
@@ -132,10 +132,10 @@ GEM
     pry (0.14.1)
       coderay (~> 1.1)
       method_source (~> 1.0)
-    psych (4.0.3)
+    psych (4.0.6)
       stringio
     racc (1.6.0)
-    rack (2.2.3)
+    rack (2.2.4)
     rack-test (1.1.0)
       rack (>= 1.0, < 3)
     rails-dom-testing (2.0.3)
@@ -182,19 +182,19 @@ GEM
       activerecord (>= 4.2.7, < 6.2.0, != 5.2.3.rc1, != 5.2.3)
       railties (>= 4.2.7, < 6.2.0, != 5.2.3.rc1, != 5.2.3)
       rake (>= 10.0)
-    stringio (3.0.1)
+    stringio (3.0.2)
     thor (1.2.1)
     thread_safe (0.3.6)
     timecop (0.9.4)
-    tzinfo (2.0.4)
+    tzinfo (2.0.5)
       concurrent-ruby (~> 1.0)
-    zeitwerk (2.5.4)
+    zeitwerk (2.6.1)
 
 PLATFORMS
   ruby
 
 DEPENDENCIES
-  activerecord (>= 5)
+  activerecord (>= 6)
   activerecord-import
   activesupport
   bundler (~> 1.0)

data/VERSION

@@ -1 +1 @@
-2.0.1
+3.1.0

data/historiographer.gemspec

@@ -2,16 +2,16 @@
 # DO NOT EDIT THIS FILE DIRECTLY
 # Instead, edit Jeweler::Tasks in Rakefile, and run 'rake gemspec'
 # -*- encoding: utf-8 -*-
-# stub: historiographer 2.0.1 ruby lib
+# stub: historiographer 3.1.0 ruby lib
 
 Gem::Specification.new do |s|
   s.name = "historiographer".freeze
-  s.version = "2.0.1"
+  s.version = "3.1.0"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0".freeze) if s.respond_to? :required_rubygems_version=
   s.require_paths = ["lib".freeze]
   s.authors = ["brettshollenberger".freeze]
-  s.date = "2022-02-04"
+  s.date = "2022-10-18"
   s.description = "Creates separate tables for each history table".freeze
   s.email = "brett.shollenberger@gmail.com".freeze
   s.extra_rdoc_files = [
@@ -40,6 +40,7 @@ Gem::Specification.new do |s|
     "lib/historiographer/postgres_migration.rb",
     "lib/historiographer/relation.rb",
     "lib/historiographer/safe.rb",
+    "lib/historiographer/silent.rb",
     "spec/db/database.yml",
     "spec/db/migrate/20161121212228_create_posts.rb",
     "spec/db/migrate/20161121212229_create_post_histories.rb",
@@ -51,6 +52,8 @@ Gem::Specification.new do |s|
     "spec/db/migrate/20191024142304_create_thing_with_compound_index.rb",
     "spec/db/migrate/20191024142352_create_thing_with_compound_index_history.rb",
     "spec/db/migrate/20191024203106_create_thing_without_history.rb",
+    "spec/db/migrate/20221018204220_create_silent_posts.rb",
+    "spec/db/migrate/20221018204255_create_silent_post_histories.rb",
     "spec/db/schema.rb",
     "spec/examples.txt",
     "spec/factories/post.rb",
@@ -67,34 +70,34 @@ Gem::Specification.new do |s|
   end
 
   if s.respond_to? :add_runtime_dependency then
-    s.add_runtime_dependency(%q<activerecord>.freeze, [">= 5"])
+    s.add_runtime_dependency(%q<activerecord>.freeze, [">= 6"])
+    s.add_runtime_dependency(%q<activerecord-import>.freeze, [">= 0"])
     s.add_runtime_dependency(%q<activesupport>.freeze, [">= 0"])
     s.add_runtime_dependency(%q<rollbar>.freeze, [">= 0"])
-    s.add_runtime_dependency(%q<activerecord-import>.freeze, [">= 0"])
+    s.add_development_dependency(%q<mysql2>.freeze, ["= 0.5"])
+    s.add_development_dependency(%q<paranoia>.freeze, [">= 0"])
     s.add_development_dependency(%q<pg>.freeze, [">= 0"])
     s.add_development_dependency(%q<pry>.freeze, [">= 0"])
-    s.add_development_dependency(%q<mysql2>.freeze, ["= 0.5"])
     s.add_development_dependency(%q<standalone_migrations>.freeze, [">= 0"])
     s.add_development_dependency(%q<timecop>.freeze, [">= 0"])
-    s.add_development_dependency(%q<paranoia>.freeze, [">= 0"])
-    s.add_development_dependency(%q<rdoc>.freeze, ["~> 3.12"])
     s.add_development_dependency(%q<bundler>.freeze, ["~> 1.0"])
     s.add_development_dependency(%q<jeweler>.freeze, [">= 0"])
+    s.add_development_dependency(%q<rdoc>.freeze, ["~> 3.12"])
     s.add_development_dependency(%q<simplecov>.freeze, [">= 0"])
   else
-    s.add_dependency(%q<activerecord>.freeze, [">= 5"])
+    s.add_dependency(%q<activerecord>.freeze, [">= 6"])
+    s.add_dependency(%q<activerecord-import>.freeze, [">= 0"])
     s.add_dependency(%q<activesupport>.freeze, [">= 0"])
     s.add_dependency(%q<rollbar>.freeze, [">= 0"])
-    s.add_dependency(%q<activerecord-import>.freeze, [">= 0"])
+    s.add_dependency(%q<mysql2>.freeze, ["= 0.5"])
+    s.add_dependency(%q<paranoia>.freeze, [">= 0"])
     s.add_dependency(%q<pg>.freeze, [">= 0"])
     s.add_dependency(%q<pry>.freeze, [">= 0"])
-    s.add_dependency(%q<mysql2>.freeze, ["= 0.5"])
     s.add_dependency(%q<standalone_migrations>.freeze, [">= 0"])
     s.add_dependency(%q<timecop>.freeze, [">= 0"])
-    s.add_dependency(%q<paranoia>.freeze, [">= 0"])
-    s.add_dependency(%q<rdoc>.freeze, ["~> 3.12"])
     s.add_dependency(%q<bundler>.freeze, ["~> 1.0"])
     s.add_dependency(%q<jeweler>.freeze, [">= 0"])
+    s.add_dependency(%q<rdoc>.freeze, ["~> 3.12"])
     s.add_dependency(%q<simplecov>.freeze, [">= 0"])
   end
 end

data/lib/historiographer/silent.rb

@@ -0,0 +1,29 @@
+# frozen_string_literal: true
+
+# Historiographer::Silent is intended to be used to migrate an existing model
+# to Historiographer, not as a long-term solution.
+#
+# Historiographer will throw an error if a model is saved without a user present,
+# unless you explicitly call save_without_history.
+#
+# Historiographer::Silent will not throw an error, and will not produce a Rollbar
+#
+module Historiographer
+  module Silent
+    extend ActiveSupport::Concern
+
+    included do
+      include Historiographer
+
+      def should_validate_history_user_id_present?
+        false
+      end
+
+      private
+
+      def history_user_absent_action
+        # noop
+      end
+    end
+  end
+end

data/spec/db/migrate/20221018204220_create_silent_posts.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+class CreateSilentPosts < ActiveRecord::Migration[5.1]
+  def change
+    create_table :silent_posts do |t|
+      t.string :title, null: false
+      t.text :body, null: false
+      t.integer :author_id, null: false
+      t.boolean :enabled, default: false
+      t.datetime :live_at
+      t.datetime :deleted_at
+
+      t.timestamps
+    end
+
+    add_index :silent_posts, :author_id
+    add_index :silent_posts, :enabled
+    add_index :silent_posts, :live_at
+    add_index :silent_posts, :deleted_at
+  end
+end

data/spec/db/migrate/20221018204255_create_silent_post_histories.rb

@@ -0,0 +1,9 @@
+# frozen_string_literal: true
+
+require 'historiographer/postgres_migration'
+
+class CreateSilentPostHistories < ActiveRecord::Migration[5.1]
+  def change
+    create_table :silent_post_histories, &:histories
+  end
+end

data/spec/db/schema.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 2019_10_24_203106) do
+ActiveRecord::Schema.define(version: 2022_10_18_204255) do
 
   # These are extensions that must be enabled in order to support this database
   enable_extension "plpgsql"
@@ -117,6 +117,44 @@ ActiveRecord::Schema.define(version: 2019_10_24_203106) do
     t.index ["live_at"], name: "index_safe_posts_on_live_at"
   end
 
+  create_table "silent_post_histories", force: :cascade do |t|
+    t.integer "silent_post_id", null: false
+    t.string "title", null: false
+    t.text "body", null: false
+    t.integer "author_id", null: false
+    t.boolean "enabled", default: false
+    t.datetime "live_at"
+    t.datetime "deleted_at"
+    t.datetime "created_at", null: false
+    t.datetime "updated_at", null: false
+    t.datetime "history_started_at", null: false
+    t.datetime "history_ended_at"
+    t.integer "history_user_id"
+    t.index ["author_id"], name: "index_silent_post_histories_on_author_id"
+    t.index ["deleted_at"], name: "index_silent_post_histories_on_deleted_at"
+    t.index ["enabled"], name: "index_silent_post_histories_on_enabled"
+    t.index ["history_ended_at"], name: "index_silent_post_histories_on_history_ended_at"
+    t.index ["history_started_at"], name: "index_silent_post_histories_on_history_started_at"
+    t.index ["history_user_id"], name: "index_silent_post_histories_on_history_user_id"
+    t.index ["live_at"], name: "index_silent_post_histories_on_live_at"
+    t.index ["silent_post_id"], name: "index_silent_post_histories_on_silent_post_id"
+  end
+
+  create_table "silent_posts", force: :cascade do |t|
+    t.string "title", null: false
+    t.text "body", null: false
+    t.integer "author_id", null: false
+    t.boolean "enabled", default: false
+    t.datetime "live_at"
+    t.datetime "deleted_at"
+    t.datetime "created_at", null: false
+    t.datetime "updated_at", null: false
+    t.index ["author_id"], name: "index_silent_posts_on_author_id"
+    t.index ["deleted_at"], name: "index_silent_posts_on_deleted_at"
+    t.index ["enabled"], name: "index_silent_posts_on_enabled"
+    t.index ["live_at"], name: "index_silent_posts_on_live_at"
+  end
+
   create_table "thing_with_compound_index_histories", force: :cascade do |t|
     t.integer "thing_with_compound_index_id", null: false
     t.string "key"

data/spec/historiographer_spec.rb

@@ -1,4 +1,6 @@
-require "spec_helper"
+# frozen_string_literal: true
+
+require 'spec_helper'
 
 class Post < ActiveRecord::Base
   include Historiographer
@@ -16,6 +18,14 @@ end
 class SafePostHistory < ActiveRecord::Base
 end
 
+class SilentPost < ActiveRecord::Base
+  include Historiographer::Silent
+  acts_as_paranoid
+end
+
+class SilentPostHistory < ActiveRecord::Base
+end
+
 class Author < ActiveRecord::Base
   include Historiographer
 end
@@ -45,7 +55,7 @@ describe Historiographer do
     Timecop.return
   end
 
-  let(:username) { "Test User" }
+  let(:username) { 'Test User' }
 
   let(:user) do
     User.create(name: username)
@@ -53,8 +63,8 @@ describe Historiographer do
 
   let(:create_post) do
     Post.create(
-      title: "Post 1",
-      body: "Great post",
+      title: 'Post 1',
+      body: 'Great post',
       author_id: 1,
       history_user_id: user.id
     )
@@ -62,43 +72,43 @@ describe Historiographer do
 
   let(:create_author) do
     Author.create(
-      full_name: "Breezy",
+      full_name: 'Breezy',
       history_user_id: user.id
     )
   end
 
-  describe "History counting" do
-    it "creates history on creation of primary model record" do
-      expect {
+  describe 'History counting' do
+    it 'creates history on creation of primary model record' do
+      expect do
         create_post
-      }.to change {
+      end.to change {
         PostHistory.count
       }.by 1
     end
 
-    it "appends new history on update" do
+    it 'appends new history on update' do
       post = create_post
 
-      expect {
-        post.update(title: "Better Title")
-      }.to change {
+      expect do
+        post.update(title: 'Better Title')
+      end.to change {
         PostHistory.count
       }.by 1
     end
 
-    it "does not append new history if nothing has changed" do
+    it 'does not append new history if nothing has changed' do
       post = create_post
 
-      expect { 
+      expect do
         post.update(title: post.title)
-      }.to_not change {
+      end.to_not change {
         PostHistory.count
       }
     end
   end
 
-  describe "History recording" do
-    it "records all fields from the parent" do
+  describe 'History recording' do
+    it 'records all fields from the parent' do
       post         = create_post
       post_history = post.histories.first
 
@@ -110,8 +120,8 @@ describe Historiographer do
       expect(post_history.history_ended_at).to be_nil
       expect(post_history.history_user_id).to eq user.id
 
-      post.update(title: "Better title")
-      post_histories = post.histories.reload.order("id asc")
+      post.update(title: 'Better title')
+      post_histories = post.histories.reload.order('id asc')
       first_history  = post_histories.first
       second_history = post_histories.second
 
@@ -119,24 +129,24 @@ describe Historiographer do
       expect(second_history.history_ended_at).to be_nil
     end
 
-    it "cannot create without history_user_id" do
+    it 'cannot create without history_user_id' do
       post = Post.create(
-        title: "Post 1",
-        body: "Great post",
-        author_id: 1,
-      ) 
-      expect(post.errors.to_h).to eq({ :history_user_id => "must be an integer" })
+        title: 'Post 1',
+        body: 'Great post',
+        author_id: 1
+      )
+      expect(post.errors.to_h).to eq(history_user_id: 'must be an integer')
 
-      expect {
+      expect do
         post.send(:record_history)
-      }.to raise_error(
+      end.to raise_error(
         Historiographer::HistoryUserIdMissingError
       )
     end
 
-    context "When directly hitting the database via SQL" do
-      context "#update_all" do
-        it "still updates histories" do
+    context 'When directly hitting the database via SQL' do
+      context '#update_all' do
+        it 'still updates histories' do
           FactoryBot.create_list(:post, 3, history_user_id: 1)
 
           posts = Post.all
@@ -144,12 +154,12 @@ describe Historiographer do
           expect(PostHistory.count).to eq 3
           expect(posts.map(&:histories).map(&:count)).to all (eq 1)
 
-          posts.update_all(title: "My New Post Title", history_user_id: 1)
+          posts.update_all(title: 'My New Post Title', history_user_id: 1)
 
           expect(PostHistory.count).to eq 6
           expect(PostHistory.current.count).to eq 3
           expect(posts.map(&:histories).map(&:count)).to all(eq 2)
-          expect(posts.map(&:current_history).map(&:title)).to all (eq "My New Post Title")
+          expect(posts.map(&:current_history).map(&:title)).to all (eq 'My New Post Title')
           expect(Post.all).to respond_to :has_histories?
 
           # It can update by sub-query
@@ -159,10 +169,10 @@ describe Historiographer do
           expect(posts.second.histories.count).to eq 2
           expect(posts.third.histories.count).to eq 3
           expect(posts.first.title).to eq "Brett's Post"
-          expect(posts.second.title).to eq "My New Post Title"
+          expect(posts.second.title).to eq 'My New Post Title'
           expect(posts.third.title).to eq "Brett's Post"
           expect(posts.first.current_history.title).to eq "Brett's Post"
-          expect(posts.second.current_history.title).to eq "My New Post Title"
+          expect(posts.second.current_history.title).to eq 'My New Post Title'
           expect(posts.third.current_history.title).to eq "Brett's Post"
 
           # It does not update histories if nothing changed
@@ -170,23 +180,23 @@ describe Historiographer do
           posts = Post.all.reload.order(:id)
           expect(posts.map(&:histories).map(&:count)).to all(eq 3)
 
-          posts.update_all_without_history(title: "Untracked")
+          posts.update_all_without_history(title: 'Untracked')
           expect(posts.first.histories.count).to eq 3
           expect(posts.second.histories.count).to eq 3
           expect(posts.third.histories.count).to eq 3
 
-          thing1 = ThingWithoutHistory.create(name: "Thing 1")
-          thing2 = ThingWithoutHistory.create(name: "Thing 2")
+          thing1 = ThingWithoutHistory.create(name: 'Thing 1')
+          thing2 = ThingWithoutHistory.create(name: 'Thing 2')
 
-          ThingWithoutHistory.all.update_all(name: "Thing 3")
+          ThingWithoutHistory.all.update_all(name: 'Thing 3')
 
-          expect(ThingWithoutHistory.all.map(&:name)).to all(eq "Thing 3")
+          expect(ThingWithoutHistory.all.map(&:name)).to all(eq 'Thing 3')
           expect(ThingWithoutHistory.all).to_not respond_to :has_histories?
           expect(ThingWithoutHistory.all).to_not respond_to :update_all_without_history
           expect(ThingWithoutHistory.all).to_not respond_to :delete_all_without_history
         end
-        
-        it "respects safety" do
+
+        it 'respects safety' do
           FactoryBot.create_list(:post, 3, history_user_id: 1)
 
           posts = Post.all
@@ -194,35 +204,35 @@ describe Historiographer do
           expect(PostHistory.count).to eq 3
           expect(posts.map(&:histories).map(&:count)).to all (eq 1)
 
-          expect {
-            posts.update_all(title: "My New Post Title")
-          }.to raise_error
+          expect do
+            posts.update_all(title: 'My New Post Title')
+          end.to raise_error
 
           posts.reload.map(&:title).each do |title|
-            expect(title).to_not eq "My New Post Title"
+            expect(title).to_not eq 'My New Post Title'
           end
 
           SafePost.create(
-            title: "Post 1",
-            body: "Great post",
-            author_id: 1,
+            title: 'Post 1',
+            body: 'Great post',
+            author_id: 1
           )
 
           safe_posts = SafePost.all
 
-          expect {
-            safe_posts.update_all(title: "New One")
-          }.to_not raise_error
+          expect do
+            safe_posts.update_all(title: 'New One')
+          end.to_not raise_error
 
-          expect(safe_posts.map(&:title)).to all(eq "New One")
+          expect(safe_posts.map(&:title)).to all(eq 'New One')
         end
       end
 
-      context "#delete_all" do
-        it "includes histories when not paranoid" do
+      context '#delete_all' do
+        it 'includes histories when not paranoid' do
           Timecop.freeze
           authors = 3.times.map do
-            Author.create(full_name: "Brett", history_user_id: 1)
+            Author.create(full_name: 'Brett', history_user_id: 1)
           end
           Author.delete_all(history_user_id: 1)
           expect(AuthorHistory.count).to eq 3
@@ -232,7 +242,7 @@ describe Historiographer do
           Timecop.return
         end
 
-        it "includes histories when paranoid" do
+        it 'includes histories when paranoid' do
           Timecop.freeze
           posts = FactoryBot.create_list(:post, 3, history_user_id: 1)
           Post.delete_all(history_user_id: 1)
@@ -246,9 +256,9 @@ describe Historiographer do
           Timecop.return
         end
 
-        it "allows delete_all_without_history" do
+        it 'allows delete_all_without_history' do
           authors = 3.times.map do
-            Author.create(full_name: "Brett", history_user_id: 1)
+            Author.create(full_name: 'Brett', history_user_id: 1)
           end
           Author.all.delete_all_without_history
           expect(AuthorHistory.current.count).to eq 3
@@ -256,8 +266,8 @@ describe Historiographer do
         end
       end
 
-      context "#destroy_all" do
-        it "includes histories" do
+      context '#destroy_all' do
+        it 'includes histories' do
           Timecop.freeze
           posts = FactoryBot.create_list(:post, 3, history_user_id: 1)
           Post.destroy_all(history_user_id: 1)
@@ -271,7 +281,7 @@ describe Historiographer do
           Timecop.return
         end
 
-        it "destroys without histories" do
+        it 'destroys without histories' do
           Timecop.freeze
           posts = FactoryBot.create_list(:post, 3, history_user_id: 1)
           Post.all.destroy_all_without_history
@@ -283,79 +293,125 @@ describe Historiographer do
       end
     end
 
-    context "When Safe mode" do
-      it "creates history without history_user_id" do
-        expect(Rollbar).to receive(:error).with("history_user_id must be passed in order to save record with histories! If you are in a context with no history_user_id, explicitly call #save_without_history")
+    context 'When Safe mode' do
+      it 'creates history without history_user_id' do
+        expect(Rollbar).to receive(:error).with('history_user_id must be passed in order to save record with histories! If you are in a context with no history_user_id, explicitly call #save_without_history')
 
         post = SafePost.create(
-          title: "Post 1",
-          body: "Great post",
-          author_id: 1,
-        ) 
+          title: 'Post 1',
+          body: 'Great post',
+          author_id: 1
+        )
         expect(post.errors.to_h.keys).to be_empty
         expect(post).to be_persisted
         expect(post.histories.count).to eq 1
         expect(post.histories.first.history_user_id).to be_nil
       end
 
-      it "creates history with history_user_id" do
+      it 'creates history with history_user_id' do
         expect(Rollbar).to_not receive(:error)
 
         post = SafePost.create(
-          title: "Post 1",
-          body: "Great post",
+          title: 'Post 1',
+          body: 'Great post',
           author_id: 1,
           history_user_id: user.id
-        ) 
+        )
         expect(post.errors.to_h.keys).to be_empty
         expect(post).to be_persisted
         expect(post.histories.count).to eq 1
         expect(post.histories.first.history_user_id).to eq user.id
       end
 
-       it "skips history creation if desired" do
+      it 'skips history creation if desired' do
         post = SafePost.new(
-          title: "Post 1",
-          body: "Great post",
+          title: 'Post 1',
+          body: 'Great post',
           author_id: 1
-        ) 
+        )
 
         post.save_without_history
         expect(post).to be_persisted
         expect(post.histories.count).to eq 0
-       end
+      end
     end
 
-    it "can override without history_user_id" do
-      expect { 
-        post = Post.new(
-          title: "Post 1",
-          body: "Great post",
+    context 'When Silent mode' do
+      it 'creates history without history_user_id' do
+        expect(Rollbar).to_not receive(:error)
+
+        post = SilentPost.create(
+          title: 'Post 1',
+          body: 'Great post',
+          author_id: 1
+        )
+        expect(post.errors.to_h.keys).to be_empty
+        expect(post).to be_persisted
+        expect(post.histories.count).to eq 1
+        expect(post.histories.first.history_user_id).to be_nil
+
+        post.update(title: 'New Title')
+        post.reload
+        expect(post.title).to eq 'New Title' # No error was raised
+      end
+
+      it 'creates history with history_user_id' do
+        expect(Rollbar).to_not receive(:error)
+
+        post = SilentPost.create(
+          title: 'Post 1',
+          body: 'Great post',
           author_id: 1,
-        ) 
+          history_user_id: user.id
+        )
+        expect(post.errors.to_h.keys).to be_empty
+        expect(post).to be_persisted
+        expect(post.histories.count).to eq 1
+        expect(post.histories.first.history_user_id).to eq user.id
+      end
+
+      it 'skips history creation if desired' do
+        post = SilentPost.new(
+          title: 'Post 1',
+          body: 'Great post',
+          author_id: 1
+        )
+
+        post.save_without_history
+        expect(post).to be_persisted
+        expect(post.histories.count).to eq 0
+      end
+    end
+    it 'can override without history_user_id' do
+      expect do
+        post = Post.new(
+          title: 'Post 1',
+          body: 'Great post',
+          author_id: 1
+        )
 
         post.save_without_history
-      }.to_not raise_error
+      end.to_not raise_error
     end
 
-    it "can override without history_user_id" do
-      expect { 
+    it 'can override without history_user_id' do
+      expect do
         post = Post.new(
-          title: "Post 1",
-          body: "Great post",
-          author_id: 1,
-        ) 
+          title: 'Post 1',
+          body: 'Great post',
+          author_id: 1
+        )
 
         post.save_without_history!
-      }.to_not raise_error
+      end.to_not raise_error
     end
 
-    it "does not record histories when main model fails to save" do
+    it 'does not record histories when main model fails to save' do
       class Post
         after_save :raise_error, prepend: true
 
         def raise_error
-          raise "Oh no, db issue!"
+          raise 'Oh no, db issue!'
         end
       end
 
@@ -367,21 +423,21 @@ describe Historiographer do
     end
   end
 
-  describe "Scopes" do
-    it "finds current histories" do
+  describe 'Scopes' do
+    it 'finds current histories' do
       post1 = create_post
-      post1.update(title: "Better title")
+      post1.update(title: 'Better title')
 
       post2 = create_post
-      post2.update(title: "Better title")
+      post2.update(title: 'Better title')
 
-      expect(PostHistory.current.pluck(:title)).to all eq "Better title"
-      expect(post1.current_history.title).to eq "Better title"
+      expect(PostHistory.current.pluck(:title)).to all eq 'Better title'
+      expect(post1.current_history.title).to eq 'Better title'
     end
   end
 
-  describe "Associations" do
-    it "names associated records" do
+  describe 'Associations' do
+    it 'names associated records' do
       post1 = create_post
       expect(post1.histories.first).to be_a(PostHistory)
 
@@ -394,27 +450,27 @@ describe Historiographer do
     end
   end
 
-  describe "Histories" do
-    it "does not allow direct updates of histories" do
+  describe 'Histories' do
+    it 'does not allow direct updates of histories' do
       post1 = create_post
       hist1 = post1.histories.first
 
-      expect(hist1.update(title: "A different title")).to be false
+      expect(hist1.update(title: 'A different title')).to be false
       expect(hist1.reload.title).to eq post1.title
 
-      expect(hist1.update!(title: "A different title")).to be false
+      expect(hist1.update!(title: 'A different title')).to be false
       expect(hist1.reload.title).to eq post1.title
 
-      hist1.title = "A different title"
+      hist1.title = 'A different title'
       expect(hist1.save).to be false
       expect(hist1.reload.title).to eq post1.title
 
-      hist1.title = "A different title"
+      hist1.title = 'A different title'
       expect(hist1.save!).to be false
       expect(hist1.reload.title).to eq post1.title
     end
 
-    it "does not allow destroys of histories" do
+    it 'does not allow destroys of histories' do
       post1                  = create_post
       hist1                  = post1.histories.first
       original_history_count = post1.histories.count
@@ -425,19 +481,19 @@ describe Historiographer do
       expect(post1.histories.count).to be original_history_count
     end
   end
-  
-  describe "Deletion" do
-    it "records deleted_at and history_user_id on primary and history if you use acts_as_paranoid" do
+
+  describe 'Deletion' do
+    it 'records deleted_at and history_user_id on primary and history if you use acts_as_paranoid' do
       post = Post.create(
-        title: "Post 1",
-        body: "Great post",
+        title: 'Post 1',
+        body: 'Great post',
         author_id: 1,
         history_user_id: user.id
       )
 
-      expect {
+      expect do
         post.destroy(history_user_id: 2)
-      }.to change {
+      end.to change {
         PostHistory.count
       }.by 1
 
@@ -447,23 +503,23 @@ describe Historiographer do
       expect(PostHistory.last.history_user_id).to eq 2
     end
 
-    it "works with Historiographer::Safe" do
-      post = SafePost.create(title: "HELLO", body: "YO", author_id: 1)
+    it 'works with Historiographer::Safe' do
+      post = SafePost.create(title: 'HELLO', body: 'YO', author_id: 1)
 
-      expect {
+      expect do
         post.destroy
-      }.to_not raise_error
+      end.to_not raise_error
 
       expect(SafePost.count).to eq 0
       expect(post.deleted_at).to_not be_nil
       expect(SafePostHistory.count).to eq 2
       expect(SafePostHistory.current.last.deleted_at).to eq post.deleted_at
 
-      post2 = SafePost.create(title: "HELLO", body: "YO", author_id: 1)
+      post2 = SafePost.create(title: 'HELLO', body: 'YO', author_id: 1)
 
-      expect {
+      expect do
         post2.destroy!
-      }.to_not raise_error
+      end.to_not raise_error
 
       expect(SafePost.count).to eq 0
       expect(post2.deleted_at).to_not be_nil
@@ -472,18 +528,18 @@ describe Historiographer do
     end
   end
 
-  describe "Scopes" do
-    it "finds current" do
+  describe 'Scopes' do
+    it 'finds current' do
       post = create_post
-      post.update(title: "New Title")
-      post.update(title: "New Title 2")
+      post.update(title: 'New Title')
+      post.update(title: 'New Title 2')
 
       expect(PostHistory.current.count).to be 1
     end
   end
 
-  describe "User associations" do
-    it "links to user" do
+  describe 'User associations' do
+    it 'links to user' do
       post = create_post
       author = create_author
 
@@ -492,10 +548,10 @@ describe Historiographer do
     end
   end
 
-  describe "Migrations with compound indexes" do
-    it "supports renaming compound indexes and migrating them to history tables" do
-      indices_sql = %q(
-        SELECT 
+  describe 'Migrations with compound indexes' do
+    it 'supports renaming compound indexes and migrating them to history tables' do
+      indices_sql = "
+        SELECT
           DISTINCT(
             ARRAY_TO_STRING(ARRAY(
              SELECT pg_get_indexdef(idx.indexrelid, k + 1, true)
@@ -514,19 +570,19 @@ describe Historiographer do
         AND a.attnum = ANY(idx.indkey)
         AND t.relkind = 'r'
         AND t.relname = ?;
-      )
+      "
 
       indices_query_array = [indices_sql, :thing_with_compound_index_histories]
       indices_sanitized_query = ThingWithCompoundIndexHistory.send(:sanitize_sql_array, indices_query_array)
 
-      indexes = ThingWithCompoundIndexHistory.connection.execute(indices_sanitized_query).to_a.map(&:values).flatten.map { |i| i.split(",") }
+      indexes = ThingWithCompoundIndexHistory.connection.execute(indices_sanitized_query).to_a.map(&:values).flatten.map { |i| i.split(',') }
 
-      expect(indexes).to include(["history_started_at"])
-      expect(indexes).to include(["history_ended_at"])
-      expect(indexes).to include(["history_user_id"])
-      expect(indexes).to include(["id"])
-      expect(indexes).to include(["key", "value"])
-      expect(indexes).to include(["thing_with_compound_index_id"])
+      expect(indexes).to include(['history_started_at'])
+      expect(indexes).to include(['history_ended_at'])
+      expect(indexes).to include(['history_user_id'])
+      expect(indexes).to include(['id'])
+      expect(indexes).to include(%w[key value])
+      expect(indexes).to include(['thing_with_compound_index_id'])
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: historiographer
 version: !ruby/object:Gem::Version
-  version: 2.0.1
+  version: 3.1.0
 platform: ruby
 authors:
 - brettshollenberger
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-02-04 00:00:00.000000000 Z
+date: 2022-10-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -16,16 +16,16 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '5'
+        version: '6'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '5'
+        version: '6'
 - !ruby/object:Gem::Dependency
-  name: activesupport
+  name: activerecord-import
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -39,7 +39,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rollbar
+  name: activesupport
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -53,7 +53,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: activerecord-import
+  name: rollbar
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -67,21 +67,21 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: pg
+  name: mysql2
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '='
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.5'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '='
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.5'
 - !ruby/object:Gem::Dependency
-  name: pry
+  name: paranoia
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -95,21 +95,21 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: mysql2
+  name: pg
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '0.5'
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '0.5'
+        version: '0'
 - !ruby/object:Gem::Dependency
-  name: standalone_migrations
+  name: pry
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -123,7 +123,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: timecop
+  name: standalone_migrations
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -137,7 +137,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: paranoia
+  name: timecop
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -151,47 +151,47 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rdoc
+  name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.12'
+        version: '1.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.12'
+        version: '1.0'
 - !ruby/object:Gem::Dependency
-  name: bundler
+  name: jeweler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '0'
 - !ruby/object:Gem::Dependency
-  name: jeweler
+  name: rdoc
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '3.12'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '3.12'
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
@@ -235,6 +235,7 @@ files:
 - lib/historiographer/postgres_migration.rb
 - lib/historiographer/relation.rb
 - lib/historiographer/safe.rb
+- lib/historiographer/silent.rb
 - spec/db/database.yml
 - spec/db/migrate/20161121212228_create_posts.rb
 - spec/db/migrate/20161121212229_create_post_histories.rb
@@ -246,6 +247,8 @@ files:
 - spec/db/migrate/20191024142304_create_thing_with_compound_index.rb
 - spec/db/migrate/20191024142352_create_thing_with_compound_index_history.rb
 - spec/db/migrate/20191024203106_create_thing_without_history.rb
+- spec/db/migrate/20221018204220_create_silent_posts.rb
+- spec/db/migrate/20221018204255_create_silent_post_histories.rb
 - spec/db/schema.rb
 - spec/examples.txt
 - spec/factories/post.rb