Preserve failAfter directive in isoltest ASTJSON reformatting

tavian-dev · clonker · commit ea0a8c137ff8 · 2026-05-07T13:39:25.000+02:00
When isoltest updates test expectations for ASTJSON tests, it calls printSource() to rewrite the .sol input file. The function was not outputting the failAfter directive that was parsed and stored in m_expectedFailAfter, causing the directive to be silently dropped. Fixes #14093
diff --git a/test/libsolidity/ASTJSONTest.cpp b/test/libsolidity/ASTJSONTest.cpp
@@ -295,6 +295,9 @@ void ASTJSONTest::printSource(std::ostream& _stream, std::string const& _linePre
 		printPrefixed(_stream, source.second, _linePrefix);
 		_stream << std::endl;
 	}
+
+	if (m_expectedFailAfter.has_value())
+		printPrefixed(_stream, "// failAfter: " + compilerStateToString(m_expectedFailAfter.value()) + "\n", _linePrefix);
 }
 
 void ASTJSONTest::printUpdatedExpectations(std::ostream&, std::string const&) const

Original file line number	Diff line number	Diff line change
`@@ -295,6 +295,9 @@ void ASTJSONTest::printSource(std::ostream& _stream, std::string const& _linePre`
`295`	`295`	`printPrefixed(_stream, source.second, _linePrefix);`
`296`	`296`	`_stream << std::endl;`
`297`	`297`	`}`
	`298`	`+`
	`299`	`+ if (m_expectedFailAfter.has_value())`
	`300`	`+ printPrefixed(_stream, "// failAfter: " + compilerStateToString(m_expectedFailAfter.value()) + "\n", _linePrefix);`
`298`	`301`	`}`
`299`	`302`
`300`	`303`	`void ASTJSONTest::printUpdatedExpectations(std::ostream&, std::string const&) const`