falcon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Paul Isaychuk" <pisayc...@hortonworks.com>
Subject Re: Review Request 41747: [FALCON-1698] New tests for ProcessSetupTest, ClusterSetupTest, UI test fixes
Date Tue, 05 Jan 2016 14:38:10 GMT

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

(Updated Jan. 5, 2016, 2:38 p.m.)


Review request for Falcon.


Changes
-------

1. Making delay OS independent
2. Removing code related to another patch.
3. Adding comment for clickLink() method


Bugs: FALCON-1698
    https://issues.apache.org/jira/browse/FALCON-1698


Repository: falcon-git


Description
-------

Adding new tests to ProcessSetupTest, ClusterSetupTest; UI test fixes related to UI changes.


Diffs (updated)
-----

  falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/Entities/ClusterMerlin.java
fe1338a 
  falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/Entities/ProcessMerlin.java
7607aa6 
  falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/core/bundle/Bundle.java
67d9ee2 
  falcon-regression/merlin/src/main/java/org/apache/falcon/regression/ui/search/ClusterWizardPage.java
bcada4a 
  falcon-regression/merlin/src/main/java/org/apache/falcon/regression/ui/search/EntityPage.java
f7499b7 
  falcon-regression/merlin/src/main/java/org/apache/falcon/regression/ui/search/PageHeader.java
7f87091 
  falcon-regression/merlin/src/main/java/org/apache/falcon/regression/ui/search/ProcessWizardPage.java
5dcd700 
  falcon-regression/merlin/src/test/java/org/apache/falcon/regression/searchUI/ClusterSetupTest.java
5efa5b2 
  falcon-regression/merlin/src/test/java/org/apache/falcon/regression/searchUI/MirrorSummaryTest.java
989e4b3 
  falcon-regression/merlin/src/test/java/org/apache/falcon/regression/searchUI/ProcessSetupTest.java
eae5137 

Diff: https://reviews.apache.org/r/41747/diff/


Testing
-------

done


Thanks,

Paul Isaychuk


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