From 4bfb52397424dfafc4c819a1b542bc7fadd56c19 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Marek=20Marczykowski-G=C3=B3recki?= Date: Sat, 3 Sep 2016 02:18:56 +0200 Subject: [PATCH] tests: add context manager to catch stdout This will avoid code duplication in tools tests. --- qubes/tests/tools/__init__.py | 43 ++++++++++++++++++++++++++++++++++- qubes/tools/__init__.py | 13 ++++++++--- 2 files changed, 52 insertions(+), 4 deletions(-) diff --git a/qubes/tests/tools/__init__.py b/qubes/tests/tools/__init__.py index 388083ed..cae5797e 100644 --- a/qubes/tests/tools/__init__.py +++ b/qubes/tests/tools/__init__.py @@ -1 +1,42 @@ -# pylint: skip-file +#!/usr/bin/python2 -O +# vim: fileencoding=utf-8 + +# +# The Qubes OS Project, https://www.qubes-os.org/ +# +# Copyright (C) 2015 Marek Marczykowski-Górecki +# +# +# This program is free software; you can redistribute it and/or modify +# it under the terms of the GNU General Public License as published by +# the Free Software Foundation; either version 2 of the License, or +# (at your option) any later version. +# +# This program is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. +# +# You should have received a copy of the GNU General Public License along +# with this program; if not, write to the Free Software Foundation, Inc., +# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. + + +import sys +try: + import StringIO +except ImportError: + from io import StringIO + + +class StdoutBuffer(object): + def __init__(self): + self.stdout = StringIO.StringIO() + + def __enter__(self): + sys.stdout = self.stdout + return self.stdout + + def __exit__(self, exc_type, exc_val, exc_tb): + sys.stdout = sys.__stdout__ + return False diff --git a/qubes/tools/__init__.py b/qubes/tools/__init__.py index f26fb3f9..43843480 100644 --- a/qubes/tools/__init__.py +++ b/qubes/tools/__init__.py @@ -525,6 +525,13 @@ def print_table(table): cmd = ['column', '-t', '-s', unit_separator] text_table = '\n'.join([unit_separator.join(row) for row in table]) - p = subprocess.Popen(cmd, stdin=subprocess.PIPE) - p.stdin.write(text_table) - p.communicate() + # for tests... + if sys.stdout != sys.__stdout__: + p = subprocess.Popen(cmd + ['-c', '80'], stdin=subprocess.PIPE, + stdout=subprocess.PIPE) + p.stdin.write(text_table) + (out, _) = p.communicate() + sys.stdout.write(out) + else: + p = subprocess.Popen(cmd, stdin=subprocess.PIPE) + p.communicate(text_table)