[jira] [Commented] (RYA-332) Rename IntegrationTests to be suffixed with IT so they are tested.

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/RYA-332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16140886#comment-16140886
 ] 

ASF GitHub Bot commented on RYA-332:


Github user asfgit commented on the issue:

https://github.com/apache/incubator-rya/pull/212
  

Refer to this link for build results (access rights to CI server needed): 

https://builds.apache.org/job/incubator-rya-master-with-optionals-pull-requests/452/



> Rename IntegrationTests to be suffixed with IT so they are tested.
> --
>
> Key: RYA-332
> URL: https://issues.apache.org/jira/browse/RYA-332
> Project: Rya
>  Issue Type: Sub-task
>  Components: build
>Reporter: Jeff Dasch
>Assignee: Andrew Smith
>
> Several Integration Tests do not get tested by the maven-failsafe-plugin 
> because they do not follow the standard IT naming convention (*IT.java).
> Also, remove the maven-surefire-plugin *IntegrationTest.java exclusion in the 
> parent pom.
> Affected tests:
> {noformat}
> $ find incubator-rya -name *IntegrationTest.java
> incubator-rya/extras/indexing/src/test/java/org/apache/rya/indexing/external/AccumuloConstantPcjIntegrationTest.java
> incubator-rya/extras/indexing/src/test/java/org/apache/rya/indexing/external/AccumuloPcjIntegrationTest.java
> incubator-rya/extras/indexing/src/test/java/org/apache/rya/indexing/external/PrecompJoinOptimizerIntegrationTest.java
> incubator-rya/extras/rya.indexing.pcj/src/test/java/org/apache/rya/indexing/pcj/storage/accumulo/PcjTablesIntegrationTest.java
> incubator-rya/osgi/camel.rya/src/test/java/org/apache/rya/camel/cbsail/CbSailIntegrationTest.java
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Commented] (RYA-332) Rename IntegrationTests to be suffixed with IT so they are tested.

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/RYA-332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16138940#comment-16138940
 ] 

ASF GitHub Bot commented on RYA-332:


Github user jdasch commented on the issue:

https://github.com/apache/incubator-rya/pull/212
  
Ditto. Looks good.  Just need to fix this test: 
org.apache.rya.camel.cbsail.CbSailIT.testCbSail


> Rename IntegrationTests to be suffixed with IT so they are tested.
> --
>
> Key: RYA-332
> URL: https://issues.apache.org/jira/browse/RYA-332
> Project: Rya
>  Issue Type: Sub-task
>  Components: build
>Reporter: Jeff Dasch
>Assignee: Andrew Smith
>
> Several Integration Tests do not get tested by the maven-failsafe-plugin 
> because they do not follow the standard IT naming convention (*IT.java).
> Also, remove the maven-surefire-plugin *IntegrationTest.java exclusion in the 
> parent pom.
> Affected tests:
> {noformat}
> $ find incubator-rya -name *IntegrationTest.java
> incubator-rya/extras/indexing/src/test/java/org/apache/rya/indexing/external/AccumuloConstantPcjIntegrationTest.java
> incubator-rya/extras/indexing/src/test/java/org/apache/rya/indexing/external/AccumuloPcjIntegrationTest.java
> incubator-rya/extras/indexing/src/test/java/org/apache/rya/indexing/external/PrecompJoinOptimizerIntegrationTest.java
> incubator-rya/extras/rya.indexing.pcj/src/test/java/org/apache/rya/indexing/pcj/storage/accumulo/PcjTablesIntegrationTest.java
> incubator-rya/osgi/camel.rya/src/test/java/org/apache/rya/camel/cbsail/CbSailIntegrationTest.java
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Commented] (RYA-332) Rename IntegrationTests to be suffixed with IT so they are tested.

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/RYA-332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16138847#comment-16138847
 ] 

ASF GitHub Bot commented on RYA-332:


Github user meiercaleb commented on the issue:

https://github.com/apache/incubator-rya/pull/212
  
I'm fine with this PR as soon as you fix the broken Camel integration tests.


> Rename IntegrationTests to be suffixed with IT so they are tested.
> --
>
> Key: RYA-332
> URL: https://issues.apache.org/jira/browse/RYA-332
> Project: Rya
>  Issue Type: Sub-task
>  Components: build
>Reporter: Jeff Dasch
>Assignee: Andrew Smith
>
> Several Integration Tests do not get tested by the maven-failsafe-plugin 
> because they do not follow the standard IT naming convention (*IT.java).
> Also, remove the maven-surefire-plugin *IntegrationTest.java exclusion in the 
> parent pom.
> Affected tests:
> {noformat}
> $ find incubator-rya -name *IntegrationTest.java
> incubator-rya/extras/indexing/src/test/java/org/apache/rya/indexing/external/AccumuloConstantPcjIntegrationTest.java
> incubator-rya/extras/indexing/src/test/java/org/apache/rya/indexing/external/AccumuloPcjIntegrationTest.java
> incubator-rya/extras/indexing/src/test/java/org/apache/rya/indexing/external/PrecompJoinOptimizerIntegrationTest.java
> incubator-rya/extras/rya.indexing.pcj/src/test/java/org/apache/rya/indexing/pcj/storage/accumulo/PcjTablesIntegrationTest.java
> incubator-rya/osgi/camel.rya/src/test/java/org/apache/rya/camel/cbsail/CbSailIntegrationTest.java
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Commented] (RYA-332) Rename IntegrationTests to be suffixed with IT so they are tested.

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/RYA-332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16137594#comment-16137594
 ] 

ASF GitHub Bot commented on RYA-332:


Github user asfgit commented on the issue:

https://github.com/apache/incubator-rya/pull/212
  

Refer to this link for build results (access rights to CI server needed): 

https://builds.apache.org/job/incubator-rya-master-with-optionals-pull-requests/427/



> Rename IntegrationTests to be suffixed with IT so they are tested.
> --
>
> Key: RYA-332
> URL: https://issues.apache.org/jira/browse/RYA-332
> Project: Rya
>  Issue Type: Sub-task
>  Components: build
>Reporter: Jeff Dasch
>Assignee: Andrew Smith
>
> Several Integration Tests do not get tested by the maven-failsafe-plugin 
> because they do not follow the standard IT naming convention (*IT.java).
> Also, remove the maven-surefire-plugin *IntegrationTest.java exclusion in the 
> parent pom.
> Affected tests:
> {noformat}
> $ find incubator-rya -name *IntegrationTest.java
> incubator-rya/extras/indexing/src/test/java/org/apache/rya/indexing/external/AccumuloConstantPcjIntegrationTest.java
> incubator-rya/extras/indexing/src/test/java/org/apache/rya/indexing/external/AccumuloPcjIntegrationTest.java
> incubator-rya/extras/indexing/src/test/java/org/apache/rya/indexing/external/PrecompJoinOptimizerIntegrationTest.java
> incubator-rya/extras/rya.indexing.pcj/src/test/java/org/apache/rya/indexing/pcj/storage/accumulo/PcjTablesIntegrationTest.java
> incubator-rya/osgi/camel.rya/src/test/java/org/apache/rya/camel/cbsail/CbSailIntegrationTest.java
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Commented] (RYA-332) Rename IntegrationTests to be suffixed with IT so they are tested.

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/RYA-332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16137299#comment-16137299
 ] 

ASF GitHub Bot commented on RYA-332:


Github user asfgit commented on the issue:

https://github.com/apache/incubator-rya/pull/212
  

Refer to this link for build results (access rights to CI server needed): 

https://builds.apache.org/job/incubator-rya-master-with-optionals-pull-requests/422/



> Rename IntegrationTests to be suffixed with IT so they are tested.
> --
>
> Key: RYA-332
> URL: https://issues.apache.org/jira/browse/RYA-332
> Project: Rya
>  Issue Type: Sub-task
>  Components: build
>Reporter: Jeff Dasch
>Assignee: Andrew Smith
>
> Several Integration Tests do not get tested by the maven-failsafe-plugin 
> because they do not follow the standard IT naming convention (*IT.java).
> Also, remove the maven-surefire-plugin *IntegrationTest.java exclusion in the 
> parent pom.
> Affected tests:
> {noformat}
> $ find incubator-rya -name *IntegrationTest.java
> incubator-rya/extras/indexing/src/test/java/org/apache/rya/indexing/external/AccumuloConstantPcjIntegrationTest.java
> incubator-rya/extras/indexing/src/test/java/org/apache/rya/indexing/external/AccumuloPcjIntegrationTest.java
> incubator-rya/extras/indexing/src/test/java/org/apache/rya/indexing/external/PrecompJoinOptimizerIntegrationTest.java
> incubator-rya/extras/rya.indexing.pcj/src/test/java/org/apache/rya/indexing/pcj/storage/accumulo/PcjTablesIntegrationTest.java
> incubator-rya/osgi/camel.rya/src/test/java/org/apache/rya/camel/cbsail/CbSailIntegrationTest.java
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Commented] (RYA-332) Rename IntegrationTests to be suffixed with IT so they are tested.

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/RYA-332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16137255#comment-16137255
 ] 

ASF GitHub Bot commented on RYA-332:


GitHub user isper3at opened a pull request:

https://github.com/apache/incubator-rya/pull/212

RYA-332 rename integration tests

## Description
>What Changed?

Some integration tests were being ignored
since the class names ended with
"IntegrationTest" instead of "IT"

### Tests
>Coverage?
N/A

### Links
[Jira](https://issues.apache.org/jira/browse/RYA-332)

### Checklist
- [ ] Code Review
- [x] Squash Commits

 People To Reivew
@jdasch
@meiercaleb 


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

$ git pull https://github.com/isper3at/incubator-rya RYA-332

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

https://github.com/apache/incubator-rya/pull/212.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 #212


commit 888d7eda61df5a1ab0c763f8ba8313879798fbe0
Author: isper3at 
Date:   2017-08-22T19:28:18Z

RYA-332 rename integration tests

Some integration tests were being ignored
since the class names ended with
"IntegrationTest" instead of "IT"




> Rename IntegrationTests to be suffixed with IT so they are tested.
> --
>
> Key: RYA-332
> URL: https://issues.apache.org/jira/browse/RYA-332
> Project: Rya
>  Issue Type: Sub-task
>  Components: build
>Reporter: Jeff Dasch
>Assignee: Andrew Smith
>
> Several Integration Tests do not get tested by the maven-failsafe-plugin 
> because they do not follow the standard IT naming convention (*IT.java).
> Also, remove the maven-surefire-plugin *IntegrationTest.java exclusion in the 
> parent pom.
> Affected tests:
> {noformat}
> $ find incubator-rya -name *IntegrationTest.java
> incubator-rya/extras/indexing/src/test/java/org/apache/rya/indexing/external/AccumuloConstantPcjIntegrationTest.java
> incubator-rya/extras/indexing/src/test/java/org/apache/rya/indexing/external/AccumuloPcjIntegrationTest.java
> incubator-rya/extras/indexing/src/test/java/org/apache/rya/indexing/external/PrecompJoinOptimizerIntegrationTest.java
> incubator-rya/extras/rya.indexing.pcj/src/test/java/org/apache/rya/indexing/pcj/storage/accumulo/PcjTablesIntegrationTest.java
> incubator-rya/osgi/camel.rya/src/test/java/org/apache/rya/camel/cbsail/CbSailIntegrationTest.java
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)