refactor(ffi): Drop support for observing AccountDataEventType::SecretStorageKey

[zecakeh]

This type cannot be observed because it is a dynamic event type and this branch will fail to compile with the next version of Ruma.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.76%. Comparing base (a9ffe5f) to head (e26c4e9).
Report is 28 commits behind head on main.

✅ All tests successful. No failed tests found.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #5326   +/-   ##
=======================================
  Coverage   88.76%   88.76%           
=======================================
  Files         333      333           
  Lines       90228    90228           
  Branches    90228    90228           
=======================================
+ Hits        80088    80091    +3     
+ Misses       6315     6312    -3     
  Partials     3825     3825           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[zecakeh]

Note that we are adding stuff to Ruma that should make this possible in the future (ruma/ruma#2136), although the event handler code will need to be adapted in the SDK.

So maybe it's better not to touch the function signature and panic for now?

[zecakeh]

Actually what we have in ruma/ruma#2136 is probably not the right answer. We might need to do another breaking change that will require changes to the event handler code and fix this.

[zecakeh]

There is a fix for this in #5337 so we might not need this. We could make #5337 smaller by merging a version of this (since the current code doesn't work anyway), and then applying the fix in another PR after #5337.

[zecakeh]

Actually we found a way where we don't need to fix anything until after Ruma is bumped.