1. Move all tests to the tests/ subfolder.
  2. Split out UI tests from functionality tests
  3. Ensure only one class is in each test file (ease of transition to D8 when the time comes) and better practice for autoloaders.
CommentFileSizeAuthor
#9 reorganize_tests-2629986-9.patch129.24 KBjoelpittet
#7 interdiff.txt1.55 KBjoelpittet
#7 reorganize_tests-2629986-7.patch129.25 KBjoelpittet
#4 interdiff.txt891 bytesjoelpittet
#4 reorganize_tests-2629986-4.patch129.25 KBjoelpittet
#2 reorganize_tests-2629986-2.patch129.24 KBjoelpittet

Comments

joelpittet created an issue. See original summary.

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Issue summary: View changes
Status: Active » Needs review
StatusFileSize
new reorganize_tests-2629986-2.patch129.24 KB

Status: Needs review » Needs work

The last submitted patch, 2: reorganize_tests-2629986-2.patch, failed testing.

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
StatusFileSize
new reorganize_tests-2629986-4.patch129.25 KB
new interdiff.txt891 bytes

Fix missing class block end

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 4: reorganize_tests-2629986-4.patch, failed testing.

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Status: Needs work » Needs review
StatusFileSize
new reorganize_tests-2629986-7.patch129.25 KB
new interdiff.txt1.55 KB

Status: Needs review » Needs work

The last submitted patch, 7: reorganize_tests-2629986-7.patch, failed testing.

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Status: Needs work » Needs review
StatusFileSize
new reorganize_tests-2629986-9.patch129.24 KB

  • joelpittet committed 866a449 on 7.x-1.x 
    Issue #2629986 by joelpittet: Reorganize Tests
joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Status: Needs review » Fixed

All fixed and committed to -dev

Status: Fixed » Closed (fixed)

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