diff --git a/exercises/practice/simple-linked-list/SimpleLinkedListTests.cs b/exercises/practice/simple-linked-list/SimpleLinkedListTests.cs index 35a89822d2..c6c7e966eb 100644 --- a/exercises/practice/simple-linked-list/SimpleLinkedListTests.cs +++ b/exercises/practice/simple-linked-list/SimpleLinkedListTests.cs @@ -1,4 +1,5 @@ using System.Linq; +using System.Collections.Generic; using Xunit; public class SimpleLinkedListTests @@ -41,8 +42,16 @@ public void Two_item_list_second_item_has_no_next() [Fact(Skip = "Remove this Skip property to run this test")] public void Implements_enumerable() { - var values = new SimpleLinkedList(2).Add(1); - Assert.Equal(new[] { 2, 1 }, values); + var list = new SimpleLinkedList(2).Add(1).Add(3); + var enumerable = Assert.IsAssignableFrom>(list); + var enumerator = enumerable.GetEnumerator(); + Assert.True(enumerator.MoveNext()); + Assert.Equal(2, enumerator.Current); + Assert.True(enumerator.MoveNext()); + Assert.Equal(1, enumerator.Current); + Assert.True(enumerator.MoveNext()); + Assert.Equal(3, enumerator.Current); + Assert.False(enumerator.MoveNext()); } [Fact(Skip = "Remove this Skip property to run this test")] @@ -56,28 +65,22 @@ public void From_enumerable() Assert.Equal(2, list.Next.Next.Next.Next.Value); } - [Theory(Skip = "Remove this Skip property to run this test")] - [InlineData(1)] - [InlineData(2)] - [InlineData(10)] - [InlineData(100)] - public void Reverse(int length) + [Fact(Skip = "Remove this Skip property to run this test")] + public void Reverse() { - var values = Enumerable.Range(1, length).ToArray(); + var values = Enumerable.Range(1, 5).ToArray(); var list = new SimpleLinkedList(values); - var reversed = list.Reverse(); + var enumerable = Assert.IsAssignableFrom>(list); + var reversed = enumerable.Reverse(); Assert.Equal(values.Reverse(), reversed); } - [Theory(Skip = "Remove this Skip property to run this test")] - [InlineData(1)] - [InlineData(2)] - [InlineData(10)] - [InlineData(100)] - public void Roundtrip(int length) + [Fact(Skip = "Remove this Skip property to run this test")] + public void Roundtrip() { - var values = Enumerable.Range(1, length); - var listValues = new SimpleLinkedList(values); - Assert.Equal(values, listValues); + var values = Enumerable.Range(1, 7); + var list = new SimpleLinkedList(values); + var enumerable = Assert.IsAssignableFrom>(list); + Assert.Equal(values, enumerable); } } \ No newline at end of file