IssueTracker

Component used: Room
Version used:1.1.1
Devices/Android versions reproduced on: All

File: FrameworkSQLiteOpenHelper

        FrameworkSQLiteDatabase getWrappedDb(SQLiteDatabase sqLiteDatabase) {
            FrameworkSQLiteDatabase dbRef = mDbRef[0];
            if (dbRef == null) {
                dbRef = new FrameworkSQLiteDatabase(sqLiteDatabase);
                mDbRef[0] = dbRef;
            }
            return mDbRef[0];
        }

Room always preserve The First Open db, even if it is closed.
Sometimes sqLiteDatabase != mDbRef[0].mDelegate, we get the wrong db from room.
This occurs when opening db, then something wrong and throws exception, but app caught it. The next time we use room, it throws re-open exception.

I thinks mDbRef[0].mDelegate should change when sqLiteDatabase is changed.
Like this:

        FrameworkSQLiteDatabase getWrappedDb(SQLiteDatabase sqLiteDatabase) {
            FrameworkSQLiteDatabase dbRef = mDbRef[0];
            if (dbRef == null || dbRef.isDatabaseChanged(sqLiteDatabase)) {
                dbRef = new FrameworkSQLiteDatabase(sqLiteDatabase);
                mDbRef[0] = dbRef;
            }
            return mDbRef[0];
        }

File: FrameworkSQLiteDatabase
    public boolean isDatabaseChanged(SQLiteDatabase db) {
        return mDelegate != db;
    }