Make GDScriptCache clearable multiple times

Fixes refactor tests memory leaks.

Author: adamscott

modules/gdscript/gdscript_cache.cpp

@@ -159,7 +159,7 @@ void GDScriptCache::move_script(const String &p_from, const String &p_to) {
 
 	MutexLock lock(singleton->mutex);
 
-	if (singleton->cleared) {
+	if (singleton->clearing) {
 		return;
 	}
 
@@ -183,7 +183,7 @@ void GDScriptCache::remove_script(const String &p_path) {
 
 	MutexLock lock(singleton->mutex);
 
-	if (singleton->cleared) {
+	if (singleton->clearing) {
 		return;
 	}
 
@@ -451,10 +451,10 @@ void GDScriptCache::clear() {
 
 	MutexLock lock(singleton->mutex);
 
-	if (singleton->cleared) {
+	if (singleton->clearing) {
 		return;
 	}
-	singleton->cleared = true;
+	singleton->clearing = true;
 
 	singleton->parser_inverse_dependencies.clear();
 
@@ -486,14 +486,16 @@ void GDScriptCache::clear() {
 	singleton->shallow_gdscript_cache.clear();
 	singleton->full_gdscript_cache.clear();
 	singleton->static_gdscript_cache.clear();
+
+	singleton->clearing = false;
 }
 
 GDScriptCache::GDScriptCache() {
 	singleton = this;
 }
 
 GDScriptCache::~GDScriptCache() {
-	if (!cleared) {
+	if (!clearing) {
 		clear();
 	}
 	singleton = nullptr;

modules/gdscript/gdscript_cache.h

@@ -94,7 +94,7 @@ class GDScriptCache {
 
 	static GDScriptCache *singleton;
 
-	bool cleared = false;
+	bool clearing = false;
 
 public:
 	static const int BINARY_MUTEX_TAG = 2;

modules/gdscript/tests/gdscript_test_runner.cpp

@@ -183,6 +183,7 @@ void init_language() {
 }
 
 void finish_language() {
+	GDScriptCache::clear();
 	GDScriptLanguage::get_singleton()->finish();
 	ScriptServer::global_classes_clear();
 }