./test doesn't work in landscape-client

Bug #381613 reported by Andreas Hasenack
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Landscape Client
Fix Released
Medium
Gustavo Niemeyer

Bug Description

Just doesn't work, failing with several dbus errors.

[landscape/tests/test_configuration.py]
...............................................................................Error occurred contacting Landscape Client. Is it running?
EError occurred contacting Landscape Client. Is it running?
EError occurred contacting Landscape Client. Is it running?
Traceback (most recent call last):
  File "./test", line 166, in <module>
    runner_func()
  File "./test", line 130, in test_with_unittest
    result = runner.run(test)
  File "/usr/lib/python2.5/unittest.py", line 705, in run
    test(result)
  File "/usr/lib/python2.5/unittest.py", line 437, in __call__
    return self.run(*args, **kwds)
  File "/usr/lib/python2.5/unittest.py", line 433, in run
    test(result)
  File "/usr/lib/python2.5/unittest.py", line 437, in __call__
    return self.run(*args, **kwds)
  File "/usr/lib/python2.5/unittest.py", line 433, in run
    test(result)
  File "/usr/lib/python2.5/site-packages/twisted/trial/unittest.py", line 632, in __call__
    return self.run(*args, **kwargs)
  File "/home/andreas/canonical/landscape-client/bound/trunk/landscape/tests/mocker.py", line 134, in run_wrapper
    return run_method(*args, **kwargs)
  File "/home/andreas/canonical/landscape-client/bound/trunk/landscape/tests/helpers.py", line 126, in run
    run_isolated(LandscapeTest, self, result)
  File "/home/andreas/canonical/landscape-client/bound/trunk/landscape/tests/subunit.py", line 359, in run_isolated
    klass.run(self, result)
  File "/usr/lib/python2.5/site-packages/twisted/trial/unittest.py", line 958, in run
    self._wait(d)
  File "/usr/lib/python2.5/site-packages/twisted/trial/unittest.py", line 1124, in _wait
    raise KeyboardInterrupt()
KeyboardInterrupt
Error in sys.exitfunc:
E
======================================================================
ERROR: test_register_failure
----------------------------------------------------------------------
RemoteException: TimeoutError: <landscape.tests.test_configuration.RegisterFunctionTest testMethod=test_register_failure> (test_register_failure) still running at 120.0 secs

======================================================================
ERROR: test_register_success
----------------------------------------------------------------------
RemoteException: TimeoutError: <landscape.tests.test_configuration.RegisterFunctionTest testMethod=test_register_success> (test_register_success) still running at 120.0 secs

======================================================================
ERROR: test_register_timeout_failure
----------------------------------------------------------------------
RemoteException: DirtyReactorAggregateError: Reactor was unclean.
DelayedCalls: (set twisted.internet.base.DelayedCall.debug = True to debug)
<DelayedCall 162274316 [119.893764019s] called=0 cancelled=0 onTimeout(<Deferred at 0x9ac124c>)>

----------------------------------------------------------------------
Ran 82 tests in 246.049s

FAILED (errors=3)
(tests=82, failures=0, errors=3)

In the meantime, what should I use to successfully run the client test suite?

Changed in landscape:
importance: Undecided → Medium
milestone: none → 1.3.2
affects: landscape → landscape-client
Changed in landscape-client:
milestone: 1.3.2 → none
visibility: private → public
Changed in landscape-client:
milestone: none → 1.3.2
Revision history for this message
Gustavo Niemeyer (niemeyer) wrote :

The attached branch replaces ./test by a simple wrapper for trial.

tags: added: review
Changed in landscape-client:
assignee: nobody → Gustavo Niemeyer (niemeyer)
status: New → In Progress
Revision history for this message
Jamu Kakar (jkakar) wrote :

Looks great, all tests pass here! +1!

Revision history for this message
Andreas Hasenack (ahasenack) wrote :

./test works fine here, qa + 1

Revision history for this message
Thomas Herve (therve) wrote :

Cool, +1!

tags: removed: review
Revision history for this message
Gustavo Niemeyer (niemeyer) wrote :

That's been committed to trunk in revision 122.

Changed in landscape-client:
status: In Progress → Fix Committed
Jamu Kakar (jkakar)
tags: added: needs-testing
Revision history for this message
Free Ekanayaka (free.ekanayaka) wrote :

Works fine on staging, qa +1.

tags: removed: needs-testing
Changed in landscape-client:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.