• Greg Thompson's avatar
    Improve reg cleanup in InstallServiceWorkItemTest and run it serially. · debdcfb3
    Greg Thompson authored
    The test was leaking an empty key on TearDown. With this change, it
    deletes the key if it was created in SetUp and also deletes an empty key
    left behind by previous test runs.
    
    Additinally, tests from this harness must run serially since they all
    interact with the ServiceControlManager and centralized machine
    state. As such, build and run them into setup_unittests rather than
    installer_util_unittests.
    
    This change also introduces a use of ScopedInstallDetails to configure
    state for system-level operations (the only case in which the code under
    test will run).
    
    BUG=1059314
    
    Change-Id: I6f3a35cda45704e9195dfae25a90401fb8915c6f
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2095540
    Auto-Submit: Greg Thompson <grt@chromium.org>
    Reviewed-by: default avatarS. Ganesh <ganesh@chromium.org>
    Commit-Queue: S. Ganesh <ganesh@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#748804}
    debdcfb3
BUILD.gn 4.7 KB