From f0b550a092cbb0b4b71ae4d49ca1991882f05e7b Mon Sep 17 00:00:00 2001
From: Michael McCandless <mikemccand@apache.org>
Date: Mon, 6 Jul 2015 14:03:46 +0000
Subject: [PATCH] LUCENE-6365: fix test to not add duplicate strings

git-svn-id: https://svn.apache.org/repos/asf/lucene/dev/trunk@1689404 13f79535-47bb-0310-9956-ffa450edef68
---
 .../lucene/util/automaton/FiniteStringsIterator.java     | 3 +++
 .../lucene/util/automaton/FiniteStringsIteratorTest.java | 9 +++++----
 2 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/lucene/core/src/java/org/apache/lucene/util/automaton/FiniteStringsIterator.java b/lucene/core/src/java/org/apache/lucene/util/automaton/FiniteStringsIterator.java
index ee332952379..454057f5104 100644
--- a/lucene/core/src/java/org/apache/lucene/util/automaton/FiniteStringsIterator.java
+++ b/lucene/core/src/java/org/apache/lucene/util/automaton/FiniteStringsIterator.java
@@ -33,6 +33,9 @@ import java.util.BitSet;
  * <p>Be aware that the iteration order is implementation dependent
  * and may change across releases.
  *
+ * <p>If the automaton is not determinized then it's possible this iterator
+ * will return duplicates.
+ *
  * @lucene.experimental
  */
 public class FiniteStringsIterator {
diff --git a/lucene/core/src/test/org/apache/lucene/util/automaton/FiniteStringsIteratorTest.java b/lucene/core/src/test/org/apache/lucene/util/automaton/FiniteStringsIteratorTest.java
index f79e5884953..01cd988851c 100644
--- a/lucene/core/src/test/org/apache/lucene/util/automaton/FiniteStringsIteratorTest.java
+++ b/lucene/core/src/test/org/apache/lucene/util/automaton/FiniteStringsIteratorTest.java
@@ -47,11 +47,12 @@ public class FiniteStringsIteratorTest extends LuceneTestCase {
     IntsRefBuilder scratch = new IntsRefBuilder();
     for(int i=0;i<numStrings;i++) {
       String s = TestUtil.randomSimpleString(random(), 1, 200);
-      automata.add(Automata.makeString(s));
       Util.toUTF32(s.toCharArray(), 0, s.length(), scratch);
-      strings.add(scratch.toIntsRef());
-      if (VERBOSE) {
-        System.out.println("  add string=" + s);
+      if (strings.add(scratch.toIntsRef())) {
+        automata.add(Automata.makeString(s));
+        if (VERBOSE) {
+          System.out.println("  add string=" + s);
+        }
       }
     }