👷 improve test code

Author: AshleyRoll

test/msg/detail/rle_codec.cpp

@@ -1,5 +1,7 @@
 #include <msg/detail/rle_codec.hpp>
 
+#include <stdx/compiler.hpp>
+
 #include <catch2/catch_test_macros.hpp>
 #include <catch2/matchers/catch_matchers_range_equals.hpp>
 
@@ -9,7 +11,7 @@
 using Catch::Matchers::RangeEquals;
 
 namespace {
-constexpr auto operator"" _b(unsigned long long v) -> std::byte {
+CONSTEVAL auto operator"" _b(unsigned long long v) -> std::byte {
     return static_cast<std::byte>(v);
 }
 } // namespace
@@ -24,53 +26,50 @@ TEST_CASE("rle_codec can encode all zeros", "[rle_codec]") {
     using bs = stdx::bitset<10>;
     using codec = rle_codec<bs>;
 
-    CHECK_THAT(codec::encode(bs{}),
-               RangeEquals(std::to_array<std::byte>({10_b})));
+    CHECK_THAT(codec::encode(bs{}), RangeEquals(std::array{10_b}));
 }
 
 TEST_CASE("rle_codec can encode all ones", "[rle_codec]") {
     using bs = stdx::bitset<12>;
     using codec = rle_codec<bs>;
 
-    CHECK_THAT(codec::encode(~bs{}),
-               RangeEquals(std::to_array<std::byte>({0_b, 12_b})));
+    CHECK_THAT(codec::encode(~bs{}), RangeEquals(std::array{0_b, 12_b}));
 }
 
 TEST_CASE("rle_codec can encode all zeros for large bit count", "[rle_codec]") {
     using bs = stdx::bitset<512>;
     using codec = rle_codec<bs>;
 
-    CHECK_THAT(codec::encode(bs{}), RangeEquals(std::to_array<std::byte>(
-                                        {255_b, 0_b, 255_b, 0_b, 2_b})));
+    CHECK_THAT(codec::encode(bs{}),
+               RangeEquals(std::array{255_b, 0_b, 255_b, 0_b, 2_b}));
 }
 
 TEST_CASE("rle_codec can encode all ones for large bit count", "[rle_codec]") {
     using bs = stdx::bitset<512>;
     using codec = rle_codec<bs>;
 
-    CHECK_THAT(codec::encode(~bs{}), RangeEquals(std::to_array<std::byte>(
-                                         {0_b, 255_b, 0_b, 255_b, 0_b, 2_b})));
+    CHECK_THAT(codec::encode(~bs{}),
+               RangeEquals(std::array{0_b, 255_b, 0_b, 255_b, 0_b, 2_b}));
 }
 
 TEST_CASE("rle_codec can encode alternating bits", "[rle_codec]") {
     using bs = stdx::bitset<8>;
     using codec = rle_codec<bs>;
 
-    CHECK_THAT(codec::encode(bs{stdx::place_bits, 0, 2, 4, 6}),
-               RangeEquals(std::to_array<std::byte>(
-                   {0_b, 1_b, 1_b, 1_b, 1_b, 1_b, 1_b, 1_b, 1_b})));
+    CHECK_THAT(
+        codec::encode(bs{stdx::place_bits, 0, 2, 4, 6}),
+        RangeEquals(std::array{0_b, 1_b, 1_b, 1_b, 1_b, 1_b, 1_b, 1_b, 1_b}));
 
     CHECK_THAT(codec::encode(bs{stdx::place_bits, 1, 3, 5, 7}),
-               RangeEquals(std::to_array<std::byte>(
-                   {1_b, 1_b, 1_b, 1_b, 1_b, 1_b, 1_b, 1_b})));
+               RangeEquals(std::array{1_b, 1_b, 1_b, 1_b, 1_b, 1_b, 1_b, 1_b}));
 }
 
 TEST_CASE("rle_decoder can iterate chunks 1", "[rle_codec]") {
     using bs = stdx::bitset<8>;
     using decoder = rle_decoder<bs>;
 
-    auto rle =
-        std::to_array<std::byte>({1_b, 1_b, 1_b, 1_b, 1_b, 1_b, 1_b, 1_b});
+    auto rle = std::array{1_b, 1_b, 1_b, 1_b, 1_b, 1_b, 1_b, 1_b};
+
     decoder dec{rle.data()};
     auto chunk = dec.make_chunk_enumerator();
 
@@ -97,7 +96,7 @@ TEST_CASE("rle_decoder can iterate chunks 2", "[rle_codec]") {
     using bs = stdx::bitset<8>;
     using decoder = rle_decoder<bs>;
 
-    auto rle = std::to_array<std::byte>({0_b, 2_b, 2_b, 2_b, 0_b, 2_b});
+    auto rle = std::array{0_b, 2_b, 2_b, 2_b, 0_b, 2_b};
     decoder dec{rle.data()};
     auto chunk = dec.make_chunk_enumerator();
 
@@ -116,7 +115,7 @@ TEST_CASE("rle_decoder can iterate chunks 3", "[rle_codec]") {
     using bs = stdx::bitset<1000>;
     using decoder = rle_decoder<bs>;
 
-    auto rle = std::to_array<std::byte>({255_b, 0_b, 255_b, 0_b, 255_b, 235_b});
+    auto rle = std::array{255_b, 0_b, 255_b, 0_b, 255_b, 235_b};
     decoder dec{rle.data()};
     auto chunk = dec.make_chunk_enumerator();
 
@@ -133,7 +132,7 @@ TEST_CASE("rle_decoder can iterate sub-chunks", "[rle_codec]") {
     using bs = stdx::bitset<8>;
     using decoder = rle_decoder<bs>;
 
-    auto rle = std::to_array<std::byte>({0_b, 2_b, 2_b, 2_b, 0_b, 2_b});
+    auto rle = std::array{0_b, 2_b, 2_b, 2_b, 0_b, 2_b};
     decoder dec{rle.data()};
     auto chunk = dec.make_chunk_enumerator();
 
@@ -166,8 +165,7 @@ TEST_CASE("rle_decoder stops after num_bits", "[rle_codec]") {
     using decoder = rle_decoder<bs>;
 
     // contains extra data which should be ignored
-    auto rle =
-        std::to_array<std::byte>({0_b, 2_b, 2_b, 2_b, 0_b, 2_b, 255_b, 255_b});
+    auto rle = std::array{0_b, 2_b, 2_b, 2_b, 0_b, 2_b, 255_b, 255_b};
     decoder dec{rle.data()};
     auto chunk = dec.make_chunk_enumerator();