LITE-27556 Add black

* Double quotes replaced with single quotes

Author: Hairash

connect/cli/ccli.py

@@ -17,6 +17,7 @@ def main():
     _ignore_openpyxl_warnings()
     try:
         import uvloop
+
         uvloop.install()
     except ImportError:
         pass
@@ -45,6 +46,7 @@ def _ignore_openpyxl_warnings():
     To avoid losing data validation, it should be re-created each time the file is saved by the cli.
     """
     import warnings
+
     warnings.filterwarnings('ignore', category=UserWarning, module='openpyxl.worksheet._reader')
 
 

connect/cli/core/base.py

@@ -23,6 +23,7 @@ def invoke(self, ctx):
 class CCliGroup(click.Group):
     def command(self, *args, **kwargs):
         from click.decorators import command
+
         kwargs['cls'] = CCliCommand
 
         def decorator(f):
@@ -34,6 +35,7 @@ def decorator(f):
 
     def group(self, *args, **kwargs):
         from click.decorators import group
+
         kwargs['cls'] = CCliGroup
 
         def decorator(f):
@@ -45,7 +47,7 @@ def decorator(f):
 
 
 def group(name=None, **attrs):
-    attrs.setdefault("cls", CCliGroup)
+    attrs.setdefault('cls', CCliGroup)
     return click.command(name, **attrs)
 
 
@@ -65,10 +67,13 @@ def print_version(ctx, param, value):
     is_eager=True,
     callback=print_version,
 )
-@click.option('-c', '--config-dir',
-              default=os.path.join(os.path.expanduser('~'), '.ccli'),
-              type=click.Path(file_okay=False),
-              help='set the config directory.')
+@click.option(
+    '-c',
+    '--config-dir',
+    default=os.path.join(os.path.expanduser('~'), '.ccli'),
+    type=click.Path(file_okay=False),
+    help='set the config directory.',
+)
 @click.option(
     '-s',
     '--silent',

connect/cli/core/config.py

@@ -86,7 +86,6 @@ def load(self, config_dir):
             data = json.load(f)
             active_account_id = data['active']
             for account_data in data['accounts']:
-
                 account = Account(
                     **account_data,
                     client=ConnectClient(

connect/cli/core/terminal.py

@@ -89,7 +89,6 @@ def update(
 
 
 class Console(_Console):
-
     def __init__(self, *args, **kwargs):
         super().__init__(*args, **kwargs)
         self._skip_confirm = False
@@ -133,7 +132,7 @@ def page_size(self, value):
     def progress(self):
         return Progress(
             SpinnerColumn(),
-            TextColumn("[progress.description]{task.description:<80}"),
+            TextColumn('[progress.description]{task.description:<80}'),
             BarColumn(style='cyan', finished_style='green3'),
             MofNCompleteColumn(),
             TaskProgressColumn(),
@@ -159,7 +158,8 @@ def confirm(
 
         self.print()
         prompt = Confirm(
-            message, console=self,
+            message,
+            console=self,
         )
 
         result = prompt(default=False)
@@ -236,10 +236,11 @@ def table(
         if not (columns and rows):
             return
 
-        chunks = [
-            rows[i:i + self.page_size]
-            for i in range(0, len(rows), self.page_size)
-        ] if not self.skip_confirm else [rows]
+        chunks = (
+            [rows[i : i + self.page_size] for i in range(0, len(rows), self.page_size)]
+            if not self.skip_confirm
+            else [rows]
+        )
 
         for chunk_count, chunk in enumerate(chunks):
             table = Table(

connect/cli/core/utils.py

@@ -69,15 +69,15 @@ def validate_output_options(output_path, output_file, default_dir_name, default_
 
     output_path = output_path or os.getcwd()
     if not os.path.exists(output_path):
-        raise click.ClickException("Output Path does not exist")
+        raise click.ClickException('Output Path does not exist')
 
     output_path = os.path.join(output_path, default_dir_name)
     if not os.path.exists(output_path):
         os.mkdir(output_path)
     elif not os.path.isdir(output_path):  # pragma: no branch
         raise click.ClickException(
             f"Exists a file with name '{os.path.basename(output_path)}' but a directory is "
-            "expected, please rename it",
+            'expected, please rename it',
         )
 
     output_file = os.path.join(output_path, output_file or f'{default_file_name}.xlsx')
@@ -88,12 +88,10 @@ def validate_output_options(output_path, output_file, default_dir_name, default_
 def field_to_check_mark(predicate, false_value=''):
     """Change the value of a field to a check mark base on a predicate.
 
-        :param predicate: bool value / expression.
-        :param false_value: customizable value if predicate evaluates to false.
-        """
-    return (
-        '\u2713' if predicate else false_value
-    )
+    :param predicate: bool value / expression.
+    :param false_value: customizable value if predicate evaluates to false.
+    """
+    return '\u2713' if predicate else false_value
 
 
 def iter_entry_points(group, name=None):

connect/cli/plugins/customer/commands.py

@@ -57,7 +57,6 @@ def cmd_export_customers(config, output_path, output_file):
     name='sync',
     short_help='Synchronize customers from an excel file.',
 )
-
 @click.argument('input_file', metavar='input_file', nargs=1, required=True)  # noqa: E304
 @pass_config
 def cmd_sync_customers(config, input_file):
@@ -70,7 +69,7 @@ def cmd_sync_customers(config, input_file):
         client=config.active.client,
         account_id=acc_id,
     )
-    warnings.filterwarnings("ignore", category=UserWarning)
+    warnings.filterwarnings('ignore', category=UserWarning)
     synchronizer.open(input_file, 'Customers')
     synchronizer.sync()
     synchronizer.save(input_file)

connect/cli/plugins/customer/export.py

@@ -13,7 +13,10 @@
 
 def dump_customers(client, account_id, output_file, output_path=None):  # noqa: CCR001
     output_file = validate_output_options(
-        output_path, output_file, default_dir_name=account_id, default_file_name='customers',
+        output_path,
+        output_file,
+        default_dir_name=account_id,
+        default_file_name='customers',
     )
     try:
         wb = Workbook()
@@ -26,7 +29,11 @@ def dump_customers(client, account_id, output_file, output_path=None):  # noqa:
         with console.progress() as progress:
             task = progress.add_task('Processing customer', total=count)
             for customer in customers:
-                progress.update(task, description=f'Processing customer {customer["id"]}', advance=1)
+                progress.update(
+                    task,
+                    description=f'Processing customer {customer["id"]}',
+                    advance=1,
+                )
                 _fill_customer_row(wb['Customers'], row_idx, customer)
                 row_idx += 1
             progress.update(task, completed=count)
@@ -83,7 +90,9 @@ def _fill_customer_row(ws, row_idx, customer):
     ws.cell(row_idx, 18, value=customer['contact_info']['contact'].get('last_name', '-'))
     ws.cell(row_idx, 19, value=customer['contact_info']['contact'].get('email', '-'))
     ws.cell(
-        row_idx, 20, value=_get_phone_number(
+        row_idx,
+        20,
+        value=_get_phone_number(
             customer['contact_info']['contact'].get(
                 'phone_number',
                 '-',
@@ -113,7 +122,7 @@ def _get_phone_number(number):
 def _prepare_worksheet(ws):
     color = Color('d3d3d3')
     fill = PatternFill('solid', color)
-    cels = ws['A1': 'T1']
+    cels = ws['A1':'T1']
     for cel in cels[0]:
         if cel.column_letter in ['J', 'K', 'L']:
             ws.column_dimensions[cel.column_letter].width = 50
@@ -136,7 +145,7 @@ def _add_countries(ws):
     ws.column_dimensions['A'].width = 15
     ws.column_dimensions['A'].auto_size = True
     ws.column_dimensions['B'].width = 50
-    cels = ws['A1': 'B1']
+    cels = ws['A1':'B1']
     for cel in cels[0]:
         cel.fill = fill
         cel.value = col_headers[cel.column_letter]

connect/cli/plugins/customer/sync.py

@@ -57,7 +57,7 @@ def _open_workbook(self, input_file):
 
     @staticmethod
     def _validate_worksheet_sheet(ws):
-        cels = ws['A1': 'T1']
+        cels = ws['A1':'T1']
         for cel in cels[0]:
             if cel.value != COL_HEADERS[cel.column_letter]:
                 raise ClickException(
@@ -90,27 +90,30 @@ def sync(self):  # noqa: CCR001
                     self.stats.error(row_errors, row_idx)
                     continue
                 if data.parent_search_criteria and not data.parent_search_value:
-                    self.stats.error("Parent search value is needed if criteria is set", row_idx)
+                    self.stats.error('Parent search value is needed if criteria is set', row_idx)
                     continue
                 if data.hub_id and (data.hub_id != '' or data.hub_id != '-'):
                     if data.hub_id not in self.hubs:
-                        self.stats.error(f"Accounts on hub {data.hub_id} can not be modified", row_idx)
+                        self.stats.error(
+                            f'Accounts on hub {data.hub_id} can not be modified',
+                            row_idx,
+                        )
                         continue
                 name = f'{data.technical_contact_first_name} {data.technical_contact_last_name}'
                 model = {
-                    "type": data.type,
-                    "name": data.company_name if data.company_name else name,
-                    "contact_info": {
-                        "address_line1": data.address_line_1,
-                        "address_line2": data.address_line_2,
-                        "city": data.city,
-                        "country": data.country,
-                        "postal_code": data.zip,
-                        "state": data.state,
-                        "contact": {
-                            "first_name": data.technical_contact_first_name,
-                            "last_name": data.technical_contact_last_name,
-                            "email": data.technical_contact_email,
+                    'type': data.type,
+                    'name': data.company_name if data.company_name else name,
+                    'contact_info': {
+                        'address_line1': data.address_line_1,
+                        'address_line2': data.address_line_2,
+                        'city': data.city,
+                        'country': data.country,
+                        'postal_code': data.zip,
+                        'state': data.state,
+                        'contact': {
+                            'first_name': data.technical_contact_first_name,
+                            'last_name': data.technical_contact_last_name,
+                            'email': data.technical_contact_email,
                         },
                     },
                 }
@@ -124,9 +127,9 @@ def sync(self):  # noqa: CCR001
                     try:
                         phone = phonenumbers.parse(data.technical_contact_phone, data.country)
                         phone_number = {
-                            "country_code": f'+{str(phone.country_code)}',
-                            "area_code": '',
-                            "extension": str(phone.extension) if phone.extension else '-',
+                            'country_code': f'+{str(phone.country_code)}',
+                            'area_code': '',
+                            'extension': str(phone.extension) if phone.extension else '-',
                             'phone_number': str(phone.national_number),
                         }
                         model['contact_info']['contact']['phone_number'] = phone_number
@@ -137,7 +140,9 @@ def sync(self):  # noqa: CCR001
                     if data.parent_search_criteria == 'id':
                         if data.parent_search_value not in parent_id:
                             try:
-                                pacc = self._client.ns('tier').accounts[data.parent_search_value].get()
+                                pacc = (
+                                    self._client.ns('tier').accounts[data.parent_search_value].get()
+                                )
                                 parent_id.append(pacc['id'])
                             except ClientError:
                                 self.stats.error(
@@ -167,7 +172,8 @@ def sync(self):  # noqa: CCR001
                                 parent_external_id[data.parent_search_value] = pacc[0]['id']
                             except ClientError:
                                 self.stats.error(
-                                    'Error when obtaining parent data from Connect', row_idx,
+                                    'Error when obtaining parent data from Connect',
+                                    row_idx,
                                 )
                                 continue
                         model['parent']['id'] = parent_external_id[data.parent_search_value]
@@ -191,7 +197,8 @@ def sync(self):  # noqa: CCR001
                                 parent_uuid[data.parent_search_value] = pacc[0]['id']
                             except ClientError:
                                 self.stats.error(
-                                    'Error when obtaining parent data from Connect', row_idx,
+                                    'Error when obtaining parent data from Connect',
+                                    row_idx,
                                 )
                                 continue
                         model['parent']['id'] = parent_uuid[data.parent_search_value]
@@ -200,7 +207,8 @@ def sync(self):  # noqa: CCR001
                         account = self._client.ns('tier').accounts.create(model)
                     except ClientError as e:
                         self.stats.error(
-                            f'Error when creating account: {str(e)}', row_idx,
+                            f'Error when creating account: {str(e)}',
+                            row_idx,
                         )
                         continue
                     self.stats.created()
@@ -211,7 +219,8 @@ def sync(self):  # noqa: CCR001
                         account = self._client.ns('tier').accounts[data.id].update(model)
                     except ClientError as e:
                         self.stats.error(
-                            f'Error when updating account: {str(e)}', row_idx,
+                            f'Error when updating account: {str(e)}',
+                            row_idx,
                         )
                         continue
                     self.stats.updated()
@@ -235,23 +244,25 @@ def _validate_row(self, row):
             errors.append(f'Customer type must be customer or reseller, not {row.type}')
             return errors
         if not all(
-                [
-                    row.address_line_1,
-                    row.city,
-                    row.state,
-                    row.zip,
-                    row.technical_contact_first_name,
-                    row.technical_contact_last_name,
-                    row.technical_contact_email,
-                ],
+            [
+                row.address_line_1,
+                row.city,
+                row.state,
+                row.zip,
+                row.technical_contact_first_name,
+                row.technical_contact_last_name,
+                row.technical_contact_email,
+            ],
         ):
             errors.append('Address line 1, city, state and zip are mandatory')
             return errors
         if row.action == 'create' and row.id is not None:
             errors.append(f'Create action must not have account id, is set to {row.id}')
             return errors
-        if row.action == 'create' and row.type == 'customer' and (
-                row.parent_search_criteria == '' or row.parent_search_criteria == '-'
+        if (
+            row.action == 'create'
+            and row.type == 'customer'
+            and (row.parent_search_criteria == '' or row.parent_search_criteria == '-')
         ):
             errors.append('Customers requires a parent account')
             return errors

connect/cli/plugins/locale/commands.py

@@ -41,10 +41,10 @@ def cmd_list_locales(config, query):
         ],
         rows=[
             (
-                resource["id"],
-                resource["name"],
-                field_to_check_mark(resource["auto_translation"], false_value='\u2716'),
-                resource["stats"]["translations"] or '-',
+                resource['id'],
+                resource['name'],
+                field_to_check_mark(resource['auto_translation'], false_value='\u2716'),
+                resource['stats']['translations'] or '-',
             )
             for resource in query_locales
         ],