Support from Acquia helps fund testing for Drupal Acquia logo

Comments

deviantintegral created an issue. See original summary.

deviantintegral’s picture

Status: Active » Postponed
FileSize
14.39 KB

This is dependent on #2614922: Remove JSONReader::setConfiguration() in favour of __construct().

This patch brings the JSONReader class up to 100% code coverage in tests. It also fixes some "bugs" in that there were code paths that could never be called. The patch is a little big because I moved the test class to match the hierarchy of the class it's testing.

KarenS’s picture

Status: Postponed » Fixed

Committed, thanks again :)

Status: Fixed » Closed (fixed)

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