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/unit/org/openstreetmap/josm/gui
Files:
9 edited

Legend:

Unmodified
Added
Removed

  • trunk/test/unit/org/openstreetmap/josm/gui/JosmUserIdentityManagerTest.groovy

    r7938 r8510  
    1212class JosmUserIdentityManagerTest {
    1313
    14         final shouldFail = new GroovyTestCase().&shouldFail
    15 
    16         @BeforeClass
    17         public static void initTestCase() {
    18             JOSMFixture.createUnitTestFixture().init()
    19         }
    20 
    21         @Test
    22         public void test_SingletonAccess() {
    23 
    24                 JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
    25 
    26                 // created ?
    27                 assert im != null
    28 
    29                 // registered as listener ?
    30                 assert Main.pref.@listeners.contains(im)
    31 
    32                 JosmUserIdentityManager im2 = JosmUserIdentityManager.getInstance()
    33 
    34                 // only one instance
    35                 assert im == im2
    36         }
    37 
    38         @Test
    39         public void test_setAnonymouse() {
    40                 JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
    41 
    42                 im.setPartiallyIdentified "test"
    43                 im.setAnonymous()
    44 
    45                 assert im.isAnonymous()
    46                 assert ! im.isPartiallyIdentified()
    47                 assert ! im.isFullyIdentified()
    48 
    49                 assert im.getUserId() == 0
    50                 assert im.getUserName() == null
    51                 assert im.getUserInfo() == null
    52         }
    53 
    54         @Test
    55         public void test_setPartiallyIdentified() {
    56                 JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
    57 
    58                 im.setPartiallyIdentified "test"
    59 
    60                 shouldFail(IllegalArgumentException) {
    61                     im.setPartiallyIdentified null
    62                 }
    63 
    64                 shouldFail(IllegalArgumentException) {
    65                         im.setPartiallyIdentified ""
    66                 }
    67 
    68                 shouldFail(IllegalArgumentException) {
    69                         im.setPartiallyIdentified "  \t  "
    70                 }
    71 
    72                 im.setPartiallyIdentified "test"
    73 
    74                 assert ! im.isAnonymous()
    75                 assert im.isPartiallyIdentified()
    76                 assert ! im.isFullyIdentified()
    77 
    78                 assert im.getUserId() == 0
    79                 assert im.getUserName() == "test"
    80                 assert im.getUserInfo() == null
    81         }
    82 
    83 
    84         @Test
    85         public void test_setFullyIdentified() {
    86                 JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
    87 
    88                 UserInfo userInfo = new UserInfo(id: 1, description: "a description")
    89 
    90                 im.setFullyIdentified "test", userInfo
    91 
    92                 shouldFail(IllegalArgumentException) {
    93                         im.setFullyIdentified null, userInfo
    94                 }
    95                 shouldFail(IllegalArgumentException) {
    96                         im.setFullyIdentified "", userInfo
    97                 }
    98                 shouldFail(IllegalArgumentException) {
    99                         im.setFullyIdentified " \t ", userInfo
    100                 }
    101                 shouldFail(IllegalArgumentException) {
    102                         im.setFullyIdentified "test", null
    103                 }
    104 
    105                 im.setFullyIdentified "test", userInfo
    106 
    107                 assert ! im.isAnonymous()
    108                 assert ! im.isPartiallyIdentified()
    109                 assert im.isFullyIdentified()
    110 
    111                 assert im.getUserId() == 1
    112                 assert im.getUserName() == "test"
    113                 assert im.getUserInfo() == userInfo
    114         }
    115 
    116         /**
    117          * Preferences include neither an url nor a user name => we have
    118          * an anonymous user
    119          */
    120         @Test
    121         public void initFromPreferences_1() {
    122                 JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
    123 
    124                 // reset it
    125                 im.@userName = null
    126                 im.@userInfo = null
    127 
    128                 Main.pref.put "osm-server.url", null
    129                 Main.pref.put "osm-server.username", null
    130 
    131                 im.initFromPreferences()
    132 
    133                 assert im.isAnonymous()
    134         }
    135 
    136         /**
    137          * Preferences include neither an url nor a user name => we have
    138          * an annoymous user
    139          */
    140         @Test
    141         public void initFromPreferences_2() {
    142                 JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
    143 
    144                 // reset it
    145                 im.@userName = null
    146                 im.@userInfo = null
    147 
    148                 // for this test we disable the listener
    149                 Main.pref.removePreferenceChangeListener im
    150 
    151                 Main.pref.put "osm-server.url", "http://api.openstreetmap.org"
    152                 Main.pref.put "osm-server.username", null
    153 
    154                 im.initFromPreferences()
    155 
    156                 assert im.isAnonymous()
    157         }
    158 
    159         /**
    160          * Preferences include an user name => we have a partially identified user
    161          */
    162         @Test
    163         public void initFromPreferences_3() {
    164                 JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
     14    final shouldFail = new GroovyTestCase().&shouldFail
     15
     16    @BeforeClass
     17    public static void initTestCase() {
     18        JOSMFixture.createUnitTestFixture().init()
     19    }
     20
     21    @Test
     22    public void test_SingletonAccess() {
     23
     24        JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
     25
     26        // created ?
     27        assert im != null
     28
     29        // registered as listener ?
     30        assert Main.pref.@listeners.contains(im)
     31
     32        JosmUserIdentityManager im2 = JosmUserIdentityManager.getInstance()
     33
     34        // only one instance
     35        assert im == im2
     36    }
     37
     38    @Test
     39    public void test_setAnonymouse() {
     40        JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
     41
     42        im.setPartiallyIdentified "test"
     43        im.setAnonymous()
     44
     45        assert im.isAnonymous()
     46        assert ! im.isPartiallyIdentified()
     47        assert ! im.isFullyIdentified()
     48
     49        assert im.getUserId() == 0
     50        assert im.getUserName() == null
     51        assert im.getUserInfo() == null
     52    }
     53
     54    @Test
     55    public void test_setPartiallyIdentified() {
     56        JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
     57
     58        im.setPartiallyIdentified "test"
     59
     60        shouldFail(IllegalArgumentException) {
     61            im.setPartiallyIdentified null
     62        }
     63
     64        shouldFail(IllegalArgumentException) {
     65            im.setPartiallyIdentified ""
     66        }
     67
     68        shouldFail(IllegalArgumentException) {
     69            im.setPartiallyIdentified "  \t  "
     70        }
     71
     72        im.setPartiallyIdentified "test"
     73
     74        assert ! im.isAnonymous()
     75        assert im.isPartiallyIdentified()
     76        assert ! im.isFullyIdentified()
     77
     78        assert im.getUserId() == 0
     79        assert im.getUserName() == "test"
     80        assert im.getUserInfo() == null
     81    }
     82
     83
     84    @Test
     85    public void test_setFullyIdentified() {
     86        JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
     87
     88        UserInfo userInfo = new UserInfo(id: 1, description: "a description")
     89
     90        im.setFullyIdentified "test", userInfo
     91
     92        shouldFail(IllegalArgumentException) {
     93            im.setFullyIdentified null, userInfo
     94        }
     95        shouldFail(IllegalArgumentException) {
     96            im.setFullyIdentified "", userInfo
     97        }
     98        shouldFail(IllegalArgumentException) {
     99            im.setFullyIdentified " \t ", userInfo
     100        }
     101        shouldFail(IllegalArgumentException) {
     102            im.setFullyIdentified "test", null
     103        }
     104
     105        im.setFullyIdentified "test", userInfo
     106
     107        assert ! im.isAnonymous()
     108        assert ! im.isPartiallyIdentified()
     109        assert im.isFullyIdentified()
     110
     111        assert im.getUserId() == 1
     112        assert im.getUserName() == "test"
     113        assert im.getUserInfo() == userInfo
     114    }
     115
     116    /**
     117     * Preferences include neither an url nor a user name => we have
     118     * an anonymous user
     119     */
     120    @Test
     121    public void initFromPreferences_1() {
     122        JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
     123
     124        // reset it
     125        im.@userName = null
     126        im.@userInfo = null
     127
     128        Main.pref.put "osm-server.url", null
     129        Main.pref.put "osm-server.username", null
     130
     131        im.initFromPreferences()
     132
     133        assert im.isAnonymous()
     134    }
     135
     136    /**
     137     * Preferences include neither an url nor a user name => we have
     138     * an annoymous user
     139     */
     140    @Test
     141    public void initFromPreferences_2() {
     142        JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
     143
     144        // reset it
     145        im.@userName = null
     146        im.@userInfo = null
    165147
    166148        // for this test we disable the listener
    167                 Main.pref.removePreferenceChangeListener im
    168 
    169                 // reset it
    170                 im.@userName = null
    171                 im.@userInfo = null
    172 
    173                 Main.pref.put "osm-server.url", "http://api.openstreetmap.org"
    174                 Main.pref.put "osm-server.username", "test"
    175 
    176                 im.initFromPreferences()
    177 
    178                 assert im.isPartiallyIdentified()
    179         }
    180 
    181         /**
    182          * Preferences include an user name which is different from the current
    183          * user name and we are currently fully identifed => josm user becomes
    184          * partially identified
    185          */
    186         @Test
    187         public void initFromPreferences_4() {
    188                 JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
     149        Main.pref.removePreferenceChangeListener im
     150
     151        Main.pref.put "osm-server.url", "http://api.openstreetmap.org"
     152        Main.pref.put "osm-server.username", null
     153
     154        im.initFromPreferences()
     155
     156        assert im.isAnonymous()
     157    }
     158
     159    /**
     160     * Preferences include an user name => we have a partially identified user
     161     */
     162    @Test
     163    public void initFromPreferences_3() {
     164        JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
    189165
    190166        // for this test we disable the listener
    191                 Main.pref.removePreferenceChangeListener im
    192 
    193                 im.setFullyIdentified "test1", new UserInfo(id: 1)
    194 
    195                 Main.pref.put "osm-server.url", "http://api.openstreetmap.org"
    196                 Main.pref.put "osm-server.username", "test2"
    197 
    198                 im.initFromPreferences()
    199 
    200                 assert im.isPartiallyIdentified()
    201         }
    202 
    203         /**
    204          * Preferences include an user name which is the same as the current
    205          * user name and we are currently fully identifed => josm user remains
    206          * fully identified
    207          */
    208         @Test
    209         public void initFromPreferences_5() {
    210                 JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
     167        Main.pref.removePreferenceChangeListener im
     168
     169        // reset it
     170        im.@userName = null
     171        im.@userInfo = null
     172
     173        Main.pref.put "osm-server.url", "http://api.openstreetmap.org"
     174        Main.pref.put "osm-server.username", "test"
     175
     176        im.initFromPreferences()
     177
     178        assert im.isPartiallyIdentified()
     179    }
     180
     181    /**
     182     * Preferences include an user name which is different from the current
     183     * user name and we are currently fully identifed => josm user becomes
     184     * partially identified
     185     */
     186    @Test
     187    public void initFromPreferences_4() {
     188        JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
    211189
    212190        // for this test we disable the listener
    213                 Main.pref.removePreferenceChangeListener im
    214 
    215                 im.setFullyIdentified "test1", new UserInfo(id: 1)
    216 
    217                 Main.pref.put "osm-server.url", "http://api.openstreetmap.org"
    218                 Main.pref.put "osm-server.username", "test1"
    219 
    220                 im.initFromPreferences()
    221 
    222                 assert im.isFullyIdentified()
    223         }
    224 
    225         @Test
    226         public void apiUrlChanged() {
    227                 JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
    228 
    229                 // make sure im is a preference change listener
    230                 Main.pref.addPreferenceChangeListener im
    231 
    232                 // reset it
    233                 im.@userName = null
    234                 im.@userInfo = null
    235 
    236                 Main.pref.put "osm-server.url", "http://api.openstreetmap.org"
    237                 assert im.isAnonymous()
     191        Main.pref.removePreferenceChangeListener im
     192
     193        im.setFullyIdentified "test1", new UserInfo(id: 1)
     194
     195        Main.pref.put "osm-server.url", "http://api.openstreetmap.org"
     196        Main.pref.put "osm-server.username", "test2"
     197
     198        im.initFromPreferences()
     199
     200        assert im.isPartiallyIdentified()
     201    }
     202
     203    /**
     204     * Preferences include an user name which is the same as the current
     205     * user name and we are currently fully identifed => josm user remains
     206     * fully identified
     207     */
     208    @Test
     209    public void initFromPreferences_5() {
     210        JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
     211
     212        // for this test we disable the listener
     213        Main.pref.removePreferenceChangeListener im
     214
     215        im.setFullyIdentified "test1", new UserInfo(id: 1)
     216
     217        Main.pref.put "osm-server.url", "http://api.openstreetmap.org"
     218        Main.pref.put "osm-server.username", "test1"
     219
     220        im.initFromPreferences()
     221
     222        assert im.isFullyIdentified()
     223    }
     224
     225    @Test
     226    public void apiUrlChanged() {
     227        JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
     228
     229        // make sure im is a preference change listener
     230        Main.pref.addPreferenceChangeListener im
     231
     232        // reset it
     233        im.@userName = null
     234        im.@userInfo = null
     235
     236        Main.pref.put "osm-server.url", "http://api.openstreetmap.org"
     237        assert im.isAnonymous()
    238238
    239239         Main.pref.put "osm-server.url", null
    240240         assert im.isAnonymous()
    241241
    242                 // reset it
    243                 im.@userName = "test"
    244                 im.@userInfo = null
    245 
    246                 Main.pref.put "osm-server.url", "http://api.openstreetmap.org"
    247                 assert im.isPartiallyIdentified()
    248                 assert im.getUserName() == "test"
    249 
    250                 Main.pref.put "osm-server.url", null
    251                 assert im.isAnonymous()
    252 
    253                 // reset it
    254                 im.@userName = "test"
    255                 im.@userInfo = new UserInfo(id:1)
    256 
    257                 Main.pref.put "osm-server.url", "http://api.openstreetmap.org"
    258                 assert im.isPartiallyIdentified()
    259                 assert im.getUserName() == "test"
    260 
    261                 // reset it
    262                 im.@userName = "test"
    263                 im.@userInfo = new UserInfo(id:1)
    264 
    265 
    266                 Main.pref.put "osm-server.url", null
    267                 assert im.isAnonymous()
    268         }
    269 
    270         @Test
    271         public void userNameChanged() {
    272                 JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
    273 
    274                 // make sure im is a preference change listener
    275                 Main.pref.addPreferenceChangeListener im
    276 
    277                 // reset it
    278                 im.@userName = null
    279                 im.@userInfo = null
    280 
    281                 Main.pref.put "osm-server.username", "test"
    282                 assert im.isPartiallyIdentified()
    283                 assert im.getUserName() == "test"
    284 
    285                 Main.pref.put "osm-server.username", null
    286                 assert im.isAnonymous()
    287 
    288                 // reset it
    289                 im.@userName = "test1"
    290                 im.@userInfo = null
    291 
    292                 Main.pref.put "osm-server.username", "test2"
    293                 assert im.isPartiallyIdentified()
    294                 assert im.getUserName() == "test2"
    295 
    296                 Main.pref.put "osm-server.username", null
    297                 assert im.isAnonymous()
    298 
    299                 // reset it
    300                 im.@userName = "test1"
    301                 im.@userInfo = new UserInfo(id:1)
    302 
    303                 Main.pref.put "osm-server.username", "test2"
    304                 assert im.isPartiallyIdentified()
    305                 assert im.getUserName() == "test2"
    306 
    307                 // reset it
    308                 im.@userName = "test1"
    309                 im.@userInfo = new UserInfo(id:1)
    310 
    311 
    312                 Main.pref.put "osm-server.username", null
    313                 assert im.isAnonymous()
    314         }
     242        // reset it
     243        im.@userName = "test"
     244        im.@userInfo = null
     245
     246        Main.pref.put "osm-server.url", "http://api.openstreetmap.org"
     247        assert im.isPartiallyIdentified()
     248        assert im.getUserName() == "test"
     249
     250        Main.pref.put "osm-server.url", null
     251        assert im.isAnonymous()
     252
     253        // reset it
     254        im.@userName = "test"
     255        im.@userInfo = new UserInfo(id:1)
     256
     257        Main.pref.put "osm-server.url", "http://api.openstreetmap.org"
     258        assert im.isPartiallyIdentified()
     259        assert im.getUserName() == "test"
     260
     261        // reset it
     262        im.@userName = "test"
     263        im.@userInfo = new UserInfo(id:1)
     264
     265
     266        Main.pref.put "osm-server.url", null
     267        assert im.isAnonymous()
     268    }
     269
     270    @Test
     271    public void userNameChanged() {
     272        JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
     273
     274        // make sure im is a preference change listener
     275        Main.pref.addPreferenceChangeListener im
     276
     277        // reset it
     278        im.@userName = null
     279        im.@userInfo = null
     280
     281        Main.pref.put "osm-server.username", "test"
     282        assert im.isPartiallyIdentified()
     283        assert im.getUserName() == "test"
     284
     285        Main.pref.put "osm-server.username", null
     286        assert im.isAnonymous()
     287
     288        // reset it
     289        im.@userName = "test1"
     290        im.@userInfo = null
     291
     292        Main.pref.put "osm-server.username", "test2"
     293        assert im.isPartiallyIdentified()
     294        assert im.getUserName() == "test2"
     295
     296        Main.pref.put "osm-server.username", null
     297        assert im.isAnonymous()
     298
     299        // reset it
     300        im.@userName = "test1"
     301        im.@userInfo = new UserInfo(id:1)
     302
     303        Main.pref.put "osm-server.username", "test2"
     304        assert im.isPartiallyIdentified()
     305        assert im.getUserName() == "test2"
     306
     307        // reset it
     308        im.@userName = "test1"
     309        im.@userInfo = new UserInfo(id:1)
     310
     311
     312        Main.pref.put "osm-server.username", null
     313        assert im.isAnonymous()
     314    }
    315315}

  • 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) {

  • trunk/test/unit/org/openstreetmap/josm/gui/dialogs/relation/sort/RelationSorterTest.java

    r7937 r8510  
    4343    private String[] getNames(List<RelationMember> members) {
    4444        String[] result = new String[members.size()];
    45         for (int i=0; i<result.length; i++) {
     45        for (int i = 0; i < result.length; i++) {
    4646            result[i] = members.get(i).getMember().get("name");
    4747        }

  • trunk/test/unit/org/openstreetmap/josm/gui/dialogs/relation/sort/WayConnectionTypeCalculatorTest.java

    r7937 r8510  
    4444    private String getConnections(List<WayConnectionType> connections) {
    4545        String[] result = new String[connections.size()];
    46         for (int i=0; i<result.length; i++) {
     46        for (int i = 0; i < result.length; i++) {
    4747            WayConnectionType wc = connections.get(i);
    4848

  • trunk/test/unit/org/openstreetmap/josm/gui/preferences/ToolbarPreferencesTest.java

    r8509 r8510  
    4444    private void checkAction(ActionDefinition a, Object... params) {
    4545        Map<String, Object> expected = new HashMap<>();
    46         for (int i=0; i<params.length; i+=2) {
    47             expected.put((String)params[i], params[i+1]);
     46        for (int i = 0; i < params.length; i += 2) {
     47            expected.put((String) params[i], params[i+1]);
    4848        }
    4949        Assert.assertEquals(expected, a.getParameters());

  • trunk/test/unit/org/openstreetmap/josm/gui/tagging/TaggingPresetReaderTest.java

    r7937 r8510  
    6767        String presetfile = "resource://data/defaultpresets.xml";
    6868        final Collection<TaggingPreset> presets = TaggingPresetReader.readAll(presetfile, true);
    69         Assert.assertTrue("Default presets are empty", presets.size()>0);
     69        Assert.assertTrue("Default presets are empty", presets.size() > 0);
    7070    }
    7171}
Note: See TracChangeset for help on using the changeset viewer.