Changeset 9942 in josm


Ignore:
Timestamp:
2016-03-06T22:42:46+01:00 (4 years ago)
Author:
Don-vip
Message:

fix problems seen in unit tests when launched manually from Eclipse

Location:
trunk
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/gui/history/HistoryBrowserModel.java

    r9734 r9942  
    710710    @Override
    711711    public void dataChanged(DataChangedEvent event) {
     712        if (history == null)
     713            return;
    712714        OsmPrimitive primitive = event.getDataset().getPrimitiveById(history.getId(), history.getType());
    713715        HistoryOsmPrimitive latest;
  • trunk/test/unit/org/openstreetmap/josm/gui/JosmUserIdentityManagerTest.groovy

    r9527 r9942  
    11// License: GPL. For details, see LICENSE file.
    22package org.openstreetmap.josm.gui
    3 
    4 import org.openstreetmap.josm.data.osm.User;
    53
    64import static org.junit.Assert.*
     
    108import org.openstreetmap.josm.JOSMFixture
    119import org.openstreetmap.josm.Main
     10import org.openstreetmap.josm.data.osm.User
    1211import org.openstreetmap.josm.data.osm.UserInfo
    1312
     
    3938
    4039    @Test
    41     public void test_setAnonymouse() {
     40    public void test_setAnonymous() {
    4241        JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
    4342
     
    120119
    121120    /**
    122      * Preferences include neither an url nor a user name => we have
    123      * an anonymous user
     121     * Preferences include neither an url nor a user name => we have an anonymous user
    124122     */
    125123    @Test
     
    140138
    141139    /**
    142      * Preferences include neither an url nor a user name => we have
    143      * an annoymous user
     140     * Preferences include neither an url nor a user name => we have an anonymous user
    144141     */
    145142    @Test
     
    154151        Main.pref.removePreferenceChangeListener im
    155152
    156         Main.pref.put "osm-server.url", "http://api.openstreetmap.org"
    157         Main.pref.put "osm-server.username", null
    158 
    159         im.initFromPreferences()
    160 
    161         assert im.isAnonymous()
     153        try {
     154            Main.pref.put "osm-server.url", "http://api.openstreetmap.org"
     155            Main.pref.put "osm-server.username", null
     156
     157            im.initFromPreferences()
     158
     159            assert im.isAnonymous()
     160        } finally {
     161            Main.pref.addPreferenceChangeListener im
     162        }
    162163    }
    163164
     
    172173        Main.pref.removePreferenceChangeListener im
    173174
    174         // reset it
    175         im.@userName = null
    176         im.@userInfo = null
    177 
    178         Main.pref.put "osm-server.url", "http://api.openstreetmap.org"
    179         Main.pref.put "osm-server.username", "test"
    180 
    181         im.initFromPreferences()
    182 
    183         assert im.isPartiallyIdentified()
     175        try {
     176            // reset it
     177            im.@userName = null
     178            im.@userInfo = null
     179
     180            Main.pref.put "osm-server.url", "http://api.openstreetmap.org"
     181            Main.pref.put "osm-server.username", "test"
     182
     183            im.initFromPreferences()
     184
     185            assert im.isPartiallyIdentified()
     186        } finally {
     187            Main.pref.addPreferenceChangeListener im
     188        }
    184189    }
    185190
     
    196201        Main.pref.removePreferenceChangeListener im
    197202
    198         im.setFullyIdentified "test1", new UserInfo(id: 1)
    199 
    200         Main.pref.put "osm-server.url", "http://api.openstreetmap.org"
    201         Main.pref.put "osm-server.username", "test2"
    202 
    203         im.initFromPreferences()
    204 
    205         assert im.isPartiallyIdentified()
     203        try {
     204            im.setFullyIdentified "test1", new UserInfo(id: 1)
     205
     206            Main.pref.put "osm-server.url", "http://api.openstreetmap.org"
     207            Main.pref.put "osm-server.username", "test2"
     208
     209            im.initFromPreferences()
     210
     211            assert im.isPartiallyIdentified()
     212        } finally {
     213            Main.pref.addPreferenceChangeListener im
     214        }
    206215    }
    207216
     
    218227        Main.pref.removePreferenceChangeListener im
    219228
    220         im.setFullyIdentified "test1", new UserInfo(id: 1)
    221 
    222         Main.pref.put "osm-server.url", "http://api.openstreetmap.org"
    223         Main.pref.put "osm-server.username", "test1"
    224 
    225         im.initFromPreferences()
    226 
    227         assert im.isFullyIdentified()
     229        try {
     230            im.setFullyIdentified "test1", new UserInfo(id: 1)
     231
     232            Main.pref.put "osm-server.url", "http://api.openstreetmap.org"
     233            Main.pref.put "osm-server.username", "test1"
     234
     235            im.initFromPreferences()
     236
     237            assert im.isFullyIdentified()
     238        } finally {
     239            Main.pref.addPreferenceChangeListener im
     240        }
    228241    }
    229242
  • trunk/test/unit/org/openstreetmap/josm/io/session/SessionReaderTest.java

    r9753 r9942  
    1414import org.junit.Test;
    1515import org.openstreetmap.josm.JOSMFixture;
     16import org.openstreetmap.josm.Main;
    1617import org.openstreetmap.josm.TestUtils;
    1718import org.openstreetmap.josm.gui.layer.GpxLayer;
     
    138139    @Test
    139140    public void testReadNotes() throws IOException, IllegalDataException {
     141        if (Main.isDisplayingMapView()) {
     142            for (NoteLayer nl : Main.map.mapView.getLayersOfType(NoteLayer.class)) {
     143                Main.map.mapView.removeLayer(nl);
     144            }
     145        }
    140146        final List<Layer> layers = testRead("notes.joz");
    141147        assertSame(layers.size(), 1);
Note: See TracChangeset for help on using the changeset viewer.