fix double-print to console of tracing events outside spans (#30)

Author: davidhewitt

src/bridges/tracing.rs

@@ -112,8 +112,9 @@ where
                 // Need to emit this as a log, because it is not part of a span and will be lost
                 // otherwise.
                 emit_event_as_log_span(tracer, event, &tracing::Span::current());
-            }
-            if let Some(writer) = &tracer.console_writer {
+            } else if let Some(writer) = &tracer.console_writer {
+                // If it's not being emitted as a top-level log, need to print it to console
+                // (if it's emitted, the console processor will handle it)
                 writer.write_tracing_event(event);
             }
         });
@@ -286,7 +287,7 @@ mod tests {
                 },
                 parent_span_id: 0000000000000000,
                 span_kind: Internal,
-                name: "event src/bridges/tracing.rs:260",
+                name: "event src/bridges/tracing.rs:261",
                 start_time: SystemTime {
                     tv_sec: 0,
                     tv_nsec: 0,
@@ -1458,7 +1459,7 @@ mod tests {
                                         "code.lineno",
                                     ),
                                     value: I64(
-                                        268,
+                                        269,
                                     ),
                                 },
                             ],
@@ -1524,7 +1525,7 @@ mod tests {
                                         "code.lineno",
                                     ),
                                     value: I64(
-                                        269,
+                                        270,
                                     ),
                                 },
                             ],
@@ -1598,15 +1599,13 @@ mod tests {
 
         assert_snapshot!(output, @r#"
         1970-01-01T00:00:00.000000Z  INFO logfire::bridges::tracing::tests root event
-        1970-01-01T00:00:00.000001Z  INFO logfire::bridges::tracing::tests root event
-        1970-01-01T00:00:00.000002Z  INFO logfire::bridges::tracing::tests root event with value field_value=1
-        1970-01-01T00:00:00.000003Z  INFO logfire::bridges::tracing::tests root event with value field_value=1
-        1970-01-01T00:00:00.000004Z  INFO logfire::bridges::tracing::tests root span
-        1970-01-01T00:00:00.000005Z  INFO logfire::bridges::tracing::tests hello world span
-        1970-01-01T00:00:00.000006Z DEBUG logfire::bridges::tracing::tests debug span
-        1970-01-01T00:00:00.000007Z DEBUG logfire::bridges::tracing::tests debug span with explicit parent
-        1970-01-01T00:00:00.000008Z  INFO logfire::bridges::tracing::tests hello world log
-        1970-01-01T00:00:00.000009Z  INFO logfire::bridges::tracing::tests hello world log with value field_value=1
+        1970-01-01T00:00:00.000001Z  INFO logfire::bridges::tracing::tests root event with value field_value=1
+        1970-01-01T00:00:00.000002Z  INFO logfire::bridges::tracing::tests root span
+        1970-01-01T00:00:00.000003Z  INFO logfire::bridges::tracing::tests hello world span
+        1970-01-01T00:00:00.000004Z DEBUG logfire::bridges::tracing::tests debug span
+        1970-01-01T00:00:00.000005Z DEBUG logfire::bridges::tracing::tests debug span with explicit parent
+        1970-01-01T00:00:00.000006Z  INFO logfire::bridges::tracing::tests hello world log
+        1970-01-01T00:00:00.000007Z  INFO logfire::bridges::tracing::tests hello world log with value field_value=1
         "#);
     }
 }