Implement Copilot Chat GUI and Test Element for JMeter integration

Author: brunoborges

…ugins/copilot/CopilotChatVisualizer.java …lugins/copilot/CopilotChatConfigGui.javasrc/main/java/org/apache/jmeter/plugins/copilot/CopilotChatVisualizer.java renamed to src/main/java/org/apache/jmeter/plugins/copilot/CopilotChatConfigGui.java src/main/java/org/apache/jmeter/plugins/copilot/CopilotChatVisualizer.java renamed to src/main/java/org/apache/jmeter/plugins/copilot/CopilotChatConfigGui.java

@@ -20,41 +20,39 @@
 
 import javax.swing.BorderFactory;
 import javax.swing.Box;
-import javax.swing.JComponent;
 import javax.swing.JPanel;
 
+import org.apache.jmeter.config.gui.AbstractConfigGui;
 import org.apache.jmeter.plugins.copilot.gui.CopilotChatPanel;
-import org.apache.jmeter.samplers.SampleResult;
 import org.apache.jmeter.testelement.TestElement;
-import org.apache.jmeter.visualizers.gui.AbstractVisualizer;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
 import com.google.auto.service.AutoService;
 
 /**
- * JMeter Visualizer plugin that provides a GitHub Copilot Chat interface
+ * JMeter Config Element plugin that provides a GitHub Copilot Chat interface
  * for generating JMeter test plans through natural language conversation.
  * 
  * This plugin allows users to describe the tests they want to create, and
  * Copilot will generate the appropriate JMeter elements and add them to
  * the test plan.
  */
 @AutoService(org.apache.jmeter.gui.JMeterGUIComponent.class)
-public class CopilotChatVisualizer extends AbstractVisualizer {
+public class CopilotChatConfigGui extends AbstractConfigGui {
 
     private static final long serialVersionUID = 1L;
-    private static final Logger log = LoggerFactory.getLogger(CopilotChatVisualizer.class);
+    private static final Logger log = LoggerFactory.getLogger(CopilotChatConfigGui.class);
 
     private CopilotChatPanel chatPanel;
     private JPanel mainPanel;
 
     /**
-     * Creates a new CopilotChatVisualizer instance.
+     * Creates a new CopilotChatConfigGui instance.
      */
-    public CopilotChatVisualizer() {
+    public CopilotChatConfigGui() {
         super();
-        log.info("Initializing Copilot Chat Visualizer");
+        log.info("Initializing Copilot Chat Config Element");
         init();
     }
 
@@ -89,8 +87,6 @@ private void init() {
      */
     private void onTestPlanGenerated(String testPlanXml) {
         log.info("Test plan generated by Copilot");
-        // This will be implemented to integrate with JMeter's GUI
-        // to add the generated elements to the test plan tree
         try {
             JMeterTestPlanIntegrator.integrateTestPlan(testPlanXml);
         } catch (Exception e) {
@@ -110,31 +106,16 @@ public String getStaticLabel() {
     }
 
     @Override
-    public void add(SampleResult sample) {
-        // This visualizer doesn't process sample results
-        // It's used for test generation, not result visualization
-    }
-
-    @Override
-    public void clearData() {
-        // Clear the chat history if needed
-        if (chatPanel != null) {
-            chatPanel.clearChat();
-        }
-    }
-
-    @Override
-    public TestElement makeTestElement() {
+    public TestElement createTestElement() {
         CopilotChatTestElement element = new CopilotChatTestElement();
         modifyTestElement(element);
         return element;
     }
 
     @Override
     public void modifyTestElement(TestElement element) {
-        super.modifyTestElement(element);
+        configureTestElement(element);
         if (element instanceof CopilotChatTestElement copilotElement) {
-            // Save any configuration from the GUI to the test element
             copilotElement.setModel(chatPanel.getSelectedModel());
         }
     }
@@ -143,18 +124,20 @@ public void modifyTestElement(TestElement element) {
     public void configure(TestElement element) {
         super.configure(element);
         if (element instanceof CopilotChatTestElement copilotElement) {
-            // Load configuration from the test element to the GUI
             chatPanel.setSelectedModel(copilotElement.getModel());
         }
     }
 
     @Override
-    public JComponent getPrintableComponent() {
-        return mainPanel;
+    public void clearGui() {
+        super.clearGui();
+        if (chatPanel != null) {
+            chatPanel.clearChat();
+        }
     }
 
     /**
-     * Clean up resources when the visualizer is being disposed.
+     * Clean up resources when the config element is being disposed.
      */
     public void dispose() {
         if (chatPanel != null) {

src/main/java/org/apache/jmeter/plugins/copilot/CopilotChatTestElement.java

@@ -16,13 +16,13 @@
  */
 package org.apache.jmeter.plugins.copilot;
 
-import org.apache.jmeter.testelement.AbstractTestElement;
+import org.apache.jmeter.config.ConfigTestElement;
 
 /**
- * Test element that holds configuration for the Copilot Chat visualizer.
- * This element stores settings like the selected AI model and chat history.
+ * Test element that holds configuration for the Copilot Chat config element.
+ * This element stores settings like the selected AI model.
  */
-public class CopilotChatTestElement extends AbstractTestElement {
+public class CopilotChatTestElement extends ConfigTestElement {
 
     private static final long serialVersionUID = 1L;
 

src/main/java/org/apache/jmeter/plugins/copilot/gui/CopilotChatPanel.java

@@ -22,8 +22,6 @@
 import java.awt.FlowLayout;
 import java.awt.Font;
 import java.awt.event.ActionEvent;
-import java.awt.event.KeyAdapter;
-import java.awt.event.KeyEvent;
 import java.io.File;
 import java.nio.charset.StandardCharsets;
 import java.nio.file.Files;
@@ -169,12 +167,9 @@ private JPanel createInputPanel() {
         // Input field
         inputField = new JTextField();
         inputField.setFont(new Font(Font.SANS_SERIF, Font.PLAIN, 14));
-        inputField.addKeyListener(new KeyAdapter() {
-            @Override
-            public void keyPressed(KeyEvent e) {
-                if (e.getKeyCode() == KeyEvent.VK_ENTER && !isProcessing) {
-                    sendMessage();
-                }
+        inputField.addActionListener(e -> {
+            if (!isProcessing) {
+                sendMessage();
             }
         });