blackified (new version)

Signed-off-by: Jürgen Eckel <juergen@riddleandcode.com>

.github/workflows/CI.yml

@@ -20,6 +20,15 @@ permissions:
   contents: write
 
 jobs:
+  lint:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v3
+      - uses: psf/black@stable
+        with:
+          options: "--check -l 119"
+          src: "."
+
   audit:
     runs-on: ubuntu-latest
     
@@ -44,14 +53,8 @@ jobs:
       - name: Audit dependencies
         run: pip-audit --ignore-vuln PYSEC-2022-42969 --ignore-vuln PYSEC-2022-203 --ignore-vuln GHSA-r9hx-vwmv-q579
 
-      - name: Install black
-        run: pip install black==22.12.0
-
-      - name: Lint - black
-        run: black --check -l 119 .
-
   test:
-    needs: audit
+    needs: lint
     runs-on: ubuntu-latest
     env:
       PLANETMINT_DATABASE_BACKEND: tarantool_db

integration/python/src/test_naughty_strings.py

@@ -115,7 +115,6 @@ def send_naughty_tx(assets, metadata):
 
 @pytest.mark.parametrize("naughty_string", naughty_strings, ids=naughty_strings)
 def test_naughty_keys(naughty_string):
-
     assets = [{"data": {naughty_string: "nice_value"}}]
     metadata = {naughty_string: "nice_value"}
 
@@ -124,7 +123,6 @@ def test_naughty_keys(naughty_string):
 
 @pytest.mark.parametrize("naughty_string", naughty_strings, ids=naughty_strings)
 def test_naughty_values(naughty_string):
-
     assets = [{"data": {"nice_key": naughty_string}}]
     metadata = {"nice_key": naughty_string}
 

integration/python/src/test_zenroom.py

@@ -17,7 +17,6 @@ def test_zenroom_signing(
     zenroom_script_input,
     condition_script_zencode,
 ):
-
     biolabs = generate_keypair()
     version = "2.0"
 

planetmint/backend/localmongodb/query.py

@@ -102,7 +102,6 @@ def store_block(conn, block):
 
 @register_query(LocalMongoDBConnection)
 def get_txids_filtered(conn, asset_ids, operation=None, last_tx=None):
-
     match = {
         Transaction.CREATE: {"operation": "CREATE", "id": {"$in": asset_ids}},
         Transaction.TRANSFER: {"operation": "TRANSFER", "asset.id": {"$in": asset_ids}},

planetmint/web/routes.py

@@ -18,9 +18,9 @@ from planetmint.web.views import (
 
 def add_routes(app):
     """Add the routes to an app"""
-    for (prefix, routes) in API_SECTIONS:
+    for prefix, routes in API_SECTIONS:
         api = Api(app, prefix=prefix)
-        for ((pattern, resource, *args), kwargs) in routes:
+        for (pattern, resource, *args), kwargs in routes:
             kwargs.setdefault("strict_slashes", False)
             api.add_resource(resource, pattern, *args, **kwargs)
 

setup.py

@@ -94,7 +94,7 @@ dev_require = ["ipdb", "ipython", "watchdog", "logging_tree", "pre-commit", "twi
 tests_require = [
     "coverage",
     "pep8",
-    "black",
+    "black>=23.1.0",
     "hypothesis>=5.3.0",
     "pytest>=3.0.0",
     "pytest-cov==2.8.1",

tests/assets/test_divisible_assets.py

@@ -34,7 +34,6 @@ def test_single_in_single_own_single_out_single_own_create(alice, user_pk, b):
 # Multiple outputs
 # Single owners_after per output
 def test_single_in_single_own_multiple_out_single_own_create(alice, user_pk, b):
-
     tx = Create.generate(
         [alice.public_key],
         [([user_pk], 50), ([user_pk], 50)],
@@ -55,7 +54,6 @@ def test_single_in_single_own_multiple_out_single_own_create(alice, user_pk, b):
 # Single output
 # Multiple owners_after
 def test_single_in_single_own_single_out_multiple_own_create(alice, user_pk, b):
-
     tx = Create.generate(
         [alice.public_key],
         [([user_pk, user_pk], 100)],
@@ -81,7 +79,6 @@ def test_single_in_single_own_single_out_multiple_own_create(alice, user_pk, b):
 # Mix: one output with a single owners_after, one output with multiple
 #      owners_after
 def test_single_in_single_own_multiple_out_mix_own_create(alice, user_pk, b):
-
     tx = Create.generate(
         [alice.public_key],
         [([user_pk], 50), ([user_pk, user_pk], 50)],
@@ -130,7 +127,6 @@ def test_single_in_multiple_own_single_out_single_own_create(alice, b, user_pk,
 # Single output
 # Single owners_after
 def test_single_in_single_own_single_out_single_own_transfer(alice, b, user_pk, user_sk):
-
     # CREATE divisible asset
     tx_create = Create.generate(
         [alice.public_key], [([user_pk], 100)], assets=[{"data": "QmaozNR7DZHQK1ZcU9p7QdrshMvXqWK6gpu5rmrkPdT3L4"}]
@@ -155,7 +151,6 @@ def test_single_in_single_own_single_out_single_own_transfer(alice, b, user_pk,
 # Multiple output
 # Single owners_after
 def test_single_in_single_own_multiple_out_single_own_transfer(alice, b, user_pk, user_sk):
-
     # CREATE divisible asset
     tx_create = Create.generate(
         [alice.public_key], [([user_pk], 100)], assets=[{"data": "QmaozNR7DZHQK1ZcU9p7QdrshMvXqWK6gpu5rmrkPdT3L4"}]
@@ -183,7 +178,6 @@ def test_single_in_single_own_multiple_out_single_own_transfer(alice, b, user_pk
 # Single output
 # Multiple owners_after
 def test_single_in_single_own_single_out_multiple_own_transfer(alice, b, user_pk, user_sk):
-
     # CREATE divisible asset
     tx_create = Create.generate(
         [alice.public_key], [([user_pk], 100)], assets=[{"data": "QmaozNR7DZHQK1ZcU9p7QdrshMvXqWK6gpu5rmrkPdT3L4"}]
@@ -219,7 +213,6 @@ def test_single_in_single_own_single_out_multiple_own_transfer(alice, b, user_pk
 # Mix: one output with a single owners_after, one output with multiple
 #      owners_after
 def test_single_in_single_own_multiple_out_mix_own_transfer(alice, b, user_pk, user_sk):
-
     # CREATE divisible asset
     tx_create = Create.generate(
         [alice.public_key], [([user_pk], 100)], assets=[{"data": "QmaozNR7DZHQK1ZcU9p7QdrshMvXqWK6gpu5rmrkPdT3L4"}]
@@ -448,7 +441,6 @@ def test_muiltiple_in_mix_own_multiple_out_mix_own_transfer(alice, b, user_pk, u
 # Single output
 # Single owners_after
 def test_multiple_in_different_transactions(alice, b, user_pk, user_sk):
-
     # CREATE divisible asset
     # `b` creates a divisible asset and assigns 50 shares to `b` and
     # 50 shares to `user_pk`
@@ -549,7 +541,6 @@ def test_threshold_same_public_key(alice, b, user_pk, user_sk):
 
 
 def test_sum_amount(alice, b, user_pk, user_sk):
-
     # CREATE divisible asset with 3 outputs with amount 1
     tx_create = Create.generate(
         [alice.public_key],
@@ -575,7 +566,6 @@ def test_sum_amount(alice, b, user_pk, user_sk):
 
 
 def test_divide(alice, b, user_pk, user_sk):
-
     # CREATE divisible asset with 1 output with amount 3
     tx_create = Create.generate(
         [alice.public_key], [([user_pk], 3)], assets=[{"data": "QmaozNR7DZHQK1ZcU9p7QdrshMvXqWK6gpu5rmrkPdT3L4"}]

tests/assets/test_zenroom_signing.py

@@ -52,7 +52,6 @@ metadata = {"units": 300, "type": "KG"}
 
 
 def test_zenroom_signing():
-
     biolabs = generate_key_pair()
     version = "3.0"
 

tests/db/test_planetmint_api.py

@@ -106,7 +106,6 @@ class TestBigchainApi(object):
             b.validate_transaction(tx)
 
     def test_write_transaction(self, b, user_sk, user_pk, alice, create_tx):
-
         asset1 = {"data": "QmaozNR7DZHQK1ZcU9p7QdrshMvXqWK6gpu5rmrkPdT3L4"}
 
         tx = Create.generate([alice.public_key], [([alice.public_key], 1)], assets=[asset1]).sign([alice.private_key])

tests/test_config_utils.py

@@ -315,7 +315,6 @@ def test_write_config():
     ),
 )
 def test_database_envs(env_name, env_value, config_key, monkeypatch):
-
     monkeypatch.setattr("os.environ", {env_name: env_value})
     planetmint.config_utils.autoconfigure()
 

tests/upsert_validator/test_upsert_validator_vote.py

@@ -156,7 +156,6 @@ def test_valid_election_votes_received(b_mock, valid_upsert_validator_election,
 
 @pytest.mark.bdb
 def test_valid_election_conclude(b_mock, valid_upsert_validator_election, ed25519_node_keys):
-
     # Node 0: cast vote
     tx_vote0 = gen_vote(valid_upsert_validator_election, 0, ed25519_node_keys)
 
@@ -215,7 +214,6 @@ def test_valid_election_conclude(b_mock, valid_upsert_validator_election, ed2551
 
 @pytest.mark.abci
 def test_upsert_validator(b, node_key, node_keys, ed25519_node_keys):
-
     if b.get_latest_block()["height"] == 0:
         generate_block(b)
 
@@ -344,6 +342,6 @@ def test_get_validator_update(b, node_keys, node_key, ed25519_node_keys):
 
 def reset_validator_set(b, node_keys, height):
     validators = []
-    for (node_pub, _) in node_keys.items():
+    for node_pub, _ in node_keys.items():
         validators.append({"public_key": {"type": "ed25519-base64", "value": node_pub}, "voting_power": 10})
     b.store_validator_set(height, validators)

tests/web/test_metadata.py

@@ -37,7 +37,6 @@ def test_get_metadata_tendermint(client, b, alice):
 
 @pytest.mark.bdb
 def test_get_metadata_limit_tendermint(client, b, alice):
-
     # create two assets
     assets1 = [{"data": multihash(marshal({"msg": "abc 1"}))}]
     meta = multihash(marshal({"key": "meta 1"}))

tests/web/test_transactions.py

@@ -340,7 +340,6 @@ def test_post_invalid_transaction(
 
 @pytest.mark.abci
 def test_post_transfer_transaction_endpoint(client, user_pk, user_sk, posted_create_tx):
-
     transfer_tx = Transfer.generate(posted_create_tx.to_inputs(), [([user_pk], 1)], asset_ids=[posted_create_tx.id])
     transfer_tx = transfer_tx.sign([user_sk])
 

tests/web/test_websocket_server.py

@@ -138,7 +138,6 @@ async def test_bridge_sync_async_queue(event_loop):
 
 @pytest.mark.asyncio
 async def test_websocket_block_event(aiohttp_client, event_loop):
-
     user_priv, user_pub = crypto.generate_key_pair()
     tx = Create.generate([user_pub], [([user_pub], 1)])
     tx = tx.sign([user_priv])
@@ -169,7 +168,6 @@ async def test_websocket_block_event(aiohttp_client, event_loop):
 
 @pytest.mark.asyncio
 async def test_websocket_transaction_event(aiohttp_client, event_loop):
-
     user_priv, user_pub = crypto.generate_key_pair()
     tx = Create.generate([user_pub], [([user_pub], 1)])
     tx = tx.sign([user_priv])