Fix bug causing when am I on call to error out.

This was happening if the person is not on any schedule.

Fixes #51

lib/pagerbot/action_manager.rb

@@ -115,7 +115,12 @@ def lookup_person(query, event_data)
       #   person goes on any on call.
       if query[:schedule] == 'call'
         next_oncall = @pagerduty.next_oncall(person.id, nil)
-        schedule = @pagerduty.find_schedule(next_oncall[:schedule][:name])
+        if next_oncall[:schedule].nil?
+          schedule = nil
+          next_oncall = nil
+        else
+          schedule = @pagerduty.find_schedule(next_oncall[:schedule][:name])
+        end
       else
         schedule = @pagerduty.find_schedule(query[:schedule])
         next_oncall = @pagerduty.next_oncall(person.id, schedule.id)
@@ -126,7 +131,7 @@ def lookup_person(query, event_data)
         person: person,
         schedule: schedule
       }
-      if next_oncall
+      unless next_oncall.nil?
         output_data[:time] = person.parse_time(next_oncall[:start])
       end
 

lib/pagerbot/utilities.rb

@@ -107,6 +107,7 @@ def self.display_time(time)
   end
 
   def self.display_schedule(schedule, text=nil)
+    return 'any schedule' if schedule.nil?
     url = PagerBot.pagerduty.uri_base+"/schedules##{schedule.id}"
     link_to(url, text || schedule.name)
   end

templates/lookup_person_irc.erb

@@ -1,5 +1,9 @@
 <% if time.nil? %>
-<%= person.name %> is not scheduled to go on <%= schedule.name %>
+<%= person.name %> is not scheduled to go on <% if schedule.nil? %>
+any schedule
+<% else %>
+<%= schedule.name %>
+<% end %>
 <% elsif time < Time.now %>
 <%= person.name %> is on <%= schedule.name %> now
 <% else %>

templates/lookup_person_slack.erb

@@ -1,5 +1,9 @@
 <% if time.nil? %>
-<%= person.name %> is not scheduled to go on <%= utilities.display_schedule(schedule) %>.
+<%= person.name %> is not scheduled to go on <% if schedule.nil? %>
+any schedule
+<% else %>
+<%= utilities.display_schedule(schedule) %>
+<% end %>
 <% elsif time < Time.now %>
 <%= person.name %> is on <%= utilities.display_schedule(schedule) %> now.
 <% else %>

test/unit/pagerbot/action_manager.rb

@@ -122,6 +122,19 @@ def event_data(opts={nick: "karl"})
 
         assert_equal("Karl-Aksel Puulmann is on Primary breakage now", response.fetch(:message))
       end
+
+      it 'special case: on schedule `call` should succeed if person is not to go on any schedule' do
+        @pagerduty.expects(:next_oncall)
+          .with('P123456', nil)
+          .returns schedule: nil
+
+        response = @manager.lookup_person({
+          :person => "karl",
+          :schedule => "call"
+        }, event_data)
+
+        assert_equal("Karl-Aksel Puulmann is not scheduled to go on any schedule", response.fetch(:message))
+      end
     end
 
     describe 'who is on schedule X' do