You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by rmetzger <gi...@git.apache.org> on 2016/06/22 15:45:17 UTC

[GitHub] flink pull request #2148: [FLINK-3995] [build] Make test utils properly avai...

GitHub user rmetzger opened a pull request:

    https://github.com/apache/flink/pull/2148

    [FLINK-3995] [build] Make test utils properly available and fix Maven dependencies

    This pull request subsumes https://github.com/apache/flink/pull/2092.
    I added a commit to Stephan's pull request fixing all the build errors.

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

    $ git pull https://github.com/rmetzger/flink test_utils

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

    https://github.com/apache/flink/pull/2148.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 #2148
    
----
commit f30b51d69cdf870f920fccbe96c2e00295ade226
Author: Stephan Ewen <se...@apache.org>
Date:   2016-05-30T18:56:05Z

    [FLINK-3995] [build] Properly structure test scopes and dependencies
    
    Makes the JUnit test utils (TestLogger, retry rules, ...) properly available to
    other projects without the 'flink-core' test-jar, via the 'flink-test-utils-junit' project.
    
    Makes the ForkableMiniCluster, TestEnvironment,  and other test utilities available in the 'main'
    scope of the 'flink-test-utils' project.
    
    Creates a 'flink-test-utils-parent' project that holds the 'flink-test-utils-junit' and
    'flink-test-utils' project.
    
    Also moves some tests between projects and inlines some very simple utility functions in
    order to simplify some test jar dependencies.

commit 82bd8b88190ab50b15be2716f0ae84afafadfffb
Author: Stephan Ewen <se...@apache.org>
Date:   2016-06-11T00:14:07Z

    [FLINK-3995] [build] flink-test-utils also contains the streaming test utilities.
    
    Test utilities include the StreamingMultipleProgramsTestBase and StreamingTestEnvironment.
    
    This moves the ITCases for streaming into 'flink-tests' to achieve that.

commit 59113586caa12a9b7c943b4c4322d00df223d691
Author: Robert Metzger <rm...@apache.org>
Date:   2016-06-20T12:20:59Z

    Make the build pass

----


---
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] flink pull request #2148: [FLINK-3995] [build] Make test utils properly avai...

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

    https://github.com/apache/flink/pull/2148


---
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] flink issue #2148: [FLINK-3995] [build] Make test utils properly available a...

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

    https://github.com/apache/flink/pull/2148
  
    Looks good to me!


---
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] flink issue #2148: [FLINK-3995] [build] Make test utils properly available a...

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

    https://github.com/apache/flink/pull/2148
  
    Thank you for the review ... merging ...


---
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.
---