Add UK-specific excluded calculated homepage domains

Requires mysociety/alaveteli#7836
mysociety · Oct 19, 2023 · 9f91e1c · 9f91e1c
1 parent 6217d9e
commit 9f91e1c
Show file tree

Hide file tree

Showing 2 changed files with 32 additions and 0 deletions.
diff --git a/lib/model_patches.rb b/lib/model_patches.rb
@@ -73,6 +73,23 @@ def late_calculator
     prepend ProAccountBans::ModelMethods
   end
 
+  PublicBody.excluded_calculated_home_page_domains += %w[
+    aol.co.uk
+    blueyonder.co.uk
+    btconnect.com
+    btinternet.com
+    btopenworld.com
+    hotmail.co.uk
+    live.co.uk
+    ntlworld.com
+    sky.com
+    talk21.com
+    talktalk.net
+    tiscali.co.uk
+    virginmedia.com
+    yahoo.co.uk
+  ]
+
   PublicBody.class_eval do
     # Return the domain part of an email address, canonicalised and with common
     # extra UK Government server name parts removed.

diff --git a/spec/public_body_spec.rb b/spec/public_body_spec.rb
@@ -0,0 +1,15 @@
+require_relative 'spec_helper'
+
+RSpec.describe PublicBody do
+  describe '.excluded_calculated_home_page_domains' do
+    subject { described_class.excluded_calculated_home_page_domains }
+
+    it 'includes default global domains' do
+      is_expected.to include('aol.com')
+    end
+
+    it 'includes expected localise domains' do
+      is_expected.to include('aol.co.uk')
+    end
+  end
+end