Add better and more complete logging for Lighthouse test runner

pingevt · Jan 1, 2024 · ead963b · ead963b
1 parent bb74c63
commit ead963b
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/modules/esm_test_lighthouse/src/Plugin/EsmTestRunner/LighthouseTestRunner.php b/modules/esm_test_lighthouse/src/Plugin/EsmTestRunner/LighthouseTestRunner.php
@@ -154,6 +154,7 @@ function ($el) {
       $this->loggerFactory->get('esm:lighthouse')->debug("Attempting to Call Github Action: <pre>" . print_r([
         $config->get('api_url'),
         $post_data,
+        json_encode($post_data),
       ], TRUE) . "</pre>", []);
 
       $ch = curl_init($config->get('api_url'));
@@ -172,7 +173,8 @@ function ($el) {
 
       $this->loggerFactory->get('esm:lighthouse')->debug("Response Code: @code", ["@code" => $response_code]);
 
-      if ($errno = curl_errno($ch)) {
+      $errno = curl_errno($ch);
+      if ($errno ) {
         $error_message = curl_strerror($errno);
         $this->loggerFactory->get('esm:lighthouse')->error("cURL erro ({@errno}): @err", [
           "@errno" => $errno,

diff --git a/modules/esm_url_group/src/Entity/UrlGroup.php b/modules/esm_url_group/src/Entity/UrlGroup.php
@@ -28,7 +28,7 @@
  *       "delete" = "Drupal\Core\Entity\ContentEntityDeleteForm"
  *     },
  *     "route_provider" = {
- *       "html" = "Drupal\Core\Entity\Routing\AdminHtmlRouteProvider",
+ *       "html" = "Drupal\external_site_monitor\Entity\Routing\EsmRouteProvider",
  *     }
  *   },
  *   base_table = "url_group",