TestCase: Treat expectations as a match even if the test case does not add a trailing newline

Author: cameel

Diff for: test/TestCase.cpp

@@ -84,7 +84,9 @@ void TestCase::printUpdatedExpectations(std::ostream& _stream, std::string const
 
 TestCase::TestResult TestCase::checkResult(std::ostream& _stream, const std::string& _linePrefix, bool const _formatted)
 {
-	if (m_expectation != m_obtainedResult)
+	// NOTE: Test cases usually use parseSimpleExpectations(), which ensures that m_expectations ends
+	// with a newline, so count m_obtainedResult as a match even if it does not.
+	if (m_expectation != m_obtainedResult && m_expectation != m_obtainedResult + '\n')
 	{
 		std::string nextIndentLevel = _linePrefix + "  ";
 		util::AnsiColorized(_stream, _formatted, {util::formatting::BOLD, util::formatting::CYAN})

Diff for: test/libevmasm/EVMAssemblyTest.cpp

@@ -174,8 +174,6 @@ TestCase::TestResult EVMAssemblyTest::run(std::ostream& _stream, std::string con
 			std::string separator = (content.empty() ? "" : (output == "Assembly" ? "\n" : " "));
 			m_obtainedResult += output + ":" + separator + content;
 		}
-	if (!m_obtainedResult.empty() && m_obtainedResult.back() != '\n')
-		m_obtainedResult += "\n";
 
 	return checkResult(_stream, _linePrefix, _formatted);
 }