diff options
author | Paul Phillips <paulp@improving.org> | 2011-03-27 19:46:12 +0000 |
---|---|---|
committer | Paul Phillips <paulp@improving.org> | 2011-03-27 19:46:12 +0000 |
commit | c17e46682a9b42eaf4d33d9f60b50c826ab4009e (patch) | |
tree | 2c18c246de913d41b513354d5f3785693a2cd699 /test/files/run/t3361.scala | |
parent | fbdda78887e6dc594d08bf2376d0bba164d14009 (diff) | |
download | scala-c17e46682a9b42eaf4d33d9f60b50c826ab4009e.tar.gz scala-c17e46682a9b42eaf4d33d9f60b50c826ab4009e.tar.bz2 scala-c17e46682a9b42eaf4d33d9f60b50c826ab4009e.zip |
Fix for linked lists closes #4080 and proves my...
Fix for linked lists closes #4080 and proves my desire not to ship
obviously broken code is even greater than my will to hold out for any
help. I threw in a free fix for this which I noticed while in there.
scala> scala.collection.mutable.LinkedList[Int]().head
res0: Int = 0
Also was reminded how useless tests can be:
val ten = DoubleLinkedList(1 to 10: _*)
ten.insert(DoubleLinkedList(11))
// Post-insert position test
require(ten.last == 11)
Fortunately a test confirming buggy behavior still serves a purpose by
breaking when you fix the bug which allowed it to pass, thus letting you
fix the broken test too. Life's (very) little compensations. Linked list
code should still be presumed broken. No review.
Diffstat (limited to 'test/files/run/t3361.scala')
-rw-r--r-- | test/files/run/t3361.scala | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/test/files/run/t3361.scala b/test/files/run/t3361.scala index 17af89a67c..892e36dbd9 100644 --- a/test/files/run/t3361.scala +++ b/test/files/run/t3361.scala @@ -39,10 +39,8 @@ object Test extends App { def insert_1 { val ten = DoubleLinkedList(1 to 10: _*) - ten.insert(DoubleLinkedList(11)) match { - case _: Unit => require(true) - case _ => require(false) - } + ten.append(DoubleLinkedList(11)) + // Post-insert size test require(11 == ten.size) // Post-insert data test |