bitwalker/timex_ecto

Expand test suite

Open

#20 geöffnet am 12. Apr. 2016

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Elixir (68 Forks)github user discovery
enhancementhelp wanted

Repository-Metriken

Stars
 (163 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

The current test suite contains a single test for Timex.Ecto.Time.cast/1 when the input is a string. Other tests are likely needed here:

  • Timex.Ecto.Time.cast/1 when input is binary
  • Timex.Ecto.Time.cast/1when input is a timestamp
  • Timex.Ecto.Time.cast/1 when input is a map like %{"calendar" => . . . }
  • Timex.Ecto.Time.cast/1 when input is time
  • Timex.Ecto.Time.load/1
  • Timex.Ecto.Time.dump/1
  • Timex.Ecto.DateTime.cast/1 functions here
  • Timex.Ecto.DateTimeWithTimezone .cast/1, .dump/1, .load/1 functions here
  • Timex.Ecto.Date functions here

I'm happy to put up a PR for these and others that you might think are necessary!

Contributor Guide

Research-Richtung
Schreiben Sie Tests für die aufgelisteten cast/load/dump Funktionen, indem Sie dem Muster in der vorhandenen Testdatei folgen.
Tech Stack
elixir
Domain
backend
Issue Type
Test
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
ElixirEcto
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60