Changeset 1049

Show
Ignore:
Timestamp:
11/19/07 15:15:29 (10 months ago)
Author:
richi
Message:

fixed bug (stubMemoryLogAppender could hold only one data record per target)

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • trunk/src/main/php/net/stubbles/util/log/stubMemoryLogAppender.php

    r1047 r1049  
    2727     * @return array<string: targetName<string: logData>> 
    2828     */ 
    29     public function getLogDataStorage()  
     29    public function getLogDataStorage() 
    3030    { 
    31         return $this->logDataStorage;     
     31        return $this->logDataStorage; 
    3232    } 
    3333 
    …… 
    6060    public function append(stubLogData $logData) 
    6161    { 
    62         $this->logDataStorage[$logData->getTarget()] = explode(stubLogData::SEPERATOR, $logData->get()); 
     62        $this->logDataStorage[$logData->getTarget()][] = explode(stubLogData::SEPERATOR, $logData->get()); 
    6363    } 
    6464 

  • trunk/src/test/php/net/stubbles/util/log/stubMemoryLogAppenderTestCase.php

    r1046 r1049  
    2323     */ 
    2424    protected $memoryLogAppender; 
    25      
     25 
    2626    /** 
    2727     * set up test environment 
    2828     */ 
    29     public function setUp()  
     29    public function setUp() 
    3030    { 
    3131        $this->memoryLogAppender =  new stubMemoryLogAppender(); 
    3232    } 
    33      
     33 
    3434    /** 
    35      * assure that appended data is stored in array 
     35     * assure that appended data is stored in array (one record) 
    3636     */ 
    37     public function testAppend()  
     37    public function testAppendWithOneDataRecord() 
    3838    { 
    3939        $mockSession  = new MockstubSession(); 
    4040        $mockLogData  = new MockstubLogData(); 
    4141        $mockLogData->setReturnValue('getTarget', 'myTestTarget'); 
    42         $mockLogData->setReturnValue('get', 'a|b|c');  
     42        $mockLogData->setReturnValue('get', 'a|b|c'); 
    4343        $this->memoryLogAppender->append($mockLogData); 
    44         $this->assertEqual($this->memoryLogAppender->getLogDataStorage(),  array( 
    45                                                                                 'myTestTarget' => array( 
    46                                                                                         0 => 'a', 
    47                                                                                         1 => 'b', 
    48                                                                                         2 => 'c', 
     44 
     45        $this->assertEqual($this->memoryLogAppender->getLogDataStorage(),  array ( 
     46                                                                                   'myTestTarget' => array ( 
     47                                                                                     0 => array ( 
     48                                                                                       0 => 'a', 
     49                                                                                       1 => 'b', 
     50                                                                                       2 => 'c', 
     51                                                                                     ) 
     52                                                                                   ) 
    4953                                                                                 ) 
    50                                                                             ) 
    5154        ); 
    5255    } 
    53      
     56 
     57    /** 
     58     * assure that appended data is stored in array (more than one records) 
     59     */ 
     60    public function testAppendWithMoreThanOneDataRecord() 
     61    { 
     62        $mockSession  = new MockstubSession(); 
     63        $mockLogData  = new MockstubLogData(); 
     64        $mockLogData->setReturnValue('getTarget', 'myTestTarget'); 
     65        $mockLogData->setReturnValue('get', 'a|b|c'); 
     66        $this->memoryLogAppender->append($mockLogData); 
     67        $this->memoryLogAppender->append($mockLogData); 
     68 
     69        $this->assertEqual($this->memoryLogAppender->getLogDataStorage(),  array ( 
     70                                                                                   'myTestTarget' => array ( 
     71                                                                                     0 => array ( 
     72                                                                                       0 => 'a', 
     73                                                                                       1 => 'b', 
     74                                                                                       2 => 'c', 
     75                                                                                     ), 
     76                                                                                     1 => array ( 
     77                                                                                       0 => 'a', 
     78                                                                                       1 => 'b', 
     79                                                                                       2 => 'c', 
     80                                                                                     ) 
     81                                                                                   ) 
     82                                                                                 ) 
     83        ); 
     84    } 
     85 
    5486} 
    5587?>