From 89e8af70d6dac5243965074060ffa69c559811fc Mon Sep 17 00:00:00 2001
From: Iustin Pop <iustin@google.com>
Date: Thu, 18 Nov 2010 10:23:48 +0100
Subject: [PATCH] QA: check that doubly modifying an OS state is OK
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
This would have prevented the bug fixed in the previous patch :(
Signed-off-by: Iustin Pop <iustin@google.com>
Reviewed-by: RenΓ© Nussbaumer <rn@google.com>
---
qa/qa_os.py | 3 +++
1 file changed, 3 insertions(+)
diff --git a/qa/qa_os.py b/qa/qa_os.py
index 143fedc53..e6e259aad 100644
--- a/qa/qa_os.py
+++ b/qa/qa_os.py
@@ -86,6 +86,9 @@ def _TestOsStates():
new_cmd = cmd + ["--%s" % param, val, _TEMP_OS_NAME]
AssertEqual(StartSSH(master["primary"],
utils.ShellQuoteArgs(new_cmd)).wait(), 0)
+ # check that double-running the command is OK
+ AssertEqual(StartSSH(master["primary"],
+ utils.ShellQuoteArgs(new_cmd)).wait(), 0)
def _SetupTempOs(node, dir, valid):
--
GitLab