Add processors tests

98teg · 98teg · commit de75867834b7 · 2020-09-06T11:53:45.000+02:00
diff --git a/main.gd b/main.gd
@@ -19,3 +19,4 @@ func _ready():
 	preload("res://test/test_suites/file_tests.gd").execute(tests_results)
 	preload("res://test/test_suites/json_config_file_tests.gd").execute(tests_results)
 	preload("res://test/test_suites/image_tests.gd").execute(tests_results)
+	preload("res://test/test_suites/processor_tests.gd").execute(tests_results)
diff --git a/project.godot b/project.godot
@@ -25,6 +25,11 @@ _global_script_classes=[ {
 "path": "res://src/json_config_file/json_config_file.gd"
 }, {
 "base": "Reference",
+"class": "JSONConfigProcessor",
+"language": "GDScript",
+"path": "res://src/json_config_file/json_config_processor.gd"
+}, {
+"base": "Reference",
 "class": "JSONProperty",
 "language": "GDScript",
 "path": "res://src/json_config_file/json_properties/json_property.gd"
@@ -98,6 +103,7 @@ _global_script_class_icons={
 "Case": "",
 "Comparator": "",
 "JSONConfigFile": "",
+"JSONConfigProcessor": "",
 "JSONProperty": "",
 "JSONPropertyArray": "",
 "JSONPropertyBool": "",
diff --git a/src b/src
@@ -1 +1 @@
-Subproject commit 43b85d2208b73a8ea70a3b54a0813d28a5a716d4
+Subproject commit 57d206b46015aa01b2e91372bd864b5571aa320c
diff --git a/test/test_files/processor/add_id_processor.gd b/test/test_files/processor/add_id_processor.gd
@@ -0,0 +1,13 @@
+extends JSONConfigProcessor
+
+
+func _process(dic : Dictionary):
+	if not has_variable("id"):
+		dic.id = 0
+		set_variable("id", 0)
+	else:
+		var id = get_variable("id") + 1
+		dic.id = id
+		set_variable("id", id)
+
+	return dic
diff --git a/test/test_files/processor/add_id_processor.json b/test/test_files/processor/add_id_processor.json
@@ -0,0 +1,13 @@
+{
+    "objects": [
+        {
+            "name": "table"
+        },
+        {
+            "name": "lamp"
+        },
+        {
+            "name": "chair"
+        }
+    ]
+}
diff --git a/test/test_files/processor/inside_of_the_range_value.json b/test/test_files/processor/inside_of_the_range_value.json
@@ -0,0 +1,5 @@
+{
+    "min": 0,
+    "max": 3,
+    "a_value": 2
+}
diff --git a/test/test_files/processor/out_of_range_value.json b/test/test_files/processor/out_of_range_value.json
@@ -0,0 +1,5 @@
+{
+    "min": 0,
+    "max": 3,
+    "a_value": 4
+}
diff --git a/test/test_files/processor/set_max_processor.gd b/test/test_files/processor/set_max_processor.gd
@@ -0,0 +1,7 @@
+extends JSONConfigProcessor
+
+
+func _process(maximum : int):
+	set_variable("max", maximum)
+
+	return maximum
diff --git a/test/test_files/processor/set_min_processor.gd b/test/test_files/processor/set_min_processor.gd
@@ -0,0 +1,7 @@
+extends JSONConfigProcessor
+
+
+func _process(minimum : int):
+	set_variable("min", minimum)
+
+	return minimum
diff --git a/test/test_files/processor/set_range_processor.gd b/test/test_files/processor/set_range_processor.gd
@@ -0,0 +1,8 @@
+extends JSONConfigProcessor
+
+
+func _process(_value):
+	if has_variable("min"):
+		get_property().set_min_value(get_variable("min"))
+	if has_variable("max"):
+		get_property().set_max_value(get_variable("max"))
diff --git a/test/test_suites/processor_tests.gd b/test/test_suites/processor_tests.gd
@@ -0,0 +1,111 @@
+static func execute(tests_results):
+	var min_value = JSONPropertyInteger.new()
+	min_value.set_postprocessor(preload("res://test/test_files/processor/set_min_processor.gd").new())
+
+	var max_value = JSONPropertyInteger.new()
+	max_value.set_postprocessor(preload("res://test/test_files/processor/set_max_processor.gd").new())
+
+	var a_value = JSONPropertyInteger.new()
+	a_value.set_preprocessor(preload("res://test/test_files/processor/set_range_processor.gd").new())
+
+	var configuration = JSONConfigFile.new()
+	configuration.add_property("min", min_value)
+	configuration.add_property("max", max_value)
+	configuration.add_property("a_value", a_value)
+
+	var case = Case.new()
+	case.set_name("Preprocessing the value adds some restrictions")
+	case.set_configuration(configuration)
+
+	var test
+
+	test = Test.new()
+	test.set_name("The value is inside of the range")
+	test.set_file_path("test/test_files/processor/inside_of_the_range_value.json")
+	test.set_expected_result({
+		"min": 0,
+		"max": 3,
+		"a_value": 2,
+	})
+
+	case.add_test(test)
+
+	test = Test.new()
+	test.set_name("The value is outside of the range")
+	test.set_file_path("test/test_files/processor/out_of_range_value.json")
+	test.set_expected_result({
+		"min": 0,
+		"max": 3,
+		"a_value": null,
+	})
+	test.add_expected_error({
+		"error": JSONProperty.Errors.NUMBER_VALUE_MORE_THAN_MAX,
+		"max": 3,
+		"value": 4,
+		"context": "a_value",
+	})
+
+	case.add_test(test)
+
+	case.check_tests(tests_results)
+
+	var element = JSONPropertyObject.new()
+	element.add_property("name", JSONPropertyString.new())
+
+	element.set_postprocessor(preload("res://test/test_files/processor/add_id_processor.gd").new())
+
+	var property = JSONPropertyArray.new()
+	property.set_element_property(element)
+
+	configuration = JSONConfigFile.new()
+	configuration.add_property("objects", property)
+
+	case = Case.new()
+	case.set_name("Postprocessing the result adds an id to the objects")
+	case.set_configuration(configuration)
+
+	test = Test.new()
+	test.set_name("The id is added")
+	test.set_file_path("test/test_files/processor/add_id_processor.json")
+	test.set_expected_result({
+		"objects": [
+			{
+				"id": 0,
+				"name": "table",
+			},
+			{
+				"id": 1,
+				"name": "lamp",
+			},
+			{
+				"id": 2,
+				"name": "chair",
+			},
+		]
+	})
+
+	case.add_test(test)
+
+	test = Test.new()
+	test.set_name("The id is added again, this tests if the variables reset")
+	test.set_file_path("test/test_files/processor/add_id_processor.json")
+	test.set_expected_result({
+		"objects": [
+			{
+				"id": 0,
+				"name": "table",
+			},
+			{
+				"id": 1,
+				"name": "lamp",
+			},
+			{
+				"id": 2,
+				"name": "chair",
+			},
+		]
+	})
+
+	case.add_test(test)
+
+	case.check_tests(tests_results)
