[Refactor] Remover dependência do Modular ChatMainTalksPage

[RoxyRoses]

Proposta de Pull Request: Refatoração do ChatMainTalksPage para incluir IChatMainTalksController como dependência

Resumo

Este pull request faz alterações na implementação de ChatMainPage e ChatMainTalksPage para utilizar a injeção de dependência de um IChatMainTalksController. Isso melhora a testabilidade e desacopla ainda mais o código.

Alterações principais

1. Refatoração de ChatMainPage:

   • Adicionada uma propriedade obrigatória iChatMainTalksController.
   • Repasse do controller para o ChatMainTalksPage.

2. Modificação em ChatMainTalksPage:

   • Introduzido o controller como uma dependência obrigatória.
   • Removida a herança de ModularState.
   • Modificada a lógica de construção para referenciar o controller da propriedade.

3. Atualização de testes:

   • Criado um MockIChatMainTalksController para substituir dependências reais nos testes.
   • Adicionados casos de teste utilizando estados como ChatMainTalksState.error e ChatMainTalksState.loaded.
   • Refatorados widgets de teste para usar a nova assinatura.

Arquivos modificados

1. lib/app/features/chat/presentation/chat_main_module.dart
   Alterado para passar o IChatMainTalksController na criação da ChatMainPage.

2. lib/app/features/chat/presentation/chat_main_page.dart
   Adicionado o parâmetro obrigatório iChatMainTalksController na ChatMainPage.

3. lib/app/features/chat/presentation/talk/chat_main_talks_page.dart
   Alterado para aceitar um IChatMainTalksController explicitamente na construção da página.

4. test/app/features/chat/presentation/talk/chat_main_talks_page_test.dart
   Adicionadas as configurações de teste para o novo design.

5. Goldens atualizados:

   • test/app/features/chat/presentation/talk/goldens/ci/chat_main_talks_page_error_state.png
   • test/app/features/chat/presentation/talk/goldens/ci/chat_main_talks_page_initial_state.png

Diferença de código (Git Diff)

# lib/app/features/chat/presentation/chat_main_module.dart
...
-  Widget get view => const ChatMainPage();
+  Widget get view => ChatMainPage(iChatMainTalksController: Modular.get<IChatMainTalksController>(),);
...

# lib/app/features/chat/presentation/chat_main_page.dart
...
-  const ChatMainPage({Key? key}) : super(key: key);
+  const ChatMainPage({Key? key, required this.iChatMainTalksController}) : super(key: key);
...
+  final IChatMainTalksController iChatMainTalksController;

# lib/app/features/chat/presentation/talk/chat_main_talks_page.dart
...
-  const ChatMainTalksPage({Key? key}) : super(key: key);
+  const ChatMainTalksPage({Key? key, required this.controller}) : super(key: key);
...
- class _ChatMainTalksPageState extends ModularState<ChatMainTalksPage, IChatMainTalksController> {
+ class _ChatMainTalksPageState extends State<ChatMainTalksPage> {
...
+  IChatMainTalksController get controller => widget.controller;

# test/app/features/chat/presentation/talk/chat_main_talks_page_test.dart
...
+class MockIChatMainTalksController extends Mock implements IChatMainTalksController {}
...
+      when(() => controller.currentState)
+          .thenAnswer((_) => ChatMainTalksState.error('error'));
...

# Goldens atualizados
- Arquivos visuais de estados como `error` e `initial` foram atualizados para refletir as mudanças de layout.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[codeclimate]

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

View more on Code Climate.