godotengine
/
godot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

GDScript: Make all warnings enabled in test generation 

The test generation doesn't initialize the language (since it's already
initialized in main), but it still needs the warning enabled so it
matches the actual tests.
This commit is contained in:
George Marques 2021-10-06 11:37:13 -03:00
parent 8a0db53d0f
commit d6c799006a
No known key found for this signature in database
GPG Key ID: 046BD46A3201E43D
2 changed files with 10 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
modules/gdscript/tests/gdscript_test_runner.cpp
View File

@ -133,14 +133,13 @@ GDScriptTestRunner::GDScriptTestRunner(const String &p_source_dir, bool p_init_l
if (do_init_languages) { if (do_init_languages) {
init_language(p_source_dir); init_language(p_source_dir);
}
// Enable all warnings for GDScript, so we can test them. // Enable all warnings for GDScript, so we can test them.
ProjectSettings::get_singleton()->set_setting("debug/gdscript/warnings/enable", true); ProjectSettings::get_singleton()->set_setting("debug/gdscript/warnings/enable", true);
for (int i = 0; i < (int)GDScriptWarning::WARNING_MAX; i++) { for (int i = 0; i < (int)GDScriptWarning::WARNING_MAX; i++) {
String warning = GDScriptWarning::get_name_from_code((GDScriptWarning::Code)i).to_lower(); String warning = GDScriptWarning::get_name_from_code((GDScriptWarning::Code)i).to_lower();
ProjectSettings::get_singleton()->set_setting("debug/gdscript/warnings/" + warning, true); ProjectSettings::get_singleton()->set_setting("debug/gdscript/warnings/" + warning, true);
} }
}
// Enable printing to show results // Enable printing to show results
_print_line_enabled = true; _print_line_enabled = true;

4
modules/gdscript/tests/scripts/analyzer/features/auto_inferred_type_dont_error.out
View File

@ -1,2 +1,6 @@
GDTEST_OK GDTEST_OK
>> WARNING
>> Line: 6
>> UNSAFE_METHOD_ACCESS
>> The method 'free' is not present on the inferred type 'Variant' (but may be present on a subtype).
Ok Ok