Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1139)

Unified Diff: au_test_harness/au_test.py

Issue 6801057: Simplify long tests to only test target image rather than back to base image. (Closed) Base URL: http://git.chromium.org/git/crostestutils.git@master
Patch Set: Nya Created 9 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: au_test_harness/au_test.py
diff --git a/au_test_harness/au_test.py b/au_test_harness/au_test.py
index cc161e042dc3b32ef923e5dd2ba50b1f04c32fe3..5a6549e1acd52fdc62716c5bc76786ba6c2f85e9 100644
--- a/au_test_harness/au_test.py
+++ b/au_test_harness/au_test.py
@@ -132,20 +132,15 @@ class AUTest(unittest.TestCase):
rather than wiping it.
"""
self.worker.InitializeResultsDirectory()
- # Just make sure some tests pass on original image. Some old images
- # don't pass many tests.
self.worker.PrepareBase(self.base_image_path)
- # TODO(sosa): move to 100% once we start testing using the autotest paired
- # with the dev channel.
- percent_passed = self.worker.VerifyImage(self, 10)
- # Update to - all tests should pass on new image.
+ # Update to
self.worker.PerformUpdate(self.target_image_path, self.base_image_path)
- percent_passed = self.worker.VerifyImage(self)
+ self.worker.VerifyImage(self)
- # Update from - same percentage should pass that originally passed.
- self.worker.PerformUpdate(self.base_image_path, self.target_image_path)
- self.worker.VerifyImage(self, percent_passed)
+ # Update from
+ self.worker.PerformUpdate(self.target_image_path, self.target_image_path)
+ self.worker.VerifyImage(self)
def testUpdateWipeStateful(self):
"""Tests if we can update after cleaning the stateful partition.
@@ -154,20 +149,17 @@ class AUTest(unittest.TestCase):
stateful partition.
"""
self.worker.InitializeResultsDirectory()
- # Just make sure some tests pass on original image. Some old images
- # don't pass many tests.
self.worker.PrepareBase(self.base_image_path)
- percent_passed = self.worker.VerifyImage(self, 10)
- # Update to - all tests should pass on new image.
+ # Update to
self.worker.PerformUpdate(self.target_image_path, self.base_image_path,
'clean')
self.worker.VerifyImage(self)
- # Update from - same percentage should pass that originally passed.
- self.worker.PerformUpdate(self.base_image_path, self.target_image_path,
+ # Update from
+ self.worker.PerformUpdate(self.target_image_path, self.target_image_path,
'clean')
- self.worker.VerifyImage(self, percent_passed)
+ self.worker.VerifyImage(self)
def testInterruptedUpdate(self):
"""Tests what happens if we interrupt payload delivery 3 times."""
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698