Fixing PR issues

lib/wal/CMakeLists.txt

@@ -33,7 +33,7 @@ add_library(wal
 
 target_include_directories(wal PUBLIC sources)
 
-add_executable(wal_tests
+add_executable(wal_tests EXCLUDE_FROM_ALL
 	tests/EntityTests.cpp
 	tests/MainTest.cpp
 	tests/EngineTests.cpp

lib/wal/sources/Component/Component.hpp

@@ -1,5 +1,6 @@
-
-
+//
+// Created by Zoe Roux on 2021-05-14.
+//
 
 #pragma once
 

lib/wal/sources/Entity/Entity.hpp

@@ -1,4 +1,6 @@
-
+//
+// Created by Zoe Roux on 2021-05-14.
+//
 
 #pragma once
 
@@ -44,7 +46,7 @@ namespace WAL
 		T &getComponent()
 		{
 			const std::type_info &type = typeid(T);
-			auto existing = std::find_if(this->_components.begin(), this->_components.end(), [&type] (auto &cmp) {
+			auto existing = std::find_if(this->_components.begin(), this->_components.end(), [&type] (const auto &cmp) {
 				return typeid(*cmp) == type;
 			});
 			if (existing == this->_components.end())
@@ -88,7 +90,7 @@ namespace WAL
 		Entity &removeComponent()
 		{
 			const std::type_info &type = typeid(T);
-			auto existing =std::find_if(this->_components.begin(), this->_components.end(), [&type] (auto &cmp) {
+			auto existing = std::find_if(this->_components.begin(), this->_components.end(), [&type] (const auto &cmp) {
 				return typeid(*cmp) == type;
 			});
 			if (existing == this->_components.end())

lib/wal/sources/Scene/Scene.cpp

@@ -8,7 +8,7 @@ namespace WAL
 {
 	std::vector<Entity> &Scene::getEntities()
 	{
-		return this->_entity;
+		return this->_entities;
 	}
 }
 

lib/wal/sources/Scene/Scene.hpp

@@ -15,7 +15,7 @@ namespace WAL
 	{
 	private:
 		//! @brief The list of registered entities
-		std::vector<Entity> _entity;
+		std::vector<Entity> _entities;
 	public:
 		//! @brief Get the list of entities.
 		std::vector<Entity> &getEntities();

lib/wal/sources/Scene/SceneManager.hpp

@@ -20,7 +20,7 @@ namespace WAL
 		//! @return The manager instance used to call this function is returned. This allow method chaining.
 		SceneManager &addScene(Scene &&scene);
 
-		//! @brief Add a scene before the current scene to allow
+		//! @brief Add a scene before the current scene. This could be useful for lobbies or scene where the next scene can be constructed.
 		//! @return The manager instance used to call this function is returned. This allow method chaining.
 		SceneManager &addBackScene(Scene &&scene);
 

lib/wal/sources/System/System.hpp

@@ -1,4 +1,6 @@
-
+//
+// Created by Zoe Roux on 2021-05-14.
+//
 
 #pragma once
 

lib/wal/sources/Wal.cpp

@@ -11,7 +11,7 @@ namespace WAL
 {
 	std::chrono::nanoseconds Wal::timestep = 8ms;
 
-	SceneManager &Wal::getSceneManger()
+	SceneManager &Wal::getSceneManager()
 	{
 		return this->_scenes;
 	}

lib/wal/sources/Wal.hpp

@@ -101,7 +101,7 @@ namespace WAL
 		}
 
 		//! @brief Get the scene manager.
-		SceneManager &getSceneManger();
+		SceneManager &getSceneManager();
 
 		//! @brief Start the game loop
 		void run();