Add test for init of ManuItem.

przemek83 · Sep 14, 2024 · 1d73370 · 1d73370
1 parent 69fcea6
commit 1d73370
Show file tree

Hide file tree

Showing 4 changed files with 57 additions and 11 deletions.
diff --git a/src/MenuItem.cpp b/src/MenuItem.cpp
@@ -2,6 +2,7 @@
 
 #include <algorithm>
 
+#include "Display.h"
 #include "ResourceType.h"
 #include "Screen.h"
 
@@ -29,16 +30,16 @@ void MenuItem::setSelected(bool selected)
         resourceType_ = ResourceType::MENU_ITEM;
 }
 
-void MenuItem::init(const Screen& screen, int position, int count)
+void MenuItem::init(const Display& display, int position, int count)
 {
-    width_ = std::max(screen.getWidth() / 3,
-                      screen.getResourceWidth(ResourceType::MENU_ITEM));
-    height_ = std::max(screen.getHeight() / 10,
-                       screen.getResourceHeight(ResourceType::MENU_ITEM));
+    width_ = std::max(display.getWidth() / 3,
+                      display.getResourceWidth(ResourceType::MENU_ITEM));
+    height_ = std::max(display.getHeight() / 10,
+                       display.getResourceHeight(ResourceType::MENU_ITEM));
 
-    setX(screen.getCenterX() - width_ / 2);
+    setX(display.getCenterX() - width_ / 2);
 
-    const int locationOfFirstItem{screen.getCenterY() - (count * height_ / 2)};
+    const int locationOfFirstItem{display.getCenterY() - (count * height_ / 2)};
     setY(locationOfFirstItem + (height_ * position));
 }
 

diff --git a/src/MenuItem.h b/src/MenuItem.h
@@ -6,12 +6,14 @@
 #include "Drawable.h"
 #include "UserChoice.h"
 
+class Display;
+
 class MenuItem : public Drawable
 {
 public:
     explicit MenuItem(UserChoice userChoice);
 
-    void init(const Screen& screen, int position, int count);
+    void init(const Display& display, int position, int count);
 
     void draw(const Screen& screen) const override;
     ResourceType getResourceType() const override;

diff --git a/test/FakeDisplay.h b/test/FakeDisplay.h
@@ -8,11 +8,20 @@ class FakeDisplay : public Display
     int getResourceWidth(
         [[maybe_unused]] ResourceType resourceType) const override
     {
-        return 0;
+        return resourceWidth_;
     }
     int getResourceHeight(
         [[maybe_unused]] ResourceType resourceType) const override
     {
-        return 0;
+        return resourceHeight_;
     }
-};
+
+    void setResourceWidth(int width) { resourceWidth_ = width; }
+
+    void setResourceHeight(int height) { resourceHeight_ = height; }
+
+private:
+    int resourceWidth_;
+    int resourceHeight_;
+};
+;
diff --git a/test/MenuItemTest.cpp b/test/MenuItemTest.cpp
@@ -3,6 +3,8 @@
 #include <src/MenuItem.h>
 #include <src/Screen.h>
 
+#include "FakeDisplay.h"
+
 TEST_CASE("MenuItem initialization and properties", "[MenuItem]")
 {
     MenuItem menuItem(UserChoice::LEVEL_MENU);
@@ -27,4 +29,36 @@ TEST_CASE("MenuItem initialization and properties", "[MenuItem]")
         menuItem.setSelected(false);
         REQUIRE(menuItem.getResourceType() == ResourceType::MENU_ITEM);
     }
+
+    SECTION("initialization")
+    {
+        const int itemsCount{3};
+        const int resourceHeight{70};
+        const int resourceWidth{340};
+
+        FakeDisplay display;
+        display.setResourceHeight(resourceHeight);
+        display.setResourceWidth(resourceWidth);
+
+        menuItem.init(display, 0, itemsCount);
+        REQUIRE(menuItem.getX() == 230);
+        REQUIRE(menuItem.getY() == 195);
+        REQUIRE(menuItem.getWidth() == resourceWidth);
+        REQUIRE(menuItem.getHeight() == resourceHeight);
+        REQUIRE(menuItem.getCenter() == Point{170, 35});
+
+        menuItem.init(display, 1, itemsCount);
+        REQUIRE(menuItem.getX() == 230);
+        REQUIRE(menuItem.getY() == 265);
+        REQUIRE(menuItem.getWidth() == resourceWidth);
+        REQUIRE(menuItem.getHeight() == resourceHeight);
+        REQUIRE(menuItem.getCenter() == Point{170, 35});
+
+        menuItem.init(display, 2, itemsCount);
+        REQUIRE(menuItem.getX() == 230);
+        REQUIRE(menuItem.getY() == 335);
+        REQUIRE(menuItem.getWidth() == resourceWidth);
+        REQUIRE(menuItem.getHeight() == resourceHeight);
+        REQUIRE(menuItem.getCenter() == Point{170, 35});
+    }
 }