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

checkstyle: enable relevant whitespace checks and fix them

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/test/unit/org/openstreetmap/josm/io/DiffResultProcessorTest.groovy

    r8351 r8510  
    1717class DiffResultProcessorTest {
    1818
     19    @Test
     20    public void testConstructor() {
     21        Node n = new Node(1)
     22        // these calls should not fail
     23        //
     24        def DiffResultProcessor processor  = new DiffResultProcessor(null)
     25        processor  = new DiffResultProcessor([])
     26        processor  = new DiffResultProcessor([n])
     27    }
    1928
    20         @Test
    21         public void testConstructor() {
    22                 Node n = new Node(1)
    23                 // these calls should not fail
    24                 //
    25                 def DiffResultProcessor processor  = new DiffResultProcessor(null)
    26                 processor  = new DiffResultProcessor([])
    27                 processor  = new DiffResultProcessor([n])
    28         }
     29    @Test
     30    public void testParse_NOK_Cases() {
     31        def DiffResultProcessor processor  = new DiffResultProcessor([])
    2932
    30         @Test
    31         public void testParse_NOK_Cases() {
    32                 def DiffResultProcessor processor  = new DiffResultProcessor([])
     33        final shouldFail = new GroovyTestCase().&shouldFail
    3334
    34                 final shouldFail = new GroovyTestCase().&shouldFail
     35        shouldFail(IllegalArgumentException) {
     36            processor.parse null, NullProgressMonitor.INSTANCE
     37        }
    3538
    36                 shouldFail(IllegalArgumentException) {
    37                     processor.parse null, NullProgressMonitor.INSTANCE
    38                 }
     39        shouldFail(XmlParsingException) {
     40            processor.parse "", NullProgressMonitor.INSTANCE
     41        }
    3942
    40                 shouldFail(XmlParsingException) {
    41                         processor.parse "", NullProgressMonitor.INSTANCE
    42                 }
     43        shouldFail(XmlParsingException) {
     44            processor.parse "<x></x>", NullProgressMonitor.INSTANCE
     45        }
     46    }
    4347
    44                 shouldFail(XmlParsingException) {
    45                         processor.parse "<x></x>", NullProgressMonitor.INSTANCE
    46                 }
    47         }
    48 
    49         @Test
    50         public void testParse_OK_Cases() {
    51                 def DiffResultProcessor processor  = new DiffResultProcessor([])
    52                 String doc = """\
    53                 <diffResult version="0.6" generator="Test Data">
    54                     <node old_id="-1" new_id="1" new_version="1"/>
    55                     <way old_id="-2" new_id="2" new_version="1"/>
    56                     <relation old_id="-3" new_id="3" new_version="1"/>
    57                 </diffResult>
    58                 """
    59 
    60                 processor.parse doc, null
    61                 assert processor.@diffResults.size() == 3
    62                 SimplePrimitiveId id = new SimplePrimitiveId(-1, OsmPrimitiveType.NODE)
    63                 def entry = processor.@diffResults[id]
    64                 assert entry != null
    65                 assert entry.newId == 1
    66                 assert entry.newVersion == 1
    67 
    68                 id = new SimplePrimitiveId(-2, OsmPrimitiveType.WAY)
    69                 entry = processor.@diffResults[id]
    70                 assert entry != null
    71                 assert entry.newId == 2
    72                 assert entry.newVersion == 1
    73 
    74                 id = new SimplePrimitiveId(-3, OsmPrimitiveType.RELATION)
    75                 entry = processor.@diffResults[id]
    76                 assert entry != null
    77                 assert entry.newId == 3
    78                 assert entry.newVersion == 1
    79         }
    80 
    81         @Test
    82         public void testPostProcess_Invocation_Variants() {
    83                 def DiffResultProcessor processor  = new DiffResultProcessor([])
    84                 String doc = """\
     48    @Test
     49    public void testParse_OK_Cases() {
     50        def DiffResultProcessor processor  = new DiffResultProcessor([])
     51        String doc = """\
    8552        <diffResult version="0.6" generator="Test Data">
    8653            <node old_id="-1" new_id="1" new_version="1"/>
     
    9057        """
    9158
    92                 processor.parse doc, null
     59        processor.parse doc, null
     60        assert processor.@diffResults.size() == 3
     61        SimplePrimitiveId id = new SimplePrimitiveId(-1, OsmPrimitiveType.NODE)
     62        def entry = processor.@diffResults[id]
     63        assert entry != null
     64        assert entry.newId == 1
     65        assert entry.newVersion == 1
    9366
    94                 // should all be ok
    95                 //
    96                 processor.postProcess null, null
    97                 processor.postProcess null, NullProgressMonitor.INSTANCE
    98                 processor.postProcess new Changeset(1), null
    99                 processor.postProcess new Changeset(1), NullProgressMonitor.INSTANCE
    100         }
     67        id = new SimplePrimitiveId(-2, OsmPrimitiveType.WAY)
     68        entry = processor.@diffResults[id]
     69        assert entry != null
     70        assert entry.newId == 2
     71        assert entry.newVersion == 1
    10172
    102         @Test
    103         public void testPostProcess_OK() {
     73        id = new SimplePrimitiveId(-3, OsmPrimitiveType.RELATION)
     74        entry = processor.@diffResults[id]
     75        assert entry != null
     76        assert entry.newId == 3
     77        assert entry.newVersion == 1
     78    }
    10479
    105                 Node n = new Node()
    106                 Way w = new Way()
    107                 Relation r = new Relation()
     80    @Test
     81    public void testPostProcess_Invocation_Variants() {
     82        def DiffResultProcessor processor  = new DiffResultProcessor([])
     83        String doc = """\
     84        <diffResult version="0.6" generator="Test Data">
     85            <node old_id="-1" new_id="1" new_version="1"/>
     86            <way old_id="-2" new_id="2" new_version="1"/>
     87            <relation old_id="-3" new_id="3" new_version="1"/>
     88        </diffResult>
     89        """
    10890
    109                 String doc = """\
    110                 <diffResult version="0.6" generator="Test Data">
    111                     <node old_id="${n.uniqueId}" new_id="1" new_version="10"/>
    112                     <way old_id="${w.uniqueId}" new_id="2" new_version="11"/>
    113                     <relation old_id="${r.uniqueId}" new_id="3" new_version="12"/>
    114                 </diffResult>
    115                 """
     91        processor.parse doc, null
    11692
    117                 def DiffResultProcessor processor  = new DiffResultProcessor([n,w,r])
    118                 processor.parse doc, null
    119                 def processed = processor.postProcess(new Changeset(5), null)
    120                 assert processed.size() == 3
    121                 n = processed.find {it.uniqueId == 1}
    122                 assert n.changesetId == 5
    123                 assert n.version == 10
     93        // should all be ok
     94        //
     95        processor.postProcess null, null
     96        processor.postProcess null, NullProgressMonitor.INSTANCE
     97        processor.postProcess new Changeset(1), null
     98        processor.postProcess new Changeset(1), NullProgressMonitor.INSTANCE
     99    }
    124100
    125                 w = processed.find {it.uniqueId == 2}
    126                 assert w.changesetId == 5
    127                 assert w.version == 11
     101    @Test
     102    public void testPostProcess_OK() {
    128103
    129                 r = processed.find {it.uniqueId == 3}
    130                 assert r.changesetId == 5
    131                 assert r.version == 12
    132         }
     104        Node n = new Node()
     105        Way w = new Way()
     106        Relation r = new Relation()
    133107
    134         @Test
    135         public void testPostProcess_ForCreatedElement() {
     108        String doc = """\
     109            <diffResult version="0.6" generator="Test Data">
     110                <node old_id="${n.uniqueId}" new_id="1" new_version="10"/>
     111                <way old_id="${w.uniqueId}" new_id="2" new_version="11"/>
     112                <relation old_id="${r.uniqueId}" new_id="3" new_version="12"/>
     113            </diffResult>
     114            """
    136115
    137                 Node n = new Node()
    138                 String doc = """\
     116        def DiffResultProcessor processor  = new DiffResultProcessor([n,w,r])
     117        processor.parse doc, null
     118        def processed = processor.postProcess(new Changeset(5), null)
     119        assert processed.size() == 3
     120        n = processed.find {it.uniqueId == 1}
     121        assert n.changesetId == 5
     122        assert n.version == 10
     123
     124        w = processed.find {it.uniqueId == 2}
     125        assert w.changesetId == 5
     126        assert w.version == 11
     127
     128        r = processed.find {it.uniqueId == 3}
     129        assert r.changesetId == 5
     130        assert r.version == 12
     131    }
     132
     133    @Test
     134    public void testPostProcess_ForCreatedElement() {
     135
     136        Node n = new Node()
     137        String doc = """\
    139138            <diffResult version="0.6" generator="Test Data">
    140139                <node old_id="${n.uniqueId}" new_id="1" new_version="1"/>
     
    142141            """
    143142
    144                 def DiffResultProcessor processor  = new DiffResultProcessor([n])
    145                 processor.parse doc, null
    146                 def processed = processor.postProcess(new Changeset(5), null)
    147                 assert processed.size() == 1
    148                 n = processed.find {it.uniqueId == 1}
    149                 assert n.changesetId == 5
    150                 assert n.version == 1
    151         }
     143        def DiffResultProcessor processor  = new DiffResultProcessor([n])
     144        processor.parse doc, null
     145        def processed = processor.postProcess(new Changeset(5), null)
     146        assert processed.size() == 1
     147        n = processed.find {it.uniqueId == 1}
     148        assert n.changesetId == 5
     149        assert n.version == 1
     150    }
    152151
    153         @Test
    154         public void testPostProcess_ForModifiedElement() {
     152    @Test
     153    public void testPostProcess_ForModifiedElement() {
    155154
    156                 Node n = new Node(1)
    157                 n.coor = new LatLon(1,1)
    158                 n.setOsmId 1, 2
    159                 n.modified = true
    160                 String doc = """\
     155        Node n = new Node(1)
     156        n.coor = new LatLon(1,1)
     157        n.setOsmId 1, 2
     158        n.modified = true
     159        String doc = """\
    161160            <diffResult version="0.6" generator="Test Data">
    162161                <node old_id="${n.uniqueId}" new_id="${n.uniqueId}" new_version="3"/>
     
    164163            """
    165164
    166                 def DiffResultProcessor processor  = new DiffResultProcessor([n])
    167                 processor.parse doc, null
    168                 def processed = processor.postProcess(new Changeset(5), null)
    169                 assert processed.size() == 1
    170                 n = processed.find {it.uniqueId == 1}
    171                 assert n.changesetId == 5
    172                 assert n.version == 3
    173         }
     165        def DiffResultProcessor processor  = new DiffResultProcessor([n])
     166        processor.parse doc, null
     167        def processed = processor.postProcess(new Changeset(5), null)
     168        assert processed.size() == 1
     169        n = processed.find {it.uniqueId == 1}
     170        assert n.changesetId == 5
     171        assert n.version == 3
     172    }
    174173
    175         @Test
    176         public void testPostProcess_ForDeletedElement() {
     174    @Test
     175    public void testPostProcess_ForDeletedElement() {
    177176
    178                 Node n = new Node(1)
    179                 n.coor = new LatLon(1,1)
    180                 n.setOsmId 1, 2
    181                 n.deleted = true
    182                 String doc = """\
     177        Node n = new Node(1)
     178        n.coor = new LatLon(1,1)
     179        n.setOsmId 1, 2
     180        n.deleted = true
     181        String doc = """\
    183182            <diffResult version="0.6" generator="Test Data">
    184183                <node old_id="${n.uniqueId}"/>
     
    186185            """
    187186
    188                 def DiffResultProcessor processor  = new DiffResultProcessor([n])
    189                 processor.parse doc, null
    190                 def processed = processor.postProcess(new Changeset(5), null)
    191                 assert processed.size() == 1
    192                 n = processed.find {it.uniqueId == 1}
    193                 assert n.changesetId == 5
    194                 assert n.version == 2
    195         }
     187        def DiffResultProcessor processor  = new DiffResultProcessor([n])
     188        processor.parse doc, null
     189        def processed = processor.postProcess(new Changeset(5), null)
     190        assert processed.size() == 1
     191        n = processed.find {it.uniqueId == 1}
     192        assert n.changesetId == 5
     193        assert n.version == 2
     194    }
    196195}
Note: See TracChangeset for help on using the changeset viewer.