facebook/TestSlide

Add constructor input checks for matchers

Open

#233 geöffnet am 1. Sept. 2020

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (1 zugewiesene Person)Python (58 Forks)auto 404
Hacktoberfestenhancement

Repository-Metriken

Stars
 (147 Stars)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

When we do:

AnyInstanceOf(not_a_type)

we end up raising an exception not at the constructor, but rather when the comparison is being made, and isinstanceof (at AnyInstanceOf's code) will complain that not_a_type is... not a type.

Let's revisit all matchers & ensure that all of them raise ValueError if the constructor arguments are invalid from the constructor itself.

CC @deathowl

Contributor Guide

Research-Richtung
Überprüfen Sie alle Matcher Klassen in der TestSlide Codebasis und fügen Sie in deren init Methoden Eingabevalidierungen hinzu, um bei ungültigen Argumenten einen ValueError auszulösen.
Tech Stack
python
Domain
testing
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
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
Python
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70