Follow-up of #1447736: Adopt Guzzle library to replace drupal_http_request(), quoting myself:

Another thing, SimpletestHttpRequestSubscriber is currently added by CoreBundle, but it should be in simpletest.module, as @Crell already pointed out after the first patches here.

But, that means that the event will only be executed in tests that have simpletest.module enabled, which is kinda pointless ;) So we either need to keep it there or add a TestBundle or something like that? I'm not exactly sure what the point of that subscriber is, anyway, is it just for requests done by drupalPost()/drupalGet()? Another follow-up, I guess...

Comments

sun’s picture

Berdir’s picture

Title: Decide what to do with SimpletestHttpRequestSubscriber » Improve documentation of SimpletestHttpRequestSubscriber

Sorry, I missed that.

Changing this task to add this information to the class then and also fix the coding style. We're not using @inheritdoc, for exampe.

Version: 8.0.x-dev » 8.1.x-dev

Drupal 8.0.6 was released on April 6 and is the final bugfix release for the Drupal 8.0.x series. Drupal 8.0.x will not receive any further development aside from security fixes. Drupal 8.1.0-rc1 is now available and sites should prepare to update to 8.1.0.

Bug reports should be targeted against the 8.1.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.2.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.