Issue with building keypath in C API

[papafe]

Considering the method build_key_path_array, it seems it does not allow to create an empty KeyPathArray:

realm-core/src/realm/object-store/c_api/notifications.cpp

Lines 67 to 74 in d318021

	std::optional<KeyPathArray> build_key_path_array(realm_key_path_array_t* key_path_array)
	{
	std::optional<KeyPathArray> ret;
	if (key_path_array && key_path_array->size()) {
	ret.emplace(std::move(*key_path_array));
	}
	return ret;
	}

The method returns std::nullopt in two scenarios:

1. When key_path_array is nullptr.
2. When key_path_array points to an array with a length of zero.

But in case 2 it should return an empty KeyPathArray.

[sync-by-unito]

➤ PM Bot commented:

Jira ticket: RCORE-2167

[kiburtse]

ok, so It's already so in c++ api, but it looks like there was some back and forth with it in #6122 and then #7087. @jedelbo @nicola-cab was it just an oversight due to missing test cases?