Don’t treat NONE and NULL as the same

lib/src/sql/value/value.rs

@@ -602,11 +602,11 @@ impl Value {
 	// -----------------------------------
 
 	pub fn is_none(&self) -> bool {
-		matches!(self, Value::None | Value::Null)
+		matches!(self, Value::None)
 	}
 
 	pub fn is_null(&self) -> bool {
-		matches!(self, Value::None | Value::Null)
+		matches!(self, Value::Null)
 	}
 
 	pub fn is_some(&self) -> bool {
@@ -1002,6 +1002,7 @@ impl Value {
 				true => self,
 				_ => Value::None,
 			},
+			Kind::Option(v) => todo!(),
 		}
 	}
 
@@ -1530,14 +1531,14 @@ mod tests {
 	#[test]
 	fn check_none() {
 		assert_eq!(true, Value::None.is_none());
-		assert_eq!(true, Value::Null.is_none());
+		assert_eq!(false, Value::Null.is_none());
 		assert_eq!(false, Value::from(1).is_none());
 	}
 
 	#[test]
 	fn check_null() {
-		assert_eq!(true, Value::None.is_null());
 		assert_eq!(true, Value::Null.is_null());
+		assert_eq!(false, Value::None.is_null());
 		assert_eq!(false, Value::from(1).is_null());
 	}
 

lib/tests/compare.rs

@@ -0,0 +1,62 @@
+mod parse;
+use surrealdb::dbs::Session;
+use surrealdb::err::Error;
+use surrealdb::kvs::Datastore;
+use surrealdb::sql::Value;
+
+#[tokio::test]
+async fn compare_empty() -> Result<(), Error> {
+	let sql = r#"
+		RETURN NONE = NONE;
+		RETURN NULL = NULL;
+		RETURN NONE = NULL;
+		RETURN [] = [];
+		RETURN {} = {};
+		RETURN [] = {};
+		RETURN 0 = 0;
+		RETURN 0 = 0.0;
+		RETURN 0 = 0.1;
+	"#;
+	let dbs = Datastore::new("memory").await?;
+	let ses = Session::for_kv().with_ns("test").with_db("test");
+	let res = &mut dbs.execute(&sql, &ses, None, false).await?;
+	assert_eq!(res.len(), 9);
+	//
+	let tmp = res.remove(0).result?;
+	let val = Value::True;
+	assert_eq!(tmp, val);
+	//
+	let tmp = res.remove(0).result?;
+	let val = Value::True;
+	assert_eq!(tmp, val);
+	//
+	let tmp = res.remove(0).result?;
+	let val = Value::False;
+	assert_eq!(tmp, val);
+	//
+	let tmp = res.remove(0).result?;
+	let val = Value::True;
+	assert_eq!(tmp, val);
+	//
+	let tmp = res.remove(0).result?;
+	let val = Value::True;
+	assert_eq!(tmp, val);
+	//
+	let tmp = res.remove(0).result?;
+	let val = Value::False;
+	assert_eq!(tmp, val);
+	//
+	let tmp = res.remove(0).result?;
+	let val = Value::True;
+	assert_eq!(tmp, val);
+	//
+	let tmp = res.remove(0).result?;
+	let val = Value::True;
+	assert_eq!(tmp, val);
+	//
+	let tmp = res.remove(0).result?;
+	let val = Value::False;
+	assert_eq!(tmp, val);
+	//
+	Ok(())
+}