fixed registration_agency check for bmc_fulltext. #381

app/models/sources/bmc_fulltext.rb

@@ -1,7 +1,7 @@
 class BmcFulltext < Source
   def get_query_url(work, options = {})
     # don't query if work is BMC article
-    return {} if work.doi =~ /^10.1186/ && registration_agencies.include?(work.registration_agency)
+    return {} if work.doi =~ /^10.1186/ || !registration_agencies.include?(work.registration_agency)
 
     query_string = get_query_string(work)
     return {} unless query_string.present?

spec/models/bmc_fulltext_spec.rb

@@ -3,7 +3,7 @@
 describe BmcFulltext, type: :model, vcr: true do
   subject { FactoryGirl.create(:bmc_fulltext) }
 
-  let(:work) { FactoryGirl.create(:work, doi: nil, canonical_url: "https://github.com/najoshi/sickle") }
+  let(:work) { FactoryGirl.create(:work, doi: nil, canonical_url: "https://github.com/najoshi/sickle", registration_agency: "github") }
 
   context "lookup canonical URL" do
     it "should look up canonical URL if there is no work url" do
@@ -29,7 +29,7 @@
     end
 
     it "should report if there are no events returned by the BMC Search API" do
-      work = FactoryGirl.create(:work, doi: nil, canonical_url: "https://github.com/pymor/pymor")
+      work = FactoryGirl.create(:work, doi: nil, canonical_url: "https://github.com/pymor/pymor", registration_agency: "github")
       response = subject.get_data(work)
       expect(response).to eq("entries"=>[])
     end