Skip to content
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

Jump to bottom

Rework FieldListUpdateTest #6143

Closed
grzesiek2010 opened this issue May 21, 2024 · 0 comments · Fixed by #6400
Closed

Rework FieldListUpdateTest #6143

grzesiek2010 opened this issue May 21, 2024 · 0 comments · Fixed by #6400
Assignees
@grzesiek2010
Milestone
v2024.4

Comments

@grzesiek2010
Copy link
Member

Problem description

According to the conversation in #6103 (comment) FieldListUpdateTest should be reworked so that it uses FormEntryActivityTestRule instead of BlankFormTestRule.
The newly added test changeInValueUsedToDetermineIfAQuestionIsRequired_ShouldUpdateTheRelatedRequiredQuestion should use it's own small form.
@github-project-automation github-project-automation bot moved this to not ready in ODK Collect May 21, 2024
@grzesiek2010 grzesiek2010 mentioned this issue May 21, 2024
Merged
5 tasks
@seadowg seadowg added this to the v2024.2 milestone May 21, 2024
@seadowg seadowg moved this from not ready to ready in ODK Collect May 21, 2024
@seadowg seadowg modified the milestones: v2024.2, v2024.3 Jun 10, 2024
@grzesiek2010 grzesiek2010 self-assigned this Sep 5, 2024
@grzesiek2010 grzesiek2010 moved this from ready to in progress in ODK Collect Sep 5, 2024
@grzesiek2010 grzesiek2010 mentioned this issue Sep 6, 2024
Merged
6 tasks
@seadowg seadowg modified the milestones: v2024.3, v2024.4 Oct 23, 2024
@github-project-automation github-project-automation bot moved this from in progress to done in ODK Collect Oct 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@grzesiek2010 grzesiek2010

Labels
None yet
Projects
ODK Collect
Status: done
Milestone
v2024.4
Development

Successfully merging a pull request may close this issue.

Reworked FieldListUpdateTest grzesiek2010/collect
2 participants
@seadowg @grzesiek2010