Skip to content

Commit

Permalink
Fix-up and re-enable regression test for issue #226 (#2703)
Browse files Browse the repository at this point in the history
  • Loading branch information
benclifford authored May 23, 2023
1 parent e241fd0 commit 3e66d9a
Showing 1 changed file with 3 additions and 12 deletions.
15 changes: 3 additions & 12 deletions parsl/tests/test_regression/test_226.py
Original file line number Diff line number Diff line change
Expand Up @@ -7,9 +7,6 @@
import pytest

from parsl.app.app import bash_app, python_app
from parsl.tests.configs.local_threads import config

local_config = config


class Foo:
Expand Down Expand Up @@ -46,26 +43,20 @@ def echo(msg, postfix='there', stdout='std.out'):
return 'echo {} {}'.format(msg, postfix)


blacklist = os.path.join(os.path.dirname(os.path.dirname(__file__)), 'configs', '*ipp*')


# @pytest.mark.blacklist(blacklist, reason='hangs on Travis')
@pytest.mark.skip("Broke somewhere between PR #525 and PR #652")
def test_no_eq():
res = get_foo_x('foo').result()
assert res == 1, 'Expected 1, returned {}'.format(res)


# @pytest.mark.blacklist(blacklist, reason='hangs on Travis')
@pytest.mark.skip("Broke somewhere between PR #525 and PR #652")
def test_get_dataframe():
res = get_dataframe().result()
assert res.equals(data), 'Unexpected dataframe'


# @pytest.mark.blacklist(blacklist, reason='hangs on Travis')
@pytest.mark.skip("Broke somewhere between PR #525 and PR #652")
def test_bash_default_arg():
if os.path.exists('std.out'):
os.remove('std.out')

echo('hello').result()
with open('std.out', 'r') as f:
assert f.read().strip() == 'hello there', "Output should be 'hello there'"

0 comments on commit 3e66d9a

Please sign in to comment.