Skip to content

Commit 1132e39

Browse files
committed
Merge pull request #107516 from kitbdev/fix-resource-duplicate-test-errors
Fix Resource Duplication test errors
2 parents 2a71b21 + 477a37a commit 1132e39

File tree

1 file changed

+3
-8
lines changed

1 file changed

+3
-8
lines changed

tests/core/io/test_resource.h

Lines changed: 3 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -384,13 +384,8 @@ TEST_CASE("[Resource] Duplication") {
384384
INFO(std::string(String(orig->get_class_name()).utf8().get_data()));
385385

386386
orig->call("set_defaults");
387-
388387
const Ref<Resource> &dupe = p_duplicate_fn(orig);
389-
if ((p_test_mode == TEST_MODE_RESOURCE_DUPLICATE_DEEP_WITH_MODE || p_test_mode == TEST_MODE_VARIANT_DUPLICATE_DEEP_WITH_MODE) && p_deep_mode == RESOURCE_DEEP_DUPLICATE_MAX) {
390-
CHECK(dupe.is_null());
391-
} else {
392-
dupe->call("verify_duplication", orig, p_test_mode, p_deep_mode);
393-
}
388+
dupe->call("verify_duplication", orig, p_test_mode, p_deep_mode);
394389
}
395390
};
396391

@@ -414,7 +409,7 @@ TEST_CASE("[Resource] Duplication") {
414409

415410
SUBCASE("Resource::duplicate_deep()") {
416411
static int deep_mode = 0;
417-
for (deep_mode = 0; deep_mode <= RESOURCE_DEEP_DUPLICATE_MAX; deep_mode++) {
412+
for (deep_mode = 0; deep_mode < RESOURCE_DEEP_DUPLICATE_MAX; deep_mode++) {
418413
_run_test(
419414
TEST_MODE_RESOURCE_DUPLICATE_DEEP_WITH_MODE,
420415
(ResourceDeepDuplicateMode)deep_mode,
@@ -467,7 +462,7 @@ TEST_CASE("[Resource] Duplication") {
467462

468463
SUBCASE("Variant::duplicate_deep()") {
469464
static int deep_mode = 0;
470-
for (deep_mode = 0; deep_mode <= RESOURCE_DEEP_DUPLICATE_MAX; deep_mode++) {
465+
for (deep_mode = 0; deep_mode < RESOURCE_DEEP_DUPLICATE_MAX; deep_mode++) {
471466
_run_test(
472467
TEST_MODE_VARIANT_DUPLICATE_DEEP_WITH_MODE,
473468
(ResourceDeepDuplicateMode)deep_mode,

0 commit comments

Comments
 (0)