First test of cursor getType()

[brody4hire]

I made pull request 80 on android-database-sqlite to get the type of retrieved data. Here is a small test to verify the change is working.

[developernotes]

Hi brodyspark,

Thanks for your interest in the SQLCipher for Android test suite. The test you modified is testing a specific scenario, one that is different than checking the column type from a Cursor. We would welcome an additional test, however we would like to to exist on its own as an individual test. Could you extract your test from this change and add it to the test suite runner? Thanks!

[developernotes]

Hi brodyspark,

This pull request appears to be targeting a newer version of the Cursor, specifically the call to getType(...) which was added at API level 11. Do you have a pull request that adds the getType(...) function support to the SQLiteCursor class within the SQLCipher for Android project?

[brody4hire]

"I made pull request 80 on android-database-sqlite" - sqlcipher/android-database-sqlcipher#80

Also: I needed the changes in sqlcipher/android-database-sqlcipher#79 to build on OSX, if I installed Android SDK & NDK using Homebrew.

[developernotes]

Hi brodyspark,

Thanks - I am just reviewing those now. I think we will still want to split the test out a a separate item.

[brody4hire]

Cool. I just entered a couple more pull requests for your review.

On Wed, Jan 2, 2013 at 4:13 PM, Nick Parker [email protected]:

Hi brodyspark,

Thanks - I am just reviewing those now. I think we will still want to
split the test out a a separate item.

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/1#issuecomment-11811053.

[developernotes]

Hi brodyspark,

We use this test suite to verify support from platform version 2.1-4.1 currently, running across different emulators. Due to the pull requests you've made, not all code is applicable across versions (e.g., Cursor.getType(...) was added in API 11). These tests would then cause build failures for lower versions that do not support that. While we appreciate the effort, we are going to hold off on merging the tests for now. Thanks!