[possible Bug]: ChangeAwareCache not capturing initialized data

Aberta
#669 2 comentários 0 reações 0 responsáveis Ver no GitHub

Mantenedores costumam responder em até 1 dia

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
3/5
Tempo estimado
1-2 dias
Facilidade para iniciantes
35/100
Tipo de issue
Bug
Clareza
Razoavelmente clara
Status de atividade
Estagnada
Stack de tecnologia
csharp
Domínio
data

Direção de pesquisa

Start with src/DynamicData/Cache/ChangeAwareCache.cs at the linked line 54, then compare its initialization and CaptureChanges() behavior with the ChangeAwareList code mentioned in the issue. Check whether an initialized dictionary should produce an initial ChangeSet and whether existing tests cover that behavior. Done means the expected behavior is clear and covered by a regression test; the issue does not identify a test file.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

bug
Describe the bug 🐞

looking through the code because of something unrelated the following took me by surprise:

https://github.com/reactivemarbles/DynamicData/blob/main/src/DynamicData/Cache/ChangeAwareCache.cs#L54

initializing the ChangeAwareCache with a Dictionary and subsequently calling CaptureChanges() did not produce an initial ChangeSet!

I don't know if that might have an effect on the whole library as for example the ChangeAwareList does have code for the initial ChangeSet

Step to reproduce

just code review

Reproduction repository

https://github.com/reactivemarbles/DynamicData

Expected behavior

to send an initial ChangeSet

Screenshots 🖼️

No response

IDE

No response

Operating system

No response

Version

No response

Device

No response

DynamicData Version

No response

Additional information ℹ️

No response

Linguagem predominante
C#
Estrelas
1.9k
Forks
194
Merge médio
12min
PRs com merge (30d)
1

Preparar o ambiente

Este projeto não oferece contêiner de desenvolvimento, Dockerfile nem guia de contribuição, então a configuração fica por sua conta: comece pelo README e veja nosso guia da primeira contribuição para os passos gerais.

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de reactivemarbles/DynamicData

Todas as issues de reactivemarbles/DynamicData

Issues semelhantes

Mais issues de C#

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.