Core created the new hook_runtime_requirements() a couple of weeks ago. See the change record. We only use hook_requirements() for runtime validation. So implementing this will make the module even more OO-oriented.

Doing it may also allow us to convert the RequirementsTest to a Unit test too. I'm not 100% sure about that yet. The entire reason it's a Functional test is because there's a constant that isn't available unless Drupal is bootstrapped. We might be able to mock it instead.

Issue fork usfedgov_google_analytics-3503814

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

dcam created an issue. See original summary.

dcam’s picture

dcam commented
Issue summary: View changes

dcam opened merge request !29

  • dcam committed 9e99948b on 3.x 
    Issue #3503814 by dcam: Implement hook_runtime_requirements()
dcam’s picture

dcam commented
Status: Active » Fixed

Status: Fixed » Closed (fixed)

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