From a797e43178022d5bc33487a1b028f2301496f3b4 Mon Sep 17 00:00:00 2001
From: Peter Serwylo <peter@ivt.com.au>
Date: Wed, 19 Feb 2014 09:32:54 +1100
Subject: [PATCH] Test apk insert more comprehensivly.

---
 .../org/fdroid/fdroid/ApkProviderTest.java    | 31 ++++++++++++++++++-
 1 file changed, 30 insertions(+), 1 deletion(-)

diff --git a/test/src/org/fdroid/fdroid/ApkProviderTest.java b/test/src/org/fdroid/fdroid/ApkProviderTest.java
index 0785a87db..4c808a066 100644
--- a/test/src/org/fdroid/fdroid/ApkProviderTest.java
+++ b/test/src/org/fdroid/fdroid/ApkProviderTest.java
@@ -214,8 +214,37 @@ public class ApkProviderTest extends FDroidProviderTest<ApkProvider> {
             assertResultCount(0, queryAllApks());
         }
 
-        // ApkProvider.DataColumns.REPO
+        ContentValues values = new ContentValues();
+        values.put(ApkProvider.DataColumns.REPO_ID, 10);
+        values.put(ApkProvider.DataColumns.REPO_ADDRESS, "http://example.com");
+        values.put(ApkProvider.DataColumns.REPO_VERSION, 3);
+        values.put(ApkProvider.DataColumns.FEATURES, "Some features");
+        Uri uri = insertApk("com.example.com", 1, values);
 
+        assertResultCount(1, queryAllApks());
+
+        String[] projections = {
+            ApkProvider.DataColumns.REPO_ID,
+            ApkProvider.DataColumns.REPO_ADDRESS,
+            ApkProvider.DataColumns.REPO_VERSION,
+            ApkProvider.DataColumns.FEATURES,
+            ApkProvider.DataColumns.APK_ID,
+            ApkProvider.DataColumns.VERSION_CODE
+        };
+        
+        Cursor cursor = getMockContentResolver().query(uri, projections, null, null, null);
+        cursor.moveToFirst();
+        Apk apk = new Apk(cursor);
+
+        // These should have quietly been dropped when we tried to save them...
+        assertEquals(null, apk.repoAddress);
+        assertEquals(0, apk.repoVersion);
+
+        // But this should have saved correctly...
+        assertEquals("Some features", apk.features.toString());
+        assertEquals("com.example.com", apk.id);
+        assertEquals(1, apk.vercode);
+        assertEquals(10, apk.repo);
     }
 
     public void testIgnore() {