feat: tweak write_bytecodes output (#217)

Use hex as everywhere else, show return code as if normal return value
without the "x bytes of code"

src/tracing/writer.rs

@@ -242,7 +242,7 @@ impl<W: Write> TraceWriter<W> {
                 label = trace.decoded.label.as_deref().unwrap_or("<unknown>")
             )?;
             if self.config.write_bytecodes {
-                write!(self.writer, "({})", hex::encode(&trace.data))?;
+                write!(self.writer, "({})", trace.data)?;
             }
         } else {
             let (func_name, inputs) = match &trace.decoded.call_data {
@@ -399,11 +399,8 @@ impl<W: Write> TraceWriter<W> {
             return self.writer.write_all(decoded.as_bytes());
         }
 
-        if trace.kind.is_any_create() && trace.status.is_ok() {
+        if !self.config.write_bytecodes && (trace.kind.is_any_create() && trace.status.is_ok()) {
             write!(self.writer, "{} bytes of code", trace.output.len())?;
-            if self.config.write_bytecodes {
-                write!(self.writer, " ({})", hex::encode(&trace.output))?;
-            }
         } else if !trace.output.is_empty() {
             write!(self.writer, "{}", trace.output)?;
         }

tests/it/writer.rs

@@ -21,7 +21,7 @@ fn test_trace_printing() {
 
     let mut evm = TestEvm::new();
 
-    let mut tracer = TracingInspector::new(TracingInspectorConfig::all().disable_steps());
+    let mut tracer = TracingInspector::new(TracingInspectorConfig::all());
     let address = evm.deploy(CREATION_CODE.parse().unwrap(), &mut tracer).unwrap();
 
     let mut index = 0;
@@ -69,7 +69,7 @@ fn deploy_fail() {
     let base_path = &Path::new(OUT_DIR).join("deploy_fail");
 
     let mut evm = TestEvm::new();
-    let mut tracer = TracingInspector::new(TracingInspectorConfig::all().disable_steps());
+    let mut tracer = TracingInspector::new(TracingInspectorConfig::all());
     let _ = evm.try_deploy(bytes!("604260005260206000fd"), &mut tracer).unwrap();
 
     assert_traces(base_path, Some("raw"), None, true, &mut tracer);

tests/it/writer/deploy_fail/decoded.write_bytecodes.txt

@@ -1,2 +1,2 @@
-  [18] → new RevertingConstructor@0xBd770416a3345F91E4B34576cb804a576fa48EB1(604260005260206000fd)
+  [18] → new RevertingConstructor@0xBd770416a3345F91E4B34576cb804a576fa48EB1(0x604260005260206000fd)
     └─ ← [Revert] 42

tests/it/writer/deploy_fail/raw.write_bytecodes.txt

@@ -1,2 +1,2 @@
-  [18] → new <unknown>@0xBd770416a3345F91E4B34576cb804a576fa48EB1(604260005260206000fd)
+  [18] → new <unknown>@0xBd770416a3345F91E4B34576cb804a576fa48EB1(0x604260005260206000fd)
     └─ ← [Revert] 0x0000000000000000000000000000000000000000000000000000000000000042