1 files changed, 18 insertions, 2 deletions
diff --git a/libphonenumber/test/com/google/i18n/phonenumbers/ExampleNumbersTest.java b/libphonenumber/test/com/google/i18n/phonenumbers/ExampleNumbersTest.java
index f265ad6b..ebea7b7d 100644
--- a/libphonenumber/test/com/google/i18n/phonenumbers/ExampleNumbersTest.java
+++ b/libphonenumber/test/com/google/i18n/phonenumbers/ExampleNumbersTest.java
@@ -259,7 +259,6 @@ public class ExampleNumbersTest extends TestCase {
   public void testCarrierSpecificShortNumbers() throws Exception {
     int wrongTagCounter = 0;
     for (String regionCode : shortNumberInfo.getSupportedRegions()) {
-      // Test the carrier-specific tag.
       PhoneNumberDesc desc =
           MetadataManager.getShortNumberMetadataForRegion(regionCode).getCarrierSpecific();
       if (desc.hasExampleNumber()) {
@@ -271,7 +270,24 @@ public class ExampleNumbersTest extends TestCase {
           logger.log(Level.SEVERE, "Carrier-specific test failed for " + regionCode);
         }
       }
-      // TODO: Test other tags here.
+    }
+    assertEquals(0, wrongTagCounter);
+  }
+
+  public void testSmsServiceShortNumbers() throws Exception {
+    int wrongTagCounter = 0;
+    for (String regionCode : shortNumberInfo.getSupportedRegions()) {
+      PhoneNumberDesc desc =
+          MetadataManager.getShortNumberMetadataForRegion(regionCode).getSmsServices();
+      if (desc.hasExampleNumber()) {
+        String exampleNumber = desc.getExampleNumber();
+        PhoneNumber smsServiceNumber = phoneNumberUtil.parse(exampleNumber, regionCode);
+        if (!shortNumberInfo.isPossibleShortNumberForRegion(smsServiceNumber, regionCode)
+            || !shortNumberInfo.isSmsServiceForRegion(smsServiceNumber, regionCode)) {
+          wrongTagCounter++;
+          logger.log(Level.SEVERE, "SMS service test failed for " + regionCode);
+        }
+      }
     }
     assertEquals(0, wrongTagCounter);
   }