slack_message 1.6.0 → 1.8.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e66bdedefd2ac86c6c49d974e5f5d0474cc9b3756ce3a64e52df66ccc6e39cf4
-  data.tar.gz: b085edf556d957516d1cbbc9b260b7aff0e5d34b4561d27ae0d5f6b2dda9c93b
+  metadata.gz: 94e7433c5374c318cd955ad43f6a02705b62876498253c4e7587a7ba5ae5277f
+  data.tar.gz: 0ca96c1266c9dc55af30c6df7f07e2ea86a64bc5fad99258a10efe10e4127e9b
 SHA512:
-  metadata.gz: ea935db68ca4771e11b40828d5cbd2fef02cf10c248fab3ede05ea82aef6a3a6d7ad96953debdc9214833ee0f281222e5e09adefd923c70d21090c996dba31a0
-  data.tar.gz: 46d2163aa9ec0e9df582700575d0c8b47b389b808f93275129025bc277086eb02fbeb4caa0754408dbc6d6fba9f06b6d8720493280550e0d3664bc735059f7c1
+  metadata.gz: 2f4c15c91c6a7741c1f387b820cc36689cd7530233e9582fb3db73407d9f39eb69656a071b65789cd33ec359d5fa1478efe212336a3834202c30f96269885895
+  data.tar.gz: ed86a645c481c0c91510acedc5b0abd2bf3923f9ea33e9a23671db0e107005c07f64a1dfc52f4f5f47f826630712756bd5ce43ddfd06622a10ed24afa7731465

data/CHANGELOG.md

@@ -2,6 +2,23 @@
 
 ## [Unreleased]
 
+## [1.8.1] - 2021-10-08
+- Cleaned that rspec code a bit, added more matchers for real world use.
+
+## [1.8.0] - 2021-10-07
+- Added the ability to test in RSpec.
+
+## [1.7.1] - 2021-10-06
+- Fixed literally a syntax issue.
+- Fixed specs.
+- Fixed API to include JSON since consumers may not have loaded it.
+
+## [1.7.0] - 2021-10-06
+- THIS RELEASE IS BADLY BROKEN.
+- Added new error messages when API configuration is wrong / missing.
+- Fixed issue with `instance_eval` and using methods within block.
+- Fixed issue with sectionless `list_item`.
+
 ## [1.6.0] - 2021-10-04
 - Added `:default_channel` and `post_as` to deal with repetitive channel usage.
 
@@ -9,7 +26,7 @@
 - Added `ol` and `ul` to sections w/ some formatting.
 
 ## [1.4.0] - 2021-09-27
-- Moved image to accessory_image to differentiate between the image block
+- Changed `image` to `accessory_image` to differentiate between the image block
   and the accessory image within a block.
 
 ## [1.3.0] - 2021-09-27
@@ -18,13 +35,12 @@
 - Added warnings for potentially invalid URLs.
 
 ## [1.2.0] - 2021-09-26
-- Turns out gemspec was broken. Fixed that.
+- Fixed gemspec, which was entirely broken.
 
 ## [1.1.0] - 2021-09-26
 - Expanded the README significantly w/ usage instructions.
 - Added lots of error handling to requests.
 
 ## [1.0.0] - 2021-09-25
-
 - Added the base gem w/ a DSL for constructing blocks using sections.
 - Added a changelog, apparently.

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    slack_message (1.2.0)
+    slack_message (1.8.0)
 
 GEM
   remote: https://rubygems.org/

data/README.md

@@ -214,6 +214,36 @@ SlackMessage.post_to('#general') do
 end
 ```
 
+### Testing
+
+You can do some basic testing against SlackMessage, at least if you use RSpec!
+You'll need to require and include the testing behavior like this, in your
+spec_helper file:
+
+```ruby
+require 'slack_message/rspec'
+
+RSpec.configure do |config|
+  include SlackMessage::RSpec
+
+  # your other config
+end
+```
+
+This will stop API calls for posting messages, and will allow you access to
+some custom matchers:
+
+```ruby
+expect {
+  SlackMessage.post_to('#general') { text "foo" }
+}.to post_slack_message_to('#general').with_content_matching(/foo/)
+```
+
+Be forewarned, I'm frankly not that great at more complicated RSpec matchers,
+so I'm guessing there are some bugs. Also, because the content of a message
+gets turned into a complex JSON object, matching against content isn't capable
+of very complicated regexes.
+
 Opinionated Stances
 ------------
 
@@ -225,6 +255,8 @@ opinionated stances on how to make use of that API. For instance:
 * Generally, same goes for the `emoji` flag on almost every text element.
 * It's possible to ask for a `blank_line` in sections, even though that concept
   isn't real. In this case, a text line containing only an emspace is rendered.
+* It's easy to configure a bot for consistent name / channel use. My previous
+  use of SlackNotifier led to frequently inconsistent names.
 
 What it Doesn't Do
 ------------
@@ -240,11 +272,11 @@ DSL to include more of the block API itself.
 
 Also, some behaviors that are still planned but not yet added:
 
+* some API documentation amirite?
 * allow custom http_options in configuration
 * more of BlockKit's options
 * any interactive elements at all (I don't understand them yet)
 * more interesting return types for your message
-* some way to specify default channel for a given profile (and omit param to post_to)
 * richer text formatting (ul is currently a hack)
 
 Contributing

data/lib/slack_message/api.rb

@@ -1,5 +1,6 @@
 require 'net/http'
 require 'net/https'
+require 'json'
 
 class SlackMessage::Api
   def self.user_id_for(email)
@@ -17,9 +18,16 @@ class SlackMessage::Api
 
     if response.code != "200"
       raise "Got an error back from the Slack API (HTTP #{response.code}):\n#{response.body}"
+    elsif response.body == ""
+      raise "Received empty 200 response from Slack when looking up user info. Check your API key."
+    end
+
+    begin
+      payload = JSON.parse(response.body)
+    rescue
+      raise "Unable to parse JSON response from Slack API\n#{response.body}"
     end
 
-    payload = JSON.parse(response.body)
     if payload.include?("error") && payload["error"] == "invalid_auth"
       raise "Received an error because your authentication token isn't properly configured:\n#{response.body}"
     elsif payload.include?("error")
@@ -37,9 +45,9 @@ class SlackMessage::Api
       channel: target,
       username: profile[:name],
       blocks: payload
-    }.to_json
+    }
 
-    response = Net::HTTP.post_form uri, { payload: params }
+    response = execute_post_form(uri, params, profile[:handle])
 
     # let's try to be helpful about error messages
     if response.body == "invalid_token"
@@ -48,10 +56,16 @@ class SlackMessage::Api
       raise "Tried to send Slack message to non-existent channel or user '#{target}'"
     elsif response.body == "missing_text_or_fallback_or_attachments"
       raise "Tried to send Slack message with invalid payload."
+    elsif response.code == "302"
+      raise "Got 302 response while posting to Slack. Check your webhook URL for '#{profile[:handle]}'."
     elsif response.code != "200"
       raise "Got an error back from the Slack API (HTTP #{response.code}):\n#{response.body}"
     end
 
     response
   end
+
+  def self.execute_post_form(uri, params, _profile)
+    Net::HTTP.post_form uri, { payload: params.to_json }
+  end
 end

data/lib/slack_message/configuration.rb

@@ -27,6 +27,10 @@ module SlackMessage::Configuration
 
   ###
 
+  def self.clear_profiles! # test harness, mainly
+    @@profiles = {}
+  end
+
   def self.add_profile(handle = :default, name:, url:, default_channel: nil)
     if @@profiles.include?(handle)
       warn "WARNING: Overriding profile '#{handle}' in SlackMessage config"

data/lib/slack_message/dsl.rb

@@ -3,7 +3,11 @@ class SlackMessage::Dsl
 
   EMSPACE = " " # unicode emspace
 
-  def initialize
+  def initialize(block)
+    # Delegate missing methods to caller scope. Thanks 2008:
+    # https://www.dan-manges.com/blog/ruby-dsls-instance-eval-with-delegation
+    @caller_self = eval("self", block.binding)
+
     @body = []
     @default_section = Section.new
     @custom_bot_name = nil
@@ -81,13 +85,17 @@ class SlackMessage::Dsl
     @body
   end
 
+  def method_missing(meth, *args, &blk)
+    @caller_self.send meth, *args, &blk
+  end
+
   private
 
   # when doing things that would generate new top-levels, first try
   # to finish the implicit section.
   def finalize_default_section
     if default_section.has_content?
-      @body.push(default_section.body)
+      @body.push(default_section.render)
     end
 
     @default_section = Section.new

data/lib/slack_message/rspec.rb

@@ -0,0 +1,168 @@
+require 'rspec/expectations'
+require 'rspec/mocks'
+
+# Honestly, this code is what happens when you do not understand the RSpec
+# custom expectation API really at all, but you really want to create your
+# matcher. This code is soo baaad.
+#
+# We override API calls by entirely replacing the low-level API method. Then we
+# use our overridden version to capture and record calls. When someone creates
+# a new expectation, an object is created, so we allow that object to register
+# itself to receive notification when a slack message _would have_ been posted.
+#
+# Then once the expectation is fulfilled, that class unregisters itself so that
+# it can be cleaned up properly.
+#
+
+module SlackMessage::RSpec
+  extend RSpec::Matchers::DSL
+
+  @@listeners = []
+
+  def self.register_expectation_listener(expectation_instance)
+    @@listeners << expectation_instance
+  end
+
+  def self.unregister_expectation_listener(expectation_instance)
+    @@listeners.delete(expectation_instance)
+  end
+
+  FauxResponse = Struct.new(:code, :body)
+
+  def self.included(_)
+    SlackMessage::Api.singleton_class.undef_method(:execute_post_form)
+    SlackMessage::Api.define_singleton_method(:execute_post_form) do |uri, params, profile|
+      @@listeners.each do |listener|
+        listener.record_call(params.merge(profile: profile, uri: uri))
+      end
+
+      return FauxResponse.new('200', 'ok')
+    end
+  end
+
+  # w/ channel
+  matcher :post_slack_message_to do |expected|
+    match do |actual|
+      @instance ||= PostTo.new
+      @instance.with_channel(expected)
+
+      actual.call
+      @instance.enforce_expectations
+    end
+
+    chain :with_content_matching do |content|
+      @instance ||= PostTo.new
+      @instance.with_content_matching(content)
+    end
+
+    failure_message { @instance.failure_message }
+    failure_message_when_negated { @instance.failure_message_when_negated }
+
+    supports_block_expectations
+  end
+
+  # no channel
+  matcher :post_to_slack do |expected|
+    match do |actual|
+      @instance ||= PostTo.new
+
+      actual.call
+      @instance.enforce_expectations
+    end
+
+    chain :with_content_matching do |content|
+      @instance ||= PostTo.new
+      @instance.with_content_matching(content)
+    end
+
+    failure_message { @instance.failure_message }
+    failure_message_when_negated { @instance.failure_message_when_negated }
+
+    supports_block_expectations
+  end
+
+  # name / profile matcher
+  matcher :post_slack_message_as do |expected|
+     match do |actual|
+      @instance ||= PostTo.new
+      @instance.with_profile(expected)
+
+      actual.call
+      @instance.enforce_expectations
+    end
+
+    chain :with_content_matching do |content|
+      @instance ||= PostTo.new
+      @instance.with_content_matching(content)
+    end
+
+    failure_message { @instance.failure_message }
+    failure_message_when_negated { @instance.failure_message_when_negated }
+
+    supports_block_expectations
+  end
+
+  class PostTo
+    def initialize
+      @captured_calls = []
+      @content = nil
+      @channel = nil
+      @profile = nil
+
+      SlackMessage::RSpec.register_expectation_listener(self)
+    end
+
+    def record_call(deets)
+      @captured_calls.push(deets)
+    end
+
+    def with_channel(channel)
+      @channel = channel
+    end
+
+    def with_content_matching(content)
+      raise ArgumentError unless content.is_a? Regexp
+      @content = content
+    end
+
+    def with_profile(profile)
+      @profile = profile
+    end
+
+    def enforce_expectations
+      SlackMessage::RSpec.unregister_expectation_listener(self)
+
+      @captured_calls
+        .filter { |call| !@channel || call[:channel] == @channel }
+        .filter { |call| !@profile || [call[:profile], call[:username]].include?(@profile) }
+        .filter { |call| !@content || call.fetch(:blocks).to_s =~ @content }
+        .any?
+    end
+
+    def failure_message
+      "expected block to #{failure_expression}"
+    end
+
+    def failure_message_when_negated
+      "expected block not to #{failure_expression}"
+    end
+
+    def failure_expression
+      concat = []
+
+      if @channel
+        concat << "post a slack message to '#{@channel}'"
+      elsif @profile
+        concat << "post a slack message as '#{@profile}'"
+      else
+        concat << "post a slack message"
+      end
+
+      if @content
+        concat << "with content matching #{@content.inspect}"
+      end
+
+      concat.join " "
+    end
+  end
+end

data/lib/slack_message.rb

@@ -16,7 +16,7 @@ module SlackMessage
   end
 
   def self.post_to(target, as: :default, &block)
-    payload = Dsl.new.tap do |instance|
+    payload = Dsl.new(block).tap do |instance|
       instance.instance_eval(&block)
     end
 
@@ -27,7 +27,7 @@ module SlackMessage
   end
 
   def self.post_as(profile_name, &block)
-    payload = Dsl.new.tap do |instance|
+    payload = Dsl.new(block).tap do |instance|
       instance.instance_eval(&block)
     end
 
@@ -43,7 +43,7 @@ module SlackMessage
   end
 
   def self.build(&block)
-    Dsl.new.tap do |instance|
+    Dsl.new(block).tap do |instance|
       instance.instance_eval(&block)
     end.send(:render)
   end

data/slack_message.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |gem|
   gem.name        = 'slack_message'
-  gem.version     = "1.6.0"
+  gem.version     = "1.8.1"
   gem.summary     = "A nice DSL for composing rich messages in Slack"
   gem.authors     = ["Joe Mastey"]
   gem.email       = 'hello@joemastey.com'

data/spec/slack_message_spec.rb

@@ -1,15 +1,12 @@
 require 'spec_helper'
 
 RSpec.describe SlackMessage do
-  it "includes a bunch of stuff" do
-    expect(SlackMessage).to respond_to(:post_to)
-  end
-
   describe "API convenience" do
     it "can grab user IDs" do
-      allow(SlackMessage::Api).to receive(:api_request)
-        .with(/hello@joemastey.com/)
-        .and_return({ "user" => { "id" => "ABC123" }})
+      SlackMessage.configure { |c| c.api_token = "asdf" }
+      allow(Net::HTTP).to receive(:start).and_return(
+        double(code: "200", body: '{ "user": { "id": "ABC123" }}')
+      )
 
       result = SlackMessage.user_id_for("hello@joemastey.com")
       expect(result).to eq("ABC123")
@@ -49,7 +46,7 @@ RSpec.describe SlackMessage do
 
     it "raises errors for missing configuration" do
       SlackMessage.configure do |config|
-        #config.api_token = "abc123"
+        config.api_token = nil
       end
 
       expect {
@@ -63,7 +60,6 @@ RSpec.describe SlackMessage do
         config.add_profile(:nonstandard, name: 'another profile', url: 'http://hooks.slack.com/1234/')
       end
 
-
       expect(SlackMessage.configuration.profile(:default)[:name]).to eq('default profile')
       expect(SlackMessage.configuration.profile(:nonstandard)[:name]).to eq('another profile')
 
@@ -72,4 +68,55 @@ RSpec.describe SlackMessage do
       }.to raise_error(ArgumentError)
     end
   end
+
+  describe "custom expectations" do
+    before do
+      SlackMessage.configure do |config|
+        config.clear_profiles!
+        config.add_profile(name: 'default profile', url: 'http://hooks.slack.com/1234/')
+      end
+    end
+
+    it "can assert expectations against posts" do
+      expect {
+        SlackMessage.post_to('#lieutenant') { text "foo" }
+      }.not_to post_slack_message_to('#general')
+
+      expect {
+        SlackMessage.post_to('#general') { text "foo" }
+      }.to post_slack_message_to('#general').with_content_matching(/foo/)
+    end
+
+    it "resets state properly" do
+      expect {
+        SlackMessage.post_to('#general') { text "foo" }
+      }.to post_slack_message_to('#general')
+
+      expect { }.not_to post_slack_message_to('#general')
+    end
+
+    it "lets you assert by profile name" do
+      SlackMessage.configure do |config|
+        config.add_profile(:schmoebot, name: 'Schmoe', url: 'http://hooks.slack.com/1234/', default_channel: '#schmoes')
+      end
+
+      expect {
+        SlackMessage.post_as(:schmoebot) { text "foo" }
+      }.to post_slack_message_to('#schmoes')
+
+      expect {
+        SlackMessage.post_as(:schmoebot) { text "foo" }
+      }.to post_slack_message_as(:schmoebot)
+
+      expect {
+        SlackMessage.post_as(:schmoebot) { text "foo" }
+      }.to post_slack_message_as('Schmoe').with_content_matching(/foo/)
+    end
+
+    it "can assert more generally too tbh" do
+      expect {
+        SlackMessage.post_to('#general') { text "foo" }
+      }.to post_to_slack.with_content_matching(/foo/)
+    end
+  end
 end

data/spec/spec_helper.rb

@@ -1,4 +1,5 @@
 require_relative '../lib/slack_message'
+require_relative '../lib/slack_message/rspec'
 
 # This file was generated by the `rspec --init` command. Conventionally, all
 # specs live under a `spec` directory, which RSpec adds to the `$LOAD_PATH`.
@@ -49,4 +50,6 @@ RSpec.configure do |config|
   # test failures related to randomization by passing the same `--seed` value
   # as the one that triggered the failure.
   Kernel.srand config.seed
+
+  include SlackMessage::RSpec
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: slack_message
 version: !ruby/object:Gem::Version
-  version: 1.6.0
+  version: 1.8.1
 platform: ruby
 authors:
 - Joe Mastey
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-10-04 00:00:00.000000000 Z
+date: 2021-10-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
@@ -70,6 +70,7 @@ files:
 - lib/slack_message/api.rb
 - lib/slack_message/configuration.rb
 - lib/slack_message/dsl.rb
+- lib/slack_message/rspec.rb
 - slack_message.gemspec
 - spec/slack_message_spec.rb
 - spec/spec_helper.rb