[ 
https://issues.apache.org/jira/browse/GORA-259?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Apostolos Giannakidis updated GORA-259:
---------------------------------------

    Attachment: GORA-259.patch

I just attached a patch for this.

Files modified:
TestCassandraStore.java
TestDynamoDBStore.java
TestHBaseStoreCountQuery.java
TestHBaseStoreWordCount.java
TestHBaseStore.java

I created the patch using: "git diff --no-prefix" so please let me know if 
there are any issues because of this.
                
> Removal of the main methods from the test case classes
> ------------------------------------------------------
>
>                 Key: GORA-259
>                 URL: https://issues.apache.org/jira/browse/GORA-259
>             Project: Apache Gora
>          Issue Type: Test
>    Affects Versions: 0.3
>            Reporter: Apostolos Giannakidis
>            Assignee: Apostolos Giannakidis
>            Priority: Minor
>              Labels: junit
>             Fix For: 0.4
>
>         Attachments: GORA-259.patch
>
>
> Some JUnit test case classes, such as the TestHBaseStore.java contain a main 
> method. Is this method necessary? As far as I know, having main methods in 
> the JUnit test case classes is not a recommended practice [1].
> [1] 
> http://sqa.fyicenter.com/FAQ/JUnit/Do_You_Need_to_Write_a_main_Method_in_a_JUnit_.html

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to