fix(api): do not init db connection in event subscriber (#11543)

doc/API/centreon-api-v22.04.yaml

@@ -3543,7 +3543,7 @@ paths:
             application/json:
               schema:
                 type: object
-                required: ["installed_version", "has_upgrade_available"]
+                required: ["is_installed", "has_upgrade_available"]
                 properties:
                   is_installed:
                     type: boolean

src/EventSubscriber/UpdateEventSubscriber.php

@@ -47,6 +47,10 @@ public function __construct(
      */
     public static function getSubscribedEvents(): array
     {
+        if (! file_exists(_CENTREON_ETC_ . DIRECTORY_SEPARATOR . 'centreon.conf.php')) {
+            return [];
+        }
+
         return [
             KernelEvents::REQUEST => [
                 ['validateCentreonWebVersionOrFail', 35],
@@ -55,19 +59,13 @@ public static function getSubscribedEvents(): array
     }
 
     /**
-     * validation centreon web installed version
+     * validate centreon web installed version when update endpoint is called
      *
      * @param RequestEvent $event
      * @throws \Exception
      */
     public function validateCentreonWebVersionOrFail(RequestEvent $event): void
     {
-        $this->debug('Checking if database configuration file exists to know if centreon is already installed');
-        if (! file_exists(_CENTREON_ETC_ . DIRECTORY_SEPARATOR . 'centreon.conf.php')) {
-            $this->debug('Centreon database configuration file not found');
-            return;
-        }
-
         $this->debug('Checking if route matches updates endpoint');
         if (
             $event->getRequest()->getMethod() === Request::METHOD_PATCH

tests/api/Context/PlatformInstallationStatusContext.php

@@ -0,0 +1,52 @@
+<?php
+
+/*
+ * Copyright 2005 - 2022 Centreon (https://www.centreon.com/)
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ *
+ * For more information : contact@centreon.com
+ *
+ */
+
+namespace Centreon\Test\Api\Context;
+
+use Centreon\Test\Behat\Api\Context\ApiContext;
+
+class PlatformInstallationStatusContext extends ApiContext
+{
+    /**
+     * Remove centreon databases, configuration file and symfony cache
+     *
+     * @Given Centreon Web is not installed
+     */
+    public function centreonWebIsNotInstalled()
+    {
+        $this->getContainer()->execute(
+            'mysql -e "DROP DATABASE centreon_storage"',
+            'web'
+        );
+        $this->getContainer()->execute(
+            'mysql -e "DROP DATABASE centreon"',
+            'web'
+        );
+        $this->getContainer()->execute(
+            'rm -f /etc/centreon/centreon.conf.php',
+            'web'
+        );
+        $this->getContainer()->execute(
+            'rm -rf /var/cache/centreon/symfony',
+            'web'
+        );
+    }
+}

tests/api/behat.yml

@@ -72,6 +72,10 @@ default:
       paths: [ "%paths.base%/features/PlatformInformation.feature" ]
       contexts:
         - Centreon\Test\Api\Context\PlatformInformationContext
+    platform_fresh_install:
+      paths: [ "%paths.base%/features/PlatformInstallationStatus.feature" ]
+      contexts:
+        - Centreon\Test\Api\Context\PlatformInstallationStatusContext
     platform_update:
       paths: [ "%paths.base%/features/PlatformUpdate.feature" ]
       contexts:

tests/api/features/PlatformInstallationStatus.feature

@@ -0,0 +1,18 @@
+Feature:
+    In order to maintain centreon platform
+    As an administrator
+    I want to known the platform installation status
+
+    Background:
+        Given a running instance of Centreon Web API
+        And the endpoints are described in Centreon Web API documentation
+
+    Scenario: Update platform information
+        When I send a GET request to '/api/latest/platform/installation/status'
+        Then the response code should be "200"
+        And the JSON node "is_installed" should be equal to true
+
+        Given Centreon Web is not installed
+        When I send a GET request to '/api/latest/platform/installation/status'
+        Then the response code should be "200"
+        And the JSON node "is_installed" should be equal to false