Import#update_status: Update `ended_at` when import has finished status

Previously we were only updating the `ended_at` field when the import
had a 'successful' status. But there are other statuses that indicate
the import finished, and the `ended_at` field should be updated in these
cases also.

Committing what I have now which seems to work but I didn't write specs
for it because it was a pain to test (what with creating a child, etc.).
The `#update_status` method should be refactored to make testing the
`ended_at` update easier.

This needs to be committed without specs now because I'm off to go to a
client meeting and this work/branch is blocking QA of imports both
internally and externally. So I guess doing things right comes later.
That is to say, probably a lot later. Who knows.

1 files changed, 3 insertions, 1 deletions

diff --git a/app/models/import.rb b/app/models/import.rb
index e5da74d94..d79d93e4a 100644
--- a/app/models/import.rb
+++ b/app/models/import.rb
@@ -61,9 +61,11 @@ class Import < ActiveRecord::Base
     status = self.status
     if children_failed_count > 0
       status = 'failed'
-      # TODO: Update `ended_at`
     elsif status_count['successful'] == children_count
       status = 'successful'
+    end
+
+    if self.class.finished_statuses.include?(status)
       attributes[:ended_at] = Time.now
     end