set labels from environment variables (fixes #91, via #117)

Author: remorhaz

.gitignore

@@ -4,6 +4,6 @@ vendor/*
 composer.phar
 composer.lock
 /build/
-/test/codeception/_support/_generated/
+/test/codeception*/_support/_generated/
 .phpunit.result.cache
 

codeception-report.yml

@@ -0,0 +1,17 @@
+namespace: Qameta\Allure\Codeception\Test\Report
+
+settings:
+  lint: true
+paths:
+  tests: test/codeception-report
+  output: build
+  support: test/codeception-report/_support
+  data: test/codeception-report/_data
+extensions:
+  enabled:
+    - Qameta\Allure\Codeception\AllureCodeception
+  config:
+    Qameta\Allure\Codeception\AllureCodeception:
+      outputDirectory: allure-results
+      linkTemplates:
+        issue: https://example.org/issues/%s

codeception.yml

@@ -7,11 +7,3 @@ paths:
   output: build
   support: test/codeception/_support
   data: test/codeception/_data
-extensions:
-  enabled:
-    - Qameta\Allure\Codeception\AllureCodeception
-  config:
-    Qameta\Allure\Codeception\AllureCodeception:
-      outputDirectory: allure-results
-      linkTemplates:
-        issue: https://example.org/issues/%s

composer.json

@@ -23,11 +23,10 @@
     "require": {
 	    "php": "^8",
         "ext-json": "*",
-        "codeception/codeception": "^5",
-        "allure-framework/allure-php-commons": "^2"
+        "codeception/codeception": "^5.0.3",
+        "allure-framework/allure-php-commons": "^2.3.1"
     },
     "require-dev": {
-        "phpunit/phpunit": "^9.5",
         "psalm/plugin-phpunit": "^0.18.4",
         "remorhaz/php-json-data": "^0.5.3",
         "remorhaz/php-json-path": "^0.7.7",
@@ -41,29 +40,34 @@
     },
     "autoload-dev": {
         "psr-4": {
-            "Qameta\\Allure\\Codeception\\Test\\Functional\\": "test/codeception/functional/",
-            "Qameta\\Allure\\Codeception\\Test\\Acceptance\\": "test/codeception/acceptance/",
-            "Qameta\\Allure\\Codeception\\Test\\Unit\\": "test/codeception/unit/",
-            "Qameta\\Allure\\Codeception\\Test\\": [
-                "test/codeception/_support/",
-                "test/report/"
-            ]
+            "Qameta\\Allure\\Codeception\\Test\\": "test/codeception/_support/",
+            "Qameta\\Allure\\Codeception\\Test\\Unit\\": [
+                "test/codeception/report-check/",
+                "test/codeception/unit/"
+            ],
+            "Qameta\\Allure\\Codeception\\Test\\Report\\": "test/codeception-report/_support/",
+            "Qameta\\Allure\\Codeception\\Test\\Report\\Functional\\": "test/codeception-report/functional/",
+            "Qameta\\Allure\\Codeception\\Test\\Report\\Acceptance\\": "test/codeception-report/acceptance/",
+            "Qameta\\Allure\\Codeception\\Test\\Report\\Unit\\": "test/codeception-report/unit/"
         }
     },
     "scripts": {
         "build": [
             "vendor/bin/codecept build",
-            "vendor/bin/codecept gherkin:snippets acceptance"
+            "vendor/bin/codecept build -c codeception-report.yml",
+            "vendor/bin/codecept gherkin:snippets acceptance -c codeception-report.yml"
         ],
         "test-cs": "vendor/bin/phpcs -sp",
+        "test-unit": "vendor/bin/codecept run unit --coverage-text",
         "test-report-generate": [
-            "rm -rf ./build/log/",
-            "vendor/bin/codecept run --no-exit --report"
+            "rm -rf ./build/allure-results/",
+            "vendor/bin/codecept run  -c codeception-report.yml --no-exit --report"
         ],
-        "test-report-check": "vendor/bin/phpunit --testsuite=report",
+        "test-report-check": "vendor/bin/codecept run report-check",
         "test-psalm": "vendor/bin/psalm --shepherd",
         "test": [
             "@test-cs",
+            "@test-unit",
             "@test-report-generate",
             "@test-report-check",
             "@test-psalm"

phpunit.xml.dist

@@ -1,14 +1,20 @@
 <?xml version="1.0"?>
 <psalm
         errorLevel="1"
+        findUnusedBaselineEntry="true"
+        findUnusedCode="false"
         xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
         xmlns="https://getpsalm.org/schema/config"
         xsi:schemaLocation="https://getpsalm.org/schema/config vendor/vimeo/psalm/config.xsd">
     <projectFiles>
         <directory name="src"/>
-        <directory name="test/report/"/>
+        <directory name="test/codeception/"/>
+        <directory name="test/codeception-report/unit/"/>
         <ignoreFiles>
+            <file name="src/StatusDetector.php" />
             <directory name="vendor"/>
+            <directory name="test/codeception/_support/"/>
+            <directory name="test/codeception-report/_support/"/>
         </ignoreFiles>
     </projectFiles>
     <plugins>

psalm.xml.dist

@@ -74,7 +74,6 @@ public function moduleInit(): void
         $this->testLifecycle = null;
         $this->threadDetector = null;
         QametaAllure::getLifecycleConfigurator()
-            ->setStatusDetector(new StatusDetector(new DefaultStatusDetector()))
             ->setOutputDirectory($this->getOutputDirectory());
         foreach ($this->getLinkTemplates() as $linkType => $linkTemplate) {
             QametaAllure::getLifecycleConfigurator()->addLinkTemplate($linkType, $linkTemplate);
@@ -191,18 +190,26 @@ public function testError(FailEvent $failEvent): void
         $this
             ->getTestLifecycle()
             ->switchToTest($failEvent->getTest())
-            ->updateTestFailure($failEvent->getFail());
+            ->updateTestFailure(
+                $failEvent->getFail(),
+                Status::broken(),
+            );
     }
 
     /**
      * @psalm-suppress MissingDependency
      */
     public function testFail(FailEvent $failEvent): void
     {
+        $error = $failEvent->getFail();
         $this
             ->getTestLifecycle()
             ->switchToTest($failEvent->getTest())
-            ->updateTestFailure($failEvent->getFail(), Status::failed());
+            ->updateTestFailure(
+                $failEvent->getFail(),
+                Status::failed(),
+                new StatusDetails(message: $error->getMessage(), trace: $error->getTraceAsString()),
+            );
     }
 
     /**
@@ -294,6 +301,7 @@ private function getTestLifecycle(): TestLifecycleInterface
             Allure::getConfig()->getStatusDetector(),
             $this->getThreadDetector(),
             Allure::getConfig()->getLinkTemplates(),
+            $_ENV,
         );
     }
 }

src/AllureCodeception.php

@@ -13,6 +13,7 @@
 use Qameta\Allure\AllureLifecycleInterface;
 use Qameta\Allure\Codeception\Setup\ThreadDetectorInterface;
 use Qameta\Allure\Io\DataSourceFactory;
+use Qameta\Allure\Model\EnvProvider;
 use Qameta\Allure\Model\ModelProviderChain;
 use Qameta\Allure\Model\Parameter;
 use Qameta\Allure\Model\ResultFactoryInterface;
@@ -46,19 +47,13 @@ final class TestLifecycle implements TestLifecycleInterface
      */
     private WeakMap $stepStarts;
 
-    /**
-     * @param AllureLifecycleInterface        $lifecycle
-     * @param ResultFactoryInterface          $resultFactory
-     * @param StatusDetectorInterface         $statusDetector
-     * @param ThreadDetectorInterface         $threadDetector
-     * @param LinkTemplateCollectionInterface $linkTemplates
-     */
     public function __construct(
         private AllureLifecycleInterface $lifecycle,
         private ResultFactoryInterface $resultFactory,
         private StatusDetectorInterface $statusDetector,
         private ThreadDetectorInterface $threadDetector,
         private LinkTemplateCollectionInterface $linkTemplates,
+        private array $env,
     ) {
         /** @psalm-var WeakMap<Step, StepStartInfo> $this->stepStarts */
         $this->stepStarts = new WeakMap();
@@ -143,6 +138,7 @@ public function create(): self
     public function updateTest(): self
     {
         $provider = new ModelProviderChain(
+            new EnvProvider($this->env),
             ...SuiteProvider::createForChain($this->getCurrentSuite(), $this->linkTemplates),
             ...TestInfoProvider::createForChain($this->getCurrentTest()),
             ...$this->createModelProvidersForTest($this->getCurrentTest()->getOriginalTest()),

src/Internal/TestLifecycle.php

@@ -117,6 +117,6 @@ public function getDescriptionHtml(): ?string
 
     public function getFullName(): ?string
     {
-        return null;
+        return $this->test->getTestCase()::class . '::' . $this->test->getMetadata()->getName();
     }
 }

src/Internal/UnitProvider.php

@@ -12,6 +12,10 @@
 use Qameta\Allure\Setup\StatusDetectorInterface;
 use Throwable;
 
+/**
+ * @deprecated This class is not used anymore and will be removed in next major version.
+ * @psalm-suppress UnusedClass
+ */
 final class StatusDetector implements StatusDetectorInterface
 {
     public function __construct(