From 3bedbf466b65b2d7abd80594b7bdefdb0d441e75 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Cl=C3=A9ment=20Schreiner?= Date: Sun, 19 Aug 2012 01:34:21 +0200 Subject: [PATCH] Add a LintianSeverity for lintian-clean packages. + PEP8 compliance --- debexpo/plugins/qa/lintian.py | 17 +++++++++++++---- 1 file changed, 13 insertions(+), 4 deletions(-) diff --git a/debexpo/plugins/qa/lintian.py b/debexpo/plugins/qa/lintian.py index 54c3e7b5..569275ae 100644 --- a/debexpo/plugins/qa/lintian.py +++ b/debexpo/plugins/qa/lintian.py @@ -67,12 +67,16 @@ severities = { constants.PLUGIN_SEVERITY_INFO), 'X': LintianSeverity('Package has lintian experimental tags', 0, constants.PLUGIN_SEVERITY_INFO), + '': LintianSeverity('Package is lintian clean', -1, + constants.PLUGIN_SEVERITY_INFO), } + @test_result class LintianTest(PluginResult): """ Summary of the lintian results """ + # -1 -> package is clean max_lintian_severity = string_field('max_lintian_severity') severity = int_field('severity') @@ -159,11 +163,16 @@ class LintianPlugin(QAPlugin): + " ".join(override_comments) + ")") override_comments = [] for line in lintian_data: - self.new_result(LintianWarning, package=package, severity=severity, - tag=lintian_tag, data=line) + self.new_result(LintianWarning, package=package, + severity=severity, tag=lintian_tag, + data=line) + + if not lintian_severities: + max_lintian_severity = '' - max_lintian_severity = max(lintian_severities, - key=lambda s: severities[s].int) + else: + max_lintian_severity = max(lintian_severities, + key=lambda s: severities[s].int) severity = severities[max_lintian_severity].plugin_severity self.new_test_result(severity=severity, -- GitLab