Merge pull request #71 from mdwheele/fix-america-edu

Adds america.edu to blacklist
This commit is contained in:
Aidan Feldman 2014-07-16 09:28:52 -04:00
commit 049eff3900
2 changed files with 2 additions and 1 deletions

View File

@ -9,6 +9,7 @@ module Swot
# Note: validated domain must not end with the blacklisted string
BLACKLIST = %w(
si.edu
america.edu
).freeze
class << self

View File

@ -62,7 +62,7 @@ describe Swot do
end
it "fail blacklisted domains" do
["si.edu", " si.edu ", "imposter@si.edu", "foo.si.edu"].each do |domain|
["si.edu", " si.edu ", "imposter@si.edu", "foo.si.edu", "america.edu"].each do |domain|
assert_equal false, Swot::is_academic?(domain), "#{domain} should be denied"
end
end