From d65d79bf5e705851491ec29e2c0240bd6ff88273 Mon Sep 17 00:00:00 2001
From: Ben Hauser <ben@hauser.id>
Date: Sat, 30 May 2020 14:48:23 +0400
Subject: [PATCH] test: add test case for abi with topics, log without

---
 tests/test_incorrect_topics.py | 53 ++++++++++++++++++++++++----------
 1 file changed, 37 insertions(+), 16 deletions(-)

diff --git a/tests/test_incorrect_topics.py b/tests/test_incorrect_topics.py
index 4b04776..a28edf9 100644
--- a/tests/test_incorrect_topics.py
+++ b/tests/test_incorrect_topics.py
@@ -14,6 +14,17 @@
     "type": "event",
 }
 
+abi_two_indexes = {
+    "anonymous": False,
+    "inputs": [
+        {"indexed": True, "name": "sender", "type": "address"},
+        {"indexed": True, "name": "receiver", "type": "address"},
+        {"indexed": False, "name": "value", "type": "uint256"},
+    ],
+    "name": "Transfer",
+    "type": "event",
+}
+
 log_no_indexes = {
     "logIndex": 0,
     "transactionIndex": 0,
@@ -28,19 +39,24 @@
     "type": "mined",
 }
 
-
-abi_indexes = {
-    "anonymous": False,
-    "inputs": [
-        {"indexed": True, "name": "sender", "type": "address"},
-        {"indexed": True, "name": "receiver", "type": "address"},
-        {"indexed": False, "name": "value", "type": "uint256"},
+log_one_index = {
+    "logIndex": 0,
+    "transactionIndex": 0,
+    "transactionHash": HexBytes(
+        "0xe6eba721271cedb7a492c513d5dfedab73cc54f1a5a337fb027d23f523cada49"
+    ),
+    "blockHash": HexBytes("0x764e1736dc9cba192b2ebf2911abcad9168d4db35850a678600637f5a6de27ed"),
+    "blockNumber": 4,
+    "address": "0xE7eD6747FaC5360f88a2EFC03E00d25789F69291",
+    "data": "0x00000000000000000000000066ab6d9362d4f35596279692f0251db6351658710000000000000000000000000000000000000000000000000000000000002710",
+    "topics": [
+        HexBytes("0xddf252ad1be2c89b69c2b068fc378daa952ba7f163c4a11628f55a4df523b3ef"),
+        HexBytes("0x00000000000000000000000033a4622b82d4c04a53e170c638b944ce27cffce3"),
     ],
-    "name": "Transfer",
-    "type": "event",
+    "type": "mined",
 }
 
-log_indexes = {
+log_two_indexes = {
     "logIndex": 0,
     "transactionIndex": 0,
     "transactionHash": HexBytes(
@@ -59,13 +75,18 @@
 }
 
 
-def test_too_many_topics():
-    topic_map = get_topic_map([abi_indexes])
-    with pytest.raises(EventError, match="more topics than expected"):
-        decode_log(log_no_indexes, topic_map)
+def test_no_topics_in_log():
+    topic_map = get_topic_map([abi_two_indexes])
+    decode_log(log_no_indexes, topic_map)
 
 
-def test_insufficient_topics():
+def test_too_many_log_topics():
     topic_map = get_topic_map([abi_no_indexes])
     with pytest.raises(EventError, match="does not contain enough topics"):
-        decode_log(log_indexes, topic_map)
+        decode_log(log_two_indexes, topic_map)
+
+
+def test_insufficient_log_topics():
+    topic_map = get_topic_map([abi_two_indexes])
+    with pytest.raises(EventError, match="more topics than expected"):
+        decode_log(log_one_index, topic_map)