-
Notifications
You must be signed in to change notification settings - Fork 87
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update to Core next-major #1464
Conversation
final managedMap = accessor(testObject); | ||
await runManagedNotificationTests(testData, managedMap, testObject); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These were accidentally committed as commented out.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good.
I'm wondering about the timing? Will we be releasing before next-major is deemed stable? If so, we should probably wait with merging this a bit.
next-major in Core should be released soon, though I'm just putting this as a baseline on top of which I'll be implementing collections in mixed. I don't think we need to merge this until collections in mixed is ready. |
* Initial stab at collections in mixed * regenerate ffi bindings * Add more tests * Add notification tests * Return the types * Address some PR feedback * Add RealmValueType * Fix some tests * Add query tests * Add tests for indexOf/contains; add tests for sets * Create a symlink for test data inside tests * Add a few more tests * Correct Core issue link * enable notifications tests * Enable some tests * Re-generate ffi * Fix test * Added a changelog entry
Pull Request Test Coverage Report for Build 7742423493
💛 - Coveralls |
This will be the base PR on top of which collections in mixed will be based.