From 31da36bedff56f36ca7d3e05751c0fe017b298a0 Mon Sep 17 00:00:00 2001
From: Ilya Gorbunov <ilya.gorbunov@jetbrains.com>
Date: Mon, 6 Feb 2017 22:38:32 +0300
Subject: [PATCH] Minor: Add test cases for adding and removing null value
 to/from set. Fix unused value in removeAll test.

---
 libraries/stdlib/test/js/SetJsTest.kt | 16 ++++++++++++++--
 1 file changed, 14 insertions(+), 2 deletions(-)

diff --git a/libraries/stdlib/test/js/SetJsTest.kt b/libraries/stdlib/test/js/SetJsTest.kt
index 2e25da23993..48f2c8e5cc3 100644
--- a/libraries/stdlib/test/js/SetJsTest.kt
+++ b/libraries/stdlib/test/js/SetJsTest.kt
@@ -147,6 +147,10 @@ abstract class SetJsTest {
         assertFalse(data.add("baz"))
         assertEquals(3, data.size)
         assertTrue(data.containsAll(arrayListOf("foo", "bar", "baz")))
+
+        val nullableSet = createEmptyMutableSetWithNullableValues()
+        assertTrue(nullableSet.add(null))
+        assertFalse(nullableSet.add(null))
     }
 
     @Test
@@ -157,6 +161,12 @@ abstract class SetJsTest {
         assertFalse(data.remove("foo"))
         assertEquals(1, data.size)
         assertTrue(data.contains("bar"))
+
+        val nullableSet = createEmptyMutableSetWithNullableValues()
+        nullableSet.add(null)
+
+        assertTrue(nullableSet.remove(null))
+        assertFalse(nullableSet.remove(null))
     }
 
     @Test
@@ -181,9 +191,11 @@ abstract class SetJsTest {
         assertTrue(data.removeAll(arrayListOf("foo", "bar")))
         assertEquals(0, data.size)
 
-        val data2 = createTestMutableSet()
         assertFalse(data.removeAll(arrayListOf("foo", "bar", "baz")))
-        assertTrue(data.isEmpty())
+
+        val data2 = createTestMutableSet()
+        assertTrue(data2.removeAll(arrayListOf("foo", "bar", "baz")))
+        assertTrue(data2.isEmpty())
     }
 
     @Test