We should have basic test coverage for File Browser, covering the basic flow of selecting existing files in the view widget.

CommentFileSizeAuthor
#7 file-browser-test-coverage-2872175-7.patch4.78 KBsamuel.mortenson
#5 file-browser-test-coverage-2872175-5.patch4.85 KBsamuel.mortenson
#3 file-browser-test-coverage.patch4.54 KBsamuel.mortenson
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

samuel.mortenson created an issue. See original summary.

samuel.mortenson’s picture

samuel.mortenson
he/him
Primary language English
CreditAttribution: samuel.mortenson at Acquia commented
Issue summary: View changes
samuel.mortenson’s picture

samuel.mortenson
he/him
Primary language English
CreditAttribution: samuel.mortenson at Acquia commented
Status: Active » Needs review
FileSize
file-browser-test-coverage.patch4.54 KB

Here's about as simple as it gets.

Status: Needs review » Needs work

The last submitted patch, 3: file-browser-test-coverage.patch, failed testing.

samuel.mortenson’s picture

samuel.mortenson
he/him
Primary language English
CreditAttribution: samuel.mortenson at Acquia commented
Status: Needs work » Needs review
FileSize
file-browser-test-coverage-2872175-5.patch4.85 KB

Added test dependencies.

Status: Needs review » Needs work

The last submitted patch, 5: file-browser-test-coverage-2872175-5.patch, failed testing.

samuel.mortenson’s picture

samuel.mortenson
he/him
Primary language English
CreditAttribution: samuel.mortenson at Acquia commented
Status: Needs work » Needs review
FileSize
file-browser-test-coverage-2872175-7.patch4.78 KB

Status: Needs review » Needs work

The last submitted patch, 7: file-browser-test-coverage-2872175-7.patch, failed testing.

samuel.mortenson’s picture

samuel.mortenson
he/him
Primary language English
CreditAttribution: samuel.mortenson at Acquia commented
Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.