Validate when updating accession

Author: maimux2x

app/controllers/accession_renewals_controller.rb

@@ -0,0 +1,22 @@
+class AccessionRenewalsController < ApplicationController
+  def show
+    @renewal = current_user.accession_renewals.find(params.expect(:id))
+  end
+
+  def create
+    accession = current_user.accessions.find_by!(number: params.expect(:accession_number))
+    @renewal  = accession.renewals.create!(renewal_params)
+
+    RenewAccessionJob.perform_later @renewal
+
+    render :show, status: :created
+  end
+
+  private
+
+  def renewal_params
+    params.expect(accession_renewal: [
+      :file
+    ])
+  end
+end

app/jobs/renew_accession_job.rb

@@ -0,0 +1,26 @@
+class RenewAccessionJob < ApplicationJob
+  def perform(renewal)
+    renewal.update! progress: :running, started_at: Time.current
+
+    sleep 3
+
+    ActiveRecord::Base.transaction do
+      begin
+        JSON.parse renewal.file.download
+      rescue JSON::ParserError => e
+        renewal.validation_details.create!(
+          severity: 'error',
+          message:  e.message
+        )
+      end
+
+      if renewal.validation_details.empty?
+        renewal.validity_valid!
+      else
+        renewal.validity_invalid!
+      end
+    end
+  ensure
+    renewal.update! progress: :finished, finished_at: Time.current unless renewal.canceled?
+  end
+end

app/models/accession.rb

@@ -1,3 +1,5 @@
 class Accession < ApplicationRecord
   belongs_to :submission
+
+  has_many :renewals, dependent: :destroy, class_name: 'AccessionRenewal'
 end

app/models/accession_renewal.rb

@@ -0,0 +1,10 @@
+class AccessionRenewal < ApplicationRecord
+  belongs_to :accession, inverse_of: :renewals
+
+  has_many :validation_details, dependent: :destroy, class_name: 'AccessionRenewalValidationDetail', inverse_of: :renewal
+
+  has_one_attached :file
+
+  enum :progress, %w[waiting running finished canceled].index_by(&:to_sym)
+  enum :validity, %w[valid invalid error].index_by(&:to_sym), prefix: true
+end

app/models/accession_renewal_validation_detail.rb

@@ -0,0 +1,3 @@
+class AccessionRenewalValidationDetail < ApplicationRecord
+  belongs_to :renewal, class_name: 'AccessionRenewal'
+end

app/models/database/trad/ddbj_record_validator.rb

@@ -1,6 +1,6 @@
 class Database::Trad::DDBJRecordValidator
   def validate(validation)
-    obj = validation.objs.without_base.sole
+    obj = validation.objs.without_base.last
 
     begin
       JSON.parse(obj.file.download)

app/models/user.rb

@@ -5,8 +5,9 @@ def self.generate_api_key
 
   has_many :validations, dependent: :destroy
 
-  has_many :submissions, through: :validations
-  has_many :accessions,  through: :submissions
+  has_many :submissions,        through: :validations
+  has_many :accessions,         through: :submissions
+  has_many :accession_renewals, through: :accessions, source: :renewals
 
   before_create do |user|
     user.api_key ||= self.class.generate_api_key

app/views/accession_renewals/_accession_renewal.json.jb

@@ -0,0 +1,20 @@
+{
+  **accession_renewal.slice(
+    :id,
+    :started_at,
+    :finished_at,
+    :progress,
+    :validity,
+    :created_at
+  ),
+
+  validation_details: accession_renewal.validation_details.map {|detail|
+    {
+      **detail.slice(
+        :severity
+      ),
+
+      message: detail.message.scrub
+    }
+  }
+}

app/views/accession_renewals/show.json.jb

@@ -0,0 +1 @@
+render(@renewal)

app/views/accessions/_accession.json.jb

@@ -1,8 +1,12 @@
 # locals: (accession:)
 
-accession.slice(
-  :number,
-  :entry_id,
-  :version,
-  :last_updated_at
-)
+{
+  **accession.slice(
+    :number,
+    :entry_id,
+    :version,
+    :last_updated_at
+  ),
+
+  renewals: render(partial: 'accession_renewals/accession_renewal', collection: accession.renewals)
+}