about summary refs log tree commit diff stats
path: root/003trace.test.cc
diff options
context:
space:
mode:
authorKartik K. Agaram <vc@akkartik.com>2015-07-25 00:02:20 -0700
committerKartik K. Agaram <vc@akkartik.com>2015-07-25 00:02:20 -0700
commit35064671ef90ec6e35eafd9b15363058bf4f23f4 (patch)
tree3a0b4b02cbdd1b304deb52d1673083091a1ed12d /003trace.test.cc
parenta17fd65ca0124f544bd4de844f1bfe2d91b46ae6 (diff)
downloadmu-35064671ef90ec6e35eafd9b15363058bf4f23f4.tar.gz
1844 - explicitly end each trace line
More verbose, but it saves trouble when debugging; there's never
something you thought should be traced but just never came out the other
end.

Also got rid of fatal errors entirely. Everything's a warning now, and
code after a warning isn't guaranteed to run.
Diffstat (limited to '003trace.test.cc')
-rw-r--r--003trace.test.cc34
1 files changed, 17 insertions, 17 deletions
diff --git a/003trace.test.cc b/003trace.test.cc
index 146f9dc2..a16457a1 100644
--- a/003trace.test.cc
+++ b/003trace.test.cc
@@ -1,58 +1,58 @@
 void test_trace_check_compares() {
-  trace("test layer") << "foo";
+  trace("test layer") << "foo" << end();
   CHECK_TRACE_CONTENTS("test layer: foo");
 }
 
 void test_trace_check_ignores_other_layers() {
-  trace("test layer 1") << "foo";
-  trace("test layer 2") << "bar";
+  trace("test layer 1") << "foo" << end();
+  trace("test layer 2") << "bar" << end();
   CHECK_TRACE_CONTENTS("test layer 1: foo");
   CHECK_TRACE_DOESNT_CONTAIN("test layer 2: foo");
 }
 
 void test_trace_check_ignores_other_lines() {
-  trace("test layer 1") << "foo";
-  trace("test layer 1") << "bar";
+  trace("test layer 1") << "foo" << end();
+  trace("test layer 1") << "bar" << end();
   CHECK_TRACE_CONTENTS("test layer 1: foo");
 }
 
 void test_trace_check_ignores_other_lines2() {
-  trace("test layer 1") << "foo";
-  trace("test layer 1") << "bar";
+  trace("test layer 1") << "foo" << end();
+  trace("test layer 1") << "bar" << end();
   CHECK_TRACE_CONTENTS("test layer 1: bar");
 }
 
 void test_trace_ignores_trailing_whitespace() {
-  trace("test layer 1") << "foo\n";
+  trace("test layer 1") << "foo\n" << end();
   CHECK_TRACE_CONTENTS("test layer 1: foo");
 }
 
 void test_trace_ignores_trailing_whitespace2() {
-  trace("test layer 1") << "foo ";
+  trace("test layer 1") << "foo " << end();
   CHECK_TRACE_CONTENTS("test layer 1: foo");
 }
 
 void test_trace_orders_across_layers() {
-  trace("test layer 1") << "foo";
-  trace("test layer 2") << "bar";
-  trace("test layer 1") << "qux";
+  trace("test layer 1") << "foo" << end();
+  trace("test layer 2") << "bar" << end();
+  trace("test layer 1") << "qux" << end();
   CHECK_TRACE_CONTENTS("test layer 1: footest layer 2: bartest layer 1: qux");
 }
 
 void test_trace_supports_count() {
-  trace("test layer 1") << "foo";
-  trace("test layer 1") << "foo";
+  trace("test layer 1") << "foo" << end();
+  trace("test layer 1") << "foo" << end();
   CHECK_EQ(trace_count("test layer 1", "foo"), 2);
 }
 
 void test_trace_supports_count2() {
-  trace("test layer 1") << "foo";
-  trace("test layer 1") << "bar";
+  trace("test layer 1") << "foo" << end();
+  trace("test layer 1") << "bar" << end();
   CHECK_EQ(trace_count("test layer 1"), 2);
 }
 
 void test_trace_count_ignores_trailing_whitespace() {
-  trace("test layer 1") << "foo\n";
+  trace("test layer 1") << "foo\n" << end();
   CHECK(trace_count("test layer 1", "foo") == 1);
 }