Problem/Motivation

There is one unit test (for PrivateAccess plugin). There should be some more.

Proposed resolution

Add some additional unit tests!

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#3 2765049-03.patch18.31 KBjhedstrom
#3 interdiff.txt7.42 KBjhedstrom
#2 2765049-02.patch10.89 KBjhedstrom
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jhedstrom created an issue. See original summary.

jhedstrom’s picture

jhedstrom
Primary language English
Location Portland, OR
CreditAttribution: jhedstrom at Phase2 for Workday, Inc. commented
Assigned: jhedstrom » Unassigned
Status: Active » Needs review
FileSize
2765049-02.patch10.89 KB

Here's a start for now in case anybody else wants to pick this up. Otherwise, I should be able to get back to it next week.

jhedstrom’s picture

jhedstrom
Primary language English
Location Portland, OR
CreditAttribution: jhedstrom at Phase2 for Workday, Inc. commented
FileSize
interdiff.txt7.42 KB
2765049-03.patch18.31 KB

A couple more. This should about do it for now I think.

  • jhedstrom committed 3276436 on 8.x-1.x 
    Issue #2765049 by jhedstrom: Add some additional unit tests
jhedstrom’s picture

jhedstrom
Primary language English
Location Portland, OR
CreditAttribution: jhedstrom at Phase2 for Workday, Inc. commented
Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

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