paper_trail 4.2.0 → 5.0.0

data/test/dummy/config/initializers/backtrace_silencers.rb

@@ -1,7 +1,9 @@
 # Be sure to restart your server when you modify this file.
 
-# You can add backtrace silencers for libraries that you're using but don't wish to see in your backtraces.
+# You can add backtrace silencers for libraries that you're using but don't wish
+# to see in your backtraces.
 # Rails.backtrace_cleaner.add_silencer { |line| line =~ /my_noisy_library/ }
 
-# You can also remove all the silencers if you're trying to debug a problem that might stem from framework code.
+# You can also remove all the silencers if you're trying to debug a problem that
+# might stem from framework code.
 # Rails.backtrace_cleaner.remove_silencers!

data/test/dummy/config/initializers/paper_trail.rb

@@ -1,5 +1,4 @@
-# Turn on associations tracking when the test suite is run on Travis CI
-PaperTrail.config.track_associations = true if ENV['TRAVIS']
+PaperTrail.config.track_associations = true
 
 module PaperTrail
   class Version < ActiveRecord::Base

data/test/dummy/config/initializers/secret_token.rb

@@ -4,4 +4,6 @@
 # If you change this key, all old signed cookies will become invalid!
 # Make sure the secret is at least 30 characters and all random,
 # no regular words or you'll be exposed to dictionary attacks.
-Dummy::Application.config.secret_token = '99c0312cf416079a8c7ccc63acb1e9f4f17741398ec8022a2f4fb509c57f55b72486573e9816a026f507efbcd452a9e954d45c93c95d5bacd87e664ad6805d3f'
+Dummy::Application.config.secret_token = "99c0312cf416079a8c7ccc63acb1e9f4f" +
+  "17741398ec8022a2f4fb509c57f55b72486573e9816a026f507efbcd452a9e954d45c93c" +
+  "95d5bacd87e664ad6805d3f"

data/test/dummy/config/initializers/session_store.rb

@@ -1,6 +1,6 @@
 # Be sure to restart your server when you modify this file.
 
-Dummy::Application.config.session_store :cookie_store, :key => '_dummy_session'
+Dummy::Application.config.session_store :cookie_store, key: "_dummy_session"
 
 # Use the database for sessions instead of the cookie-based default,
 # which shouldn't be used to store highly confidential information

data/test/dummy/config/routes.rb

@@ -1,4 +1,4 @@
 Dummy::Application.routes.draw do
-  resources :articles, :only => [:create]
-  resources :widgets, :only => [:create, :update, :destroy]
+  resources :articles, only: [:create]
+  resources :widgets, only: [:create, :update, :destroy]
 end

data/test/dummy/db/migrate/20110208155312_set_up_test_tables.rb

@@ -1,33 +1,49 @@
+# Keep this migration in sync with
+# `lib/generators/paper_trail/templates/create_versions.rb`
+# TODO: Is there a way to avoid duplication?
 class SetUpTestTables < ActiveRecord::Migration
-  def self.up
-    create_table :skippers, :force => true do |t|
+  MYSQL_ADAPTERS = [
+    "ActiveRecord::ConnectionAdapters::MysqlAdapter",
+    "ActiveRecord::ConnectionAdapters::Mysql2Adapter"
+  ].freeze
+  TEXT_BYTES = 1_073_741_823
+
+  def up
+    # Classes: Vehicle, Car, Truck
+    create_table :vehicles, force: true do |t|
+      t.string :name, null: false
+      t.string :type, null: false
+      t.timestamps null: false
+    end
+
+    create_table :skippers, force: true do |t|
       t.string     :name
       t.datetime   :another_timestamp
-      t.timestamps :null => true
+      t.timestamps null: true
     end
 
-    create_table :widgets, :force => true do |t|
+    create_table :widgets, force: true do |t|
       t.string    :name
       t.text      :a_text
       t.integer   :an_integer
       t.float     :a_float
-      t.decimal   :a_decimal, :precision => 6, :scale => 4
+      t.decimal   :a_decimal, precision: 6, scale: 4
       t.datetime  :a_datetime
       t.time      :a_time
       t.date      :a_date
       t.boolean   :a_boolean
       t.string    :sacrificial_column
       t.string    :type
-      t.timestamps :null => true
+      t.timestamps null: true
     end
 
-    create_table :versions, :force => true do |t|
-      t.string   :item_type, :null => false
-      t.integer  :item_id,   :null => false
-      t.string   :event,     :null => false
+    create_table :versions, versions_table_options 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.text     :object_changes
+      t.text     :object, limit: TEXT_BYTES
+      t.text     :object_changes, limit: TEXT_BYTES
       t.integer  :transaction_id
       t.datetime :created_at
 
@@ -46,16 +62,18 @@ class SetUpTestTables < ActiveRecord::Migration
 
     create_table :version_associations do |t|
       t.integer  :version_id
-      t.string   :foreign_key_name, :null => false
+      t.string   :foreign_key_name, null: false
       t.integer  :foreign_key_id
     end
     add_index :version_associations, [:version_id]
-    add_index :version_associations, [:foreign_key_name, :foreign_key_id], :name => 'index_version_associations_on_foreign_key'
-
-    create_table :post_versions, :force => true do |t|
-      t.string   :item_type, :null => false
-      t.integer  :item_id,   :null => false
-      t.string   :event,     :null => false
+    add_index :version_associations,
+      [:foreign_key_name, :foreign_key_id],
+      name: "index_version_associations_on_foreign_key"
+
+    create_table :post_versions, force: true 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
@@ -66,11 +84,11 @@ class SetUpTestTables < ActiveRecord::Migration
     end
     add_index :post_versions, [:item_type, :item_id]
 
-    if ENV['DB'] == 'postgres' && ::ActiveRecord::VERSION::MAJOR >= 4
-      create_table :json_versions, :force => true do |t|
-        t.string   :item_type, :null => false
-        t.integer  :item_id,   :null => false
-        t.string   :event,     :null => false
+    if ENV["DB"] == "postgres" && ::ActiveRecord::VERSION::MAJOR >= 4
+      create_table :json_versions, force: true do |t|
+        t.string   :item_type, null: false
+        t.integer  :item_id,   null: false
+        t.string   :event,     null: false
         t.string   :whodunnit
         t.json     :object
         t.json     :object_changes
@@ -79,168 +97,186 @@ class SetUpTestTables < ActiveRecord::Migration
       add_index :json_versions, [:item_type, :item_id]
     end
 
-    create_table :not_on_updates, :force => true do |t|
-      t.timestamps :null => true
+    create_table :not_on_updates, force: true do |t|
+      t.timestamps null: true
     end
 
-    create_table :bananas, :force => true do |t|
-      t.timestamps :null => true
+    create_table :bananas, force: true do |t|
+      t.timestamps null: true
     end
 
-    create_table :banana_versions, :force => true do |t|
-      t.string   :item_type, :null => false
-      t.integer  :item_id,   :null => false
-      t.string   :event,     :null => false
+    create_table :banana_versions, force: true 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 :banana_versions, [:item_type, :item_id]
 
-    create_table :wotsits, :force => true do |t|
+    create_table :wotsits, force: true do |t|
       t.integer :widget_id
       t.string  :name
-      t.timestamps :null => true
+      t.timestamps null: true
     end
 
-    create_table :fluxors, :force => true do |t|
+    create_table :fluxors, force: true do |t|
       t.integer :widget_id
       t.string  :name
     end
 
-    create_table :whatchamajiggers, :force => true do |t|
+    create_table :whatchamajiggers, force: true do |t|
       t.string  :owner_type
       t.integer :owner_id
       t.string  :name
     end
 
-    create_table :articles, :force => true do |t|
+    create_table :articles, force: true do |t|
       t.string :title
       t.string :content
       t.string :abstract
       t.string :file_upload
     end
 
-    create_table :books, :force => true do |t|
+    create_table :books, force: true do |t|
       t.string :title
     end
 
-    create_table :authorships, :force => true do |t|
+    create_table :authorships, force: true do |t|
       t.integer :book_id
-      t.integer :person_id
+      t.integer :author_id
     end
 
-    create_table :people, :force => true do |t|
+    create_table :people, force: true do |t|
       t.string :name
       t.string :time_zone
     end
 
-    create_table :editorships, :force => true do |t|
+    create_table :editorships, force: true do |t|
       t.integer :book_id
       t.integer :editor_id
     end
 
-    create_table :editors, :force => true do |t|
+    create_table :editors, force: true do |t|
       t.string :name
     end
 
-    create_table :songs, :force => true do |t|
+    create_table :songs, force: true do |t|
       t.integer :length
     end
 
-    create_table :posts, :force => true do |t|
+    create_table :posts, force: true do |t|
       t.string :title
       t.string :content
     end
 
-    create_table :post_with_statuses, :force => true do |t|
+    create_table :post_with_statuses, force: true do |t|
       t.integer :status
     end
 
-    create_table :animals, :force => true do |t|
+    create_table :animals, force: true do |t|
       t.string :name
-      t.string :species   # single table inheritance column
+      t.string :species # single table inheritance column
     end
 
-    create_table :documents, :force => true do |t|
+    create_table :documents, force: true do |t|
       t.string :name
     end
 
-    create_table :legacy_widgets, :force => true do |t|
+    create_table :legacy_widgets, force: true do |t|
       t.string    :name
       t.integer   :version
     end
 
-    create_table :things, :force => true do |t|
+    create_table :things, force: true do |t|
       t.string    :name
     end
 
-    create_table :translations, :force => true do |t|
+    create_table :translations, force: true do |t|
       t.string    :headline
       t.string    :content
       t.string    :language_code
       t.string    :type
     end
 
-    create_table :gadgets, :force => true do |t|
+    create_table :gadgets, force: true do |t|
       t.string    :name
       t.string    :brand
-      t.timestamps :null => true
+      t.timestamps null: true
     end
 
-    create_table :customers, :force => true do |t|
-      t.string    :name
+    create_table :customers, force: true do |t|
+      t.string :name
     end
 
-    create_table :orders, :force => true do |t|
+    create_table :orders, force: true do |t|
       t.integer :customer_id
       t.string  :order_date
     end
 
-    create_table :line_items, :force => true do |t|
+    create_table :line_items, force: true do |t|
       t.integer :order_id
       t.string  :product
     end
 
-    create_table :fruits, :force => true do |t|
+    create_table :fruits, force: true do |t|
       t.string :name
       t.string :color
     end
 
-    create_table :boolits, :force => true do |t|
+    create_table :boolits, force: true do |t|
       t.string :name
-      t.boolean :scoped, :default => true
+      t.boolean :scoped, default: true
     end
 
-    create_table :callback_modifiers, :force => true do |t|
+    create_table :callback_modifiers, force: true do |t|
       t.string  :some_content
-      t.boolean :deleted, :default => false
+      t.boolean :deleted, default: false
     end
 
-    create_table :chapters, :force => true do |t|
+    create_table :chapters, force: true do |t|
       t.string :name
     end
 
-    create_table :sections, :force => true do |t|
+    create_table :sections, force: true do |t|
       t.integer :chapter_id
       t.string :name
     end
 
-    create_table :paragraphs, :force => true do |t|
+    create_table :paragraphs, force: true do |t|
       t.integer :section_id
       t.string :name
     end
 
-    create_table :quotations, :force => true do |t|
+    create_table :quotations, force: true do |t|
       t.integer :chapter_id
     end
 
-    create_table :citations, :force => true do |t|
+    create_table :citations, force: true do |t|
       t.integer :quotation_id
     end
+
+    create_table :foo_habtms, force: true do |t|
+      t.string :name
+    end
+
+    create_table :bar_habtms, force: true do |t|
+      t.string :name
+    end
+
+    create_table :bar_habtms_foo_habtms, force: true, id: false do |t|
+      t.integer :foo_habtm_id
+      t.integer :bar_habtm_id
+    end
+    add_index :bar_habtms_foo_habtms, [:foo_habtm_id]
+    add_index :bar_habtms_foo_habtms, [:bar_habtm_id]
   end
 
-  def self.down
+  def down
+    drop_table :bar_habtms_foo_habtms
+    drop_table :foo_habtms
+    drop_table :bar_habtms
     drop_table :citations
     drop_table :quotations
     drop_table :animals
@@ -257,12 +293,12 @@ class SetUpTestTables < ActiveRecord::Migration
     drop_table :whatchamajiggers
     drop_table :fluxors
     drop_table :wotsits
-    remove_index :post_versions, :column => [:item_type, :item_id]
+    remove_index :post_versions, column: [:item_type, :item_id]
     drop_table :post_versions
-    remove_index :versions, :column => [:item_type, :item_id]
+    remove_index :versions, column: [:item_type, :item_id]
     drop_table :versions
     if JsonVersion.table_exists?
-      remove_index :json_versions, :column => [:item_type, :item_id]
+      remove_index :json_versions, column: [:item_type, :item_id]
       drop_table :json_versions
     end
     drop_table :widgets
@@ -279,9 +315,19 @@ class SetUpTestTables < ActiveRecord::Migration
     drop_table :chapters
     drop_table :sections
     drop_table :paragraphs
-    remove_index :version_associations, :column => [:version_id]
-    remove_index :version_associations, :name => 'index_version_associations_on_foreign_key'
+    remove_index :version_associations, column: [:version_id]
+    remove_index :version_associations, name: "index_version_associations_on_foreign_key"
     drop_table :version_associations
     drop_table :callback_modifiers
   end
+
+  private
+
+  def versions_table_options
+    opts = { force: true }
+    if MYSQL_ADAPTERS.include?(connection.class.name)
+      opts[:options] = "ENGINE=InnoDB DEFAULT CHARSET=utf8mb4 COLLATE=utf8mb4_general_ci"
+    end
+    opts
+  end
 end

data/test/dummy/db/schema.rb

@@ -27,7 +27,7 @@ ActiveRecord::Schema.define(version: 20110208155312) do
 
   create_table "authorships", force: :cascade do |t|
     t.integer "book_id"
-    t.integer "person_id"
+    t.integer "author_id"
   end
 
   create_table "banana_versions", force: :cascade do |t|
@@ -46,6 +46,18 @@ ActiveRecord::Schema.define(version: 20110208155312) do
     t.datetime "updated_at"
   end
 
+  create_table "bar_habtms", force: :cascade do |t|
+    t.string "name"
+  end
+
+  create_table "bar_habtms_foo_habtms", id: false, force: :cascade do |t|
+    t.integer "foo_habtm_id"
+    t.integer "bar_habtm_id"
+  end
+
+  add_index "bar_habtms_foo_habtms", ["bar_habtm_id"], name: "index_bar_habtms_foo_habtms_on_bar_habtm_id"
+  add_index "bar_habtms_foo_habtms", ["foo_habtm_id"], name: "index_bar_habtms_foo_habtms_on_foo_habtm_id"
+
   create_table "books", force: :cascade do |t|
     t.string "title"
   end
@@ -90,6 +102,10 @@ ActiveRecord::Schema.define(version: 20110208155312) do
     t.string  "name"
   end
 
+  create_table "foo_habtms", force: :cascade do |t|
+    t.string "name"
+  end
+
   create_table "fruits", force: :cascade do |t|
     t.string "name"
     t.string "color"
@@ -185,6 +201,13 @@ ActiveRecord::Schema.define(version: 20110208155312) do
     t.string "type"
   end
 
+  create_table "vehicles", force: :cascade do |t|
+    t.string   "name",       null: false
+    t.string   "type",       null: false
+    t.datetime "created_at", null: false
+    t.datetime "updated_at", null: false
+  end
+
   create_table "version_associations", force: :cascade do |t|
     t.integer "version_id"
     t.string  "foreign_key_name", null: false
@@ -195,12 +218,12 @@ ActiveRecord::Schema.define(version: 20110208155312) do
   add_index "version_associations", ["version_id"], name: "index_version_associations_on_version_id"
 
   create_table "versions", force: :cascade do |t|
-    t.string   "item_type",      null: false
-    t.integer  "item_id",        null: false
-    t.string   "event",          null: false
+    t.string   "item_type",                         null: false
+    t.integer  "item_id",                           null: false
+    t.string   "event",                             null: false
     t.string   "whodunnit"
-    t.text     "object"
-    t.text     "object_changes"
+    t.text     "object",         limit: 1073741823
+    t.text     "object_changes", limit: 1073741823
     t.integer  "transaction_id"
     t.datetime "created_at"
     t.integer  "answer"