slack-ruby-bot 0.10.3 → 0.10.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 88982b31e113e853ff4c707a7718f0739ccf260e
-  data.tar.gz: 9f818712b6659e1281a89b0a144b922f9b66e61d
+  metadata.gz: 12c173de864218ce41d7115da1a9c6ded20649eb
+  data.tar.gz: 01733d1afa189753bd3bc338a9508f138a69dbcb
 SHA512:
-  metadata.gz: 537ee07e13e6500c453b71cbfffa1b9eeda032baaf86319af4c298a6e65b3adc51de771694e6cc5107e4858d3344633bfaca7b70f23025afc6c5a1eb8bb23580
-  data.tar.gz: 6e2f758c616871af780b376613ef9eac7c0bfe6438f65e5950b035e4e450da31d207674b558b9e3726ca3287f0e468adda333988fa3c2a53b7fe0ac50757b660
+  metadata.gz: e834c192faaa6922e02c536c6fb8620fe20d32bcb58be48dea83c9f6283b75c6cbe738e2422ee67830f928d10979fbdd03cc87a8946cc565501c839c7d659dba
+  data.tar.gz: da3d808d4863af50cc8cdfc689ead159d013fc4955428d6c1e6cc6069417817112a870da7a1a8e420de283da4579c219569178848f4f3c05d64aa20abbc03bdc

data/.rubocop_todo.yml

@@ -1,40 +1,56 @@
 # This configuration was generated by
 # `rubocop --auto-gen-config`
-# on 2016-09-24 09:10:21 -0400 using RuboCop version 0.38.0.
+# on 2017-06-16 13:07:42 -0500 using RuboCop version 0.48.1.
 # The point is for the user to remove these configuration records
 # one by one as the offenses are removed from the code base.
 # Note that changes in the inspected code, or installation of new
 # versions of RuboCop, may require this file to be generated again.
 
+# Offense count: 4
+# Cop supports --auto-correct.
+# Configuration parameters: Include, TreatCommentsAsGroupSeparators.
+# Include: **/Gemfile, **/gems.rb
+Bundler/OrderedGems:
+  Exclude:
+    - 'examples/inventory/Gemfile'
+    - 'examples/market/Gemfile'
+    - 'examples/minimal/Gemfile'
+    - 'examples/weather/Gemfile'
+
 # Offense count: 1
 Lint/HandleExceptions:
   Exclude:
     - 'lib/initializers/giphy.rb'
 
-# Offense count: 5
+# Offense count: 6
 Metrics/AbcSize:
   Max: 36
 
+# Offense count: 21
+# Configuration parameters: CountComments, ExcludedMethods.
+Metrics/BlockLength:
+  Max: 120
+
 # Offense count: 2
 Metrics/CyclomaticComplexity:
-  Max: 11
+  Max: 13
 
-# Offense count: 156
-# Configuration parameters: AllowHeredoc, AllowURI, URISchemes.
+# Offense count: 199
+# Configuration parameters: AllowHeredoc, AllowURI, URISchemes, IgnoreCopDirectives, IgnoredPatterns.
 # URISchemes: http, https
 Metrics/LineLength:
   Max: 147
 
-# Offense count: 6
+# Offense count: 10
 # Configuration parameters: CountComments.
 Metrics/MethodLength:
   Max: 27
 
 # Offense count: 2
 Metrics/PerceivedComplexity:
-  Max: 11
+  Max: 13
 
-# Offense count: 26
+# Offense count: 30
 Style/Documentation:
   Enabled: false
 
@@ -43,13 +59,58 @@ Style/DoubleNegation:
   Exclude:
     - 'lib/slack-ruby-bot/commands/base.rb'
 
-# Offense count: 1
-# Configuration parameters: ExpectMatchingDefinition, Regex, IgnoreExecutableScripts.
+# Offense count: 2
+# Configuration parameters: ExpectMatchingDefinition, Regex, IgnoreExecutableScripts, AllowedAcronyms.
+# AllowedAcronyms: CLI, DSL, ACL, API, ASCII, CPU, CSS, DNS, EOF, GUID, HTML, HTTP, HTTPS, ID, IP, JSON, LHS, QPS, RAM, RHS, RPC, SLA, SMTP, SQL, SSH, TCP, TLS, TTL, UDP, UI, UID, UUID, URI, URL, UTF8, VM, XML, XMPP, XSRF, XSS
 Style/FileName:
   Exclude:
+    - 'Dangerfile'
     - 'lib/slack-ruby-bot.rb'
 
+# Offense count: 2
+# Cop supports --auto-correct.
+# Configuration parameters: EnforcedStyle, SupportedStyles.
+# SupportedStyles: auto_detection, squiggly, active_support, powerpack, unindent
+Style/IndentHeredoc:
+  Exclude:
+    - 'lib/slack-ruby-bot/commands/help.rb'
+    - 'spec/slack-ruby-bot/commands/help_spec.rb'
+
 # Offense count: 1
+# Configuration parameters: EnforcedStyle, SupportedStyles.
+# SupportedStyles: module_function, extend_self
 Style/ModuleFunction:
   Exclude:
     - 'lib/slack-ruby-bot/config.rb'
+
+# Offense count: 3
+# Cop supports --auto-correct.
+Style/MultilineIfModifier:
+  Exclude:
+    - 'lib/slack-ruby-bot/app.rb'
+    - 'lib/slack-ruby-bot/client.rb'
+    - 'lib/slack-ruby-bot/commands/base.rb'
+
+# Offense count: 5
+# Cop supports --auto-correct.
+# Configuration parameters: PreferredDelimiters.
+Style/PercentLiteralDelimiters:
+  Exclude:
+    - 'lib/slack-ruby-bot/server.rb'
+    - 'spec/slack-ruby-bot/commands/aliases_spec.rb'
+    - 'spec/slack-ruby-bot/hooks/hook_support_spec.rb'
+    - 'spec/slack-ruby-bot/server_spec.rb'
+
+# Offense count: 3
+# Cop supports --auto-correct.
+# Configuration parameters: SupportedStyles.
+# SupportedStyles: percent, brackets
+Style/SymbolArray:
+  EnforcedStyle: brackets
+
+# Offense count: 2
+# Configuration parameters: EnforcedStyle, SupportedStyles.
+# SupportedStyles: snake_case, normalcase, non_integer
+Style/VariableNumber:
+  Exclude:
+    - 'spec/slack-ruby-bot/hooks/set_spec.rb'

data/CHANGELOG.md

@@ -1,3 +1,9 @@
+### 0.10.4 (07/05/2017)
+
+* [#149](https://github.com/slack-ruby/slack-ruby-bot/pull/149): Add `logger` configuration to set a custom logger - [@upscent](https://github.com/upscent).
+* [#147](https://github.com/slack-ruby/slack-ruby-bot/pull/147): Adds `server.on` as a shortcut for `hooks.add` and deprecate `hooks` method - [@laertispappas](https://github.com/laertispappas).
+* [#143](https://github.com/slack-ruby/slack-ruby-bot/pull/143): Provide `permitted?` method to allow for simple authorization extensions - [@chuckremes](https://github.com/chuckremes).
+
 ### 0.10.3 (06/15/2017)
 
 * [#145](https://github.com/slack-ruby/slack-ruby-bot/pull/145): Map multiple command strings to same controller method - [@chuckremes](https://github.com/chuckremes).

data/README.md

@@ -212,6 +212,25 @@ class Phone < SlackRubyBot::Commands::Base
 end
 ```
 
+### Authorization
+
+The framework does not provide any user authentication or command authorization capability out of the box. However, the `SlackRubyBot::Commands::Base` class does check every command invocation for permission prior to executing the command. The default method always returns true.
+
+Therefore, subclasses of `SlackRubyBot::Commands::Base` can override the `permitted?` private method to provide its own authorization logic. This method is intended to be exploited by user code or external gems that want to provide custom authorization logic for command execution.
+
+```ruby
+class AuthorizedBot < SlackRubyBot::Commands::Base
+  command 'phone home' do |client, data, match|
+    client.say(channel: data.channel, text: 'Elliot!')
+  end
+
+  # Only allow user 'Uxyzabc' to run this command
+  def self.permitted?(client, data, match)
+    data && data.user && data.user == 'Uxyzabc'
+  end
+end
+```
+
 ### Animated GIFs
 
 The `SlackRubyBot::Client` implementation comes with GIF support.
@@ -261,22 +280,37 @@ Hooks are event handlers and respond to Slack RTM API [events](https://api.slack
 
 A Hook Handler is any object that respond to a `call` message, like a proc, instance of an object, class with a `call` class method, etc.
 
-Hooks are registered onto the `SlackRubyBot::Server` instance, by way of a configuration hash
+Hooks can be registered using different methods based on user preference / use case. 
+Currently someone can use one of the following methods: 
+
+* Pass `hooks` in `SlackRubyBot::Server` initialization.
+* Register `hooks` on `SlackRubyBot::Server` using `on` class method.
+* Register `hooks` on `SlackRubyBot::Server` using `on` instance method.
+
+
+##### Hooks registration on `SlackRubyBot::Server` initialization
 
 ```ruby
 SlackRubyBot::Server.new(hook_handlers: {
-  hello: MyBot::Hooks::UserChange.new
+  hello: MyBot::Hooks::UserChange.new,
+  user_change: [->(client, data) {  }, ->(client, data) {}]
 })
 ```
 
-or at any time by pushing it to the `HookSet`
+##### Hooks registration on a `SlackRubyBot::Server` instance
 
 ```ruby
-# Push an object that implements the
-server.hooks.add(:hello, MyBot::Hooks::UserChange.new)
+# Register an object that implements `call` method
+class MyBot::Hooks::Hello
+  def call(client, data)
+    puts "Hello"
+  end
+end
 
-# Push a lambda to handle the event
-server.hooks.add(:hello, ->(client, data) { puts "Hello!" })
+server.on(:hello, MyBot::Hooks::Hello.new)
+
+# or register a lambda function to handle the event
+server.on(:hello, ->(client, data) { puts "Hello!" })
 ```
 
 For example, the following hook handles [user_change](https://api.slack.com/events/user_change), an event sent when a team member updates their profile or data. This can be useful to update the local user cache when a user is renamed.
@@ -288,14 +322,16 @@ module MyBot
       def call(client, data)
         # data['user']['id'] contains the user ID
         # data['user']['name'] contains the new user name
-        ...
+        # ...
       end
     end
   end
 end
 ```
 
-Hooks can also be written as blocks inside the `SlackRubyBot::Server` class, for example
+##### Hooks registration on `SlackRubyBot::Server` class
+
+Example:
 
 ```ruby
 module MyBot
@@ -304,6 +340,11 @@ module MyBot
       # data['user']['id'] contains the user ID
       # data['user']['name'] contains the new user name
     end
+    
+    on 'user_change', ->(client, data) {      
+      # data['user']['id'] contains the user ID
+      # data['user']['name'] contains the new user name
+    }
   end
 end
 ```
@@ -312,6 +353,19 @@ These will get pushed into the hook set on initialization.
 
 Either by configuration, explicit assignment or hook blocks, multiple handlers can exist for the same event type.
 
+
+#### Deprecated hook registration
+
+Registering a hook method using `hooks.add` is considered deprecated and
+will be removed on future versions.
+
+```ruby
+# [DEPRECATED]
+server.hooks.add(:hello, MyBot::Hooks::UserChange.new)
+server.hooks.add(:hello, ->(client, data) { puts "Hello!" })
+
+```
+
 ### Message Loop Protection
 
 By default bots do not respond to their own messages. If you wish to change that behavior, set `allow_message_loops` to `true`.
@@ -324,12 +378,22 @@ end
 
 ### Logging
 
-By default bots set a logger to `STDOUT` with `DEBUG` level. The logger is used in both the RealTime and Web clients. Silence logger as follows.
+By default bots set a logger to `$stdout` with `DEBUG` level. The logger is used in both the RealTime and Web clients.
+
+Silence logger as follows.
 
 ```ruby
 SlackRubyBot::Client.logger.level = Logger::WARN
 ```
 
+If you wish to customize logger, set `logger` to your logger.
+
+```ruby
+SlackRubyBot.configure do |config|
+  config.logger = Logger.new("slack-ruby-bot.log", "daily")
+end
+```
+
 ### Advanced Integration
 
 You may want to integrate a bot or multiple bots into other systems, in which case a globally configured bot may not work for you. You may create instances of [SlackRubyBot::Server](lib/slack-ruby-bot/server.rb) which accepts `token`, `aliases` and `send_gifs`.

data/UPGRADING.md

@@ -1,6 +1,25 @@
 Upgrading SlackRubyBot
 ======================
 
+### Upgrading to >= 0.10.4
+
+#### Replace `server.hooks.add` with `server.on`
+
+We have deprecated `SlackRubyBot::Server#hooks` in favor of `SlackRubyBot::Server#on` instance method. All users using `SlackRubyBot::Server#hooks` method should
+change their codebase and use the new method instead. Method signature is not affected.
+
+Example:
+
+```ruby
+  # Given server is an instance of SlackRubyBot::Server
+  #
+  # Before
+  server.hooks.add :hello, Greet.new
+
+  # After
+  server.on :hello, Greet.new
+```
+
 ### Upgrading to >= 0.9.0
 
 #### Add giphy to your Gemfile for GIF support

data/lib/slack-ruby-bot/commands/base.rb

@@ -68,9 +68,9 @@ module SlackRubyBot
             called = true
             call = options[:call]
             if call
-              call.call(client, data, match)
+              call.call(client, data, match) if permitted?(client, data, match)
             elsif respond_to?(:call)
-              send(:call, client, data, match)
+              send(:call, client, data, match) if permitted?(client, data, match)
             else
               raise NotImplementedError, data.text
             end
@@ -115,6 +115,12 @@ module SlackRubyBot
           return if routes && routes.any?
           command default_command_name
         end
+
+        # Intended to be overridden by subclasses to hook in an
+        # authorization mechanism.
+        def permitted?(_client, _data, _match)
+          true
+        end
       end
     end
   end

data/lib/slack-ruby-bot/config.rb

@@ -2,7 +2,7 @@ module SlackRubyBot
   module Config
     extend self
 
-    ATTRS = [:token, :url, :aliases, :user, :user_id, :team, :team_id, :allow_message_loops, :send_gifs].freeze
+    ATTRS = [:token, :url, :aliases, :user, :user_id, :team, :team_id, :allow_message_loops, :send_gifs, :logger].freeze
     attr_accessor(*ATTRS)
 
     def allow_message_loops?

data/lib/slack-ruby-bot/hooks/hook_support.rb

@@ -18,7 +18,12 @@ module SlackRubyBot
 
       # Instance stuff
       def hooks
-        @hooks ||= SlackRubyBot::Hooks::Set.new
+        warn Kernel.caller.first + ' [DEPRECATION] `hooks` method is deprecated. Please use `server.on` instead to register a hook.'
+        _hooks
+      end
+
+      def on(event_name, handler)
+        _hooks.add(event_name, handler)
       end
 
       def flush_hook_blocks
@@ -27,11 +32,18 @@ module SlackRubyBot
         add_hook_handlers(self.class.hook_blocks)
       end
 
+      # TODO: This should be deprecated in favor of `on`
       def add_hook_handlers(handler_hash)
         handler_hash.each do |hook, handlers|
-          Array(handlers).each { |handler| hooks.add(hook, handler) }
+          Array(handlers).each { |handler| on(hook, handler) }
         end
       end
+
+      # Temp use this method in order to deprecate `hooks` and revisit
+      def _hooks
+        @hooks ||= SlackRubyBot::Hooks::Set.new
+      end
+      private :_hooks
     end
   end
 end

data/lib/slack-ruby-bot/hooks/set.rb

@@ -11,12 +11,10 @@ module SlackRubyBot
       end
 
       def add(hook_name, handler)
-        if handlers[hook_name].present?
-          handlers[hook_name] << handler
-        else
-          handlers[hook_name] = [handler]
-          register_callback(hook_name)
-        end
+        handlers[hook_name] ||= ::Set.new
+        handlers[hook_name] << handler
+
+        register_callback(hook_name)
       end
 
       def client=(client)

data/lib/slack-ruby-bot/rspec/support/slack-ruby-bot/not_respond.rb

@@ -0,0 +1,25 @@
+require 'rspec/expectations'
+
+RSpec::Matchers.define :not_respond do
+  match do |actual|
+    client = if respond_to?(:client)
+               send(:client)
+             else
+               SlackRubyBot::Client.new
+             end
+
+    message_command = SlackRubyBot::Hooks::Message.new
+    channel, user, message = parse(actual)
+
+    expect(client).not_to receive(:message)
+    message_command.call(client, Hashie::Mash.new(text: message, channel: channel, user: user))
+    true
+  end
+
+  private
+
+  def parse(actual)
+    actual = { message: actual } unless actual.is_a?(Hash)
+    [actual[:channel] || 'channel', actual[:user] || 'user', actual[:message]]
+  end
+end

data/lib/slack-ruby-bot/server.rb

@@ -101,7 +101,7 @@ module SlackRubyBot
           @client = nil
           restart! unless @stopping
         end
-        hooks.client = client
+        _hooks.client = client
 
         client
       end

data/lib/slack-ruby-bot/support/loggable.rb

@@ -7,9 +7,9 @@ module SlackRubyBot
 
     module ClassMethods
       def logger
-        @logger ||= begin
+        @logger ||= SlackRubyBot::Config.logger || begin
           $stdout.sync = true
-          Logger.new(STDOUT)
+          Logger.new($stdout)
         end
       end
     end

data/lib/slack-ruby-bot/version.rb

@@ -1,3 +1,3 @@
 module SlackRubyBot
-  VERSION = '0.10.3'.freeze
+  VERSION = '0.10.4'.freeze
 end

data/spec/slack-ruby-bot/commands/commands_permitted_spec.rb

@@ -0,0 +1,34 @@
+describe SlackRubyBot::Commands, 'permitted?' do
+  let! :command do
+    Class.new(SlackRubyBot::Commands::Base) do
+      class << self
+        attr_accessor :_flag1
+        attr_accessor :_flag2
+        def permitted?(_client, _data, match)
+          match[:command] == 'saylo'
+        end
+      end
+      @_flag1 = false
+      @_flag2 = false
+      command 'saylo' do |_client, _data, _match|
+        self._flag1 = true
+      end
+      command 'sayallowed' do |_client, _data, _match|
+        self._flag2 = true
+      end
+    end
+  end
+  def app
+    SlackRubyBot::App.new
+  end
+
+  it 'allows command when returning true' do
+    expect(message: "#{SlackRubyBot::Config.user} saylo").to not_respond
+    expect(command._flag1).to be true
+  end
+
+  it 'denies command when returning false' do
+    expect(message: "#{SlackRubyBot::Config.user} sayallowed").to not_respond
+    expect(command._flag2).to be false
+  end
+end

data/spec/slack-ruby-bot/hooks/hook_support_spec.rb

@@ -35,11 +35,34 @@ describe SlackRubyBot::Hooks::HookSupport do
     it 'registers class hook blocks as hook handlers in set' do
       object = subject.new
 
-      expect(object.hooks).to receive(:add).exactly(3).times.and_call_original
+      expect(object.send(:_hooks)).to receive(:add).exactly(3).times.and_call_original
 
       expect do
         object.flush_hook_blocks
-      end.to change { object.hooks.handlers.size }.by(2)
+      end.to change { object.send(:_hooks).handlers.size }.by(2)
+    end
+  end
+
+  describe '#on' do
+    subject { super().new }
+    it 'delegates to `hooks.add` method' do
+      event_name = :message_received
+      handler = ->(_, _) {}
+
+      expect(subject.send(:_hooks)).to receive(:add).with(event_name, handler).and_call_original
+      subject.on(event_name, handler)
+    end
+  end
+
+  describe '#hooks' do
+    subject { super().new }
+    it { expect(subject.hooks).to eq subject.send(:_hooks) }
+  end
+
+  describe '#_hooks' do
+    it 'returns a SlackRubyBot::Hooks::Set instance' do
+      hooks_set = subject.new.send(:_hooks)
+      expect(hooks_set).to be_a SlackRubyBot::Hooks::Set
     end
   end
 end

data/spec/slack-ruby-bot/hooks/set_spec.rb

@@ -12,7 +12,7 @@ describe SlackRubyBot::Hooks::Set do
       end.to change(subject, :handlers)
 
       expect(subject.handlers).to have_key(:message)
-      expect(subject.handlers[:message]).to eq [handler]
+      expect(subject.handlers[:message]).to eq Set.new.add(handler)
     end
 
     it 'lets you add multiple handlers for the same hook' do
@@ -25,7 +25,7 @@ describe SlackRubyBot::Hooks::Set do
       end.to change(subject, :handlers)
 
       expect(subject.handlers).to have_key(:message)
-      expect(subject.handlers[:message]).to eq [handler_1, handler_2]
+      expect(subject.handlers[:message]).to eq Set.new([handler_1, handler_2])
     end
   end
 

data/spec/slack-ruby-bot/support/loggable_spec.rb

@@ -1,23 +1,23 @@
 describe SlackRubyBot::Loggable do
-  let! :class_with_logger do
-    Class.new(SlackRubyBot::Commands::Base) do
-      def public_logger
-        logger
+  let!(:class_with_logger)       { Class.new SlackRubyBot::Commands::Base }
+  let!(:child_class_with_logger) { Class.new class_with_logger }
+  describe 'logger for class' do
+    context 'set logger by config' do
+      let(:logger) { double 'logger' }
+      it do
+        SlackRubyBot.configure do |config|
+          config.logger = logger
+        end
+        expect(class_with_logger.logger).to eq logger
+        expect(child_class_with_logger.logger).to eq logger
       end
     end
-  end
-  let! :child_class_with_logger do
-    Class.new(class_with_logger) do
-      def public_logger
-        logger
+    context 'default logger' do
+      it do
+        expect(class_with_logger.logger).to be_kind_of Logger
+        expect(child_class_with_logger.logger).to be_kind_of Logger
       end
     end
-  end
-  describe 'logger for class' do
-    it do
-      expect(class_with_logger.logger).to be_kind_of Logger
-      expect(child_class_with_logger.logger).to be_kind_of Logger
-    end
     it 'should be cached' do
       first_logger = class_with_logger.logger
       second_logger = class_with_logger.logger
@@ -30,18 +30,9 @@ describe SlackRubyBot::Loggable do
     end
   end
   describe 'logger for instance' do
-    it do
-      expect(class_with_logger.new.public_logger).to be_kind_of Logger
-    end
-    it 'should be cached' do
-      first_logger = class_with_logger.new.public_logger
-      second_logger = class_with_logger.new.public_logger
-      expect(first_logger.object_id).to eq second_logger.object_id
-    end
-    it 'should not be shared by a child class' do
-      first_logger = class_with_logger.new.public_logger
-      second_logger = child_class_with_logger.new.public_logger
-      expect(first_logger.object_id).to_not eq second_logger.object_id
+    it 'same with one for class' do
+      expect(class_with_logger.new.logger.object_id).to eq class_with_logger.logger.object_id
+      expect(child_class_with_logger.new.logger.object_id).to eq child_class_with_logger.logger.object_id
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: slack-ruby-bot
 version: !ruby/object:Gem::Version
-  version: 0.10.3
+  version: 0.10.4
 platform: ruby
 authors:
 - Daniel Doubrovkine
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-06-15 00:00:00.000000000 Z
+date: 2017-07-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: hashie
@@ -187,6 +187,7 @@ files:
 - lib/slack-ruby-bot/rspec/support/bots_for_tests.rb
 - lib/slack-ruby-bot/rspec/support/fixtures/slack/migration_in_progress.yml
 - lib/slack-ruby-bot/rspec/support/slack-ruby-bot/it_behaves_like_a_slack_bot.rb
+- lib/slack-ruby-bot/rspec/support/slack-ruby-bot/not_respond.rb
 - lib/slack-ruby-bot/rspec/support/slack-ruby-bot/respond_with_error.rb
 - lib/slack-ruby-bot/rspec/support/slack-ruby-bot/respond_with_slack_message.rb
 - lib/slack-ruby-bot/rspec/support/slack_api_key.rb
@@ -213,6 +214,7 @@ files:
 - spec/slack-ruby-bot/commands/bot_message_spec.rb
 - spec/slack-ruby-bot/commands/bot_spec.rb
 - spec/slack-ruby-bot/commands/commands_command_classes_spec.rb
+- spec/slack-ruby-bot/commands/commands_permitted_spec.rb
 - spec/slack-ruby-bot/commands/commands_precedence_spec.rb
 - spec/slack-ruby-bot/commands/commands_regexp_escape_spec.rb
 - spec/slack-ruby-bot/commands/commands_spaces_spec.rb
@@ -278,6 +280,7 @@ test_files:
 - spec/slack-ruby-bot/commands/bot_message_spec.rb
 - spec/slack-ruby-bot/commands/bot_spec.rb
 - spec/slack-ruby-bot/commands/commands_command_classes_spec.rb
+- spec/slack-ruby-bot/commands/commands_permitted_spec.rb
 - spec/slack-ruby-bot/commands/commands_precedence_spec.rb
 - spec/slack-ruby-bot/commands/commands_regexp_escape_spec.rb
 - spec/slack-ruby-bot/commands/commands_spaces_spec.rb