Merge pull request #1668 from DFE-Digital/2241-trim-whitespace-from-f…

…ilters

Remove leading and trailing whitespace from filters

app/lib/filters/email.rb

@@ -4,7 +4,7 @@ module Filters
   class Email < Base
     def apply
       if email.present?
-        scope.joins(:teacher).where("teachers.email = ?", email.downcase)
+        scope.joins(:teacher).where("teachers.email = ?", email.strip.downcase)
       else
         scope
       end

app/lib/filters/name.rb

@@ -5,7 +5,10 @@ class Name < Base
     def apply
       return scope if name.blank?
 
-      scope.where("CONCAT(given_names, ' ', family_name) ilike ?", "%#{name}%")
+      scope.where(
+        "CONCAT(given_names, ' ', family_name) ilike ?",
+        "%#{name.strip}%",
+      )
     end
 
     private

app/lib/filters/reference.rb

@@ -4,7 +4,7 @@ module Filters
   class Reference < Base
     def apply
       if reference.present?
-        scope.where("reference ILIKE ?", "%#{reference}%")
+        scope.where("reference ILIKE ?", "%#{reference.strip}%")
       else
         scope
       end

spec/lib/filters/email_spec.rb

@@ -31,6 +31,40 @@
 
       it { is_expected.to contain_exactly(included) }
     end
+
+    context "with trailing whitespace" do
+      let(:params) { { email: "[email protected]    " } }
+      let(:scope) { ApplicationForm.all }
+      let!(:included) { create(:application_form, teacher:) }
+
+      before do
+        create(
+          :application_form,
+          teacher: create(:teacher, email: "[email protected]"),
+        )
+      end
+
+      it "returns a filtered scope" do
+        expect(subject).to contain_exactly(included)
+      end
+    end
+
+    context "with leading whitespace" do
+      let(:params) { { email: "    [email protected]" } }
+      let(:scope) { ApplicationForm.all }
+      let!(:included) { create(:application_form, teacher:) }
+
+      before do
+        create(
+          :application_form,
+          teacher: create(:teacher, email: "[email protected]"),
+        )
+      end
+
+      it "returns a filtered scope" do
+        expect(subject).to contain_exactly(included)
+      end
+    end
   end
 
   context "the params don't include email" do

spec/lib/filters/name_spec.rb

@@ -97,6 +97,36 @@
       end
     end
 
+    context "with trailing whitespace" do
+      let(:params) { { name: "Jane Smith    " } }
+      let(:scope) { ApplicationForm.all }
+      let!(:included) do
+        create(:application_form, given_names: "Jane", family_name: "Smith")
+      end
+      let!(:excluded) do
+        create(:application_form, given_names: "Tom", family_name: "Bombadil")
+      end
+
+      it "returns a filtered scope" do
+        expect(subject).to contain_exactly(included)
+      end
+    end
+
+    context "with leading whitespace" do
+      let(:params) { { name: "    Jane Smith" } }
+      let(:scope) { ApplicationForm.all }
+      let!(:included) do
+        create(:application_form, given_names: "Jane", family_name: "Smith")
+      end
+      let!(:excluded) do
+        create(:application_form, given_names: "Tom", family_name: "Bombadil")
+      end
+
+      it "returns a filtered scope" do
+        expect(subject).to contain_exactly(included)
+      end
+    end
+
     context "match with different case" do
       let(:params) { { name: "daVe" } }
 

spec/lib/filters/reference_spec.rb

@@ -40,4 +40,26 @@
 
     it { is_expected.to eq(scope) }
   end
+
+  context "with trailing whitespace" do
+    let(:params) { { reference: "ABCDEF    " } }
+    let(:scope) { ApplicationForm.all }
+    let!(:included) { create(:application_form, reference: "ABCDEF") }
+    let!(:excluded) { create(:application_form, reference: "QRHGF") }
+
+    it "returns a filtered scope" do
+      expect(subject).to contain_exactly(included)
+    end
+  end
+
+  context "with leading whitespace" do
+    let(:params) { { reference: "    ABCDEF" } }
+    let(:scope) { ApplicationForm.all }
+    let!(:included) { create(:application_form, reference: "ABCDEF") }
+    let!(:excluded) { create(:application_form, reference: "QRHGF") }
+
+    it "returns a filtered scope" do
+      expect(subject).to contain_exactly(included)
+    end
+  end
 end