test: fix duplicate operations in golden files and add operation

invariants

Fixed 4 failing /construction/parse golden tests that had duplicate
operations at index 3. Tests expected 5 operations but API correctly
returned 4.

Added operation invariant tests to prevent ordering bugs:
- assert_operations_ordered() - ensures operations sorted by index
- assert_operations_sequential() - ensures indices [0, 1, 2, ...]

Covers /search/transactions, /block, and /block/transaction endpoints.

This would have caught the bug where /construction/parse returns
operations in processing order (inputs → outputs → certs → withdrawals)
instead of sorted by operation_identifier.index.

Author: linconvidal

tests/data-endpoints/conftest.py

@@ -160,3 +160,16 @@ def get_error_message(error_response):
     details_message = error_response.get("details", {}).get("message", "")
     return (message + " " + details_message).strip()
 
+
+def assert_operations_ordered(operations):
+    """Operations must be sorted by operation_identifier.index."""
+    indices = [op["operation_identifier"]["index"] for op in operations]
+    assert indices == sorted(indices), f"Operations not ordered: {indices}"
+
+
+def assert_operations_sequential(operations):
+    """Operation indices must be [0, 1, 2, ..., n-1]."""
+    indices = [op["operation_identifier"]["index"] for op in operations]
+    expected = list(range(len(operations)))
+    assert indices == expected, f"Expected {expected}, got {indices}"
+

tests/data-endpoints/test_block_endpoints.py

@@ -6,7 +6,7 @@
 
 import pytest
 import allure
-from conftest import get_error_message
+from conftest import get_error_message, assert_operations_ordered, assert_operations_sequential
 
 pytestmark = pytest.mark.pr
 
@@ -314,3 +314,75 @@ def test_invalid_transaction_hash_returns_error(self, client, network):
             transaction_identifier={"hash": "invalid_hash"},
         )
         assert response.status_code == 500
+
+
+@allure.feature("Block")
+@allure.story("Operation Invariants")
+class TestBlockOperationInvariants:
+    """Operations must be ordered and sequential in all block transactions."""
+
+    def test_block_operations_ordered_by_index(self, client, network):
+        """Operations in /block response must be sorted by index."""
+        # Find block with transactions
+        search = client.search_transactions(network=network)
+        block_id = search.json()["transactions"][0]["block_identifier"]
+
+        response = client.block(network=network, block_identifier={"index": block_id["index"]})
+        assert response.status_code == 200
+
+        for tx in response.json()["block"]["transactions"]:
+            assert_operations_ordered(tx["operations"])
+
+    def test_block_operations_sequential_indices(self, client, network):
+        """Operations in /block must have sequential indices [0, 1, 2, ...]."""
+        # Find block with transactions
+        search = client.search_transactions(network=network)
+        block_id = search.json()["transactions"][0]["block_identifier"]
+
+        response = client.block(network=network, block_identifier={"index": block_id["index"]})
+        assert response.status_code == 200
+
+        for tx in response.json()["block"]["transactions"]:
+            assert_operations_sequential(tx["operations"])
+
+
+@allure.feature("Block Transaction")
+@allure.story("Operation Invariants")
+class TestBlockTransactionOperationInvariants:
+    """Operations must be ordered and sequential in /block/transaction."""
+
+    def test_block_transaction_operations_ordered_by_index(self, client, network):
+        """Operations in /block/transaction must be sorted by index."""
+        # Find transaction
+        search = client.search_transactions(network=network)
+        block_tx = search.json()["transactions"][0]
+
+        response = client.block_transaction(
+            network=network,
+            block_identifier={
+                "index": block_tx["block_identifier"]["index"],
+                "hash": block_tx["block_identifier"]["hash"]
+            },
+            transaction_identifier={"hash": block_tx["transaction"]["transaction_identifier"]["hash"]}
+        )
+        assert response.status_code == 200
+
+        assert_operations_ordered(response.json()["transaction"]["operations"])
+
+    def test_block_transaction_operations_sequential_indices(self, client, network):
+        """Operations in /block/transaction must have sequential indices."""
+        # Find transaction
+        search = client.search_transactions(network=network)
+        block_tx = search.json()["transactions"][0]
+
+        response = client.block_transaction(
+            network=network,
+            block_identifier={
+                "index": block_tx["block_identifier"]["index"],
+                "hash": block_tx["block_identifier"]["hash"]
+            },
+            transaction_identifier={"hash": block_tx["transaction"]["transaction_identifier"]["hash"]}
+        )
+        assert response.status_code == 200
+
+        assert_operations_sequential(response.json()["transaction"]["operations"])

tests/data-endpoints/test_search_transactions.py

@@ -6,7 +6,7 @@
 import os
 import pytest
 import allure
-from conftest import get_error_message
+from conftest import get_error_message, assert_operations_ordered, assert_operations_sequential
 
 
 # Network configuration from environment - works on ANY network
@@ -793,3 +793,29 @@ def test_native_asset_filtering_with_policy_id(self, client, network, network_da
                 f"Transaction must contain asset with policyId {asset['policy_id']} "
                 f"and symbol {asset['symbol_hex']}. Found assets: {assets_in_tx}"
             )
+
+
+class TestOperationInvariants:
+    """Operations must be ordered and sequential in all transactions."""
+
+    @allure.feature("Search Transactions")
+    @allure.story("Operation Invariants")
+    def test_operations_ordered_by_index(self, client, network):
+        """Operations array must be sorted by operation_identifier.index."""
+        response = client.search_transactions(network=network, limit=10)
+        assert response.status_code == 200
+
+        for block_tx in response.json()["transactions"]:
+            operations = block_tx["transaction"]["operations"]
+            assert_operations_ordered(operations)
+
+    @allure.feature("Search Transactions")
+    @allure.story("Operation Invariants")
+    def test_operations_sequential_indices(self, client, network):
+        """Operation indices must be [0, 1, 2, ..., n-1] with no gaps."""
+        response = client.search_transactions(network=network, limit=10)
+        assert response.status_code == 200
+
+        for block_tx in response.json()["transactions"]:
+            operations = block_tx["transaction"]["operations"]
+            assert_operations_sequential(operations)

tests/integration/golden_examples/rosetta_java/construction/parse/complex_transactions/combined_stake_and_drep_delegation.json

@@ -91,25 +91,6 @@
             "type": "key_hash"
           }
         }
-      },
-      {
-        "operation_identifier": {
-          "index": 3
-        },
-        "type": "dRepVoteDelegation",
-        "account": {
-          "address": "stake_test1uql3k2h9kskfma8y53vth7wmptmlw8zxx67cx7c3pwj8sqs6zl0wr"
-        },
-        "metadata": {
-          "staking_credential": {
-            "hex_bytes": "fdbee86a702c49d23f45ab80e697b93ec48b744d5f413ef59c338c3f7b2d2de8",
-            "curve_type": "edwards25519"
-          },
-          "drep": {
-            "id": "f72050aa252ccc6bf75747184910c0b9386298167656f935d6b6c26a",
-            "type": "key_hash"
-          }
-        }
       }
     ],
     "account_identifier_signers": []

tests/integration/golden_examples/rosetta_java/construction/parse/complex_transactions/drep_delegation_with_native_assets.json

@@ -144,25 +144,6 @@
             "type": "script_hash"
           }
         }
-      },
-      {
-        "operation_identifier": {
-          "index": 3
-        },
-        "type": "dRepVoteDelegation",
-        "account": {
-          "address": "stake_test1uql3k2h9kskfma8y53vth7wmptmlw8zxx67cx7c3pwj8sqs6zl0wr"
-        },
-        "metadata": {
-          "staking_credential": {
-            "hex_bytes": "fdbee86a702c49d23f45ab80e697b93ec48b744d5f413ef59c338c3f7b2d2de8",
-            "curve_type": "edwards25519"
-          },
-          "drep": {
-            "id": "2d4cb680b5f400d3521d272b4295d61150e0eff3950ef4285406a953",
-            "type": "script_hash"
-          }
-        }
       }
     ],
     "account_identifier_signers": []

tests/integration/golden_examples/rosetta_java/construction/parse/complex_transactions/drep_delegation_with_stake_key_registration.json

@@ -89,24 +89,6 @@
             "type": "abstain"
           }
         }
-      },
-      {
-        "operation_identifier": {
-          "index": 3
-        },
-        "type": "dRepVoteDelegation",
-        "account": {
-          "address": "stake_test1uql3k2h9kskfma8y53vth7wmptmlw8zxx67cx7c3pwj8sqs6zl0wr"
-        },
-        "metadata": {
-          "staking_credential": {
-            "hex_bytes": "fdbee86a702c49d23f45ab80e697b93ec48b744d5f413ef59c338c3f7b2d2de8",
-            "curve_type": "edwards25519"
-          },
-          "drep": {
-            "type": "abstain"
-          }
-        }
       }
     ],
     "account_identifier_signers": []

tests/integration/golden_examples/rosetta_java/construction/parse/complex_transactions/withdrawal_with_drep_delegation.json

@@ -55,25 +55,6 @@
           }
         }
       },
-      {
-        "operation_identifier": {
-          "index": 3
-        },
-        "type": "dRepVoteDelegation",
-        "status": "",
-        "account": {
-          "address": "stake_test1uql3k2h9kskfma8y53vth7wmptmlw8zxx67cx7c3pwj8sqs6zl0wr"
-        },
-        "metadata": {
-          "staking_credential": {
-            "hex_bytes": "fdbee86a702c49d23f45ab80e697b93ec48b744d5f413ef59c338c3f7b2d2de8",
-            "curve_type": "edwards25519"
-          },
-          "drep": {
-            "type": "no_confidence"
-          }
-        }
-      },
       {
         "operation_identifier": {
           "index": 2
@@ -102,6 +83,7 @@
           "index": 3
         },
         "type": "dRepVoteDelegation",
+        "status": "",
         "account": {
           "address": "stake_test1uql3k2h9kskfma8y53vth7wmptmlw8zxx67cx7c3pwj8sqs6zl0wr"
         },