Fix pattern (?<query>.*) to (?<query>+*)

tbpgr · Dec 9, 2014 · 6da633c · 6da633c
1 parent 36adb6b
commit 6da633c
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/README.md b/README.md
@@ -29,7 +29,7 @@ Or install it yourself as:
 
 |Command|Pattern|Description|
 |:--|:--|:--|
-|help|/help *(?&lt;query&gt;.*)\z/|Show all help commands that match &lt;query&gt;.|
+|help|/help *(?&lt;query&gt;.+)\z/|Show all help commands that match &lt;query&gt;.|
 
 ## Usage
 ### help

diff --git a/lib/ruboty/handlers/help_query.rb b/lib/ruboty/handlers/help_query.rb
@@ -5,7 +5,7 @@ module Handlers
     # HelpQuery
     class HelpQuery < Base
       on(
-        /help\s*(?<query>.*)\z/,
+        /help\s*(?<query>.+)\z/,
         name: 'help',
         description: 'Show all help commands that match <query>.'
       )

diff --git a/spec/ruboty/help_query/actions/help_spec.rb b/spec/ruboty/help_query/actions/help_spec.rb
@@ -33,7 +33,7 @@ def reply(message)
         case_title: 'exist user case',
         query: '.*p.*',
         expected: <<-EOS
-robot /help\\s*(?<query>.*)\\z/ - Show all help commands that match <query>.
+robot /help\\s*(?<query>.+)\\z/ - Show all help commands that match <query>.
 robot /help( me)?\\z/i         - Show this help message
 robot /ping\\z/i               - Return PONG to PING
         EOS