fix emit statments being printed on the same line

Update test/libsolidity/semanticTests/emit_three_identical_events.sol Co-authored-by: Kamil Śliwak <[email protected]> Update test/libsolidity/semanticTests/emit_two_identical_events.sol Co-authored-by: Nikola Matić <[email protected]>
sgilflores3 · Oct 25, 2022 · 54965fb · 54965fb
1 parent 84cdcec
commit 54965fb
Show file tree

Hide file tree

Showing 3 changed files with 30 additions and 6 deletions.
diff --git a/test/libsolidity/semanticTests/emit_three_identical_events.sol b/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()
diff --git a/test/libsolidity/semanticTests/emit_two_identical_events.sol b/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()
diff --git a/test/libsolidity/util/TestFunctionCall.cpp b/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);