summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTristan Van Berkom <tristan.vanberkom@codethink.co.uk>2017-09-03 23:02:47 -0400
committerTristan Van Berkom <tristan.vanberkom@codethink.co.uk>2017-09-03 23:04:56 -0400
commit9834da72329b6d2fc7d5db357a3c836e4710cab5 (patch)
tree60b91b11d6422ba40d3db812eec1ebdf67d1e51a
parent84c871b8a60f7826392c35a36735ac52df29ef74 (diff)
downloadbuildstream-9834da72329b6d2fc7d5db357a3c836e4710cab5.tar.gz
plugin.py: Removed special case for test cases
Turns out this doesnt work once you try to actually run the full buildstream in a test case (in foreground it screws with other parsing, in background it fails in the test context with a strange error).
-rw-r--r--buildstream/plugin.py4
-rw-r--r--setup.cfg2
-rwxr-xr-xsetup.py1
3 files changed, 1 insertions, 6 deletions
diff --git a/buildstream/plugin.py b/buildstream/plugin.py
index ae5f4018f..775215462 100644
--- a/buildstream/plugin.py
+++ b/buildstream/plugin.py
@@ -526,9 +526,7 @@ class Plugin():
#
@contextmanager
def _output_file(self):
- if 'BST_TEST_SUITE' in os.environ:
- yield sys.stdout
- elif not self.__log:
+ if not self.__log:
with open(os.devnull, "w") as output:
yield output
else:
diff --git a/setup.cfg b/setup.cfg
index 8950a0c1b..e5414a8b0 100644
--- a/setup.cfg
+++ b/setup.cfg
@@ -12,5 +12,3 @@ pep8ignore =
tmp/* ALL
*/lib/python3* ALL
*/bin/* ALL
-env =
- D:BST_TEST_SUITE=True
diff --git a/setup.py b/setup.py
index a13a06472..f66f76cc3 100755
--- a/setup.py
+++ b/setup.py
@@ -145,6 +145,5 @@ setup(name='BuildStream',
'pytest-datafiles',
'pytest-pep8',
'pytest-cov',
- 'pytest-env',
'pytest'],
zip_safe=False)