added tests

updated readme and changelog
fixed erroneous behavior
updated version to stable

Author: ologe

CHANGELOG.md

@@ -1,3 +1,15 @@
 # Release notes
+
+## 1.0.0
+- Fixed crashed
+- Now is **mandatory** to specify `ORDER BY` when using `LIMIT`
+- Now is **mandatory** to specify `WHERE` when using `GROUP BY`
+- Now throws an exception when using `HAVING` without `GROUP BY`
+- Now throws an exception when using `OFFSET` without `LIMIT`
+- Added tests 
+
+## 1.0.0-beta02
+- Renamed `contentResolver.queryParser(..)` to `contentResolver.querySql(..)` for better discoverability 
+
 ## 1.0.0-beta01
-- first release
+- First release

README.md

@@ -18,6 +18,10 @@ Allows to write SQL statements instead of using `contentResolver.query(...)`.
 The library add an extension function to ContentResolver named 
 `querySql(query: String, selectionArgs: Array<String>? = null)`
 
+###Limitations
+When using `LIMIT` keyword, you need to specify also `ORDER BY`.<p>
+When using `GROUP BY` keyword, you need to specify also  `WHERE`.
+
 ##Getting started
 Step 1. Add the JitPack repository to your build file
 Add it in your root build.gradle at the end of repositories:

app/src/androidTest/java/dev/olog/contentresolversql/example/ExampleInstrumentedTest.kt

@@ -1,5 +1,6 @@
 package dev.olog.contentresolversql.example
 
+import android.provider.MediaStore
 import androidx.test.InstrumentationRegistry
 import androidx.test.runner.AndroidJUnit4
 
@@ -21,4 +22,17 @@ class ExampleInstrumentedTest {
         val appContext = InstrumentationRegistry.getTargetContext()
         assertEquals("dev.olog.contentresolversql", appContext.packageName)
     }
+
+    @Test
+    fun assertSimpleQuery(){
+        val appContext = InstrumentationRegistry.getTargetContext()
+
+
+
+        var cursor = appContext.contentResolver.query(
+            MediaStore.Audio.Media.EXTERNAL_CONTENT_URI,
+            null, null, null, null
+        )
+    }
+
 }

app/src/main/java/dev/olog/contentresolversql/example/MainActivity.kt

@@ -1,6 +1,5 @@
 package dev.olog.contentresolversql.example
 
-import android.net.Uri
 import android.os.Bundle
 import android.provider.MediaStore
 import android.provider.MediaStore.Audio.AlbumColumns.ARTIST
@@ -14,6 +13,7 @@ class MainActivity : AppCompatActivity() {
         super.onCreate(savedInstanceState)
         setContentView(R.layout.activity_main)
 
+        var start = System.currentTimeMillis()
         var query = """
             SELECT *
             FROM ${MediaStore.Audio.Media.EXTERNAL_CONTENT_URI}
@@ -22,6 +22,9 @@ class MainActivity : AppCompatActivity() {
         var cursor = contentResolver.querySql(query)
         cursor.close()
 
+        val time1 = System.currentTimeMillis() - start
+        start = System.currentTimeMillis()
+
         query = """
             SELECT distinct $ARTIST_ID, $ARTIST, count(*) as songs, count(distinct $ALBUM_ID) as albums
             FROM ${MediaStore.Audio.Media.EXTERNAL_CONTENT_URI}
@@ -33,7 +36,6 @@ class MainActivity : AppCompatActivity() {
             OFFSET 2
         """.trimIndent()
 
-        contentResolver.query(Uri.parse(""), null, null, null, null)
         cursor = contentResolver.querySql(query)
         val result = mutableListOf<Artist>()
         while (cursor.moveToNext()){
@@ -46,6 +48,8 @@ class MainActivity : AppCompatActivity() {
             result.add(item)
         }
 
+        val time2 = System.currentTimeMillis() - start
+
         cursor.close()
     }
 }

library/build.gradle

@@ -8,8 +8,8 @@ android {
         minSdkVersion 16
         targetSdkVersion 28
         // sdk_version, major, minor, fixes
-        versionCode 28_1_0_02
-        versionName "1.0-beta02"
+        versionCode 28_1_0_10
+        versionName "1.0"
 
         testInstrumentationRunner "androidx.test.runner.AndroidJUnitRunner"