Fixed minor issues in the tests

Author: liuzicheng1987

tests/generic/test_optional.cpp

@@ -1,17 +1,18 @@
+#include <gtest/gtest.h>
+
 #include <rfl.hpp>
 #include <string>
-#include <gtest/gtest.h>
 
 namespace test_optional {
 
 struct Person {
-    std::string name;
-    std::optional<rfl::Box<Person>> spouse;
+  std::string name;
+  std::optional<rfl::Box<Person>> spouse;
 };
 
 TEST(generic, test_optional) {
-    rfl::Generic generic;
-    // This failed to compile due to optional copying rather than moving
-    rfl::from_generic<Person>(generic);
+  rfl::Generic generic;
+  // This failed to compile due to optional copying rather than moving
+  rfl::from_generic<Person>(generic).value();
 }
-}  // namespace test_map
+}  // namespace test_optional

tests/json/test_save_load.cpp

@@ -1,12 +1,12 @@
+#include <gtest/gtest.h>
+
 #include <cassert>
 #include <iostream>
 #include <rfl.hpp>
 #include <rfl/json.hpp>
 #include <string>
 #include <vector>
 
-#include <gtest/gtest.h>
-
 namespace test_save_load {
 
 using Age = rfl::Validator<unsigned int,
@@ -49,14 +49,15 @@ TEST(json, test_save_load) {
              .email = "homer@simpson.com",
              .children = std::vector<Person>({bart, lisa, maggie})};
 
-  rfl::json::save("homer.json", homer1);
+  rfl::json::save("homer.json", homer1).value();
 
   const auto homer2 = rfl::json::load<Person>("homer.json").value();
 
   const auto string1 = rfl::json::write(homer1);
   const auto string2 = rfl::json::write(homer2);
 
-  EXPECT_EQ(string1, string2) << "Test failed. Content was not identical." << std::endl
-              << std::endl;
+  EXPECT_EQ(string1, string2)
+      << "Test failed. Content was not identical." << std::endl
+      << std::endl;
 }
 }  // namespace test_save_load