Changeset 8510 in josm for trunk/test/functional


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

checkstyle: enable relevant whitespace checks and fix them

Location:
trunk/test/functional/org/openstreetmap/josm
Files:
9 edited

Legend:

Unmodified
Added
Removed

  • trunk/test/functional/org/openstreetmap/josm/gui/conflict/pair/nodes/NodeListMergerTest.java

    r7937 r8510  
    2222        Node n1;
    2323        w1.addNode(n1 = new Node(1));
    24         for (int i=0; i < 20; i++) {
     24        for (int i = 0; i < 20; i++) {
    2525            n1.put("key" + i, "value" + i);
    2626        }
    2727        StringBuilder note = new StringBuilder();
    28         for (int i=0; i < 50; i++) {
     28        for (int i = 0; i < 50; i++) {
    2929            note.append(" A very long text ");
    3030        }
    … …  
    4949
    5050        Way w2 = new Way();
    51         for (int i = 1; i < 200; i+=2) {
     51        for (int i = 1; i < 200; i += 2) {
    5252            w2.addNode(new Node(i));
    5353        }
    … …  
    7272    static public void main(String args[]) {
    7373        NodeListMergerTest test = new NodeListMergerTest();
    74         test.setSize(600,600);
     74        test.setSize(600, 600);
    7575        test.setVisible(true);
    7676    }

  • trunk/test/functional/org/openstreetmap/josm/gui/conflict/pair/properties/PropertiesMergerTest.java

    r7937 r8510  
    2828    protected void populate() {
    2929        Node my = new Node(1);
    30         my.setCoor(new LatLon(1,1));
     30        my.setCoor(new LatLon(1, 1));
    3131        my.setDeleted(true);
    3232
    3333        Node their = new Node(2);
    34         their.setCoor(new LatLon(10,10));
     34        their.setCoor(new LatLon(10, 10));
    3535
    3636        merger.getModel().populate(new Conflict<OsmPrimitive>(my, their));

  • trunk/test/functional/org/openstreetmap/josm/gui/conflict/pair/relation/RelationMemberMergerTest.java

    r7937 r8510  
    5151    static public void main(String args[]) {
    5252        RelationMemberMergerTest test = new RelationMemberMergerTest();
    53         test.setSize(600,600);
     53        test.setSize(600, 600);
    5454        test.setVisible(true);
    5555    }

  • trunk/test/functional/org/openstreetmap/josm/gui/conflict/pair/tags/TagMergerTest.java

    r8509 r8510  
    2929        tagMerger.getModel().addItem(new TagMergeItem("a very long key asdfasdf asdfasdf", "a very long value asdfasdf",
    3030                "a very long value asdfasdf"));
    31         for (int i=0; i< 50; i++) {
     31        for (int i = 0; i < 50; i++) {
    3232          tagMerger.getModel().addItem(new TagMergeItem("key", "myvalue", "theirvalue"));
    3333        }
    … …  
    3636    public static void main(String args[]) {
    3737        TagMergerTest test  = new TagMergerTest();
    38         test.setSize(600,600);
     38        test.setSize(600, 600);
    3939        test.setVisible(true);
    4040    }

  • trunk/test/functional/org/openstreetmap/josm/gui/dialogs/ConflictResolutionDialogTest.java

    r7937 r8510  
    1616
    1717    protected void build() {
    18         setSize(100,100);
     18        setSize(100, 100);
    1919        dialog = new ConflictResolutionDialog(this);
    20         dialog.setSize(600,600);
     20        dialog.setSize(600, 600);
    2121    }
    2222

  • trunk/test/functional/org/openstreetmap/josm/gui/history/HistoryBrowserTest.java

    r7937 r8510  
    2929
    3030    protected void build() {
    31         setSize(500,500);
     31        setSize(500, 500);
    3232        getContentPane().setLayout(new BorderLayout());
    3333        browser = new HistoryBrowser();
    … …  
    4040        try {
    4141            ds = reader.parseHistory(NullProgressMonitor.INSTANCE);
    42         } catch(OsmTransferException e) {
     42        } catch (OsmTransferException e) {
    4343            Main.error(e);
    4444            return;
    … …  
    5151     * Constructs a new {@code HistoryBrowserTest}.
    5252     */
    53     public HistoryBrowserTest(){
     53    public HistoryBrowserTest() {
    5454        build();
    5555        //populate(OsmPrimitiveType.NODE,354117);
    5656        //populate(OsmPrimitiveType.WAY,37951);
    57         populate(OsmPrimitiveType.RELATION,5055);
     57        populate(OsmPrimitiveType.RELATION, 5055);
    5858
    5959    }

  • trunk/test/functional/org/openstreetmap/josm/io/MultiFetchServerObjectReaderTest.java

    r8509 r8510  
    5757        // create a set of nodes
    5858        //
    59         for (int i=0; i< numNodes; i++) {
     59        for (int i = 0; i < numNodes; i++) {
    6060            Node n = new Node();
    61             n.setCoor(new LatLon(-36.6,47.6));
     61            n.setCoor(new LatLon(-36.6, 47.6));
    6262            n.put("name", "node-"+i);
    6363            ds.addPrimitive(n);
    … …  
    6565        }
    6666
    67         // create a set of ways, each with a random number of
    68         // nodes
     67        // create a set of ways, each with a random number of nodes
    6968        //
    70         for (int i=0; i< numWays; i++) {
     69        for (int i = 0; i < numWays; i++) {
    7170            Way w = new Way();
    72             int numNodesInWay = 2 + (int)Math.round(Math.random() * 5);
    73             int start = (int)Math.round(Math.random() * numNodes);
    74             for (int j = 0; j < numNodesInWay;j++) {
     71            int numNodesInWay = 2 + (int) Math.round(Math.random() * 5);
     72            int start = (int) Math.round(Math.random() * numNodes);
     73            for (int j = 0; j < numNodesInWay; j++) {
    7574                int idx = (start + j) % numNodes;
    7675                Node n = nodes.get(idx);
    … …  
    8281        }
    8382
    84         // create a set of relations each with a random number of nodes,
    85         // and ways
     83        // create a set of relations each with a random number of nodes, and ways
    8684        //
    87         for (int i=0; i< numRelations; i++) {
     85        for (int i = 0; i < numRelations; i++) {
    8886            Relation r = new Relation();
    8987            r.put("name", "relation-" +i);
    90             int numNodesInRelation = (int)Math.round(Math.random() * 10);
    91             int start = (int)Math.round(Math.random() * numNodes);
    92             for (int j = 0; j < numNodesInRelation;j++) {
     88            int numNodesInRelation = (int) Math.round(Math.random() * 10);
     89            int start = (int) Math.round(Math.random() * numNodes);
     90            for (int j = 0; j < numNodesInRelation; j++) {
    9391                int idx = (start + j) % 500;
    9492                Node n = nodes.get(idx);
    9593                r.addMember(new RelationMember("role-" + j, n));
    9694            }
    97             int numWaysInRelation = (int)Math.round(Math.random() * 10);
    98             start = (int)Math.round(Math.random() * numWays);
    99             for (int j = 0; j < numWaysInRelation;j++) {
     95            int numWaysInRelation = (int) Math.round(Math.random() * 10);
     96            start = (int) Math.round(Math.random() * numWays);
     97            for (int j = 0; j < numWaysInRelation; j++) {
    10098                int idx = (start + j) % 500;
    10199                Way w = ways.get(idx);
    … …  
    167165                w.footer();
    168166            }
    169         } catch(IOException e) {
     167        } catch (IOException e) {
    170168            fail(MessageFormat.format("failed to open file ''{0}'' for writing", dataSetCacheOutputFile.toString()));
    171169        }
    … …  
    191189        MultiFetchServerObjectReader reader = new MultiFetchServerObjectReader();
    192190        ArrayList<Node> nodes = new ArrayList<>(ds.getNodes());
    193         for (int i =0; i< 10; i++) {
     191        for (int i = 0; i < 10; i++) {
    194192            reader.append(nodes.get(i));
    195193        }
    … …  
    197195        assertEquals(10, out.getNodes().size());
    198196        for (Node n1:out.getNodes()) {
    199             Node n2 = (Node)ds.getPrimitiveById(n1);
     197            Node n2 = (Node) ds.getPrimitiveById(n1);
    200198            assertNotNull(n2);
    201             assertEquals(n2.get("name"),n2.get("name"));
     199            assertEquals(n2.get("name"), n2.get("name"));
    202200        }
    203201        assertTrue(reader.getMissingPrimitives().isEmpty());
    … …  
    207205    public void testMultiGet10Ways() throws OsmTransferException {
    208206        MultiFetchServerObjectReader reader = new MultiFetchServerObjectReader();
    209         ArrayList<Way> ways= new ArrayList<>(ds.getWays());
    210         for (int i =0; i< 10; i++) {
     207        ArrayList<Way> ways = new ArrayList<>(ds.getWays());
     208        for (int i = 0; i < 10; i++) {
    211209            reader.append(ways.get(i));
    212210        }
    … …  
    214212        assertEquals(10, out.getWays().size());
    215213        for (Way w1: out.getWays()) {
    216             Way w2 = (Way)ds.getPrimitiveById(w1);
     214            Way w2 = (Way) ds.getPrimitiveById(w1);
    217215            assertNotNull(w2);
    218216            assertEquals(w2.getNodesCount(), w1.getNodesCount());
    219             assertEquals(w2.get("name"),w1.get("name"));
     217            assertEquals(w2.get("name"), w1.get("name"));
    220218        }
    221219        assertTrue(reader.getMissingPrimitives().isEmpty());
    … …  
    225223    public void testMultiGet10Relations() throws OsmTransferException {
    226224        MultiFetchServerObjectReader reader = new MultiFetchServerObjectReader();
    227         ArrayList<Relation> relations= new ArrayList<>(ds.getRelations());
    228         for (int i =0; i< 10; i++) {
     225        ArrayList<Relation> relations = new ArrayList<>(ds.getRelations());
     226        for (int i = 0; i < 10; i++) {
    229227            reader.append(relations.get(i));
    230228        }
    … …  
    232230        assertEquals(10, out.getRelations().size());
    233231        for (Relation r1: out.getRelations()) {
    234             Relation r2 = (Relation)ds.getPrimitiveById(r1);
     232            Relation r2 = (Relation) ds.getPrimitiveById(r1);
    235233            assertNotNull(r2);
    236234            assertEquals(r2.getMembersCount(), r1.getMembersCount());
    237             assertEquals(r2.get("name"),r2.get("name"));
     235            assertEquals(r2.get("name"), r2.get("name"));
    238236        }
    239237        assertTrue(reader.getMissingPrimitives().isEmpty());
    … …  
    244242        MultiFetchServerObjectReader reader = new MultiFetchServerObjectReader();
    245243        ArrayList<Node> nodes = new ArrayList<>(ds.getNodes());
    246         for (int i =0; i< 812; i++) {
     244        for (int i = 0; i < 812; i++) {
    247245            reader.append(nodes.get(i));
    248246        }
    … …  
    250248        assertEquals(812, out.getNodes().size());
    251249        for (Node n1:out.getNodes()) {
    252             Node n2 = (Node)ds.getPrimitiveById(n1);
     250            Node n2 = (Node) ds.getPrimitiveById(n1);
    253251            assertNotNull(n2);
    254             assertEquals(n2.get("name"),n2.get("name"));
     252            assertEquals(n2.get("name"), n2.get("name"));
    255253        }
    256254        assertTrue(reader.getMissingPrimitives().isEmpty());
    … …  
    261259        MultiFetchServerObjectReader reader = new MultiFetchServerObjectReader();
    262260        ArrayList<Node> nodes = new ArrayList<>(ds.getNodes());
    263         for (int i =0; i< 10; i++) {
     261        for (int i = 0; i < 10; i++) {
    264262            reader.append(nodes.get(i));
    265263        }
    … …  
    269267        assertEquals(10, out.getNodes().size());
    270268        for (Node n1:out.getNodes()) {
    271             Node n2 = (Node)ds.getPrimitiveById(n1);
     269            Node n2 = (Node) ds.getPrimitiveById(n1);
    272270            assertNotNull(n2);
    273             assertEquals(n2.get("name"),n2.get("name"));
     271            assertEquals(n2.get("name"), n2.get("name"));
    274272        }
    275273        assertFalse(reader.getMissingPrimitives().isEmpty());

  • trunk/test/functional/org/openstreetmap/josm/io/OsmServerBackreferenceReaderTest.java

    r8509 r8510  
    7373
    7474    protected static void populateTestDataSetWithNodes(DataSet ds) {
    75         for (int i=0;i<100;i++) {
     75        for (int i = 0; i < 100; i++) {
    7676            Node n = new Node();
    77             n.setCoor(new LatLon(-36.6,47.6));
     77            n.setCoor(new LatLon(-36.6, 47.6));
    7878            n.put("name", "node-"+i);
    7979            ds.addPrimitive(n);
    … …  
    8282
    8383    protected static void populateTestDataSetWithWays(DataSet ds) {
    84         for (int i=0;i<20;i++) {
     84        for (int i = 0; i < 20; i++) {
    8585            Way w = new Way();
    86             for (int j = 0; j < 10;j++) {
     86            for (int j = 0; j < 10; j++) {
    8787                w.addNode(lookupNode(ds, i+j));
    8888            }
    … …  
    9393
    9494    protected static void populateTestDataSetWithRelations(DataSet ds) {
    95         for (int i=0;i<10;i++) {
     95        for (int i = 0; i < 10; i++) {
    9696            Relation r = new Relation();
    9797            r.put("name", "relation-" +i);
    98             for (int j =0; j < 10; j++) {
     98            for (int j = 0; j < 10; j++) {
    9999                RelationMember member = new RelationMember("node-" + j, lookupNode(ds, i + j));
    100100                r.addMember(member);
    101101            }
    102             for (int j =0; j < 5; j++) {
     102            for (int j = 0; j < 5; j++) {
    103103                RelationMember member = new RelationMember("way-" + j, lookupWay(ds, i + j));
    104104                r.addMember(member);
    105105            }
    106106            if (i > 5) {
    107                 for (int j =0; j < 3; j++) {
     107                for (int j = 0; j < 3; j++) {
    108108                    RelationMember member = new RelationMember("relation-" + j, lookupRelation(ds, j));
    109109                    logger.info(MessageFormat.format("adding relation {0} to relation {1}", j, i));
    … …  
    185185                w.footer();
    186186            }
    187         } catch(IOException e) {
     187        } catch (IOException e) {
    188188            fail(MessageFormat.format("failed to open file ''{0}'' for writing", dataSetCacheOutputFile.toString()));
    189189        }
    … …  
    456456    protected static Set<Long> getNodeIdsInWay(Way way) {
    457457        HashSet<Long> ret = new HashSet<>();
    458         if (way == null)return ret;
     458        if (way == null) return ret;
    459459        for (Node n: way.getNodes()) {
    460460            ret.add(n.getId());

  • trunk/test/functional/org/openstreetmap/josm/io/UploadStrategySelectionPanelTest.java

    r7937 r8510  
    2727        getContentPane().add(uploadStrategySelectionPanel, BorderLayout.CENTER);
    2828        getContentPane().add(buildControlPanel(), BorderLayout.SOUTH);
    29         setSize(400,400);
     29        setSize(400, 400);
    3030    }
    31 
    3231
    3332    protected JPanel buildControlPanel() {
    … …  
    4241                        try {
    4342                            n = Integer.parseInt(tf.getText());
    44                         } catch(NumberFormatException e) {
     43                        } catch (NumberFormatException e) {
    4544                            Main.error(e);
    4645                            return;
    … …  
    6160    }
    6261
    63     public static void main(String args[]) throws OsmApiInitializationException, OsmTransferCanceledException{
     62    public static void main(String args[]) throws OsmApiInitializationException, OsmTransferCanceledException {
    6463        OsmApi.getOsmApi().initialize(NullProgressMonitor.INSTANCE);
    6564        new UploadStrategySelectionPanelTest().setVisible(true);
Note: See TracChangeset for help on using the changeset viewer.