bitwalker / bitwalker/libgraph

Failing tests

Offen
#83 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Elixir
Sterne
571
Forks
76
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

test sizeof/1 (Graph.UtilsTest)
`assert 440 = sizeof(String.duplicate("bar", 128))`
```
left: 440
right: 456
```
> I can't comment on what's going on here, but on my system 456 seems to be correct as `:erlang.system_info(:wordsize) == 8` so `sizeof(String.duplicate("bar", 128))` evaluates to 72 + 384 = 456.

test inspect (PriorityQueue.Test)
`assert "#PriorityQueue" = str`
```
left: "#PriorityQueue"
right: "#PriorityQueue"
```
> I think this was caused by an Elixir update at some point that prefers the sigil notation over the single-quoted strings. If reproducible would update the test expected value to use the sigil or change the inspect function to convert the sigil to the single-quoted string format.

test inspect (GraphTest)
`assert "#Graph :b, :a -> :b, :b -[{:complex, :label}]-> :a, :b -> :c]>" = str`
```
left: "#Graph :b, :a -> :b, :b -[{:complex, :label}]-> :a, :b -> :c]>"
right: "#Graph :b, :a -[foo]-> :b, :b -[{:complex, :label}]-> :a, :b -> :c]>"
```
> Since edges come from a map the order is not reliable, need to sort before printing to ensure test can pass.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Beginne damit, die fehlschlagenden Tests in Graph.UtilsTest, PriorityQueue.Test und GraphTest auszuführen. Prüfe die gemeldete sizeof und vergleiche die Erwartungen mit dem aktuellen Verhalten von Elixir; verifiziere anschließend, dass die Ausgabe der Graph-Kanten deterministisch ist. Erledigt ist die Aufgabe, wenn alle drei Fehler bestehen, ohne sich auf eine plattformspezifische Wortgröße oder die Iterationsreihenfolge von Maps zu verlassen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
elixir
Bereich
testing
Issue-Typ
Bug
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.