Signal-android: ConversationAdapterTest.testGetItemIdEquals fails

Created on 29 May 2018  路  1Comment  路  Source: signalapp/Signal-Android


Bug description

Building from the latest master branch (as of today), the following test fails with a NullPointerException: ConversationAdapterTest. testGetItemIdEquals

java.lang.NullPointerException
    at net.sqlcipher.database.SQLiteDatabase.loadLibs(SQLiteDatabase.java:203)
    at org.thoughtcrime.securesms.database.DatabaseFactory.<init>(DatabaseFactory.java:148)
    at org.thoughtcrime.securesms.database.DatabaseFactory.getInstance(DatabaseFactory.java:64)
    at org.thoughtcrime.securesms.database.DatabaseFactory.getAttachmentDatabase(DatabaseFactory.java:87)
    at org.thoughtcrime.securesms.ConversationAdapter.getItemId(ConversationAdapter.java:272)
    at org.thoughtcrime.securesms.ConversationAdapterTest.testGetItemIdEquals(ConversationAdapterTest.java:29)
[... many more lines ...]

There are several issues open already about this, the latest one being for example #7742, and according to these closed tickets, the issue is supposed to be solved. Of course it can be "circumvented" by skipping the tests (e.g. doing gradle assemblerelease) but that's besides the point of tests, is it not?

Steps to reproduce

  • Build according to the Wiki instructions (https://github.com/signalapp/Signal-Android/wiki/How-to-build-Signal-from-the-sources), the test fails as one of the steps of gradle build.
    Building on Manjaro with latest Android SDK (from Android Studio) for Linux.

Expected results: All tests should pass successfully.

>All comments

It's an open wiki; everything on there is "community contributed" and not official. You could update the wiki to replace "build" with "assembleDebug" or "assembleRelease" if you'd like. Thanks!

Was this page helpful?
0 / 5 - 0 ratings

Related issues

FeuRenard picture FeuRenard  路  3Comments

derWalter picture derWalter  路  3Comments

gmsl360 picture gmsl360  路  3Comments

notthematrix picture notthematrix  路  3Comments

hiredgunhouse picture hiredgunhouse  路  3Comments