From 759aa0e31b093a47971fd67554a8118fbe41c03f Mon Sep 17 00:00:00 2001
From: David Humphrey <david.humphrey@senecacollege.ca>
Date: Tue, 30 Jan 2018 19:04:40 -0500
Subject: [PATCH] Added rest of tests

---
 .../phonenumbers/phonenumbermatcher_test.js   | 122 ++++++++++++++++++
 1 file changed, 122 insertions(+)

diff --git a/javascript/i18n/phonenumbers/phonenumbermatcher_test.js b/javascript/i18n/phonenumbers/phonenumbermatcher_test.js
index 9b5fdf8c4..1cc014dcd 100644
--- a/javascript/i18n/phonenumbers/phonenumbermatcher_test.js
+++ b/javascript/i18n/phonenumbers/phonenumbermatcher_test.js
@@ -845,6 +845,128 @@ function testMaxMatchesMixed() {
     }
 }
 
+// XXX_FAILING: ZZ region not valid?
+/**
+function testNonPlusPrefixedNumbersNotFoundForInvalidRegion() {
+    // Does not start with a "+", we won't match it.
+    var iterator = phoneUtil.findNumbers("1 456 764 156", RegionCode.ZZ);
+
+    assertFalse(iterator.hasNext());
+    try {
+      iterator.next();
+      fail("Violation of the Iterator contract.");
+    } catch (e) {
+        // Success
+    }
+    assertFalse(iterator.hasNext());
+  }
+
+function testEmptyIteration() {
+    var iterator = phoneUtil.findNumbers("", RegionCode.ZZ);
+
+    assertFalse(iterator.hasNext());
+    assertFalse(iterator.hasNext());
+    try {
+      iterator.next();
+      fail("Violation of the Iterator contract.");
+    } catch (e) {
+        // Success
+    }
+    assertFalse(iterator.hasNext());
+}
+
+public void testSingleIteration() {
+    var iterator = phoneUtil.findNumbers("+14156667777", RegionCode.ZZ);
+
+    // With hasNext() -> next().
+    // Double hasNext() to ensure it does not advance.
+    assertTrue(iterator.hasNext());
+    assertTrue(iterator.hasNext());
+    assertNotNull(iterator.next());
+    assertFalse(iterator.hasNext());
+    try {
+      iterator.next();
+      fail("Violation of the Iterator contract.");
+    } catch (e) {
+        // Success
+    }
+    assertFalse(iterator.hasNext());
+
+    // With next() only.
+    assertNotNull(iterator.next());
+    try {
+      iterator.next();
+      fail("Violation of the Iterator contract.");
+    } catch (e) {
+        // Success
+    }
+}
+
+function testDoubleIteration() {
+    var iterator =
+        phoneUtil.findNumbers("+14156667777 foobar +14156667777 ", RegionCode.ZZ);
+
+    // With hasNext() -> next().
+    // Double hasNext() to ensure it does not advance.
+    assertTrue(iterator.hasNext());
+    assertTrue(iterator.hasNext());
+    assertNotNull(iterator.next());
+    assertTrue(iterator.hasNext());
+    assertTrue(iterator.hasNext());
+    assertNotNull(iterator.next());
+    assertFalse(iterator.hasNext());
+    try {
+      iterator.next();
+      fail("Violation of the Iterator contract.");
+    } catch (e) { 
+        // Success
+    }
+    assertFalse(iterator.hasNext());
+
+    // With next() only.
+    assertNotNull(iterator.next());
+    assertNotNull(iterator.next());
+    try {
+      iterator.next();
+      fail("Violation of the Iterator contract.");
+    } catch (e) {
+        // Success
+    }
+}
+
+function testRemovalNotSupported() {
+    var = phoneUtil.findNumbers("+14156667777", RegionCode.ZZ);
+
+    try {
+      iterator.remove();
+      fail("Iterator must not support remove.");
+    } catch (e) {
+        // success
+    }
+
+    assertTrue(iterator.hasNext());
+
+    try {
+      iterator.remove();
+      fail("Iterator must not support remove.");
+    } catch (e) {
+        // success
+    }
+
+    assertNotNull(iterator.next());
+
+    try {
+      iterator.remove();
+      fail("Iterator must not support remove.");
+    } catch (e) { 
+        // success
+    }
+
+    assertFalse(iterator.hasNext());
+}
+
+*/
+
 /**
  * Tests numbers found by {@link PhoneNumberUtil#findNumbers(CharSequence, String)} in various
  * textual contexts.