Fix Tests

Signed-off-by: Rohit Agrawal <[email protected]>

Author: agrawroh

test/extensions/filters/network/dubbo_proxy/decoder_test.cc

@@ -176,7 +176,8 @@ TEST_F(DubboDecoderStateMachineTest, ProtocolStateNameTest) {
   // Test that ProtocolStateNameValues::name() returns the expected string value
   EXPECT_EQ("StopIteration", ProtocolStateNameValues::name(ProtocolState::StopIteration));
   EXPECT_EQ("WaitForData", ProtocolStateNameValues::name(ProtocolState::WaitForData));
-  EXPECT_EQ("OnDecodeStreamHeader", ProtocolStateNameValues::name(ProtocolState::OnDecodeStreamHeader));
+  EXPECT_EQ("OnDecodeStreamHeader",
+            ProtocolStateNameValues::name(ProtocolState::OnDecodeStreamHeader));
   EXPECT_EQ("OnDecodeStreamData", ProtocolStateNameValues::name(ProtocolState::OnDecodeStreamData));
   EXPECT_EQ("Done", ProtocolStateNameValues::name(ProtocolState::Done));
 }

test/extensions/internal_redirect/allow_listed_routes/BUILD

@@ -21,4 +21,4 @@ envoy_extension_cc_test(
         "//test/test_common:utility_lib",
         "@envoy_api//envoy/extensions/internal_redirect/allow_listed_routes/v3:pkg_cc_proto",
     ],
-) 
+)

test/extensions/internal_redirect/allow_listed_routes/config_test.cc

@@ -3,8 +3,8 @@
 #include "envoy/router/internal_redirect.h"
 
 #include "source/common/stream_info/filter_state_impl.h"
-#include "source/extensions/internal_redirect/allow_listed_routes/config.h"
 #include "source/extensions/internal_redirect/allow_listed_routes/allow_listed_routes.h"
+#include "source/extensions/internal_redirect/allow_listed_routes/config.h"
 
 #include "gmock/gmock.h"
 #include "gtest/gtest.h"
@@ -22,10 +22,10 @@ class AllowListedRoutesTest : public testing::Test {
     factory_ = Registry::FactoryRegistry<Router::InternalRedirectPredicateFactory>::getFactory(
         "envoy.internal_redirect_predicates.allow_listed_routes");
     config_ = factory_->createEmptyConfigProto();
-    
-    envoy::extensions::internal_redirect::allow_listed_routes::v3::AllowListedRoutesConfig* typed_config =
-        dynamic_cast<envoy::extensions::internal_redirect::allow_listed_routes::v3::AllowListedRoutesConfig*>(
-            config_.get());
+
+    envoy::extensions::internal_redirect::allow_listed_routes::v3::AllowListedRoutesConfig*
+        typed_config = dynamic_cast<envoy::extensions::internal_redirect::allow_listed_routes::v3::
+                                        AllowListedRoutesConfig*>(config_.get());
     typed_config->add_allowed_route_names("route_2");
     typed_config->add_allowed_route_names("route_3");
   }
@@ -41,10 +41,10 @@ TEST_F(AllowListedRoutesTest, PredicateTest) {
 
   // Test route that is in the allow list (allowed)
   EXPECT_TRUE(predicate->acceptTargetRoute(filter_state_, "route_2", false, false));
-  
+
   // Test another route that is in the allow list (allowed)
   EXPECT_TRUE(predicate->acceptTargetRoute(filter_state_, "route_3", false, false));
-  
+
   // Test route that is not in the allow list (not allowed)
   EXPECT_FALSE(predicate->acceptTargetRoute(filter_state_, "route_1", false, false));
 }
@@ -58,4 +58,4 @@ TEST_F(AllowListedRoutesTest, VerifyPredicateNameFunction) {
 } // namespace
 } // namespace InternalRedirect
 } // namespace Extensions
-} // namespace Envoy 
+} // namespace Envoy

test/extensions/internal_redirect/safe_cross_scheme/BUILD

@@ -21,4 +21,4 @@ envoy_extension_cc_test(
         "//test/test_common:utility_lib",
         "@envoy_api//envoy/extensions/internal_redirect/safe_cross_scheme/v3:pkg_cc_proto",
     ],
-) 
+)

test/extensions/internal_redirect/safe_cross_scheme/config_test.cc

@@ -35,13 +35,13 @@ TEST_F(SafeCrossSchemeTest, PredicateTest) {
 
   // Test when downstream is HTTPS and target is HTTPS (allowed)
   EXPECT_TRUE(predicate->acceptTargetRoute(filter_state_, "route_1", true, true));
-  
+
   // Test when downstream is HTTPS and target is HTTP (allowed)
   EXPECT_TRUE(predicate->acceptTargetRoute(filter_state_, "route_1", true, false));
-  
+
   // Test when downstream is HTTP and target is HTTP (allowed)
   EXPECT_TRUE(predicate->acceptTargetRoute(filter_state_, "route_1", false, false));
-  
+
   // Test when downstream is HTTP and target is HTTPS (not allowed)
   EXPECT_FALSE(predicate->acceptTargetRoute(filter_state_, "route_1", false, true));
 }
@@ -55,4 +55,4 @@ TEST_F(SafeCrossSchemeTest, VerifyPredicateNameFunction) {
 } // namespace
 } // namespace InternalRedirect
 } // namespace Extensions
-} // namespace Envoy 
+} // namespace Envoy