Skip to content

Automatiza teste para devolver processo à fase 'Análise Jurídica'#391

Merged
jgaguiarm merged 4 commits into
developfrom
feature/automates-test-return-proccess-legal-analysis-tab
Jul 14, 2026
Merged

Automatiza teste para devolver processo à fase 'Análise Jurídica'#391
jgaguiarm merged 4 commits into
developfrom
feature/automates-test-return-proccess-legal-analysis-tab

Conversation

@jgaguiarm

@jgaguiarm jgaguiarm commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

✅ Descrição do propósito desse Pull Request

Automatiza teste para devolver processo à fase 'Análise Jurídica'

🧭 Referência a Issue

#382

❓ O que foi feito para atingir isso?


🏃‍♀️ Tipo de mudança

Marque as opções relevantes:

  • Bug fix (correção de bug)
  • Nova feature (mudança não retrocompatível que adiciona funcionalidade)
  • Mudança de breaking (correção ou feature que faria com que a funcionalidade existente não funcionasse como esperado)
  • Documentação (somente mudanças ou atualizações na documentação)

🕵️ Como foi testado?

  • Critério de aceitação
  • Testes de software (TDD, BDD, UNITÁRIO, INTEGRAÇÃO, E2E)

Checklist: ✔️

  • Meu código segue as diretrizes do projeto
  • Eu fiz um code review com minha equipe
  • Eu comentei meu código, especialmente em áreas de difícil entendimento
  • Eu atualizei a documentação correspondente
  • Testes novos e existentes passaram localmente com minhas alterações

Observação:

Summary by CodeRabbit

  • New Features

    • Added the ability to return a process from Formalization to the Legal Analysis phase.
    • Users can provide a return motive before submitting the process.
    • Added confirmation feedback when the return is completed successfully.
  • Bug Fixes

    • Improved navigation to ensure the Formalization tab opens correctly when viewing project details.

@jgaguiarm
jgaguiarm requested a review from Junior-Shyko July 10, 2026 19:53
@coderabbitai

coderabbitai Bot commented Jul 10, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 2f41c1c6-a764-4910-afd9-3d3f75618c9a

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/automates-test-return-proccess-legal-analysis-tab

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🧹 Nitpick comments (2)
cypress/pages/project/formalizationTab/elements.js (1)

5-7: 📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low value

Fix typographical error in selector variable name. The variable name contains a typo (retunr instead of return).

  • cypress/pages/project/formalizationTab/elements.js#L5-L7: rename the property to returnProcessSentButton.
  • cypress/pages/project/formalizationTab/FormalizationTab.js#L58-L60: update the usage to el.returnProcessSentButton.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@cypress/pages/project/formalizationTab/elements.js` around lines 5 - 7,
Rename the misspelled retunrProcessSentButton property to
returnProcessSentButton in cypress/pages/project/formalizationTab/elements.js
(lines 5-7), and update its usage in
cypress/pages/project/formalizationTab/FormalizationTab.js (lines 58-60) to
reference el.returnProcessSentButton.
cypress/support/workflows/FormalizationWorkflow.js (1)

93-100: 📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low value

Fix typographical error in workflow method name. The method name contains a typo (Leal instead of Legal).

  • cypress/support/workflows/FormalizationWorkflow.js#L93-L100: rename the method to returnProcessToLegalAnalysisTab.
  • cypress/e2e/efomento/projects/formalization.cy.js#L48-L58: update the caller to FormalizationWorkflow.returnProcessToLegalAnalysisTab.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@cypress/support/workflows/FormalizationWorkflow.js` around lines 93 - 100,
Rename the FormalizationWorkflow method returnProcessToLealAnalysisTab to
returnProcessToLegalAnalysisTab in
cypress/support/workflows/FormalizationWorkflow.js (lines 93-100), and update
its caller in cypress/e2e/efomento/projects/formalization.cy.js (lines 48-58) to
use the corrected method name.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@cypress/pages/project/ProjectPage.js`:
- Line 24: Update the URL assertion in the shared goToProjectDetailsPage method
to avoid hardcoding a specific tab query parameter; make the match tab-agnostic
so any active tab is accepted, or support an optional expected-tab argument
while preserving existing callers.

---

Nitpick comments:
In `@cypress/pages/project/formalizationTab/elements.js`:
- Around line 5-7: Rename the misspelled retunrProcessSentButton property to
returnProcessSentButton in cypress/pages/project/formalizationTab/elements.js
(lines 5-7), and update its usage in
cypress/pages/project/formalizationTab/FormalizationTab.js (lines 58-60) to
reference el.returnProcessSentButton.

In `@cypress/support/workflows/FormalizationWorkflow.js`:
- Around line 93-100: Rename the FormalizationWorkflow method
returnProcessToLealAnalysisTab to returnProcessToLegalAnalysisTab in
cypress/support/workflows/FormalizationWorkflow.js (lines 93-100), and update
its caller in cypress/e2e/efomento/projects/formalization.cy.js (lines 48-58) to
use the corrected method name.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 08ad0a6d-cf31-4212-b9c2-322546c26535

📥 Commits

Reviewing files that changed from the base of the PR and between 311e659 and 1b5923d.

📒 Files selected for processing (9)
  • cypress/e2e/efomento/projects/formalization.cy.js
  • cypress/pages/project/ProjectPage.js
  • cypress/pages/project/formalizationTab/FormalizationTab.js
  • cypress/pages/project/formalizationTab/elements.js
  • cypress/pages/project/formalizationTab/index.js
  • cypress/support/constants/documents.js
  • cypress/support/workflows/FormalizationWorkflow.js
  • resources/js/Components/ReturnProcessModal.vue
  • resources/js/Pages/ProjectDetails/Partials/Tabs/FormalizationTab.vue
💤 Files with no reviewable changes (1)
  • cypress/pages/project/formalizationTab/index.js

Comment thread cypress/pages/project/ProjectPage.js Outdated
@jgaguiarm
jgaguiarm merged commit 0c25f06 into develop Jul 14, 2026
4 checks passed
@jgaguiarm
jgaguiarm deleted the feature/automates-test-return-proccess-legal-analysis-tab branch July 14, 2026 17:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants