Commite6f84f2

committed

modified return type of TreasureChestItemIterator's constructor to be Iterator

1 parent1c2ddfa commite6f84f2Copy full SHA for e6f84f2

File tree

-3

lines changed

iterator/src
- main/java/com/iluwatar/iterator/list
  - TreasureChest.java
- test/java/com/iluwatar/iterator/list
  - TreasureChestTest.java

-3

lines changed

Lines changed: 2 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -18,6 +18,7 @@`
`18`	`18`	`*/`
`19`	`19`	`packagecom.iluwatar.iterator.list;`
`20`	`20`
	`21`	`+importcom.iluwatar.iterator.Iterator;`
`21`	`22`	`importjava.util.ArrayList;`
`22`	`23`	`importjava.util.List;`
`23`	`24`
`@@ -47,7 +48,7 @@ public TreasureChest() {`
`47`	`48`	`items.add(newItem(ItemType.WEAPON,"Dagger of poison"));`
`48`	`49`	`}`
`49`	`50`
`50`		`-publicTreasureChestItemIteratoriterator(ItemTypeitemType) {`
	`51`	`+publicIteratoriterator(ItemTypeitemType) {`
`51`	`52`	`returnnewTreasureChestItemIterator(this,itemType);`
`52`	`53`	`}`
`53`	`54`

Lines changed: 3 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -22,6 +22,7 @@`
`22`	`22`	`importstaticorg.junit.jupiter.api.Assertions.assertNotNull;`
`23`	`23`	`importstaticorg.junit.jupiter.api.Assertions.fail;`
`24`	`24`
	`25`	`+importcom.iluwatar.iterator.Iterator;`
`25`	`26`	`importjava.util.ArrayList;`
`26`	`27`	`importjava.util.List;`
`27`	`28`	`importorg.junit.jupiter.params.ParameterizedTest;`
`@@ -62,11 +63,11 @@ public static List<Object[]> dataProvider() {`
`62`	`63`	`@MethodSource("dataProvider")`
`63`	`64`	`publicvoidtestIterator(ItemexpectedItem) {`
`64`	`65`	`finalTreasureChestchest =newTreasureChest();`
`65`		`-finalTreasureChestItemIteratoriterator =chest.iterator(expectedItem.getType());`
	`66`	`+finalIteratoriterator =chest.iterator(expectedItem.getType());`
`66`	`67`	`assertNotNull(iterator);`
`67`	`68`
`68`	`69`	`while (iterator.hasNext()) {`
`69`		`-finalItemitem =iterator.next();`
	`70`	`+finalItemitem =(Item)iterator.next();`
`70`	`71`	`assertNotNull(item);`
`71`	`72`	`assertEquals(expectedItem.getType(),item.getType());`
`72`	`73`

Comments

(0)