From d0890f94d17a32a924a12f754a735ebfd95eaca6 Mon Sep 17 00:00:00 2001
From: ArthurSonzogni <sonzogniarthur@gmail.com>
Date: Wed, 27 Apr 2022 18:57:48 +0200
Subject: [PATCH] execute IWYU and add some coverage tests.

---
 src/ftxui/component/animation.cpp             | 10 ++--
 src/ftxui/component/animation_test.cpp        | 47 +++++++------------
 src/ftxui/component/input.cpp                 |  2 +-
 src/ftxui/component/menu.cpp                  |  5 +-
 src/ftxui/component/menu_test.cpp             | 47 +++++++++++++++++++
 src/ftxui/component/radiobox.cpp              |  5 +-
 .../component/terminal_input_parser_test.cpp  |  2 +-
 src/ftxui/dom/blink_test.cpp                  | 10 ++--
 src/ftxui/dom/bold_test.cpp                   | 10 ++--
 src/ftxui/dom/border_test.cpp                 |  8 ++--
 src/ftxui/dom/canvas_test.cpp                 | 13 +++--
 src/ftxui/dom/color_test.cpp                  |  8 ++--
 src/ftxui/dom/dim_test.cpp                    | 10 ++--
 src/ftxui/dom/flexbox_helper.cpp              |  1 -
 src/ftxui/dom/scroll_indicator_test.cpp       |  6 +--
 src/ftxui/dom/separator_test.cpp              |  8 ++--
 src/ftxui/dom/spinner_test.cpp                |  8 ++--
 src/ftxui/dom/underlined_test.cpp             | 10 ++--
 src/ftxui/screen/color_test.cpp               |  8 ++--
 src/ftxui/screen/string_test.cpp              |  2 +-
 20 files changed, 124 insertions(+), 96 deletions(-)

diff --git a/src/ftxui/component/animation.cpp b/src/ftxui/component/animation.cpp
index cbee0a2b..ce8b4157 100644
--- a/src/ftxui/component/animation.cpp
+++ b/src/ftxui/component/animation.cpp
@@ -197,13 +197,13 @@ float ElasticOut(float p) {
 // y = (1/2)*sin(13pi/2*(2*x))*pow(2, 10 * ((2*x) - 1))      ; [0,0.5)
 // y = (1/2)*(sin(-13pi/2*((2x-1)+1))*pow(2,-10(2*x-1)) + 2) ; [0.5, 1]
 float ElasticInOut(float p) {
-  if (p < 0.5F) {                                             // NOLINT
+  if (p < 0.5F) {                                               // NOLINT
     return 0.5 * std::sin(13.F * kPi2 * (2 * p)) *              // NOLINT
-           std::pow(2, 10 * ((2 * p) - 1));                   // NOLINT
-  } else {                                                    // NOLINT
+           std::pow(2, 10 * ((2 * p) - 1));                     // NOLINT
+  } else {                                                      // NOLINT
     return 0.5 * (std::sin(-13.F * kPi2 * ((2 * p - 1) + 1)) *  // NOLINT
-                      std::pow(2, -10 * (2 * p - 1)) +        // NOLINT
-                  2);                                         // NOLINT
+                      std::pow(2, -10 * (2 * p - 1)) +          // NOLINT
+                  2);                                           // NOLINT
   }
 }
 
diff --git a/src/ftxui/component/animation_test.cpp b/src/ftxui/component/animation_test.cpp
index 0ea1277f..e183a04a 100644
--- a/src/ftxui/component/animation_test.cpp
+++ b/src/ftxui/component/animation_test.cpp
@@ -11,37 +11,22 @@ namespace ftxui {
 
 TEST(AnimationTest, StartAndEnd) {
   std::vector<animation::easing::Function> functions = {
-    animation::easing::Linear,
-    animation::easing::QuadraticIn,
-    animation::easing::QuadraticOut,
-    animation::easing::QuadraticInOut,
-    animation::easing::CubicIn,
-    animation::easing::CubicOut,
-    animation::easing::CubicInOut,
-    animation::easing::QuarticIn,
-    animation::easing::QuarticOut,
-    animation::easing::QuarticInOut,
-    animation::easing::QuinticIn,
-    animation::easing::QuinticOut,
-    animation::easing::QuinticInOut,
-    animation::easing::SineIn,
-    animation::easing::SineOut,
-    animation::easing::SineInOut,
-    animation::easing::CircularIn,
-    animation::easing::CircularOut,
-    animation::easing::CircularInOut,
-    animation::easing::ExponentialIn,
-    animation::easing::ExponentialOut,
-    animation::easing::ExponentialInOut,
-    animation::easing::ElasticIn,
-    animation::easing::ElasticOut,
-    animation::easing::ElasticInOut,
-    animation::easing::BackIn,
-    animation::easing::BackOut,
-    animation::easing::BackInOut,
-    animation::easing::BounceIn,
-    animation::easing::BounceOut,
-    animation::easing::BounceInOut,
+      animation::easing::Linear,         animation::easing::QuadraticIn,
+      animation::easing::QuadraticOut,   animation::easing::QuadraticInOut,
+      animation::easing::CubicIn,        animation::easing::CubicOut,
+      animation::easing::CubicInOut,     animation::easing::QuarticIn,
+      animation::easing::QuarticOut,     animation::easing::QuarticInOut,
+      animation::easing::QuinticIn,      animation::easing::QuinticOut,
+      animation::easing::QuinticInOut,   animation::easing::SineIn,
+      animation::easing::SineOut,        animation::easing::SineInOut,
+      animation::easing::CircularIn,     animation::easing::CircularOut,
+      animation::easing::CircularInOut,  animation::easing::ExponentialIn,
+      animation::easing::ExponentialOut, animation::easing::ExponentialInOut,
+      animation::easing::ElasticIn,      animation::easing::ElasticOut,
+      animation::easing::ElasticInOut,   animation::easing::BackIn,
+      animation::easing::BackOut,        animation::easing::BackInOut,
+      animation::easing::BounceIn,       animation::easing::BounceOut,
+      animation::easing::BounceInOut,
   };
   for (auto& it : functions) {
     EXPECT_NEAR(0.f, it(0.f), 1.0e-4);
diff --git a/src/ftxui/component/input.cpp b/src/ftxui/component/input.cpp
index 0ff193e7..6883dc11 100644
--- a/src/ftxui/component/input.cpp
+++ b/src/ftxui/component/input.cpp
@@ -17,7 +17,7 @@
 #include "ftxui/screen/box.hpp"    // for Box
 #include "ftxui/screen/string.hpp"  // for GlyphPosition, GlyphCount, to_string, CellToGlyphIndex, to_wstring
 #include "ftxui/screen/util.hpp"  // for clamp
-#include "ftxui/util/ref.hpp"  // for StringRef, Ref, ConstStringRef
+#include "ftxui/util/ref.hpp"     // for StringRef, Ref, ConstStringRef
 
 namespace ftxui {
 
diff --git a/src/ftxui/component/menu.cpp b/src/ftxui/component/menu.cpp
index 0727c32e..f275ca3e 100644
--- a/src/ftxui/component/menu.cpp
+++ b/src/ftxui/component/menu.cpp
@@ -115,9 +115,8 @@ class MenuBase : public ComponentBase {
       bool is_focused = (focused_entry() == i) && is_menu_focused;
       bool is_selected = (*selected_ == i);
 
-      auto focus_management = !is_selected      ? nothing
-                              : is_menu_focused ? focus
-                                                : nothing;
+      auto focus_management =
+          !is_selected ? nothing : is_menu_focused ? focus : nothing;
       EntryState state = {
           entries_[i],
           false,
diff --git a/src/ftxui/component/menu_test.cpp b/src/ftxui/component/menu_test.cpp
index fd84a26a..a49aa388 100644
--- a/src/ftxui/component/menu_test.cpp
+++ b/src/ftxui/component/menu_test.cpp
@@ -43,6 +43,53 @@ TEST(MenuTest, RemoveEntries) {
   EXPECT_EQ(focused_entry, 1);
 }
 
+TEST(MenuTest, Directions) {
+  int selected = 0;
+  std::vector<std::string> entries = {"1", "2", "3"};
+  MenuOption option;
+  auto menu = Menu(&entries, &selected, &option);
+
+  {
+    option.direction = MenuOption::Down;
+    Screen screen(4, 3);
+    Render(screen, menu->Render());
+    EXPECT_EQ(screen.ToString(),
+              "\x1B[1m\x1B[7m> 1 \x1B[22m\x1B[27m\r\n"
+              "  2 \r\n"
+              "  3 ");
+  }
+
+  {
+    option.direction = MenuOption::Up;
+    Screen screen(4, 3);
+    Render(screen, menu->Render());
+    EXPECT_EQ(screen.ToString(),
+              "  3 \r\n"
+              "  2 \r\n"
+              "\x1B[1m\x1B[7m> 1 \x1B[22m\x1B[27m");
+  }
+
+  {
+    option.direction = MenuOption::Right;
+    Screen screen(10, 1);
+    Render(screen, menu->Render());
+    EXPECT_EQ(screen.ToString(),
+              "\x1B[1m\x1B[7m> 1\x1B[22m\x1B[27m"
+              "  2"
+              "  3 ");
+  }
+
+  {
+    option.direction = MenuOption::Left;
+    Screen screen(10, 1);
+    Render(screen, menu->Render());
+    EXPECT_EQ(screen.ToString(),
+              "  3"
+              "  2"
+              "\x1B[1m\x1B[7m> 1\x1B[22m\x1B[27m ");
+  }
+}
+
 }  // namespace ftxui
 
 // Copyright 2022 Arthur Sonzogni. All rights reserved.
diff --git a/src/ftxui/component/radiobox.cpp b/src/ftxui/component/radiobox.cpp
index 298518d3..f3f32b0d 100644
--- a/src/ftxui/component/radiobox.cpp
+++ b/src/ftxui/component/radiobox.cpp
@@ -39,9 +39,8 @@ class RadioboxBase : public ComponentBase {
     for (int i = 0; i < size(); ++i) {
       bool is_focused = (focused_entry() == i) && is_menu_focused;
       bool is_selected = (hovered_ == i);
-      auto focus_management = !is_selected      ? nothing
-                              : is_menu_focused ? focus
-                                                : select;
+      auto focus_management =
+          !is_selected ? nothing : is_menu_focused ? focus : select;
       auto state = EntryState{
           entries_[i],
           *selected_ == i,
diff --git a/src/ftxui/component/terminal_input_parser_test.cpp b/src/ftxui/component/terminal_input_parser_test.cpp
index 42d6d8b9..3611b22f 100644
--- a/src/ftxui/component/terminal_input_parser_test.cpp
+++ b/src/ftxui/component/terminal_input_parser_test.cpp
@@ -310,7 +310,7 @@ TEST(Event, Control) {
   cases.push_back({char(26), true});
   cases.push_back({char(127), false});
 
-  for(auto test : cases) {
+  for (auto test : cases) {
     auto event_receiver = MakeReceiver<Task>();
     {
       auto parser = TerminalInputParser(event_receiver->MakeSender());
diff --git a/src/ftxui/dom/blink_test.cpp b/src/ftxui/dom/blink_test.cpp
index 1b02a4fe..a4451178 100644
--- a/src/ftxui/dom/blink_test.cpp
+++ b/src/ftxui/dom/blink_test.cpp
@@ -1,9 +1,9 @@
 #include <gtest/gtest-message.h>  // for Message
 #include <gtest/gtest-test-part.h>  // for SuiteApiResolver, TestFactoryImpl, TestPartResult
-#include "gtest/gtest_pred_impl.h"       // for Test, EXPECT_EQ, TEST
 #include <string>                   // for allocator
-#include "ftxui/dom/elements.hpp"        // for text, flexbox
-#include "ftxui/screen/screen.hpp"       // for Screen
+#include "ftxui/dom/elements.hpp"   // for text, flexbox
+#include "ftxui/screen/screen.hpp"  // for Screen
+#include "gtest/gtest_pred_impl.h"  // for Test, EXPECT_EQ, TEST
 
 namespace ftxui {
 
@@ -11,10 +11,10 @@ TEST(BlinkTest, Basic) {
   auto element = text("text") | blink;
   Screen screen(5, 1);
   Render(screen, element);
-  EXPECT_TRUE(screen.PixelAt(0,0).blink);
+  EXPECT_TRUE(screen.PixelAt(0, 0).blink);
 }
 
-} // namespace ftxui
+}  // namespace ftxui
 
 // Copyright 2022 Arthur Sonzogni. All rights reserved.
 // Use of this source code is governed by the MIT license that can be found in
diff --git a/src/ftxui/dom/bold_test.cpp b/src/ftxui/dom/bold_test.cpp
index d1f87cab..ff8ccee9 100644
--- a/src/ftxui/dom/bold_test.cpp
+++ b/src/ftxui/dom/bold_test.cpp
@@ -1,9 +1,9 @@
 #include <gtest/gtest-message.h>  // for Message
 #include <gtest/gtest-test-part.h>  // for SuiteApiResolver, TestFactoryImpl, TestPartResult
-#include "gtest/gtest_pred_impl.h"       // for Test, EXPECT_EQ, TEST
 #include <string>                   // for allocator
-#include "ftxui/dom/elements.hpp"        // for text, flexbox
-#include "ftxui/screen/screen.hpp"       // for Screen
+#include "ftxui/dom/elements.hpp"   // for text, flexbox
+#include "ftxui/screen/screen.hpp"  // for Screen
+#include "gtest/gtest_pred_impl.h"  // for Test, EXPECT_EQ, TEST
 
 namespace ftxui {
 
@@ -11,10 +11,10 @@ TEST(BoldTest, Basic) {
   auto element = text("text") | bold;
   Screen screen(5, 1);
   Render(screen, element);
-  EXPECT_TRUE(screen.PixelAt(0,0).bold);
+  EXPECT_TRUE(screen.PixelAt(0, 0).bold);
 }
 
-} // namespace ftxui
+}  // namespace ftxui
 
 // Copyright 2022 Arthur Sonzogni. All rights reserved.
 // Use of this source code is governed by the MIT license that can be found in
diff --git a/src/ftxui/dom/border_test.cpp b/src/ftxui/dom/border_test.cpp
index 9f1706db..501ef6bc 100644
--- a/src/ftxui/dom/border_test.cpp
+++ b/src/ftxui/dom/border_test.cpp
@@ -1,9 +1,9 @@
 #include <gtest/gtest-message.h>  // for Message
 #include <gtest/gtest-test-part.h>  // for SuiteApiResolver, TestFactoryImpl, TestPartResult
-#include "gtest/gtest_pred_impl.h"       // for Test, EXPECT_EQ, TEST
 #include <string>                   // for allocator
-#include "ftxui/dom/elements.hpp"        // for text, flexbox
-#include "ftxui/screen/screen.hpp"       // for Screen
+#include "ftxui/dom/elements.hpp"   // for text, flexbox
+#include "ftxui/screen/screen.hpp"  // for Screen
+#include "gtest/gtest_pred_impl.h"  // for Test, EXPECT_EQ, TEST
 
 namespace ftxui {
 
@@ -99,7 +99,7 @@ TEST(BorderTest, Window) {
             "╰────────╯");
 }
 
-} // namespace ftxui
+}  // namespace ftxui
 
 // Copyright 2022 Arthur Sonzogni. All rights reserved.
 // Use of this source code is governed by the MIT license that can be found in
diff --git a/src/ftxui/dom/canvas_test.cpp b/src/ftxui/dom/canvas_test.cpp
index ec790c1f..e1fff825 100644
--- a/src/ftxui/dom/canvas_test.cpp
+++ b/src/ftxui/dom/canvas_test.cpp
@@ -1,9 +1,9 @@
 #include <gtest/gtest-message.h>  // for Message
 #include <gtest/gtest-test-part.h>  // for SuiteApiResolver, TestFactoryImpl, TestPartResult
-#include "gtest/gtest_pred_impl.h"       // for Test, EXPECT_EQ, TEST
 #include <string>                   // for allocator
-#include "ftxui/dom/elements.hpp"        // for text, flexbox
-#include "ftxui/screen/screen.hpp"       // for Screen
+#include "ftxui/dom/elements.hpp"   // for text, flexbox
+#include "ftxui/screen/screen.hpp"  // for Screen
+#include "gtest/gtest_pred_impl.h"  // for Test, EXPECT_EQ, TEST
 
 namespace ftxui {
 
@@ -16,7 +16,7 @@ uint32_t Hash(const std::string s) {
   }
   return hash;
 }
-}
+}  // namespace
 
 TEST(CanvasTest, GoldPoint) {
   Terminal::SetColorSupport(Terminal::Color::TrueColor);
@@ -40,7 +40,7 @@ TEST(CanvasTest, GoldPointColor) {
   Terminal::SetColorSupport(Terminal::Color::TrueColor);
   auto element = canvas([](Canvas& c) {  //
     c.DrawPoint(3, 3, 1, Color::Red);
-    c.DrawPointLine(3, 7, 10, 19,Color::Blue);
+    c.DrawPointLine(3, 7, 10, 19, Color::Blue);
     c.DrawPointCircle(10, 5, 3, Color::Yellow);
     c.DrawPointCircleFilled(20, 5, 3, Color::White);
     c.DrawPointEllipse(10, 10, 5, 2, Color::Black);
@@ -84,7 +84,6 @@ TEST(CanvasTest, GoldBlockColor) {
   EXPECT_EQ(Hash(screen.ToString()), 2869205941);
 }
 
-
 TEST(CanvasTest, GoldText) {
   Terminal::SetColorSupport(Terminal::Color::TrueColor);
   Canvas c(10, 10);
@@ -97,7 +96,7 @@ TEST(CanvasTest, GoldText) {
   EXPECT_EQ(Hash(screen.ToString()), 1074960375);
 }
 
-} // namespace ftxui
+}  // namespace ftxui
 
 // Copyright 2022 Arthur Sonzogni. All rights reserved.
 // Use of this source code is governed by the MIT license that can be found in
diff --git a/src/ftxui/dom/color_test.cpp b/src/ftxui/dom/color_test.cpp
index b49306e8..c4028040 100644
--- a/src/ftxui/dom/color_test.cpp
+++ b/src/ftxui/dom/color_test.cpp
@@ -1,9 +1,9 @@
 #include <gtest/gtest-message.h>  // for Message
 #include <gtest/gtest-test-part.h>  // for SuiteApiResolver, TestFactoryImpl, TestPartResult
-#include "gtest/gtest_pred_impl.h"       // for Test, EXPECT_EQ, TEST
 #include <string>                   // for allocator
-#include "ftxui/dom/elements.hpp"        // for text, flexbox
-#include "ftxui/screen/screen.hpp"       // for Screen
+#include "ftxui/dom/elements.hpp"   // for text, flexbox
+#include "ftxui/screen/screen.hpp"  // for Screen
+#include "gtest/gtest_pred_impl.h"  // for Test, EXPECT_EQ, TEST
 
 namespace ftxui {
 
@@ -23,7 +23,7 @@ TEST(ColorTest, Background) {
   EXPECT_EQ(screen.PixelAt(0, 0).background_color, Color::Red);
 }
 
-} // namespace ftxui
+}  // namespace ftxui
 
 // Copyright 2022 Arthur Sonzogni. All rights reserved.
 // Use of this source code is governed by the MIT license that can be found in
diff --git a/src/ftxui/dom/dim_test.cpp b/src/ftxui/dom/dim_test.cpp
index b0e0056a..a64c4a69 100644
--- a/src/ftxui/dom/dim_test.cpp
+++ b/src/ftxui/dom/dim_test.cpp
@@ -1,9 +1,9 @@
 #include <gtest/gtest-message.h>  // for Message
 #include <gtest/gtest-test-part.h>  // for SuiteApiResolver, TestFactoryImpl, TestPartResult
-#include "gtest/gtest_pred_impl.h"       // for Test, EXPECT_EQ, TEST
 #include <string>                   // for allocator
-#include "ftxui/dom/elements.hpp"        // for text, flexbox
-#include "ftxui/screen/screen.hpp"       // for Screen
+#include "ftxui/dom/elements.hpp"   // for text, flexbox
+#include "ftxui/screen/screen.hpp"  // for Screen
+#include "gtest/gtest_pred_impl.h"  // for Test, EXPECT_EQ, TEST
 
 namespace ftxui {
 
@@ -11,10 +11,10 @@ TEST(DimTest, Basic) {
   auto element = text("text") | dim;
   Screen screen(5, 1);
   Render(screen, element);
-  EXPECT_TRUE(screen.PixelAt(0,0).dim);
+  EXPECT_TRUE(screen.PixelAt(0, 0).dim);
 }
 
-} // namespace ftxui
+}  // namespace ftxui
 
 // Copyright 2022 Arthur Sonzogni. All rights reserved.
 // Use of this source code is governed by the MIT license that can be found in
diff --git a/src/ftxui/dom/flexbox_helper.cpp b/src/ftxui/dom/flexbox_helper.cpp
index 4bc80511..56e19f6d 100644
--- a/src/ftxui/dom/flexbox_helper.cpp
+++ b/src/ftxui/dom/flexbox_helper.cpp
@@ -358,7 +358,6 @@ void Compute(Global& global) {
   Compute1(global);
 }
 
-
 }  // namespace ftxui::flexbox_helper
 
 // Copyright 2021 Arthur Sonzogni. All rights reserved.
diff --git a/src/ftxui/dom/scroll_indicator_test.cpp b/src/ftxui/dom/scroll_indicator_test.cpp
index 171d9f68..6272df1b 100644
--- a/src/ftxui/dom/scroll_indicator_test.cpp
+++ b/src/ftxui/dom/scroll_indicator_test.cpp
@@ -1,9 +1,9 @@
 #include <gtest/gtest-message.h>  // for Message
 #include <gtest/gtest-test-part.h>  // for SuiteApiResolver, TestFactoryImpl, TestPartResult
-#include "gtest/gtest_pred_impl.h"       // for Test, EXPECT_EQ, TEST
 #include <string>                   // for allocator
-#include "ftxui/dom/elements.hpp"        // for text, flexbox
-#include "ftxui/screen/screen.hpp"       // for Screen
+#include "ftxui/dom/elements.hpp"   // for text, flexbox
+#include "ftxui/screen/screen.hpp"  // for Screen
+#include "gtest/gtest_pred_impl.h"  // for Test, EXPECT_EQ, TEST
 
 namespace ftxui {
 
diff --git a/src/ftxui/dom/separator_test.cpp b/src/ftxui/dom/separator_test.cpp
index f678da91..eb0a25f7 100644
--- a/src/ftxui/dom/separator_test.cpp
+++ b/src/ftxui/dom/separator_test.cpp
@@ -1,9 +1,9 @@
 #include <gtest/gtest-message.h>  // for Message
 #include <gtest/gtest-test-part.h>  // for SuiteApiResolver, TestFactoryImpl, TestPartResult
-#include "gtest/gtest_pred_impl.h"       // for Test, EXPECT_EQ, TEST
 #include <string>                   // for allocator
-#include "ftxui/dom/elements.hpp"        // for text, flexbox
-#include "ftxui/screen/screen.hpp"       // for Screen
+#include "ftxui/dom/elements.hpp"   // for text, flexbox
+#include "ftxui/screen/screen.hpp"  // for Screen
+#include "gtest/gtest_pred_impl.h"  // for Test, EXPECT_EQ, TEST
 
 namespace ftxui {
 
@@ -107,7 +107,7 @@ TEST(SeparatorTest, WithPixel) {
             "down");
 }
 
-} // namespace ftxui
+}  // namespace ftxui
 
 // Copyright 2022 Arthur Sonzogni. All rights reserved.
 // Use of this source code is governed by the MIT license that can be found in
diff --git a/src/ftxui/dom/spinner_test.cpp b/src/ftxui/dom/spinner_test.cpp
index d15ba162..5c5b9188 100644
--- a/src/ftxui/dom/spinner_test.cpp
+++ b/src/ftxui/dom/spinner_test.cpp
@@ -1,9 +1,9 @@
 #include <gtest/gtest-message.h>  // for Message
 #include <gtest/gtest-test-part.h>  // for SuiteApiResolver, TestFactoryImpl, TestPartResult
-#include "gtest/gtest_pred_impl.h"       // for Test, EXPECT_EQ, TEST
 #include <string>                   // for allocator
-#include "ftxui/dom/elements.hpp"        // for text, flexbox
-#include "ftxui/screen/screen.hpp"       // for Screen
+#include "ftxui/dom/elements.hpp"   // for text, flexbox
+#include "ftxui/screen/screen.hpp"  // for Screen
+#include "gtest/gtest_pred_impl.h"  // for Test, EXPECT_EQ, TEST
 
 namespace ftxui {
 
@@ -35,7 +35,7 @@ TEST(SpinnerTest, Spinner4) {
   EXPECT_EQ(screen.ToString(), ".   ");
 }
 
-} // namespace ftxui
+}  // namespace ftxui
 
 // Copyright 2022 Arthur Sonzogni. All rights reserved.
 // Use of this source code is governed by the MIT license that can be found in
diff --git a/src/ftxui/dom/underlined_test.cpp b/src/ftxui/dom/underlined_test.cpp
index 2b0e6b88..2109a4cf 100644
--- a/src/ftxui/dom/underlined_test.cpp
+++ b/src/ftxui/dom/underlined_test.cpp
@@ -1,9 +1,9 @@
 #include <gtest/gtest-message.h>  // for Message
 #include <gtest/gtest-test-part.h>  // for SuiteApiResolver, TestFactoryImpl, TestPartResult
-#include "gtest/gtest_pred_impl.h"       // for Test, EXPECT_EQ, TEST
 #include <string>                   // for allocator
-#include "ftxui/dom/elements.hpp"        // for text, flexbox
-#include "ftxui/screen/screen.hpp"       // for Screen
+#include "ftxui/dom/elements.hpp"   // for text, flexbox
+#include "ftxui/screen/screen.hpp"  // for Screen
+#include "gtest/gtest_pred_impl.h"  // for Test, EXPECT_EQ, TEST
 
 namespace ftxui {
 
@@ -11,10 +11,10 @@ TEST(UnderlinedTest, Basic) {
   auto element = text("text") | underlined;
   Screen screen(5, 1);
   Render(screen, element);
-  EXPECT_TRUE(screen.PixelAt(0,0).underlined);
+  EXPECT_TRUE(screen.PixelAt(0, 0).underlined);
 }
 
-} // namespace ftxui
+}  // namespace ftxui
 
 // Copyright 2022 Arthur Sonzogni. All rights reserved.
 // Use of this source code is governed by the MIT license that can be found in
diff --git a/src/ftxui/screen/color_test.cpp b/src/ftxui/screen/color_test.cpp
index bcceeec0..25fbcbf4 100644
--- a/src/ftxui/screen/color_test.cpp
+++ b/src/ftxui/screen/color_test.cpp
@@ -26,18 +26,18 @@ TEST(ColorTest, PrintColor256) {
 
 TEST(ColorTest, PrintTrueCOlor) {
   Terminal::SetColorSupport(Terminal::Color::TrueColor);
-  EXPECT_EQ(Color::RGB(1,2,3).Print(false), "38;2;1;2;3");
-  EXPECT_EQ(Color::RGB(1,2,3).Print(true), "48;2;1;2;3");
+  EXPECT_EQ(Color::RGB(1, 2, 3).Print(false), "38;2;1;2;3");
+  EXPECT_EQ(Color::RGB(1, 2, 3).Print(true), "48;2;1;2;3");
 }
 
 TEST(ColorTest, FallbackTo256) {
   Terminal::SetColorSupport(Terminal::Color::Palette256);
-  EXPECT_EQ(Color::RGB(1,2,3).Print(false), "38;5;16");
+  EXPECT_EQ(Color::RGB(1, 2, 3).Print(false), "38;5;16");
 }
 
 TEST(ColorTest, FallbackTo16) {
   Terminal::SetColorSupport(Terminal::Color::Palette16);
-  EXPECT_EQ(Color::RGB(1,2,3).Print(false), "30");
+  EXPECT_EQ(Color::RGB(1, 2, 3).Print(false), "30");
 }
 
 TEST(ColorTest, Litterals) {
diff --git a/src/ftxui/screen/string_test.cpp b/src/ftxui/screen/string_test.cpp
index 5518d249..76da3512 100644
--- a/src/ftxui/screen/string_test.cpp
+++ b/src/ftxui/screen/string_test.cpp
@@ -120,7 +120,7 @@ TEST(StringTest, CellToGlyphIndex) {
   EXPECT_EQ(combining[2], 2);
 }
 
-}
+}  // namespace ftxui
 // Copyright 2020 Arthur Sonzogni. All rights reserved.
 // Use of this source code is governed by the MIT license that can be found in
 // the LICENSE file.
-- 
GitLab