Skip to content

Commit

Permalink
comments
Browse files Browse the repository at this point in the history
Signed-off-by: Alyssa Wilk <[email protected]>
  • Loading branch information
alyssawilk committed Nov 15, 2023
1 parent 910385a commit 20beffb
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -109,6 +109,7 @@ sampling_window: 1337s
admission_control_filter_factory
.createFilterFactoryFromProtoTyped(proto, "whatever", dual_info_,
factory_context.getServerFactoryContext())
.status()
.IgnoreError(),
EnvoyException, "Success rate threshold cannot be less than 1.0%.");
}
Expand Down
5 changes: 3 additions & 2 deletions test/extensions/filters/http/aws_lambda/config_test.cc
Original file line number Diff line number Diff line change
Expand Up @@ -112,8 +112,9 @@ arn: "arn:aws:lambda:region:424242:fun"
testing::NiceMock<Server::Configuration::MockFactoryContext> context;
AwsLambdaFilterFactory factory;

EXPECT_THROW(factory.createFilterFactoryFromProto(proto_config, "stats", context).value(),
EnvoyException);
EXPECT_THROW(
factory.createFilterFactoryFromProto(proto_config, "stats", context).status().IgnoreError(),
EnvoyException);
}

TEST(AwsLambdaFilterConfigTest, PerRouteConfigWithInvalidARNThrows) {
Expand Down
5 changes: 3 additions & 2 deletions test/extensions/filters/http/cache/config_test.cc
Original file line number Diff line number Diff line change
Expand Up @@ -54,8 +54,9 @@ TEST_F(CacheFilterFactoryTest, NoTypedConfig) {
TEST_F(CacheFilterFactoryTest, UnregisteredTypedConfig) {
config_.mutable_typed_config()->PackFrom(
envoy::extensions::filters::http::cache::v3::CacheConfig());
EXPECT_THROW(factory_.createFilterFactoryFromProto(config_, "stats", context_).IgnoreError(),
EnvoyException);
EXPECT_THROW(
factory_.createFilterFactoryFromProto(config_, "stats", context_).status().IgnoreError(),
EnvoyException);
}

} // namespace
Expand Down

0 comments on commit 20beffb

Please sign in to comment.