Merge pull request #13649 from Andy53/output-emit-logs-correctly

fix emit statments being printed on the same line

Author: cameel

test/libsolidity/semanticTests/emit_three_identical_events.sol

@@ -0,0 +1,14 @@
+contract C {
+    event Terminated();
+
+    function terminate() external {
+        emit Terminated();
+        emit Terminated();
+        emit Terminated();
+    }
+}
+// ----
+// terminate() ->
+// ~ emit Terminated()
+// ~ emit Terminated()
+// ~ emit Terminated()

test/libsolidity/semanticTests/emit_two_identical_events.sol

@@ -0,0 +1,12 @@
+contract C {
+    event Terminated();
+
+    function terminate() external {
+        emit Terminated();
+        emit Terminated();
+    }
+}
+// ----
+// terminate() ->
+// ~ emit Terminated()
+// ~ emit Terminated()

test/libsolidity/util/TestFunctionCall.cpp

@@ -208,12 +208,10 @@ string TestFunctionCall::format(
 		if (!sideEffects.empty())
 		{
 			stream << std::endl;
-			for (string const& effect: sideEffects)
-			{
-				stream << _linePrefix << "// ~ " << effect;
-				if (effect != *sideEffects.rbegin())
-					stream << std::endl;
-			}
+			size_t i = 0;
+			for (; i < sideEffects.size() - 1; ++i)
+				stream << _linePrefix << "// ~ " << sideEffects[i] << std::endl;
+			stream << _linePrefix << "// ~ " << sideEffects[i];
 		}
 
 		stream << formatGasExpectations(_linePrefix, _renderMode == RenderMode::ExpectedValuesActualGas, _interactivePrint);