Update to_bytes tests

test/test_utils.cpp

@@ -6,54 +6,66 @@
 namespace science::libndtp {
 
 TEST(UtilsTest, ToBytesBasicFunctionality) {
-  auto [result1, offset1] = to_bytes<uint64_t>({1, 2, 3, 0}, 2);
+  auto [result1, offset1, success1] = to_bytes<uint64_t>({1, 2, 3, 0}, 2);
   EXPECT_EQ(result1, (std::vector<uint8_t>{0x6C}));
   EXPECT_EQ(offset1, 0);
+  EXPECT_TRUE(success1);
 
-  auto [result2, offset2] = to_bytes<uint64_t>({1, 2, 3, 2, 1}, 2);
+  auto [result2, offset2, success2] = to_bytes<uint64_t>({1, 2, 3, 2, 1}, 2);
   EXPECT_EQ(result2, (std::vector<uint8_t>{0x6E, 0x40}));
   EXPECT_EQ(offset2, 2);
+  EXPECT_TRUE(success2);
 
-  auto [result3, offset3] = to_bytes<uint64_t>({7, 5, 3, 1}, 12);
+  auto [result3, offset3, success3] = to_bytes<uint64_t>({7, 5, 3, 1}, 12);
   EXPECT_EQ(result3, (std::vector<uint8_t>{0x00, 0x70, 0x05, 0x00, 0x30, 0x01}));
   EXPECT_EQ(offset3, 0);
+  EXPECT_TRUE(success3);
 
-  auto [result4, offset4] = to_bytes<int64_t>({-7, -5, -3, -1}, 12, {}, 0, true);
+  auto [result4, offset4, success4] = to_bytes<int64_t>({-7, -5, -3, -1}, 12, {}, 0, true);
   EXPECT_EQ(result4, (std::vector<uint8_t>{0xFF, 0x9F, 0xFB, 0xFF, 0xDF, 0xFF}));
   EXPECT_EQ(offset4, 0);
+  EXPECT_TRUE(success4);
 
-  auto [result5, offset5] = to_bytes<uint64_t>({7, 5, 3}, 12, {0x01, 0x00}, 4);
+  auto [result5, offset5, success5] = to_bytes<uint64_t>({7, 5, 3}, 12, {0x01, 0x00}, 4);
   EXPECT_EQ(result5, (std::vector<uint8_t>{0x01, 0x00, 0x07, 0x00, 0x50, 0x03}));
   EXPECT_EQ(offset5, 0);
+  EXPECT_TRUE(success5);
 
-  auto [result6, offset6] = to_bytes<int64_t>({-7, -5, -3}, 12, {0x01, 0x00}, 4, true);
+  auto [result6, offset6, success6] = to_bytes<int64_t>({-7, -5, -3}, 12, {0x01, 0x00}, 4, true);
   EXPECT_EQ(result6, (std::vector<uint8_t>{0x01, 0x0F, 0xF9, 0xFF, 0xBF, 0xFD}));
   EXPECT_EQ(offset6, 0);
+  EXPECT_TRUE(success6);
 
-  auto [result7, offset7] = to_bytes<uint64_t>({7, 5, 3}, 12);
+  auto [result7, offset7, success7] = to_bytes<uint64_t>({7, 5, 3}, 12);
   EXPECT_EQ(result7, (std::vector<uint8_t>{0x00, 0x70, 0x05, 0x00, 0x30}));
   EXPECT_EQ(offset7, 4);
+  EXPECT_TRUE(success7);
 
-  auto [result8, offset8] = to_bytes<uint64_t>({1, 2, 3, 4}, 8);
+  auto [result8, offset8, success8] = to_bytes<uint64_t>({1, 2, 3, 4}, 8);
   EXPECT_EQ(result8, (std::vector<uint8_t>{0x01, 0x02, 0x03, 0x04}));
   EXPECT_EQ(offset8, 0);
+  EXPECT_TRUE(success8);
 
   std::vector<uint8_t> result9;
   int offset9;
-  std::tie(result9, offset9) = to_bytes<uint64_t>({7, 5, 3}, 12);
+  bool success9;
+  std::tie(result9, offset9, success9) = to_bytes<uint64_t>({7, 5, 3}, 12);
   EXPECT_EQ(result9, (std::vector<uint8_t>{0x00, 0x70, 0x05, 0x00, 0x30}));
   EXPECT_EQ(result9.size(), 5);
   EXPECT_EQ(offset9, 4);
+  EXPECT_TRUE(success9);
 
-  auto [result10, offset10] = to_bytes<uint64_t>({3, 5, 7}, 12, result9, offset9);
+  auto [result10, offset10, success10] = to_bytes<uint64_t>({3, 5, 7}, 12, result9, offset9);
   EXPECT_EQ(result10, (std::vector<uint8_t>{0x00, 0x70, 0x05, 0x00, 0x30, 0x03, 0x00, 0x50, 0x07}));
   EXPECT_EQ(result10.size(), 9);
   EXPECT_EQ(offset10, 0);
+  EXPECT_TRUE(success10);
 }
 
 TEST(UtilsTest, ToBytesErrorCases) {
   // Test case: 8 doesn't fit in 3 bits
-  EXPECT_THROW(to_bytes<uint64_t>({8}, 3), std::invalid_argument);
+  auto [_1, _2, success] = to_bytes<uint64_t>({8}, 3);
+  EXPECT_FALSE(success);
 
   // Test case: Invalid bit width
   EXPECT_THROW(to_bytes<uint64_t>({1, 2, 3, 0}, 0), std::invalid_argument);