Tests and fix objobject ne impl

OddCoincidence · OddCoincidence · commit a510aed86aee · 2019-02-10T10:13:30.000-08:00
diff --git a/tests/snippets/builtin_complex.py b/tests/snippets/builtin_complex.py
@@ -8,14 +8,18 @@
 
 assert complex(1, -1) == complex(1, -1)
 assert complex(1, 0) == 1
-assert not complex(1, 1) == 1
+assert 1 == complex(1, 0)
+assert complex(1, 1) != 1
+assert 1 != complex(1, 1)
 assert complex(1, 0) == 1.0
-assert not complex(1, 1) == 1.0
-assert not complex(1, 0) == 1.5
+assert 1.0 == complex(1, 0)
+assert complex(1, 1) != 1.0
+assert 1.0 != complex(1, 1)
+assert complex(1, 0) != 1.5
+assert not 1.0 != complex(1, 0)
 assert bool(complex(1, 0))
-assert not complex(1, 2) == complex(1, 1)
-# Currently broken - see issue #419
-# assert complex(1, 2) != 'foo'
+assert complex(1, 2) != complex(1, 1)
+assert complex(1, 2) != 'foo'
 assert complex(1, 2).__eq__('foo') == NotImplemented
 
 # __neg__
diff --git a/vm/src/obj/objobject.rs b/vm/src/obj/objobject.rs
@@ -40,7 +40,8 @@ fn object_ne(vm: &mut VirtualMachine, args: PyFuncArgs) -> PyResult {
         args,
         required = [(zelf, Some(vm.ctx.object())), (other, None)]
     );
-    let eq = vm.call_method(zelf, "__eq__", vec![other.clone()])?;
+
+    let eq = vm._eq(zelf.clone(), other.clone())?;
     objbool::not(vm, &eq)
 }
 

Original file line number	Diff line number	Diff line change
`@@ -40,7 +40,8 @@ fn object_ne(vm: &mut VirtualMachine, args: PyFuncArgs) -> PyResult {`
`40`	`40`	`args,`
`41`	`41`	`required = [(zelf, Some(vm.ctx.object())), (other, None)]`
`42`	`42`	`);`
`43`		`- let eq = vm.call_method(zelf, "__eq__", vec![other.clone()])?;`
	`43`	`+`
	`44`	`+ let eq = vm._eq(zelf.clone(), other.clone())?;`
`44`	`45`	`objbool::not(vm, &eq)`
`45`	`46`	`}`
`46`	`47`