Move the tests imto a separate directory.

Author: moigagoo

pyproject.lock

@@ -208,6 +208,18 @@ platform = "*"
 coverage = ">=3.7.1"
 pytest = ">=2.6.0"
 [[package]]
+name = "pytest-datadir"
+version = "1.0.1"
+description = "pytest plugin for test data directories and files"
+category = "dev"
+optional = false
+python-versions = "*"
+platform = "*"
+
+[package.dependencies]
+pytest = ">=2.7.0"
+pathlib = "*"
+[[package]]
 name = "requests"
 version = "2.18.4"
 description = "Python HTTP for Humans."
@@ -260,7 +272,7 @@ platform = "*"
 [metadata]
 python-versions = "^3.6"
 platform = "*"
-content-hash = "7ce2c03447eb45ba59dcdf80124b6e5462a7d84d881a638ce132cfb1662b1f2d"
+content-hash = "523ec577d845a8cb3ea985f5d255ebed924269e3b0eeda7fb6cef8fc3ba00b6c"
 
 [metadata.hashes]
 astroid = [ "032f6e09161e96f417ea7fad46d3fac7a9019c775f202182c22df0e4f714cb1c", "dea42ae6e0b789b543f728ddae7ddb6740ba33a49fb52c4a4d9cb7bb4aa6ec09",]
@@ -282,6 +294,7 @@ pygments = [ "78f3f434bcc5d6ee09020f92ba487f95ba50f1e3ef83ae96b9d5ffa1bab25c5d",
 pylint = [ "aa519865f8890a5905fa34924fed0f3bfc7d84fc9f9142c16dac52ffecd25a39", "c353d8225195b37cc3aef18248b8f3fe94c5a6a95affaf885ae21a24ca31d8eb",]
 pytest = [ "c76e93f3145a44812955e8d46cdd302d8a45fbfc7bf22be24fe231f9d8d8853a", "39555d023af3200d004d09e51b4dd9fdd828baa863cded3fd6ba2f29f757ae2d",]
 pytest-cov = [ "890fe5565400902b0c78b5357004aab1c814115894f4f21370e2433256a3eeec", "03aa752cf11db41d281ea1d807d954c4eda35cfa1b21d6971966cc041bbf6e2d",]
+pytest-datadir = [ "68226fbb6780366950a9409bb705cd4d05b11bff07a4569d4cc589c69c259def", "3e5d1892c68d0dfb6863e87d6a1f683e67be60802fee798a3ccae5dbda1ab378",]
 requests = [ "6a1b267aa90cac58ac3a765d067950e7dbbf75b1da07e895d1f594193a40a38b", "9c443e7324ba5b85070c4a818ade28bfabedf16ea10206da1132edaa6dda237e",]
 rope = [ "a09edfd2034fd50099a67822f9bd851fbd0f4e98d3b87519f6267b60e50d80d1",]
 six = [ "832dc0e10feb1aa2c68dcc57dbb658f1c7e65b9b61af69048abc87a2db00a0eb", "70e8a77beed4562e7f14fe23a786b54f6296e34344c23bc42f07b15018ff98e9",]

pyproject.toml

@@ -20,3 +20,4 @@ pytest-cov = "^2.5"
 rope = "^0.10.7"
 codecov = "^2.0"
 pygments = "^2.2"
+pytest-datadir = "^1.0"

test_cliar.py tests/test_basicmath.py

@@ -1,193 +1,192 @@
 from subprocess import run
 from math import factorial, log
 
+from pytest import fixture
 
-def test_positional_args(capfd):
+
+def test_positional_args(capfd, datadir):
     x, y = 12, 34
 
-    run(f'python basicmath.py add {x} {y}', shell=True)
+    run(f'python {datadir/"basicmath.py"} add {x} {y}', shell=True)
     assert int(capfd.readouterr().out) == x + y
 
 
-def test_optional_args(capfd):
+def test_optional_args(capfd, datadir):
     x, power = 12, 3
 
-    run(f'python basicmath.py power {x} --power {power}', shell=True)
+    run(f'python {datadir/"basicmath.py"} power {x} --power {power}', shell=True)
     assert int(capfd.readouterr().out) == x ** power
 
-    run(f'python basicmath.py power {x} -p {power}', shell=True)
+    run(f'python {datadir/"basicmath.py"} power {x} -p {power}', shell=True)
     assert int(capfd.readouterr().out) == x ** power
 
-    run(f'python basicmath.py power {x}', shell=True)
+    run(f'python {datadir/"basicmath.py"} power {x}', shell=True)
     assert int(capfd.readouterr().out) == x ** 2
 
 
-def test_no_args(capfd):
+def test_no_args(capfd, datadir):
     from math import pi
 
-    run('python basicmath.py pi', shell=True)
+    run(f'python {datadir/"basicmath.py"} pi', shell=True)
     assert float(capfd.readouterr().out) == pi
 
-    run('python basicmath.py avg', shell=True)
+    run(f'python {datadir/"basicmath.py"} avg', shell=True)
     assert float(capfd.readouterr().out) == sum((1, 2, 3))/3
 
 
-def test_root_command(capfd):
+def test_root_command(capfd, datadir):
     version = '0.1.0'
 
-    run('python basicmath.py', shell=True)
+    run(f'python {datadir/"basicmath.py"}', shell=True)
     assert capfd.readouterr().out.strip() == 'Welcome to math!'
 
-    run('python basicmath.py --version', shell=True)
+    run(f'python {datadir/"basicmath.py"} --version', shell=True)
     assert capfd.readouterr().out.strip() == version
 
-    run('python basicmath.py -v', shell=True)
+    run(f'python {datadir/"basicmath.py"} -v', shell=True)
     assert capfd.readouterr().out.strip() == version
 
 
-def test_nargs(capfd):
+def test_nargs(capfd, datadir):
     numbers = 1, 2, 42, 101
 
-    run(f'python basicmath.py summ {" ".join(str(number) for number in numbers)}', shell=True)
+    run(f'python {datadir/"basicmath.py"} summ {" ".join(str(number) for number in numbers)}', shell=True)
     assert int(capfd.readouterr().out) == sum(numbers)
 
     numbers = 1, 2, 42.2, 101.1
 
     run(
-        f'python basicmath.py avg --numbers {" ".join(str(number) for number in numbers)}',
+        f'python {datadir/"basicmath.py"} avg --numbers {" ".join(str(number) for number in numbers)}',
         shell=True
     )
     assert float(capfd.readouterr().out) == sum(numbers)/len(numbers)
 
-    run(f'python basicmath.py avg -n {" ".join(str(number) for number in numbers)}', shell=True)
+    run(f'python {datadir/"basicmath.py"} avg -n {" ".join(str(number) for number in numbers)}', shell=True)
     assert float(capfd.readouterr().out) == sum(numbers)/len(numbers)
 
 
-def test_negative_numbers(capfd):
+def test_negative_numbers(capfd, datadir):
     x, y = 12, -34
 
-    run(f'python basicmath.py add {x} {y}', shell=True)
+    run(f'python {datadir/"basicmath.py"} add {x} {y}', shell=True)
     assert int(capfd.readouterr().out) == x + y
 
 
-def test_type_casting(capfd):
+def test_type_casting(capfd, datadir):
     error = "argument {arg}: invalid int value: '{value}'"
     x, y = 12, 34.0
 
-    run(f'python basicmath.py add {x} {y}', shell=True)
+    run(f'python {datadir/"basicmath.py"} add {x} {y}', shell=True)
     assert capfd.readouterr().err.splitlines()[-1].endswith(error.format(arg='y', value=y))
 
     x, y = 12.0, 34.0
 
-    run(f'python basicmath.py add {x} {y}', shell=True)
+    run(f'python {datadir/"basicmath.py"} add {x} {y}', shell=True)
     assert capfd.readouterr().err.splitlines()[-1].endswith(error.format(arg='x', value=x))
 
     x, y = 'foo', 42
 
-    run(f'python basicmath.py add {x} {y}', shell=True)
+    run(f'python {datadir/"basicmath.py"} add {x} {y}', shell=True)
     assert capfd.readouterr().err.splitlines()[-1].endswith(error.format(arg='x', value=x))
 
 
-def test_missing_args(capfd):
+def test_missing_args(capfd, datadir):
     error = 'the following arguments are required: {args}'
     x = 12
 
-    run(f'python basicmath.py add {x}', shell=True)
+    run(f'python {datadir/"basicmath.py"} add {x}', shell=True)
     assert capfd.readouterr().err.splitlines()[-1].endswith(error.format(args='y'))
 
-    run(f'python basicmath.py add', shell=True)
+    run(f'python {datadir/"basicmath.py"} add', shell=True)
     assert capfd.readouterr().err.splitlines()[-1].endswith(error.format(args='x, y'))
 
 
-def test_redundant_args(capfd):
+def test_redundant_args(capfd, datadir):
     error = 'unrecognized arguments: {args}'
     x, y, z = 12, 34, 56
 
-    run(f'python basicmath.py add {x} {y} {z}', shell=True)
+    run(f'python {datadir/"basicmath.py"} add {x} {y} {z}', shell=True)
     assert capfd.readouterr().err.splitlines()[-1].endswith(error.format(args=z))
 
     x, y, z = 12, 34, '-f'
 
-    run(f'python basicmath.py add {x} {y} {z}', shell=True)
+    run(f'python {datadir/"basicmath.py"} add {x} {y} {z}', shell=True)
     assert capfd.readouterr().err.splitlines()[-1].endswith(error.format(args=z))
 
     x, y, z, q = 12, 34, '--flag', 'value'
 
-    run(f'python basicmath.py add {x} {y} {z} {q}', shell=True)
+    run(f'python {datadir/"basicmath.py"} add {x} {y} {z} {q}', shell=True)
     assert capfd.readouterr().err.splitlines()[-1].endswith(error.format(args=f'{z} {q}'))
 
 
-def test_open(capfd):
-    filename = 'numbers.txt'
+def test_open(capfd, datadir):
+    filename = datadir / 'numbers.txt'
     with open(filename) as file:
         numbers = (float(line) for line in file.readlines())
 
-    run('python basicmath.py sumfile numbers.txt', shell=True)
+    run(f'python {datadir/"basicmath.py"} sumfile {filename}', shell=True)
     assert float(capfd.readouterr().out) == sum(numbers)
 
 
-def test_aliases(capfd):
+def test_aliases(capfd, datadir):
     x, y = 12, 34
 
-    run(f'python basicmath.py sum {x} {y}', shell=True)
+    run(f'python {datadir/"basicmath.py"} sum {x} {y}', shell=True)
     assert int(capfd.readouterr().out) == x + y
 
-    run(f'python basicmath.py plus {x} {y}', shell=True)
+    run(f'python {datadir/"basicmath.py"} plus {x} {y}', shell=True)
     assert int(capfd.readouterr().out) == x + y
 
 
-def test_help(capfd):
-    run('python basicmath.py --help', shell=True)
-    assert 'Basic math operations.' in capfd.readouterr().out
-
-    run('python basicmath.py -h', shell=True)
+def test_help(capfd, datadir):
+    run(f'python {datadir/"basicmath.py"} --help', shell=True)
     assert 'Basic math operations.' in capfd.readouterr().out
 
-    run('python noroot.py', shell=True)
+    run(f'python {datadir/"basicmath.py"} -h', shell=True)
     assert 'Basic math operations.' in capfd.readouterr().out
 
-    run('python basicmath.py add --help', shell=True)
+    run(f'python {datadir/"basicmath.py"} add --help', shell=True)
     help_message = capfd.readouterr().out
     assert 'Add two numbers.' in help_message
     assert 'First operand' in help_message
     assert 'Second operand' in help_message
 
-    run('python basicmath.py add -h', shell=True)
+    run(f'python {datadir/"basicmath.py"} add -h', shell=True)
     help_message = capfd.readouterr().out
     assert 'Add two numbers.' in help_message
     assert 'First operand' in help_message
     assert 'Second operand' in help_message
 
 
-def test_ignore(capfd):
+def test_ignore(capfd, datadir):
     n = 6
 
-    run(f'python basicmath.py calculate-factorial {n}', shell=True)
+    run(f'python {datadir/"basicmath.py"} calculate-factorial {n}', shell=True)
     assert "invalid choice: 'calculate-factorial'" in capfd.readouterr().err.splitlines()[-1]
 
 
-def test_pseudonym(capfd):
+def test_pseudonym(capfd, datadir):
     n = 6
 
-    run(f'python basicmath.py fac 6', shell=True)
+    run(f'python {datadir/"basicmath.py"} fac 6', shell=True)
     assert int(capfd.readouterr().out) == factorial(n)
 
 
-def test_arg_map(capfd):
+def test_arg_map(capfd, datadir):
     x, base = 1000, 10
 
-    run(f'python basicmath.py log {x} --to {base}', shell=True)
+    run(f'python {datadir/"basicmath.py"} log {x} --to {base}', shell=True)
     assert float(capfd.readouterr().out) == log(x, base)
 
-    run(f'python basicmath.py log {x} -t {base}', shell=True)
+    run(f'python {datadir/"basicmath.py"} log {x} -t {base}', shell=True)
     assert float(capfd.readouterr().out) == log(x, base)
 
 
-def test_metavars(capfd):
-    run('python basicmath.py log --help', shell=True)
+def test_metavars(capfd, datadir):
+    run(f'python {datadir/"basicmath.py"} log --help', shell=True)
     assert '-t BASE, --to BASE' in capfd.readouterr().out
 
-    run('python basicmath.py log -h', shell=True)
+    run(f'python {datadir/"basicmath.py"} log -h', shell=True)
     assert '-t BASE, --to BASE' in capfd.readouterr().out
 
 
@@ -205,7 +204,7 @@ def _(self):
     assert 'Command name cannot be empty' in str(excinfo.value)
 
 
-def test_str_arg(capfd):
+def test_str_arg(capfd, datadir):
     message = 'Hello Cliar'
-    run(f'python basicmath.py echo "{message}"', shell=True)
+    run(f'python {datadir/"basicmath.py"} echo "{message}"', shell=True)
     assert capfd.readouterr().out.strip() == message

basicmath.py tests/test_basicmath/basicmath.py

@@ -0,0 +1,6 @@
+from subprocess import run
+
+
+def test_help(capfd, datadir):
+    run(f'python {datadir/"noroot.py"}', shell=True)
+    assert 'Basic math operations.' in capfd.readouterr().out