Skip to content

Commit 5ec1df7

Browse files
committed
Fix tests
1 parent eb716ce commit 5ec1df7

File tree

2 files changed

+36
-0
lines changed

2 files changed

+36
-0
lines changed

collect_app/src/test/java/org/odk/collect/android/widgets/ArbitraryFileWidgetTest.kt

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,8 @@ package org.odk.collect.android.widgets
33
import androidx.compose.ui.test.junit4.createAndroidComposeRule
44
import androidx.compose.ui.test.onNodeWithText
55
import androidx.compose.ui.test.performClick
6+
import org.hamcrest.CoreMatchers.equalTo
7+
import org.hamcrest.MatcherAssert.assertThat
68
import org.javarosa.core.model.Constants
79
import org.javarosa.core.model.data.StringData
810
import org.junit.Before
@@ -132,4 +134,20 @@ class ArbitraryFileWidgetTest : FileWidgetTest<ArbitraryFileWidget>() {
132134
composeRule.onNodeWithClickLabel(activity.getString(string.choose_file)).assertDoesNotExist()
133135
composeRule.onNodeWithText(initialAnswer.displayText).assertExists()
134136
}
137+
138+
@Test
139+
override fun settingANewAnswerShouldCallDeleteMediaToRemoveTheOldFile() {
140+
val file = questionMediaManager.addAnswerFile(File.createTempFile("document", ".pdf"))
141+
formEntryPrompt = MockFormEntryPromptBuilder(formEntryPrompt)
142+
.withAnswer(StringData(file.name))
143+
.build()
144+
145+
val widget = createWidget()
146+
widget.setData(createBinaryData(nextAnswer))
147+
148+
assertThat(
149+
questionMediaManager.originalFiles[formEntryPrompt.index.toString()],
150+
equalTo(file.absolutePath)
151+
)
152+
}
135153
}

collect_app/src/test/java/org/odk/collect/android/widgets/ExArbitraryFileWidgetTest.kt

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,8 @@ package org.odk.collect.android.widgets
33
import androidx.compose.ui.test.junit4.createAndroidComposeRule
44
import androidx.compose.ui.test.onNodeWithText
55
import androidx.compose.ui.test.performClick
6+
import org.hamcrest.CoreMatchers.equalTo
7+
import org.hamcrest.MatcherAssert.assertThat
68
import org.javarosa.core.model.Constants
79
import org.javarosa.core.model.data.StringData
810
import org.junit.Before
@@ -135,4 +137,20 @@ class ExArbitraryFileWidgetTest : FileWidgetTest<ExArbitraryFileWidget>() {
135137
composeRule.onNodeWithClickLabel(activity.getString(string.launch_app)).assertDoesNotExist()
136138
composeRule.onNodeWithText(initialAnswer.displayText).assertExists()
137139
}
140+
141+
@Test
142+
override fun settingANewAnswerShouldCallDeleteMediaToRemoveTheOldFile() {
143+
val file = questionMediaManager.addAnswerFile(File.createTempFile("document", ".pdf"))
144+
formEntryPrompt = MockFormEntryPromptBuilder(formEntryPrompt)
145+
.withAnswer(StringData(file.name))
146+
.build()
147+
148+
val widget = createWidget()
149+
widget.setData(createBinaryData(nextAnswer))
150+
151+
assertThat(
152+
questionMediaManager.originalFiles[formEntryPrompt.index.toString()],
153+
equalTo(file.absolutePath)
154+
)
155+
}
138156
}

0 commit comments

Comments
 (0)