You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@brooklyn.apache.org by aledsage <gi...@git.apache.org> on 2016/09/23 17:23:50 UTC

[GitHub] brooklyn-server pull request #349: Fix or disable integration tests

GitHub user aledsage opened a pull request:

    https://github.com/apache/brooklyn-server/pull/349

    Fix or disable integration tests

    See individual commits for further description (copy-pasted here for your convenience):
    
    ```
    commit 384de14a710e1bbbccebd83a0f61ab158f29f116
    Author: Aled Sage <al...@gmail.com>
    Date:   Fri Sep 23 18:18:10 2016 +0100
    
        Re-write CreateUserPolicyTest to use RecordingSshTool
        
        Previously it use its own RecordingSshMachineLocation, so changing it
        to use our more standard mechanism.
    
    commit ab5d7b84940a7802bf807e590c1ccf1c6808c05a
    Author: Aled Sage <al...@gmail.com>
    Date:   Fri Sep 23 18:17:10 2016 +0100
    
        Fix vanilla-software-process-with-resource.yaml
        
        Typo of \u201ctempalte\u201d broke integration test, which broke
        org.apache.brooklyn.rest.test.entity.VanillaSoftwareProcessTest
    
    commit 42a547913e86b1e8a1b4d1de2cf9f68df0f5dc4d
    Author: Aled Sage <al...@gmail.com>
    Date:   Fri Sep 23 18:15:13 2016 +0100
    
        Mark blueprint integration tests as \u201cBroken\u201d
        
        The brooklyn-server repo does not include entities in brooklyn-library,
        such as MongoDB, so cannot be tested here.
        
        It should really be deleted (once we move them to a better home such
        as use of the test-framework, with QA running regularly against the
        catalog).
    ```

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

    $ git pull https://github.com/aledsage/brooklyn-server fix-or-disable-integration-tests

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

    https://github.com/apache/brooklyn-server/pull/349.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 #349
    
----
commit 42a547913e86b1e8a1b4d1de2cf9f68df0f5dc4d
Author: Aled Sage <al...@gmail.com>
Date:   2016-09-23T17:15:13Z

    Mark blueprint integration tests as \u201cBroken\u201d
    
    The brooklyn-server repo does not include entities in brooklyn-library,
    such as MongoDB, so cannot be tested here.
    
    It should really be deleted (once we move them to a better home such
    as use of the test-framework, with QA running regularly against the
    catalog).

commit ab5d7b84940a7802bf807e590c1ccf1c6808c05a
Author: Aled Sage <al...@gmail.com>
Date:   2016-09-23T17:17:10Z

    Fix vanilla-software-process-with-resource.yaml
    
    Typo of \u201ctempalte\u201d broke integration test, which broke
    org.apache.brooklyn.rest.test.entity.VanillaSoftwareProcessTest

commit 384de14a710e1bbbccebd83a0f61ab158f29f116
Author: Aled Sage <al...@gmail.com>
Date:   2016-09-23T17:18:10Z

    Re-write CreateUserPolicyTest to use RecordingSshTool
    
    Previously it use its own RecordingSshMachineLocation, so changing it
    to use our more standard mechanism.

----


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] brooklyn-server pull request #349: Fix or disable integration tests

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/brooklyn-server/pull/349


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] brooklyn-server issue #349: Fix or disable integration tests

Posted by aledsage <gi...@git.apache.org>.
Github user aledsage commented on the issue:

    https://github.com/apache/brooklyn-server/pull/349
  
    This only touches tests, so merging now.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---