Use ID to match DB collections with content provider collections

[sunkup]

Purpose

We are currently still using the URL which is error prone. It might change and break the link.

Short description

• Update ical4android
• Syncer: Use ID to match DB collections with content provider collections
• LocalCollections: Save the database collection ID in content provider collections instead of the resource URL
• LocalCollectionsStores: Adapt accordingly
• Adapt tests

See also https://github.com/bitfireAT/davx5/issues/603#issuecomment-2314844880

Checklist

• The PR has a proper title, description and label.
• I have self-reviewed the PR.
• I have added documentation to complex functions and functions that can be used by other modules.
• I have added reasonable tests or consciously decided to not add tests.

[sunkup]

@rfc2822 Test testMigrate_AddressBook_InvalidCollection fails only in CI again. I would guess it's also due to #1273 but I am not sure.

[rfc2822]

@rfc2822 Test testMigrate_AddressBook_InvalidCollection fails only in CI again. I would guess it's also due to #1273 but I am not sure.

Hm the error message looks different at least

 at.bitfire.davdroid.settings.migration.AccountSettingsMigration18Test > testMigrate_AddressBook_InvalidCollection[virtual] FAILED 
	java.lang.AssertionError: Verification failed: call 1 of 1: AccountManager(object AccountManager).setUserData(eq(Account {name=Address Book, type=at.bitfire.davdroid.address_book}), any(), any())) should not be called

[sunkup]

Yes, and it happened after I removed the exception throw.

[rfc2822]

Yes, and it happened after I removed the exception throw.

Guess it's then more related to that. Maybe the migration test has to be adapted

[sunkup]

Yes, most likely. After reverting the commit the tests succeed again. This is weird though. It should not have any effect on the test 🤔

[sunkup]

Now it succeeded. No changes whatsoever 👍

[rfc2822]

Looks good :)