[Asterisk-code-review] runtests: Relabel the dry-run option from '-n' to '-d' (testsuite[master])

Anonymous Coward asteriskteam at digium.com
Sun Oct 25 10:11:55 CDT 2015


Anonymous Coward #1000019 has submitted this change and it was merged.

Change subject: runtests: Relabel the dry-run option from '-n' to '-d'
......................................................................


runtests: Relabel the dry-run option from '-n' to '-d'

This patch relabels the dry-run option from '-n' to '-d'. This is being
done for two reasons:
(1) -n doesn't go as well with --dry-run as -d
(2) This free's up -n for use with a later patch

Change-Id: Ifb6daa6710782aaa59f6ad7364d2e0b1e8620e38
---
M runtests.py
1 file changed, 3 insertions(+), 3 deletions(-)

Approvals:
  Anonymous Coward #1000019: Verified
  Matt Jordan: Looks good to me, approved
  Joshua Colp: Looks good to me, but someone else must approve



diff --git a/runtests.py b/runtests.py
index 4f84595..82744b1 100755
--- a/runtests.py
+++ b/runtests.py
@@ -605,6 +605,9 @@
     parser.add_option("-c", "--cleanup", action="store_true",
                       dest="cleanup", default=False,
                       help="Cleanup tmp directory after each successful test")
+    parser.add_option("-d", "--dry-run", action="store_true",
+                      dest="dry_run", default=False,
+                      help="Only show which tests would be run.")
     parser.add_option("-g", "--tag", action="append",
                       dest="tags",
                       help="Specify one or more tags to select a subset of tests.")
@@ -617,9 +620,6 @@
     parser.add_option("-L", "--list-tags", action="store_true",
                       dest="list_tags", default=False,
                       help="List available tags")
-    parser.add_option("-n", "--dry-run", action="store_true",
-                      dest="dry_run", default=False,
-                      help="Only show which tests would be run.")
     parser.add_option("-t", "--test", action="append", default=[],
                       dest="tests",
                       help=("Run a single specified test (directory) instead "

-- 
To view, visit https://gerrit.asterisk.org/1527
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ifb6daa6710782aaa59f6ad7364d2e0b1e8620e38
Gerrit-PatchSet: 2
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Matt Jordan <mjordan at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>



More information about the asterisk-code-review mailing list