commit | b71e7e2a602008a7ac9611737bcf9a795c673e41 | [log] [tgz] |
---|---|---|
author | Scott Lee <ddoman@chromium.org> | Fri May 17 01:23:58 2019 |
committer | Commit Bot <commit-bot@chromium.org> | Fri May 17 01:23:58 2019 |
tree | 6aa90fb69dd18c186c4dc825adfa56cf5684fadb | |
parent | 1f50d1ed09e38683f0d05f8d5f8e7d7229d4bf95 [diff] |
Access Dispatcher via context so that each test can install own dispatcher. The flaky failure was because the dispatcher object was a global variable, and therefore, it was unintentionally shared between tests. Test in assingee_test.go and issue_test.go didn't even invoke registerTaskHandlers(), but it was OK, as long as backend_test.go gets executed before assignee_test.go and issue_test.go. Tests in assignee_test.go and issue_test.go used accidentally the dispatcher set by backend_test.go. Interestingly, it seems that go-test runs all the tests within a single process with the same order repeatedly, at least in my workstation. Thus, repeatedly executing go-test over 1000 times doesn't produce the error in my workstation, but does randomly in some other bots at a high rate. I ran CQ runs multiple times to ensure that the test results are stable. R: tandrii Bug: 963695 Change-Id: Id66707d2a03a34746faccf1bfb46263c113c60a1 Reviewed-on: https://chromium-review.googlesource.com/c/infra/infra/+/1614598 Commit-Queue: Scott Lee <ddoman@chromium.org> Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org> Cr-Commit-Position: refs/heads/master@{#23023}
Welcome to the Chrome Infra repository!
Wondering where to start? Check out General Chrome Infrastructure documentation. In particular, to check out this repo and the rest of the infrastructure code, follow the instructions here. The rest of this page is specific to this repo.
sys.path
modifications.from infra.path_hacks.common import <stg>
is actually getting <stg>
from build/scripts/common.git push origin <updated hash>:deployed
git push
commandIf you've added a new module, integrate your tests with test.py:
Double-check that your tests are getting picked up when you want them to be: ./test.py test <path-to-package>
.
Tests still not getting picked up by test.py? Double-check to make sure you have init.py files in each directory of your module so Python recognizes it as a package.