Skip to content

Commit

Permalink
Merge pull request #270 from a16bitsysop/tests
Browse files Browse the repository at this point in the history
test for nodejs or node in tests
  • Loading branch information
ekalinin authored Aug 29, 2020
2 parents 95c500d + 766f9e3 commit 797c354
Showing 1 changed file with 21 additions and 10 deletions.
31 changes: 21 additions & 10 deletions tests/nodeenv_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,25 @@
HERE = os.path.abspath(os.path.dirname(__file__))


if subprocess.run(["which", "nodejs"],capture_output=True).returncode == 0:
is_nodejs = True
else:
is_nodejs = False


def call_nodejs(ev_path):
assert os.path.exists(ev_path)
activate = pipes.quote(os.path.join(ev_path, 'bin', 'activate'))
if is_nodejs:
subprocess.check_call([
'sh', '-c', '. {} && nodejs --version'.format(activate),
])
else:
subprocess.check_call([
'sh', '-c', '. {} && node --version'.format(activate),
])


@pytest.mark.integration
def test_smoke(tmpdir):
nenv_path = tmpdir.join('nenv').strpath
Expand All @@ -23,11 +42,7 @@ def test_smoke(tmpdir):
'coverage', 'run', '-p',
'-m', 'nodeenv', '--prebuilt', nenv_path,
])
assert os.path.exists(nenv_path)
activate = pipes.quote(os.path.join(nenv_path, 'bin', 'activate'))
subprocess.check_call([
'sh', '-c', '. {} && nodejs --version'.format(activate),
])
call_nodejs(nenv_path)


@pytest.mark.integration
Expand All @@ -38,11 +53,7 @@ def test_smoke_n_system_special_chars(tmpdir):
'coverage', 'run', '-p',
'-m', 'nodeenv', '-n', 'system', nenv_path,
))
assert os.path.exists(nenv_path)
activate = pipes.quote(os.path.join(nenv_path, 'bin', 'activate'))
subprocess.check_call([
'sh', '-c', '. {} && nodejs --version'.format(activate),
])
call_nodejs(nenv_path)


@pytest.yield_fixture
Expand Down

0 comments on commit 797c354

Please sign in to comment.