remove delivery_class field from Endpoint model and endpoints table

app/models/endpoint.rb

@@ -4,14 +4,6 @@ class Endpoint < ApplicationRecord
   has_many :preserved_copies, dependent: :restrict_with_exception
   has_and_belongs_to_many :preservation_policies
 
-  # @note Hash values cannot be modified without migrating any associated persisted data.
-  # @see [enum docs] http://api.rubyonrails.org/classes/ActiveRecord/Enum.html
-  # TODO: deprecated, remove this field (and drop DB col) once this has transitioned to ArchiveEndpoint
-  enum delivery_class: {
-    S3WestDeliveryJob => 1,
-    S3EastDeliveryJob => 2
-  }
-
   validates :endpoint_name, presence: true, uniqueness: true
   validates :endpoint_node, presence: true
   validates :storage_location, presence: true

db/migrate/20180717191117_remove_delivery_class_from_endpoints.rb

@@ -0,0 +1,5 @@
+class RemoveDeliveryClassFromEndpoints < ActiveRecord::Migration[5.1]
+  def change
+    remove_column :endpoints, :delivery_class, :integer
+  end
+end

db/schema.rb

@@ -10,7 +10,8 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 20180712203748) do
+ActiveRecord::Schema.define(version: 20180717191117) do
+
   # These are extensions that must be enabled in order to support this database
   enable_extension "plpgsql"
 
@@ -47,7 +48,6 @@
     t.datetime "updated_at", null: false
     t.string "endpoint_node", null: false
     t.string "storage_location", null: false
-    t.integer "delivery_class"
     t.index ["endpoint_name"], name: "index_endpoints_on_endpoint_name", unique: true
     t.index ["endpoint_node"], name: "index_endpoints_on_endpoint_node"
     t.index ["storage_location"], name: "index_endpoints_on_storage_location"

spec/factories/endpoint.rb

@@ -6,6 +6,5 @@
   end
 
   factory :archive_endpoint_deprecated, parent: :endpoint do
-    delivery_class 1
   end
 end

spec/models/endpoint_spec.rb

@@ -28,10 +28,6 @@
     expect { endpoint.dup.save(validate: false) }.to raise_error(ActiveRecord::RecordNotUnique)
   end
 
-  it 'has multiple delivery_classes' do
-    expect(described_class.delivery_classes).to include(S3WestDeliveryJob, S3EastDeliveryJob)
-  end
-
   it { is_expected.to have_many(:preserved_copies) }
   it { is_expected.to have_db_index(:endpoint_name) }
   it { is_expected.to have_db_index(:endpoint_node) }