highline_wrapper 0.1.0 → 1.0.0

data/spec/highline_wrapper/open_ended_question_spec.rb

@@ -0,0 +1,94 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+require 'highline_wrapper'
+
+describe HighlineWrapper::OpenEndedQuestion do
+  let(:response) { double(:response, readline: true, to_i: 3) }
+  let(:highline) { double(:highline_cli, ask: response) }
+
+  before do
+    allow(HighLine).to receive(:new).and_return(highline)
+    allow(subject).to receive(:puts)
+  end
+
+  after do
+    HighlineWrapper::OpenEndedQuestion.instance_variable_set('@highline', nil)
+  end
+
+  subject { HighlineWrapper::OpenEndedQuestion }
+
+  context 'with the options as defaults' do
+    let(:options) do
+      {
+        secret: false,
+        default: '',
+        required: false
+      }
+    end
+
+    it 'should ask the highline client ask' do
+      expect(highline).to receive(:ask)
+      subject.ask(Faker::Lorem.sentence, options)
+    end
+
+    it 'should return the value the user selects' do
+      answer = Faker::Lorem.sentence
+      allow(highline).to receive(:ask).and_return(answer)
+      resp = subject.ask(Faker::Lorem.sentence, options)
+      expect(resp).to eq(answer)
+    end
+
+    it 'should return empty string if the user skips' do
+      allow(highline).to receive(:ask).and_return('')
+      resp = subject.ask(Faker::Lorem.sentence, options)
+      expect(resp).to eq('')
+    end
+  end
+
+  context 'with required set to true' do
+    let(:options) do
+      {
+        secret: false,
+        default: 'something-goes-here',
+        required: true
+      }
+    end
+
+    it 'should return the value the user selects' do
+      answer = Faker::Lorem.sentence
+      allow(highline).to receive(:ask).and_return(answer)
+      resp = subject.ask(Faker::Lorem.sentence, options)
+      expect(resp).to eq(answer)
+    end
+
+    it 'should recurse multiple times if the user skips' do
+      allow(highline).to receive(:ask).and_return('', '', Faker::Lorem.sentence)
+      expect(subject).to receive(:ask).exactly(3).times.and_call_original
+      subject.ask(Faker::Lorem.sentence, options)
+    end
+  end
+
+  context 'with the default set and the required set to false' do
+    let(:default_string) { Faker::Lorem.sentence }
+    let(:options) do
+      {
+        secret: false,
+        default: default_string,
+        required: false
+      }
+    end
+
+    it 'should return the value the user selects' do
+      answer = Faker::Lorem.sentence
+      allow(highline).to receive(:ask).and_return(answer)
+      resp = subject.ask(Faker::Lorem.sentence, options)
+      expect(resp).to eq(answer)
+    end
+
+    it 'should return the default value the user skips' do
+      allow(highline).to receive(:ask).and_return('')
+      expect(subject.ask(Faker::Lorem.sentence, options)).to eq(default_string)
+    end
+  end
+end

data/spec/highline_wrapper/question_spec.rb

@@ -0,0 +1,63 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+require 'highline_wrapper'
+
+describe HighlineWrapper::Question do
+  let(:response) { double(:response, readline: true, to_i: 3) }
+  let(:highline) { double(:highline_cli, ask: response) }
+
+  before do
+    allow(HighLine).to receive(:new).and_return(highline)
+    allow(subject).to receive(:puts)
+  end
+
+  after do
+    HighlineWrapper::Question.instance_variable_set('@highline', nil)
+  end
+
+  subject { HighlineWrapper::Question }
+
+  describe '#highline' do
+    it 'should start a new highline client' do
+      expect(HighLine).to receive(:new)
+      subject.send(:highline)
+    end
+  end
+
+  describe '#ask_highline' do
+    it 'should ask the highline client a question' do
+      expect(HighLine).to receive(:new)
+      subject.send(:ask_highline, Faker::Lorem.sentence)
+    end
+
+    it 'should accept an optional secret parameter' do
+      expect(HighLine).to receive(:new)
+      subject.send(:ask_highline, Faker::Lorem.sentence, secret: true)
+    end
+  end
+
+  describe '#format_options' do
+    it 'should format the prompt and choices into a single string with new lines' do
+      response = "Prompt\n1. Choice 1\n2. Choice 2"
+      expect(subject.format_options('Prompt', ['Choice 1', 'Choice 2'])).to eq(response)
+    end
+  end
+
+  describe '#format_selection' do
+    let(:choices) { %w[one two three] }
+    let(:index) { 1 }
+
+    context 'with_index as false' do
+      it 'should format the selection based on the index into a hash' do
+        expect(subject.format_selection(choices, index, false)).to eq({ value: 'two' })
+      end
+    end
+
+    context 'with_index as true' do
+      it 'should format the selection based on the index into a hash' do
+        expect(subject.format_selection(choices, index, true)).to eq({ value: 'two', index: 1 })
+      end
+    end
+  end
+end

data/spec/highline_wrapper/yes_no_question_spec.rb

@@ -0,0 +1,88 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+require 'highline_wrapper'
+
+describe HighlineWrapper::YesNoQuestion do
+  let(:response) { double(:response, readline: true, to_i: 3) }
+  let(:highline) { double(:highline_cli, ask: response) }
+
+  before do
+    allow(HighLine).to receive(:new).and_return(highline)
+    allow(subject).to receive(:puts)
+  end
+
+  after do
+    HighlineWrapper::YesNoQuestion.instance_variable_set('@highline', nil)
+  end
+
+  subject { HighlineWrapper::YesNoQuestion }
+
+  context 'with the options as defaults' do
+    let(:options) do
+      {
+        default: true,
+        required: false
+      }
+    end
+
+    it 'should ask the highline client ask' do
+      expect(highline).to receive(:ask)
+      subject.ask(Faker::Lorem.sentence, options)
+    end
+
+    it 'should return the value the user selects' do
+      allow(highline).to receive(:ask).and_return('n')
+      resp = subject.ask(Faker::Lorem.sentence, options)
+      expect(resp).to eq(false)
+    end
+
+    it 'should return true if the user skips' do
+      allow(highline).to receive(:ask).and_return('')
+      resp = subject.ask(Faker::Lorem.sentence, options)
+      expect(resp).to eq(true)
+    end
+  end
+
+  context 'with required set to true' do
+    let(:options) do
+      {
+        default: false,
+        required: true
+      }
+    end
+
+    it 'should return the value the user selects' do
+      allow(highline).to receive(:ask).and_return('y')
+      resp = subject.ask(Faker::Lorem.sentence, options)
+      expect(resp).to eq(true)
+    end
+
+    it 'should recurse multiple times if the user skips' do
+      allow(highline).to receive(:ask).and_return('', '', 'y')
+      expect(subject).to receive(:ask).exactly(3).times.and_call_original
+      subject.ask(Faker::Lorem.sentence, options)
+    end
+  end
+
+  context 'with required set to false' do
+    let(:options) do
+      {
+        default: false,
+        required: false
+      }
+    end
+
+    it 'should return the value the user selects' do
+      allow(highline).to receive(:ask).and_return('y')
+      resp = subject.ask(Faker::Lorem.sentence, options)
+      expect(resp).to eq(true)
+    end
+
+    it 'should return the default if the user skips' do
+      allow(highline).to receive(:ask).and_return('')
+      resp = subject.ask(Faker::Lorem.sentence, options)
+      expect(resp).to eq(false)
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: highline_wrapper
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 1.0.0
 platform: ruby
 authors:
 - Emma Sax
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-03-02 00:00:00.000000000 Z
+date: 2021-03-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: highline
@@ -112,32 +112,41 @@ 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'
+        version: '1.10'
 description: Making it easier to ask simple questions, such as multiple choice questions,
-  yes/no questions, etc, using Highline
+  yes/no questions, etc, using HighLine
 email:
 executables: []
 extensions: []
 extra_rdoc_files: []
 files:
 - Gemfile
+- Gemfile.lock
 - Guardfile
 - LICENSE
 - README.md
 - Rakefile
 - lib/highline_wrapper.rb
-- lib/highline_wrapper/client.rb
+- lib/highline_wrapper/checkbox_question.rb
+- lib/highline_wrapper/multiple_choice_question.rb
+- lib/highline_wrapper/open_ended_question.rb
+- lib/highline_wrapper/question.rb
 - lib/highline_wrapper/version.rb
-- spec/highline_wrapper/client_spec.rb
+- lib/highline_wrapper/yes_no_question.rb
+- spec/highline_wrapper/checkbox_question_spec.rb
+- spec/highline_wrapper/multiple_choice_question_spec.rb
+- spec/highline_wrapper/open_ended_question_spec.rb
+- spec/highline_wrapper/question_spec.rb
+- spec/highline_wrapper/yes_no_question_spec.rb
 - spec/spec_helper.rb
 homepage: https://github.com/emmahsax/highline_wrapper
 licenses:
@@ -161,7 +170,11 @@ requirements: []
 rubygems_version: 3.2.3
 signing_key:
 specification_version: 4
-summary: A little wrapper for Highline
+summary: A little wrapper for HighLine
 test_files:
 - spec/spec_helper.rb
-- spec/highline_wrapper/client_spec.rb
+- spec/highline_wrapper/checkbox_question_spec.rb
+- spec/highline_wrapper/multiple_choice_question_spec.rb
+- spec/highline_wrapper/open_ended_question_spec.rb
+- spec/highline_wrapper/question_spec.rb
+- spec/highline_wrapper/yes_no_question_spec.rb

data/lib/highline_wrapper/client.rb

@@ -1,68 +0,0 @@
-# frozen_string_literal: true
-
-class HighlineWrapper
-  class Client
-    def ask(prompt, secret)
-      highline.ask(prompt) do |conf|
-        conf.readline = true
-        if secret
-          conf.echo = false
-          conf.echo = '*'
-        end
-      end.to_s
-    end
-
-    def ask_yes_no(prompt, preference)
-      answer = highline.ask(prompt) do |conf|
-        conf.readline = true
-      end.to_s
-
-      answer.empty? ? preference : !!(answer =~ /^y/i)
-    end
-
-    def ask_multiple_choice(prompt, choices, with_index)
-      index = highline.ask(format_options(prompt, choices)) do |conf|
-        conf.readline = true
-      end.to_i - 1
-
-      if with_index
-        { choice: choices[index], index: index }
-      else
-        choices[index]
-      end
-    end
-
-    # rubocop:disable Metrics/AbcSize
-    # rubocop:disable Metrics/MethodLength
-    def ask_checkbox(prompt, choices, provide_indices)
-      indices = []
-      selected = []
-
-      answer = highline.ask(format_options(prompt, choices)) do |conf|
-        conf.readline = true
-      end
-
-      answer.split(',').each { |i| indices << i.strip.to_i - 1 }
-
-      if provide_indices
-        indices.each { |index| selected << { choice: choices[index], index: index } }
-      else
-        indices.each { |index| selected << choices[index] }
-      end
-
-      selected
-    end
-    # rubocop:enable Metrics/MethodLength
-    # rubocop:enable Metrics/AbcSize
-
-    private def format_options(prompt, choices)
-      choices_as_string_options = ''.dup
-      choices.each_with_index { |choice, index| choices_as_string_options << "#{index + 1}. #{choice}\n" }
-      "#{prompt}\n#{choices_as_string_options.strip}"
-    end
-
-    private def highline
-      @highline ||= HighLine.new
-    end
-  end
-end

data/spec/highline_wrapper/client_spec.rb

@@ -1,109 +0,0 @@
-# frozen_string_literal: true
-
-require 'spec_helper'
-require 'highline_wrapper'
-
-describe HighlineWrapper::Client do
-  let(:response) { double(:response, readline: true, to_i: 3) }
-  let(:highline) { double(:highline_cli, ask: response) }
-
-  before do
-    allow(HighLine).to receive(:new).and_return(highline)
-  end
-
-  describe '#ask' do
-    it 'should ask the highline client ask' do
-      expect(highline).to receive(:ask)
-      subject.ask(Faker::Lorem.sentence, false)
-    end
-
-    it 'should return a string' do
-      expect(subject.ask(Faker::Lorem.sentence, true)).to be_a(String)
-    end
-  end
-
-  describe '#ask_yes_no' do
-    it 'should ask the highline client ask' do
-      expect(highline).to receive(:ask)
-      subject.ask_yes_no(Faker::Lorem.sentence, true)
-    end
-
-    it 'should return a boolean' do
-      expect(subject.ask_yes_no(Faker::Lorem.sentence, false)).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, false)).to be_truthy
-    end
-
-    it 'should return false if we say yes' do
-      allow(response).to receive(:to_s).and_return('no')
-      expect(subject.ask_yes_no(Faker::Lorem.sentence, true)).to be_falsey
-    end
-
-    context 'when preferring true' do
-      it 'should return true if empty' do
-        allow(response).to receive(:to_s).and_return('')
-        expect(subject.ask_yes_no(Faker::Lorem.sentence, true)).to be_truthy
-      end
-
-      it 'should return false if empty' do
-        allow(response).to receive(:to_s).and_return('')
-        expect(subject.ask_yes_no(Faker::Lorem.sentence, false)).to be_falsey
-      end
-    end
-  end
-
-  describe '#ask_multiple_choice' do
-    it 'should ask the highline client ask' do
-      expect(highline).to receive(:ask)
-      subject.ask_multiple_choice(Faker::Lorem.sentence, %w[one two three], false)
-    end
-
-    context 'when not including the index' do
-      it 'should return an array of one from the options' do
-        resp = subject.ask_multiple_choice(Faker::Lorem.sentence, %w[one two three], false)
-        expect(resp).to be_a(String)
-        expect(resp).to eq('three')
-      end
-    end
-
-    context 'when including the index' do
-      it 'should return an array of two' do
-        resp = subject.ask_multiple_choice(Faker::Lorem.sentence, %w[one two three], true)
-        expect(resp).to be_a(Hash)
-        expect(resp[:choice]).to eq('three')
-        expect(resp[:index]).to eq(2)
-      end
-    end
-  end
-
-  describe '#ask_checkbox' do
-    it 'should ask the highline client ask' do
-      expect(highline).to receive(:ask).and_return('2')
-      subject.ask_checkbox(Faker::Lorem.sentence, %w[one two three], false)
-    end
-
-    context 'when not including the index' do
-      it 'should return an array of one from the options' do
-        allow(highline).to receive(:ask).and_return('1, 3')
-        resp = subject.ask_checkbox(Faker::Lorem.sentence, %w[one two three], false)
-        expect(resp).to be_a(Array)
-        expect(resp.count).to eq(2)
-        expect(resp).to include('one')
-      end
-    end
-
-    context 'when including the index' do
-      it 'should return an array of two' do
-        allow(highline).to receive(:ask).and_return('1, 3')
-        resp = subject.ask_checkbox(Faker::Lorem.sentence, %w[one two three], true)
-        expect(resp).to be_a(Array)
-        expect(resp.count).to eq(2)
-        expect(resp).to include({ choice: 'one', index: 0 })
-        expect(resp).to include({ choice: 'three', index: 2 })
-      end
-    end
-  end
-end