More tests for SwingStreamUtils.asIterable(TreeModel)

Author: parubok

src/main/java/io/github/parubok/stream/SwingStreamUtils.java

@@ -275,6 +275,9 @@ public static Iterable<KTreePath> asIterable(TreeModel treeModel, TreeTraversalT
         return asIterable(treeModel, traversalType, true);
     }
 
+    /**
+     * @return Iterator which does not check for the tree model modifications during the iteration.
+     */
     public static Iterable<KTreePath> asIterable(TreeModel treeModel, TreeTraversalType traversalType,
                                                  boolean failOnModification) {
         requireNonNull(treeModel);

src/test/java/io/github/parubok/stream/SwingStreamUtilsTest.java

@@ -1264,11 +1264,15 @@ public void removeTreeModelListener(TreeModelListener l) {
             };
             Assertions.assertEquals(0, listeners.size());
             Iterator<KTreePath> iterator = SwingStreamUtils.asIterable(model).iterator();
-            Assertions.assertEquals(1, listeners.size());
+            Assertions.assertEquals(1, listeners.size()); // must be removed after iteration
             while (iterator.hasNext()) {
                 iterator.next();
             }
             Assertions.assertEquals(0, listeners.size());
+
+            // without modification check - no listener is added
+            iterator = SwingStreamUtils.asIterable(model, SwingStreamUtils.DEFAULT_TREE_TRAVERSAL_TYPE, false).iterator();
+            Assertions.assertEquals(0, listeners.size());
         });
     }
 }