Skip to content

Commit

Permalink
python: fix integration test
Browse files Browse the repository at this point in the history
A recent refactor broke integration tests by generating config envoy would reject.

Signed-off-by: Lance Austin <[email protected]>
  • Loading branch information
Lance Austin authored and LanceEa committed Jan 4, 2024
1 parent c8edb16 commit f9e27c9
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions python/ambassador/ir/irgofilter.py
Original file line number Diff line number Diff line change
Expand Up @@ -63,6 +63,7 @@ def setup(self, ir: "IR", _: Config) -> bool:
self.logger.error(
"%s not found, envoy configuration will fail to apply", GO_FILTER_LIBRARY_PATH
)
return False
self.config = GOFilterConfig(library_path=GO_FILTER_LIBRARY_PATH)
return True
return False
Expand Down
2 changes: 1 addition & 1 deletion python/tests/unit/test_gofilter.py
Original file line number Diff line number Diff line change
Expand Up @@ -116,7 +116,7 @@ def test_gofilter_missing_object_file(go_library, caplog):
econf = get_envoy_config(MAPPING)
filters = _get_go_filters(econf.as_dict())

assert len(filters) == 4
assert len(filters) == 0

assert "/ambassador/filter.so not found" in caplog.text

Expand Down

0 comments on commit f9e27c9

Please sign in to comment.