Skip to content
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

Closed
wants to merge 5 commits into from
Closed

Update to Core next-major #1464

wants to merge 5 commits into from

Conversation

nirinchev
Copy link
Member

This will be the base PR on top of which collections in mixed will be based.

Comment on lines +633 to +634
final managedMap = accessor(testObject);
await runManagedNotificationTests(testData, managedMap, testObject);
Copy link
Member Author

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.

Copy link
Contributor

@nielsenko nielsenko left a 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.

@nirinchev
Copy link
Member Author

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
@cla-bot cla-bot bot added the cla: yes label Feb 1, 2024
Copy link

Pull Request Test Coverage Report for Build 7742423493

  • -18 of 186 (90.32%) changed or added relevant lines in 9 files are covered.
  • 3 unchanged lines in 3 files lost coverage.
  • Overall coverage increased (+1.2%) to 88.568%

Changes Missing Coverage Covered Lines Changed/Added Lines %
lib/src/map.dart 14 16 87.5%
lib/src/results.dart 3 5 60.0%
lib/src/list.dart 15 19 78.95%
lib/src/native/realm_core.dart 76 86 88.37%
Files with Coverage Reduction New Missed Lines %
lib/src/realm_class.dart 1 94.43%
lib/src/realm_object.dart 1 86.33%
lib/src/set.dart 1 85.0%
Totals Coverage Status
Change from base Build 7741947640: 1.2%
Covered Lines: 3897
Relevant Lines: 4400

💛 - Coveralls

@nirinchev nirinchev closed this Feb 23, 2024
@nirinchev nirinchev deleted the ni/core-vnext branch February 23, 2024 13:35
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 24, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants