command_kit 0.1.0 → 0.2.0

data/spec/pager_spec.rb

@@ -19,8 +19,8 @@ describe CommandKit::Pager do
 
       subject { command_class.new(env: {'PAGER' => pager}) }
 
-      it "must set @pager to the PAGER env variable" do
-        expect(subject.instance_variable_get('@pager')).to eq(pager)
+      it "must set @pager_command to the PAGER env variable" do
+        expect(subject.instance_variable_get('@pager_command')).to eq(pager)
       end
     end
 
@@ -29,7 +29,7 @@ describe CommandKit::Pager do
         subject { command_class.new(env: {'PATH' => ENV['PATH']}) }
 
         it "must search PATH for one of the pagers" do
-          expect(subject.instance_variable_get('@pager')).to eq("less -r")
+          expect(subject.instance_variable_get('@pager_command')).to eq("less -r")
         end
       end
     end
@@ -46,9 +46,9 @@ describe CommandKit::Pager do
       end
     end
 
-    context "when @pager isn't initialized" do
+    context "when @pager_command isn't initialized" do
       before do
-        subject.instance_variable_set('@pager',nil)
+        subject.instance_variable_set('@pager_command',nil)
       end
 
       it "must yield stdout" do
@@ -58,7 +58,7 @@ describe CommandKit::Pager do
       end
     end
 
-    context "when stdout is a TTY and @pager is initialized" do
+    context "when stdout is a TTY and @pager_command is initialized" do
       let(:pager) { 'less -r' }
 
       subject { command_class.new(env: {'PAGER' => pager}) }
@@ -149,4 +149,69 @@ describe CommandKit::Pager do
       end
     end
   end
+
+  describe "#pipe_to_pager" do
+    context "when @pager_command is initialized" do
+      let(:pager) { 'less' }
+
+      subject do
+        command_class.new(env: {'PAGER' => pager})
+      end
+
+      context "and when given a single String" do
+        let(:command) { "find ." }
+
+        it "must run the command but piped into the pager" do
+          expect(subject).to receive(:system).with("#{command} | #{pager}")
+
+          subject.pipe_to_pager(command)
+        end
+      end
+
+      context "and when given a String and additional arguments" do
+        let(:command)   { 'find'          }
+        let(:arguments) { %[. -name *.md] }
+
+        let(:escaped_command) do
+          Shellwords.shelljoin([command,*arguments])
+        end
+
+        it "must shell escape the command and arguments" do
+          expect(subject).to receive(:system).with(
+            "#{escaped_command} | #{pager}"
+          )
+
+          subject.pipe_to_pager(command,*arguments)
+        end
+      end
+    end
+
+    context "when @pager_command is not initialized" do
+      before do
+        subject.instance_variable_set('@pager_command',nil)
+      end
+
+      let(:command)   { 'find' }
+      let(:arguments) { %[. -name *.md] }
+
+      it "must pass the command and any additional arguments to #system" do
+        expect(subject).to receive(:system).with(command,*arguments)
+
+        subject.pipe_to_pager(command,*arguments)
+      end
+
+      context "when one of the arguments is not a String" do
+        let(:command)   { :find }
+        let(:arguments) { ['.', :"-name", "*.md"] }
+
+        it "must convert the command to a String before calling #system" do
+          expect(subject).to receive(:system).with(
+            command.to_s, *arguments.map(&:to_s)
+          )
+
+          subject.pipe_to_pager(command,*arguments)
+        end
+      end
+    end
+  end
 end

data/spec/sudo_spec.rb

@@ -0,0 +1,51 @@
+require 'spec_helper'
+require 'command_kit/sudo'
+
+describe CommandKit::Sudo do
+  module TestSudo
+    class TestCommand
+      include CommandKit::Sudo
+    end
+  end
+
+  let(:command_class) { TestSudo::TestCommand }
+  subject { command_class.new }
+
+  describe "#sudo" do
+    let(:command) { "ls" }
+    let(:arguments) { ["-la", "~root"] }
+    let(:status) { double(:status) }
+
+    context "on UNIX" do
+      context "when the UID is 0" do
+        before { allow(Process).to receive(:uid).and_return(0) }
+
+        it "must execute the command without sudo" do
+          expect(subject).to receive(:system).with(command,*arguments).and_return(status)
+
+          expect(subject.sudo(command,*arguments)).to be(status)
+        end
+      end
+
+      context "when the UID is not 0" do
+        before { allow(Process).to receive(:uid).and_return(1000) }
+
+        it "must execute the command with 'sudo ...'" do
+          expect(subject).to receive(:system).with('sudo',command,*arguments).and_return(status)
+
+          expect(subject.sudo(command,*arguments)).to be(status)
+        end
+      end
+    end
+
+    context "on Windows" do
+      subject { command_class.new(os: :windows) }
+
+      it "must execute the command with 'runas /user:administrator ...'" do
+        expect(subject).to receive(:system).with('runas','/user:administrator',command,*arguments).and_return(status)
+
+        expect(subject.sudo(command,*arguments)).to be(status)
+      end
+    end
+  end
+end

data/spec/terminal_spec.rb

@@ -43,6 +43,36 @@ describe CommandKit::Terminal do
     end
   end
 
+  describe "#tty?" do
+    context "when stdout is connected to a TTY" do
+      subject { command_class.new(stdout: STDOUT) }
+
+      it do
+        skip "STDOUT is not a TTY" unless STDOUT.tty?
+
+        expect(subject.tty?).to be(true)
+      end
+    end
+
+    context "when stdout is not connected to a TTY" do
+      subject { command_class.new(stdout: StringIO.new) }
+
+      it do
+        expect(subject.tty?).to be(false)
+      end
+    end
+
+    context "when IO.console is missing" do
+      before do
+        expect(IO).to receive(:respond_to?).with(:console).and_return(false)
+      end
+
+      it do
+        expect(subject.tty?).to be(false)
+      end
+    end
+  end
+
   describe "#terminal" do
     context "when stdout is connected to a TTY" do
       subject { command_class.new(stdout: STDOUT) }

data/spec/usage_spec.rb

@@ -182,7 +182,7 @@ describe CommandKit::Usage do
           [
             "#{subject.command_name} #{command_class.usage[0]}",
             "#{subject.command_name} #{command_class.usage[1]}",
-            "#{subject.command_name} #{command_class.usage[2]}",
+            "#{subject.command_name} #{command_class.usage[2]}"
           ]
         )
       end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: command_kit
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.2.0
 platform: ruby
 authors:
 - Postmodern
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-07-17 00:00:00.000000000 Z
+date: 2021-08-31 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -38,6 +38,7 @@ files:
 - ".github/workflows/ruby.yml"
 - ".gitignore"
 - ".rspec"
+- ".rubocop.yml"
 - ".yardopts"
 - ChangeLog.md
 - Gemfile
@@ -75,6 +76,8 @@ files:
 - lib/command_kit/inflector.rb
 - lib/command_kit/interactive.rb
 - lib/command_kit/main.rb
+- lib/command_kit/man.rb
+- lib/command_kit/open_app.rb
 - lib/command_kit/options.rb
 - lib/command_kit/options/option.rb
 - lib/command_kit/options/option_value.rb
@@ -83,11 +86,14 @@ files:
 - lib/command_kit/options/verbose.rb
 - lib/command_kit/options/version.rb
 - lib/command_kit/os.rb
+- lib/command_kit/os/linux.rb
+- lib/command_kit/package_manager.rb
 - lib/command_kit/pager.rb
 - lib/command_kit/printing.rb
 - lib/command_kit/printing/indent.rb
 - lib/command_kit/program_name.rb
 - lib/command_kit/stdio.rb
+- lib/command_kit/sudo.rb
 - lib/command_kit/terminal.rb
 - lib/command_kit/usage.rb
 - lib/command_kit/version.rb
@@ -120,24 +126,33 @@ files:
 - spec/inflector_spec.rb
 - spec/interactive_spec.rb
 - spec/main_spec.rb
+- spec/man_spec.rb
+- spec/open_app_spec.rb
 - spec/options/option_spec.rb
 - spec/options/option_value_spec.rb
 - spec/options/parser_spec.rb
 - spec/options_spec.rb
+- spec/os/linux_spec.rb
 - spec/os_spec.rb
+- spec/package_manager_spec.rb
 - spec/pager_spec.rb
 - spec/printing/indent_spec.rb
 - spec/printing_spec.rb
 - spec/program_name_spec.rb
 - spec/spec_helper.rb
 - spec/stdio_spec.rb
+- spec/sudo_spec.rb
 - spec/terminal_spec.rb
 - spec/usage_spec.rb
 - spec/xdg_spec.rb
 homepage: https://github.com/postmodern/command_kit.rb#readme
 licenses:
 - MIT
-metadata: {}
+metadata:
+  documentation_uri: https://rubydoc.info/gems/command_kit
+  source_code_uri: https://github.com/postmodern/command_kit.rb
+  bug_tracker_uri: https://github.com/postmodern/command_kit.rb/issues
+  changelog_uri: https://github.com/postmodern/command_kit.rb/blob/main/ChangeLog.md
 post_install_message:
 rdoc_options: []
 require_paths:
@@ -153,7 +168,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.15
+rubygems_version: 3.2.22
 signing_key:
 specification_version: 4
 summary: A toolkit for building Ruby CLI commands