Check to make sure Zotero integration is enabled before asking user i…

…nstances if they have a Zotero user ID. Fixes #1235

app/views/users/_user_info.html.erb

@@ -5,7 +5,7 @@
   <dd class="col-xs-7"><%= link_to user.orcid, user.orcid, { target: '_blank' } %></dd>
 <% end %>
 
-<% if user.zotero_userid.present? %>
+<% if Sufia.config.arkivo_api && user.zotero_userid.present? %>
   <dt class="col-xs-5"><%= zotero_label(html_class: 'profile') %></dt>
   <dd class="col-xs-7"><%= link_to zotero_profile_url(user.zotero_userid), zotero_profile_url(user.zotero_userid), { target: '_blank' } %></dd>
 <% end %>

spec/views/users/_user_info.html.erb_spec.rb

@@ -0,0 +1,29 @@
+require 'spec_helper'
+
+describe 'users/_user_info.html.erb', type: :view do
+  let(:user) { stub_model(User, user_key: 'jdoe42') }
+
+  context 'with Zotero disabled' do
+    before do
+      allow(Sufia.config).to receive(:arkivo_api) { false }
+      allow(user).to receive(:zotero_userid).and_raise(NoMethodError)
+      render "users/user_info", user: user
+    end
+
+    it 'does not display a Zotero profile link' do
+      expect(rendered).not_to match(/Zotero Profile/)
+    end
+  end
+
+  context 'with Zotero enabled' do
+    before do
+      allow(Sufia.config).to receive(:arkivo_api) { true }
+      allow(user).to receive(:zotero_userid) { 'jdoe42zotero' }
+      render "users/user_info", user: user
+    end
+
+    it 'displays a Zotero profile link' do
+      expect(rendered).to match(/Zotero Profile/)
+    end
+  end
+end