From 35064671ef90ec6e35eafd9b15363058bf4f23f4 Mon Sep 17 00:00:00 2001 From: "Kartik K. Agaram" Date: Sat, 25 Jul 2015 00:02:20 -0700 Subject: 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. --- 003trace.test.cc | 34 +++++++++++++++++----------------- 1 file changed, 17 insertions(+), 17 deletions(-) (limited to '003trace.test.cc') 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); } -- cgit 1.4.1-2-gfad0