Merge #12996: tests: Remove redundant bytes(…) calls

9db48c5 tests: Remove redundant bytes² (practicalswift)

Pull request description:

  This is a follow-up to #12993. As @jnewbery noted `bytes()` is idempotent.

Tree-SHA512: 0eb25e0c2c46f9abaac30f964c5eb422bece1414c840a717d86794424294cb19d995a6db7c8df2a2f4ec84776b05274a637f2c111738f397051f510e57184752

Author: MarcoFalke

test/functional/test_framework/script.py

@@ -379,7 +379,7 @@ def encode(obj):
             r.append(0x80 if neg else 0)
         elif neg:
             r[-1] |= 0x80
-        return bytes(bytes([len(r)]) + r)
+        return bytes([len(r)]) + r
 
 
 class CScript(bytes):
@@ -404,9 +404,9 @@ def __coerce_instance(cls, other):
                 other = CScriptNum.encode(other)
         elif isinstance(other, int):
             if 0 <= other <= 16:
-                other = bytes(bytes([CScriptOp.encode_op_n(other)]))
+                other = bytes([CScriptOp.encode_op_n(other)])
             elif other == -1:
-                other = bytes(bytes([OP_1NEGATE]))
+                other = bytes([OP_1NEGATE])
             else:
                 other = CScriptOp.encode_op_pushdata(bn2vch(other))
         elif isinstance(other, (bytes, bytearray)):