Skip to content
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

[refactor] Remover dependência do Modular ZodiacPage #384

Merged
merged 3 commits into from
Jan 13, 2025

Conversation

shonorio
Copy link
Collaborator

Este PR introduz alterações na página de zodíaco (zodiac_page.dart) e no módulo de zodíaco (zodiac_module.dart), além de ajustes nos testes relacionados. As mudanças visam simplificar a injeção de dependências, melhorar a organização do código e garantir a consistência dos testes.

Principais Alterações:

  1. Refatoração da Injeção de Dependências:

    • Remoção da dependência direta do Modular na página ZodiacPage.
    • O controlador (ZodiacController) agora é injetado diretamente no construtor da página, tornando o código mais explícito e testável.
    • Ajuste no ZodiacModule para passar o controlador corretamente ao construir a página.
  2. Simplificação do Estado da Página:

    • Remoção da herança de ModularState e substituição por um State comum, já que o controlador é injetado diretamente.
    • Adição de um getter para acessar o controlador (controller) de forma mais clara.
  3. Correção de Texto:

    • Correção do texto "Diário astrólogico" para "Diário astrológico", mantendo a ortografia correta.
  4. Refatoração dos Testes:

    • Remoção da configuração do Modular nos testes, simplificando a inicialização do controlador.
    • O controlador é criado diretamente nos testes, eliminando a necessidade de mocks complexos e inicialização de módulos.
    • Ajuste nos testes para utilizar a nova forma de injeção do controlador.
  5. Testes de Screenshot:

    • O teste de screenshot foi atualizado para utilizar a nova forma de construção da página, garantindo que a renderização continue consistente.

Issues:

Fixes #331

@shonorio shonorio requested a review from a team as a code owner January 12, 2025 14:36
@shonorio shonorio self-assigned this Jan 12, 2025
@shonorio shonorio added dependencies Pull requests that update a dependency file deprecation Para destacar o uso de componentes obsoletos refactor Envolve melhorias no código, como substituir o componentes obsoletos labels Jan 12, 2025
@shonorio shonorio changed the title Refactor Remover dependencias do Modular ZodiacPage Refactor Remover dependência do Modular ZodiacPage Jan 12, 2025
@shonorio shonorio changed the title Refactor Remover dependência do Modular ZodiacPage [refactor] Remover dependência do Modular ZodiacPage Jan 12, 2025
- Removida a dependência do Modular e do `ModularState` na `ZodiacPage`.
- Injetado o `ZodiacController` diretamente na `ZodiacPage`.
- Adicionado getter para acessar o controller no estado da página.
- Removido o parâmetro `title` não utilizado.
- Corrigido texto do botão de "Diário astrólogico" para "Diário astrológico".
Copy link

codeclimate bot commented Jan 13, 2025

Code Climate has analyzed commit d4573c1 and detected 0 issues on this pull request.

View more on Code Climate.

@shonorio shonorio merged commit ece955a into main Jan 13, 2025
8 checks passed
@shonorio shonorio deleted the shonorio/issue331 branch January 13, 2025 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file deprecation Para destacar o uso de componentes obsoletos refactor Envolve melhorias no código, como substituir o componentes obsoletos
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Substituir ModularState por State e injetar ZodiacController via parâmetro no ZodiacPage
2 participants