add missing cache test - [merged]

[benoitdm-oslandia]

_Merges fix/missing_cachetest -> develop

==> 2 bugs found!

ref #50

[benoitdm-oslandia]

requested review from @jmkerloch

[benoitdm-oslandia]

added 30m of time spent

[benoitdm-oslandia]

In GitLab by @azarz on Nov 4, 2022, 15:54

Commented on internal/data/listener.go line 140

Je me demande si le mutex est toujours utile ici vu que @nrevelant l'a implémenté dans la classe Cacher ?

[benoitdm-oslandia]

In GitLab by @azarz on Nov 4, 2022, 15:56

Commented on internal/service/db_test/runner_db_test.go line 96

Déplacer vers la ligne 145 ? (tests associés au listen)

[benoitdm-oslandia]

added 2 commits

• 281ab934 - fix(catalog_db): non-check error when posted object has only required fields.
• 553f76cb - fix(cache): add missing test TestCacheModifiedAfterUpdate

Compare with previous version

[benoitdm-oslandia]

In GitLab by @jmkerloch on Nov 4, 2022, 16:04

approved this merge request

[benoitdm-oslandia]

en effet

[benoitdm-oslandia]

ok

[benoitdm-oslandia]

resolved all threads

[benoitdm-oslandia]

changed this line in version 3 of the diff

[benoitdm-oslandia]

changed this line in version 3 of the diff

[benoitdm-oslandia]

added 2 commits

• f8a20856 - fix(cache): add missing test TestCacheModifiedAfterUpdate
• a00c42b4 - fix(listener): remove useless mutex

Compare with previous version

[benoitdm-oslandia]

resolved all threads

[benoitdm-oslandia]

@jmkerloch @azarz c'est bon ?

[benoitdm-oslandia]

In GitLab by @azarz on Nov 4, 2022, 16:35

approved this merge request

[benoitdm-oslandia]

approved this merge request