From c683fad3f3c12626ea623a967e53e0e0b1d47655 Mon Sep 17 00:00:00 2001
From: simon lehericey <mail@simon.lehericey.net>
Date: Mon, 12 Apr 2021 13:15:55 +0200
Subject: [PATCH] [#6084] add after_party task to fill missing data

---
 ...0210412093054_fill_missing_date_of_fc.rake | 39 ++++++++++
 ...0412093054_fill_missing_date_of_fc_spec.rb | 77 +++++++++++++++++++
 2 files changed, 116 insertions(+)
 create mode 100644 lib/tasks/deployment/20210412093054_fill_missing_date_of_fc.rake
 create mode 100644 spec/lib/tasks/deployment/20210412093054_fill_missing_date_of_fc_spec.rb

diff --git a/lib/tasks/deployment/20210412093054_fill_missing_date_of_fc.rake b/lib/tasks/deployment/20210412093054_fill_missing_date_of_fc.rake
new file mode 100644
index 000000000..f41ce8e2b
--- /dev/null
+++ b/lib/tasks/deployment/20210412093054_fill_missing_date_of_fc.rake
@@ -0,0 +1,39 @@
+namespace :after_party do
+  desc 'Deployment task: fill_missing_date_of_fc'
+  task fill_missing_date_of_fc: :environment do
+    rake_puts "Remove invalid FranceConnectInformation records with no associated user…"
+    FranceConnectInformation
+      .where(user_id: nil)
+      .destroy_all
+
+    rake_puts "Fill-in missing created_at from updated_at column on FranceConnectInformation records…"
+    created_from_updated_sql = <<~EOF
+      created_at = updated_at,
+      data = '{ "note": "missing created_at has been copied from updated_at" }'
+    EOF
+
+    FranceConnectInformation
+      .where(created_at: nil)
+      .where.not(updated_at: nil)
+      .update_all(created_from_updated_sql)
+
+    rake_puts "Fill-in missing created_at, updated_at columns from users.created on FranceConnectInformation records…"
+    created_updated_from_user_created_sql = <<~EOF
+      UPDATE france_connect_informations
+      SET created_at = users.created_at,
+          updated_at = users.created_at,
+          data = '{ "note": "missing created_at, updated_at have been copied from users.created_at" }'
+      FROM  users
+      WHERE users.id = france_connect_informations.user_id
+        AND france_connect_informations.created_at IS NULL
+        AND france_connect_informations.updated_at IS NULL
+    EOF
+
+    FranceConnectInformation
+      .connection
+      .execute(created_updated_from_user_created_sql)
+
+    AfterParty::TaskRecord
+      .create version: AfterParty::TaskRecorder.new(__FILE__).timestamp
+  end
+end
diff --git a/spec/lib/tasks/deployment/20210412093054_fill_missing_date_of_fc_spec.rb b/spec/lib/tasks/deployment/20210412093054_fill_missing_date_of_fc_spec.rb
new file mode 100644
index 000000000..79657ced7
--- /dev/null
+++ b/spec/lib/tasks/deployment/20210412093054_fill_missing_date_of_fc_spec.rb
@@ -0,0 +1,77 @@
+describe '20210412093054_fill_missing_date_of_fc' do
+  let(:rake_task) { Rake::Task['after_party:fill_missing_date_of_fc'] }
+
+  let!(:user) { create(:user, created_at: Time.zone.parse('2000/01/01')) }
+
+  let!(:valid_fci) do
+    FranceConnectInformation.create!(
+      user: user,
+      france_connect_particulier_id: '123',
+      created_at: Time.zone.parse('2010/01/01'),
+      updated_at: Time.zone.parse('2012/01/01')
+    )
+  end
+
+  let!(:missing_created_fci) do
+    fci = FranceConnectInformation.create!(
+      user: user,
+      france_connect_particulier_id: '123',
+      updated_at: Time.zone.parse('2013/01/01')
+    )
+
+    fci.update_column('created_at', nil)
+    fci
+  end
+
+  let!(:missing_created_updated_fci) do
+    fci = FranceConnectInformation.create!(
+      user: user,
+      france_connect_particulier_id: '123'
+    )
+
+    fci.update_column('created_at', nil)
+    fci.update_column('updated_at', nil)
+    fci
+  end
+
+  let!(:missing_created_updated_without_user_fci) do
+    fci = FranceConnectInformation.create!(
+      france_connect_particulier_id: '123'
+    )
+
+    fci.update_column('created_at', nil)
+    fci.update_column('updated_at', nil)
+    fci
+  end
+
+  before do
+    rake_task.invoke
+  end
+
+  after { rake_task.reenable }
+
+  it "does not change valid fci" do
+    valid_fci.reload
+    expect(valid_fci.created_at).to eq(Time.zone.parse('2010/01/01'))
+    expect(valid_fci.updated_at).to eq(Time.zone.parse('2012/01/01'))
+    expect(valid_fci.data).to be_nil
+  end
+
+  it "fills missing created from updated" do
+    missing_created_fci.reload
+    expect(missing_created_fci.created_at).to eq(Time.zone.parse('2013/01/01'))
+    expect(missing_created_fci.data['note']).to eq("missing created_at has been copied from updated_at")
+  end
+
+  it "fills missing created, updated from users created" do
+    missing_created_updated_fci.reload
+    expect(missing_created_updated_fci.created_at).to eq(Time.zone.parse('2000/01/01'))
+    expect(missing_created_updated_fci.updated_at).to eq(Time.zone.parse('2000/01/01'))
+    expect(missing_created_updated_fci.data['note']).to eq("missing created_at, updated_at have been copied from users.created_at")
+  end
+
+  it "destroys fci when there is no user" do
+    expect { missing_created_updated_without_user_fci.reload }
+      .to raise_error(ActiveRecord::RecordNotFound)
+  end
+end