Merge pull request symfony#4 from stealth35/test_mo_loader

stof · stof · commit d974a4a465b7 · 2011-11-09T03:13:48.000-08:00
[Translation] fix gettext tests
diff --git a/tests/Symfony/Tests/Component/Translation/Loader/MoFileLoaderTest.php b/tests/Symfony/Tests/Component/Translation/Loader/MoFileLoaderTest.php
@@ -27,14 +27,13 @@ public function testLoad()
         $this->assertEquals(array(new FileResource($resource)), $catalogue->getResources());
     }
 
-    public function testLoadDoesNothingIfEmpty()
+    /**
+     * @expectedException \InvalidArgumentException
+     */
+    public function testLoadInvalidResource()
     {
         $loader = new MoFileLoader();
         $resource = __DIR__.'/../fixtures/empty.mo';
         $catalogue = $loader->load($resource, 'en', 'domain1');
-
-        $this->assertEquals(array(), $catalogue->all('domain1'));
-        $this->assertEquals('en', $catalogue->getLocale());
-        $this->assertEquals(array(new FileResource($resource)), $catalogue->getResources());
     }
 }
diff --git a/tests/Symfony/Tests/Component/Translation/fixtures/resources.po b/tests/Symfony/Tests/Component/Translation/fixtures/resources.po
@@ -1,2 +1,3 @@
 msgid "foo"
-msgstr "bar"
+msgstr "bar"
+

Original file line number	Diff line number	Diff line change
`@@ -27,14 +27,13 @@ public function testLoad()`
`27`	`27`	`$this->assertEquals(array(new FileResource($resource)), $catalogue->getResources());`
`28`	`28`	`}`
`29`	`29`
`30`		`- public function testLoadDoesNothingIfEmpty()`
	`30`	`+ /**`
	`31`	`+ * @expectedException \InvalidArgumentException`
	`32`	`+ */`
	`33`	`+ public function testLoadInvalidResource()`
`31`	`34`	`{`
`32`	`35`	`$loader = new MoFileLoader();`
`33`	`36`	`$resource = __DIR__.'/../fixtures/empty.mo';`
`34`	`37`	`$catalogue = $loader->load($resource, 'en', 'domain1');`
`35`		`-`
`36`		`- $this->assertEquals(array(), $catalogue->all('domain1'));`
`37`		`- $this->assertEquals('en', $catalogue->getLocale());`
`38`		`- $this->assertEquals(array(new FileResource($resource)), $catalogue->getResources());`
`39`	`38`	`}`
`40`	`39`	`}`