Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
Parent ticket: #2616404: Create SimpleTest for basic creating, importing, exporting, etc.
Need test coverage for assignment plugins.
Proposed resolution
Remaining tasks
User interface changes
API changes
Data model changes
Comment | File | Size | Author |
---|---|---|---|
#2 | assignment-tests-2619650-2.patch | 8.73 KB | nedjo |
|
Comments
Comment #2
nedjoHere are draft tests for the base, core, and dependency assignment plugins.
Comment #3
dawehnerIn general the tests are looking great! Small tests, and limits in its scope.
Given that features knows which code exists I'd have typehinted against concrete classes, which makes it a bit easier to follow.
yeah, for adding helper methods here.
There are a little bit of whitespace ... I'd also just not use messages anymore, its kinda pointless IMHO quite often.
There is assertContains if you want to use that. Its a bit better to read at the end.
Comment #4
dawehnerFixed those nitpicks and committed to 8.x-3.x
Thank you!
Comment #7
nedjo