Skip to content

Ai gen testcase Issues #4001

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: development
Choose a base branch
from
Open

Ai gen testcase Issues #4001

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
87 changes: 59 additions & 28 deletions ...eatures/rest-explorer/components/request-tests/sub-components/pre-script/PreScript.svelte
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@

// Persistent highlighting variables
let observer: MutationObserver | null = null;
let highlightInterval: number | null = null;
let highlightInterval: ReturnType<typeof setInterval> | null = null;
let rafId: number | null = null;
let clickHandlers: (() => void)[] = [];
let isUserLimitReached: boolean = false;
Expand Down Expand Up @@ -97,8 +97,32 @@
const handleCodeMirrorChange = (e: any) => {
const newContent = e.detail;

// If we have generated test actions showing, protect the generated content
// If we have generated test actions showing, check if generated content still exists
if (showGeneratedTestActions) {
const generatedText = generatedTestContent.trim().replace(/\s+/g, "");
const newContentNormalized = newContent.replace(/\s+/g, "");

// Check if the generated content was completely removed by the user
if (
generatedText.length > 0 &&
!newContentNormalized.includes(generatedText)
) {
// User has deleted the generated content - hide buttons and reset
showGeneratedTestActions = false;
removeHighlight();
generatedTestContent = "";
temporaryDisplayContent = "";
currentPrompt = "";
originalLineCount = 0;
generatedContentStartLine = 0;
generatedContentEndLine = 0;
contentAddedDuringGeneration = "";

// Save the new content normally
onTestsChange({ ...tests, preScript: newContent });
return;
}

const newLines = newContent.split("\n");

// Check if lines in the generated range have been modified
Expand Down Expand Up @@ -273,15 +297,26 @@
$: rightPanelWidth = isLeftPanelCollapsed ? "calc(100% - 60px)" : "75%";

const handleGenerateTestCases = async () => {
// Validate input: trim and check for empty or special characters only
const trimmedPrompt = testCasePrompt.trim();

// Check if prompt is empty or contains only special characters (no alphanumeric)
if (!trimmedPrompt || !/[a-zA-Z0-9]/.test(trimmedPrompt)) {
isError = true;
errorMessage =
"Please enter a valid prompt with alphanumeric characters.";
return;
}

// Store the original content and current prompt
originalTestContent = tests?.preScript || "";
contentAddedDuringGeneration = "";
originalLineCount = originalTestContent.trim()
? originalTestContent.split("\n").length
: 0;
currentPrompt = testCasePrompt;
currentPrompt = trimmedPrompt;

const result = await onGeneratePreScript(testCasePrompt);
const result = await onGeneratePreScript(trimmedPrompt);
if (result?.error) {
if (result?.message === "Limit reached. Please try again later.") {
isUserLimitReached = true;
Expand Down Expand Up @@ -513,8 +548,13 @@
const acceptGeneratedTest = () => {
removeHighlight();

// Ensure we're only working with the current displayed content
const contentToSave = showGeneratedTestActions
? temporaryDisplayContent
: tests?.preScript || "";

// Save the complete content (original + snippets + generated)
onTestsChange({ ...tests, preScript: temporaryDisplayContent });
onTestsChange({ ...tests, preScript: contentToSave });

// Clear temporary state
showGeneratedTestActions = false;
Expand All @@ -531,19 +571,23 @@
const rejectGeneratedTest = () => {
removeHighlight();

// Calculate what content to revert to:
// Original content + any snippets added during generation
let revertedContent = originalTestContent || "";
// If buttons are showing, revert to original + snippets
// Otherwise, just clear the state (shouldn't happen, but safety check)
if (showGeneratedTestActions) {
// Calculate what content to revert to:
// Original content + any snippets added during generation
let revertedContent = originalTestContent || "";

if (contentAddedDuringGeneration) {
revertedContent = revertedContent
? `${revertedContent}\n${contentAddedDuringGeneration}`
: contentAddedDuringGeneration;
}

if (contentAddedDuringGeneration) {
revertedContent = revertedContent
? `${revertedContent}\n${contentAddedDuringGeneration}`
: contentAddedDuringGeneration;
// Save the reverted content (original + snippets, but no AI generation)
onTestsChange({ ...tests, preScript: revertedContent });
}

// Save the reverted content (original + snippets, but no AI generation)
onTestsChange({ ...tests, preScript: revertedContent });

// Clear all temporary state
showGeneratedTestActions = false;
generatedTestContent = "";
Expand Down Expand Up @@ -886,12 +930,6 @@
color: var(--text-ds-neutral-50);
}

.results-count {
font-weight: 400;
color: var(--text-ds-neutral-400);
font-size: 11px;
}

.snippet-suggestion-container {
padding: 8px;
cursor: pointer;
Expand Down Expand Up @@ -936,13 +974,6 @@
margin: 0 0 8px 0;
}

.no-results-suggestion {
font-weight: 400;
font-size: 11px;
color: var(--text-ds-neutral-400);
margin: 0;
}

:global(.search-highlight) {
background-color: transparent;
color: var(--text-ds-neutral-50);
Expand Down
87 changes: 59 additions & 28 deletions ...s/src/features/rest-explorer/components/request-tests/sub-components/script/Script.svelte
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@

// Persistent highlighting variables
let observer: MutationObserver | null = null;
let highlightInterval: number | null = null;
let highlightInterval: ReturnType<typeof setInterval> | null = null;
let rafId: number | null = null;
let clickHandlers: (() => void)[] = [];
let isUserLimitReached: boolean = false;
Expand Down Expand Up @@ -97,8 +97,32 @@
const handleCodeMirrorChange = (e: any) => {
const newContent = e.detail;

// If we have generated test actions showing, protect the generated content
// If we have generated test actions showing, check if generated content still exists
if (showGeneratedTestActions) {
const generatedText = generatedTestContent.trim().replace(/\s+/g, "");
const newContentNormalized = newContent.replace(/\s+/g, "");

// Check if the generated content was completely removed by the user
if (
generatedText.length > 0 &&
!newContentNormalized.includes(generatedText)
) {
// User has deleted the generated content - hide buttons and reset
showGeneratedTestActions = false;
removeHighlight();
generatedTestContent = "";
temporaryDisplayContent = "";
currentPrompt = "";
originalLineCount = 0;
generatedContentStartLine = 0;
generatedContentEndLine = 0;
contentAddedDuringGeneration = "";

// Save the new content normally
onTestsChange({ ...tests, script: newContent });
return;
}

const newLines = newContent.split("\n");

// Check if lines in the generated range have been modified
Expand Down Expand Up @@ -277,15 +301,26 @@
$: rightPanelWidth = isLeftPanelCollapsed ? "calc(100% - 60px)" : "75%";

const handleGenerateTestCases = async () => {
// Validate input: trim and check for empty or special characters only
const trimmedPrompt = testCasePrompt.trim();

// Check if prompt is empty or contains only special characters (no alphanumeric)
if (!trimmedPrompt || !/[a-zA-Z0-9]/.test(trimmedPrompt)) {
isError = true;
errorMessage =
"Please enter a valid prompt with alphanumeric characters.";
return;
}

// Store the original content and current prompt
originalTestContent = tests?.script || "";
contentAddedDuringGeneration = "";
originalLineCount = originalTestContent.trim()
? originalTestContent.split("\n").length
: 0;
currentPrompt = testCasePrompt;
currentPrompt = trimmedPrompt;

const result = await onGenerateTestCases(testCasePrompt);
const result = await onGenerateTestCases(trimmedPrompt);
if (result?.error) {
if (result?.message === "Limit reached. Please try again later.") {
isUserLimitReached = true;
Expand Down Expand Up @@ -517,8 +552,13 @@
const acceptGeneratedTest = () => {
removeHighlight();

// Ensure we're only working with the current displayed content
const contentToSave = showGeneratedTestActions
? temporaryDisplayContent
: tests?.script || "";

// Save the complete content (original + snippets + generated)
onTestsChange({ ...tests, script: temporaryDisplayContent });
onTestsChange({ ...tests, script: contentToSave });

// Clear temporary state
showGeneratedTestActions = false;
Expand All @@ -535,19 +575,23 @@
const rejectGeneratedTest = () => {
removeHighlight();

// Calculate what content to revert to:
// Original content + any snippets added during generation
let revertedContent = originalTestContent || "";
// If buttons are showing, revert to original + snippets
// Otherwise, just clear the state (shouldn't happen, but safety check)
if (showGeneratedTestActions) {
// Calculate what content to revert to:
// Original content + any snippets added during generation
let revertedContent = originalTestContent || "";

if (contentAddedDuringGeneration) {
revertedContent = revertedContent
? `${revertedContent}\n${contentAddedDuringGeneration}`
: contentAddedDuringGeneration;
}

if (contentAddedDuringGeneration) {
revertedContent = revertedContent
? `${revertedContent}\n${contentAddedDuringGeneration}`
: contentAddedDuringGeneration;
// Save the reverted content (original + snippets, but no AI generation)
onTestsChange({ ...tests, script: revertedContent });
}

// Save the reverted content (original + snippets, but no AI generation)
onTestsChange({ ...tests, script: revertedContent });

// Clear all temporary state
showGeneratedTestActions = false;
generatedTestContent = "";
Expand Down Expand Up @@ -890,12 +934,6 @@
color: var(--text-ds-neutral-50);
}

.results-count {
font-weight: 400;
color: var(--text-ds-neutral-400);
font-size: 11px;
}

.snippet-suggestion-container {
padding: 8px;
cursor: pointer;
Expand Down Expand Up @@ -940,13 +978,6 @@
margin: 0 0 8px 0;
}

.no-results-suggestion {
font-weight: 400;
font-size: 11px;
color: var(--text-ds-neutral-400);
margin: 0;
}

:global(.search-highlight) {
background-color: transparent;
color: var(--text-ds-neutral-50);
Expand Down
Loading