Spec form Fillin valid

Author: Quentinchampenois

app/controllers/decidim/extends/controllers/confirmation_controller.rb

@@ -1,48 +1,31 @@
 # frozen_string_literal: true
 
-# An example authorization handler used so that users can be verified against
-# third party systems.
-#
-# You should probably rename this class and file to match your needs.
-#
-# If you need a custom form to be rendered, you can create a file matching the
-# class name named "_form".
-#
-# Example:
-#
-#   A handler named Decidim::CensusHandler would look for its partial in:
-#   decidim/census/form
-#
-# When testing your authorization handler, add this line to be sure it has a
-# valid public api:
-#
-#   it_behaves_like "an authorization handler"
-#
-# See Decidim::AuthorizationHandler for more documentation.
-class SocioDemographicAuthorizationHandler < Decidim::AuthorizationHandler
-
-  attribute :scope, String
-  attribute :gender, String
-  attribute :age, String
-
-  GENDER = %w[man woman undefined].freeze
-  SCOPES = Decidim::Scope.all.map { |scope| scope.code.downcase }
-  AGE_SLICE = %w[16-25 26-45 46-65 65+].freeze
-
-  validates :scope,
-            inclusion: { in: SCOPES },
-            presence: true
-
-  validates :gender,
-            inclusion: { in: GENDER },
-            presence: true
-
-  validates :age,
-            inclusion: { in: AGE_SLICE },
-            presence: true
-
-  def metadata
-    super.merge(scope: scope, gender: gender, age: age)
-  end
 
-end
+# Allows to create a form for simple Socio Demographic authorization
+  class SocioDemographicAuthorizationHandler < Decidim::AuthorizationHandler
+
+    attribute :scope, String
+    attribute :gender, String
+    attribute :age, String
+
+    SCOPES = Decidim::Scope.all.map { |scope| scope.code.downcase }
+    GENDER = %w(man woman undefined).freeze
+    AGE_SLICE = %w(16-25 26-45 46-65 65+).freeze
+
+    validates :scope,
+              inclusion: { in: SCOPES },
+              presence: true
+
+    validates :gender,
+              inclusion: { in: GENDER },
+              presence: true
+
+    validates :age,
+              inclusion: { in: AGE_SLICE },
+              presence: true
+
+    def metadata
+      super.merge(scope: scope, gender: gender, age: age)
+    end
+
+  end

app/controllers/decidim/socio_demographic_authorization_handler/admin/application_controller.rb

@@ -14,7 +14,6 @@ def after_confirmation_path_for(resource_name, resource)
       sign_in(resource) unless user_signed_in? # In case you want to sign in the user
 
       if first_login_and_not_authorized?(resource)
-        # TODO: new_authorization_path est une méthode d'ENGINE !!
         Decidim::Verifications.find_workflow_manifest :socio_demographic_authorization_handler .new_authorization_path(action: :new)
       else
         super

app/controllers/decidim/socio_demographic_authorization_handler/application_controller.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 require "decidim/socio_demographic_authorization_handler/workflow"
-require "decidim/socio_demographic_authorization_handler/socio_demographic_authorization_handler"
+#require "decidim/socio_demographic_authorization_handler/socio_demographic_authorization_handler"
 
 module Decidim
   # This namespace holds the logic of the `SocioDemographicAuthorizationHandler` component. This component

app/controllers/decidim/socio_demographic_authorization_handler/socio_demographic_authorization_controller.rb

@@ -9,6 +9,7 @@
 
   context "when a new user" do
     let(:organization) { create :organization, available_authorizations: authorizations }
+    let!(:scopes) { create_list(:scope, 9, organization: organization) }
 
     let(:user) { create(:user, :confirmed, organization: organization) }
 
@@ -27,9 +28,9 @@
       end
 
       it "redirects the user to the authorization form after the first sign in" do
-        fill_in :scope, with: organization.scopes
-        fill_in :gender, with: %w(Man Woman Undefined)
-        fill_in :age, with: %w(16-25 26-45 46-65 65+)
+        find('select#authorization_handler_scope').find("option", match: :first).select_option
+        find('select#authorization_handler_gender').find("option", match: :first).select_option
+        find('select#authorization_handler_age').find("option", match: :first).select_option
 
         click_button "Send"
         expect(page).to have_content("You've been successfully authorized")
@@ -82,9 +83,10 @@
         click_link "Authorizations"
         click_link "Socio Demographic Authorization"
 
-        fill_in "Document number", with: "123456789X"
-        page.execute_script("$('#authorization_handler_birthday').focus()")
-        page.find(".datepicker-dropdown .day", text: "12").click
+        fill_in :scope, with: organization.scopes
+        fill_in :gender, with: %w(Man Woman Undefined)
+        fill_in :age, with: %w(16-25 26-45 46-65 65+)
+
         click_button "Send"
 
         expect(page).to have_content("You've been successfully authorized")
@@ -107,9 +109,10 @@
         click_link "Authorizations"
         click_link "Socio Demographic Authorization"
 
-        fill_in "Document number", with: "12345678"
-        page.execute_script("$('#authorization_handler_birthday').focus()")
-        page.find(".datepicker-dropdown .day", text: "12").click
+        fill_in :scope, with: organization.scopes
+        fill_in :gender, with: %w(Man Woman Undefined)
+        fill_in :age, with: %w(16-25 26-45 46-65 65+)
+
         click_button "Send"
 
         expect(page).to have_content("There was a problem creating the authorization.")
@@ -171,7 +174,7 @@
             click_link "Socio Demographic Authorization"
           end
 
-          fill_in "Document number", with: "123456789X"
+          fill_in :scope, with: organization.scopes
           click_button "Send"
 
           expect(page).to have_content("You've been successfully authorized")