Patchwork [U-Boot,v2,4/4] patman: Add settings to the list of modules to doctest

login
register
mail settings
Submitter Doug Anderson
Date Dec. 4, 2012, 12:43 a.m.
Message ID <1354581799-775-4-git-send-email-dianders@chromium.org>
Download mbox | patch
Permalink /patch/203518/
State Accepted
Commit 656cffeb49394f1cc935001f90337028e01472c5
Delegated to: Simon Glass
Headers show

Comments

Doug Anderson - Dec. 4, 2012, 12:43 a.m.
The settings modules now has doctests, so run them.

Signed-off-by: Doug Anderson <dianders@chromium.org>
---
Changes in v2: None

 tools/patman/patman.py |    5 +++--
 1 files changed, 3 insertions(+), 2 deletions(-)
Simon Glass - Jan. 31, 2013, 11:53 p.m.
On Tue, Dec 4, 2012 at 1:43 PM, Doug Anderson <dianders@chromium.org> wrote:
> The settings modules now has doctests, so run them.
>
> Signed-off-by: Doug Anderson <dianders@chromium.org>
> ---
> Changes in v2: None

Applied to u-boot-x86, thanks.

Patch

diff --git a/tools/patman/patman.py b/tools/patman/patman.py
index 2e9e5dc..e56dd01 100755
--- a/tools/patman/patman.py
+++ b/tools/patman/patman.py
@@ -85,8 +85,9 @@  if options.test:
     result = unittest.TestResult()
     suite.run(result)
 
-    suite = doctest.DocTestSuite('gitutil')
-    suite.run(result)
+    for module in ['gitutil', 'settings']:
+        suite = doctest.DocTestSuite(module)
+        suite.run(result)
 
     # TODO: Surely we can just 'print' result?
     print result