diff options
author | Tristan Van Berkom <tristan.vanberkom@codethink.co.uk> | 2016-12-10 15:14:25 +0900 |
---|---|---|
committer | Tristan Van Berkom <tristan.vanberkom@codethink.co.uk> | 2016-12-10 15:14:25 +0900 |
commit | 02c8a6b1e2e032b7dd43d62057af611b72eea6b4 (patch) | |
tree | 57ae117e5c4d240e8905345be93747b2348329a7 /tests/context | |
parent | 7b1233ff1410bcf60cfa00ff1f12dabeab5b0634 (diff) | |
download | buildstream-02c8a6b1e2e032b7dd43d62057af611b72eea6b4.tar.gz |
pep8 fixes in context test case
Diffstat (limited to 'tests/context')
-rw-r--r-- | tests/context/context.py | 11 |
1 files changed, 9 insertions, 2 deletions
diff --git a/tests/context/context.py b/tests/context/context.py index ae4effad1..9864da8a8 100644 --- a/tests/context/context.py +++ b/tests/context/context.py @@ -9,14 +9,16 @@ DATA_DIR = os.path.join( 'data', ) + # Simple fixture to create a PluginBase object that # we use for loading plugins. @pytest.fixture() def context_fixture(): return { - 'context' : Context('x86_64') + 'context': Context('x86_64') } + ####################################### # Test instantiation # ####################################### @@ -25,6 +27,7 @@ def test_context_create(context_fixture): assert(isinstance(context, Context)) assert(context.arch == 'x86_64') + ####################################### # Test configuration loading # ####################################### @@ -39,6 +42,7 @@ def test_context_load(context_fixture): assert(context.artifactdir == '~/buildstream/artifacts') assert(context.ccachedir == '~/buildstream/ccache') + # Test that values in a user specified config file # override the defaults @pytest.mark.datafiles(os.path.join(DATA_DIR)) @@ -57,10 +61,10 @@ def test_context_load_user_config(context_fixture, datafiles): assert(context.artifactdir == '~/buildstream/artifacts') assert(context.ccachedir == '~/buildstream/ccache') + ####################################### # Test failure modes # ####################################### - @pytest.mark.datafiles(os.path.join(DATA_DIR)) def test_context_load_missing_config(context_fixture, datafiles): context = context_fixture['context'] @@ -75,6 +79,7 @@ def test_context_load_missing_config(context_fixture, datafiles): assert (exc.value.reason == LoadErrorReason.MISSING_FILE) + @pytest.mark.datafiles(os.path.join(DATA_DIR)) def test_context_load_malformed_config(context_fixture, datafiles): context = context_fixture['context'] @@ -89,6 +94,7 @@ def test_context_load_malformed_config(context_fixture, datafiles): assert (exc.value.reason == LoadErrorReason.INVALID_YAML) + @pytest.mark.datafiles(os.path.join(DATA_DIR)) def test_context_load_notdict_config(context_fixture, datafiles): context = context_fixture['context'] @@ -104,6 +110,7 @@ def test_context_load_notdict_config(context_fixture, datafiles): # XXX Should this be a different LoadErrorReason ? assert (exc.value.reason == LoadErrorReason.INVALID_YAML) + @pytest.mark.datafiles(os.path.join(DATA_DIR)) def test_context_load_invalid_type(context_fixture, datafiles): context = context_fixture['context'] |