Merge pull request #5804 from betagouv/dev

2020-12-14-01

.gitignore

@@ -27,6 +27,7 @@ yarn-debug.log*
 .yarn-integrity
 /.vscode
 /.idea
+/public/assets
 /public/packs
 /public/packs-test
 /node_modules

app/models/bill_signature.rb

@@ -92,25 +92,27 @@ class BillSignature < ApplicationRecord
   end
 
   def read_signature
-    if attachment_changes['signature']
+    read_attachment('signature')
-      io = io_for_changes(attachment_changes['signature'])
-      io.read if io.present?
-    elsif signature.attached?
-      signature.download
-    end
   end
 
   def read_serialized
-    if attachment_changes['serialized']
+    read_attachment('serialized')
-      io = io_for_changes(attachment_changes['serialized'])
+  end
-      io.read if io.present?
+
+  private
+
+  def read_attachment(attachment)
+    if attachment_changes[attachment]
+      io = io_for_changes(attachment_changes[attachment])
+      if io.present?
+        io.rewind
+        io.read
+      end
     elsif serialized.attached?
       serialized.download
     end
   end
 
-  private
-
   def io_for_changes(attachment_changes)
     attachable = attachment_changes.attachable
     case attachable

app/models/procedure.rb

@@ -568,7 +568,7 @@ class Procedure < ApplicationRecord
     if logo.attached?
       Rails.application.routes.url_helpers.url_for(logo)
     else
-      ActionController::Base.helpers.image_url("republique-francaise-logo.svg")
+      ActionController::Base.helpers.image_url(PROCEDURE_DEFAULT_LOGO_SRC)
     end
   end
 

app/views/new_administrateur/services/_form.html.haml

@@ -19,7 +19,7 @@
   %h2.header-section Informations de contact
 
   %p.explication
-    Votre démarche sera hébergée par demarche-simplifiees.fr – mais nous ne pouvons pas assurer le support des démarches. Et malgré la dématérialisation, les usagers se poseront parfois des questions légitimes sur le processus administratif.
+    Votre démarche sera hébergée par #{APPLICATION_NAME} – mais nous ne pouvons pas assurer le support des démarches. Et malgré la dématérialisation, les usagers se poseront parfois des questions légitimes sur le processus administratif.
     %br
     Il est donc important que les usagers puissent vous contacter s'ils ont des questions sur votre démarche.
     %br

config/env.example.optional

@@ -41,5 +41,8 @@ APPLICATION_BASE_URL="https://www.demarches-simplifiees.fr"
 # Personnalisation d'instance - Logo dans l'entête des emails ---> à placer dans "app/assets/images"
 # MAILER_LOGO_SRC="mailer/instructeur_mailer/logo.png"
 
+# Personnalisation d'instance - Logo par défaut d'une procédure  ---> à placer dans "app/assets/images"
+# PROCEDURE_DEFAULT_LOGO_SRC="republique-francaise-logo.svg"
+
 # Personnalisation d'instance - fichier utilisé pour poser un filigrane sur les pièces d'identité
 # WATERMARK_FILE=""

config/initializers/images.rb

@@ -11,3 +11,6 @@ HEADER_LOGO_HEIGHT = ENV.fetch("HEADER_LOGO_HEIGHT", "56")
 
 # Mailer logo
 MAILER_LOGO_SRC = ENV.fetch("MAILER_LOGO_SRC", "mailer/instructeur_mailer/logo.png")
+
+# Default logo of a procedure
+PROCEDURE_DEFAULT_LOGO_SRC = ENV.fetch("PROCEDURE_DEFAULT_LOGO_SRC", "republique-francaise-logo.svg")

spec/models/bill_signature_spec.rb

@@ -72,48 +72,41 @@ RSpec.describe BillSignature, type: :model do
     end
 
     describe 'check_signature_contents' do
+      let(:signature) { File.open('spec/fixtures/files/bill_signature/signature.der') }
+      let(:signature_date) { DateTime.parse('2019-04-30 15:30:20') }
+      let(:signature_digest) { Digest::SHA256.hexdigest('CECI EST UN BLOB') }
+      let(:current_date) { Time.zone.now }
+
       before do
-        bill_signature.signature.attach(io: StringIO.new(signature), filename: 'file') if signature.present?
+        Timecop.freeze(current_date)
-        allow(ASN1::Timestamp).to receive(:signature_time).and_return(signature_time)
+        bill_signature.signature.attach(io: signature, filename: 'file') if signature.present?
-        allow(ASN1::Timestamp).to receive(:signed_digest).and_return(signed_digest)
+        bill_signature.digest = signature_digest
-        bill_signature.digest = digest
         bill_signature.valid?
+        Timecop.return
       end
 
-      context 'when the signature is correct' do
+      subject { bill_signature.errors.details[:signature] }
-        let(:signature) { 'signature' }
-        let(:signature_time) { 1.day.ago }
-        let(:digest) { 'abcd' }
-        let(:signed_digest) { 'abcd' }
 
-        it { expect(bill_signature.errors.details[:signature]).to be_empty }
+      context 'when the signature is correct' do
+        it { is_expected.to be_empty }
       end
 
       context 'when the signature isn’t set' do
         let(:signature) { nil }
-        let(:signature_time) { 1.day.ago }
-        let(:digest) { 'abcd' }
-        let(:signed_digest) { 'abcd' }
 
-        it { expect(bill_signature.errors.details[:signature]).to eq [error: :blank] }
+        it { is_expected.to eq [error: :blank] }
       end
 
       context 'when the signature time is in the future' do
-        let(:signature) { 'signature' }
+        let(:current_date) { signature_date - 1.day }
-        let(:signature_time) { 1.day.from_now }
-        let(:digest) { 'abcd' }
-        let(:signed_digest) { 'abcd' }
 
-        it { expect(bill_signature.errors.details[:signature]).to eq [error: :invalid_date] }
+        it { is_expected.to eq [error: :invalid_date] }
       end
 
       context 'when the signature doesn’t match the digest' do
-        let(:signature) { 'signature' }
+        let(:signature_digest) { 'dcba' }
-        let(:signature_time) { 1.day.ago }
-        let(:digest) { 'abcd' }
-        let(:signed_digest) { 'dcba' }
 
-        it { expect(bill_signature.errors.details[:signature]).to eq [error: :invalid] }
+        it { is_expected.to eq [error: :invalid] }
       end
     end
   end