fix: ♻️ ModID in dependencies and incompatibilities

[Blade67]

Adds validation function validate_dependencies_and_incompatibilities_conflicts() to mod_manifest.gd

Validates a mod's dependencies and incompatibilities to ensure they don't conflict.

closes #169

[Qubus0]

good so far, pls test

[KANAjetzt]

I actually generated some unit tests for this 👀
Just to see if it works.

Also tested it "by hand" and works fine.

Tests

extends "res://addons/gut/test.gd"


func test_validate_dependencies_and_incompatibilities_conflicts():
	# Test when there are no overlaps between dependencies and incompatibilities
	var mod_id = "test-mod"
	var dependencies = PoolStringArray(["mod1-mod1", "mod2-mod2"])
	var incompatibilities = PoolStringArray(["mod3-mod3", "mod4-mod4"])
	var result = ModManifest.validate_dependencies_and_incompatibilities_conflicts(mod_id, dependencies, incompatibilities)
	assert_eq(result, false)

	# Test when there is an overlap between dependencies and incompatibilities
	mod_id = "test-mod2"
	dependencies = PoolStringArray(["mod1-mod1", "mod2-mod2", "mod3-mod3"])
	incompatibilities = PoolStringArray(["mod3-mod3", "mod4-mod4"])
	result = ModManifest.validate_dependencies_and_incompatibilities_conflicts(mod_id, dependencies, incompatibilities, true)
	assert_eq(result, true)

	# Test when the function is called with no incompatibilities
	mod_id = "test-mod3"
	dependencies = PoolStringArray(["mod1-mod1", "mod2-mod2", "mod3-mod3"])
	incompatibilities = PoolStringArray([])
	result = ModManifest.validate_dependencies_and_incompatibilities_conflicts(mod_id, dependencies, incompatibilities)
	assert_eq(result, false)

	# Test when the function is called with no dependencies
	mod_id = "test-mod4"
	dependencies = PoolStringArray([])
	incompatibilities = PoolStringArray(["mod1-mod1", "mod2-mod2"])
	result = ModManifest.validate_dependencies_and_incompatibilities_conflicts(mod_id, dependencies, incompatibilities)
	assert_eq(result, false)

[Qubus0]

nice, thanks. not sure if we need it but technically, an incompatibility should also not be an optional dependency. might not be in the scope here since the optional ones were added after this issue