Tests and fix objobject ne impl

OddCoincidence · OddCoincidence · commit 86362fd7b325 · 2019-02-10T10:12:53.000-08:00
diff --git a/tests/snippets/builtin_complex.py b/tests/snippets/builtin_complex.py
@@ -14,11 +14,12 @@
 assert complex(1, 0) == 1.0
 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 complex(1, 2) != complex(1, 1)
-# Currently broken - see issue #419
-# assert complex(1, 2) != 'foo'
+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`