-
-
Notifications
You must be signed in to change notification settings - Fork 40
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.
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
384 e2e gen add retry attempts for failed tasks #385
384 e2e gen add retry attempts for failed tasks #385
Conversation
Code Review❗ Attention Required: This PR has potential issues. 🚨 Class Import ChangeThe import statement has been changed from UITestGenerator to E2ETestGenerator.Potential Solution: Ensure that the E2ETestGenerator class exists and provides the same or enhanced functionality as the UITestGenerator.
Method Name UpdateThe method call has been changed from generate_ui_tests to generate_e2e_tests.Potential Solution: Verify that the generate_e2e_tests method exists in the E2ETestGenerator class and provides the expected functionality.
DocumentationChanges made to Sensetive filePotential Solution: NA
Useful Commands
|
@@ -1,7 +1,7 @@ | |||
import click | |||
import os | |||
import json | |||
from kaizen.generator.ui import UITestGenerator | |||
from kaizen.generator.e2e_tests import E2ETestGenerator |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Comment: The import statement has been changed from UITestGenerator to E2ETestGenerator.
Solution: Ensure that the E2ETestGenerator class exists and provides the same or enhanced functionality as the UITestGenerator.
Reason For Comment: This change affects the core functionality of the test generation process.
@@ -82,7 +82,7 @@ | |||
@click.argument("url", required=True) | |||
def ui_tests(url): | |||
"""Run ui test generation""" | |||
UITestGenerator().generate_ui_tests(url) | |||
E2ETestGenerator().generate_e2e_tests(url) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Comment: The method call has been changed from generate_ui_tests to generate_e2e_tests.
Solution: Verify that the generate_e2e_tests method exists in the E2ETestGenerator class and provides the expected functionality.
Reason For Comment: This change affects how tests are generated and may impact existing workflows.
@@ -75,7 +75,7 @@ Kaizen is an open-source AI-powered suite that revolutionizes your code quality | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Comment: Changes made to Sensetive file
Solution: NA
Reason For Comment: Changes were made to README.md, which needs review
E2E Test Generation with Retry Attempts
Overview
This pull request introduces E2E test generation with retry attempts for failed tasks, enhancing the robustness of the testing framework. It refactors the codebase to utilize a new
E2ETestGenerator
class and updates the CLI to reflect these changes.Changes
UITestGenerator
class with a newE2ETestGenerator
class.E2ETestGenerator
class and retry attempts.E2ETestGenerator
class.Original Description
None