From 90e6893b967cded4bf45380d643d3bfcb19ce0d4 Mon Sep 17 00:00:00 2001 From: Yujun Zhang Date: Tue, 18 Apr 2017 18:57:14 +0800 Subject: Set default width to 80 for justify filter - 80 default screen width for most consoles - template content should not be part of console reporter test - remove fix width splitter in console report templates Change-Id: Ib060b447bd52c211a4bf1e4ded2f898a9201b6d8 Signed-off-by: Yujun Zhang --- tests/unit/cli/cmd_report_test.py | 66 +++++++------------------------------ tests/unit/reporter/console_test.py | 66 +++++++------------------------------ 2 files changed, 22 insertions(+), 110 deletions(-) (limited to 'tests/unit') diff --git a/tests/unit/cli/cmd_report_test.py b/tests/unit/cli/cmd_report_test.py index 21e757d8..064ed8fd 100644 --- a/tests/unit/cli/cmd_report_test.py +++ b/tests/unit/cli/cmd_report_test.py @@ -26,60 +26,16 @@ def result_path(): return result -def test_dhrystone(runner, result_path): +@pytest.mark.parametrize(['report_name'], [ + ('dhrystone',), + ('whetstone',), + ('dpi',), + ('ramspeed',), + ('ssl',) +]) +def test_dhrystone(report_name, runner, result_path): """Test dhrystone report""" - result = runner.invoke(cli, ['report', 'show', 'dhrystone', '-p', result_path]) - assert "Benchmark..............................................................." \ - "...................dhrystone" in result.output - assert "Score..................................................................." \ - "..................63529.6" - - -def test_whetstone(runner, result_path): - """ Test whetstone output""" - - result = runner.invoke(cli, ['report', 'show', 'whetstone', '-p', result_path]) - assert "Number................................................................." \ - "...........................40" in result.output - assert "Total CPUs............................................................." \ - "...........................40" - - -def test_dpi(runner, result_path): - """ Test dpi report""" - result = runner.invoke(cli, ['report', 'show', 'dpi', '-p', result_path]) - assert "Bits per Second..................................................." \ - ".............................3.638" in result.output - assert "Packets per Second................................................" \ - ".............................1.458" in result.output - - -def test_ramspeed(runner, result_path): - """ Test ramspeed report """ - result = runner.invoke(cli, ['report', 'show', 'ramspeed', '-p', result_path]) - assert "Float Addition........................................................" \ - "......................10522.33" in result.output - assert "Integer Average........................................................" \ - ".....................11466.52" in result.output - - -def test_ssl(runner, result_path): - """ Test ssl report""" - - result = runner.invoke(cli, ['report', 'show', 'ssl', '-p', result_path]) - assert "8192............................................................." \ - ".........................667303.94k" in result.output - assert "1024.............................................................." \ - "..........................100088.3" in result.output - - -def test_sys(runner, result_path): - """ Test sys_info """ - - result = runner.invoke(cli, ['report', 'show', 'ssl', '-p', result_path]) - assert "System Information" in result.output - assert "Host Name........................................................" \ - ".................node-38.zte.com.cn" in result.output - assert "Operating System.................................................." \ - "...............Ubuntu 16.04 xenial" in result.output + result = runner.invoke(cli, ['report', 'show', report_name, '-p', result_path]) + for line in str(result).split('\n'): + assert len(line) <= 80 diff --git a/tests/unit/reporter/console_test.py b/tests/unit/reporter/console_test.py index 7ece4663..11fb3f71 100644 --- a/tests/unit/reporter/console_test.py +++ b/tests/unit/reporter/console_test.py @@ -29,60 +29,16 @@ def test_constructor(console_reporter): assert isinstance(console_reporter, ConsoleReporter) -def test_dhrystone(console_reporter, result_path): +@pytest.mark.parametrize(['template_name'], [ + ('dhrystone',), + ('whetstone',), + ('dpi',), + ('ramspeed',), + ('ssl',) +]) +def test_templates(template_name, console_reporter, result_path): """ Test dhrystone report""" - result = console_reporter.render('dhrystone', result_path) - assert "Benchmark..............................................................." \ - "...................dhrystone" in result - - -def test_whetstone(console_reporter, result_path): - """ Test whetstone output""" - - result = console_reporter.render('whetstone', result_path) - assert "Number................................................................." \ - "...........................40" in result - assert "Total CPUs............................................................." \ - "...........................40" in result - - -def test_dpi(console_reporter, result_path): - """ Test dpi report""" - - result = console_reporter.render('dpi', result_path) - assert "Bits per Second..................................................." \ - ".............................3.638" in result - assert "Packets per Second................................................" \ - ".............................1.458" in result - - -def test_ramspeed(console_reporter, result_path): - """ Test ramspeed report """ - - result = console_reporter.render('ramspeed', result_path) - assert "Float Addition........................................................" \ - "......................10522.33" in result - assert "Integer Average........................................................" \ - ".....................11466.52" in result - - -def test_ssl(console_reporter, result_path): - """ Test ssl report""" - - result = console_reporter.render('ssl', result_path) - assert "8192............................................................." \ - ".........................667303.94k" in result - assert "1024.............................................................." \ - "..........................100088.3" in result - - -def test_sys(console_reporter, result_path): - """ Test sys_info """ - - result = console_reporter.render('ssl', result_path) - assert "System Information" in result - assert "Host Name........................................................" \ - ".................node-38.zte.com.cn" in result - assert "Operating System.................................................." \ - "...............Ubuntu 16.04 xenial" in result + result = console_reporter.render(template_name, result_path) + for line in result.split('\n'): + assert len(line) <= 80 -- cgit 1.2.3-korg