Fix indentation of empty sequences and add test

c67d701a · Alejandro-FA · Jesse Beder · 9eb11429 · c67d701a · c67d701a
Commit c67d701a authored Jan 25, 2024 by Alejandro-FA Committed by Jesse Beder Jan 26, 2024
Hide whitespace changes
Inline Side-by-side

Showing with 13 additions and 1 deletion

src/emitter.cpp src/emitter.cpp +2 -1

test/integration/emitter_test.cpp test/integration/emitter_test.cpp +11 -0

No files found.
--- a/src/emitter.cpp
+++ b/src/emitter.cpp
@@ -213,7 +213,8 @@ void Emitter::EmitEndSeq() {
  if (m_pState->CurGroupFlowType() == FlowType::Flow) {
    if (m_stream.comment())
      m_stream << "\n";
-    m_stream << IndentTo(m_pState->CurIndent());
+    if (originalType == FlowType::Block || m_pState->HasBegunNode())
+      m_stream << IndentTo(m_pState->CurIndent());
    if (originalType == FlowType::Block) {
      m_stream << "[";
    } else {

--- a/test/integration/emitter_test.cpp
+++ b/test/integration/emitter_test.cpp
@@ -176,6 +176,17 @@ TEST_F(EmitterTest, EmptyFlowSeqWithBegunContent) {
  ]])");
 }

+TEST_F(EmitterTest, EmptyFlowSeqInMap) {
+  out << BeginMap;
+  out << Key << Flow << BeginSeq << EndSeq;
+  out << Value << 1;
+  out << Key << 2;
+  out << Value << Flow << BeginSeq << EndSeq;
+  out << EndMap;
+
+  ExpectEmit("[]: 1\n2: []");
+}
+
 TEST_F(EmitterTest, EmptyFlowMapWithBegunContent) {
  out << Flow;
  out << BeginSeq;