[IMP] update dotfiles

Author: OCA-git-bot

.copier-answers.yml

@@ -1,16 +1,19 @@
 # Do NOT update manually; changes here will be overwritten by Copier
-_commit: v1.5.1
+_commit: v1.5.2
 _src_path: gh:oca/oca-addons-repo-template
 ci: GitHub
 dependency_installation_mode: PIP
 generate_requirements_txt: true
 include_wkhtmltopdf: true
 odoo_version: 14.0
+org_name: Odoo Community Association (OCA)
+org_slug: OCA
 rebel_module_groups:
 - fetchmail_incoming_log,fetchmail_notify_error_to_sender_test,fetchmail_notify_error_to_sender
 repo_description: This project aims to deal with modules related to manage Odoo server
   environment and provide useful tools.
 repo_name: Tools for server environment(s)
 repo_slug: server-tools
+repo_website: https://github.com/OCA/server-tools
 travis_apt_packages: []
 travis_apt_sources: []

.eslintrc.yml

@@ -4,7 +4,7 @@ env:
 
 # See https://github.com/OCA/odoo-community.org/issues/37#issuecomment-470686449
 parserOptions:
-  ecmaVersion: 2017
+  ecmaVersion: 2019
 
 overrides:
   - files:

.github/workflows/pre-commit.yml

@@ -2,21 +2,19 @@ name: pre-commit
 
 on:
   pull_request:
+    branches:
+      - "14.0*"
   push:
+    branches:
+      - "14.0"
+      - "14.0-ocabot-*"
 
 jobs:
   pre-commit:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
       - uses: actions/setup-python@v2
-        with:
-          # The pylint-odoo version we use here does not support python 3.10
-          # https://github.com/OCA/oca-addons-repo-template/issues/80
-          # We also need to pin to an older version of python for older odoo versions
-          # where we are not using black > 21. Older black versions won't work with
-          # Python 3.9.8+, and we can't bump black without reformatting.
-          python-version: "3.9.7"
       - name: Get python version
         run: echo "PY=$(python -VV | sha256sum | cut -d' ' -f1)" >> $GITHUB_ENV
       - uses: actions/cache@v1

.github/workflows/test.yml

@@ -6,7 +6,8 @@ on:
       - "14.0*"
   push:
     branches:
-      - "14.0*"
+      - "14.0"
+      - "14.0-ocabot-*"
 
 jobs:
   unreleased-deps:

.pre-commit-config.yaml

@@ -46,7 +46,7 @@ repos:
           - --remove-duplicate-keys
           - --remove-unused-variables
   - repo: https://github.com/psf/black
-    rev: 20.8b1
+    rev: 22.3.0
     hooks:
       - id: black
   - repo: https://github.com/pre-commit/mirrors-prettier
@@ -120,7 +120,7 @@ repos:
         name: flake8
         additional_dependencies: ["flake8-bugbear==20.1.4"]
   - repo: https://github.com/PyCQA/pylint
-    rev: pylint-2.5.3
+    rev: v2.11.1
     hooks:
       - id: pylint
         name: pylint with optional checks
@@ -129,7 +129,7 @@ repos:
           - --exit-zero
         verbose: true
         additional_dependencies: &pylint_deps
-          - pylint-odoo==3.5.0
+          - pylint-odoo==5.0.5
       - id: pylint
         name: pylint with mandatory checks
         args:

README.md

@@ -1,3 +1,4 @@
+
 [![Runboat](https://img.shields.io/badge/runboat-Try%20me-875A7B.png)](https://runboat.odoo-community.org/builds?repo=OCA/server-tools&target_branch=14.0)
 [![Pre-commit Status](https://github.com/OCA/server-tools/actions/workflows/pre-commit.yml/badge.svg?branch=14.0)](https://github.com/OCA/server-tools/actions/workflows/pre-commit.yml?query=branch%3A14.0)
 [![Build Status](https://github.com/OCA/server-tools/actions/workflows/test.yml/badge.svg?branch=14.0)](https://github.com/OCA/server-tools/actions/workflows/test.yml?query=branch%3A14.0)
@@ -82,12 +83,11 @@ addon | version | maintainers | summary
 
 This repository is licensed under [AGPL-3.0](LICENSE).
 
-However, each module can have a totally different license, as long as they adhere to OCA
+However, each module can have a totally different license, as long as they adhere to Odoo Community Association (OCA)
 policy. Consult each module's `__manifest__.py` file, which contains a `license` key
 that explains its license.
 
 ----
-
 OCA, or the [Odoo Community Association](http://odoo-community.org/), is a nonprofit
 organization whose mission is to support the collaborative development of Odoo features
 and promote its widespread use.

auditlog/models/log.py

@@ -27,7 +27,7 @@ class AuditlogLog(models.Model):
 
     @api.model_create_multi
     def create(self, vals_list):
-        """ Insert model_name and model_model field values upon creation. """
+        """Insert model_name and model_model field values upon creation."""
         for vals in vals_list:
             if not vals.get("model_id"):
                 raise UserError(_("No model defined to create log."))

auditlog/tests/test_auditlog.py

@@ -301,7 +301,7 @@ def assert_values(self):
         self.assertEqual(self.auditlog_rule.model_model, "x_test.model")
 
     def test_01_field_and_model_removal(self):
-        """ Test field and model removal to check auditlog line persistence """
+        """Test field and model removal to check auditlog line persistence"""
         self.assert_values()
 
         # Remove the field

auto_backup/tests/test_db_backup.py

@@ -35,7 +35,7 @@ def setUp(self):
 
     @contextmanager
     def mock_assets(self):
-        """ It provides mocked core assets """
+        """It provides mocked core assets"""
         self.path_join_val = "/this/is/a/path"
         with patch("%s.db" % model) as db:
             with patch("%s.os" % model) as os:
@@ -49,15 +49,15 @@ def mock_assets(self):
 
     @contextmanager
     def patch_filtered_sftp(self, record):
-        """ It patches filtered record and provides a mock """
+        """It patches filtered record and provides a mock"""
         with patch("%s.filtered" % class_name) as filtered:
             filtered.side_effect = [], [record]
             with patch("%s.backup_log" % class_name):
                 with patch("%s.sftp_connection" % class_name):
                     yield filtered
 
     def new_record(self, method="sftp"):
-        vals = {"name": u"Têst backup", "method": method, "days_to_keep": 1}
+        vals = {"name": "Têst backup", "method": method, "days_to_keep": 1}
         if method == "sftp":
             vals.update(
                 {
@@ -72,7 +72,7 @@ def new_record(self, method="sftp"):
         return self.Model.create(vals)
 
     def test_compute_name_sftp(self):
-        """ It should create proper SFTP URI """
+        """It should create proper SFTP URI"""
         rec_id = self.new_record()
         self.assertEqual(
             "sftp://%(user)s@%(host)s:%(port)s%(folder)s"
@@ -86,7 +86,7 @@ def test_compute_name_sftp(self):
         )
 
     def test_check_folder(self):
-        """ It should not allow recursive backups """
+        """It should not allow recursive backups"""
         rec_id = self.new_record("local")
         with self.assertRaises(UserError):
             rec_id.write(
@@ -98,7 +98,7 @@ def test_check_folder(self):
 
     @patch("%s._" % model)
     def test_action_sftp_test_connection_success(self, _):
-        """ It should raise connection succeeded warning """
+        """It should raise connection succeeded warning"""
         with patch("%s.sftp_connection" % class_name, new_callable=PropertyMock):
             rec_id = self.new_record()
             with self.assertRaises(UserError):
@@ -107,7 +107,7 @@ def test_action_sftp_test_connection_success(self, _):
 
     @patch("%s._" % model)
     def test_action_sftp_test_connection_fail(self, _):
-        """ It should raise connection fail warning """
+        """It should raise connection fail warning"""
         with patch(
             "%s.sftp_connection" % class_name, new_callable=PropertyMock
         ) as conn:
@@ -118,15 +118,15 @@ def test_action_sftp_test_connection_fail(self, _):
             _.assert_called_once_with("Connection Test Failed!")
 
     def test_action_backup_local(self):
-        """ It should backup local database """
+        """It should backup local database"""
         rec_id = self.new_record("local")
         filename = rec_id.filename(datetime.now())
         rec_id.action_backup()
         generated_backup = [f for f in os.listdir(rec_id.folder) if f >= filename]
         self.assertEqual(1, len(generated_backup))
 
     def test_action_backup_local_cleanup(self):
-        """ Backup local database and cleanup old databases """
+        """Backup local database and cleanup old databases"""
         rec_id = self.new_record("local")
         old_date = datetime.now() - timedelta(days=3)
         filename = rec_id.filename(old_date)
@@ -142,7 +142,7 @@ def test_action_backup_local_cleanup(self):
         self.assertEqual(1, len(generated_backup))
 
     def test_action_backup_sftp_mkdirs(self):
-        """ It should create remote dirs """
+        """It should create remote dirs"""
         rec_id = self.new_record()
         with self.mock_assets():
             with self.patch_filtered_sftp(rec_id):
@@ -152,7 +152,7 @@ def test_action_backup_sftp_mkdirs(self):
                     conn.makedirs.assert_called_once_with(rec_id.folder)
 
     def test_action_backup_sftp_mkdirs_conn_exception(self):
-        """ It should guard from ConnectionException on remote.mkdirs """
+        """It should guard from ConnectionException on remote.mkdirs"""
         rec_id = self.new_record()
         with self.mock_assets():
             with self.patch_filtered_sftp(rec_id):
@@ -164,7 +164,7 @@ def test_action_backup_sftp_mkdirs_conn_exception(self):
                     self.assertTrue(True)
 
     def test_action_backup_sftp_remote_open(self):
-        """ It should open remote file w/ proper args """
+        """It should open remote file w/ proper args"""
         rec_id = self.new_record()
         with self.mock_assets() as assets:
             with self.patch_filtered_sftp(rec_id):
@@ -174,22 +174,22 @@ def test_action_backup_sftp_remote_open(self):
                     conn.open.assert_called_once_with(assets["os"].path.join(), "wb")
 
     def test_action_backup_all_search(self):
-        """ It should search all records """
+        """It should search all records"""
         rec_id = self.new_record()
         with patch("%s.search" % class_name, new_callable=PropertyMock):
             rec_id.action_backup_all()
             rec_id.search.assert_called_once_with([])
 
     def test_action_backup_all_return(self):
-        """ It should return result of backup operation """
+        """It should return result of backup operation"""
         rec_id = self.new_record()
         with patch("%s.search" % class_name, new_callable=PropertyMock):
             res = rec_id.action_backup_all()
             self.assertEqual(rec_id.search().action_backup(), res)
 
     @patch("%s.pysftp" % model)
     def test_sftp_connection_init_passwd(self, pysftp):
-        """ It should initiate SFTP connection w/ proper args and pass """
+        """It should initiate SFTP connection w/ proper args and pass"""
         rec_id = self.new_record()
         rec_id.sftp_connection()
         pysftp.Connection.assert_called_once_with(
@@ -201,7 +201,7 @@ def test_sftp_connection_init_passwd(self, pysftp):
 
     @patch("%s.pysftp" % model)
     def test_sftp_connection_init_key(self, pysftp):
-        """ It should initiate SFTP connection w/ proper args and key """
+        """It should initiate SFTP connection w/ proper args and key"""
         rec_id = self.new_record()
         rec_id.write({"sftp_private_key": "pkey", "sftp_password": "pkeypass"})
         rec_id.sftp_connection()
@@ -215,7 +215,7 @@ def test_sftp_connection_init_key(self, pysftp):
 
     @patch("%s.pysftp" % model)
     def test_sftp_connection_return(self, pysftp):
-        """ It should return new sftp connection """
+        """It should return new sftp connection"""
         rec_id = self.new_record()
         res = rec_id.sftp_connection()
         self.assertEqual(
@@ -224,19 +224,19 @@ def test_sftp_connection_return(self, pysftp):
         )
 
     def test_filename_default(self):
-        """ It should not error and should return a .dump.zip file str """
+        """It should not error and should return a .dump.zip file str"""
         now = datetime.now()
         res = self.Model.filename(now)
         self.assertTrue(res.endswith(".dump.zip"))
 
     def test_filename_zip(self):
-        """ It should return a dump.zip filename"""
+        """It should return a dump.zip filename"""
         now = datetime.now()
         res = self.Model.filename(now, ext="zip")
         self.assertTrue(res.endswith(".dump.zip"))
 
     def test_filename_dump(self):
-        """ It should return a dump filename"""
+        """It should return a dump filename"""
         now = datetime.now()
         res = self.Model.filename(now, ext="dump")
         self.assertTrue(res.endswith(".dump"))

base_changeset/models/record_changeset_change.py

@@ -192,7 +192,7 @@ def get_new_value(self):
         return self[field_name]
 
     def set_old_value(self):
-        """ Copy the value of the record to the 'old' field """
+        """Copy the value of the record to the 'old' field"""
         for change in self:
             # copy the existing record's value for the history
             old_value_for_write = self._value_for_changeset(
@@ -261,7 +261,7 @@ def _check_previous_changesets(self, changeset):
                 )
 
     def cancel(self):
-        """ Reject the change """
+        """Reject the change"""
         for change in self:
             if not change.user_can_validate_changeset:
                 raise UserError(_("You don't have the rights to reject the changes."))