Skip to content

Commit

Permalink
fix(integration-tests): fix LoaderSetDummy class
Browse files Browse the repository at this point in the history
`params` class attribute for `LoaderSetDummy` started to be required
(due introduction of dns_names). Missing it is causing some integration
tests to fail.

This commit adds `params` attribute to `LoaderSetDummy` class with empty
content (as it's not required)
  • Loading branch information
soyacz authored and fruch committed Nov 9, 2023
1 parent 1f3267e commit 2400d90
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
2 changes: 1 addition & 1 deletion unit_tests/dummy_remote.py
Original file line number Diff line number Diff line change
Expand Up @@ -74,8 +74,8 @@ class LocalLoaderSetDummy(BaseCluster):
# pylint: disable=super-init-not-called,abstract-method
def __init__(self, nodes=None):
self.name = "LocalLoaderSetDummy"
self.nodes = nodes if nodes is not None else [LocalNode("loader_node", parent_cluster=self)]
self.params = {}
self.nodes = nodes if nodes is not None else [LocalNode("loader_node", parent_cluster=self)]

@staticmethod
def get_db_auth():
Expand Down
1 change: 1 addition & 0 deletions unit_tests/test_manual.py
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,7 @@ class DbNode: # pylint: disable=no-init,too-few-public-methods
class LoaderSetDummy: # pylint: disable=no-init,too-few-public-methods
def __init__(self):
self.nodes = [Node()]
self.params = {}

@staticmethod
def get_db_auth():
Expand Down

0 comments on commit 2400d90

Please sign in to comment.