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

test #95

Closed
wants to merge 18 commits into from
Closed

test #95

wants to merge 18 commits into from

Conversation

alazier
Copy link
Contributor

@alazier alazier commented Oct 24, 2015

test

appden and others added 18 commits October 21, 2015 13:11
Rather than using Object.preventExtensions to throw exceptions when setting any out-of-bounds index, we must instead compromise by only throwing for the common cases, which are -1 and list.length. Having Object.keys() support is essential for List to work with React Native ListViewDataSource.

Resolves #80
React Native now leaves the original object accessible when it "polyfills" objects. So we check for that, but still have the previous logic in place in case they change their minds.
React Native 0.13.0 fixes the bug we experienced by using const.
Test failures were not pointing to the correct JS file path when it was in the src directory. This is where it belongs anyways.
This helps the editor catch simple syntax issues before finding out about them when running a test.
Added some tests. Turns out our Chrome layer already did this too, so no changes were needed there.

Resolves #70
Make ReactTests runnable from RealmJS project
Update version/Fix cordova example
@alazier alazier closed this Oct 26, 2015
@alazier alazier deleted the al-old branch October 26, 2015 19:45
alazier pushed a commit that referenced this pull request Sep 14, 2016
* Bump version

* update version
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 17, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants