Ignore whitespace-only lines in addition to purely blank lines.

Fixes #38.
nmlorg · May 13, 2019 · 6223d4c · 6223d4c
1 parent fa59b02
commit 6223d4c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/metabot/modules/echo.py b/metabot/modules/echo.py
@@ -18,7 +18,7 @@ def moddispatch(ctx, msg, modconf):  # pylint: disable=missing-docstring
 
 
 def echo(ctx, msg, message):  # pylint: disable=missing-docstring
-    lines = [line for line in message.splitlines() if line]
+    lines = [line for line in message.splitlines() if line.strip()]
     page = ctx.text.isdigit() and int(ctx.text) or 1
     for line in lines[:page]:
         msg.add('%s', line)

diff --git a/metabot/modules/test_echo.py b/metabot/modules/test_echo.py
@@ -31,6 +31,7 @@ def test_echo(conversation):  # pylint: disable=redefined-outer-name
     conversation.multibot.conf['bots']['modulestestbot']['issue37']['echo']['about'] = (
         'First line.\n'
         'Second line.\n'
+        ' \n'
         'Last line.')
 
     assert conversation.message('/about') == """\