Skip to content

Commit

Permalink
No longer ignore some tests since they pass now; fix whitespace issue
Browse files Browse the repository at this point in the history
  • Loading branch information
JordanMartinez committed Oct 1, 2017
1 parent 7f4b164 commit b105dc4
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@
import org.fxmisc.richtext.CodeArea;
import org.fxmisc.richtext.InlineCssTextAreaAppTest;
import org.junit.Before;
import org.junit.Ignore;
import org.junit.Test;
import org.junit.runner.RunWith;
import org.testfx.framework.junit.ApplicationTest;
Expand Down Expand Up @@ -210,7 +209,7 @@ private void runAssert() {
}

@Test
public void cut() {
public void cut() {
// this test fails on Linux; Windows is untested
// so for now, only run on Mac
// TODO: update if test succeeds on Windows, too
Expand Down Expand Up @@ -252,9 +251,8 @@ private void runAssert() {
assertEquals(beginning + text + end, area.getText());
}

@Ignore("Flaky test when all others of equivalent tests pass")
@Test
public void paste() {
public void paste() {
// this test fails on Linux; Windows is untested
// so for now, only run on Mac
// TODO: update if test succeeds on Windows, too
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,6 @@ public void page_up_moves_caret_to_top_of_viewport() {
assertTrue(beforeBounds.getMinY() > afterBounds.getMinY());
}

@Ignore("doesn't work despite 'testShiftPageDown' working fine using the same code")
@Test
public void page_down_moves_caret_to_bottom_of_viewport() throws Exception {
interact(() -> {
Expand Down

0 comments on commit b105dc4

Please sign in to comment.