Banner message change

Better work with nonexist action call exception

Author: Vitexus

src/AbraFlexi/RO.php

@@ -652,7 +652,7 @@ public function setDataValue(string $columnName, $value) {
         }
         return $result;
     }
-    
+
     /**
      * Strip all non-identifier data
      * 
@@ -664,7 +664,7 @@ public function stripBody(array $keep = []) {
         $extIds = $this->getExternalIDs();
         $restoreData = [];
         $originalData = $this->getData();
-        foreach ($keep as $column){
+        foreach ($keep as $column) {
             \Ease\Functions::divDataArray($originalData, $restoreData, $column);
         }
         $this->dataReset();
@@ -1193,7 +1193,7 @@ public function parseResponse($responseDecoded, $responseCode) {
                 $this->addStatusMessage($this->lastResponseCode . ': ' . $this->curlInfo['url'] . ' (' . $this->format . ') ' . json_encode($this->getErrors()), 'warning');
                 if ($this->throwException) {
                     $errors = $this->getErrors();
-                    throw new Exception( empty($errors) ? 'Problem ' : $errors[0]['message'] , $this);
+                    throw new Exception(empty($errors) ? 'Problem ' : $errors[0]['message'], $this);
                 }
                 break;
         }
@@ -2801,7 +2801,7 @@ function ($key) {
     public function logBanner($prefix = null, $suffix = null) {
         parent::logBanner($prefix,
                 'ServerURL ' . str_replace('://', '://' . $this->user . '@',
-                        $this->getApiUrl()) . ' library v' . self::$libVersion . ' (AbraFlexi ' . EvidenceList::$version . ')', $suffix);
+                        $this->getApiUrl()) . ' php-abraflexi v' . self::$libVersion . ' (AbraFlexi ' . EvidenceList::$version . ')', $suffix);
     }
 
     /**

testing/src/AbraFlexi/ROTest.php

@@ -61,11 +61,20 @@ protected function tearDown(): void {
 
     }
 
+    private function withEvidenceOnly() {
+        if (empty($this->object->getEvidence())) {
+            $this->markTestSkipped('Evidence is not set');
+        }
+    }
+
     /**
      * @covers AbraFlexi\AbraFlexiRO::logBanner
      */
     public function testLogBanner() {
+        \Ease\Logger\Regent::singleton()->cleanMessages();
         $this->object->logBanner(addslashes(get_class($this)));
+        $messages = \Ease\Logger\Regent::singleton()->getMessages();
+        $this->assertFalse(empty(strstr($messages[0]->body, 'ServerURL http')));
     }
 
     /**
@@ -112,6 +121,7 @@ public function testCurlInit() {
      * @covers AbraFlexi\AbraFlexiRO::processInit
      */
     public function testProcessInit() {
+
         $this->object->processInit(1);
         $this->object->processInit(['id' => 1]);
         $this->assertEquals(1, $this->object->getDataValue('id'));