This is a long shot, but I think I may have just figured out a way for themes to provide actual tests.

CommentFileSizeAuthor
#8 provide_unittests_for-2138693-8.patch1 KBmarkhalliwell
#7 provide_unittests_for-2138693-7.patch1002 bytesmarkhalliwell
bootstrap-unittests.patch1.17 KBmarkhalliwell

Comments

Status: Needs review » Needs work

The last submitted patch, bootstrap-unittests.patch, failed testing.

markhalliwell’s picture

markhalliwell
Primary language English
commented
Parent issue: » #2138733: run-test.sh needs full path for --file

The last submitted patch, bootstrap-unittests.patch, failed testing.

Status: Needs work » Needs review

markcarver queued bootstrap-unittests.patch for re-testing.

Status: Needs review » Needs work

The last submitted patch, bootstrap-unittests.patch, failed testing.

markhalliwell’s picture

markhalliwell
Primary language English
commented
Status: Needs work » Postponed

Still not working, postponed on #2138733: run-test.sh needs full path for --file.

markhalliwell’s picture

markhalliwell
Primary language English
commented
Version: 7.x-3.x-dev » 8.x-3.x-dev
Status: Postponed » Active
StatusFileSize
new provide_unittests_for-2138693-7.patch1002 bytes
markhalliwell’s picture

markhalliwell
Primary language English
commented
StatusFileSize
new provide_unittests_for-2138693-8.patch1 KB
markhalliwell’s picture

markhalliwell
Primary language English
commented
Version: 8.x-3.x-dev » 8.x-4.x-dev
Status: Active » Needs work

Sigh... this still isn't possible. Probably won't happen for 8.x-3.x. I bumped the Drupal CI related issue... again.

markhalliwell’s picture

markhalliwell
Primary language English
commented
Status: Needs work » Postponed
Parent issue: #2138733: run-test.sh needs full path for --file » #2769599: [PP-1] Allow themes to be testable

Err... this is actually postponed on this parent issue.

markhalliwell’s picture

markhalliwell
Primary language English
commented
Status: Postponed » Needs review

Maybe... just maybe?!

Status: Needs review » Needs work

The last submitted patch, 8: provide_unittests_for-2138693-8.patch, failed testing. View results

markhalliwell’s picture

markhalliwell
Primary language English
commented
Version: 8.x-4.x-dev » 8.x-3.x-dev
Status: Needs work » Needs review

Oops, no code currently in 8.x-4.x heh. Let's try this again.

Status: Needs review » Needs work

The last submitted patch, 8: provide_unittests_for-2138693-8.patch, failed testing. View results

Mixologic’s picture

Mixologic
He/Him/His
Primary language English
Location Portland, OR
commented
Status: Needs work » Postponed

There's definitely still some things inside both pift and drupalci that are stopping this from working. I'll keep at them.

Mixologic’s picture

Mixologic
He/Him/His
Primary language English
Location Portland, OR
commented
Status: Postponed » Needs review

I purged everything that was module specific, and it ran your one test. yay. I think we've now got theme testing working.

markhalliwell’s picture

markhalliwell
Primary language English
commented
Status: Needs review » Fixed

Fantastic!!! Thank you @Mixologic!

markhalliwell’s picture

markhalliwell
Primary language English
commented
Status: Fixed » Closed (fixed)