[IMP] base_external_dbsource_mysql: black, isort, prettier

This commit is contained in:
David Alonso (Solvos)
2020-09-01 18:09:29 +02:00
committed by Víctor Martínez
parent 14df31c0ec
commit 52e0147d3e
5 changed files with 34 additions and 47 deletions

View File

@@ -2,25 +2,14 @@
# Copyright 2016 LasLabs Inc.
# License LGPL-3.0 or later (http://www.gnu.org/licenses/lgpl).
{
'name': 'External Database Source - MySQL',
'version': '12.0.1.0.0',
'category': 'Tools',
'author': "Daniel Reis, "
"LasLabs, "
"Odoo Community Association (OCA)",
'website': 'https://github.com/OCA/server-backend',
'license': 'LGPL-3',
'depends': [
'base_external_dbsource',
],
'external_dependencies': {
'python': [
'sqlalchemy',
'MySQLdb',
],
},
'demo': [
'demo/base_external_dbsource.xml',
],
'installable': True,
"name": "External Database Source - MySQL",
"version": "12.0.1.0.0",
"category": "Tools",
"author": "Daniel Reis, " "LasLabs, " "Odoo Community Association (OCA)",
"website": "https://github.com/OCA/server-backend",
"license": "LGPL-3",
"depends": ["base_external_dbsource",],
"external_dependencies": {"python": ["sqlalchemy", "MySQLdb",],},
"demo": ["demo/base_external_dbsource.xml",],
"installable": True,
}

View File

@@ -1,8 +1,10 @@
<?xml version="1.0"?>
<?xml version="1.0" ?>
<odoo>
<record model="base.external.dbsource" id="demo_mysql">
<field name="name">MySQL Demo</field>
<field name="conn_string">Server=myServerAddress;Database=myDataBase;Uid=myUsername;</field>
<field
name="conn_string"
>Server=myServerAddress;Database=myDataBase;Uid=myUsername;</field>
<field name="password">password</field>
<field name="connector">mysql</field>
</record>

View File

@@ -9,24 +9,27 @@ from odoo import models
_logger = logging.getLogger(__name__)
try:
from odoo.addons.base_external_dbsource.models import (
base_external_dbsource,
)
from odoo.addons.base_external_dbsource.models import base_external_dbsource
CONNECTORS = base_external_dbsource.BaseExternalDbsource.CONNECTORS
try:
import MySQLdb
CONNECTORS.append(('mysql', 'MySQL'))
CONNECTORS.append(("mysql", "MySQL"))
assert MySQLdb
except (ImportError, AssertionError):
_logger.info('MySQLdb not available. Please install "mysqlclient" '
'python package.')
_logger.info(
'MySQLdb not available. Please install "mysqlclient" ' "python package."
)
try:
import sqlalchemy
except ImportError:
_logger.info('SQLAlchemy library not available. Please '
'install "sqlalchemy" python package.')
_logger.info(
"SQLAlchemy library not available. Please "
'install "sqlalchemy" python package.'
)
except ImportError:
_logger.info('base_external_dbsource Odoo module not found.')
_logger.info("base_external_dbsource Odoo module not found.")
class BaseExternalDbsource(models.Model):

View File

@@ -4,18 +4,15 @@ import mock
from odoo.tests import common
ADAPTER = ('odoo.addons.base_external_dbsource_mysql.models'
'.base_external_dbsource.MySQLdb')
ADAPTER = (
"odoo.addons.base_external_dbsource_mysql.models" ".base_external_dbsource.MySQLdb"
)
class TestBaseExternalDbsource(common.TransactionCase):
def setUp(self):
super(TestBaseExternalDbsource, self).setUp()
self.dbsource = self.env.ref(
'base_external_dbsource_mysql.demo_mysql',
)
self.dbsource = self.env.ref("base_external_dbsource_mysql.demo_mysql",)
def test_connection_close_mysql(self):
""" It should close the connection """
@@ -25,17 +22,13 @@ class TestBaseExternalDbsource(common.TransactionCase):
def test_connection_open_mysql(self):
""" It should call SQLAlchemy open """
with mock.patch.object(
self.dbsource, 'connection_open_mysql'
) as parent_method:
with mock.patch.object(self.dbsource, "connection_open_mysql") as parent_method:
self.dbsource.connection_open_mysql()
parent_method.assert_called_once_with()
def test_excecute_mysql(self):
""" It should pass args to SQLAlchemy execute """
expect = 'sqlquery', 'sqlparams', 'metadata'
with mock.patch.object(
self.dbsource, 'execute_mysql'
) as parent_method:
expect = "sqlquery", "sqlparams", "metadata"
with mock.patch.object(self.dbsource, "execute_mysql") as parent_method:
self.dbsource.execute_mysql(*expect)
parent_method.assert_called_once_with(*expect)