Feb 21, 2025 · Feb 20, 2025 · Feb 20, 2025 · Feb 20, 2025 · Feb 21, 2025 · Feb 21, 2025
diff --git a/testgres/operations/remote_ops.py b/testgres/operations/remote_ops.py
        )
        return conn

    @staticmethod
    def _make_exec_env_list() -> list[str]:
        result = list[str]()
        for envvar in os.environ.items():

    sm_envs_for_exec_cmd = ["LANG", "LANGUAGE"]

    @staticmethod
    def _does_put_envvar_into_exec_cmd(name: str) -> bool:
        assert type(name) == str  # noqa: E721
        name = name.upper()
            return True
        return False

    @staticmethod
    def _quote_envvar(value) -> str:
        assert type(value) == str  # noqa: E721
        result = "\""
Original file line number	Diff line number	Diff line change
Expand Up		@@ -515,6 +515,7 @@ def db_connect(self, dbname, user, password=None, host="localhost", port=5432):
		)
		return conn

		@staticmethod
		def _make_exec_env_list() -> list[str]:
Copy link Contributor demonolockFeb 21, 2025 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. тут нужно self, либо добавить@staticmethod Copy link CollaboratorAuthor dmitry-lipetskFeb 21, 2025 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. self тут не нужен. Это статический метод. Хорошо, я добавлю этот декоратор. Я там в других местах тоже статические методы определял... Тогда я пока только в новом коде этого PR его добавлю, а потом, отдельным коммитом, добавим его во все остальные случаи. PS. Я этот питон на ходу изучаю, поэтому многие вещи использую неправильно ))))
		result = list[str]()
		for envvar in os.environ.items():
Expand All		@@ -529,6 +530,7 @@ def _make_exec_env_list() -> list[str]:

		sm_envs_for_exec_cmd = ["LANG", "LANGUAGE"]

		@staticmethod
		def _does_put_envvar_into_exec_cmd(name: str) -> bool:
Copy link Contributor demonolockFeb 21, 2025 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. тут нужно self, либо добавить@staticmethod
		assert type(name) == str # noqa: E721
		name = name.upper()
Expand All		@@ -538,6 +540,7 @@ def _does_put_envvar_into_exec_cmd(name: str) -> bool:
		return True
		return False

		@staticmethod
		def _quote_envvar(value) -> str:
Copy link Contributor demonolockFeb 21, 2025 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. тут нужно self, либо добавить@staticmethod
		assert type(value) == str # noqa: E721
		result = "\""
Expand Down