fix test in SetBuiltins.remove

cosminbasca · cosminbasca · commit 9efbf6754ca1 · 2018-05-11T16:10:43.000+02:00
diff --git a/graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/set/SetBuiltins.java b/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;

Original file line number	Diff line number	Diff line change
`@@ -103,7 +103,7 @@ abstract static class RemoveNode extends PythonBinaryBuiltinNode {`
`103`	`103`	`Object remove(PBaseSet self, Object other,`
`104`	`104`	`@Cached("create()") HashingStorageNodes.DelItemNode delItemNode) {`
`105`	`105`
`106`		`- if (delItemNode.execute(self, self.getDictStorage(), other)) {`
	`106`	`+ if (!delItemNode.execute(self, self.getDictStorage(), other)) {`
`107`	`107`	`throw raise(PythonErrorType.KeyError, "%s", other);`
`108`	`108`	`}`
`109`	`109`	`return PNone.NONE;`