refactor: format and linting fix

Author: bsrdjan

setup.py

@@ -33,10 +33,7 @@ def readme_md():
 # Check if SAP NWRFC SDK configured
 SAPNWRFC_HOME = os.environ.get("SAPNWRFC_HOME")
 if not SAPNWRFC_HOME:
-    sys.exit(
-        "Environment variable SAPNWRFC_HOME not set.\n"
-        "Please specify this variable with the root directory of the SAP NWRFC Library."
-    )
+    sys.exit("Environment variable SAPNWRFC_HOME not set.\n" "Please specify this variable with the root directory of the SAP NWRFC Library.")
 
 print("pyrfc version", __version__)
 
@@ -217,7 +214,7 @@ def readme_md():
         "Programming Language :: Python :: 3.9",
         "Programming Language :: Python :: 3.10",
         "Programming Language :: Python :: 3.11",
-        "Topic :: Software Development :: Libraries"
+        "Topic :: Software Development :: Libraries",
     ],
     keywords=f"{MODULE_NAME} {PYPIPACKAGE} pyrfc sap rfc nwrfc sapnwrfc",
     author="SAP SE",
@@ -234,6 +231,7 @@ def readme_md():
     extras_require={':"linux" in sys_platform': ["cython"]},
     cmdclass=CMDCLASS,  # type: ignore
     ext_modules=cythonize(PYRFC_EXT, annotate=True, compiler_directives={"language_level": "1"})  # type: ignore
-        if BUILD_CYTHON else [PYRFC_EXT],  # type: ignore
+    if BUILD_CYTHON
+    else [PYRFC_EXT],  # type: ignore
     test_suite=MODULE_NAME,
 )

src/pyrfc/__init__.py

@@ -59,5 +59,5 @@
 
 __author__ = "SAP SE"
 __email__ = "srdjan.boskovic@sap.com"
-__version_info__ = (2, 7, 2)
+__version_info__ = (2, 7, 3)
 __version__ = ".".join(str(i) for i in __version_info__)

src/pyrfc/_cyrfc.cpp

@@ -56,7 +56,7 @@ class RFCLibError(RFCError):
         30: "RFC_AUTHENTICATION_FAILURE",
         31: "RFC_CRYPTOLIB_FAILURE",
         32: "RFC_IO_FAILURE",
-        33: "RFC_LOCKING_FAILURE"
+        33: "RFC_LOCKING_FAILURE",
     }
 
     def __init__(

src/pyrfc/_exception.py

@@ -10,7 +10,7 @@ _MASK_DTIME = 0x01
 _MASK_RETURN_IMPORT_PARAMS = 0x02
 _MASK_RSTRIP = 0x04
 
-_LOCALE_RADIX = localeconv()['decimal_point']
+_LOCALE_RADIX = localeconv()["decimal_point"]
 
 ################################################################################
 # Enumerators, external and internal use
@@ -32,7 +32,7 @@ class RfcFieldType(Enum):
     RFCTYPE_BCD = RFCTYPE.RFCTYPE_BCD
     RFCTYPE_TIME = RFCTYPE.RFCTYPE_TIME
     RFCTYPE_BYTE = RFCTYPE.RFCTYPE_BYTE
-    RFCTYPE_TABLE= RFCTYPE.RFCTYPE_TABLE
+    RFCTYPE_TABLE = RFCTYPE.RFCTYPE_TABLE
     RFCTYPE_NUM = RFCTYPE.RFCTYPE_NUM
     RFCTYPE_FLOAT = RFCTYPE.RFCTYPE_FLOAT
     RFCTYPE_INT = RFCTYPE.RFCTYPE_INT

src/pyrfc/config.pyx

@@ -146,9 +146,11 @@ def set_locale_radix(value=None):
     _LOCALE_RADIX = value
     return _LOCALE_RADIX
 
+
 def enum_names(enum_obj):
     return set(e.name for e in enum_obj)
 
+
 def enum_values(enum_obj):
     return set(e.value for e in enum_obj)
 

src/pyrfc/nwrfcsdk.pyx

@@ -47,16 +47,8 @@ def compare_function_description(fd1, fd2):
             assert q["type_description"] is None
         else:
             # compare fields
-            fd1_fields = (
-                p["type_description"]["fields"]
-                if type(p["type_description"]) is dict
-                else p["type_description"].fields
-            )
-            fd2_fields = (
-                q["type_description"]["fields"]
-                if type(q["type_description"]) is dict
-                else q["type_description"].fields
-            )
+            fd1_fields = p["type_description"]["fields"] if type(p["type_description"]) is dict else p["type_description"].fields
+            fd2_fields = q["type_description"]["fields"] if type(q["type_description"]) is dict else q["type_description"].fields
             assert len(fd1_fields) == len(fd2_fields)
             assert fd1_fields == fd2_fields
 

tests/function_description_utils.py

@@ -22,9 +22,7 @@ def my_auth_check(func_name=False, request_context={}):
 set_ini_file_directory(dir_path)
 
 # create server instance
-server = Server(
-    {"dest": "gateway"}, {"dest": "MME"}, {"port": 8081, "server_log": False}
-)
+server = Server({"dest": "gateway"}, {"dest": "MME"}, {"port": 8081, "server_log": False})
 
 # expose python function my_stfc_connection as ABAP function STFC_CONNECTION, that ABAP server can call
 server.add_function("STFC_CONNECTION", my_stfc_connection)

tests/server/server.py

@@ -15,7 +15,6 @@
 
 
 class TestConnection:
-
     def test_config_rstrip_false(self):
         conn = pyrfc.Connection(config={"rstrip": False}, **config_sections["coevi51"])
 

tests/test_client_config.py

@@ -98,10 +98,7 @@ def test_call_over_closed_connection(self):
         with pytest.raises(pyrfc.RFCError) as ex:
             conn.call("STFC_CONNECTION", REQUTEXT=hello)
         error = ex.value
-        assert (
-            error.args[0]
-            == "Remote function module 'STFC_CONNECTION' invocation rejected because the connection is closed"
-        )
+        assert error.args[0] == "Remote function module 'STFC_CONNECTION' invocation rejected because the connection is closed"
 
     def test_ping(self):
         assert self.conn.alive
@@ -112,10 +109,7 @@ def test_ping(self):
         error = ex.value
         assert error.code == 13
         assert error.key == "RFC_INVALID_HANDLE"
-        assert (
-            error.message == "An invalid handle 'RFC_CONNECTION_HANDLE' was passed to the API call"
-            or error.message == "An invalid handle was passed to the API call"
-        )
+        assert error.message == "An invalid handle 'RFC_CONNECTION_HANDLE' was passed to the API call" or error.message == "An invalid handle was passed to the API call"
 
     def test_RFM_name_string(self):
         result = self.conn.call("STFC_CONNECTION", REQUTEXT=UNICODETEST)
@@ -195,9 +189,9 @@ def test_STFC_STRUCTURE(self):
         )
         out = dict(
             RFCFLOAT=imp["RFCFLOAT"] + 1,  # type: ignore
-            RFCINT2=imp["RFCINT2"] + 1,   # type: ignore
-            RFCINT1=imp["RFCINT1"] + 1,   # type: ignore
-            RFCINT4=imp["RFCINT4"] + 1,   # type: ignore
+            RFCINT2=imp["RFCINT2"] + 1,  # type: ignore
+            RFCINT1=imp["RFCINT1"] + 1,  # type: ignore
+            RFCINT4=imp["RFCINT4"] + 1,  # type: ignore
             RFCHEX3=b"\xf1\xf2\xf3",
             RFCCHAR1="X",
             RFCCHAR2="YZ",