dev #56994 : fix deletion flag on patrons

VERSIONS_WIP/56994

+ - ticket #56994 : Outils de migration de comptes lecteurs
\ No newline at end of file

cosmogramme/php/classes/classe_abonne.php

@@ -152,6 +152,7 @@ class abonne
 
     $user
       ->updateAttributes($data)
+      ->setStatut(0)
       ->saveWithoutValidation();
 
     Class_Users::clearCache();

tests/library/Class/Cosmogramme/Integration/PhasePatronsTest.php

@@ -62,6 +62,14 @@ abstract class PhasePatronsTestCase extends Class_Cosmogramme_Integration_PhaseT
                     'role_level' => ZendAfi_Acl_AdminControllerRoles::ABONNE_SIGB,
                     'bib' => $bib_annecy,
                     'idabon' => '666']);
+
+    $this->fixture('Class_Users',
+                   ['id' => 2,
+                    'login' => 'A-000208',
+                    'password' => '2001',
+                    'role_level' => ZendAfi_Acl_AdminControllerRoles::ABONNE_SIGB,
+                    'bib' => $bib_annecy,
+                    'idabon' => 'A-000208']);
   }
 
 
@@ -104,15 +112,15 @@ class PhasePatronsFullImportTest extends PhasePatronsTestCase {
 
 
   /** @test */
-  public function secondAbonneShouldBeBarryWhite() {
-    $barry = Class_Users::find(3);
+  public function barryWhiteShouldBeImported() {
+    $barry = Class_Users::findFirstBy(['login' => 'A-000033']);
     $this->assertEquals('white BARRY', $barry->getNomComplet());
     return $barry;
   }
 
 
   /**
-   * @depends secondAbonneShouldBeBarryWhite
+   * @depends barryWhiteShouldBeImported
    * @test
    */
   public function barryIdAbonShouldBeA000033($barry) {
@@ -121,7 +129,7 @@ class PhasePatronsFullImportTest extends PhasePatronsTestCase {
 
 
   /**
-   * @depends secondAbonneShouldBeBarryWhite
+   * @depends barryWhiteShouldBeImported
    * @test
    */
   public function barryIdSIGBShouldBe47($barry) {
@@ -131,7 +139,7 @@ class PhasePatronsFullImportTest extends PhasePatronsTestCase {
 
 
   /**
-   * @depends secondAbonneShouldBeBarryWhite
+   * @depends barryWhiteShouldBeImported
    * @test
    */
   public function barryOrdreabonShouldBeOne($barry) {
@@ -140,7 +148,7 @@ class PhasePatronsFullImportTest extends PhasePatronsTestCase {
 
 
   /**
-   * @depends secondAbonneShouldBeBarryWhite
+   * @depends barryWhiteShouldBeImported
    * @test
    */
   public function barryMailShouldBeEmpty($barry) {
@@ -149,7 +157,7 @@ class PhasePatronsFullImportTest extends PhasePatronsTestCase {
 
 
   /**
-   * @depends secondAbonneShouldBeBarryWhite
+   * @depends barryWhiteShouldBeImported
    * @test
    */
   public function barryPasswordShouldBe1978($barry) {
@@ -158,7 +166,7 @@ class PhasePatronsFullImportTest extends PhasePatronsTestCase {
 
 
   /**
-   * @depends secondAbonneShouldBeBarryWhite
+   * @depends barryWhiteShouldBeImported
    * @test
    */
   public function barryNaissanceShouldBe1978_05_19($barry) {
@@ -167,7 +175,7 @@ class PhasePatronsFullImportTest extends PhasePatronsTestCase {
 
 
   /**
-   * @depends secondAbonneShouldBeBarryWhite
+   * @depends barryWhiteShouldBeImported
    * @test
    */
   public function barryShouldNotHaveNULLAttribute($barry) {
@@ -176,7 +184,7 @@ class PhasePatronsFullImportTest extends PhasePatronsTestCase {
 
 
   /**
-   * @depends secondAbonneShouldBeBarryWhite
+   * @depends barryWhiteShouldBeImported
    * @test
    */
   public function barryFinShouldBe2006_03_23($barry) {
@@ -185,7 +193,7 @@ class PhasePatronsFullImportTest extends PhasePatronsTestCase {
 
 
   /**
-   * @depends secondAbonneShouldBeBarryWhite
+   * @depends barryWhiteShouldBeImported
    * @test
    */
   public function barryStatutShouldBeZero($barry) {
@@ -205,6 +213,12 @@ class PhasePatronsFullImportTest extends PhasePatronsTestCase {
     $this->assertEquals(Class_Users::STATUT_TO_BE_DELETED,
                         Class_Users::find(1)->getStatut());
   }
+
+
+  /** @test */
+  public function mieszkalskiShouldNotBeMarkedForDeletion() {
+    $this->assertEquals(0, Class_Users::find(2)->getStatut());
+  }
 }