Skip to content

Commit ed5d975

Browse files
authored
Using GdUnit3MonoAPI class wrapper to load GdUnit3Mono assembly (#308)
1 parent 0f38912 commit ed5d975

12 files changed

+47
-114
lines changed

addons/gdUnit3/bin/GdUnitCmdTool.gd

+1-1
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ class CLIRunner extends Node:
5454
_executor.fail_fast(true)
5555

5656
if GdUnitTools.is_mono_supported():
57-
_cs_executor = GdUnit3MonoBridge.create_executor(self)
57+
_cs_executor = GdUnit3MonoAPI.create_executor(self)
5858

5959
var err := _executor.connect("send_event", self, "_on_executor_event")
6060
if err != OK:

addons/gdUnit3/src/core/GdObjects.gd

+1-1
Original file line numberDiff line numberDiff line change
@@ -327,7 +327,7 @@ static func is_script(value) -> bool:
327327
return is_object(value) and value is Script
328328

329329
static func is_test_suite(script :Script) -> bool:
330-
return is_gd_testsuite(script) or GdUnit3MonoBridge.is_test_suite(script.resource_path)
330+
return is_gd_testsuite(script) or GdUnit3MonoAPI.is_test_suite(script.resource_path)
331331

332332
static func is_native_class(value) -> bool:
333333
return is_object(value) and value.to_string() != null and value.to_string().find("GDScriptNativeClass") != -1

addons/gdUnit3/src/core/GdUnitRunner.gd

+1-1
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ func _init():
3030
_signal_handler = GdUnitSingleton.get_or_create_singleton(SignalHandler.SINGLETON_NAME, "res://addons/gdUnit3/src/core/event/SignalHandler.gd")
3131
# store current runner instance to engine meta data to can be access in as a singleton
3232
Engine.set_meta(GDUNIT_RUNNER, self)
33-
_cs_executor = GdUnit3MonoBridge.create_executor(self)
33+
_cs_executor = GdUnit3MonoAPI.create_executor(self)
3434

3535
func _ready():
3636
var config_result := _config.load()

addons/gdUnit3/src/core/GdUnitTestSuiteBuilder.gd

+1-1
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ func create(source :Script, line_number :int) -> Result:
2323
_save_and_close_script(test_suite_path)
2424

2525
if GdObjects.is_cs_script(source):
26-
return GdUnit3MonoBridge.create_test_suite(source.resource_path, line_number+1, test_suite_path)
26+
return GdUnit3MonoAPI.create_test_suite(source.resource_path, line_number+1, test_suite_path)
2727

2828
var parser := GdScriptParser.new()
2929
var lines := source.source_code.split("\n")

addons/gdUnit3/src/core/_TestSuiteScanner.gd

+3-3
Original file line numberDiff line numberDiff line change
@@ -55,8 +55,8 @@ static func _file(dir :Directory, file_name :String) -> String:
5555
func _parse_is_test_suite(resource_path :String) -> Node:
5656
if not _is_script_format_supported(resource_path):
5757
return null
58-
if GdUnit3MonoBridge.is_test_suite(resource_path):
59-
return GdUnit3MonoBridge.parse_test_suite(resource_path)
58+
if GdUnit3MonoAPI.is_test_suite(resource_path):
59+
return GdUnit3MonoAPI.parse_test_suite(resource_path)
6060
var script :Script = ResourceLoader.load(resource_path)
6161
if not GdObjects.is_test_suite(script):
6262
return null
@@ -68,7 +68,7 @@ static func _is_script_format_supported(resource_path :String) -> bool:
6868
var ext := resource_path.get_extension()
6969
if ext == "gd":
7070
return true
71-
return GdUnit3MonoBridge.is_csharp_file(resource_path)
71+
return GdUnit3MonoAPI.is_csharp_file(resource_path)
7272

7373
func _parse_test_suite(script :GDScript) -> GdUnitTestSuite:
7474
var test_suite = script.new()
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
2+
// GdUnit3 c# API wrapper
3+
public class GdUnit3MonoAPI : GdUnit3.GdUnit3MonoAPI
4+
{
5+
}

addons/gdUnit3/src/mono/GdUnit3MonoBridge.gd addons/gdUnit3/src/mono/GdUnit3MonoAPI.gd

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
11
extends Reference
2-
class_name GdUnit3MonoBridge
2+
class_name GdUnit3MonoAPI
33

44
const _instance : = []
55

66
static func instance() :
77
if not _instance.empty():
88
return _instance[0]
9-
var instance = load("res://addons/gdUnit3/src/mono/GdUnit3MonoBridge.cs").new()
9+
var instance = load("res://addons/gdUnit3/src/mono/GdUnit3MonoAPI.cs").new()
1010
_instance.append(instance)
1111
return _instance[0]
1212

addons/gdUnit3/src/mono/GdUnit3MonoBridge.cs

-71
This file was deleted.
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
// GdUnit generated TestSuite
2+
using Godot;
3+
using GdUnit3;
4+
using System;
5+
6+
namespace GdUnit3
7+
{
8+
using static Assertions;
9+
using static Utils;
10+
11+
[TestSuite]
12+
public class GdUnit3MonoAPITest
13+
{
14+
// TestSuite generated from
15+
private const string sourceClazzPath = "d:/develop/workspace/gdUnit3/addons/gdUnit3/src/mono/GdUnit3MonoAPI.cs";
16+
[TestCase]
17+
public void IsTestSuite()
18+
{
19+
AssertThat(GdUnit3MonoAPI.IsTestSuite("res://addons/gdUnit3/src/mono/GdUnit3MonoAPI.cs")).IsFalse();
20+
AssertThat(GdUnit3MonoAPI.IsTestSuite("res://addons/gdUnit3/test/mono/GdUnit3MonoAPITest.cs")).IsTrue();
21+
}
22+
}
23+
}

addons/gdUnit3/test/mono/GdUnit3MonoBridgeTest.gd addons/gdUnit3/test/mono/GdUnit3MonoAPITest.gd

+4-4
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,11 @@
11
# GdUnit generated TestSuite
22
#warning-ignore-all:unused_argument
33
#warning-ignore-all:return_value_discarded
4-
class_name GdUnit3MonoBridgeTest
4+
class_name GdUnit3MonoAPITest
55
extends GdUnitTestSuite
66

77
# TestSuite generated from
8-
const __source = 'res://addons/gdUnit3/src/mono/GdUnit3MonoBridge.gd'
8+
const __source = 'res://addons/gdUnit3/src/mono/GdUnit3MonoAPI.gd'
99
var _example_source_cs :String
1010

1111
func before_test():
@@ -20,8 +20,8 @@ func test_create_test_suite() -> void:
2020
return
2121
var source := load(_example_source_cs)
2222
var test_suite_path := _TestSuiteScanner.resolve_test_suite_path(source.resource_path, "test")
23-
var result := GdUnit3MonoBridge.create_test_suite(source.resource_path, 18, test_suite_path)
24-
23+
var result := GdUnit3MonoAPI.create_test_suite(source.resource_path, 18, test_suite_path)
24+
2525
assert_result(result).is_success()
2626
var info := result.value() as Dictionary
2727
assert_str(info.get("path")).is_equal("user://tmp/test/examples/TestPersonTest.cs")

addons/gdUnit3/test/mono/GdUnit3MonoBridgeTest.cs

-24
This file was deleted.

project.godot

+6-6
Original file line numberDiff line numberDiff line change
@@ -305,14 +305,14 @@ _global_script_classes=[ {
305305
"path": "res://addons/gdUnit3/src/core/parse/GdScriptParser.gd"
306306
}, {
307307
"base": "Reference",
308-
"class": "GdUnit3MonoBridge",
308+
"class": "GdUnit3MonoAPI",
309309
"language": "GDScript",
310-
"path": "res://addons/gdUnit3/src/mono/GdUnit3MonoBridge.gd"
310+
"path": "res://addons/gdUnit3/src/mono/GdUnit3MonoAPI.gd"
311311
}, {
312312
"base": "GdUnitTestSuite",
313-
"class": "GdUnit3MonoBridgeTest",
313+
"class": "GdUnit3MonoAPITest",
314314
"language": "GDScript",
315-
"path": "res://addons/gdUnit3/test/mono/GdUnit3MonoBridgeTest.gd"
315+
"path": "res://addons/gdUnit3/test/mono/GdUnit3MonoAPITest.gd"
316316
}, {
317317
"base": "Reference",
318318
"class": "GdUnit3Version",
@@ -1249,8 +1249,8 @@ _global_script_class_icons={
12491249
"GdMarkDownReaderTest": "",
12501250
"GdObjects": "",
12511251
"GdScriptParser": "",
1252-
"GdUnit3MonoBridge": "",
1253-
"GdUnit3MonoBridgeTest": "",
1252+
"GdUnit3MonoAPI": "",
1253+
"GdUnit3MonoAPITest": "",
12541254
"GdUnit3Version": "",
12551255
"GdUnit3VersionTest": "",
12561256
"GdUnitArgumentMatcher": "",

0 commit comments

Comments
 (0)