Merge pull request #877 from sgmap/fix_attestation

Fix attestation

app/uploaders/attestation_uploader.rb

@@ -17,4 +17,14 @@ class AttestationUploader < BaseUploader
       "uploads/#{model.class.to_s.underscore}/#{mounted_as}/#{model.id}"
     end
   end
+
+  def filename
+    "attestation-#{secure_token}.pdf"
+  end
+
+  private
+
+  def secure_token
+    model.content_secure_token ||= SecureRandom.uuid
+  end
 end

db/migrate/20171019085515_add_content_secure_token_column_to_attestation.rb

@@ -0,0 +1,5 @@
+class AddContentSecureTokenColumnToAttestation < ActiveRecord::Migration[5.0]
+  def change
+    add_column :attestations, :content_secure_token, :string
+  end
+end

db/schema.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 20170927092716) do
+ActiveRecord::Schema.define(version: 20171019085515) do
 
   # These are extensions that must be enabled in order to support this database
   enable_extension "plpgsql"
@@ -90,6 +90,7 @@ ActiveRecord::Schema.define(version: 20170927092716) do
     t.integer  "dossier_id",           null: false
     t.datetime "created_at",           null: false
     t.datetime "updated_at",           null: false
+    t.string   "content_secure_token"
     t.index ["dossier_id"], name: "index_attestations_on_dossier_id", using: :btree
   end
 

spec/models/attestation_template_spec.rb

@@ -142,7 +142,7 @@ describe AttestationTemplate, type: :model do
 
     it 'provides a pseudo file' do
       expect(attestation.pdf.file).to exist
-      expect(attestation.pdf.filename).to eq('attestation')
+      expect(attestation.pdf.filename).to start_with('attestation')
     end
 
     context 'when the dossier and the procedure has an individual' do