chore!: drop backup.py direct calls + pragma nocov
This commit is contained in:
parent
aa1f03239c
commit
290a3df68d
3 changed files with 7 additions and 74 deletions
6
.github/helper/ci.py
vendored
6
.github/helper/ci.py
vendored
|
|
@ -24,6 +24,9 @@ STANDARD_EXCLUSIONS = [
|
|||
# tested via commands' test suite
|
||||
TESTED_VIA_CLI = [
|
||||
"*/frappe/installer.py",
|
||||
"*/frappe/utils/install.py",
|
||||
"*/frappe/utils/scheduler.py",
|
||||
"*/frappe/utils/doctor.py",
|
||||
"*/frappe/build.py",
|
||||
"*/frappe/database/__init__.py",
|
||||
"*/frappe/database/db_manager.py",
|
||||
|
|
@ -35,10 +38,13 @@ FRAPPE_EXCLUSIONS = [
|
|||
"*/commands/*",
|
||||
"*/frappe/change_log/*",
|
||||
"*/frappe/exceptions*",
|
||||
"*/frappe/desk/page/setup_wizard/setup_wizard.py",
|
||||
"*/frappe/coverage.py",
|
||||
"*frappe/setup.py",
|
||||
"*/frappe/hooks.py",
|
||||
"*/doctype/*/*_dashboard.py",
|
||||
"*/patches/*",
|
||||
"*/.github/helper/ci.py",
|
||||
] + TESTED_VIA_CLI
|
||||
|
||||
|
||||
|
|
|
|||
|
|
@ -219,7 +219,7 @@ def start_worker(
|
|||
rq_password: str | None = None,
|
||||
burst: bool = False,
|
||||
strategy: Literal["round_robin", "random"] | None = None,
|
||||
) -> NoReturn | None:
|
||||
) -> NoReturn | None: # pragma: no cover
|
||||
"""Wrapper to start rq worker. Connects to redis and monitors these queues."""
|
||||
DEQUEUE_STRATEGIES = {"round_robin": RoundRobinWorker, "random": RandomWorker}
|
||||
|
||||
|
|
|
|||
|
|
@ -710,76 +710,3 @@ def backup(
|
|||
"backup_path_files": odb.backup_path_files,
|
||||
"backup_path_private_files": odb.backup_path_private_files,
|
||||
}
|
||||
|
||||
|
||||
if __name__ == "__main__":
|
||||
import sys
|
||||
|
||||
from frappe.utils.commands import warn
|
||||
|
||||
warn(
|
||||
"Calling the backup script directly is deprecated. "
|
||||
"Use the backup command instead. This script will be removed in Frappe v15.",
|
||||
category=DeprecationWarning,
|
||||
)
|
||||
|
||||
cmd = sys.argv[1]
|
||||
|
||||
db_type = "mariadb"
|
||||
try:
|
||||
db_type = sys.argv[6]
|
||||
except IndexError:
|
||||
pass
|
||||
|
||||
db_port = 3306
|
||||
try:
|
||||
db_port = int(sys.argv[7])
|
||||
except IndexError:
|
||||
pass
|
||||
|
||||
if cmd == "is_file_old":
|
||||
odb = BackupGenerator(
|
||||
sys.argv[2],
|
||||
sys.argv[3],
|
||||
sys.argv[4],
|
||||
sys.argv[5] or "localhost",
|
||||
db_type=db_type,
|
||||
db_port=db_port,
|
||||
)
|
||||
is_file_old(odb.db_file_name)
|
||||
|
||||
if cmd == "get_backup":
|
||||
odb = BackupGenerator(
|
||||
sys.argv[2],
|
||||
sys.argv[3],
|
||||
sys.argv[4],
|
||||
sys.argv[5] or "localhost",
|
||||
db_type=db_type,
|
||||
db_port=db_port,
|
||||
)
|
||||
odb.get_backup()
|
||||
|
||||
if cmd == "take_dump":
|
||||
odb = BackupGenerator(
|
||||
sys.argv[2],
|
||||
sys.argv[3],
|
||||
sys.argv[4],
|
||||
sys.argv[5] or "localhost",
|
||||
db_type=db_type,
|
||||
db_port=db_port,
|
||||
)
|
||||
odb.take_dump()
|
||||
|
||||
if cmd == "send_email":
|
||||
odb = BackupGenerator(
|
||||
sys.argv[2],
|
||||
sys.argv[3],
|
||||
sys.argv[4],
|
||||
sys.argv[5] or "localhost",
|
||||
db_type=db_type,
|
||||
db_port=db_port,
|
||||
)
|
||||
odb.send_email()
|
||||
|
||||
if cmd == "delete_temp_backups":
|
||||
delete_temp_backups()
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue