commit 5c71041b3759afbdfaa030a62de3892eabd609a4
author Brian Silverman <bsilver16384@gmail.com> Tue Feb 09 19:09:32 2021 -0800
committer Brian Silverman <bsilver16384@gmail.com> Tue Feb 09 19:09:32 2021 -0800
tree aa48127dcd2959a6aca026685ea742a4e51cdd2a
parent 0f0dac3deeb7ea68af2d889b8abb45ee7cfda419

Fix and test JSON conversions on nonconsecutive enums

Change-Id: I057940c95bd6e371568be5ca1f426a40a6a73e95

aos/json_to_flatbuffer_test.cc

diff --git a/aos/json_to_flatbuffer_test.cc b/aos/json_to_flatbuffer_test.cc
index 4379ae9..dd70908 100644
--- a/aos/json_to_flatbuffer_test.cc
+++ b/aos/json_to_flatbuffer_test.cc
@@ -1,9 +1,8 @@
 #include "aos/json_to_flatbuffer.h"
 
-#include "gtest/gtest.h"
-
 #include "aos/json_to_flatbuffer_generated.h"
 #include "flatbuffers/minireflect.h"
+#include "gtest/gtest.h"
 
 namespace aos {
 namespace testing {
@@ -64,6 +63,9 @@
   EXPECT_TRUE(JsonAndBack("{ \"foo_enum_default\": \"UType\" }"));
 
   EXPECT_TRUE(JsonAndBack("{ \"foo_string\": \"baz\" }"));
+
+  EXPECT_TRUE(JsonAndBack("{ \"foo_enum_nonconsecutive\": \"Zero\" }"));
+  EXPECT_TRUE(JsonAndBack("{ \"foo_enum_nonconsecutive\": \"Big\" }"));
 }
 
 // Tests that NaN is handled correctly
@@ -88,6 +90,11 @@
   EXPECT_FALSE(JsonAndBack("{ \"foo_enum\": \"5ype\" }"));
 
   EXPECT_FALSE(JsonAndBack("{ \"foo_enum_default\": \"7ype\" }"));
+
+  EXPECT_FALSE(JsonAndBack("{ \"foo_enum_nonconsecutive\": \"Nope\" }"));
+
+  EXPECT_FALSE(
+      JsonAndBack("{ \"foo_enum_nonconsecutive_default\": \"Nope\" }"));
 }
 
 // Test that adding a duplicate field results in an error.