refactor: sport module migrate (#684)

include/faker-cxx/Sport.h

@@ -2,11 +2,8 @@
 
 #include <string_view>
 
-namespace faker
+namespace faker::sport
 {
-class Sport
-{
-public:
     /**
      * @brief Returns a random sport.
      *
@@ -16,7 +13,8 @@ class Sport
      * Sport::sport() // "football"
      * @endcode
      */
-    static std::string_view sport();
+
+    std::string_view sportName();
 
     /**
      * @brief Returns a random soccer team.
@@ -27,7 +25,7 @@ class Sport
      * Sport::soccerTeam() // "Manchester United"
      * @endcode
      */
-    static std::string_view soccerTeam();
+    std::string_view soccerTeam();
 
     /**
      * @brief Returns a random male athlete.
@@ -38,7 +36,7 @@ class Sport
      * Sport::maleAthlete() // "Cristiano Ronaldo"
      * @endcode
      */
-    static std::string_view maleAthlete();
+    std::string_view maleAthlete();
 
     /**
      * @brief Returns a random female athlete.
@@ -49,7 +47,7 @@ class Sport
      * Sport::femaleAthlete() // "Serena Williams"
      * @endcode
      */
-    static std::string_view femaleAthlete();
+    std::string_view femaleAthlete();
 
     /**
      * @brief Returns a random Sport Event.
@@ -60,6 +58,5 @@ class Sport
      * Sport::sportEvent() // "Super Bowl"
      * @endcode
      */
-    static std::string_view sportEvent();
-};
+    std::string_view sportEvent();
 }

src/modules/sport/Sport.cpp

@@ -5,30 +5,30 @@
 #include "faker-cxx/Helper.h"
 #include "SportData.h"
 
-namespace faker
+namespace faker::sport
 {
-std::string_view Sport::sport()
+std::string_view sportName()
 {
-    return Helper::arrayElement(sport::sportNames);
+    return Helper::arrayElement(sportNames);
 }
 
-std::string_view Sport::soccerTeam()
+std::string_view soccerTeam()
 {
-    return Helper::arrayElement(sport::soccerTeams);
+    return Helper::arrayElement(soccerTeams);
 }
 
-std::string_view Sport::maleAthlete()
+std::string_view maleAthlete()
 {
-    return Helper::arrayElement(sport::maleAthletes);
+    return Helper::arrayElement(maleAthletes);
 }
 
-std::string_view Sport::femaleAthlete()
+std::string_view femaleAthlete()
 {
-    return Helper::arrayElement(sport::femaleAthletes);
+    return Helper::arrayElement(femaleAthletes);
 }
 
-std::string_view Sport::sportEvent()
+std::string_view sportEvent()
 {
-    return Helper::arrayElement(sport::sportEvents);
+    return Helper::arrayElement(sportEvents);
 }
 }

tests/modules/sport/SportTest.cpp

@@ -9,6 +9,7 @@
 
 using namespace ::testing;
 using namespace faker;
+using namespace faker::sport;
 
 class SportTest : public Test
 {
@@ -17,41 +18,41 @@ class SportTest : public Test
 
 TEST_F(SportTest, shouldGenerateSport)
 {
-    const auto generatedSport = Sport::sport();
+    const auto generatedSport = sportName();
 
-    ASSERT_TRUE(std::ranges::any_of(sport::sportNames, [generatedSport](const std::string_view& sport)
+    ASSERT_TRUE(std::ranges::any_of(sportNames, [generatedSport](const std::string_view& sport)
                                     { return sport == generatedSport; }));
 }
 
 TEST_F(SportTest, shouldGenerateSoccerTeam)
 {
-    const auto generatedSoccerTeam = Sport::soccerTeam();
+    const auto generatedSoccerTeam = soccerTeam();
 
-    ASSERT_TRUE(std::ranges::any_of(sport::soccerTeams, [generatedSoccerTeam](const std::string_view& soccerTeam)
+    ASSERT_TRUE(std::ranges::any_of(soccerTeams, [generatedSoccerTeam](const std::string_view& soccerTeam)
                                     { return soccerTeam == generatedSoccerTeam; }));
 }
 
 TEST_F(SportTest, shouldGenerateSportEvent)
 {
-    const auto generatedSportEvent = Sport::sportEvent();
+    const auto generatedSportEvent = sportEvent();
 
-    ASSERT_TRUE(std::ranges::any_of(sport::sportEvents, [generatedSportEvent](const std::string_view& sportEvent)
+    ASSERT_TRUE(std::ranges::any_of(sportEvents, [generatedSportEvent](const std::string_view& sportEvent)
                                     { return sportEvent == generatedSportEvent; }));
 }
 
 TEST_F(SportTest, shouldGenerateMaleAthlete)
 {
-    const auto generatedMaleAthlete = Sport::maleAthlete();
+    const auto generatedMaleAthlete = maleAthlete();
 
-    ASSERT_TRUE(std::ranges::any_of(sport::maleAthletes, [generatedMaleAthlete](const std::string_view& maleAthlete)
+    ASSERT_TRUE(std::ranges::any_of(maleAthletes, [generatedMaleAthlete](const std::string_view& maleAthlete)
                                     { return maleAthlete == generatedMaleAthlete; }));
 }
 
 TEST_F(SportTest, shouldGenerateFemaleAthlete)
 {
-    const auto generatedFemaleAthlete = Sport::femaleAthlete();
+    const auto generatedFemaleAthlete = femaleAthlete();
 
-    ASSERT_TRUE(std::ranges::any_of(sport::femaleAthletes,
+    ASSERT_TRUE(std::ranges::any_of(femaleAthletes,
                                     [generatedFemaleAthlete](const std::string_view& femaleAthlete)
                                     { return femaleAthlete == generatedFemaleAthlete; }));
 }