grape-slack-bot 1.0.5 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 01e2933c0e98e856b11a9a89da5feb9d01bd6ed059a31e2ae0132560eb4a15e6
-  data.tar.gz: 18b35789632851892cf8afe19d3b8b50b5c4b6b616551b305a4f603e307f7423
+  metadata.gz: f84c93d79a89045d58deda55e64382c8ef74daf4975e0f47732b480c08619020
+  data.tar.gz: 6f3e952d9a8f85e48de8f045aab1c35f0d69e9ac39b8328c240f38c77c578ad0
 SHA512:
-  metadata.gz: 831eb6dd2c71b08bc91f120f9243ccc4cab043baa7f0666992ed744ec31abddc9975e69200c3aa136997fd4c571c0be460a16bde90f017ce0ec73bd0d15ef9c9
-  data.tar.gz: 23463749cc8381847530940e62ccc9afc53e59ec31d15985b89dbfa611251a5bc7b85958ceae4acc50b577d95336cf2c33d876fc365204b34472f67ec7acb899
+  metadata.gz: 458e378569349642ecf2433fd573d3ad40ef69def84a6253206ff5bcef2b9d983c24b0d1e53a30887de7a8203dcf733ba868bbbbc9b4e215ded1f71487c3dd12
+  data.tar.gz: dd5ef3106d1a1255cb788bef313d905ca986133bc5b3b3e53166c70b9727f8d3b996cce595f724e80b9d08ec4338ca91fbedcf135945878c01a61d32872375ef

data/CHANGELOG.md

@@ -1,3 +1,11 @@
+# 1.2.0
+
+* Remove `Rails.logger` dependency, make logger configurable
+
+# 1.1.0
+
+* Set minimum ruby version requirement to 2.5.0
+
 # 1.0.5
 
 * Add superclass `SlackBot::Error` for all errors

data/README.md

@@ -133,6 +133,7 @@ References:
 Create `app/api/slack_bot_api.rb`, it will contain bot configuration and endpoints setup:
 
 ```ruby
+SlackBot::DevConsole.logger = Rails.logger
 SlackBot::DevConsole.enabled = Rails.env.development?
 SlackBot::Config.configure do
   callback_storage Rails.cache

data/grape-slack-bot.gemspec

@@ -29,7 +29,7 @@ Gem::Specification.new do |gem|
   gem.files = Dir.glob("lib/**/*.rb") + Dir.glob("bin/**/*") + root_files
   gem.test_files = Dir.glob("spec/**/*_spec.rb")
 
-  gem.required_ruby_version = ">= 1.9.3"
+  gem.required_ruby_version = ">= 2.5.0"
   gem.require_paths = ["lib"]
 
   gem.add_runtime_dependency 'rack', '> 2'
@@ -38,6 +38,9 @@ Gem::Specification.new do |gem|
   gem.add_runtime_dependency 'activesupport', '> 5'
 
   gem.add_development_dependency 'bundler', '~> 2'
-  gem.add_development_dependency 'pry-byebug', '~> 3'
+  gem.add_development_dependency 'pry', '~> 0.14'
   gem.add_development_dependency 'rspec', '~> 3'
+  gem.add_development_dependency 'webmock', '~> 3'
+  gem.add_development_dependency 'simplecov', '~> 0.21'
+  gem.add_development_dependency 'simplecov-cobertura', '~> 2'
 end

data/lib/slack_bot/dev_console.rb

@@ -1,4 +1,9 @@
 module SlackBot
+  class Logger
+    def info(*args, **kwargs)
+      puts args, kwargs
+    end
+  end
   class DevConsole
     def self.enabled=(value)
       @enabled = value
@@ -8,24 +13,32 @@ module SlackBot
       @enabled
     end
 
-    def self.log(message = nil, &)
+    def self.logger=(value)
+      @logger = value
+    end
+
+    def self.logger
+      @logger ||= Logger.new
+    end
+
+    def self.log(message = nil, &block)
       return unless enabled?
 
       message = yield if block_given?
-      Rails.logger.info(message)
+      logger.info(message)
     end
 
-    def self.log_input(message = nil, &)
+    def self.log_input(message = nil, &block)
       message = yield if block_given?
       log(">>> #{message}")
     end
 
-    def self.log_output(message = nil, &)
+    def self.log_output(message = nil, &block)
       message = yield if block_given?
       log("<<< #{message}")
     end
 
-    def self.log_check(message = nil, &)
+    def self.log_check(message = nil, &block)
       message = yield if block_given?
       log("!!! #{message}")
     end

data/lib/slack_bot.rb

@@ -21,5 +21,5 @@ require 'slack_bot/pager'
 require 'slack_bot/grape_extension'
 
 module SlackBot
-  VERSION = '1.0.5'.freeze
+  VERSION = '1.2.0'.freeze
 end

data/spec/slack_bot/dev_console_spec.rb

@@ -0,0 +1,88 @@
+module SlackBot
+  class Logger
+    def info(*args, **kwargs)
+      puts args, kwargs
+    end
+  end
+  class DevConsole
+    def self.enabled=(value)
+      @enabled = value
+    end
+
+    def self.enabled?
+      @enabled
+    end
+
+    def self.logger=(value)
+      @logger = value
+    end
+
+    def self.logger
+      @logger ||= Logger.new
+    end
+
+    def self.log(message = nil, &block)
+      return unless enabled?
+
+      message = yield if block_given?
+      logger.info(message)
+    end
+
+    def self.log_input(message = nil, &block)
+      message = yield if block_given?
+      log(">>> #{message}")
+    end
+
+    def self.log_output(message = nil, &block)
+      message = yield if block_given?
+      log("<<< #{message}")
+    end
+
+    def self.log_check(message = nil, &block)
+      message = yield if block_given?
+      log("!!! #{message}")
+    end
+  end
+end
+
+require 'spec_helper'
+
+describe SlackBot::DevConsole do
+  let(:logger) { instance_double(SlackBot::Logger) }
+  before do
+    described_class.enabled = true
+    described_class.logger = logger
+  end
+
+  describe '.enabled=' do
+    it 'sets the enabled value' do
+      expect { described_class.enabled = false }.to change(described_class, :enabled?).from(true).to(false)
+    end
+  end
+
+  describe '.logger=' do
+    let(:new_logger) { instance_double(SlackBot::Logger) }
+    after { described_class.logger = logger }
+    it 'sets the logger' do
+      expect { described_class.logger = new_logger }.to change(described_class, :logger).from(logger).to(new_logger)
+    end
+  end
+
+  describe '.log' do
+    context 'when enabled' do
+      before { described_class.enabled = true }
+      it 'logs the message' do
+        expect(described_class.logger).to receive(:info).with('test')
+        described_class.log('test')
+      end
+    end
+
+    context 'when disabled' do
+      before { described_class.enabled = false }
+      it 'does not log the message' do
+        expect(described_class.logger).not_to receive(:info)
+        described_class.log('test')
+      end
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: grape-slack-bot
 version: !ruby/object:Gem::Version
-  version: 1.0.5
+  version: 1.2.0
 platform: ruby
 authors:
 - Andrei Makarov
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-08-29 00:00:00.000000000 Z
+date: 2023-08-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack
@@ -81,7 +81,21 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '2'
 - !ruby/object:Gem::Dependency
-  name: pry-byebug
+  name: pry
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.14'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.14'
+- !ruby/object:Gem::Dependency
+  name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
@@ -95,7 +109,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '3'
 - !ruby/object:Gem::Dependency
-  name: rspec
+  name: webmock
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
@@ -108,6 +122,34 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '3'
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.21'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.21'
+- !ruby/object:Gem::Dependency
+  name: simplecov-cobertura
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2'
 description: Slack bot implementation for ruby-grape
 email:
 - andrei@kiskolabs.com
@@ -140,6 +182,7 @@ files:
 - spec/slack_bot/callback_spec.rb
 - spec/slack_bot/command_spec.rb
 - spec/slack_bot/config_spec.rb
+- spec/slack_bot/dev_console_spec.rb
 - spec/slack_bot/event_spec.rb
 - spec/slack_bot/grape_extension_spec.rb
 - spec/slack_bot/interaction_spec.rb
@@ -164,7 +207,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 1.9.3
+      version: 2.5.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
@@ -181,6 +224,7 @@ test_files:
 - spec/slack_bot/callback_spec.rb
 - spec/slack_bot/command_spec.rb
 - spec/slack_bot/config_spec.rb
+- spec/slack_bot/dev_console_spec.rb
 - spec/slack_bot/event_spec.rb
 - spec/slack_bot/grape_extension_spec.rb
 - spec/slack_bot/interaction_spec.rb