ref: improve logging when validation is false

Signed-off-by: Johnny Hujol <itudoben@users.noreply.github.com>

Author: itudoben

src/main/java/io/github/delirius325/jmeter/backendlistener/elasticsearch/ElasticsearchBackendClient.java

@@ -228,32 +228,43 @@ private void setSSLConfiguration(BackendListenerContext context) {
 
     @Override
     public void handleSampleResults(List<SampleResult> results, BackendListenerContext context) {
-        for (SampleResult sr : results) {
-            ElasticSearchMetric metric = new ElasticSearchMetric(sr, context.getParameter(ES_TEST_MODE),
-                    context.getParameter(ES_TIMESTAMP), this.buildNumber,
-                    context.getBooleanParameter(ES_PARSE_REQ_HEADERS, false),
-                    context.getBooleanParameter(ES_PARSE_RES_HEADERS, false), fields);
+      if (results.isEmpty()) {
+        logger.warn("There are no sampler results to handle!");
+        return;
+      }
 
-            if (validateSample(context, sr)) {
-                try {
-                    this.sender.addToList(new Gson().toJson(metric.getMetric(context)));
-                } catch (Exception e) {
-                    logger.error(
-                            "The ElasticSearch Backend Listener was unable to add sampler to the list of samplers to send... More info in JMeter's console.");
-                    e.printStackTrace();
-                }
-            }
+      for (SampleResult sr : results) {
+        logger.info(String.format("Handle sampler result %s: %s", sr.getSampleLabel(), sr.getResponseDataAsString()));
+        ElasticSearchMetric metric = new ElasticSearchMetric(sr, context.getParameter(ES_TEST_MODE),
+          context.getParameter(ES_TIMESTAMP), this.buildNumber,
+          context.getBooleanParameter(ES_PARSE_REQ_HEADERS, false),
+          context.getBooleanParameter(ES_PARSE_RES_HEADERS, false), fields);
+
+        if (validateSample(context, sr)) {
+          try {
+            this.sender.addToList(new Gson().toJson(metric.getMetric(context)));
+          }
+          catch (Exception e) {
+            logger.error(
+              "The ElasticSearch Backend Listener was unable to add sampler to the list of samplers to send... More info in JMeter's console.");
+            e.printStackTrace();
+          }
+        } else {
+          logger.warn(String.format("Not valid sampler result %s: %s", sr.getSampleLabel(), sr.getResponseDataAsString()));
         }
+      }
 
-        if (this.sender.getListSize() >= this.bulkSize) {
-            try {
-                this.sender.sendRequest(this.esVersion);
-            } catch (Exception e) {
-                logger.error("Error occured while sending bulk request.", e);
-            } finally {
-                this.sender.clearList();
-            }
+      if (this.sender.getListSize() >= this.bulkSize) {
+        try {
+          this.sender.sendRequest(this.esVersion);
+        }
+        catch (Exception e) {
+          logger.error("Error occurred while sending bulk request.", e);
+        }
+        finally {
+          this.sender.clearList();
         }
+      }
     }
 
     @Override
@@ -305,6 +316,8 @@ private boolean validateSample(BackendListenerContext context, SampleResult sr)
                     valid = true;
                     break;
                 } else {
+                  logger.warn(String.format("Invalidate sample label %s, it may start with '!!' or it may not contain filter %s. " +
+                    "Check the %s property value in the JMX file. Leave it empty to increase validation", sampleLabel, filter, ES_SAMPLE_FILTER));
                     valid = false;
                 }
             }