Skip to content

Commit

Permalink
Move from tmpdir pytest fixture to tmp_path (#206)
Browse files Browse the repository at this point in the history
  • Loading branch information
adamchainz committed Jan 16, 2023
1 parent ea5e9bf commit 6af8099
Showing 1 changed file with 36 additions and 36 deletions.
72 changes: 36 additions & 36 deletions tests/blacken_docs_test.py
Expand Up @@ -157,7 +157,7 @@ def test_format_src_latex_minted_pycon_indented():
)


def test_src_pythontex(tmpdir):
def test_src_pythontex():
before = (
'hello\n'
'\\begin{pyblock}\n'
Expand Down Expand Up @@ -305,49 +305,49 @@ def test_format_src_rst_python_inside_non_python_code_block():
assert after == before


def test_integration_ok(tmpdir, capsys):
f = tmpdir.join('f.md')
f.write(
def test_integration_ok(tmp_path, capsys):
f = tmp_path / 'f.md'
f.write_text(
'```python\n'
'f(1, 2, 3)\n'
'```\n',
)
assert not blacken_docs.main((str(f),))
assert not capsys.readouterr()[1]
assert f.read() == (
assert f.read_text() == (
'```python\n'
'f(1, 2, 3)\n'
'```\n'
)


def test_integration_modifies(tmpdir, capsys):
f = tmpdir.join('f.md')
f.write(
def test_integration_modifies(tmp_path, capsys):
f = tmp_path / 'f.md'
f.write_text(
'```python\n'
'f(1,2,3)\n'
'```\n',
)
assert blacken_docs.main((str(f),))
out, _ = capsys.readouterr()
assert out == f'{f}: Rewriting...\n'
assert f.read() == (
assert f.read_text() == (
'```python\n'
'f(1, 2, 3)\n'
'```\n'
)


def test_integration_line_length(tmpdir):
f = tmpdir.join('f.md')
f.write(
def test_integration_line_length(tmp_path):
f = tmp_path / 'f.md'
f.write_text(
'```python\n'
'foo(very_very_very_very_very_very_very, long_long_long_long_long)\n'
'```\n',
)
assert not blacken_docs.main((str(f), '--line-length=80'))
assert blacken_docs.main((str(f), '--line-length=50'))
assert f.read() == (
assert f.read_text() == (
'```python\n'
'foo(\n'
' very_very_very_very_very_very_very,\n'
Expand All @@ -357,9 +357,9 @@ def test_integration_line_length(tmpdir):
)


def test_integration_py36(tmpdir):
f = tmpdir.join('f.md')
f.write(
def test_integration_py36(tmp_path):
f = tmp_path / 'f.md'
f.write_text(
'```python\n'
'def very_very_long_function_name(\n'
' very_very_very_very_very_very,\n'
Expand All @@ -371,7 +371,7 @@ def test_integration_py36(tmpdir):
)
assert not blacken_docs.main((str(f),))
assert blacken_docs.main((str(f), '--target-version=py36'))
assert f.read() == (
assert f.read_text() == (
'```python\n'
'def very_very_long_function_name(\n'
' very_very_very_very_very_very,\n'
Expand All @@ -383,9 +383,9 @@ def test_integration_py36(tmpdir):
)


def test_integration_filename_last(tmpdir):
f = tmpdir.join('f.md')
f.write(
def test_integration_filename_last(tmp_path):
f = tmp_path / 'f.md'
f.write_text(
'```python\n'
'def very_very_long_function_name(\n'
' very_very_very_very_very_very,\n'
Expand All @@ -397,7 +397,7 @@ def test_integration_filename_last(tmpdir):
)
assert not blacken_docs.main((str(f),))
assert blacken_docs.main(('--target-version', 'py36', str(f)))
assert f.read() == (
assert f.read_text() == (
'```python\n'
'def very_very_long_function_name(\n'
' very_very_very_very_very_very,\n'
Expand All @@ -409,9 +409,9 @@ def test_integration_filename_last(tmpdir):
)


def test_integration_multiple_target_version(tmpdir):
f = tmpdir.join('f.md')
f.write(
def test_integration_multiple_target_version(tmp_path):
f = tmp_path / 'f.md'
f.write_text(
'```python\n'
'def very_very_long_function_name(\n'
' very_very_very_very_very_very,\n'
Expand All @@ -427,41 +427,41 @@ def test_integration_multiple_target_version(tmpdir):
)


def test_integration_skip_string_normalization(tmpdir):
f = tmpdir.join('f.md')
f.write(
def test_integration_skip_string_normalization(tmp_path):
f = tmp_path / 'f.md'
f.write_text(
'```python\n'
"f('hi')\n"
'```\n',
)
assert not blacken_docs.main((str(f), '--skip-string-normalization'))
assert f.read() == (
assert f.read_text() == (
'```python\n'
"f('hi')\n"
'```\n'
)


def test_integration_syntax_error(tmpdir, capsys):
f = tmpdir.join('f.md')
f.write(
def test_integration_syntax_error(tmp_path, capsys):
f = tmp_path / 'f.md'
f.write_text(
'```python\n'
'f(\n'
'```\n',
)
assert blacken_docs.main((str(f),))
out, _ = capsys.readouterr()
assert out.startswith(f'{f}:1: code block parse error')
assert f.read() == (
assert f.read_text() == (
'```python\n'
'f(\n'
'```\n'
)


def test_integration_ignored_syntax_error(tmpdir, capsys):
f = tmpdir.join('f.md')
f.write(
def test_integration_ignored_syntax_error(tmp_path, capsys):
f = tmp_path / 'f.md'
f.write_text(
'```python\n'
'f( )\n'
'```\n'
Expand All @@ -472,7 +472,7 @@ def test_integration_ignored_syntax_error(tmpdir, capsys):
)
assert blacken_docs.main((str(f), '--skip-errors'))
out, _ = capsys.readouterr()
assert f.read() == (
assert f.read_text() == (
'```python\n'
'f()\n'
'```\n'
Expand Down

0 comments on commit 6af8099

Please sign in to comment.