MirroredRepos/thefuck
1
0
Fork 0
mirror of https://github.com/nvbn/thefuck.git synced 2025-04-20 09:40:46 +01:00
Code Issues Releases Wiki Activity

fix rm dir rule to make it case insensitive

Browse Source 
In bash the output for the command `rm -f foo/` is:

    rm: cannot remove ‘foo/’: Is a directory

And not:

    rm: cannot remove ‘foo/’: is a directory
This commit is contained in:
Timo Furrer 2015-04-22 19:04:52 +02:00
parent b4b599df80
commit c788dfbc14
2 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
tests/rules/test_rm_dir.py
View File

@ -4,6 +4,7 @@ from thefuck.rules.rm_dir import match, get_new_command
def test_match(): def test_match():
assert match(Command('rm foo', '', 'rm: foo: is a directory'), None) assert match(Command('rm foo', '', 'rm: foo: is a directory'), None)
assert match(Command('rm foo', '', 'rm: foo: Is a directory'), None)
assert not match(Command('rm foo', '', ''), None) assert not match(Command('rm foo', '', ''), None)
assert not match(Command('rm foo', '', 'foo bar baz'), None) assert not match(Command('rm foo', '', 'foo bar baz'), None)
assert not match(Command('', '', ''), None) assert not match(Command('', '', ''), None)

2
thefuck/rules/rm_dir.py
View File

@ -5,7 +5,7 @@ from thefuck.utils import sudo_support
@sudo_support @sudo_support
def match(command, settings): def match(command, settings):
return ('rm' in command.script return ('rm' in command.script
and 'is a directory' in command.stderr) and 'is a directory' in command.stderr.lower())
@sudo_support @sudo_support