~registry/chromium-os-update-engine/trunk

Viewing all changes in revision 450.

  • Committer: ChromeBot
  • Author(s): Gilad Arnold
  • Date: 2013-07-31 20:16:00 UTC
  • Revision ID: git-v1:70e476ed0a09f8ca2d0a9e3f0e1cd78ccefd5fbe
AU/unittest: allow use of a parametric update completed marker

This should allow unit tests to be run in parallel. We introduce
a private constructor available only to specific test classes of
UpdateAttempter, which allows us to set a different marker.

Note that, in all unit tests that do not specifically make use of this
marker, we actually want UpdateAttempter to ignore it entirely. I'm not
entirely positive whether not ignoring would lead to interferences
between concurrently run tests, but I think it's better to err on the
safe side.  (The marker is never ignored when used with any code other
than UpdateAttempter unit testing code.)

BUG=chromium:236465
TEST=Unit tests pass incl ignore semantics when not needed

Change-Id: I30fbed2ae2c21368d79127ed44811007e2a66e77
Reviewed-on: https://gerrit.chromium.org/gerrit/63840
Tested-by: Gilad Arnold <garnold@chromium.org>
Reviewed-by: Alex Deymo <deymo@chromium.org>
Commit-Queue: Gilad Arnold <garnold@chromium.org>

expand all expand all

Show diffs side-by-side

added added

removed removed

Lines of Context: