Changeset 18444 in josm for trunk/test


Ignore:
Timestamp:
2022-05-10T18:54:08+02:00 (3 years ago)
Author:
taylor.smock
Message:

Fix #21775: Remove deprecated code in ImageData

The code in question has been deprecated since r18246 (2021-10-04).

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/test/unit/org/openstreetmap/josm/data/ImageDataTest.java

    r18049 r18444  
    4545        assertEquals(0, data.getImages().size());
    4646        assertNull(data.getSelectedImage());
    47         data.selectFirstImage();
     47        data.setSelectedImage(data.getFirstImage());
    4848        assertNull(data.getSelectedImage());
    49         data.selectLastImage();
     49        data.setSelectedImage(data.getLastImage());
    5050        assertNull(data.getSelectedImage());
    51         data.selectFirstImage();
     51        data.setSelectedImage(data.getFirstImage());
    5252        assertNull(data.getSelectedImage());
    53         data.selectPreviousImage();
     53        data.setSelectedImage(data.getPreviousImage());
    5454        assertNull(data.getSelectedImage());
    5555        assertFalse(data.hasNextImage());
    … …  
    9696
    9797        ImageData data = new ImageData(list);
    98         data.selectFirstImage();
     98        data.setSelectedImage(data.getFirstImage());
    9999        assertEquals(1, data.getSelectedImages().size());
    100100        assertEquals(list.get(0), data.getSelectedImages().get(0));
    … …  
    107107
    108108        ImageData data = new ImageData(list);
    109         data.selectLastImage();
     109        data.setSelectedImage(data.getLastImage());
    110110        assertEquals(1, data.getSelectedImages().size());
    111111        assertEquals(list.get(1), data.getSelectedImages().get(0));
    … …  
    118118        ImageData data = new ImageData(list);
    119119        assertTrue(data.hasNextImage());
    120         data.selectNextImage();
     120        data.setSelectedImage(data.getNextImage());
    121121        assertEquals(1, data.getSelectedImages().size());
    122122        assertEquals(list.get(0), data.getSelectedImages().get(0));
    123123        assertFalse(data.hasNextImage());
    124         data.selectNextImage();
    125         assertEquals(list.get(0), data.getSelectedImages().get(0));
     124        data.setSelectedImage(data.getNextImage());
     125        assertTrue(data.getSelectedImages().isEmpty());
    126126    }
    127127
    … …  
    133133        ImageData data = new ImageData(list);
    134134        assertFalse(data.hasPreviousImage());
    135         data.selectLastImage();
     135        data.setSelectedImage(data.getLastImage());
    136136        assertTrue(data.hasPreviousImage());
    137         data.selectPreviousImage();
    138         assertEquals(1, data.getSelectedImages().size());
    139         assertEquals(list.get(0), data.getSelectedImages().get(0));
    140         data.selectPreviousImage();
    141         assertEquals(list.get(0), data.getSelectedImages().get(0));
     137        data.setSelectedImage(data.getPreviousImage());
     138        assertEquals(1, data.getSelectedImages().size());
     139        assertEquals(list.get(0), data.getSelectedImages().get(0));
     140        data.setSelectedImage(data.getPreviousImage());
     141        assertTrue(data.getSelectedImages().isEmpty());
    142142    }
    143143
    … …  
    177177        }};
    178178        data.addImageDataUpdateListener(listener);
    179         data.selectFirstImage();
    180         data.selectFirstImage();
     179        data.setSelectedImage(data.getFirstImage());
     180        data.setSelectedImage(data.getFirstImage());
    181181    }
    182182
    … …  
    187187
    188188        ImageData data = new ImageData(list);
    189         data.selectFirstImage();
     189        data.setSelectedImage(data.getFirstImage());
    190190        data.addImageToSelection(list.get(1));
    191191        data.removeSelectedImages();
    … …  
    199199        list.add(new ImageEntry());
    200200        list.add(new ImageEntry());
    201 
    202         ImageData data = new ImageData(list);
    203         data.selectLastImage();
     201        // The images cannot be equal -- otherwise, some code gives us the wrong result.
     202        list.get(1).setPos(LatLon.NORTH_POLE);
     203
     204        ImageData data = new ImageData(list);
     205        data.setSelectedImage(data.getLastImage());
    204206        data.addImageToSelection(list.get(1));
    205207        data.removeSelectedImages();
    … …  
    214216
    215217        ImageData data = new ImageData(list);
    216         data.selectFirstImage();
     218        data.setSelectedImage(data.getFirstImage());
    217219        data.removeSelectedImages();
    218220        assertEquals(1, data.getImages().size());
    … …  
    227229
    228230        ImageData data = new ImageData(list);
    229         data.selectLastImage();
     231        data.setSelectedImage(data.getLastImage());
    230232        data.removeSelectedImages();
    231233        assertEquals(1, data.getImages().size());
    … …  
    250252        }};
    251253        data.addImageDataUpdateListener(listener);
    252         data.selectFirstImage();
     254        data.setSelectedImage(data.getFirstImage());
    253255        data.removeSelectedImages();
    254256    }
    … …  
    270272        }};
    271273        data.addImageDataUpdateListener(listener);
    272         data.selectFirstImage();
     274        data.setSelectedImage(data.getFirstImage());
    273275        data.removeSelectedImages();
    274276    }
    … …  
    355357
    356358        ImageData data = new ImageData(list);
    357         data.selectLastImage();
     359        data.setSelectedImage(data.getLastImage());
    358360        data.removeImageToSelection(list.get(1));
    359361        assertEquals(0, data.getSelectedImages().size());
    360         data.selectFirstImage();
     362        data.setSelectedImage(data.getFirstImage());
    361363        assertEquals(1, data.getSelectedImages().size());
    362364    }
    … …  
    369371        ImageData data = new ImageData(list);
    370372        assertFalse(data.isImageSelected(list.get(0)));
    371         data.selectFirstImage();
     373        data.setSelectedImage(data.getFirstImage());
    372374        assertTrue(data.isImageSelected(list.get(0)));
    373375        data.addImageToSelection(list.get(1));
    … …  
    394396        assertEquals(0, data.getSelectedImages().size());
    395397        data.addImageToSelection(list.get(1));
    396         data.selectFirstImage();
     398        data.setSelectedImage(data.getFirstImage());
    397399        assertEquals(1, data.getSelectedImages().size());
    398400    }
    … …  
    414416        }};
    415417        data.addImageDataUpdateListener(listener);
    416         data.selectFirstImage();
     418        data.setSelectedImage(data.getFirstImage());
    417419        data.addImageToSelection(list.get(1));
    418420        data.removeImageToSelection(list.get(0));
Note: See TracChangeset for help on using the changeset viewer.