server_management_lib has no tests? |
||
Issue descriptionThe server management libraries broke because of a puppet directory rename. Allen found the that the problem is in: venv/server_management_lib/constants.KEY_FILENAME I updated that value locally and went looking for where to add a test to catch problems in the future before they are submitted. But I can't seem to find the unittests for the server management library. If they exist, can you add instructions for running them to the README? If not.... we need some. At least enough to catch syntax errors and to give us a place to add tests over time as things break. I can help make sure they are run on builders before changes are submitted.
,
Oct 18 2017
The following revision refers to this bug: https://chrome-internal.googlesource.com/chromeos/chromeos-admin/+/75279b81d218badddc9262dbe5cad7085770f293 commit 75279b81d218badddc9262dbe5cad7085770f293 Author: Don Garrett <dgarrett@google.com> Date: Wed Oct 18 03:49:10 2017
,
Apr 2 2018
|
||
►
Sign in to add a comment |
||
Comment 1 by dgarr...@chromium.org
, Oct 5 2017