[Tests] disable Memoryhuey

Author: GuillaumeDSM

octobot_node/app/core/config.py

@@ -85,7 +85,6 @@ def all_cors_origins(self) -> list[str]:
 
     SENTRY_DSN: HttpUrl | None = None
     SCHEDULER_REDIS_URL: AnyUrl | None = None
-    USE_MEMORY_SCHEDULER: bool = False
     SCHEDULER_SQLITE_FILE: str = "tasks.db"
     SCHEDULER_WORKERS: int = 0  # 0 disables consumers, >0 enables consumers
     IS_MASTER_MODE: bool = False  # Enable master node mode

octobot_node/scheduler/scheduler.py

@@ -14,7 +14,7 @@
 #  You should have received a copy of the GNU General Public
 #  License along with OctoBot. If not, see <https://www.gnu.org/licenses/>.
 
-from huey import Huey, RedisHuey, SqliteHuey, MemoryHuey
+from huey import Huey, RedisHuey, SqliteHuey
 from huey.registry import Message
 from huey.utils import Error as HueyError
 from typing import Optional, Any
@@ -52,10 +52,6 @@ def create(self):
             ) if settings.REDIS_STORAGE_CERTS_PATH is not None else None
 
             self.INSTANCE = RedisHuey(DEFAULT_NAME, connection_pool=connection_pool) if connection_pool is not None else RedisHuey(DEFAULT_NAME, url=str(settings.SCHEDULER_REDIS_URL))
-        elif settings.USE_MEMORY_SCHEDULER:
-            # warning: this mode has no persistence, tasks will be lost when the process is stopped
-            self.logger.info("Initializing scheduler with memory backend")
-            self.INSTANCE = MemoryHuey(DEFAULT_NAME)
         else:
             self.logger.info(
                 "Initializing scheduler with sqlite backend at %s", settings.SCHEDULER_SQLITE_FILE

tests/__init__.py

@@ -13,7 +13,3 @@
 #
 #  You should have received a copy of the GNU Lesser General Public
 #  License along with this library.
-
-import octobot_node.app.core.config
-# force memory scheduler for tests to avoid disk I/O or redis dependencies
-octobot_node.app.core.config.settings.USE_MEMORY_SCHEDULER = True