tests: include the test for the tests

Some implementations send an error message and at the same time
will close the connection. Locally this will cause a reading from
a closed socket.

However, ldk now has a different life cycle for the connection,
and we should receive the error message before.

```
 lnprototest.errors.EventError: `Peer did not disconnect` on event [{"event": "ExpectDisconnect", "file": "test_bolt2-01-open_channel.py", "pos": "345"},]
=========================== short test summary info ============================
FAILED ../lnprototest/tests/test_bolt2-01-open_channel.py::test_open_channel_opener_side_wrong_announcement_signatures
============= 1 failed, 29 passed, 17 skipped in 96.15s (0:01:36) ==============
```

Link: #134
Reported-by: @Psycho-Pirate
Signed-off-by: Vincenzo Palazzo <[email protected]>

Author: vincenzopalazzo

tests/test_bolt1-02-unknown-messages.py

@@ -5,7 +5,7 @@
 
 from typing import Any
 
-from lnprototest import TryAll, Connect, ExpectMsg, Msg, RawMsg, Runner
+from lnprototest import Connect, ExpectMsg, Msg, RawMsg, Runner
 from lnprototest.event import ExpectDisconnect
 from lnprototest.utils import run_runner
 

tests/test_bolt2-01-open_channel.py

@@ -1,13 +1,12 @@
 # Variations on open_channel, accepter + opener perspectives
 from lnprototest import (
     TryAll,
-    OneOf,
+    Sequence,
     ExpectDisconnect,
     Block,
     FundChannel,
     ExpectMsg,
     ExpectTx,
-    ExpectError,
     Msg,
     RawMsg,
     AcceptFunding,
@@ -26,8 +25,6 @@
     remote_funding_privkey,
     bitfield,
     Block,
-    ExpectError,
-    Disconnect,
 )
 from lnprototest.stash import (
     sent,
@@ -291,6 +288,8 @@ def test_open_channel_opener_side(runner: Runner) -> None:
 def test_open_channel_opener_side_wrong_announcement_signatures(runner: Runner) -> None:
     """Testing the case where the channel is announces in the correct way but one node
     send the wrong signature inside the `announcement_signatures` message."""
+    from lnprototest.clightning import Runner as CLightningRunner
+
     connections_events = connect_to_node_helper(
         runner=runner,
         tx_spendable=tx_spendable,
@@ -302,6 +301,8 @@ def test_open_channel_opener_side_wrong_announcement_signatures(runner: Runner)
 
     dummy_sign = "138c93afb2013c39f959e70a163c3d6d8128cf72f8ae143f87b9d1fd6bb0ad30321116b9c58d69fca9fb33c214f681b664e53d5640abc2fdb972dc62a5571053"
     short_channel_id = opts["short_channel_id"]
+
+    is_cln = isinstance(runner, CLightningRunner)
     test_events = [
         # BOLT 2:
         #
@@ -342,6 +343,7 @@ def test_open_channel_opener_side_wrong_announcement_signatures(runner: Runner)
         #
         # So we should change the OneOf to all exception and stop when
         # the first one succided
-        ExpectDisconnect(),
+        Sequence(ExpectDisconnect(), enable=is_cln),
+        Sequence(ExpectMsg("error"), enable=(not is_cln)),
     ]
     run_runner(runner, merge_events_sequences(pre_events, test_events))