Add missing signals

Author: BenjaTK

addons/gaea/nodes/generator.gd

@@ -8,8 +8,10 @@ extends Node
 
 ## Emitted when [GaeaGraph] is changed.
 signal data_changed
-@warning_ignore("unused_signal")
+## Emitted when the graph is about to generate.
+signal about_to_generate
 ## Emitted when the graph is done with the generation.
+@warning_ignore("unused_signal")
 signal generation_finished(grid: GaeaGrid)
 ## Emitted when this generator wants to trigger a reset. See [method GaeaRenderer._reset].
 signal reset_requested
@@ -39,6 +41,7 @@ signal area_erased(area: AABB)
 ## Start the generaton process. First resets the current generation, then generates the whole
 ## [member world_size].
 func generate() -> void:
+	about_to_generate.emit()
 	if random_seed_on_generate:
 		seed = randi()
 	request_reset()

addons/gaea/nodes/renderers/gridmap_renderer.gd

@@ -31,7 +31,7 @@ func _render(grid: GaeaGrid) -> void:
 				grid_maps[layer_idx].set_cell_item(cell, value.item_idx)
 
 
-func _on_area_erased(area: AABB) -> void:
+func _erase_area(area: AABB) -> void:
 	for x in range(area.position.x, area.end.x):
 		for y in range(area.position.y, area.end.y):
 			for z in range(area.position.z, area.end.z):

addons/gaea/nodes/renderers/renderer.gd

@@ -8,17 +8,23 @@ extends Node
 ## This is an abstract class. On its own, it doesn't do anything,
 ## but it can be extended to customize the way your generation will be rendered.
 
+## Emitted when the node is done with rendering an area.
+signal render_finished
+## Emitted when [method reset] is called.
+signal render_reset
+## Emitted when an area of the render has been erased.
+signal area_erased(area: AABB)
 
 ## Will render this [GaeaGenerator]'s generation results.
 @export var generator: GaeaGenerator :
 	set(value):
 		if is_instance_valid(generator):
-			if generator.generation_finished.is_connected(_render):
-				generator.generation_finished.disconnect(_render)
-			if generator.area_erased.is_connected(_on_area_erased):
-				generator.area_erased.disconnect(_on_area_erased)
-			if generator.reset_requested.is_connected(_reset):
-				generator.reset_requested.disconnect(_reset)
+			if generator.generation_finished.is_connected(render):
+				generator.generation_finished.disconnect(render)
+			if generator.area_erased.is_connected(erase_area):
+				generator.area_erased.disconnect(erase_area)
+			if generator.reset_requested.is_connected(reset):
+				generator.reset_requested.disconnect(reset)
 		generator = value
 		_connect_signals()
 
@@ -32,12 +38,29 @@ func _connect_signals() -> void:
 	if not is_instance_valid(generator):
 		return
 
-	if not generator.generation_finished.is_connected(_render):
-		generator.generation_finished.connect(_render)
-	if not generator.area_erased.is_connected(_on_area_erased):
-		generator.area_erased.connect(_on_area_erased)
-	if not generator.reset_requested.is_connected(_reset):
-		generator.reset_requested.connect(_reset)
+	if not generator.generation_finished.is_connected(render):
+		generator.generation_finished.connect(render)
+	if not generator.area_erased.is_connected(erase_area):
+		generator.area_erased.connect(erase_area)
+	if not generator.reset_requested.is_connected(reset):
+		generator.reset_requested.connect(reset)
+
+
+## Public version of [method _render].
+func render(grid: GaeaGrid) -> void:
+	_render(grid)
+	render_finished.emit()
+	
+
+## Public version of [method _reset].
+func reset() -> void:
+	_reset()
+	render_reset.emit()
+	
+
+func erase_area(area: AABB) -> void:
+	_erase_area(area)
+	area_erased.emit(area)
 
 
 ## Should be overridden with custom behavior for rendering the [param grid].
@@ -47,11 +70,11 @@ func _render(_grid: GaeaGrid) -> void
 
 ## Should be overridden with custom behavior for erasing the rendered [param area].
 @abstract
-func _on_area_erased(_area: AABB) -> void
+func _erase_area(_area: AABB) -> void
 
 
-## Should be overridden with custom behavior to clear/_reset the previously-rendered generation.
-## Should return the _render to a 'default' state.
+## Should be overridden with custom behavior to clear/reset the previously-rendered generation.
+## Should return the render to a 'default' state.
 @abstract
 func _reset() -> void
 

addons/gaea/nodes/renderers/tilemap_renderer.gd

@@ -43,7 +43,7 @@ func _render(grid: GaeaGrid) -> void:
 				)
 
 
-func _on_area_erased(area: AABB) -> void:
+func _erase_area(area: AABB) -> void:
 	for x in range(area.position.x, area.end.x):
 		for y in range(area.position.y, area.end.y):
 			for layer in tile_map_layers:

testing/rendering/tilemap_renderer_test.gd

@@ -19,6 +19,7 @@ func before() -> void:
 func test_full_area() -> void:
 	scene.generator.generate()
 	await scene.generator.generation_finished
+	await scene.renderer.render_finished
 
 	var grass_hash: int = scene.renderer.tile_map_layers[0].get_used_cells_by_id(0, Vector2i(0, 0)).hash()
 	assert_int(grass_hash)\