Back to Repositories

Testing Git Rebase Merge Directory Handling in TheFuck

This test suite validates the git_rebase_merge_dir rule functionality in TheFuck project, focusing on handling git rebase conflicts and directory management. The tests ensure proper detection and resolution of rebase-merge directory scenarios.

Test Coverage Overview

The test suite provides comprehensive coverage of git rebase merge directory handling scenarios.

Tests match() function for detecting rebase-merge conflicts
Validates various git rebase command variations
Covers error message parsing and command suggestions
Tests both positive and negative matching cases

Implementation Analysis

The implementation uses pytest’s parametrize feature to test multiple scenarios efficiently. The tests validate the rule’s ability to detect rebase conflicts and suggest appropriate resolution commands.

Parametrized test cases for different git rebase commands
Fixture-based output simulation
Command object usage for input/output handling

Technical Details

Uses pytest framework for test organization
Implements Command type from thefuck.types
Utilizes pytest.fixture for test data setup
Employs pytest.mark.parametrize for test case variations

Best Practices Demonstrated

The test suite demonstrates several testing best practices for command-line tool validation.

Separation of test cases into match and command generation
Comprehensive error scenario coverage
Clear test case organization
Efficient test data management through fixtures

nvbn/thefuck

tests/rules/test_git_rebase_merge_dir.py

            
import pytest
from thefuck.rules.git_rebase_merge_dir import match, get_new_command
from thefuck.types import Command


@pytest.fixture
def output():
    return ('

It seems that there is already a rebase-merge directory, and
'
            'I wonder if you are in the middle of another rebase.  If that is the
'
            'case, please try
'
            '\tgit rebase (--continue | --abort | --skip)
'
            'If that is not the case, please
'
            '\trm -fr "/foo/bar/baz/egg/.git/rebase-merge"
'
            'and run me again.  I am stopping in case you still have something
'
            'valuable there.
')


@pytest.mark.parametrize('script', [
    'git rebase master',
    'git rebase -skip',
    'git rebase'])
def test_match(output, script):
    assert match(Command(script, output))


@pytest.mark.parametrize('script', ['git rebase master', 'git rebase -abort'])
def test_not_match(script):
    assert not match(Command(script, ''))


@pytest.mark.parametrize('script, result', [
    ('git rebase master', [
        'git rebase --abort', 'git rebase --skip', 'git rebase --continue',
        'rm -fr "/foo/bar/baz/egg/.git/rebase-merge"']),
    ('git rebase -skip', [
        'git rebase --skip', 'git rebase --abort', 'git rebase --continue',
        'rm -fr "/foo/bar/baz/egg/.git/rebase-merge"']),
    ('git rebase', [
        'git rebase --skip', 'git rebase --abort', 'git rebase --continue',
        'rm -fr "/foo/bar/baz/egg/.git/rebase-merge"'])])
def test_get_new_command(output, script, result):
    assert get_new_command(Command(script, output)) == result