The tests were "ported" to PHPUnit in #3021997: Convert automated tests from Simpletest to PHPUnit, but that issue was committed and marked "Fixed" without actually completing the job.

As you can see from the DrupalCI test results at https://www.drupal.org/pift-ci-job/1616679 - NO test are being run for the D8 version of this module. This has a simple explanation - the namespace of the test classes is wrong so these tests are never discovered by DrupalCI.

Note, this may not be all that's wrong, but we won't know until the test run ... this patch makes the tests run locally for me ...

CommentFileSizeAuthor
#3 3131963-3-votingapi-test-namespace.patch1.18 KBtr
votingapi-test-namespace.patch736 bytestr

Comments

TR created an issue. See original summary.

tr’s picture

tr commented
Issue summary: View changes
tr’s picture

tr commented
StatusFileSize
new 3131963-3-votingapi-test-namespace.patch1.18 KB

As you can see, with this patch we now have 4 tests that run (and pass!) on Drupal 8.

The testbot shows 3 coding standards problems in these tests. Two of them are triggered by commented-out code and shouldn't be fixed. But the third can be fixed, so let's do that at the same time as the namespace fix.

Here's a new patch with is the same as the above but with the coding standards fix.

tr’s picture

tr commented

This is blocking many other issues ...

pifagor’s picture

pifagor commented
Status: Needs review » Reviewed & tested by the community

  • pifagor committed 613d05a on 8.x-3.x authored by TR 
    Issue #3131963 by TR, pifagor: Drupal 8 tests aren't running because of...
pifagor’s picture

pifagor commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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