[Intl] Update assertions to verify data is not empty

jakzal · jakzal · commit a1a551ddbd86 · 2017-11-04T07:26:32.000Z
diff --git a/src/Symfony/Component/Intl/Tests/Data/Provider/AbstractLanguageDataProviderTest.php b/src/Symfony/Component/Intl/Tests/Data/Provider/AbstractLanguageDataProviderTest.php
@@ -856,6 +856,7 @@ public function testGetNames($displayLocale)
 
         sort($languages);
 
+        $this->assertNotEmpty($languages);
         $this->assertEmpty(array_diff($languages, static::$languages));
     }
 
diff --git a/src/Symfony/Component/Intl/Tests/Data/Provider/AbstractLocaleDataProviderTest.php b/src/Symfony/Component/Intl/Tests/Data/Provider/AbstractLocaleDataProviderTest.php
@@ -58,6 +58,7 @@ public function testGetNames($displayLocale)
 
         // We can't assert on exact list of locale, as there's too many variations.
         // The best we can do is to make sure getNames() returns a subset of what getLocales() returns.
+        $this->assertNotEmpty($locales);
         $this->assertEmpty(array_diff($locales, $this->getLocales()));
     }
 
diff --git a/src/Symfony/Component/Intl/Tests/Data/Provider/AbstractScriptDataProviderTest.php b/src/Symfony/Component/Intl/Tests/Data/Provider/AbstractScriptDataProviderTest.php
@@ -282,6 +282,7 @@ public function testGetNames($displayLocale)
 
         // We can't assert on exact list of scripts, as there's too many variations between locales.
         // The best we can do is to make sure getNames() returns a subset of what getScripts() returns.
+        $this->assertNotEmpty($scripts);
         $this->assertEmpty(array_diff($scripts, self::$scripts));
     }
 

Original file line number	Diff line number	Diff line change
`@@ -856,6 +856,7 @@ public function testGetNames($displayLocale)`
`856`	`856`
`857`	`857`	`sort($languages);`
`858`	`858`
	`859`	`+ $this->assertNotEmpty($languages);`
`859`	`860`	`$this->assertEmpty(array_diff($languages, static::$languages));`
`860`	`861`	`}`
`861`	`862`
Original file line number	Diff line number	Diff line change
`@@ -58,6 +58,7 @@ public function testGetNames($displayLocale)`
`58`	`58`
`59`	`59`	`// We can't assert on exact list of locale, as there's too many variations.`
`60`	`60`	`// The best we can do is to make sure getNames() returns a subset of what getLocales() returns.`
	`61`	`+ $this->assertNotEmpty($locales);`
`61`	`62`	`$this->assertEmpty(array_diff($locales, $this->getLocales()));`
`62`	`63`	`}`
`63`	`64`
Original file line number	Diff line number	Diff line change
`@@ -282,6 +282,7 @@ public function testGetNames($displayLocale)`
`282`	`282`
`283`	`283`	`// We can't assert on exact list of scripts, as there's too many variations between locales.`
`284`	`284`	`// The best we can do is to make sure getNames() returns a subset of what getScripts() returns.`
	`285`	`+ $this->assertNotEmpty($scripts);`
`285`	`286`	`$this->assertEmpty(array_diff($scripts, self::$scripts));`
`286`	`287`	`}`
`287`	`288`