Move and organize tests into subfolders

tests/SCsub

@@ -22,6 +22,11 @@ if env_tests["platform"] == "windows":
 if env_tests.msvc:
     env_tests.Append(CCFLAGS=["/bigobj"])
 
+env_tests.add_source_files(env.tests_sources, "core/*.cpp")
+env_tests.add_source_files(env.tests_sources, "core/math/*.cpp")
+env_tests.add_source_files(env.tests_sources, "core/templates/*.cpp")
+env_tests.add_source_files(env.tests_sources, "scene/*.cpp")
+env_tests.add_source_files(env.tests_sources, "servers/*.cpp")
 env_tests.add_source_files(env.tests_sources, "*.cpp")
 
 lib = env_tests.add_library("tests", env.tests_sources)

tests/core/io/test_config_file.h

@@ -32,6 +32,7 @@
 #define TEST_CONFIG_FILE_H
 
 #include "core/io/config_file.h"
+#include "core/os/os.h"
 
 #include "tests/test_macros.h"
 

tests/core/io/test_file_access.h

@@ -32,7 +32,8 @@
 #define TEST_FILE_ACCESS_H
 
 #include "core/io/file_access.h"
-#include "test_utils.h"
+#include "tests/test_macros.h"
+#include "tests/test_utils.h"
 
 namespace TestFileAccess {
 

tests/core/io/test_image.h

@@ -31,10 +31,10 @@
 #ifndef TEST_IMAGE_H
 #define TEST_IMAGE_H
 
-#include "core/io/file_access_pack.h"
 #include "core/io/image.h"
-#include "test_utils.h"
+#include "core/os/os.h"
 
+#include "tests/test_utils.h"
 #include "thirdparty/doctest/doctest.h"
 
 namespace TestImage {

tests/core/io/test_pck_packer.h

@@ -35,6 +35,7 @@
 #include "core/io/pck_packer.h"
 #include "core/os/os.h"
 
+#include "tests/test_utils.h"
 #include "thirdparty/doctest/doctest.h"
 
 namespace TestPCKPacker {

tests/core/io/test_xml_parser.h

@@ -31,10 +31,7 @@
 #ifndef TEST_XML_PARSER_H
 #define TEST_XML_PARSER_H
 
-#include <inttypes.h>
-
 #include "core/io/xml_parser.h"
-#include "core/string/ustring.h"
 
 #include "tests/test_macros.h"
 

tests/core/math/test_aabb.h

@@ -32,10 +32,8 @@
 #define TEST_AABB_H
 
 #include "core/math/aabb.h"
-#include "core/string/print_string.h"
-#include "tests/test_macros.h"
 
-#include "thirdparty/doctest/doctest.h"
+#include "tests/test_macros.h"
 
 namespace TestAABB {
 

tests/core/math/test_astar.h

@@ -32,11 +32,6 @@
 #define TEST_ASTAR_H
 
 #include "core/math/a_star.h"
-#include "core/math/math_funcs.h"
-#include "core/os/os.h"
-
-#include <math.h>
-#include <stdio.h>
 
 #include "tests/test_macros.h"
 

tests/core/math/test_basis.h

@@ -31,9 +31,8 @@
 #ifndef TEST_BASIS_H
 #define TEST_BASIS_H
 
+#include "core/math/basis.h"
 #include "core/math/random_number_generator.h"
-#include "core/os/os.h"
-#include "core/string/ustring.h"
 
 #include "tests/test_macros.h"
 

tests/core/math/test_color.h

@@ -33,7 +33,7 @@
 
 #include "core/math/color.h"
 
-#include "thirdparty/doctest/doctest.h"
+#include "tests/test_macros.h"
 
 namespace TestColor {
 

tests/core/math/test_geometry_2d.h

@@ -32,7 +32,6 @@
 #define TEST_GEOMETRY_2D_H
 
 #include "core/math/geometry_2d.h"
-#include "core/templates/vector.h"
 
 #include "thirdparty/doctest/doctest.h"
 

tests/core/math/test_geometry_3d.h

@@ -32,11 +32,7 @@
 #define TEST_GEOMETRY_3D_H
 
 #include "core/math/geometry_3d.h"
-#include "core/math/plane.h"
-#include "core/math/random_number_generator.h"
-#include "core/math/vector3.h"
 #include "tests/test_macros.h"
-#include "vector"
 
 namespace TestGeometry3D {
 TEST_CASE("[Geometry3D] Closest Points Between Segments") {

tests/core/math/test_math.cpp

@@ -30,23 +30,11 @@
 
 #include "test_math.h"
 
-#include "core/io/file_access.h"
-#include "core/math/basis.h"
 #include "core/math/camera_matrix.h"
 #include "core/math/delaunay_3d.h"
 #include "core/math/geometry_2d.h"
-#include "core/math/math_funcs.h"
-#include "core/math/transform_3d.h"
-#include "core/os/keyboard.h"
+#include "core/os/main_loop.h"
 #include "core/os/os.h"
-#include "core/string/print_string.h"
-#include "core/string/ustring.h"
-#include "core/templates/vmap.h"
-#include "core/variant/method_ptrcall.h"
-#include "core/variant/variant.h"
-#include "scene/main/node.h"
-#include "scene/resources/texture.h"
-#include "servers/rendering/shader_language.h"
 
 namespace TestMath {
 

tests/core/math/test_math.h

@@ -31,7 +31,7 @@
 #ifndef TEST_MATH_H
 #define TEST_MATH_H
 
-#include "core/os/main_loop.h"
+class MainLoop;
 
 namespace TestMath {
 

tests/core/object/test_class_db.h

@@ -31,14 +31,9 @@
 #ifndef TEST_CLASS_DB_H
 #define TEST_CLASS_DB_H
 
-#include "core/register_core_types.h"
-
+#include "core/core_bind.h"
 #include "core/core_constants.h"
-#include "core/os/os.h"
-#include "core/string/string_name.h"
-#include "core/string/ustring.h"
-#include "core/templates/ordered_hash_map.h"
-#include "core/variant/variant.h"
+#include "core/object/class_db.h"
 
 #include "tests/test_macros.h"
 

tests/core/object/test_method_bind.h

@@ -35,10 +35,6 @@
 
 #include "tests/test_macros.h"
 
-#include <inttypes.h>
-#include <stdio.h>
-#include <wchar.h>
-
 namespace TestMethodBind {
 
 class MethodBindTester : public Object {

tests/core/object/test_object.h

@@ -32,9 +32,11 @@
 #define TEST_OBJECT_H
 
 #include "core/core_string_names.h"
+#include "core/object/class_db.h"
 #include "core/object/object.h"
+#include "core/object/script_language.h"
 
-#include "thirdparty/doctest/doctest.h"
+#include "tests/test_macros.h"
 
 // Declared in global namespace because of GDCLASS macro warning (Windows):
 // "Unqualified friend declaration referring to type outside of the nearest enclosing namespace

tests/core/string/test_node_path.h

@@ -33,7 +33,7 @@
 
 #include "core/string/node_path.h"
 
-#include "thirdparty/doctest/doctest.h"
+#include "tests/test_macros.h"
 
 namespace TestNodePath {
 

tests/core/string/test_string.h

@@ -31,15 +31,7 @@
 #ifndef TEST_STRING_H
 #define TEST_STRING_H
 
-#include <inttypes.h>
-#include <stdio.h>
-#include <wchar.h>
-
-#include "core/io/ip_address.h"
-#include "core/os/main_loop.h"
-#include "core/os/os.h"
 #include "core/string/ustring.h"
-#include "core/variant/variant.h"
 
 #include "tests/test_macros.h"
 

tests/core/string/test_translation.h

@@ -39,7 +39,8 @@
 #include "editor/import/resource_importer_csv_translation.h"
 #endif
 
-#include "thirdparty/doctest/doctest.h"
+#include "tests/test_macros.h"
+#include "tests/test_utils.h"
 
 namespace TestTranslation {
 

tests/core/templates/test_command_queue.h

@@ -33,12 +33,10 @@
 
 #include "core/config/project_settings.h"
 #include "core/math/random_number_generator.h"
-#include "core/os/mutex.h"
 #include "core/os/os.h"
-#include "core/os/semaphore.h"
 #include "core/os/thread.h"
 #include "core/templates/command_queue_mt.h"
-#include "test_macros.h"
+#include "tests/test_macros.h"
 
 #if !defined(NO_THREADS)
 

tests/core/templates/test_lru.h

@@ -32,7 +32,6 @@
 #define TEST_LRU_H
 
 #include "core/templates/lru.h"
-#include "core/templates/vector.h"
 
 #include "tests/test_macros.h"
 

tests/core/templates/test_oa_hash_map.h

@@ -31,7 +31,7 @@
 #ifndef TEST_OA_HASH_MAP_H
 #define TEST_OA_HASH_MAP_H
 
-#include "core/os/main_loop.h"
+class MainLoop;
 
 namespace TestOAHashMap {
 

tests/core/templates/test_ordered_hash_map.h

@@ -31,10 +31,7 @@
 #ifndef TEST_ORDERED_HASH_MAP_H
 #define TEST_ORDERED_HASH_MAP_H
 
-#include "core/os/os.h"
 #include "core/templates/ordered_hash_map.h"
-#include "core/templates/pair.h"
-#include "core/templates/vector.h"
 
 #include "tests/test_macros.h"
 

tests/core/test_crypto.h

@@ -33,7 +33,6 @@
 
 #include "core/crypto/crypto.h"
 #include "tests/test_macros.h"
-#include <stdio.h>
 
 namespace TestCrypto {
 

tests/core/variant/test_array.h

@@ -31,13 +31,7 @@
 #ifndef TEST_ARRAY_H
 #define TEST_ARRAY_H
 
-#include "core/object/class_db.h"
-#include "core/object/script_language.h"
-#include "core/templates/hashfuncs.h"
-#include "core/templates/vector.h"
 #include "core/variant/array.h"
-#include "core/variant/container_type_validate.h"
-#include "core/variant/variant.h"
 #include "tests/test_macros.h"
 #include "tests/test_tools.h"
 

tests/core/variant/test_dictionary.h

@@ -31,10 +31,7 @@
 #ifndef TEST_DICTIONARY_H
 #define TEST_DICTIONARY_H
 
-#include "core/templates/ordered_hash_map.h"
-#include "core/templates/safe_refcount.h"
 #include "core/variant/dictionary.h"
-#include "core/variant/variant.h"
 #include "tests/test_macros.h"
 
 namespace TestDictionary {

tests/scene/test_code_edit.h

@@ -31,12 +31,7 @@
 #ifndef TEST_CODE_EDIT_H
 #define TEST_CODE_EDIT_H
 
-#include "core/input/input_map.h"
-#include "core/object/message_queue.h"
-#include "core/os/keyboard.h"
-#include "core/string/string_builder.h"
 #include "scene/gui/code_edit.h"
-#include "scene/resources/default_theme/default_theme.h"
 
 #include "tests/test_macros.h"
 

tests/scene/test_curve.h

@@ -33,7 +33,7 @@
 
 #include "scene/resources/curve.h"
 
-#include "thirdparty/doctest/doctest.h"
+#include "tests/test_macros.h"
 
 namespace TestCurve {
 

tests/scene/test_gradient.h

@@ -31,8 +31,6 @@
 #ifndef TEST_GRADIENT_H
 #define TEST_GRADIENT_H
 
-#include "core/math/color.h"
-#include "core/object/class_db.h"
 #include "scene/resources/gradient.h"
 
 #include "thirdparty/doctest/doctest.h"

tests/scene/test_gui.cpp

@@ -32,29 +32,18 @@
 
 #include "test_gui.h"
 
-#include "core/io/image_loader.h"
-#include "core/os/os.h"
-#include "core/string/print_string.h"
-#include "scene/2d/sprite_2d.h"
 #include "scene/gui/button.h"
-#include "scene/gui/control.h"
 #include "scene/gui/label.h"
 #include "scene/gui/line_edit.h"
 #include "scene/gui/menu_button.h"
 #include "scene/gui/option_button.h"
 #include "scene/gui/panel.h"
-#include "scene/gui/popup_menu.h"
 #include "scene/gui/progress_bar.h"
 #include "scene/gui/rich_text_label.h"
 #include "scene/gui/scroll_bar.h"
 #include "scene/gui/spin_box.h"
 #include "scene/gui/tab_container.h"
-#include "scene/gui/texture_rect.h"
 #include "scene/gui/tree.h"
-#include "scene/main/scene_tree.h"
-
-#include "scene/3d/camera_3d.h"
-#include "scene/main/window.h"
 
 namespace TestGUI {
 
@@ -267,4 +256,4 @@ MainLoop *test() {
 }
 } // namespace TestGUI
 
-#endif
+#endif // _3D_DISABLED

tests/scene/test_gui.h

@@ -31,7 +31,7 @@
 #ifndef TEST_GUI_H
 #define TEST_GUI_H
 
-#include "core/os/main_loop.h"
+class MainLoop;
 
 namespace TestGUI {
 

tests/scene/test_path_3d.h

@@ -32,7 +32,6 @@
 #define TEST_PATH_3D_H
 
 #include "scene/3d/path_3d.h"
-#include "scene/resources/curve.h"
 
 #include "tests/test_macros.h"
 

tests/scene/test_path_follow_2d.h

@@ -32,7 +32,6 @@
 #define TEST_PATH_FOLLOW_2D_H
 
 #include "scene/2d/path_2d.h"
-#include "scene/resources/curve.h"
 
 #include "tests/test_macros.h"
 

tests/scene/test_path_follow_3d.h

@@ -32,7 +32,6 @@
 #define TEST_PATH_FOLLOW_3D_H
 
 #include "scene/3d/path_3d.h"
-#include "scene/resources/curve.h"
 
 #include "tests/test_macros.h"
 

tests/servers/test_physics_2d.cpp

@@ -31,11 +31,6 @@
 #include "test_physics_2d.h"
 
 #include "core/os/main_loop.h"
-#include "core/os/os.h"
-#include "core/string/print_string.h"
-#include "core/templates/map.h"
-#include "scene/resources/texture.h"
-#include "servers/display_server.h"
 #include "servers/physics_server_2d.h"
 #include "servers/rendering_server.h"
 

tests/servers/test_physics_2d.h

@@ -31,7 +31,7 @@
 #ifndef TEST_PHYSICS_2D_H
 #define TEST_PHYSICS_2D_H
 
-#include "core/os/main_loop.h"
+class MainLoop;
 
 namespace TestPhysics2D {
 

tests/servers/test_physics_3d.cpp

@@ -31,12 +31,8 @@
 #include "test_physics_3d.h"
 
 #include "core/math/convex_hull.h"
-#include "core/math/math_funcs.h"
+#include "core/math/geometry_3d.h"
 #include "core/os/main_loop.h"
-#include "core/os/os.h"
-#include "core/string/print_string.h"
-#include "core/templates/map.h"
-#include "servers/display_server.h"
 #include "servers/physics_server_3d.h"
 #include "servers/rendering_server.h"
 

tests/servers/test_physics_3d.h

@@ -31,7 +31,7 @@
 #ifndef TEST_PHYSICS_H
 #define TEST_PHYSICS_H
 
-#include "core/os/main_loop.h"
+class MainLoop;
 
 namespace TestPhysics3D {
 

tests/servers/test_render.cpp

@@ -31,12 +31,7 @@
 #include "test_render.h"
 
 #include "core/math/convex_hull.h"
-#include "core/math/math_funcs.h"
-#include "core/os/keyboard.h"
 #include "core/os/main_loop.h"
-#include "core/os/os.h"
-#include "core/string/print_string.h"
-#include "servers/display_server.h"
 #include "servers/rendering_server.h"
 
 #define OBJECT_COUNT 50

tests/servers/test_render.h

@@ -31,11 +31,11 @@
 #ifndef TEST_RENDER_H
 #define TEST_RENDER_H
 
-#include "core/os/main_loop.h"
+class MainLoop;
 
 namespace TestRender {
 
 MainLoop *test();
 }
 
-#endif
+#endif // TEST_RENDER_H

tests/servers/test_shader_lang.cpp

@@ -30,13 +30,8 @@
 
 #include "test_shader_lang.h"
 
-#include "core/io/file_access.h"
 #include "core/os/main_loop.h"
 #include "core/os/os.h"
-
-#include "core/string/print_string.h"
-#include "scene/gui/control.h"
-#include "scene/gui/text_edit.h"
 #include "servers/rendering/shader_language.h"
 
 typedef ShaderLanguage SL;

tests/servers/test_shader_lang.h

@@ -31,7 +31,7 @@
 #ifndef TEST_SHADER_LANG_H
 #define TEST_SHADER_LANG_H
 
-#include "core/os/main_loop.h"
+class MainLoop;
 
 namespace TestShaderLang {
 

tests/test_macros.h

@@ -31,10 +31,8 @@
 #ifndef TEST_MACROS_H
 #define TEST_MACROS_H
 
-#include "core/object/callable_method_pointer.h"
-#include "core/object/class_db.h"
-#include "core/string/print_string.h"
-#include "core/templates/map.h"
+#include "core/input/input_map.h"
+#include "core/object/message_queue.h"
 #include "core/variant/variant.h"
 
 // See documentation for doctest at:

tests/test_main.cpp

@@ -30,62 +30,62 @@
 
 #include "test_main.h"
 
-#include "core/templates/list.h"
-
-#include "test_aabb.h"
-#include "test_array.h"
-#include "test_astar.h"
-#include "test_basis.h"
-#include "test_class_db.h"
-#include "test_code_edit.h"
-#include "test_color.h"
-#include "test_command_queue.h"
-#include "test_config_file.h"
-#include "test_crypto.h"
-#include "test_curve.h"
-#include "test_dictionary.h"
-#include "test_expression.h"
-#include "test_file_access.h"
-#include "test_geometry_2d.h"
-#include "test_geometry_3d.h"
-#include "test_gradient.h"
-#include "test_gui.h"
-#include "test_hashing_context.h"
-#include "test_image.h"
-#include "test_json.h"
-#include "test_list.h"
-#include "test_local_vector.h"
-#include "test_lru.h"
-#include "test_marshalls.h"
-#include "test_math.h"
-#include "test_method_bind.h"
-#include "test_node_path.h"
-#include "test_oa_hash_map.h"
-#include "test_object.h"
-#include "test_ordered_hash_map.h"
-#include "test_paged_array.h"
-#include "test_path_3d.h"
-#include "test_pck_packer.h"
-#include "test_physics_2d.h"
-#include "test_physics_3d.h"
-#include "test_random_number_generator.h"
-#include "test_rect2.h"
-#include "test_render.h"
-#include "test_resource.h"
-#include "test_shader_lang.h"
-#include "test_string.h"
-#include "test_text_server.h"
-#include "test_time.h"
-#include "test_translation.h"
-#include "test_validate_testing.h"
-#include "test_variant.h"
-#include "test_vector.h"
-#include "test_xml_parser.h"
+#include "tests/core/io/test_config_file.h"
+#include "tests/core/io/test_file_access.h"
+#include "tests/core/io/test_image.h"
+#include "tests/core/io/test_json.h"
+#include "tests/core/io/test_marshalls.h"
+#include "tests/core/io/test_pck_packer.h"
+#include "tests/core/io/test_resource.h"
+#include "tests/core/io/test_xml_parser.h"
+#include "tests/core/math/test_aabb.h"
+#include "tests/core/math/test_astar.h"
+#include "tests/core/math/test_basis.h"
+#include "tests/core/math/test_color.h"
+#include "tests/core/math/test_expression.h"
+#include "tests/core/math/test_geometry_2d.h"
+#include "tests/core/math/test_geometry_3d.h"
+#include "tests/core/math/test_math.h"
+#include "tests/core/math/test_random_number_generator.h"
+#include "tests/core/math/test_rect2.h"
+#include "tests/core/object/test_class_db.h"
+#include "tests/core/object/test_method_bind.h"
+#include "tests/core/object/test_object.h"
+#include "tests/core/string/test_node_path.h"
+#include "tests/core/string/test_string.h"
+#include "tests/core/string/test_translation.h"
+#include "tests/core/templates/test_command_queue.h"
+#include "tests/core/templates/test_list.h"
+#include "tests/core/templates/test_local_vector.h"
+#include "tests/core/templates/test_lru.h"
+#include "tests/core/templates/test_oa_hash_map.h"
+#include "tests/core/templates/test_ordered_hash_map.h"
+#include "tests/core/templates/test_paged_array.h"
+#include "tests/core/templates/test_vector.h"
+#include "tests/core/test_crypto.h"
+#include "tests/core/test_hashing_context.h"
+#include "tests/core/test_time.h"
+#include "tests/core/variant/test_array.h"
+#include "tests/core/variant/test_dictionary.h"
+#include "tests/core/variant/test_variant.h"
+#include "tests/scene/test_code_edit.h"
+#include "tests/scene/test_curve.h"
+#include "tests/scene/test_gradient.h"
+#include "tests/scene/test_gui.h"
+#include "tests/scene/test_path_3d.h"
+#include "tests/servers/test_physics_2d.h"
+#include "tests/servers/test_physics_3d.h"
+#include "tests/servers/test_render.h"
+#include "tests/servers/test_shader_lang.h"
+#include "tests/servers/test_text_server.h"
+#include "tests/test_validate_testing.h"
 
 #include "modules/modules_tests.gen.h"
 
 #include "tests/test_macros.h"
 
+#include "scene/resources/default_theme/default_theme.h"
+
 int test_main(int argc, char *argv[]) {
 	bool run_tests = true;
 

tests/test_tools.h

@@ -31,8 +31,6 @@
 #ifndef TEST_TOOLS_H
 #define TEST_TOOLS_H
 
-#include "core/error/error_macros.h"
-
 struct ErrorDetector {
 	ErrorDetector() {
 		eh.errfunc = _detect_error;

tests/test_utils.cpp

@@ -28,7 +28,7 @@
 /* SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.                */
 /*************************************************************************/
 
-#include "test_utils.h"
+#include "tests/test_utils.h"
 
 #include "core/os/os.h"
 

tests/test_utils.h

@@ -31,7 +31,7 @@
 #ifndef TEST_UTILS_H
 #define TEST_UTILS_H
 
-#include "core/string/ustring.h"
+class String;
 
 namespace TestUtils {