There is currently no explicit testing for form redirection.

Things which should be tested

  • Basic form redirection
  • Advanced form redirection via query parameter
  • No form redirection
CommentFileSizeAuthor
#1 1261842-form_redirect.patch2.75 KBdawehner

Comments

dawehner’s picture

dawehner
Primary language German
commented
Status: Active » Needs review
Issue tags: +simpletests
StatusFileSize
new 1261842-form_redirect.patch2.75 KB

Here is a start, feedback is highly welcomed.

Status: Needs review » Needs work

The last submitted patch, 1261842-form_redirect.patch, failed testing.

dawehner’s picture

dawehner
Primary language German
commented

Oh using post might convert the value to '', so it's not false at the end.

Does someone has a better idea to test it? Sure it would be possible to add a parameter to the url, which set's the form_state

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
commented
Status: Needs work » Closed (duplicate)
Issue tags: -simpletests

I adapted these tests and merged them into #171267-16: form redirects removes get variables like sort and order, so I'm going to close this.