From 25cc98a21a3450a046caf418f08713c82a290805 Mon Sep 17 00:00:00 2001 From: Pablo Santiago Blum de Aguiar Date: Fri, 3 Jul 2015 13:18:48 -0300 Subject: [PATCH] fix(rules.ls_lah): make sure script starts with ls Fix #271 --- tests/rules/test_ls_lah.py | 3 +++ thefuck/rules/ls_lah.py | 4 +++- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/tests/rules/test_ls_lah.py b/tests/rules/test_ls_lah.py index c453f6e5..66bc8365 100644 --- a/tests/rules/test_ls_lah.py +++ b/tests/rules/test_ls_lah.py @@ -3,9 +3,12 @@ from thefuck.rules.ls_lah import match, get_new_command def test_match(): + assert match(Mock(script='ls'), None) assert match(Mock(script='ls file.py'), None) assert match(Mock(script='ls /opt'), None) assert not match(Mock(script='ls -lah /opt'), None) + assert not match(Mock(script='pacman -S binutils'), None) + assert not match(Mock(script='lsof'), None) def test_get_new_command(): diff --git a/thefuck/rules/ls_lah.py b/thefuck/rules/ls_lah.py index 7eba5bda..70f6baab 100644 --- a/thefuck/rules/ls_lah.py +++ b/thefuck/rules/ls_lah.py @@ -1,5 +1,7 @@ def match(command, settings): - return 'ls' in command.script and not ('ls -' in command.script) + return (command.script == 'ls' + or command.script.startswith('ls ') + and not ('ls -' in command.script)) def get_new_command(command, settings):