CommentFileSizeAuthor
#3 add_test_coverage-2678368-3.patch13.48 KBbdone
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bdone created an issue. See original summary.

bdone’s picture

Assigned: Unassigned » bdone

going to try and add some test coverage for r4032login.

bdone’s picture

attaching a wip unit test for this. not yet sure how to mock the base url, without using the internal route base: prefix.

  • Nixou committed a78e0b4 on 8.x-1.x authored by bdone
    Issue #2678368 by bdone, nixou: Add test coverage
    
Nixou’s picture

Status: Active » Fixed

Thanks for the proposition.

I had to adapt or remove some tests because they was obsolete as they were writed 2 years ago.
All remaining tests are now ok so we have a base of test we can grow up.

Status: Fixed » Closed (fixed)

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