New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rework and add test for checking of Test Runner plugin #6401
Conversation
6140007
to
a9e1e6d
Compare
a9e1e6d
to
d3c23df
Compare
ci-build |
ws.getId(), | ||
Paths.get(resource.toURI()), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Consider porting this class to master to avoid painful merge conflicts.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks OK, but I have too little experience in this area, so up 2 @tolusha
assertTrue(pluginConsole.getAllMethodsMarkedDefinedStatus(PASSED).size() == 1); | ||
assertTrue(pluginConsole.getAllMethodsMarkedDefinedStatus(FAILED).size() == 1); | ||
Assert.assertTrue(pluginConsole.getAllNamesOfMethodsMarkedDefinedStatus(PASSED).size() == 1); | ||
Assert.assertTrue(pluginConsole.getAllNamesOfMethodsMarkedDefinedStatus(FAILED).size() == 1); | ||
String testErrorMessage = pluginConsole.getTestErrorMessage(); | ||
assertTrue( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use static import everywhere if you star using it
43ae545
to
27e9b0d
Compare
* | ||
* @param xoffset offset in pixels for shifting | ||
*/ | ||
public void dragConsolesInDefinePosition(int xoffset) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If the name of the method is self descriptive I don't want to see any java doc.
Review all javadocs you have added in this PR. https://dev.liferay.com/participate/javadoc-guidelines
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The name of the variables xoffset
seems weird.
import org.testng.annotations.BeforeClass; | ||
import org.testng.annotations.Test; | ||
|
||
public class JavaTestPluginJuinit4CheckRunSuitesAndScopesTest { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo in the name of the class
projectExplorer.waitItem(JUNIT4_PROJECT); | ||
runCompileCommandByPallete(compileCommand); | ||
notifications.waitProgressPopupPanelClose(); | ||
consoles.dragConsolesInDefinePosition(-100); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
-100
says nothing to me
+ " at org.junit.Assert.assertFalse(Assert.java:64)\n" | ||
+ " at org.junit.Assert.assertFalse(Assert.java:74)\n" | ||
+ " at org.eclipse.che.examples.AppAnotherTest.shouldFailOfAppAnother(AppAnotherTest.java:33)"; | ||
+ " at org.junit. fail( java:86)\n" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is weird to see a space before method.
import org.testng.annotations.BeforeClass; | ||
import org.testng.annotations.Test; | ||
|
||
public class JavaTestPluginJunit4CheckRunSuitesAndScopesTest { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Author?
} | ||
|
||
/** | ||
* click on the item in the result tree. If will be some items with the same name - will select |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Uppercase?
Build success. https://ci.codenvycorp.com/job/che-pullrequests-build/3865/ |
What does this PR do?
@garagatyi, @vparfonov, @tolusha