Louvorg/ReaderForSelfoss-multiplatform
Louvorg/ReaderForSelfoss-multiplatform
3
2
Fork 2
Code Issues 16 Pull Requests Actions Projects 1 Releases 68 Wiki Activity

Repository Unit Tests #50

Merged
AmineB merged 38 commits from davidoskky/ReaderForSelfoss-multiplatform:repository_tests into master 2022-09-30 11:31:55 +00:00
Conversation 7 Commits 38 Files Changed 7 +2 -3
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 79fd115f5e - Show all commits

5
shared/src/commonMain/kotlin/bou/amine/apps/readerforselfossv2/repository/RepositoryImpl.kt
View File

@ -457,9 +457,8 @@ class Repository(private val api: SelfossApi, private val appSettingsService: Ap
val newItems = getMaxItemsForBackground(ItemType.UNREAD)
val allItems = getMaxItemsForBackground(ItemType.ALL)
AmineB marked this conversation as resolved Outdated
AmineB commented 2022-09-30 09:44:53 +00:00
Outdated
Review
Copy Link

I think that the DB handles the duplicates.

I think that the DB handles the duplicates.
davidoskky commented 2022-09-30 09:51:56 +00:00
Outdated
Review
Copy Link

Does it use the id to check for duplicates? If that is the case then all is good.

Does it use the id to check for duplicates? If that is the case then all is good.
val starredItems = getMaxItemsForBackground(ItemType.STARRED)
val fullItemsList = newItems + allItems + starredItems
insertDBItems(fullItemsList)
return fullItemsList
insertDBItems(newItems + allItems + starredItems)
return newItems
AmineB marked this conversation as resolved Outdated
AmineB commented 2022-09-30 09:38:26 +00:00
Outdated
Review
Copy Link

This is useless.

This is useless.
} catch (e: Throwable) {
// We do nothing
}
AmineB commented 2022-09-30 10:00:09 +00:00
Outdated
Review
Copy Link

As the function name describes, this must only return the new items.

As the function name describes, this must only return the new items.
davidoskky commented 2022-09-30 11:13:32 +00:00
Outdated
Review
Copy Link

Ops, you're right I made a mistake while changing things...

Ops, you're right I made a mistake while changing things...