ruff B009 (getattr with constant attribute value)

mmgen/cfg.py

@@ -577,7 +577,7 @@ class Config(Lockable):
 
 		if 'usage' in self._uopts: # requires self.coin
 			import importlib
-			getattr(importlib.import_module(UserOpts.help_pkg), 'usage')(self) # exits
+			importlib.import_module(UserOpts.help_pkg).usage(self) # exits
 
 		# self.color is finalized, so initialize color:
 		if self.color: # MMGEN_DISABLE_COLOR sets this to False

mmgen/keygen.py

@@ -70,9 +70,8 @@ def get_backends(pubkey_type):
 
 def get_pubkey_type_cls(pubkey_type):
 	import importlib
-	return getattr(
-		importlib.import_module(f'mmgen.proto.{backend_data[pubkey_type]["package"]}.keygen'),
-		'backend')
+	return importlib.import_module(
+		f'mmgen.proto.{backend_data[pubkey_type]["package"]}.keygen').backend
 
 def _check_backend(cfg, backend, pubkey_type, *, desc='keygen backend'):
 

mmgen/main_tool.py

@@ -348,11 +348,11 @@ def process_result(ret, *, pager=False, print_result=False):
 def get_cmd_cls(cmd):
 	for modname, cmdlist in mods.items():
 		if cmd in cmdlist:
-			return getattr(importlib.import_module(f'mmgen.tool.{modname}'), 'tool_cmd')
+			return importlib.import_module(f'mmgen.tool.{modname}').tool_cmd
 	return False
 
 def get_mod_cls(modname):
-	return getattr(importlib.import_module(f'mmgen.tool.{modname}'), 'tool_cmd')
+	return importlib.import_module(f'mmgen.tool.{modname}').tool_cmd
 
 if gc.prog_name.endswith('-tool'):
 

mmgen/msg.py

@@ -365,9 +365,8 @@ def _get_obj(clsname, cfg, *args, coin=None, network='mainnet', infile=None, dat
 		coin_msg.base.get_proto_from_file(cfg, infile))
 
 	try:
-		msg_cls = getattr(
-			importlib.import_module(f'mmgen.proto.{proto.base_proto_coin.lower()}.msg'),
-			'coin_msg')
+		msg_cls = importlib.import_module(
+			f'mmgen.proto.{proto.base_proto_coin.lower()}.msg').coin_msg
 	except:
 		die(1, f'Message signing operations not supported for {proto.base_proto} protocol')
 

mmgen/proto/cosmos/tx/protobuf.py

@@ -113,7 +113,7 @@ class TxMsg(BaseMessage):
 		msg_cls = getattr(cls.msgs_cls, id.removeprefix('/types.'))
 		me = BaseMessage.__new__(msg_cls)
 		me.id = id
-		me.body = getattr(msg_cls, 'Body').loads(bodyBytes)
+		me.body = msg_cls.Body.loads(bodyBytes)
 		return me
 
 @dataclass

mmgen/tx/completed.py

@@ -64,7 +64,7 @@ class Completed(Base):
 		from .unsigned import Unsigned, AutomountUnsigned
 		from .online import Sent, AutomountSent
 		for cls in (Unsigned, AutomountUnsigned, Sent, AutomountSent):
-			if ext == getattr(cls, 'ext'):
+			if ext == cls.ext:
 				return cls
 
 		if proto.tokensym:
@@ -73,7 +73,7 @@ class Completed(Base):
 		else:
 			from .signed import Signed, AutomountSigned
 		for cls in (Signed, AutomountSigned):
-			if ext == getattr(cls, 'ext'):
+			if ext == cls.ext:
 				return cls
 
 	def check_swap_memo(self):

mmgen/util.py

@@ -408,12 +408,12 @@ class oneshot_warning:
 
 		def do_warning():
 			from . import color
-			msg(getattr(color, getattr(wcls, 'color'))('WARNING: ' + getattr(wcls, 'message').format(*fmt_args)))
+			msg(getattr(color, wcls.color)('WARNING: ' + wcls.message.format(*fmt_args)))
 
 		if not hasattr(wcls, 'data'):
 			setattr(wcls, 'data', [])
 
-		data = getattr(wcls, 'data')
+		data = wcls.data
 		condition = (div in data) if reverse else (not div in data)
 
 		if not div in data:

mmgen/wallet/__init__.py

@@ -79,15 +79,12 @@ def get_wallet_cls(
 		ext         = None,
 		die_on_fail = False):
 
-	return getattr(
-		importlib.import_module('mmgen.wallet.{}'.format(
+	return importlib.import_module('mmgen.wallet.{}'.format(
 			wtype or
 			get_wallet_data(
 				fmt_code    = fmt_code,
 				ext         = ext,
-				die_on_fail = die_on_fail).type
-		)),
-		'wallet')
+				die_on_fail = die_on_fail).type)).wallet
 
 def get_wallet_extensions(key):
 	return {
@@ -109,7 +106,7 @@ def format_fmt_codes():
 	return '\n'.join(ret) + '\n'
 
 def _get_me(modname):
-	return MMGenObject.__new__(getattr(importlib.import_module(f'mmgen.wallet.{modname}'), 'wallet'))
+	return MMGenObject.__new__(importlib.import_module(f'mmgen.wallet.{modname}').wallet)
 
 def Wallet(
 	cfg,

pyproject.toml

@@ -18,7 +18,6 @@ indent-style = "tab"
 ignore = [
 	"ASYNC221", # blocking method in async function (mmgen/rpc/backends/curl.py)
 	"B006", # Do not use mutable data structures for argument defaults
-	"B009", # Do not call `getattr` with a constant attribute value. It is not any safer than normal property access.
 	"B010", # Do not call `setattr` with a constant attribute value. It is not any safer than normal property access.
 	"B018", # Found useless expression. Either assign it to a variable or remove it.
 	"B023", # Function definition does not bind loop variable `confs`

test/include/unit_test.py

@@ -106,8 +106,8 @@ if cfg.list_subtests:
 		for test in all_tests:
 			mod = importlib.import_module(f'test.{test_subdir}.{test}')
 			if hasattr(mod, 'unit_tests'):
-				t = getattr(mod, 'unit_tests')
-				subtests = [k for k, v in t.__dict__.items() if type(v).__name__ == 'function' and k[0] != '_']
+				subtests = [k for k, v in mod.unit_tests.__dict__.items()
+					if type(v).__name__ == 'function' and k[0] != '_']
 				yield fs.format(test, ' '.join(f'{subtest}' for subtest in subtests))
 			else:
 				yield test
@@ -173,7 +173,7 @@ def run_test(test, subtest=None):
 			silence()
 
 		if hasattr(t, '_pre_subtest'):
-			getattr(t, '_pre_subtest')(test, subtest, UnitTestHelpers(subtest))
+			t._pre_subtest(test, subtest, UnitTestHelpers(subtest))
 
 		try:
 			func = getattr(t, subtest.replace('-', '_'))
@@ -198,7 +198,7 @@ def run_test(test, subtest=None):
 			raise
 
 		if hasattr(t, '_post_subtest'):
-			getattr(t, '_post_subtest')(test, subtest, UnitTestHelpers(subtest))
+			t._post_subtest(test, subtest, UnitTestHelpers(subtest))
 
 		if getattr(t, 'silence_output', False):
 			end_silence()
@@ -217,7 +217,7 @@ def run_test(test, subtest=None):
 	mod = importlib.import_module(f'test.{test_subdir}.{test}')
 
 	if hasattr(mod, 'unit_tests'): # new class-based API
-		t = getattr(mod, 'unit_tests')()
+		t = mod.unit_tests()
 		altcoin_deps = getattr(t, 'altcoin_deps', ())
 		win_skip = getattr(t, 'win_skip', ())
 		mac_skip = getattr(t, 'mac_skip', ())

test/objattrtest.py

@@ -167,7 +167,7 @@ def do_loop():
 	import importlib
 	modname = f'test.objattrtest_d.{proto.coin.lower()}_{proto.network}'
 	mod = importlib.import_module(modname)
-	test_data = getattr(mod, 'tests')
+	test_data = mod.tests
 	gmsg(f'Running immutable attribute tests for {proto.coin} {proto.network}')
 
 	utests = cfg._args

test/objtest.py

@@ -183,7 +183,7 @@ def do_loop():
 	import importlib
 	modname = f'test.objtest_d.{proto.coin.lower()}_{proto.network}'
 	mod = importlib.import_module(modname)
-	test_data = getattr(mod, 'tests')
+	test_data = mod.tests
 	gmsg(f'Running data object tests for {proto.coin} {proto.network}')
 
 	clr = None

test/tooltest2.py

@@ -338,8 +338,8 @@ if cfg.tool_api:
 if cfg.list_tests:
 	Msg('Available tests:')
 	for modname, cmdlist in main_tool.mods.items():
-		cls = getattr(importlib.import_module(f'mmgen.tool.{modname}'), 'tool_cmd')
-		Msg(f'  {modname:6} - {docstring_head(cls)}')
+		cls = importlib.import_module(f'mmgen.tool.{modname}').tool_cmd
+		Msg(f'  {modname:10} - {docstring_head(cls)}')
 	sys.exit(0)
 
 if cfg.list_tested_cmds: