Problem/Motivation

Branch didn't pass tests.

Proposed resolution

  • Fix tests
  • Test against Drupal 8.2.x

Remaining tasks

Fix tests and/or convert simpletest tests

User interface changes

-

API changes

-

Data model changes

-

CommentFileSizeAuthor
#9 interdiff.txt6.85 KBslasher13
#9 2788543-fix-tests-8.patch128.12 KBslasher13
#8 interdiff.txt41.96 KBslasher13
#8 2788543-fix-tests-7.patch121.28 KBslasher13
#6 interdiff.txt4.63 KBslasher13
#6 2788543-fix-tests-6.patch79.31 KBslasher13
#5 interdiff.txt4.59 KBslasher13
#5 2788543-fix-tests-5.patch74.68 KBslasher13
#4 interdiff.txt14.23 KBslasher13
#4 2788543-fix-tests-4.patch70.09 KBslasher13
#3 interdiff.txt52.72 KBslasher13
#3 2788543-fix-tests-3.patch55.85 KBslasher13
#2 2788543-fix-tests-2.patch3.13 KBslasher13
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

slasher13 created an issue. See original summary.

slasher13’s picture

slasher13’s picture

slasher13’s picture

slasher13’s picture

slasher13’s picture

AsadKamil’s picture

Hi All,
The Patch works fine,Thanks

root@asad-Vostro-3550:/var/www/html/git/purge# git apply -v interdiff_22473.txt Checking patch src/Tests/Purger/PluginManagerTest.php...
Checking patch tests/src/Unit/Purger/PluginManagerTest.php...
Applied patch src/Tests/Purger/PluginManagerTest.php cleanly.
Applied patch tests/src/Unit/Purger/PluginManagerTest.php cleanly.
root@asad-Vostro-3550:/var/www/html/git/purge#
slasher13’s picture

slasher13’s picture

nielsvm’s picture

I'm aware that the tests aren't passing, or at least, are giving issues on drupal.org at minimum. Although this patch hasn't been applied (outdated by now), I'm more and more going to make sure that tests pass here instead of just in the simpletest webview.

Wim Leers’s picture

marcingy’s picture

Status: Needs review » Closed (duplicate)