Skip to content

Return Params start #776

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
AdminCrystal wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Return Params start #776

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
2499a3e
Working
AdminCrystal Oct 3, 2025
d2d4c98
Working
AdminCrystal Oct 3, 2025
60128d2
Working
AdminCrystal Oct 3, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 8 additions & 8 deletions addons/gut/spy.gd
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ func _get_params_as_string(params):
return to_return


func add_call(variant, method_name, parameters=null):
func add_call(variant, method_name, parameters=null) -> void:
if(!_calls.has(variant)):
_calls[variant] = {}

Expand All @@ -52,7 +52,7 @@ func add_call(variant, method_name, parameters=null):
_calls[variant][method_name].append(parameters)


func was_called(variant, method_name, parameters=null):
func was_called(variant, method_name, parameters=null) -> bool:
var to_return = false
if(_calls.has(variant) and _calls[variant].has(method_name)):
if(parameters):
Expand All @@ -62,7 +62,7 @@ func was_called(variant, method_name, parameters=null):
return to_return


func get_call_parameters(variant, method_name, index=-1):
func get_call_parameters(variant, method_name, index=-1) -> Variant:
Copy link
Owner

@bitwes bitwes Oct 6, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Array, optionally Array[Variant], I'm fine with either, we just have to be consistent.

var to_return = null
var get_index = -1

Expand All @@ -82,7 +82,7 @@ func get_call_parameters(variant, method_name, index=-1):
return to_return


func call_count(instance, method_name, parameters=null):
func call_count(instance, method_name, parameters=null) -> int:
var to_return = 0

if(was_called(instance, method_name)):
Expand All @@ -95,11 +95,11 @@ func call_count(instance, method_name, parameters=null):
return to_return


func clear():
func clear() -> void:
_calls = {}


func get_call_list_as_string(instance):
func get_call_list_as_string(instance) -> String:
var to_return = ''
if(_calls.has(instance)):
for method in _calls[instance]:
Expand All @@ -108,9 +108,9 @@ func get_call_list_as_string(instance):
return to_return


func get_logger():
func get_logger() -> Variant:
return _lgr


func set_logger(logger):
func set_logger(logger) -> void:
_lgr = logger
Loading