mirror of
https://github.com/pre-commit/pre-commit-hooks.git
synced 2026-04-07 04:26:52 +00:00
fix tests to handle default branch main or master
This commit is contained in:
parent
31903eabdb
commit
c96572c378
3 changed files with 24 additions and 6 deletions
|
|
@ -14,3 +14,8 @@ def get_resource_path(path):
|
||||||
def git_commit(*args, **kwargs):
|
def git_commit(*args, **kwargs):
|
||||||
cmd = ('git', 'commit', '--no-gpg-sign', '--no-verify', '--no-edit', *args)
|
cmd = ('git', 'commit', '--no-gpg-sign', '--no-verify', '--no-edit', *args)
|
||||||
subprocess.check_call(cmd, **kwargs)
|
subprocess.check_call(cmd, **kwargs)
|
||||||
|
|
||||||
|
|
||||||
|
def get_default_branch():
|
||||||
|
cmd = ('git', 'config', '--get', 'init.defaultBranch')
|
||||||
|
return subprocess.getoutput(cmd).strip() or 'master'
|
||||||
|
|
|
||||||
|
|
@ -7,10 +7,14 @@ import pytest
|
||||||
|
|
||||||
from pre_commit_hooks.check_merge_conflict import main
|
from pre_commit_hooks.check_merge_conflict import main
|
||||||
from pre_commit_hooks.util import cmd_output
|
from pre_commit_hooks.util import cmd_output
|
||||||
|
from testing.util import get_default_branch
|
||||||
from testing.util import get_resource_path
|
from testing.util import get_resource_path
|
||||||
from testing.util import git_commit
|
from testing.util import git_commit
|
||||||
|
|
||||||
|
|
||||||
|
default_branch = get_default_branch()
|
||||||
|
|
||||||
|
|
||||||
@pytest.fixture
|
@pytest.fixture
|
||||||
def f1_is_a_conflict_file(tmpdir):
|
def f1_is_a_conflict_file(tmpdir):
|
||||||
# Make a merge conflict
|
# Make a merge conflict
|
||||||
|
|
@ -150,7 +154,12 @@ def test_worktree_merge_conflicts(f1_is_a_conflict_file, tmpdir, capsys):
|
||||||
cmd_output('git', 'worktree', 'add', str(worktree))
|
cmd_output('git', 'worktree', 'add', str(worktree))
|
||||||
with worktree.as_cwd():
|
with worktree.as_cwd():
|
||||||
cmd_output(
|
cmd_output(
|
||||||
'git', 'pull', '--no-rebase', 'origin', 'master', retcode=None,
|
'git',
|
||||||
|
'pull',
|
||||||
|
'--no-rebase',
|
||||||
|
'origin',
|
||||||
|
default_branch,
|
||||||
|
retcode=None,
|
||||||
)
|
)
|
||||||
msg = f1_is_a_conflict_file.join('.git/worktrees/worktree/MERGE_MSG')
|
msg = f1_is_a_conflict_file.join('.git/worktrees/worktree/MERGE_MSG')
|
||||||
assert msg.exists()
|
assert msg.exists()
|
||||||
|
|
|
||||||
|
|
@ -5,19 +5,23 @@ import pytest
|
||||||
from pre_commit_hooks.no_commit_to_branch import is_on_branch
|
from pre_commit_hooks.no_commit_to_branch import is_on_branch
|
||||||
from pre_commit_hooks.no_commit_to_branch import main
|
from pre_commit_hooks.no_commit_to_branch import main
|
||||||
from pre_commit_hooks.util import cmd_output
|
from pre_commit_hooks.util import cmd_output
|
||||||
|
from testing.util import get_default_branch
|
||||||
from testing.util import git_commit
|
from testing.util import git_commit
|
||||||
|
|
||||||
|
|
||||||
|
default_branch = get_default_branch()
|
||||||
|
|
||||||
|
|
||||||
def test_other_branch(temp_git_dir):
|
def test_other_branch(temp_git_dir):
|
||||||
with temp_git_dir.as_cwd():
|
with temp_git_dir.as_cwd():
|
||||||
cmd_output('git', 'checkout', '-b', 'anotherbranch')
|
cmd_output('git', 'checkout', '-b', 'anotherbranch')
|
||||||
assert is_on_branch({'master'}) is False
|
assert is_on_branch({default_branch}) is False
|
||||||
|
|
||||||
|
|
||||||
def test_multi_branch(temp_git_dir):
|
def test_multi_branch(temp_git_dir):
|
||||||
with temp_git_dir.as_cwd():
|
with temp_git_dir.as_cwd():
|
||||||
cmd_output('git', 'checkout', '-b', 'another/branch')
|
cmd_output('git', 'checkout', '-b', 'another/branch')
|
||||||
assert is_on_branch({'master'}) is False
|
assert is_on_branch({default_branch}) is False
|
||||||
|
|
||||||
|
|
||||||
def test_multi_branch_fail(temp_git_dir):
|
def test_multi_branch_fail(temp_git_dir):
|
||||||
|
|
@ -28,7 +32,7 @@ def test_multi_branch_fail(temp_git_dir):
|
||||||
|
|
||||||
def test_master_branch(temp_git_dir):
|
def test_master_branch(temp_git_dir):
|
||||||
with temp_git_dir.as_cwd():
|
with temp_git_dir.as_cwd():
|
||||||
assert is_on_branch({'master'}) is True
|
assert is_on_branch({default_branch}) is True
|
||||||
|
|
||||||
|
|
||||||
def test_main_branch_call(temp_git_dir):
|
def test_main_branch_call(temp_git_dir):
|
||||||
|
|
@ -50,11 +54,11 @@ def test_branch_pattern_fail(temp_git_dir):
|
||||||
assert is_on_branch(set(), {'another/.*'}) is True
|
assert is_on_branch(set(), {'another/.*'}) is True
|
||||||
|
|
||||||
|
|
||||||
@pytest.mark.parametrize('branch_name', ('master', 'another/branch'))
|
@pytest.mark.parametrize('branch_name', (default_branch, 'another/branch'))
|
||||||
def test_branch_pattern_multiple_branches_fail(temp_git_dir, branch_name):
|
def test_branch_pattern_multiple_branches_fail(temp_git_dir, branch_name):
|
||||||
with temp_git_dir.as_cwd():
|
with temp_git_dir.as_cwd():
|
||||||
cmd_output('git', 'checkout', '-b', branch_name)
|
cmd_output('git', 'checkout', '-b', branch_name)
|
||||||
assert main(('--branch', 'master', '--pattern', 'another/.*'))
|
assert main(('--branch', default_branch, '--pattern', 'another/.*'))
|
||||||
|
|
||||||
|
|
||||||
def test_main_default_call(temp_git_dir):
|
def test_main_default_call(temp_git_dir):
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue