[Refactor] Remover dependência do Modular `ChatMainTalksPage` #395

RoxyRoses · 2025-01-14T15:07:51Z

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

Refatoração de ChatMainPage:
- Adicionada uma propriedade obrigatória iChatMainTalksController.
- Repasse do controller para o ChatMainTalksPage.
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.
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

lib/app/features/chat/presentation/chat_main_module.dart
Alterado para passar o IChatMainTalksController na criação da ChatMainPage.
lib/app/features/chat/presentation/chat_main_page.dart
Adicionado o parâmetro obrigatório iChatMainTalksController na ChatMainPage.
lib/app/features/chat/presentation/talk/chat_main_talks_page.dart
Alterado para aceitar um IChatMainTalksController explicitamente na construção da página.
test/app/features/chat/presentation/talk/chat_main_talks_page_test.dart
Adicionadas as configurações de teste para o novo design.
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.

… into 366-substituir-modularstate-por-state-em-chatmaintalkspage

…tmaintalkspage

sonarqubecloud · 2025-01-16T13:05:52Z

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 · 2025-01-16T13:05:56Z

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

View more on Code Climate.

shonorio

Necessita algumas correções

shonorio · 2025-01-25T18:42:34Z

lib/app/features/chat/presentation/chat_main_module.dart

+        chatMainPeopleController: ChatMainPeopleController(
+            skillRepository: Modular.get<IFilterSkillRepository>(),
+            usersRepository: Modular.get<IUsersRepository>()),
+        controller: ChatMainController(
+            chatToggleFeature: ChatPrivateToggleFeature(
+                modulesServices: Modular.get<IAppModulesServices>())),
+        chatMainTalksController: ChatMainTalksController(
+            chatChannelRepository: Modular.get<IChatChannelRepository>()),


Para melhorar a lógica, é melhor manter a criação do controllers no Bind (onde já estão) e aqui consumir a injeção, melhorando a organização e a manutenibilidade do código.

Suggested change

chatMainPeopleController: ChatMainPeopleController(

skillRepository: Modular.get<IFilterSkillRepository>(),

usersRepository: Modular.get<IUsersRepository>()),

controller: ChatMainController(

chatToggleFeature: ChatPrivateToggleFeature(

modulesServices: Modular.get<IAppModulesServices>())),

chatMainTalksController: ChatMainTalksController(

chatChannelRepository: Modular.get<IChatChannelRepository>()),

chatMainPeopleController: Modular.get<ChatMainPeopleController>(),

controller: Modular.get<ChatMainController>(),

chatMainTalksController: Modular.get<ChatMainTalksController>(),

shonorio · 2025-01-25T19:06:53Z

lib/app/features/chat/presentation/talk/chat_main_talks_page.dart

+  @override
+  void initState() {
+    super.initState();
+    controller.initialize();


Melhor garantir que o controller seja acionado apenas quando as Widgets estiverem prontas para isto

Suggested change

controller.initialize();

WidgetsBinding.instance.addPostFrameCallback((_) {

controller.initialize();

});

shonorio · 2025-01-25T19:14:50Z

lib/app/features/support_center/presentation/support_center_module.dart

+        controller: SupportCenterController(
+            supportCenterUseCase: SupportCenterUseCase(
+                locationService: Modular.get<ILocationServices>(),
+                supportCenterRepository: SupportCenterRepository(
+                    apiProvider: Modular.get<ApiProvider>()))),


Não entendi a necessidade de alterar este código, não entendi a relação desta alteração com o PR

reafactor modular state chat main module

ca7a23c

RoxyRoses added deprecation Para destacar o uso de componentes obsoletos refactor Envolve melhorias no código, como substituir o componentes obsoletos labels Jan 14, 2025

RoxyRoses self-assigned this Jan 14, 2025

RoxyRoses linked an issue Jan 14, 2025 that may be closed by this pull request

Substituir ModularState por State em ChatMainTalksPage #366

Open

RoxyRoses added 6 commits January 14, 2025 14:10

Merge branch 'main' of github.com-personal:institutoazmina/penhas-app…

f694e4e

… into 366-substituir-modularstate-por-state-em-chatmaintalkspage

refactor test

c1ed3ed

fix mock

68735eb

correcao testes em desenv

4705308

fix testes

e1173d2

fix lint e widget module nao encontrando widgets

8986620

RoxyRoses marked this pull request as ready for review January 15, 2025 19:26

RoxyRoses requested a review from a team as a code owner January 15, 2025 19:26

Merge branch 'main' into 366-substituir-modularstate-por-state-em-cha…

60d1946

…tmaintalkspage

RoxyRoses requested review from shonorio and pedrox-hs January 17, 2025 01:54

shonorio reviewed Jan 25, 2025

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Refactor] Remover dependência do Modular `ChatMainTalksPage` #395

[Refactor] Remover dependência do Modular `ChatMainTalksPage` #395

RoxyRoses commented Jan 14, 2025

sonarqubecloud bot commented Jan 16, 2025

codeclimate bot commented Jan 16, 2025

shonorio left a comment

shonorio Jan 25, 2025

shonorio Jan 25, 2025

shonorio Jan 25, 2025

[Refactor] Remover dependência do Modular ChatMainTalksPage #395

Are you sure you want to change the base?

[Refactor] Remover dependência do Modular ChatMainTalksPage #395

Conversation

RoxyRoses commented Jan 14, 2025

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

Resumo

Alterações principais

Arquivos modificados

Diferença de código (Git Diff)

sonarqubecloud bot commented Jan 16, 2025

Quality Gate passed

codeclimate bot commented Jan 16, 2025

shonorio left a comment

Choose a reason for hiding this comment

shonorio Jan 25, 2025

Choose a reason for hiding this comment

shonorio Jan 25, 2025

Choose a reason for hiding this comment

shonorio Jan 25, 2025

Choose a reason for hiding this comment

[Refactor] Remover dependência do Modular `ChatMainTalksPage` #395

[Refactor] Remover dependência do Modular `ChatMainTalksPage` #395