[autotest] reenable django or simplejson requiring unit tests
For a refactor of control_type, a lot of the relevant unit test coverage
is currently blacklisted in utils/unittest_suite.py due to requiring
Django or simplejson.
This CL un-blacklists those tests. It also therefore
fixes a few tests which are failing or broken. In particular:
tko/rpc_interface_unittest was throwing
`DatabaseError: only a single result allowed for a SELECT that is
part of an expression` all over the place. I couldn't blacklist just
this test file, since it has the same filename as
afe/rpc_interface_unittest.py which we do not want to blacklist and with
the way blacklists work in utils/unittest_suite I couldn't blacklist
just one of them. Instead, I have renamed the broken test file to
rpc_interface_unittest_fixme.py. tko/resources_test imports this failing
module, so I had to rename that one too.
One test in resources_test was throwing the same excepiion as above, so
I commented it out. (test_keyval_filtering)
in monitor_db_unittest, the test
test_HostScheduler_get_host_atomic_group_id throws KeyErrors that seem
to be related to labels not being correctly set up or committed to the
test database in the test setup phase. After trying to blindly track
this down a bit, I realized I was in over my head and just commented out
this specific test).
monitor_db_functional_test was throwing
`DatabaseError: near "TRUNCATE": syntax error` all over the place, so I
blacklisted that test file in utils/unittest_suite.py
BUG=chromium:240643
TEST=utils/unittest_suite.py # All tests pass
Change-Id: I8fdbe048b04516548e96bd888ed74e9fc82a2d88
Reviewed-on: https://gerrit.chromium.org/gerrit/51190
Commit-Queue: Aviv Keshet <akeshet@chromium.org>
Reviewed-by: Aviv Keshet <akeshet@chromium.org>
Tested-by: Aviv Keshet <akeshet@chromium.org>
diff --git a/utils/unittest_suite.py b/utils/unittest_suite.py
index 9c9436c..32af377 100755
--- a/utils/unittest_suite.py
+++ b/utils/unittest_suite.py
@@ -87,13 +87,12 @@
'guest_test.py',
'ap_configurator_test.py',
'chaos_base_test.py',
- 'chaos_interop_test.py'
+ 'chaos_interop_test.py',
+ 'monitor_db_functional_test.py'
))
-LONG_TESTS = (REQUIRES_DJANGO |
- REQUIRES_MYSQLDB |
+LONG_TESTS = (REQUIRES_MYSQLDB |
REQUIRES_GWT |
- REQUIRES_SIMPLEJSON |
REQUIRES_HTTPLIB2 |
REQUIRES_AUTH |
REQUIRES_PROTOBUFS |