You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ambari.apache.org by Dmitro Lisnichenko <dl...@hortonworks.com> on 2014/04/10 19:06:35 UTC

Review Request 20222: failing testGenerateManifest, test_overwrite_repos

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20222/
-----------------------------------------------------------

Review request for Ambari and Andrew Onischuk.


Bugs: AMBARI-5428
    https://issues.apache.org/jira/browse/AMBARI-5428


Repository: ambari


Description
-------

Sometimes unittests fail (got 3 fails for 20 runs)
======================================================================
FAIL: testGenerateManifest (TestManifestGenerator.TestManifestGenerator)
----------------------------------------------------------------------
Traceback (most recent call last):
File "/home/user/ambari/ambari-common/src/test/python/mock/mock.py", line 1199, in patched
return func(*args, **keywargs)
File "/home/user/ambari/ambari-agent/src/test/python/ambari_agent/TestManifestGenerator.py", line 76, in testGenerateManifest
self.assertTrue(writeParamsMock.called)
AssertionError
======================================================================
FAIL: test_overwrite_repos (TestPuppetExecutor.TestPuppetExecutor)
----------------------------------------------------------------------
Traceback (most recent call last):
File "/home/user/ambari/ambari-common/src/test/python/mock/mock.py", line 1199, in patched
return func(*args, **keywargs)
File "/home/user/ambari/ambari-agent/src/test/python/ambari_agent/TestPuppetExecutor.py", line 142, in test_overwrite_repos
self.assertTrue(puppetInstance.reposInstalled)
AssertionError


Diffs
-----

  ambari-agent/src/test/python/ambari_agent/TestManifestGenerator.py 7e6ab81 
  ambari-agent/src/test/python/ambari_agent/TestPuppetExecutor.py c5c3bdc 

Diff: https://reviews.apache.org/r/20222/diff/


Testing
-------

Ran unit tests 20 times with no fails


Thanks,

Dmitro Lisnichenko


Re: Review Request 20222: failing testGenerateManifest, test_overwrite_repos

Posted by Andrew Onischuk <ao...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20222/#review40045
-----------------------------------------------------------

Ship it!


Ship It!

- Andrew Onischuk


On April 10, 2014, 5:06 p.m., Dmitro Lisnichenko wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20222/
> -----------------------------------------------------------
> 
> (Updated April 10, 2014, 5:06 p.m.)
> 
> 
> Review request for Ambari and Andrew Onischuk.
> 
> 
> Bugs: AMBARI-5428
>     https://issues.apache.org/jira/browse/AMBARI-5428
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Sometimes unittests fail (got 3 fails for 20 runs)
> ======================================================================
> FAIL: testGenerateManifest (TestManifestGenerator.TestManifestGenerator)
> ----------------------------------------------------------------------
> Traceback (most recent call last):
> File "/home/user/ambari/ambari-common/src/test/python/mock/mock.py", line 1199, in patched
> return func(*args, **keywargs)
> File "/home/user/ambari/ambari-agent/src/test/python/ambari_agent/TestManifestGenerator.py", line 76, in testGenerateManifest
> self.assertTrue(writeParamsMock.called)
> AssertionError
> ======================================================================
> FAIL: test_overwrite_repos (TestPuppetExecutor.TestPuppetExecutor)
> ----------------------------------------------------------------------
> Traceback (most recent call last):
> File "/home/user/ambari/ambari-common/src/test/python/mock/mock.py", line 1199, in patched
> return func(*args, **keywargs)
> File "/home/user/ambari/ambari-agent/src/test/python/ambari_agent/TestPuppetExecutor.py", line 142, in test_overwrite_repos
> self.assertTrue(puppetInstance.reposInstalled)
> AssertionError
> 
> 
> Diffs
> -----
> 
>   ambari-agent/src/test/python/ambari_agent/TestManifestGenerator.py 7e6ab81 
>   ambari-agent/src/test/python/ambari_agent/TestPuppetExecutor.py c5c3bdc 
> 
> Diff: https://reviews.apache.org/r/20222/diff/
> 
> 
> Testing
> -------
> 
> Ran unit tests 20 times with no fails
> 
> 
> Thanks,
> 
> Dmitro Lisnichenko
> 
>