Reduce the max class length after splitting the changeset comments controller tests out

[rails.git] / db / migrate / 025_add_end_time_to_changesets.rb

diff --git a/db/migrate/025_add_end_time_to_changesets.rb b/db/migrate/025_add_end_time_to_changesets.rb
index 4941b72b92fb4b5264bfe42d84b2189394608e14..5bd2d5879d4d30f45693395ad54b69486716b3e6 100644 (file)
--- a/db/migrate/025_add_end_time_to_changesets.rb
+++ b/db/migrate/025_add_end_time_to_changesets.rb
@@ -1,23 +1,25 @@
-class AddEndTimeToChangesets < ActiveRecord::Migration
+require "migrate"
+
+class AddEndTimeToChangesets < ActiveRecord::Migration[5.0]
   def self.up
     # swap the boolean closed-or-not for a time when the changeset will
     # close or has closed.
     add_column(:changesets, :closed_at, :datetime, :null => false)
-    
+
     # it appears that execute will only accept string arguments, so
     # this is an ugly, ugly hack to get some sort of mysql/postgres
     # independence. now i have to go wash my brain with bleach.
-    execute("update changesets set closed_at=(now()-#{interval_constant('1 hour')}) where open=(1=0)")
-    execute("update changesets set closed_at=(now()+#{interval_constant('1 hour')}) where open=(1=1)")
+    execute("update changesets set closed_at=(now()-'1 hour'::interval) where open=(1=0)")
+    execute("update changesets set closed_at=(now()+'1 hour'::interval) where open=(1=1)")
 
-    # remove the open column as it is unnecessary now and denormalises 
+    # remove the open column as it is unnecessary now and denormalises
     # the table.
     remove_column :changesets, :open
 
     # add a column to keep track of the number of changes in a changeset.
     # could probably work out how many changes there are here, but i'm not
     # sure its actually important.
-    add_column(:changesets, :num_changes, :integer, 
+    add_column(:changesets, :num_changes, :integer,
                :null => false, :default => 0)
   end