Skip to content

Don't copy existing media files when updating media only #6946

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
grzesiek2010 merged 19 commits into from
Dec 2, 2025
Merged

Don't copy existing media files when updating media only #6946

Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
f8996ca
Remove test replaced by FormDownloadActionTest
seadowg Oct 23, 2025
bf57e78
Remove unneeded open
seadowg Oct 23, 2025
b797772
Move ServerFormDetailsFetcher logic to use case
seadowg Oct 23, 2025
36560a9
Isolate authentication overriding to where its used
seadowg Oct 23, 2025
6ab2581
Convert ServerFormDetailsFetcher to interface
seadowg Oct 23, 2025
aaf058b
Only use interface for Java code
seadowg Oct 24, 2025
6e09b31
Separate out tests for ServerFormUseCases
seadowg Oct 24, 2025
98fd70d
Remove unused annotation
seadowg Oct 24, 2025
acb89bf
Add enum to replace booleans
seadowg Oct 24, 2025
f7de496
Convert boolean to proxies for type
seadowg Oct 24, 2025
a224e46
Remove isNotOnDevice
seadowg Oct 24, 2025
17cfc35
Replace isUpdated property with helper
seadowg Oct 24, 2025
ad16c3d
Change target for benchmark
seadowg Oct 27, 2025
d71bfcc
Optimize updating media
seadowg Oct 27, 2025
7643da9
Add clarifying comment
seadowg Oct 27, 2025
f9db6d5
Remove unneeded cache clear
seadowg Oct 27, 2025
73f7eb7
Remove unused variable
seadowg Nov 26, 2025
ee6de92
Fix comment
seadowg Nov 26, 2025
493711e
Update test names
seadowg Nov 26, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Remove unneeded cache clear
  • Loading branch information
@seadowg
seadowg committed Nov 26, 2025
commit f9db6d5c75b231a624cb07b50ad41043d2ccef12
9 changes: 0 additions & 9 deletions collect_app/src/androidTest/java/org/odk/collect/android/benchmark/EntitiesBenchmarkTest.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
package org.odk.collect.android.benchmark

import android.app.Application
import androidx.test.core.app.ApplicationProvider
import androidx.test.ext.junit.runners.AndroidJUnit4
import org.hamcrest.MatcherAssert.assertThat
import org.hamcrest.Matchers.blankOrNullString
Expand Down Expand Up @@ -44,7 +42,6 @@ class EntitiesBenchmarkTest {
ENTITIES_FILTER_PROJECT_URL,
not(blankOrNullString())
)
clearAndroidCache()

val benchmarker = Benchmarker()

Expand Down Expand Up @@ -142,9 +139,3 @@ class EntitiesBenchmarkTest {
benchmarker.assertResults()
}
}

private fun clearAndroidCache() {
val application = ApplicationProvider.getApplicationContext<Application>()
application.cacheDir.deleteRecursively()
application.cacheDir.mkdir()
}