summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKyrylo Silin <silin@kyrylo.org>2020-04-12 15:17:32 +0800
committerGitHub <noreply@github.com>2020-04-12 15:17:32 +0800
commit7a4166fb04e077536b7b8931c2649a0985cf1a83 (patch)
tree9e4efe5886e2693d0dea07ef4dfc971278bb5343
parent9094b5f8d47e61c1cf64ac07847d46b09e7ef932 (diff)
parenta164846c4dc69386432a19a684ea64c2a9e737b5 (diff)
downloadpry-7a4166fb04e077536b7b8931c2649a0985cf1a83.tar.gz
Merge pull request #2120 from barrettkingram/fix-help-command-listing
Improve appearance of help command listing
-rw-r--r--lib/pry/command_set.rb2
-rw-r--r--lib/pry/commands/bang.rb2
-rw-r--r--spec/command_set_spec.rb7
3 files changed, 8 insertions, 3 deletions
diff --git a/lib/pry/command_set.rb b/lib/pry/command_set.rb
index 70ec2e53..339e6424 100644
--- a/lib/pry/command_set.rb
+++ b/lib/pry/command_set.rb
@@ -193,7 +193,7 @@ class Pry
options = original_options.merge!(
desc: "Alias for `#{action}`",
- listing: match
+ listing: match.is_a?(String) ? match : match.inspect
).merge!(options)
# ensure default description is used if desc is nil
diff --git a/lib/pry/commands/bang.rb b/lib/pry/commands/bang.rb
index ca42834e..1879c64e 100644
--- a/lib/pry/commands/bang.rb
+++ b/lib/pry/commands/bang.rb
@@ -6,7 +6,7 @@ class Pry
match(/^\s*!\s*$/)
group 'Editing'
description 'Clear the input buffer.'
- command_options use_prefix: false
+ command_options use_prefix: false, listing: '!'
banner <<-'BANNER'
Clear the input buffer. Useful if the parsing process goes wrong and you get
diff --git a/spec/command_set_spec.rb b/spec/command_set_spec.rb
index fc67d140..c5243999 100644
--- a/spec/command_set_spec.rb
+++ b/spec/command_set_spec.rb
@@ -185,11 +185,16 @@ RSpec.describe Pry::CommandSet do
expect(new_command.description).to eq('Alias for `test`')
end
- it "sets aliased command's listing" do
+ it "sets aliased command's listing for string alias" do
new_command = subject.alias_command('new-test', 'test')
expect(new_command.options).to include(listing: 'new-test')
end
+ it "sets aliased command's listing for regex alias" do
+ new_command = subject.alias_command(/test[!?]+/, 'test')
+ expect(new_command.options[:listing].to_s).to eq('/test[!?]+/')
+ end
+
it "sets group for the aliased command automatically" do
new_command = subject.alias_command('new-test', 'test')
expect(new_command.group).to eq('Aliases')