composer/iTop
6
0
Fork 0
mirror of https://github.com/Combodo/iTop.git synced 2026-05-19 07:12:26 +02:00
Code Issues Packages Projects Releases Wiki Activity

✅ Fix CI

Browse Source
This commit is contained in:
Eric Espie
2026-04-17 15:32:12 +02:00
parent 0fb15e72d4
commit 7819620013
2 changed files with 14 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
setup/sequencers/DataAuditSequencer.php
View File

@@ -72,7 +72,9 @@ class DataAuditSequencer extends StepSequencer
return $this->GetNextStep('complete', 'Check Completed', 100); return $this->GetNextStep('complete', 'Check Completed', 100);
case 'setup-audit': case 'setup-audit':
$this->oRunTimeEnvironment->DataToCleanupAudit(); if ($this->IsDataAuditRequired()) {
$this->oRunTimeEnvironment->DataToCleanupAudit();
}
return $this->GetNextStep('complete', 'Check Completed', 100); return $this->GetNextStep('complete', 'Check Completed', 100);
case 'complete': case 'complete':

23
tests/php-unit-tests/unitary-tests/setup/sequencers/DataAuditSequencerTest.php
View File

@@ -5,7 +5,6 @@ namespace Combodo\iTop\Test\UnitTest\Setup\Sequencers;
use Combodo\iTop\Test\UnitTest\ItopTestCase; use Combodo\iTop\Test\UnitTest\ItopTestCase;
use DataAuditSequencer; use DataAuditSequencer;
use PHPParameters; use PHPParameters;
use PHPUnit\Framework\TestCase;
class DataAuditSequencerTest extends ItopTestCase class DataAuditSequencerTest extends ItopTestCase
{ {
@@ -106,8 +105,8 @@ class DataAuditSequencerTest extends ItopTestCase
$aExpected = [ $aExpected = [
'status' => 1, 'status' => 1,
'message' => '', 'message' => '',
'next-step' => '', 'next-step' => 'complete',
'next-step-label' => 'Completed', 'next-step-label' => 'Check Completed',
'percentage-completed' => 100, 'percentage-completed' => 100,
]; ];
$this->assertEquals($aExpected, $aRes); $this->assertEquals($aExpected, $aRes);
@@ -129,8 +128,8 @@ class DataAuditSequencerTest extends ItopTestCase
$aExpected = [ $aExpected = [
'status' => 1, 'status' => 1,
'message' => '', 'message' => '',
'next-step' => '', 'next-step' => 'complete',
'next-step-label' => 'Completed', 'next-step-label' => 'Check Completed',
'percentage-completed' => 100, 'percentage-completed' => 100,
]; ];
$this->assertEquals($aExpected, $aRes); $this->assertEquals($aExpected, $aRes);
@@ -159,7 +158,7 @@ class DataAuditSequencerTest extends ItopTestCase
{ {
$oRunTimeEnvironment = $this->createMock(\RunTimeEnvironment::class); $oRunTimeEnvironment = $this->createMock(\RunTimeEnvironment::class);
$oRunTimeEnvironment->expects($this->once())->method('DataToCleanupAudit'); $oRunTimeEnvironment->expects($this->once())->method('DataToCleanupAudit');
$oRunTimeEnvironment->expects($this->once())->method('GetFinalEnv') $oRunTimeEnvironment->expects($this->any())->method('GetFinalEnv')
->willReturn('production'); ->willReturn('production');
$aAdditionalParams = [ $aAdditionalParams = [
@@ -171,8 +170,8 @@ class DataAuditSequencerTest extends ItopTestCase
$aExpected = [ $aExpected = [
'status' => 1, 'status' => 1,
'message' => '', 'message' => '',
'next-step' => '', 'next-step' => 'complete',
'next-step-label' => 'Completed', 'next-step-label' => 'Check Completed',
'percentage-completed' => 100, 'percentage-completed' => 100,
]; ];
$this->assertEquals($aExpected, $aRes); $this->assertEquals($aExpected, $aRes);
@@ -193,8 +192,8 @@ class DataAuditSequencerTest extends ItopTestCase
$aExpected = [ $aExpected = [
'status' => 1, 'status' => 1,
'message' => '', 'message' => '',
'next-step' => '', 'next-step' => 'complete',
'next-step-label' => 'Completed', 'next-step-label' => 'Check Completed',
'percentage-completed' => 100, 'percentage-completed' => 100,
]; ];
$this->assertEquals($aExpected, $aRes); $this->assertEquals($aExpected, $aRes);
@@ -213,8 +212,8 @@ class DataAuditSequencerTest extends ItopTestCase
$aExpected = [ $aExpected = [
'status' => 1, 'status' => 1,
'message' => '', 'message' => '',
'next-step' => '', 'next-step' => 'complete',
'next-step-label' => 'Completed', 'next-step-label' => 'Check Completed',
'percentage-completed' => 100, 'percentage-completed' => 100,
]; ];
$this->assertEquals($aExpected, $aRes); $this->assertEquals($aExpected, $aRes);