falcon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Raghav Gautam" <rag...@apache.org>
Subject Re: Review Request 34240: Feed Wizard multiple tests
Date Thu, 21 May 2015 00:16:44 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/34240/#review84625
-----------------------------------------------------------



falcon-regression/merlin/src/main/java/org/apache/falcon/regression/ui/search/AbstractSearchPage.java
<https://reviews.apache.org/r/34240/#comment135962>

    In general, it is better to use list instead of arrays. The benefits are better toString
and being able to search for element without for loop.



falcon-regression/merlin/src/test/java/org/apache/falcon/regression/searchUI/FeedSetupTest.java
<https://reviews.apache.org/r/34240/#comment135966>

    Assert failure without appropriate message don't convey much. It will be great if you
can add appropriate messages to be printed when assert fails.
    
    Applicable to other assertions as well.



falcon-regression/merlin/src/test/java/org/apache/falcon/regression/searchUI/FeedSetupTest.java
<https://reviews.apache.org/r/34240/#comment135969>

    You can set tags and groups even if they are not null ?
    
    In case you need to do the check, using StringUtil.isEmpty() would be more robust.
    
    Applicable to other places also.



falcon-regression/merlin/src/test/java/org/apache/falcon/regression/searchUI/FeedSetupTest.java
<https://reviews.apache.org/r/34240/#comment135973>

    Is this string same as the one that has been set a few lines above. In that case extract
a constant so, that this fact is more obvious.
    
    If not, then how did you come up with this string ?



falcon-regression/merlin/src/test/java/org/apache/falcon/regression/searchUI/FeedSetupTest.java
<https://reviews.apache.org/r/34240/#comment135974>

    Can you use all the path to be starting with prefix baseTestHDFSDir ?
    
    That way by looking at the path in the printed log/screenshot etc, we would know what
test are we talking about.
    
    Applicable at multiple places.


- Raghav Gautam


On May 19, 2015, 6:49 p.m., Namit Maheshwari wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/34240/
> -----------------------------------------------------------
> 
> (Updated May 19, 2015, 6:49 p.m.)
> 
> 
> Review request for Falcon.
> 
> 
> Bugs: FALCON-1222
>     https://issues.apache.org/jira/browse/FALCON-1222
> 
> 
> Repository: falcon-git
> 
> 
> Description
> -------
> 
> Feed Wizard multiple tests
> 
> 
> Diffs
> -----
> 
>   falcon-regression/merlin/src/main/java/org/apache/falcon/regression/ui/search/AbstractSearchPage.java
e72ce6706d7fe3331744d723894193bfbaff586a 
>   falcon-regression/merlin/src/main/java/org/apache/falcon/regression/ui/search/FeedWizardPage.java
6eed4cef86c1974c3d30f0133b34894e24ac1750 
>   falcon-regression/merlin/src/test/java/org/apache/falcon/regression/searchUI/FeedSetupTest.java
8920f41129175a207ae6dee0659a514fef17bcfc 
> 
> Diff: https://reviews.apache.org/r/34240/diff/
> 
> 
> Testing
> -------
> 
> Below new tests added. Tested the same.
> testWizardCancel
> testWizardXmlPreview
> testGeneralStepEditXml
> testGeneralStepDefaultScenario
> testGeneralStepXmlPreview
> testGeneralStepAddRemoveTag
> testGeneralStepBlankOptionalFields
> testTimingStepDefaultScenario
> testTimingStepXmlPreview
> testTimingStepEditXml
> testTimingStepDropDownLists
> testTimingStepAddDeleteProperties
> testLocationStepValidValuesFS
> testLocationStepValidCatalogStorage
> testLocationStepBothLocationsAndTableUri
> testLocationStepEditXml
> testClustersStepDefaultScenario
> testClustersStepDropDownLists
> testClustersStepDefaultLocations
> testClusterStepEditXml
> testSummaryStepDefaultScenario
> 
> 
> Thanks,
> 
> Namit Maheshwari
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message