Skip to content

Commit e6f84f2

Browse files
committed
modified return type of TreasureChestItemIterator's constructor to be Iterator
1 parent 1c2ddfa commit e6f84f2

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

iterator/src/main/java/com/iluwatar/iterator/list/TreasureChest.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@
1818
*/
1919
package com.iluwatar.iterator.list;
2020

21+
import com.iluwatar.iterator.Iterator;
2122
import java.util.ArrayList;
2223
import java.util.List;
2324

@@ -47,7 +48,7 @@ public TreasureChest() {
4748
items.add(new Item(ItemType.WEAPON, "Dagger of poison"));
4849
}
4950

50-
public TreasureChestItemIterator iterator(ItemType itemType) {
51+
public Iterator iterator(ItemType itemType) {
5152
return new TreasureChestItemIterator(this, itemType);
5253
}
5354

iterator/src/test/java/com/iluwatar/iterator/list/TreasureChestTest.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@
2222
import static org.junit.jupiter.api.Assertions.assertNotNull;
2323
import static org.junit.jupiter.api.Assertions.fail;
2424

25+
import com.iluwatar.iterator.Iterator;
2526
import java.util.ArrayList;
2627
import java.util.List;
2728
import org.junit.jupiter.params.ParameterizedTest;
@@ -62,11 +63,11 @@ public static List<Object[]> dataProvider() {
6263
@MethodSource("dataProvider")
6364
public void testIterator(Item expectedItem) {
6465
final TreasureChest chest = new TreasureChest();
65-
final TreasureChestItemIterator iterator = chest.iterator(expectedItem.getType());
66+
final Iterator iterator = chest.iterator(expectedItem.getType());
6667
assertNotNull(iterator);
6768

6869
while (iterator.hasNext()) {
69-
final Item item = iterator.next();
70+
final Item item = (Item) iterator.next();
7071
assertNotNull(item);
7172
assertEquals(expectedItem.getType(), item.getType());
7273

0 commit comments

Comments
 (0)