lita-pagerduty 0.2.0 → 1.0.0

data/spec/spec_helper.rb

@@ -8,204 +8,18 @@ SimpleCov.start { add_filter '/spec/' }
 
 require 'lita-pagerduty'
 require 'lita/rspec'
-
 Lita.version_3_compatibility_mode = false
 
 RSpec.configure do |config|
   config.before do
-    registry.register_handler(Lita::Handlers::PagerdutyAck)
-    registry.register_handler(Lita::Handlers::PagerdutyIncident)
-    registry.register_handler(Lita::Handlers::PagerdutyNote)
-    registry.register_handler(Lita::Handlers::PagerdutyResolve)
-    registry.register_handler(Lita::Handlers::PagerdutyUtility)
-  end
-
-  config.expect_with :rspec do |expectations|
-    expectations.include_chain_clauses_in_custom_matcher_descriptions = true
-  end
-
-  config.mock_with :rspec do |mocks|
-    mocks.verify_partial_doubles = true
+    Lita.config.handlers.pagerduty.api_key = 'foo'
+    Lita.config.handlers.pagerduty.email = 'foo@pagerduty.com'
+    Lita.config.redis[:host] = ENV['REDIS_HOST'] if ENV['REDIS_HOST']
   end
 
   config.filter_run :focus
   config.run_all_when_everything_filtered = true
   config.default_formatter = 'doc' if config.files_to_run.one?
   config.order = :random
-
   Kernel.srand config.seed
 end
-
-RSpec.shared_context 'basic fixtures' do
-  let(:no_incident) do
-    client = double
-    expect(client).to receive(:get_incident) { 'No results' }
-    client
-  end
-
-  let(:no_incidents) do
-    client = double
-    expect(client).to receive(:incidents) do
-      double(
-        incidents: []
-      )
-    end
-    client
-  end
-
-  let(:incidents) do
-    client = double
-    expect(client).to receive(:incidents) do
-      double(
-        incidents: [
-          double(
-            id: 'ABC123',
-            status: 'resolved',
-            html_url: 'https://acme.pagerduty.com/incidents/ABC123',
-            trigger_summary_data: double(subject: 'something broke'),
-            assigned_to_user: double(email: 'foo@example.com')
-          ),
-          double(
-            id: 'ABC789',
-            status: 'triggered',
-            html_url: 'https://acme.pagerduty.com/incidents/ABC789',
-            trigger_summary_data: double(subject: 'Still broke'),
-            assigned_to_user: double(email: 'bar@example.com')
-          )
-        ]
-      )
-    end
-    allow(client).to receive(:get_incident) do
-      double(
-        status: 'triggered',
-        html_url: 'https://acme.pagerduty.com/incidents/ABC789',
-        trigger_summary_data: double(subject: 'Still broke'),
-        assigned_to_user: double(email: 'bar@example.com'),
-        acknowledge: { 'id' => 'ABC789', 'status' => 'acknowledged' },
-        resolve: { 'id' => 'ABC789', 'status' => 'resolved' },
-        notes: double(notes: [])
-      )
-    end
-    client
-  end
-
-  let(:new_incident) do
-    client = double
-    expect(client).to receive(:get_incident) do
-      double(
-        id: 'ABC123',
-        status: 'triggered',
-        html_url: 'https://acme.pagerduty.com/incidents/ABC123',
-        trigger_summary_data: double(subject: 'something broke'),
-        assigned_to_user: double(email: 'foo@example.com'),
-        acknowledge: { 'id' => 'ABC123', 'status' => 'acknowledged' },
-        resolve: { 'id' => 'ABC123', 'status' => 'resolved' },
-        notes: double(notes: [])
-      )
-    end
-    client
-  end
-
-  let(:incident_without_assigned_user) do
-    client = double
-    expect(client).to receive(:get_incident) do
-      double(
-        id: 'ABC456',
-        status: 'triggered',
-        html_url: 'https://acme.pagerduty.com/incidents/ABC456',
-        trigger_summary_data: double(subject: 'something broke'),
-        assigned_to_user: nil
-      )
-    end
-    client
-  end
-
-  let(:acknowledged_incident) do
-    client = double
-    expect(client).to receive(:get_incident) do
-      double(
-        status: 'acknowledged',
-        html_url: 'https://acme.pagerduty.com/incidents/ABC123',
-        trigger_summary_data: double(subject: 'something broke'),
-        assigned_to_user: double(email: 'foo@example.com'),
-        acknowledge: { 'error' =>
-          { 'message' => 'Incident Already Acknowledged', 'code' => 1002 }
-        },
-        resolve:  { 'id' => 'ABC123', 'status' => 'resolved' },
-        notes: double(notes: [])
-      )
-    end
-    client
-  end
-
-  let(:resolved_incident) do
-    client = double
-    expect(client).to receive(:get_incident) do
-      double(
-        status: 'resolved',
-        html_url: 'https://acme.pagerduty.com/incidents/ABC123',
-        trigger_summary_data: double(subject: 'something broke'),
-        assigned_to_user: double(email: 'foo@example.com'),
-        notes: double(notes: [])
-      )
-    end
-    client
-  end
-
-  let(:unable_to_resolve_incident) do
-    client = double
-    expect(client).to receive(:get_incident) do
-      double(
-        status: 'notresolved',
-        html_url: 'https://acme.pagerduty.com/incidents/ABC123',
-        resolve:  { 'id' => 'ABC123', 'status' => 'notresolved' }
-      )
-    end
-    client
-  end
-
-  let(:unable_to_ack_incident) do
-    client = double
-    expect(client).to receive(:get_incident) do
-      double(
-        status: 'notacked',
-        html_url: 'https://acme.pagerduty.com/incidents/ABC123',
-        acknowledge: { 'error' => {} },
-        resolve:  { 'id' => 'ABC123', 'status' => 'notacked' }
-      )
-    end
-    client
-  end
-
-  let(:incident_with_notes) do
-    client = double
-    expect(client).to receive(:get_incident) do
-      double(
-        id: 'ABC123',
-        status: 'resolved',
-        html_url: 'https://acme.pagerduty.com/incidents/ABC123',
-        trigger_summary_data: double(subject: 'something broke'),
-        assigned_to_user: double(email: 'foo@example.com'),
-        notes: double(
-          notes: [double(content: 'Hi!',
-                         user: double(email: 'foo@example.com'))]
-        )
-      )
-    end
-    client
-  end
-
-  let(:incident_with_long_id) do
-    client = double
-    expect(client).to receive(:get_incident) do
-      double(
-        id: 'ABC123456789',
-        status: 'triggered',
-        html_url: 'https://acme.pagerduty.com/incidents/ABC123456789',
-        trigger_summary_data: double(subject: 'something broke'),
-        assigned_to_user: double(email: 'foo@example.com')
-      )
-    end
-    client
-  end
-end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: lita-pagerduty
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 1.0.0
 platform: ruby
 authors:
 - Eric Sigler
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-03-11 00:00:00.000000000 Z
+date: 2018-10-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: lita
@@ -24,20 +24,6 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '4.0'
-- !ruby/object:Gem::Dependency
-  name: pagerduty-sdk
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 1.0.9
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 1.0.9
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -138,23 +124,48 @@ files:
 - LICENSE
 - README.md
 - Rakefile
+- lib/exceptions.rb
 - lib/lita-pagerduty.rb
-- lib/lita/handlers/pagerduty_ack.rb
-- lib/lita/handlers/pagerduty_incident.rb
-- lib/lita/handlers/pagerduty_note.rb
-- lib/lita/handlers/pagerduty_resolve.rb
-- lib/lita/handlers/pagerduty_utility.rb
-- lib/pagerduty_helper/incident.rb
-- lib/pagerduty_helper/regex.rb
-- lib/pagerduty_helper/utility.rb
+- lib/lita/commands/ack.rb
+- lib/lita/commands/ack_all.rb
+- lib/lita/commands/ack_mine.rb
+- lib/lita/commands/base.rb
+- lib/lita/commands/forget.rb
+- lib/lita/commands/identify.rb
+- lib/lita/commands/incident.rb
+- lib/lita/commands/incidents_all.rb
+- lib/lita/commands/incidents_mine.rb
+- lib/lita/commands/notes.rb
+- lib/lita/commands/on_call_list.rb
+- lib/lita/commands/on_call_lookup.rb
+- lib/lita/commands/pager_me.rb
+- lib/lita/commands/resolve.rb
+- lib/lita/commands/resolve_all.rb
+- lib/lita/commands/resolve_mine.rb
+- lib/lita/handlers/commands.yml
+- lib/lita/handlers/pagerduty.rb
+- lib/pagerduty.rb
+- lib/store.rb
 - lita-pagerduty.gemspec
 - locales/en.yml
-- spec/lita/handlers/pagerduty_ack_spec.rb
-- spec/lita/handlers/pagerduty_incident_spec.rb
-- spec/lita/handlers/pagerduty_note_spec.rb
-- spec/lita/handlers/pagerduty_resolve_spec.rb
-- spec/lita/handlers/pagerduty_utility_spec.rb
+- spec/lita/handlers/ack_all_spec.rb
+- spec/lita/handlers/ack_id_spec.rb
+- spec/lita/handlers/ack_mine_spec.rb
+- spec/lita/handlers/forget_spec.rb
+- spec/lita/handlers/identify_email_spec.rb
+- spec/lita/handlers/incident_id_spec.rb
+- spec/lita/handlers/incidents_all_spec.rb
+- spec/lita/handlers/incidents_mine_spec.rb
+- spec/lita/handlers/notes_spec.rb
+- spec/lita/handlers/on_call_list_spec.rb
+- spec/lita/handlers/on_call_lookup_spec.rb
+- spec/lita/handlers/pager_me_spec.rb
+- spec/lita/handlers/resolve_all_spec.rb
+- spec/lita/handlers/resolve_id_spec.rb
+- spec/lita/handlers/resolve_mine_spec.rb
+- spec/pagerduty_spec.rb
 - spec/spec_helper.rb
+- templates/.gitkeep
 homepage: http://github.com/esigler/lita-pagerduty
 licenses:
 - MIT
@@ -176,14 +187,25 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.5
+rubygems_version: 2.6.13
 signing_key: 
 specification_version: 4
 summary: A Lita handler to interact with PagerDuty
 test_files:
-- spec/lita/handlers/pagerduty_ack_spec.rb
-- spec/lita/handlers/pagerduty_incident_spec.rb
-- spec/lita/handlers/pagerduty_note_spec.rb
-- spec/lita/handlers/pagerduty_resolve_spec.rb
-- spec/lita/handlers/pagerduty_utility_spec.rb
+- spec/lita/handlers/ack_all_spec.rb
+- spec/lita/handlers/ack_id_spec.rb
+- spec/lita/handlers/ack_mine_spec.rb
+- spec/lita/handlers/forget_spec.rb
+- spec/lita/handlers/identify_email_spec.rb
+- spec/lita/handlers/incident_id_spec.rb
+- spec/lita/handlers/incidents_all_spec.rb
+- spec/lita/handlers/incidents_mine_spec.rb
+- spec/lita/handlers/notes_spec.rb
+- spec/lita/handlers/on_call_list_spec.rb
+- spec/lita/handlers/on_call_lookup_spec.rb
+- spec/lita/handlers/pager_me_spec.rb
+- spec/lita/handlers/resolve_all_spec.rb
+- spec/lita/handlers/resolve_id_spec.rb
+- spec/lita/handlers/resolve_mine_spec.rb
+- spec/pagerduty_spec.rb
 - spec/spec_helper.rb

data/lib/lita/handlers/pagerduty_ack.rb

@@ -1,77 +0,0 @@
-# Lita-related code
-module Lita
-  # Plugin-related code
-  module Handlers
-    # Acknowledge-related routes
-    class PagerdutyAck < Handler
-      namespace 'Pagerduty'
-
-      include ::PagerdutyHelper::Incident
-      include ::PagerdutyHelper::Regex
-      include ::PagerdutyHelper::Utility
-
-      route(
-        /^pager\sack\sall$/,
-        :ack_all,
-        command: true,
-        help: {
-          t('help.ack_all.syntax') => t('help.ack_all.desc')
-        }
-      )
-
-      route(
-        /^pager\sack\smine$/,
-        :ack_mine,
-        command: true,
-        help: {
-          t('help.ack_mine.syntax') => t('help.ack_mine.desc')
-        }
-      )
-
-      route(
-        /^pager\sack\s#{INCIDENT_ID_PATTERN}$/,
-        :ack,
-        command: true,
-        help: {
-          t('help.ack.syntax') => t('help.ack.desc')
-        }
-      )
-
-      # rubocop:disable Metrics/AbcSize
-      def ack_all(response)
-        incidents = fetch_all_incidents
-        return response.reply(t('incident.none')) unless incidents.count > 0
-        completed = []
-        incidents.each do |incident|
-          result = acknowledge_incident(incident.id)
-          completed.push(incident.id) if result == "#{incident.id}: Incident acknowledged"
-          response.reply(t('all.acknowledged', list: completed.join(', ')))
-        end
-      end
-      # rubocop:enable Metrics/AbcSize
-
-      # rubocop:disable Metrics/AbcSize
-      def ack_mine(response)
-        email = fetch_user(response.user)
-        return response.reply(t('identify.missing')) unless email
-        incidents = fetch_my_incidents(email)
-        return response.reply(t('incident.none_mine')) unless incidents.count > 0
-        completed = []
-        incidents.each do |incident|
-          result = acknowledge_incident(incident.id)
-          completed.push(incident.id) if result == "#{incident.id}: Incident acknowledged"
-          response.reply(t('all.acknowledged', list: completed.join(', ')))
-        end
-      end
-      # rubocop:enable Metrics/AbcSize
-
-      def ack(response)
-        incident_id = response.match_data['incident_id']
-        return if incident_id == 'all' || incident_id == 'mine'
-        response.reply(acknowledge_incident(incident_id))
-      end
-    end
-
-    Lita.register_handler(PagerdutyAck)
-  end
-end

data/lib/lita/handlers/pagerduty_incident.rb

@@ -1,68 +0,0 @@
-# Lita-related code
-module Lita
-  # Plugin-related code
-  module Handlers
-    # Incident-related routes
-    class PagerdutyIncident < Handler
-      namespace 'Pagerduty'
-
-      include ::PagerdutyHelper::Incident
-      include ::PagerdutyHelper::Regex
-      include ::PagerdutyHelper::Utility
-
-      route(
-        /^pager\sincidents\sall$/,
-        :incidents_all,
-        command: true,
-        help: {
-          t('help.incidents_all.syntax') => t('help.incidents_all.desc')
-        }
-      )
-
-      route(
-        /^pager\sincidents\smine$/,
-        :incidents_mine,
-        command: true,
-        help: {
-          t('help.incidents_mine.syntax') => t('help.incidents_mine.desc')
-        }
-      )
-
-      route(
-        /^pager\sincident\s#{INCIDENT_ID_PATTERN}$/,
-        :incident,
-        command: true,
-        help: {
-          t('help.incident.syntax') => t('help.incident.desc')
-        }
-      )
-
-      def incidents_all(response)
-        incidents = fetch_all_incidents
-        return response.reply(t('incident.none')) unless incidents.count > 0
-        incidents.each do |incident|
-          response.reply(format_incident(incident))
-        end
-      end
-
-      def incidents_mine(response)
-        email = fetch_user(response.user)
-        return response.reply(t('identify.missing')) unless email
-        incidents = fetch_my_incidents(email)
-        response.reply(t('incident.none_mine')) unless incidents.count > 0
-        incidents.each do |incident|
-          response.reply(format_incident(incident))
-        end
-      end
-
-      def incident(response)
-        incident_id = response.match_data['incident_id']
-        incident = fetch_incident(incident_id)
-        return response.reply(t('incident.not_found', id: incident_id)) if incident == 'No results'
-        response.reply(format_incident(incident))
-      end
-    end
-
-    Lita.register_handler(PagerdutyIncident)
-  end
-end