Projekt

Allgemein

Profil

« Zurück | Weiter » 

Revision b4cc349b

Von Jan Büren vor mehr als 6 Jahren hinzugefügt

  • ID b4cc349b7fe50744832261c8b415bd146bde5520
  • Vorgänger 8dfafa71
  • Nachfolger 1de19311

SelfTest false positive vermeiden

$self->all_passed enthält nicht mehr den Zustand, ob alle
Tests erfolgreich waren. Als Workaround auf zwei negativ
Status-Meldungen prüfen, die bei Problemen gesetzt sind.

Unterschiede anzeigen:

SL/BackgroundJob/SelfTest.pm
63 63 
             $self->aggreg->failed,
64 64 
             $self->aggreg->todo_passed,
65 65 
  );
66 

  		




  67
  
  
    
  if (!$self->aggreg->all_passed || $self->config->{send_email_on_success}) {


  		




  
  66
  
    
  # if (!$self->aggreg->all_passed || $self->config->{send_email_on_success}) {


  		




  
  67
  
    
  # all_passed is not set or calculated (anymore). it is safe to check only for probs or errors


  		




  
  68
  
    
  if ($self->aggreg->has_problems || $self->aggreg->has_errors || $self->config->{send_email_on_success}) {


  		




  68
  69
  
    
    $self->_send_email;


  		




  69
  70
  
    
  }


  		




  70
  71
  
    

  		












Auch abrufbar als:  Unified diff