Permalink
Browse files

MINOR Temporarily disabled nested url specific cases inTranslatableTe…

…st->testAlternateGetByLink(), unclear functionality requirements (from r99350)

git-svn-id: svn://svn.silverstripe.com/silverstripe/open/modules/sapphire/trunk@102856 467b73ca-7a2a-4603-9d3b-597d59a354a9
  • Loading branch information...
1 parent c1f227b commit 76d57b648de2cc759d46ca1f6cd39a7c638ed46f @chillu chillu committed Apr 14, 2010
Showing with 10 additions and 6 deletions.
  1. +10 −6 tests/model/TranslatableTest.php
@@ -873,12 +873,16 @@ public function testAlternateGetByLink() {
SiteTree::get_by_link($grandchildTranslation->Link())->ID,
'Grandchild pages can be found.'
);
-
- $this->assertEquals (
- $child->ID,
- SiteTree::get_by_link($parentTranslation->Link($child->URLSegment))->ID,
- 'Links can be made up of multiple languages'
- );
+
+ // TODO Re-enable test after clarifying with ajshort (see r88503).
+ // Its unclear if this is valid behaviour, and/or necessary for translated nested URLs
+ // to work properly
+ //
+ // $this->assertEquals (
+ // $child->ID,
+ // SiteTree::get_by_link($parentTranslation->Link($child->URLSegment))->ID,
+ // 'Links can be made up of multiple languages'
+ // );
}
public function testSiteTreeGetByLinkFindsTranslationWithoutLocale() {

0 comments on commit 76d57b6

Please sign in to comment.