Development#30
Conversation
Corrigindo referência de versão dos pacotes relacionados ao framework de testes.
- Atualizado o sdk do framework de testes para a versão 3.9.3
WalkthroughTodos os arquivos de projeto de testes unitários foram atualizados para utilizar a versão 3.9.3 do MSTest SDK. Em alguns projetos, outros pacotes relacionados ao MSTest também foram atualizados para a mesma versão, além da atualização do Microsoft.NET.Test.Sdk em um caso específico. Não houve alterações em propriedades, referências ou estrutura dos projetos. Changes
Estimated code review effort1 (~5 minutos) Possibly related PRs
Poem
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
♻️ Duplicate comments (2)
InvoiceReminder.Application.UnitTests/InvoiceReminder.Application.UnitTests.csproj (2)
49-50: Veja comentário anterior sobre redundância do TestAdapter
53-54: Veja comentário anterior sobre redundância do TestFramework
🧹 Nitpick comments (12)
InvoiceReminder.DomainEntities.UnitTests/InvoiceReminder.DomainEntities.UnitTests.csproj (1)
40-52: Considere centralizar as versões de pacotes de testeComo vários projetos do repositório atualizam o trio
MSTest.Analyzers / TestAdapter / TestFrameworkpara a mesma versão, vale avaliar mover essas referências para umDirectory.Packages.props(ouDirectory.Build.props) na raiz da solução. Isso:
- Reduz duplicação de XML nos csproj.
- Evita drift de versões futuramente.
- Facilita bump único de versão nos pipelines.
Nenhuma ação obrigatória agora, apenas recomendação para o próximo ciclo de limpeza.
InvoiceReminder.JobScheduler.UnitTests/InvoiceReminder.JobScheduler.UnitTests.csproj (2)
15-17: Verifique se a dupla MSTest.Sdk × Microsoft.NET.Test.Sdk está realmente necessáriaComo o
MSTest.Sdkjá traz transitivamente oMicrosoft.NET.Test.Sdk, manter oPackageReference Update="Microsoft.NET.Test.Sdk"pode ser redundante. Se o objetivo era apenas garantir a versão 17.14.1, tudo bem — mas confirme se não há conflitos de binding no restore, principalmente agora que oMSTest.Sdkfoi atualizado.
28-40: Referências explícitas aos pacotes MSTest podem ser simplificadasCom o uso do
MSTest.Sdk, as dependênciasMSTest.TestAdapter,MSTest.TestFrameworkeMSTest.Analyzersjá vêm inclusas em versão compatível. Manter essesPackageReference Updatesó faz sentido quando se precisa forçar um pin exato. Caso contrário, considere removê-los para reduzir ruído no arquivo:- <PackageReference Update="MSTest.Analyzers" Version="3.9.3"> … </PackageReference> … - <PackageReference Update="MSTest.TestAdapter" Version="3.9.3" /> … - <PackageReference Update="MSTest.TestFramework" Version="3.9.3" />Isso deixa a manutenção de versões centralizada apenas na linha do SDK.
InvoiceReminder.Application.UnitTests/InvoiceReminder.Application.UnitTests.csproj (2)
30-30: Possível sobreposição entre MSTest.Sdk e Microsoft.NET.Test.SdkO
MSTest.Sdkjá traz oMicrosoft.NET.Test.Sdkcomo dependência transitiva; manter ambos pode resultar em conflitos de versão ou restauração desnecessária. Considere remover oUpdateexplícito ou alinhar as versões para evitar “asset clashes”.
42-45: Pacotes MSTest. podem ser redundantes*Se optar por continuar com o
MSTest.Sdk, os pacotesMSTest.Analyzers,MSTest.TestAdaptereMSTest.TestFrameworksão resolvidos automaticamente. Manter referências explícitas aumenta o tempo de restauração e gera ruído de versão.- <PackageReference Update="MSTest.Analyzers" Version="3.9.3"> … </PackageReference> - <PackageReference Update="MSTest.TestAdapter" Version="3.9.3" /> - <PackageReference Update="MSTest.TestFramework" Version="3.9.3" /> + <!-- Dependências já trazidas pelo MSTest.Sdk -->InvoiceReminder.ArchitectureTests/InvoiceReminder.ArchitectureTests.csproj (1)
28-28: Versões 17.14. quase alinhadas—avalie atualizar para 17.14.2 se disponível*
OMicrosoft.NET.Test.Sdkestá em 17.14.1, enquanto as extensões de CodeCoverage usam 17.14.2. Caso exista o patch 17.14.2 também para o Test Sdk, considere alinhar para evitar advertências de restauração.InvoiceReminder.Infrastructure.UnitTests/InvoiceReminder.Infrastructure.UnitTests.csproj (1)
46-58: Referências explícitas de MSTest redundantes
Ao definir o Sdk comoMSTest.Sdk, os pacotesMSTest.Analyzers,MSTest.TestAdaptereMSTest.TestFrameworkjá são trazidos transitivamente. Manter referências<PackageReference Update=…>duplicadas é desnecessário e pode gerar advertências de dependência nodotnet restore. Sugestão de limpeza:- <ItemGroup> - <PackageReference Update="MSTest.Analyzers" Version="3.9.3"> - <PrivateAssets>all</PrivateAssets> - <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets> - </PackageReference> - </ItemGroup> - - <ItemGroup> - <PackageReference Update="MSTest.TestAdapter" Version="3.9.3" /> - </ItemGroup> - - <ItemGroup> - <PackageReference Update="MSTest.TestFramework" Version="3.9.3" /> - </ItemGroup> + <!-- Pacotes MSTest já são trazidos pelo Sdk; remoção reduz ruído de restore -->InvoiceReminder.API.UnitTests/InvoiceReminder.API.UnitTests.csproj (2)
7-7: Considere habilitar Nullable
<Nullable>disable</Nullable>reduz a detecção de problemas de nulidade mesmo em testes. Ativar (enable) ajuda a pegar regressões cedo sem custo significativo.
32-57: ReferênciasUpdateparecem redundantes
Como o projeto já usa o SDKMSTest.Sdk, as versões de
Microsoft.NET.Test.Sdk,MSTest.TestAdapter,MSTest.TestFramework,MSTest.Analyzerse os pacotes Extensions são fixadas transitivamente. DeclararPackageReference Updatepara cada um aumenta a manutenção e o risco de divergência futura.Se não houver necessidade de sobrescrever versões específicas, remova o bloco para simplificar:
- <ItemGroup> - <PackageReference Update="Microsoft.NET.Test.Sdk" Version="17.14.1" /> - </ItemGroup> - <ItemGroup> - <PackageReference Update="Microsoft.Testing.Extensions.CodeCoverage" Version="17.14.2" /> - </ItemGroup> - <ItemGroup> - <PackageReference Update="Microsoft.Testing.Extensions.TrxReport" Version="1.7.3" /> - </ItemGroup> - <ItemGroup> - <PackageReference Update="MSTest.Analyzers" Version="3.9.3"> - <PrivateAssets>all</PrivateAssets> - <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets> - </PackageReference> - </ItemGroup> - <ItemGroup> - <PackageReference Update="MSTest.TestAdapter" Version="3.9.3" /> - </ItemGroup> - <ItemGroup> - <PackageReference Update="MSTest.TestFramework" Version="3.9.3" /> - </ItemGroup>InvoiceReminder.ExternalServices.UnitTests/InvoiceReminder.ExternalServices.UnitTests.csproj (3)
1-1: Upgrade do SDK do MSTest parece correta, mas revise pacotes redundantesA troca para
MSTest.Sdk/3.9.3no atributoSdkdispensa, na maioria dos casos, a referência explícita aMicrosoft.NET.Test.Sdk,MSTest.TestAdapter,MSTest.TestFrameworkeMSTest.Analyzers, pois o novo SDK já traz essas dependências transitivamente nas versões corretas. Manter referências manuais pode gerar duplicidade de versões e tempo de restauração maior.Caso não exista razão específica para fixar versões nesses
PackageReference Update, considere removê-los:- <ItemGroup> - <PackageReference Update="Microsoft.NET.Test.Sdk" Version="17.14.1" /> - </ItemGroup> - - <ItemGroup> - <PackageReference Update="MSTest.Analyzers" Version="3.9.3"> - <PrivateAssets>all</PrivateAssets> - <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets> - </PackageReference> - </ItemGroup> - - <ItemGroup> - <PackageReference Update="MSTest.TestAdapter" Version="3.9.3" /> - </ItemGroup> - - <ItemGroup> - <PackageReference Update="MSTest.TestFramework" Version="3.9.3" /> - </ItemGroup>Isso simplifica o
.csproje evita divergências futuras.
4-8: Avalie compatibilidade do .NET 9 preview e nullable
net9.0ainda está em pré-lançamento; certifique-se de que o pipeline e os ambientes de CI/CD possuam o SDK preview correspondente para evitar falhas de build.- O
<Nullable>disable</Nullable>desativa verificações de referência nula justamente em testes onde bugs podem ser capturados cedo. Considere habilitar (enable) para melhorar a qualidade do código de teste.
17-25: Bibliotecas iText podem ser marcadas comoPrivateAssets=allSe os pacotes
itext.bouncy-castle-*só são necessários durante testes, recomendo:<PackageReference Include="itext.bouncy-castle-adapter" Version="9.2.0" PrivateAssets="all" /> <PackageReference Include="itext.bouncy-castle-fips-adapter" Version="9.2.0" PrivateAssets="all" />Assim evitamos que dependências pesadas vazem para projetos que referenciam este de testes.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (7)
InvoiceReminder.API.UnitTests/InvoiceReminder.API.UnitTests.csproj(1 hunks)InvoiceReminder.Application.UnitTests/InvoiceReminder.Application.UnitTests.csproj(1 hunks)InvoiceReminder.ArchitectureTests/InvoiceReminder.ArchitectureTests.csproj(2 hunks)InvoiceReminder.DomainEntities.UnitTests/InvoiceReminder.DomainEntities.UnitTests.csproj(1 hunks)InvoiceReminder.ExternalServices.UnitTests/InvoiceReminder.ExternalServices.UnitTests.csproj(1 hunks)InvoiceReminder.Infrastructure.UnitTests/InvoiceReminder.Infrastructure.UnitTests.csproj(1 hunks)InvoiceReminder.JobScheduler.UnitTests/InvoiceReminder.JobScheduler.UnitTests.csproj(1 hunks)
🔇 Additional comments (9)
InvoiceReminder.DomainEntities.UnitTests/InvoiceReminder.DomainEntities.UnitTests.csproj (1)
1-1: Upgrade do MSTest SDK aprovadoA atualização para
MSTest.Sdk/3.9.3mantém o projeto alinhado aos demais testes da solução e elimina warnings conhecidos da versão 3.6.x. Sem objeções.InvoiceReminder.JobScheduler.UnitTests/InvoiceReminder.JobScheduler.UnitTests.csproj (1)
1-1: Versão do MSTest.Sdk alinhada com demais pacotes — ponto positivoAtualizar o SDK do projeto para
3.9.3corrige bugs conhecidos da série 3.6 .x e garante compatibilidade com os demais pacotes MSTest referenciados.InvoiceReminder.Application.UnitTests/InvoiceReminder.Application.UnitTests.csproj (3)
1-1: Verifique a compatibilidade do MSTest.Sdk 3.9.3 com o .NET 9.0
net9.0ainda está em preview e nem todos os SDKs foram validados oficialmente. Garanta que os testes compilam/rodam no pipeline CI local e em agentes hospedados, sobretudo em modo ‑warnaserror.
34-35: Atualização de CodeCoverage aprovadaBoa prática manter o pacote em linha com o
Microsoft.NET.Test.Sdk. Nenhum problema identificado.
38-39: Atualização de TrxReport aprovadaSem impactos visíveis; dependência segue a mesma linha de versão do ecossistema 17.14.x.
InvoiceReminder.ArchitectureTests/InvoiceReminder.ArchitectureTests.csproj (1)
1-1: Atualização doSdkpara MSTest 3.9.3 é coerente, mas valide compatibilidade com .NET 9 preview
A TFMnet9.0ainda é pré-lançamento; certifique-se de que a pipeline esteja usando um SDK do .NET 9 correspondente, caso contrário o build falhará.InvoiceReminder.Infrastructure.UnitTests/InvoiceReminder.Infrastructure.UnitTests.csproj (2)
1-1: Atualização do SDK para 3.9.3 validada
Boa escolha manter oSdk="MSTest.Sdk/3.9.3"alinhado aos demais projetos.
34-34: Uniformidade do Microsoft.NET.Test.Sdk validada
Todos os projetos de teste (Infrastructure, JobScheduler, DomainEntities, ArchitectureTests, API, ExternalServices e Application) referenciam a versão 17.14.1 do Microsoft.NET.Test.Sdk, sem divergências.InvoiceReminder.API.UnitTests/InvoiceReminder.API.UnitTests.csproj (1)
1-1: Versão do SDK atualizada corretamente, mas confira ambiente local
A atualização paraMSTest.Sdk/3.9.3está em linha com os demais projetos — ótimo. Apenas valide se todos os devs/CI já utilizam um .NET SDK preview compatível comnet9.0; caso contrário, o build poderá falhar fora do pipeline principal.
1 participant
✨
Description by Callstackai
This PR updates the MSTest SDK version and Microsoft.NET.Test.Sdk version across multiple unit test projects.
Diagrams of code changes
sequenceDiagram participant TestProjects participant MSTest participant DotNet TestProjects->>MSTest: Update SDK version to 3.9.3 Note over TestProjects,MSTest: All test projects updated to MSTest.Sdk/3.9.3 TestProjects->>DotNet: Update target framework Note over TestProjects,DotNet: All projects targeting net9.0 TestProjects->>MSTest: Update test dependencies Note over TestProjects,MSTest: Microsoft.NET.Test.Sdk updated to 17.14.1Files Changed
This PR includes files in programming languages that we currently do not support. We have not reviewed files with the extensions
.csproj. See list of supported languages.Summary by CodeRabbit