[hotfix] Set.remove

PullRequest: graalpython-open/46

Author: cosminbasca

graalpython/com.oracle.graal.python.test/src/tests/test_set.py

@@ -53,3 +53,10 @@ def test_set_or():
 
     union = s1 | s4
     assert union == {1, 2, 3}
+
+
+def test_set_remove():
+    s = {1, 2, 3}
+    assert s == {1, 2, 3}
+    s.remove(3)
+    assert s == {1, 2}

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/set/SetBuiltins.java

@@ -103,7 +103,7 @@ abstract static class RemoveNode extends PythonBinaryBuiltinNode {
         Object remove(PBaseSet self, Object other,
                         @Cached("create()") HashingStorageNodes.DelItemNode delItemNode) {
 
-            if (delItemNode.execute(self, self.getDictStorage(), other)) {
+            if (!delItemNode.execute(self, self.getDictStorage(), other)) {
                 throw raise(PythonErrorType.KeyError, "%s", other);
             }
             return PNone.NONE;