Ignore:
Timestamp:
2015-06-20T23:42:21+02:00 (10 years ago)
Author:
Don-vip
Message:

checkstyle: enable relevant whitespace checks and fix them

Location:
trunk/test/unit/org/openstreetmap/josm/gui/conflict
Files:
4 edited

Legend:

Unmodified
Added
Removed

  • trunk/test/unit/org/openstreetmap/josm/gui/conflict/nodes/NodeListMergeModelTest.java

    r8450 r8510  
    5252    protected void ensureSelected(DefaultListSelectionModel model, Object... idx) {
    5353        if (idx == null) return;
    54         for (int i=0; i < idx.length; i++) {
     54        for (int i = 0; i < idx.length; i++) {
    5555            if (idx[i] instanceof Integer) {
    56                 int j = (Integer)idx[i];
     56                int j = (Integer) idx[i];
    5757                assertTrue("expected row " + j + " to be selected", model.isSelectedIndex(j));
    5858                break;
    5959            }
    6060            try {
    61                 int rows[] = (int[])idx[i];
     61                int rows[] = (int[]) idx[i];
    6262                if (rows.length != 2) {
    6363                    fail("illegal selection range. Either null or not length 2: " + Arrays.toString(rows));
    … …  
    6969                    assertTrue("expected row " + j + " to be selected", model.isSelectedIndex(j));
    7070                }
    71             } catch(ClassCastException e) {
     71            } catch (ClassCastException e) {
    7272                fail("illegal selection range:" + idx[i]);
    7373            }
    … …  
    9595        DefaultListSelectionModel mergedSelection = inspectListSelectionModel(model, "mergedEntriesSelectionModel");
    9696        ensureSelected(mergedSelection, 0);
    97 
    98     }
    99 
     97    }
    10098
    10199    @Test
    … …  
    123121        DefaultListSelectionModel mergedSelection = inspectListSelectionModel(model, "mergedEntriesSelectionModel");
    124122        ensureSelected(mergedSelection, 0);
    125 
    126123    }
    127124
    … …  
    167164        mergedNodes.add(new Node(1));
    168165
    169         model.copyMyToTop(new int[]{1,2}); // copy node 3 and 4
     166        model.copyMyToTop(new int[]{1, 2}); // copy node 3 and 4
    170167
    171168        mergedNodes = inspectNodeList(model, "Merged");
    … …  
    176173
    177174        DefaultListSelectionModel mergedSelection = inspectListSelectionModel(model, "mergedEntriesSelectionModel");
    178         ensureSelected(mergedSelection, 0,1); // first two rows selected
    179     }
    180 
     175        ensureSelected(mergedSelection, 0, 1); // first two rows selected
     176    }
    181177
    182178    @Test
    … …  
    269265        mergedNodes.add(new Node(1));
    270266
    271         model.copyMyToEnd(new int[]{1,2}); // copy node 3 and 4
     267        model.copyMyToEnd(new int[]{1, 2}); // copy node 3 and 4
    272268
    273269        mergedNodes = inspectNodeList(model, "Merged");
    … …  
    279275
    280276        DefaultListSelectionModel mergedSelection = inspectListSelectionModel(model, "mergedEntriesSelectionModel");
    281         ensureSelected(mergedSelection, 1,2); // last two rows selected
     277        ensureSelected(mergedSelection, 1, 2); // last two rows selected
    282278    }
    283279
    … …  
    313309    }
    314310
    315 
    316311    @Test
    317312    public void test_copyMyNodesBeforeCurrent_2() {
    … …  
    329324        mergedNodes.add(new Node(12));
    330325
    331         model.copyMyBeforeCurrent(new int[]{0,1}, 0);
     326        model.copyMyBeforeCurrent(new int[]{0, 1}, 0);
    332327
    333328        assertEquals(5, mergedNodes.size());
    … …  
    339334
    340335        DefaultListSelectionModel mergedSelection = inspectListSelectionModel(model, "mergedEntriesSelectionModel");
    341         ensureSelected(mergedSelection, 0,1); // position 0 and 1 selected
     336        ensureSelected(mergedSelection, 0, 1); // position 0 and 1 selected
    342337    }
    343338
    … …  
    358353
    359354        try {
    360             model.copyMyBeforeCurrent(new int[]{0,1}, -1);
     355            model.copyMyBeforeCurrent(new int[]{0, 1}, -1);
    361356            fail("expected IllegalArgumentException");
    362         } catch(IllegalArgumentException e) {
     357        } catch (IllegalArgumentException e) {
    363358            // OK
    364359        }
    365360
    366361        try {
    367             model.copyMyBeforeCurrent(new int[]{0,1}, 4);
     362            model.copyMyBeforeCurrent(new int[]{0, 1}, 4);
    368363            fail("expected IllegalArgumentException");
    369         } catch(IllegalArgumentException e) {
     364        } catch (IllegalArgumentException e) {
    370365            // OK
    371366        }
    … …  
    402397    }
    403398
    404 
    405399    @Test
    406400    public void test_copyMyNodesAfterCurrent_2() {
    … …  
    419413        mergedNodes.add(new Node(12));
    420414
    421         model.copyMyAfterCurrent(new int[]{0,1}, 2);
     415        model.copyMyAfterCurrent(new int[]{0, 1}, 2);
    422416
    423417        assertEquals(5, mergedNodes.size());
    … …  
    428422        assertEquals(2, mergedNodes.get(4).getId());  // copied node 2 at position 4
    429423
    430 
    431         DefaultListSelectionModel mergedSelection = inspectListSelectionModel(model, "mergedEntriesSelectionModel");
    432         ensureSelected(mergedSelection, 3,4); // position 3,4 selected
     424        DefaultListSelectionModel mergedSelection = inspectListSelectionModel(model, "mergedEntriesSelectionModel");
     425        ensureSelected(mergedSelection, 3, 4); // position 3,4 selected
    433426    }
    434427
    … …  
    449442        mergedNodes.add(new Node(12));
    450443
    451         model.copyMyAfterCurrent(new int[]{0,2}, 0);
     444        model.copyMyAfterCurrent(new int[]{0, 2}, 0);
    452445
    453446        assertEquals(5, mergedNodes.size());
    … …  
    459452
    460453        DefaultListSelectionModel mergedSelection = inspectListSelectionModel(model, "mergedEntriesSelectionModel");
    461         ensureSelected(mergedSelection, 1,2); // position 1,2 selected
     454        ensureSelected(mergedSelection, 1, 2); // position 1,2 selected
    462455    }
    463456
    … …  
    509502        mergedNodes.add(new Node(14));
    510503
    511         model.moveUpMerged(new int[]{1,4});
     504        model.moveUpMerged(new int[]{1, 4});
    512505
    513506        assertEquals(5, mergedNodes.size());
    … …  
    519512
    520513        DefaultListSelectionModel mergedSelection = inspectListSelectionModel(model, "mergedEntriesSelectionModel");
    521         ensureSelected(mergedSelection, 0,3); // position 0 and 3 selecte0
     514        ensureSelected(mergedSelection, 0, 3); // position 0 and 3 selecte0
    522515    }
    523516
    … …  
    539532        mergedNodes.add(new Node(14));
    540533
    541         model.moveUpMerged(new int[]{1,2,3,4});
     534        model.moveUpMerged(new int[]{1, 2, 3, 4});
    542535
    543536        assertEquals(5, mergedNodes.size());
    … …  
    549542
    550543        DefaultListSelectionModel mergedSelection = inspectListSelectionModel(model, "mergedEntriesSelectionModel");
    551         ensureSelected(mergedSelection, 0,1,2,3);
     544        ensureSelected(mergedSelection, 0, 1, 2, 3);
    552545    }
    553546
    … …  
    598591        mergedNodes.add(new Node(14));
    599592
    600         model.moveDownMerged(new int[]{1,3});
     593        model.moveDownMerged(new int[]{1, 3});
    601594
    602595        assertEquals(5, mergedNodes.size());
    … …  
    608601
    609602        DefaultListSelectionModel mergedSelection = inspectListSelectionModel(model, "mergedEntriesSelectionModel");
    610         ensureSelected(mergedSelection, 2,4);
     603        ensureSelected(mergedSelection, 2, 4);
    611604    }
    612605
    … …  
    628621        mergedNodes.add(new Node(14));
    629622
    630         model.moveDownMerged(new int[]{1,2,3});
     623        model.moveDownMerged(new int[]{1, 2, 3});
    631624
    632625        assertEquals(5, mergedNodes.size());
    … …  
    638631
    639632        DefaultListSelectionModel mergedSelection = inspectListSelectionModel(model, "mergedEntriesSelectionModel");
    640         ensureSelected(mergedSelection, 2,3,4);
     633        ensureSelected(mergedSelection, 2, 3, 4);
    641634    }
    642635
    … …  
    712705            }
    713706        }
     707
    714708        MyListener listener = new MyListener();
    715709        model.addPropertyChangeListener(listener);
    … …  
    722716        assertEquals(!oldValue, listener.events.get(0).getNewValue());
    723717    }
    724 
    725718}

  • trunk/test/unit/org/openstreetmap/josm/gui/conflict/properties/PropertiesMergeModelTest.java

    r7937 r8510  
    9494        assertFalse(model.hasCoordConflict());
    9595
    96         n1.setCoor(new LatLon(1,1));
     96        n1.setCoor(new LatLon(1, 1));
    9797        populate(n1, n2);
    9898        assertTrue(model.hasCoordConflict());
    … …  
    100100
    101101        n1.cloneFrom(new Node(1));
    102         n2.setCoor(new LatLon(2,2));
     102        n2.setCoor(new LatLon(2, 2));
    103103        populate(n1, n2);
    104104        assertTrue(model.hasCoordConflict());
    105105
    106         n1.setCoor(new LatLon(1,1));
    107         n2.setCoor(new LatLon(2,2));
     106        n1.setCoor(new LatLon(1, 1));
     107        n2.setCoor(new LatLon(2, 2));
    108108        populate(n1, n2);
    109109        assertTrue(model.hasCoordConflict());

  • trunk/test/unit/org/openstreetmap/josm/gui/conflict/tags/TagMergeItemTest.java

    r8509 r8510  
    6868        assertEquals("key", item.getKey());
    6969        assertNull(item.getMyTagValue());
    70         assertEquals("theirvalue",item.getTheirTagValue());
     70        assertEquals("theirvalue", item.getTheirTagValue());
    7171        assertEquals(MergeDecisionType.UNDECIDED, item.getMergeDecision());
    7272    }
    73 
    7473
    7574    @Test
    … …  
    8685            item.decide(null);
    8786            fail("expected IllegalArgumentException not thrown");
    88         } catch(IllegalArgumentException e) {
     87        } catch (IllegalArgumentException e) {
    8988            // OK
    9089        }
    … …  
    104103        item.applyToMyPrimitive(n1);
    105104        assertEquals("myvalue", n1.get("key"));
    106 
    107105    }
    108106
    … …  
    133131            item.applyToMyPrimitive(n1);
    134132            fail("expected IllegalStateException");
    135         } catch(IllegalStateException e) {
     133        } catch (IllegalStateException e) {
    136134            // OK
    137135        }
    … …  
    145143            item.applyToMyPrimitive(null);
    146144            fail("expected IllegalArgumentException");
    147         } catch(IllegalArgumentException e) {
     145        } catch (IllegalArgumentException e) {
    148146            // OK
    149147        }

  • trunk/test/unit/org/openstreetmap/josm/gui/conflict/tags/TagMergeModelTest.java

    r8391 r8510  
    210210
    211211        TagMergeModel model = new TagMergeModel();
    212         for (int i=0; i < 10; i++) {
     212        for (int i = 0; i < 10; i++) {
    213213            model.addItem(new TagMergeItem("key-" + i, "myvalue-" + i, "theirvalue-" +i));
    214214        }
    … …  
    222222
    223223        model.decide(new int[] {0, 3, 5}, MergeDecisionType.KEEP_MINE);
    224         for (int i = 0; i< 10; i++) {
     224        for (int i = 0; i < 10; i++) {
    225225            TagMergeItem item = list.get(i);
    226226            if (i == 0 || i == 3 || i == 5) {
Note: See TracChangeset for help on using the changeset viewer.