Merge pull request #172 from github/no-regex-🧠

Simplify github_domain? check + add test

Author: yoannchaudet

lib/github-pages-health-check/domain.rb

@@ -283,7 +283,7 @@ def pages_dot_github_dot_com?
 
       # Is this domain owned by GitHub?
       def github_domain?
-        !!host.match(/(\A|\.)github\.com\.?\z/i)
+        host.downcase.eql?("github.com") || host.downcase.end_with?(".github.com")
       end
 
       # Is the host our Fastly CNAME?

spec/github_pages_health_check/domain_spec.rb

@@ -535,6 +535,14 @@
       end
     end
 
+    context "not github domains" do
+      let(:domain) { "somethinggithub.com" }
+
+      it "knows if the domain is not a github domain" do
+        expect(subject).to_not be_a_github_domain
+      end
+    end
+
     context "fastly domain" do
       let(:domain) { "github.map.fastly.net" }