Skip to content

Commit

Permalink
fix asm tests, mocked config object needs to use the new name
Browse files Browse the repository at this point in the history
  • Loading branch information
mabdinur committed Oct 6, 2024
1 parent e5c49e0 commit e456701
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions tests/appsec/appsec/test_processor.py
Original file line number Diff line number Diff line change
Expand Up @@ -590,7 +590,7 @@ def test_ddwaf_info_with_3_errors():
def test_ddwaf_info_with_json_decode_errors(tracer_appsec, caplog):
tracer = tracer_appsec
config = rules.Config()
config.http_tag_query_string = True
config._http_tag_query_string = True

with caplog.at_level(logging.WARNING), mock.patch(
"ddtrace.appsec._processor.json.dumps", side_effect=JSONDecodeError("error", "error", 0)
Expand Down Expand Up @@ -627,7 +627,7 @@ def test_ddwaf_run_contained_typeerror(tracer_appsec, caplog):
tracer = tracer_appsec

config = rules.Config()
config.http_tag_query_string = True
config._http_tag_query_string = True

with caplog.at_level(logging.DEBUG), mock.patch(
"ddtrace.appsec._ddwaf.ddwaf_run", side_effect=TypeError("expected c_long instead of int")
Expand Down Expand Up @@ -665,7 +665,7 @@ def test_ddwaf_run_contained_oserror(tracer_appsec, caplog):
tracer = tracer_appsec

config = rules.Config()
config.http_tag_query_string = True
config._http_tag_query_string = True

with caplog.at_level(logging.DEBUG), mock.patch(
"ddtrace.appsec._ddwaf.ddwaf_run", side_effect=OSError("ddwaf run failed")
Expand Down

0 comments on commit e456701

Please sign in to comment.