epic/OpenRA
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Improve MiniYaml nunit tests:

Browse Source 
- Fixes incorrectly used variable.
- Adds additional tests for node positions.
This commit is contained in:
Paul Chote
2018-05-20 16:12:44 +00:00
committed by abcdefg30
parent 8e11f0cf4e
commit ba9d1c3182
1 changed files with 8 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
OpenRA.Test/OpenRA.Game/MiniYamlTest.cs
View File

@@ -230,12 +230,15 @@ TestA:
TestB: TestB:
Nothing: Nothing:
"; ";
var resultDiscard = MiniYaml.FromString(yaml);
var resultDiscardLine = resultDiscard.First(n => n.Key == "TestB").Location.Line;
Assert.That(resultDiscardLine, Is.EqualTo(6), "Node TestB should report its location as line 6, but is not (discarding comments)");
Assert.That(resultDiscard[1].Key, Is.EqualTo("TestB"), "Node TestB should be the second child of the root node, but is not (discarding comments)");
var resultDiscard = MiniYaml.FromString(yaml).First(n => n.Key == "TestB"); var resultKeep = MiniYaml.FromString(yaml, discardCommentsAndWhitespace: false);
Assert.That(resultDiscard.Location.Line, Is.EqualTo(6), "Node TestB should report its location as line 6, but is not (discarding comments)"); var resultKeepLine = resultKeep.First(n => n.Key == "TestB").Location.Line;
Assert.That(resultKeepLine, Is.EqualTo(6), "Node TestB should report its location as line 6, but is not (parsing comments)");
var resultKeep = MiniYaml.FromString(yaml, discardCommentsAndWhitespace: false).First(n => n.Key == "TestB"); Assert.That(resultKeep[4].Key, Is.EqualTo("TestB"), "Node TestB should be the fifth child of the root node, but is not (parsing comments)");
Assert.That(resultDiscard.Location.Line, Is.EqualTo(6), "Node TestB should report its location as line 6, but is not (parsing comments)");
} }
[TestCase(TestName = "Comments should survive a round trip intact")] [TestCase(TestName = "Comments should survive a round trip intact")]