move tests to the end

Author: kosiew

datafusion/functions/src/datetime/now.rs

@@ -73,47 +73,6 @@ impl NowFunc {
     }
 }
 
-#[cfg(test)]
-mod tests {
-    use super::*;
-
-    #[allow(deprecated)]
-    #[test]
-    fn now_func_default_matches_config() {
-        let default_config = ConfigOptions::default();
-
-        let legacy_now = NowFunc::new();
-        let configured_now = NowFunc::new_with_config(&default_config);
-
-        let empty_fields: [FieldRef; 0] = [];
-        let empty_scalars: [Option<&ScalarValue>; 0] = [];
-
-        let legacy_field = legacy_now
-            .return_field_from_args(ReturnFieldArgs {
-                arg_fields: &empty_fields,
-                scalar_arguments: &empty_scalars,
-            })
-            .expect("legacy now() return field");
-
-        let configured_field = configured_now
-            .return_field_from_args(ReturnFieldArgs {
-                arg_fields: &empty_fields,
-                scalar_arguments: &empty_scalars,
-            })
-            .expect("configured now() return field");
-
-        assert_eq!(legacy_field.as_ref(), configured_field.as_ref());
-
-        let legacy_scalar =
-            ScalarValue::TimestampNanosecond(None, legacy_now.timezone.clone());
-        let configured_scalar =
-            ScalarValue::TimestampNanosecond(None, configured_now.timezone.clone());
-
-        assert_eq!(legacy_scalar, configured_scalar);
-        assert_eq!(Some("+00:00"), legacy_now.timezone.as_deref());
-    }
-}
-
 /// Create an implementation of `now()` that always returns the
 /// specified timestamp.
 ///
@@ -181,3 +140,44 @@ impl ScalarUDFImpl for NowFunc {
         self.doc()
     }
 }
+
+#[cfg(test)]
+mod tests {
+    use super::*;
+
+    #[allow(deprecated)]
+    #[test]
+    fn now_func_default_matches_config() {
+        let default_config = ConfigOptions::default();
+
+        let legacy_now = NowFunc::new();
+        let configured_now = NowFunc::new_with_config(&default_config);
+
+        let empty_fields: [FieldRef; 0] = [];
+        let empty_scalars: [Option<&ScalarValue>; 0] = [];
+
+        let legacy_field = legacy_now
+            .return_field_from_args(ReturnFieldArgs {
+                arg_fields: &empty_fields,
+                scalar_arguments: &empty_scalars,
+            })
+            .expect("legacy now() return field");
+
+        let configured_field = configured_now
+            .return_field_from_args(ReturnFieldArgs {
+                arg_fields: &empty_fields,
+                scalar_arguments: &empty_scalars,
+            })
+            .expect("configured now() return field");
+
+        assert_eq!(legacy_field.as_ref(), configured_field.as_ref());
+
+        let legacy_scalar =
+            ScalarValue::TimestampNanosecond(None, legacy_now.timezone.clone());
+        let configured_scalar =
+            ScalarValue::TimestampNanosecond(None, configured_now.timezone.clone());
+
+        assert_eq!(legacy_scalar, configured_scalar);
+        assert_eq!(Some("+00:00"), legacy_now.timezone.as_deref());
+    }
+}