Incorrect handling of map like structures
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 42/100
Direzione di ricerca
Inizia dal test applicato in src/tests.rs, in particolare test_option, e riproduci l’errore con il comando di test della libreria. Leggi src/apply.rs intorno alla riga 96 per capire come viene gestito il comando diff che fallisce. Il lavoro è completato quando il roundtrip di HashMap nel test riesce e i casi mirati, array, tupla e option esistenti continuano a funzionare.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
See the applied test where "map" like structure is tested.
running 4 tests
test tests::test_targeted ... ok
test tests::test_array ... ok
test tests::test_tuple ... ok
test tests::test_option ... FAILED
failures:
---- tests::test_option stdout ----
thread 'tests::test_option' panicked at 'unexpected DiffCommand Value or Remove', src/apply.rs:96:49
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
failures:
tests::test_option
test result: FAILED. 3 passed; 1 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s
error: test failed, to rerun pass '--lib'
diff --git a/src/tests.rs b/src/tests.rs
index 082846b..83d117c 100644
--- a/src/tests.rs
+++ b/src/tests.rs
@@ -1,7 +1,9 @@
use crate as serde_diff;
use crate::{Apply, Diff, SerdeDiff};
use serde::{Deserialize, Serialize};
+use std::collections::HashMap;
use std::fmt::Debug;
+use std::iter::FromIterator;
#[derive(SerdeDiff, Serialize, Deserialize, PartialEq, Debug, Copy, Clone)]
struct TestStruct {
@@ -59,6 +61,18 @@ fn test_option() {
Some(TestStruct { a: 52, b: 32. }),
Some(TestStruct { a: 42, b: 12. }),
);
+ roundtrip(
+ HashMap::from_iter([
+ (1, TestStruct { a: 1, b: 1. }),
+ (2, TestStruct { a: 2, b: 2. }),
+ (3, TestStruct { a: 3, b: 3. }),
+ ]),
+ HashMap::from_iter([
+ (1, TestStruct { a: 1, b: 1. }),
+ (3, TestStruct { a: 4, b: 4. }),
+ (4, TestStruct { a: 1, b: 1. }),
+ ]),
+ );
partial(
Some(TestStruct { a: 5, b: 2. }),
- Lingua principale
- Rust
- Stelle
- 127
- Fork
- 22
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di amethyst/serde-diff
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
amethyst/serde-diff#29 ·
-
Iterate over differences Aperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
amethyst/serde-diff#43 ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 10/100
amethyst/serde-diff#42 · 6 commenti · 2 reazioni ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 45/100
amethyst/serde-diff#35 · 1 reazione ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
amethyst/serde-diff#32 ·
Tutte le issue di amethyst/serde-diff
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
-
bug core
Difficoltà 2/5 1-3 ore Idoneità per principianti 86/100
-
JIT-compiled number -> Decimal conversion silently overflows instead of raising DECIMAL_OVERFLOW Apertafuzz
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
ClickHouse/ClickHouse#122114 ·
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 92/100
linebender/vello_svg#90 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100