GitHub user aledsage opened a pull request:

    https://github.com/apache/incubator-brooklyn/pull/264

    Fix various integration tests 20141021

    

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/aledsage/incubator-brooklyn 
fix/various-integration-tests-20141021

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-brooklyn/pull/264.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #264
    
----
commit 8faa247f077f3c7d043777f52a981880d3d132d7
Author: Aled Sage <[email protected]>
Date:   2014-10-21T19:10:50Z

    BrooklynNode test: bind to all nics
    
    - On AWS jenkins box, was binding to 127.0.0.1, but then trying to 
      connect to http://ip-10-107-4-137:8081 (which resolved to 
      ip-10-107-4-137/10.107.4.137).
    - Presumably works for me locally because in /etc/hosts I have the
      explicit entry "127.0.0.1 Aleds-MacBook-Pro.local"

commit dd98d2519df6ad2de7e89c175e6573be915c9b1d
Author: Aled Sage <[email protected]>
Date:   2014-10-21T19:13:10Z

    ActiveMQ test: fix expectation of jmx rmi url format
    
    - Not sure how this ever passed. Perhaps a difference between 
      Java 6 and Java 7? If so, should we accept both in the assertion,
      I wonder?

commit 589c936e11a7101fdba80ca7b1eb8158d0db82f1
Author: Aled Sage <[email protected]>
Date:   2014-10-21T21:57:24Z

    Fix BrooklynNode tests: on stop, entity auto-unmanaged
    
    - testStopButLeaveAppsEffector and testStopAndKillAppsEffector were
      failing because trying to use entity.driver.isRunning() on an
      unmanaged entity.
    - Instead, get the entity’s pid file and check that explicitly.

commit 4e3b899558d3c1808f49171958121f430b9406f2
Author: Aled Sage <[email protected]>
Date:   2014-10-21T22:15:44Z

    Fix failing RebindWebClusterDatabaseExampleAppIntegrationTest
    
    - app was not being managed
    - deleted unneccessary rebind() impl - just call super
    - removed race by adding “eventually” to assertHttpStatusCodeEquals

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to