slack_messaging 3.0.0 → 3.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e6742192033a64bb2935e5af6486ea356d54684f501515f22eb8a24c200fff61
-  data.tar.gz: 0d880c5cd858c778f06c9047e911927a1cb2ab2ee6b53a89b20b9e23a61ad2a9
+  metadata.gz: 8dca88b2aa6f5f3f6c6d00b959fc61757cb09585125c57f9e8b97196fecb90aa
+  data.tar.gz: a2209e3efd4abdb1a68cbfe284d1d3c037ff2132bc4c3b022625e3d232ab826b
 SHA512:
-  metadata.gz: 8458be91785ae94772e960f5f0705a2a611b36c286f694d0eb0ac4abe86fdd9d6867aa89d45dc9c44d721dbdb7bd9f34365ca5121f1d85ac0fd027d33439dbf4
-  data.tar.gz: 7485f64ff53624785968f06a25e109a8042aa1e7e63412d7c7a40b78da8209f643e06f62130b3c8e7ba8d0677db2445950b2a9fca0ceb5199b03c71d10592bd3
+  metadata.gz: '099fcbf29e6141abfb0e0d7616c8257764994ca916110db53b1f94ed9253222e62bdf2b5c67b787910c16781d206cc2ce81abd7e35e3b7d98c89c8e803ac719f'
+  data.tar.gz: 7582a545af09cf57b4c03297e56675b13bba4f76e0f2e6de204e9e8a28173347af941c87e7902c721c8bf2e16e1de1f54cd705e832bbff16ff4f450e03e1da9a

data/Gemfile

@@ -2,5 +2,4 @@
 
 source 'https://rubygems.org'
 
-# Specify your gem's dependencies in slack_messaging.gemspec
 gemspec

data/Gemfile.lock

@@ -1,34 +1,26 @@
 PATH
   remote: .
   specs:
-    slack_messaging (3.0.0)
-      activesupport (~> 6.0)
+    slack_messaging (3.2.0)
       gli (~> 2.10)
       hashie (~> 4.1)
-      highline (~> 2.0)
+      highline_wrapper (~> 1.1)
       httparty (~> 0.18)
       json (~> 2.5)
-      rack (~> 2.2)
 
 GEM
   remote: https://rubygems.org/
   specs:
-    activesupport (6.1.2.1)
-      concurrent-ruby (~> 1.0, >= 1.0.2)
-      i18n (>= 1.6, < 2)
-      minitest (>= 5.1)
-      tzinfo (~> 2.0)
-      zeitwerk (~> 2.3)
     ast (2.4.2)
     coderay (1.1.3)
-    concurrent-ruby (1.1.8)
+    concurrent-ruby (1.1.9)
     diff-lcs (1.4.4)
-    faker (2.15.1)
+    faker (2.18.0)
       i18n (>= 1.6, < 2)
-    ffi (1.14.2)
-    formatador (0.2.5)
-    gli (2.19.2)
-    guard (2.16.2)
+    ffi (1.15.3)
+    formatador (0.3.0)
+    gli (2.20.1)
+    guard (2.17.0)
       formatador (>= 0.2.4)
       listen (>= 2.7, < 4.0)
       lumberjack (>= 1.0.12, < 2.0)
@@ -44,40 +36,39 @@ GEM
       rspec (>= 2.99.0, < 4.0)
     hashie (4.1.0)
     highline (2.0.3)
+    highline_wrapper (1.3.0)
+      highline (~> 2.0)
     httparty (0.18.1)
       mime-types (~> 3.0)
       multi_xml (>= 0.5.2)
-    i18n (1.8.8)
+    i18n (1.8.10)
       concurrent-ruby (~> 1.0)
     json (2.5.1)
-    listen (3.4.1)
+    listen (3.5.1)
       rb-fsevent (~> 0.10, >= 0.10.3)
       rb-inotify (~> 0.9, >= 0.9.10)
     lumberjack (1.2.8)
     method_source (1.0.0)
     mime-types (3.3.1)
       mime-types-data (~> 3.2015)
-    mime-types-data (3.2020.1104)
-    minitest (5.14.3)
+    mime-types-data (3.2021.0704)
     multi_xml (0.6.0)
     nenv (0.3.0)
     notiffany (0.1.3)
       nenv (~> 0.1)
       shellany (~> 0.0)
     parallel (1.20.1)
-    parser (3.0.0.0)
+    parser (3.0.1.1)
       ast (~> 2.4.1)
-    pry (0.14.0)
+    pry (0.14.1)
       coderay (~> 1.1)
       method_source (~> 1.0)
-    rack (2.2.3)
     rainbow (3.0.0)
-    rake (13.0.3)
-    rb-fsevent (0.10.4)
+    rb-fsevent (0.11.0)
     rb-inotify (0.10.1)
       ffi (~> 1.0)
     regexp_parser (2.1.1)
-    rexml (3.2.4)
+    rexml (3.2.5)
     rspec (3.10.0)
       rspec-core (~> 3.10.0)
       rspec-expectations (~> 3.10.0)
@@ -91,24 +82,21 @@ GEM
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.10.0)
     rspec-support (3.10.2)
-    rubocop (1.10.0)
+    rubocop (1.18.2)
       parallel (~> 1.10)
       parser (>= 3.0.0.0)
       rainbow (>= 2.2.2, < 4.0)
       regexp_parser (>= 1.8, < 3.0)
       rexml
-      rubocop-ast (>= 1.2.0, < 2.0)
+      rubocop-ast (>= 1.7.0, < 2.0)
       ruby-progressbar (~> 1.7)
       unicode-display_width (>= 1.4.0, < 3.0)
-    rubocop-ast (1.4.1)
-      parser (>= 2.7.1.5)
+    rubocop-ast (1.7.0)
+      parser (>= 3.0.1.1)
     ruby-progressbar (1.11.0)
     shellany (0.0.1)
     thor (1.1.0)
-    tzinfo (2.0.4)
-      concurrent-ruby (~> 1.0)
     unicode-display_width (2.0.0)
-    zeitwerk (2.4.2)
 
 PLATFORMS
   x86_64-darwin-19
@@ -118,10 +106,9 @@ DEPENDENCIES
   faker (~> 2.15)
   guard-rspec (~> 4.3)
   pry (~> 0.13)
-  rake (~> 13.0)
   rspec (~> 3.9)
-  rubocop
+  rubocop (~> 1.10)
   slack_messaging!
 
 BUNDLED WITH
-   2.2.9
+   2.2.21

data/Guardfile

@@ -2,6 +2,6 @@
 
 guard :rspec, cmd: 'bundle exec rspec', all_on_start: true do
   watch(%r{^spec/.+_spec\.rb$})
-  watch(%r{^lib/(.+)\.rb$})     { |m| "spec/lib/#{m[1]}_spec.rb" }
+  watch(%r{^lib/(.+)\.rb$})     { |m| "spec/#{m[1]}_spec.rb" }
   watch('spec/spec_helper.rb')  { 'spec' }
 end

data/LICENSE.md

@@ -0,0 +1,29 @@
+BSD 3-Clause "New" or "Revised" License
+
+Copyright (c) 2014 Emma Sax.
+All rights reserved.
+
+Redistribution and use in source and binary forms, with or without
+modification, are permitted provided that the following conditions are met:
+
+1. Redistributions of source code must retain the above copyright notice, this
+   list of conditions, and the following disclaimer.
+
+2. Redistributions in binary form must reproduce the above copyright notice,
+   this list of conditions, and the following disclaimer in the documentation
+   and/or other materials provided with the distribution.
+
+3. Neither the name of the copyright holder nor the names of its
+   contributors may be used to endorse or promote products derived from
+   this software without specific prior written permission.
+
+THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
+AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
+IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
+DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE
+FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
+DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR
+SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER
+CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY,
+OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.

data/README.md

@@ -22,7 +22,7 @@ Or install it yourself as:
 gem install slack_messaging
 ```
 
-### Usage
+## Usage
 
 This project requires a config file that should look like this:
 
@@ -84,19 +84,31 @@ The output of slack_messaging will look something like this:
 
 I hope you enjoy printing fun and specialized messages to Slack!
 
-### Tests
+## Tests
 
-To run the tests, run `bundle exec rspec` from the command line. GitHub Actions will also run the tests upon every commit to make sure they're up to date and that everything is working correctly. Locally, you can also run `bundle exec guard` to automatically run tests as you develop! There are currently only tests for the `notify_slack` and `random_message` classes. Feel free to help us add more!
+To run the tests, run `bundle exec rspec` from the command-line. GitHub Actions will also run the tests upon every commit to make sure they're up to date and that everything is working correctly. Locally, you can also run `bundle exec guard` to automatically run tests as you develop!
 
-## Contributing
+---
 
-To submit a feature request, bug ticket, etc, please submit an official [GitHub Issue](https://github.com/emmahsax/slack_messaging/issues/new).
+### Contributing
 
-To report any security vulnerabilities, please view this project's [Security Policy](https://github.com/emmahsax/slack_messaging/security/policy).
+To submit a feature request, bug ticket, etc, please submit an official [GitHub issue](https://github.com/emmahsax/slack_messaging/issues/new). To copy or make changes, please [fork this repository](https://github.com/emmahsax/slack_messaging/fork). When/if you'd like to contribute back to this upstream, please create a pull request on this repository.
+
+Please follow included Issue Template(s) and Pull Request Template(s) when creating issues or pull requests.
+
+### Security Policy
+
+To report any security vulnerabilities, please view this repository's [Security Policy](https://github.com/emmahsax/slack_messaging/security/policy).
+
+### Licensing
+
+For information on licensing, please see [LICENSE.md](https://github.com/emmahsax/slack_messaging/blob/main/LICENSE.md).
+
+### Code of Conduct
 
 When interacting with this repository, please follow [Contributor Covenant's Code of Conduct](https://contributor-covenant.org).
 
-## Releasing
+### Releasing
 
 To make a new release of this gem:
 

data/bin/slack-messaging

@@ -21,19 +21,20 @@ DOCUMENTATION
     see Slack Messaging's GitHub repo: https://github.com/emmahsax/slack_messaging
 "
 
+pre do |global, _command, _options, _args|
+  SlackMessaging::Config.load(global[:config])
+  true
+end
+
 desc 'Prints a variety of messages to Slack'
 command 'send' do |c|
-  pre do |global, _command, _options, _args|
-    SlackMessaging::Config.load(global[:config])
-    true
-  end
-
   c.action do |_global_options, options, args|
     SlackMessaging::Send.execute(args, options)
   end
 end
 
 desc 'Sets up a Slack Messaging config file at ~/.slack_messaging.yml'
+skips_pre
 command 'setup' do |c|
   c.action do
     SlackMessaging::Setup.execute

data/lib/slack_messaging.rb

@@ -3,7 +3,7 @@
 require 'yaml'
 require 'hashie'
 require 'httparty'
-require 'highline'
+require 'highline_wrapper'
 require 'json'
 
 files = "#{File.expand_path(File.join(File.dirname(File.absolute_path(__FILE__)), 'slack_messaging'))}/**/*.rb"

data/lib/slack_messaging/config.rb

@@ -2,27 +2,29 @@
 
 module SlackMessaging
   class Config
-    def self.config
-      config_data.to_hash
-    end
+    class << self
+      def method_missing(method, args = false)
+        config_data.send(method, args)
+      end
 
-    def self.load(path)
-      load_config(path)
-      config
-    end
+      def load(path)
+        load_config(path)
+        config
+      end
 
-    def self.config_data
-      @config_data ||= Hashie::Mash.new
-    end
+      private def config
+        config_data.to_hash
+      end
 
-    def self.method_missing(method, args = false)
-      config_data.send(method, args)
-    end
+      private def config_data
+        @config_data ||= Hashie::Mash.new
+      end
 
-    def self.load_config(file)
-      raise MissingConfig, "Missing configuration file: #{file}" unless File.exist?(file)
+      private def load_config(file)
+        raise StandardError, "Missing configuration file: #{file}" unless File.exist?(file)
 
-      YAML.load_file(file).each { |key, value| config_data.assign_property(key, value) }
+        YAML.load_file(file).each { |key, value| config_data.assign_property(key, value) }
+      end
     end
   end
 end

data/lib/slack_messaging/default_paths.rb

@@ -2,12 +2,14 @@
 
 module SlackMessaging
   class DefaultPaths
-    def self.config
-      File.join(home, '.slack_messaging.yml')
-    end
+    class << self
+      def config
+        File.join(home, '.slack_messaging.yml')
+      end
 
-    def self.home
-      ENV['HOME'] || '.'
+      private def home
+        ENV['HOME'] || '.'
+      end
     end
   end
 end

data/lib/slack_messaging/send.rb

@@ -2,21 +2,23 @@
 
 module SlackMessaging
   class Send
-    attr_accessor :options
+    class << self
+      attr_accessor :options
 
-    def self.execute(args, _options = nil)
-      if args.empty?
-        message = SlackMessaging::RandomMessage.acquire_random_quote
-        print_message(message)
-      else
-        args.each do |arg_message|
-          print_message(arg_message)
+      def execute(args, _options = nil)
+        if args.empty?
+          message = SlackMessaging::RandomMessage.acquire_random_quote
+          print_message(message)
+        else
+          args.each do |arg_message|
+            print_message(arg_message)
+          end
         end
       end
-    end
 
-    def self.print_message(message)
-      SlackMessaging::NotifySlack.new(message).perform
+      private def print_message(message)
+        SlackMessaging::NotifySlack.new(message).perform
+      end
     end
   end
 end

data/lib/slack_messaging/setup.rb

@@ -2,83 +2,86 @@
 
 module SlackMessaging
   class Setup
-    def self.execute
-      if config_file_exists?
-        answer = highline.ask_yes_no(
-          "It looks like the #{default_config} file already exists. Do you wish to replace it? (y/n)"
-        )
-        puts
-      else
-        answer = true
-      end
-
-      create_or_update_config_file(ask_config_questions) if answer
-    end
+    class << self
+      # rubocop:disable Style/ConditionalAssignment
+      def execute
+        if config_file_exists?
+          answer = highline.ask_yes_no(
+            "It looks like the #{default_config} file already exists. Do you wish to replace it? (y/n)",
+            { required: true }
+          )
+        else
+          answer = true
+        end
 
-    def self.create_or_update_config_file(answers)
-      contents = generate_config_file(answers)
-      puts "\nCreating or updating your #{default_config} file..."
-      File.open(default_config, 'w') { |file| file.puts contents }
-      puts "\nDone!"
-    end
+        create_or_update_config_file(ask_config_questions) if answer
+      end
+      # rubocop:enable Style/ConditionalAssignment
 
-    def self.generate_config_file(answers)
-      file_contents = ''.dup
-      file_contents << "slack:\n"
-      file_contents << "  channel: #{answers[:channel].tr('#', '').strip}\n"
-      file_contents << "  username: #{answers[:username].strip}\n"
-      file_contents << "  webhook_url: #{answers[:webhook_url].strip}\n"
-      file_contents << "  icon_emoji: '#{answers[:icon_emoji].strip}'"
-      file_contents
-    end
+      private def create_or_update_config_file(answers)
+        contents = generate_config_file(answers)
+        puts "Creating or updating your #{default_config} file..."
+        File.open(default_config, 'w') { |file| file.puts contents }
+        puts 'Done!'
+      end
 
-    def self.config_file_exists?
-      File.exist?(default_config)
-    end
+      private def generate_config_file(answers)
+        file_contents = ''.dup
+        file_contents << "slack:\n"
+        file_contents << "  channel: #{answers[:channel].tr('#', '').strip}\n"
+        file_contents << "  username: #{answers[:username].strip}\n"
+        file_contents << "  webhook_url: #{answers[:webhook_url].strip}\n"
+        file_contents << "  icon_emoji: '#{answers[:icon_emoji].strip}'"
+        file_contents
+      end
 
-    # rubocop:disable Metrics/MethodLength:
-    def self.ask_config_questions
-      answers = {}
+      private def config_file_exists?
+        File.exist?(default_config)
+      end
 
-      answers[:webhook_url] = ask_question(
-        "What is your Slack webhook URL? If you don't have one yet, please navigate" \
-        ' to https://api.slack.com/messaging/webhooks to create one, and then come back' \
-        ' here and paste it in the Terminal.'
-      )
+      # rubocop:disable Metrics/MethodLength
+      private def ask_config_questions
+        answers = {}
 
-      unless answers[:webhook_url]
-        puts "\nExiting because Slack webhoook is required..."
-        exit
-      end
+        answers[:webhook_url] = ask_question(
+          "What is your Slack webhook URL? If you don't have one yet, please navigate" \
+          ' to https://api.slack.com/messaging/webhooks to create one, and then come back' \
+          ' here and paste it in the Terminal.',
+          nil,
+          required: true
+        )
 
-      answers[:channel] = ask_question(
-        "\nWhat slack channel do you wish to post to? (default is \"#general\")"
-      ) || 'general'
+        answers[:channel] = ask_question(
+          'What slack channel do you wish to post to? (default is "#general")',
+          '#general'
+        )
 
-      answers[:username] = ask_question(
-        "\nWhat slack username do you wish to post as? (default is \"Let's Get Quoty\")"
-      ) || "Let's Get Quoty"
+        answers[:username] = ask_question(
+          "What slack username do you wish to post as? (default is \"Let's Get Quoty\")",
+          "Let's Get Quoty"
+        )
 
-      answers[:icon_emoji] = ask_question(
-        "\nWhat emoji would you like to post with (include the colons at the beginning and end" \
-        ' of the emoji name)? (default is ":mailbox_with_mail:")'
-      ) || ':mailbox_with_mail:'
+        answers[:icon_emoji] = ask_question(
+          'What emoji would you like to post with (include the colons at the beginning and end' \
+          ' of the emoji name)? (default is ":mailbox_with_mail:")',
+          ':mailbox_with_mail:'
+        )
 
-      answers
-    end
-    # rubocop:enable Metrics/MethodLength
+        answers
+      end
+      # rubocop:enable Metrics/MethodLength
 
-    def self.ask_question(prompt)
-      answer = highline.ask(prompt)
-      answer.empty? ? nil : answer
-    end
+      private def ask_question(prompt, default, required: false)
+        highline.ask(prompt, { default: default, required: required })
+      end
 
-    def self.highline
-      @highline ||= SlackMessaging::HighlineCli.new
-    end
+      private def highline
+        @highline ||= HighlineWrapper.new
+      end
 
-    def self.default_config
-      SlackMessaging::DefaultPaths.config
+      private def default_config
+        SlackMessaging::DefaultPaths.config
+      end
     end
   end
 end

data/lib/slack_messaging/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module SlackMessaging
-  VERSION = '3.0.0'
+  VERSION = '3.2.0'
 end

data/spec/slack_messaging/notify_slack_spec.rb

@@ -28,20 +28,18 @@ describe SlackMessaging::NotifySlack do
     SlackMessaging::Config.load(Faker::Lorem.word)
   end
 
-  subject { SlackMessaging::NotifySlack }
+  subject { SlackMessaging::NotifySlack.new(sentence) }
 
   it 'should call HTTParty' do
     expect(HTTParty).to receive(:post)
-    message = subject.new(sentence)
-    message.perform
+    subject.perform
   end
 
   it 'should define certain values' do
-    message = subject.new(sentence)
-    expect(message.text).to eq(sentence)
-    expect(message.channel).to eq(channel)
-    expect(message.username).to eq(username)
-    expect(message.webhook_url).to eq(webhook)
-    expect(message.icon_emoji).to eq(emoji)
+    expect(subject.text).to eq(sentence)
+    expect(subject.channel).to eq(channel)
+    expect(subject.username).to eq(username)
+    expect(subject.webhook_url).to eq(webhook)
+    expect(subject.icon_emoji).to eq(emoji)
   end
 end

data/spec/slack_messaging/setup_spec.rb

@@ -4,8 +4,7 @@ require 'spec_helper'
 require 'slack_messaging'
 
 describe SlackMessaging::Setup do
-  let(:response) { double(:response, readline: true, to_s: Faker::Lorem.sentence) }
-  let(:highline_cli) { double(:highline_cli, ask: response, ask_yes_no: true) }
+  let(:highline_wrapper) { double(:highline_wrapper, ask: Faker::Lorem.word, ask_yes_no: true) }
   let(:answers) do
     {
       channel: Faker::Lorem.word,
@@ -16,7 +15,7 @@ describe SlackMessaging::Setup do
   end
 
   before do
-    allow(SlackMessaging::HighlineCli).to receive(:new).and_return(highline_cli)
+    allow(HighlineWrapper).to receive(:new).and_return(highline_wrapper)
     allow(subject).to receive(:puts)
   end
 
@@ -30,7 +29,7 @@ describe SlackMessaging::Setup do
     it 'should ask a question if the config file exists' do
       allow(subject).to receive(:config_file_exists?).and_return(true)
       allow(File).to receive(:exists?).and_return(true)
-      expect(highline_cli).to receive(:ask_yes_no).and_return(true)
+      expect(highline_wrapper).to receive(:ask_yes_no).and_return(true)
       allow(subject).to receive(:create_or_update_config_file).and_return(true)
       allow(subject).to receive(:ask_config_questions).and_return(true)
       subject.execute
@@ -38,7 +37,7 @@ describe SlackMessaging::Setup do
 
     it 'should call to create or update the config file' do
       allow(File).to receive(:exists?).and_return(true)
-      allow(highline_cli).to receive(:ask_yes_no).and_return(true)
+      allow(highline_wrapper).to receive(:ask_yes_no).and_return(true)
       expect(subject).to receive(:create_or_update_config_file).and_return(true)
       expect(subject).to receive(:ask_config_questions).and_return(true)
       subject.execute
@@ -47,7 +46,7 @@ describe SlackMessaging::Setup do
     it 'should skip if the user opts not to continue' do
       allow(subject).to receive(:config_file_exists?).and_return(true)
       allow(File).to receive(:exists?).and_return(true)
-      allow(highline_cli).to receive(:ask_yes_no).and_return(false)
+      allow(highline_wrapper).to receive(:ask_yes_no).and_return(false)
       expect(subject).not_to receive(:create_or_update_config_file)
       expect(subject).not_to receive(:ask_config_questions)
       subject.execute
@@ -88,49 +87,54 @@ describe SlackMessaging::Setup do
 
   describe '#self.ask_question' do
     it 'should use highline to ask a question' do
-      expect(highline_cli).to receive(:ask).and_return('')
-      subject.send(:ask_question, Faker::Lorem.sentence)
+      expect(highline_wrapper).to receive(:ask).and_return('')
+      subject.send(:ask_question, Faker::Lorem.sentence, nil)
     end
 
     it 'should return nil if the highline client gets an empty string' do
-      allow(highline_cli).to receive(:ask).and_return('')
-      expect(subject.send(:ask_question, Faker::Lorem.sentence)).to be_nil
+      allow(highline_wrapper).to receive(:ask).and_return(nil)
+      expect(subject.send(:ask_question, Faker::Lorem.sentence, nil)).to be_nil
     end
 
     it 'should return the answer if it is given' do
       answer = Faker::Lorem.sentence
-      allow(highline_cli).to receive(:ask).and_return(answer)
-      expect(subject.send(:ask_question, Faker::Lorem.sentence)).to be(answer)
+      allow(highline_wrapper).to receive(:ask).and_return(answer)
+      expect(subject.send(:ask_question, Faker::Lorem.sentence, :default)).to be(answer)
     end
   end
 
   describe '#self.ask_config_questions' do
     it 'should call to ask at least four questions' do
-      expect(subject).to receive(:ask_question).at_least(4).times
+      expect(subject).to receive(:ask_question).at_least(4).times.and_return(Faker::Lorem.word)
       subject.send(:ask_config_questions)
     end
 
-    it 'should exit if the slack URL is not given' do
-      allow(subject).to receive(:ask_question).and_return(nil)
-      expect { subject.send(:ask_config_questions) }.to raise_error(SystemExit)
-    end
-
     it 'should return the defaults if nothing is given' do
       slack_url = Faker::Internet.url
+      defaults = {
+        channel: 'general',
+        username: "Let's Get Quoty",
+        webhook_url: slack_url,
+        icon_emoji: ':mailbox_with_mail:'
+      }
       allow(subject).to receive(:ask_question).with(
-        "\nWhat is your Slack webhook URL? If you don't have one yet, please navigate" \
-        ' to https://api.slack.com/messaging/webhooks to create one, and then come back' \
-        ' here and paste it in the Terminal.'
+        "What is your Slack webhook URL? If you don't have one yet, please navigate to https://api.slack.com/messaging/webhooks to create one, and then come back here and paste it in the Terminal.",
+        nil,
+        required: true
       ).and_return(slack_url)
-      allow(subject).to receive(:ask_question).and_return(nil)
-      expect(subject.send(:ask_config_questions)).to eq(
-        {
-          channel: 'general',
-          username: "Let's Get Quoty",
-          webhook_url: slack_url,
-          icon_emoji: ':mailbox_with_mail:'
-        }
-      )
+      allow(subject).to receive(:ask_question).with(
+        'What slack channel do you wish to post to? (default is "#general")',
+        '#general'
+      ).and_return('general')
+      allow(subject).to receive(:ask_question).with(
+        "What slack username do you wish to post as? (default is \"Let's Get Quoty\")",
+        "Let's Get Quoty"
+      ).and_return("Let's Get Quoty")
+      allow(subject).to receive(:ask_question).with(
+        'What emoji would you like to post with (include the colons at the beginning and end of the emoji name)? (default is ":mailbox_with_mail:")',
+        ':mailbox_with_mail:'
+      ).and_return(':mailbox_with_mail:')
+      expect(subject.send(:ask_config_questions)).to eq(defaults)
     end
   end
 end

metadata

@@ -1,29 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: slack_messaging
 version: !ruby/object:Gem::Version
-  version: 3.0.0
+  version: 3.2.0
 platform: ruby
 authors:
 - Emma Sax
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-03-01 00:00:00.000000000 Z
+date: 2021-07-06 00:00:00.000000000 Z
 dependencies:
-- !ruby/object:Gem::Dependency
-  name: activesupport
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '6.0'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '6.0'
 - !ruby/object:Gem::Dependency
   name: gli
   requirement: !ruby/object:Gem::Requirement
@@ -53,19 +39,19 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '4.1'
 - !ruby/object:Gem::Dependency
-  name: highline
+  name: highline_wrapper
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: '1.1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: '1.1'
 - !ruby/object:Gem::Dependency
   name: httparty
   requirement: !ruby/object:Gem::Requirement
@@ -94,20 +80,6 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '2.5'
-- !ruby/object:Gem::Dependency
-  name: rack
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '2.2'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '2.2'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -164,20 +136,6 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.13'
-- !ruby/object:Gem::Dependency
-  name: rake
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '13.0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '13.0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -196,17 +154,18 @@ dependencies:
   name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '1.10'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
-description: Sending Personalized Slack Messages to a Slack channel of your choice.
+        version: '1.10'
+description: Sending personalized messages and quotes to a Slack channel of your choice
+  via the command-line.
 email:
 executables:
 - slack-messaging
@@ -216,7 +175,7 @@ files:
 - Gemfile
 - Gemfile.lock
 - Guardfile
-- LICENSE
+- LICENSE.md
 - QuoteExample.png
 - README.md
 - Rakefile
@@ -224,21 +183,19 @@ files:
 - lib/slack_messaging.rb
 - lib/slack_messaging/config.rb
 - lib/slack_messaging/default_paths.rb
-- lib/slack_messaging/highline_cli.rb
 - lib/slack_messaging/notify_slack.rb
 - lib/slack_messaging/random_message.rb
 - lib/slack_messaging/send.rb
 - lib/slack_messaging/setup.rb
 - lib/slack_messaging/version.rb
 - spec/slack_messaging/config_spec.rb
-- spec/slack_messaging/highline_cli_spec.rb
 - spec/slack_messaging/notify_slack_spec.rb
 - spec/slack_messaging/random_message_spec.rb
 - spec/slack_messaging/setup_spec.rb
 - spec/spec_helper.rb
 homepage: https://github.com/emmahsax/slack_messaging
 licenses:
-- MIT
+- BSD-3-Clause
 metadata: {}
 post_install_message:
 rdoc_options: []
@@ -255,14 +212,13 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.3
+rubygems_version: 3.2.15
 signing_key:
 specification_version: 4
-summary: Personalized Slack Messages
+summary: Personalized quotes and messages sent straight to Slack
 test_files:
 - spec/spec_helper.rb
 - spec/slack_messaging/config_spec.rb
-- spec/slack_messaging/highline_cli_spec.rb
 - spec/slack_messaging/notify_slack_spec.rb
 - spec/slack_messaging/random_message_spec.rb
 - spec/slack_messaging/setup_spec.rb

data/LICENSE

@@ -1,22 +0,0 @@
-Copyright (c) 2014 Emma Sax
-
-MIT License
-
-Permission is hereby granted, free of charge, to any person obtaining
-a copy of this software and associated documentation files (the
-"Software"), to deal in the Software without restriction, including
-without limitation the rights to use, copy, modify, merge, publish,
-distribute, sublicense, and/or sell copies of the Software, and to
-permit persons to whom the Software is furnished to do so, subject to
-the following conditions:
-
-The above copyright notice and this permission notice shall be
-included in all copies or substantial portions of the Software.
-
-THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
-EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
-MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
-NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE
-LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
-OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
-WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

data/lib/slack_messaging/highline_cli.rb

@@ -1,35 +0,0 @@
-# frozen_string_literal: true
-
-module SlackMessaging
-  class HighlineCli
-    def ask(prompt)
-      highline_client.ask(prompt) do |conf|
-        conf.readline = true
-      end.to_s
-    end
-
-    def ask_yes_no(prompt)
-      answer = highline_client.ask(prompt) do |conf|
-        conf.readline = true
-      end.to_s
-
-      answer.empty? ? true : !!(answer =~ /^y/i)
-    end
-
-    def ask_options(prompt, choices)
-      choices_as_string_options = ''.dup
-      choices.each { |choice| choices_as_string_options << "#{choices.index(choice) + 1}. #{choice}\n" }
-      compiled_prompt = "#{prompt}\n#{choices_as_string_options.strip}"
-
-      index = highline_client.ask(compiled_prompt) do |conf|
-        conf.readline = true
-      end.to_i - 1
-
-      choices[index]
-    end
-
-    private def highline_client
-      @highline_client ||= HighLine.new
-    end
-  end
-end

data/spec/slack_messaging/highline_cli_spec.rb

@@ -1,51 +0,0 @@
-# frozen_string_literal: true
-
-require 'spec_helper'
-require 'slack_messaging'
-
-describe SlackMessaging::HighlineCli do
-  let(:response) { double(:response, readline: true, to_i: 3) }
-  let(:highline_client) { double(:highline_cli, ask: response) }
-
-  before do
-    allow(HighLine).to receive(:new).and_return(highline_client)
-  end
-
-  describe '#ask' do
-    it 'should ask the highline client ask' do
-      expect(highline_client).to receive(:ask)
-      subject.ask(Faker::Lorem.sentence)
-    end
-
-    it 'should return a string' do
-      expect(subject.ask(Faker::Lorem.sentence)).to be_a(String)
-    end
-  end
-
-  describe '#ask_yes_no' do
-    it 'should ask the highline client ask' do
-      expect(highline_client).to receive(:ask)
-      subject.ask_yes_no(Faker::Lorem.sentence)
-    end
-
-    it 'should return a boolean' do
-      expect(subject.ask_yes_no(Faker::Lorem.sentence)).to be_falsey
-    end
-
-    it 'should return true if we say yes' do
-      allow(response).to receive(:to_s).and_return('y')
-      expect(subject.ask_yes_no(Faker::Lorem.sentence)).to be_truthy
-    end
-  end
-
-  describe '#ask_options' do
-    it 'should ask the highline client ask' do
-      expect(highline_client).to receive(:ask)
-      subject.ask_options(Faker::Lorem.sentence, %w[one two three])
-    end
-
-    it 'should return a string from the options' do
-      expect(subject.ask_options(Faker::Lorem.sentence, %w[one two three])).to be_a(String)
-    end
-  end
-end