Skip to content

Commit dca6d6a

Browse files
mmerdesrtroilo
authored andcommitted
#470: fix misleading method name
1 parent 37c0b6d commit dca6d6a

File tree

7 files changed

+16
-16
lines changed

7 files changed

+16
-16
lines changed

oshdb-util/src/test/java/org/heigit/ohsome/oshdb/util/geometry/helpers/OSMXmlReaderTagInterpreter.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -38,10 +38,10 @@ public boolean isArea(OSMEntity e) {
3838
if (e instanceof OSMWay) {
3939
OSMMember[] nds = ((OSMWay) e).getMembers();
4040
return nds.length >= 4 && nds[0].getId() == nds[nds.length - 1].getId()
41-
&& e.getTags().hasTagValue(area, areaYes);
41+
&& e.getTags().hasTag(area, areaYes);
4242
}
4343
if (e instanceof OSMRelation) {
44-
return e.getTags().hasTagValue(type, typeMultipolygon);
44+
return e.getTags().hasTag(type, typeMultipolygon);
4545
}
4646
return true;
4747
}

oshdb-util/src/test/java/org/heigit/ohsome/oshdb/util/osh/TestOSHEntityTimeUtils.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ void testGetModificationTimestampsNodeWithFilter() throws IOException {
5858
));
5959

6060
List<OSHDBTimestamp> tss =
61-
OSHEntityTimeUtils.getModificationTimestamps(hnode, e -> e.getTags().hasTagValue(1, 1));
61+
OSHEntityTimeUtils.getModificationTimestamps(hnode, e -> e.getTags().hasTag(1, 1));
6262

6363
assertNotNull(tss);
6464
assertEquals(2, tss.size());
@@ -176,7 +176,7 @@ void testGetModificationTimestampsWayWithFilter() throws IOException {
176176

177177
tss = OSHEntityTimeUtils.getModificationTimestamps(
178178
hway,
179-
osmEntity -> osmEntity.getTags().hasTagValue(2, 1)
179+
osmEntity -> osmEntity.getTags().hasTag(2, 1)
180180
);
181181
assertNotNull(tss);
182182
assertEquals(5, tss.size());

oshdb/src/main/java/org/heigit/ohsome/oshdb/OSHDBTags.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ public boolean hasTagKey(OSHDBTagKey key) {
4646
* @param value the value to search for.
4747
* @return true, if key/value combination is present.
4848
*/
49-
public abstract boolean hasTagValue(int key, int value);
49+
public abstract boolean hasTag(int key, int value);
5050

5151
/**
5252
* Test for a certain key/value combination.
@@ -136,7 +136,7 @@ public boolean hasTagKeyExcluding(int key, int[] uninterestingValues) {
136136
}
137137

138138
@Override
139-
public boolean hasTagValue(int key, int value) {
139+
public boolean hasTag(int key, int value) {
140140
return hasTag(new OSHDBTag(key, value));
141141
}
142142

oshdb/src/test/java/org/heigit/ohsome/oshdb/OSHDBTagsTest.java

+4-4
Original file line numberDiff line numberDiff line change
@@ -45,11 +45,11 @@ void testArrayHasTagKeyExcluding() {
4545
void testArrayHasTagValue() {
4646
var tags = OSHDBTags.of(kvs);
4747

48-
assertTrue(tags.hasTagValue(1, 2));
48+
assertTrue(tags.hasTag(1, 2));
4949

50-
assertFalse(tags.hasTagValue(2, 2));
51-
assertFalse(tags.hasTagValue(3, 4));
52-
assertFalse(tags.hasTagValue(5, 6));
50+
assertFalse(tags.hasTag(2, 2));
51+
assertFalse(tags.hasTag(3, 4));
52+
assertFalse(tags.hasTag(5, 6));
5353
}
5454

5555
@Test()

oshdb/src/test/java/org/heigit/ohsome/oshdb/osm/OSMNodeTest.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -172,12 +172,12 @@ void testHasTagValue() {
172172
OSMNode instance =
173173
OSM.node(1L, 1, 1L, 1L, 1, new int[] {1, 2, 2, 3}, 1000000000, 1000000000);
174174
boolean expResult = false;
175-
boolean result = instance.getTags().hasTagValue(1, 1);
175+
boolean result = instance.getTags().hasTag(1, 1);
176176
assertEquals(expResult, result);
177177

178178
instance = OSM.node(1L, 1, 1L, 1L, 1, new int[] {1, 1, 2, 3}, 1000000000, 1000000000);
179179
expResult = true;
180-
result = instance.getTags().hasTagValue(1, 1);
180+
result = instance.getTags().hasTag(1, 1);
181181
assertEquals(expResult, result);
182182
}
183183

oshdb/src/test/java/org/heigit/ohsome/oshdb/osm/OSMRelationTest.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -142,7 +142,7 @@ void testHasTagValue() {
142142
OSMRelation instance =
143143
OSM.relation(1L, 1, 1L, 1L, 1, new int[] {1, 2, 2, 3}, new OSMMember[] {part, part});
144144
boolean expResult = false;
145-
boolean result = instance.getTags().hasTagValue(1, 1);
145+
boolean result = instance.getTags().hasTag(1, 1);
146146
assertEquals(expResult, result);
147147
}
148148

@@ -152,7 +152,7 @@ void testHasTagValue2() {
152152
OSMRelation instance =
153153
OSM.relation(1L, 1, 1L, 1L, 1, new int[] {1, 1, 2, 3}, new OSMMember[] {part, part});
154154
boolean expResult = true;
155-
boolean result = instance.getTags().hasTagValue(1, 1);
155+
boolean result = instance.getTags().hasTag(1, 1);
156156
assertEquals(expResult, result);
157157
}
158158
}

oshdb/src/test/java/org/heigit/ohsome/oshdb/osm/OSMWayTest.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -141,13 +141,13 @@ void testHasTagValue() {
141141
OSMWay instance =
142142
OSM.way(1L, 1, 1L, 1L, 1, new int[] {1, 2, 2, 3}, new OSMMember[] {part, part});
143143
boolean expResult = false;
144-
boolean result = instance.getTags().hasTagValue(1, 1);
144+
boolean result = instance.getTags().hasTag(1, 1);
145145
assertEquals(expResult, result);
146146

147147
part = new OSMMember(1L, OSMType.NODE, 1);
148148
instance = OSM.way(1L, 1, 1L, 1L, 1, new int[] {1, 1, 2, 3}, new OSMMember[] {part, part});
149149
expResult = true;
150-
result = instance.getTags().hasTagValue(1, 1);
150+
result = instance.getTags().hasTag(1, 1);
151151
assertEquals(expResult, result);
152152
}
153153

0 commit comments

Comments
 (0)