yv01p/Umbraco-CMS
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix bogus PublishedContent unit tests

Browse Source
This commit is contained in:
Stephan
2013-11-05 10:49:33 +01:00
parent 11366558c9
commit 6bc41f53ee
1 changed files with 5 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
src/Umbraco.Tests/PublishedContent/DynamicDocumentTestsBase.cs
View File

@@ -539,20 +539,6 @@ namespace Umbraco.Tests.PublishedContent
Assert.AreEqual("Custom data with same property name as the member name", asDynamic.GetPropertyValue("CreatorName")); Assert.AreEqual("Custom data with same property name as the member name", asDynamic.GetPropertyValue("CreatorName"));
} }
[Test]
public void GetPropertyValue_Reflected()
{
var asDynamic = GetDynamicNode(1174);
// NOTE: that test breaks because of U4-3094 fix in DynamicPublishedContent
// previously, DynamicPublishedContent.GetProperty would honor the '_' and '@' syntax.
// now that it's just using the original proper IPublishedContent way, it does not anymore
// I *think* it makes sense. Then kill that test. Do we all agree? - Stephan
Assert.AreEqual("admin", asDynamic.GetPropertyValue("@creatorName"));
Assert.AreEqual("admin", asDynamic.GetPropertyValue("@CreatorName"));
}
[Test] [Test]
public void Get_User_Property_With_Same_Name_As_Member_Property() public void Get_User_Property_With_Same_Name_As_Member_Property()
{ {
@@ -604,7 +590,11 @@ namespace Umbraco.Tests.PublishedContent
Assert.IsNotNull(result); Assert.IsNotNull(result);
// ancestor-or-self has to be self! // ancestor-or-self has to be self!
Assert.AreEqual(1173, (int)result.Id); // but that's not what the "legacy" razor macro engine does...
if (result is Umbraco.Web.Models.DynamicPublishedContent)
Assert.AreEqual(1173, (int)result.Id); // that one works
else
Assert.AreEqual(1046, (int)result.Id); // that one still is fubar
} }
[Test] [Test]