mirror of
https://github.com/TeamNewPipe/NewPipe
synced 2024-12-23 08:30:44 +00:00
Add basic tests for settings import
This commit is contained in:
parent
19cd3a17df
commit
f778c48923
@ -1,8 +1,10 @@
|
|||||||
package org.schabi.newpipe.settings
|
package org.schabi.newpipe.settings
|
||||||
|
|
||||||
import android.content.SharedPreferences
|
import android.content.SharedPreferences
|
||||||
import com.nononsenseapps.filepicker.NewFolderFragment
|
|
||||||
import org.junit.Assert
|
import org.junit.Assert
|
||||||
|
import org.junit.Assert.assertEquals
|
||||||
|
import org.junit.Assert.assertFalse
|
||||||
|
import org.junit.Assert.assertTrue
|
||||||
import org.junit.Assume
|
import org.junit.Assume
|
||||||
import org.junit.Before
|
import org.junit.Before
|
||||||
import org.junit.BeforeClass
|
import org.junit.BeforeClass
|
||||||
@ -11,10 +13,16 @@ import org.junit.runner.RunWith
|
|||||||
import org.junit.runners.Suite
|
import org.junit.runners.Suite
|
||||||
import org.mockito.Mockito
|
import org.mockito.Mockito
|
||||||
import org.mockito.Mockito.`when`
|
import org.mockito.Mockito.`when`
|
||||||
|
import org.mockito.Mockito.anyString
|
||||||
|
import org.mockito.Mockito.anyBoolean
|
||||||
|
import org.mockito.Mockito.atLeastOnce
|
||||||
|
import org.mockito.Mockito.verify
|
||||||
|
import org.mockito.Mockito.withSettings
|
||||||
import org.mockito.junit.MockitoJUnitRunner
|
import org.mockito.junit.MockitoJUnitRunner
|
||||||
import org.schabi.newpipe.settings.ContentSettingsManagerTest.ExportTest
|
import org.schabi.newpipe.settings.ContentSettingsManagerTest.ExportTest
|
||||||
import java.io.File
|
import java.io.File
|
||||||
import java.io.ObjectInputStream
|
import java.io.ObjectInputStream
|
||||||
|
import java.nio.file.Files
|
||||||
import java.util.zip.ZipFile
|
import java.util.zip.ZipFile
|
||||||
|
|
||||||
@RunWith(Suite::class)
|
@RunWith(Suite::class)
|
||||||
@ -40,7 +48,7 @@ class ContentSettingsManagerTest {
|
|||||||
newpipeDb = File(dbPath!!)
|
newpipeDb = File(dbPath!!)
|
||||||
newpipeSettings = File(settingsPath!!)
|
newpipeSettings = File(settingsPath!!)
|
||||||
|
|
||||||
fileLocator = Mockito.mock(NewPipeFileLocator::class.java, Mockito.withSettings().stubOnly())
|
fileLocator = Mockito.mock(NewPipeFileLocator::class.java, withSettings().stubOnly())
|
||||||
`when`(fileLocator.db).thenReturn(newpipeDb)
|
`when`(fileLocator.db).thenReturn(newpipeDb)
|
||||||
`when`(fileLocator.settings).thenReturn(newpipeSettings)
|
`when`(fileLocator.settings).thenReturn(newpipeSettings)
|
||||||
}
|
}
|
||||||
@ -50,7 +58,7 @@ class ContentSettingsManagerTest {
|
|||||||
|
|
||||||
@Before
|
@Before
|
||||||
fun setupMocks() {
|
fun setupMocks() {
|
||||||
preferences = Mockito.mock(SharedPreferences::class.java, Mockito.withSettings().stubOnly())
|
preferences = Mockito.mock(SharedPreferences::class.java, withSettings().stubOnly())
|
||||||
}
|
}
|
||||||
|
|
||||||
@Test
|
@Test
|
||||||
@ -79,4 +87,100 @@ class ContentSettingsManagerTest {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@RunWith(MockitoJUnitRunner::class)
|
||||||
|
class ImportTest {
|
||||||
|
companion object {
|
||||||
|
private lateinit var fileLocator: NewPipeFileLocator
|
||||||
|
private lateinit var zip: File
|
||||||
|
private lateinit var emptyZip: File
|
||||||
|
private lateinit var db: File
|
||||||
|
private lateinit var dbJournal: File
|
||||||
|
private lateinit var dbWal: File
|
||||||
|
private lateinit var dbShm: File
|
||||||
|
private lateinit var settings: File
|
||||||
|
|
||||||
|
@JvmStatic
|
||||||
|
@BeforeClass
|
||||||
|
fun setupReadOnlyFiles() {
|
||||||
|
val zipPath = ImportTest::class.java.classLoader?.getResource("settings/newpipe.zip")?.file
|
||||||
|
val emptyZipPath = ImportTest::class.java.classLoader?.getResource("settings/empty.zip")?.file
|
||||||
|
Assume.assumeNotNull(zipPath)
|
||||||
|
Assume.assumeNotNull(emptyZipPath)
|
||||||
|
|
||||||
|
zip = File(zipPath!!)
|
||||||
|
emptyZip = File(emptyZipPath!!)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
@Before
|
||||||
|
fun setupWriteFiles() {
|
||||||
|
db = File.createTempFile("newpipe_", "")
|
||||||
|
dbJournal = File.createTempFile("newpipe_", "")
|
||||||
|
dbWal = File.createTempFile("newpipe_", "")
|
||||||
|
dbShm = File.createTempFile("newpipe_", "")
|
||||||
|
settings = File.createTempFile("newpipe_", "")
|
||||||
|
|
||||||
|
fileLocator = Mockito.mock(NewPipeFileLocator::class.java, withSettings().stubOnly())
|
||||||
|
`when`(fileLocator.db).thenReturn(db)
|
||||||
|
`when`(fileLocator.dbJournal).thenReturn(dbJournal)
|
||||||
|
`when`(fileLocator.dbShm).thenReturn(dbShm)
|
||||||
|
`when`(fileLocator.dbWal).thenReturn(dbWal)
|
||||||
|
`when`(fileLocator.settings).thenReturn(settings)
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
fun `The database must be extracted from the zip file`() {
|
||||||
|
val success = ContentSettingsManager(fileLocator).extractDb(zip.path)
|
||||||
|
|
||||||
|
assertTrue(success)
|
||||||
|
assertFalse(dbJournal.exists())
|
||||||
|
assertFalse(dbWal.exists())
|
||||||
|
assertFalse(dbShm.exists())
|
||||||
|
assertTrue("database file size is zero", Files.size(db.toPath()) > 0)
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
fun `Extracting the database from an empty zip must not work`() {
|
||||||
|
val success = ContentSettingsManager(fileLocator).extractDb(emptyZip.path)
|
||||||
|
|
||||||
|
assertFalse(success)
|
||||||
|
assertTrue(dbJournal.exists())
|
||||||
|
assertTrue(dbWal.exists())
|
||||||
|
assertTrue(dbShm.exists())
|
||||||
|
assertEquals(0, Files.size(db.toPath()))
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
fun `Contain setting must return true, if a settings file exists in the zip`() {
|
||||||
|
val contains = ContentSettingsManager(fileLocator).containSettings(zip.path)
|
||||||
|
|
||||||
|
assertTrue(contains)
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
fun `Contain setting must return false, if a no settings file exists in the zip`() {
|
||||||
|
val contains = ContentSettingsManager(fileLocator).containSettings(emptyZip.path)
|
||||||
|
|
||||||
|
assertFalse(contains)
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
fun `Preferences must be set from the settings file`() {
|
||||||
|
val preferences = Mockito.mock(SharedPreferences::class.java, withSettings().stubOnly())
|
||||||
|
val editor = Mockito.mock(SharedPreferences.Editor::class.java)
|
||||||
|
`when`(preferences.edit()).thenReturn(editor)
|
||||||
|
|
||||||
|
|
||||||
|
val manager = ContentSettingsManager(fileLocator)
|
||||||
|
manager.containSettings(zip.path)
|
||||||
|
manager.loadSharedPreferences(preferences)
|
||||||
|
|
||||||
|
verify(editor, atLeastOnce()).putBoolean(anyString(), anyBoolean())
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
}
|
}
|
||||||
|
BIN
app/src/test/resources/settings/empty.zip
Normal file
BIN
app/src/test/resources/settings/empty.zip
Normal file
Binary file not shown.
BIN
app/src/test/resources/settings/newpipe.zip
Normal file
BIN
app/src/test/resources/settings/newpipe.zip
Normal file
Binary file not shown.
Loading…
Reference in New Issue
Block a user