78025209dc
## Description Updated the tone to reflect a professional development environment instead of a teacher in a classroom. Fixes #`36934` ## Automation /ok-to-test tags="@tag.IDE" ### 🔍 Cypress test results <!-- This is an auto-generated comment: Cypress test results --> > [!WARNING] > Workflow run: <https://github.com/appsmithorg/appsmith/actions/runs/11381087559> > Commit: d07f1e616cd49f69001cba3f9a10d13fc11a2286 > <a href="https://internal.appsmith.com/app/cypress-dashboard/rundetails-65890b3c81d7400d08fa9ee5?branch=master&workflowId=11381087559&attempt=1" target="_blank">Cypress dashboard</a>. > Tags: @tag.IDE > Spec: > It seems like **no tests ran** 😔. We are not able to recognize it, please check <a href="https://github.com/appsmithorg/appsmith/actions/runs/11381087559" target="_blank">workflow here</a>. > <hr>Thu, 17 Oct 2024 08:31:44 UTC <!-- end of auto-generated comment: Cypress test results --> ## Communication Should the DevRel and Marketing teams inform users about this change? - [ ] Yes - [x] No --------- Co-authored-by: Arpit Mohan <mohanarpit@users.noreply.github.com>
39 lines
1.7 KiB
YAML
39 lines
1.7 KiB
YAML
# yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json
|
|
language: "en-US"
|
|
early_access: false
|
|
tone_instructions: 'You are an expert code reviewer with expertise in Java, Typescript, Javascript and NodeJS. You work in a professional software development team. You give code review suggestions in a concise fashion. You don't elaborate unless specifically asked to do so.'
|
|
reviews:
|
|
profile: "chill"
|
|
request_changes_workflow: false
|
|
high_level_summary: true
|
|
poem: true
|
|
review_status: true
|
|
collapse_walkthrough: false
|
|
auto_review:
|
|
enabled: true
|
|
drafts: false
|
|
base_branches: ["pg", "release"]
|
|
path_instructions:
|
|
- path: "app/client/cypress/**/**.*"
|
|
instructions: |
|
|
Review the following e2e test code written using the Cypress test library. Ensure that:
|
|
- Follow best practices for Cypress code and e2e automation.
|
|
- Avoid using cy.wait in code.
|
|
- Avoid using cy.pause in code.
|
|
- Avoid using agHelper.sleep().
|
|
- Use locator variables for locators and do not use plain strings.
|
|
- Use data-* attributes for selectors.
|
|
- Avoid Xpaths, Attributes and CSS path.
|
|
- Avoid selectors like .btn.submit or button[type=submit].
|
|
- Perform logins via API with LoginFromAPI.
|
|
- Perform logout via API with LogOutviaAPI.
|
|
- Perform signup via API with SignupFromAPI.
|
|
- Avoid using it.only.
|
|
- Avoid using after and aftereach in test cases.
|
|
- Use multiple assertions for expect statements.
|
|
- Avoid using strings for assertions.
|
|
- Do not use duplicate filenames even with different paths.
|
|
- Avoid using agHelper.Sleep, this.Sleep in any file in code.
|
|
chat:
|
|
auto_reply: true
|