Merge pull request #230 from joe733/workshop

maint: reformats `card` module, fix typo

Author: yozachar

tests/test_btc_address.py

@@ -1,7 +1,7 @@
 """Test BTC address."""
 # -*- coding: utf-8 -*-
 
-# standard
+# external
 import pytest
 
 # local

tests/test_card.py

@@ -1,44 +1,29 @@
+"""Test Card."""
 # -*- coding: utf-8 -*-
+
+# external
 import pytest
 
+# local
 from validators import (
-    amex,
+    ValidationFailure,
     card_number,
-    diners,
-    discover,
-    jcb,
     mastercard,
     unionpay,
-    ValidationFailure,
-    visa
+    discover,
+    diners,
+    visa,
+    amex,
+    jcb,
 )
 
-visa_cards = [
-    '4242424242424242',
-    '4000002760003184'
-]
-mastercard_cards = [
-    '5555555555554444',
-    '2223003122003222'
-]
-amex_cards = [
-    '378282246310005',
-    '371449635398431'
-]
-unionpay_cards = [
-    '6200000000000005'
-]
-diners_cards = [
-    '3056930009020004',
-    '36227206271667'
-]
-jcb_cards = [
-    '3566002020360505'
-]
-discover_cards = [
-    '6011111111111117',
-    '6011000990139424'
-]
+visa_cards = ["4242424242424242", "4000002760003184"]
+mastercard_cards = ["5555555555554444", "2223003122003222"]
+amex_cards = ["378282246310005", "371449635398431"]
+unionpay_cards = ["6200000000000005"]
+diners_cards = ["3056930009020004", "36227206271667"]
+jcb_cards = ["3566002020360505"]
+discover_cards = ["6011111111111117", "6011000990139424"]
 
 
 @pytest.mark.parametrize(
@@ -51,140 +36,117 @@
     + jcb_cards
     + discover_cards,
 )
-def test_returns_true_on_valid_card_number(value):
+def test_returns_true_on_valid_card_number(value: str):
+    """Test returns true on valid card number."""
     assert card_number(value)
 
 
-@pytest.mark.parametrize('value', [
-    '4242424242424240',
-    '4000002760003180',
-    '400000276000318X'
-])
-def test_returns_failed_on_valid_card_number(value):
+@pytest.mark.parametrize("value", ["4242424242424240", "4000002760003180", "400000276000318X"])
+def test_returns_failed_on_valid_card_number(value: str):
+    """Test returns failed on valid card number."""
     assert isinstance(card_number(value), ValidationFailure)
 
 
-@pytest.mark.parametrize('value', visa_cards)
-def test_returns_true_on_valid_visa(value):
+@pytest.mark.parametrize("value", visa_cards)
+def test_returns_true_on_valid_visa(value: str):
+    """Test returns true on valid visa."""
     assert visa(value)
 
 
 @pytest.mark.parametrize(
     "value",
-    mastercard_cards
-    + amex_cards
-    + unionpay_cards
-    + diners_cards
-    + jcb_cards
-    + discover_cards,
+    mastercard_cards + amex_cards + unionpay_cards + diners_cards + jcb_cards + discover_cards,
 )
-def test_returns_failed_on_valid_visa(value):
+def test_returns_failed_on_valid_visa(value: str):
+    """Test returns failed on valid visa."""
     assert isinstance(visa(value), ValidationFailure)
 
 
-@pytest.mark.parametrize('value', mastercard_cards)
-def test_returns_true_on_valid_mastercard(value):
+@pytest.mark.parametrize("value", mastercard_cards)
+def test_returns_true_on_valid_mastercard(value: str):
+    """Test returns true on valid mastercard."""
     assert mastercard(value)
 
 
 @pytest.mark.parametrize(
     "value",
-    visa_cards
-    + amex_cards
-    + unionpay_cards
-    + diners_cards
-    + jcb_cards
-    + discover_cards,
+    visa_cards + amex_cards + unionpay_cards + diners_cards + jcb_cards + discover_cards,
 )
-def test_returns_failed_on_valid_mastercard(value):
+def test_returns_failed_on_valid_mastercard(value: str):
+    """Test returns failed on valid mastercard."""
     assert isinstance(mastercard(value), ValidationFailure)
 
 
-@pytest.mark.parametrize('value', amex_cards)
-def test_returns_true_on_valid_amex(value):
+@pytest.mark.parametrize("value", amex_cards)
+def test_returns_true_on_valid_amex(value: str):
+    """Test returns true on valid amex."""
     assert amex(value)
 
 
 @pytest.mark.parametrize(
     "value",
-    visa_cards
-    + mastercard_cards
-    + unionpay_cards
-    + diners_cards
-    + jcb_cards
-    + discover_cards,
+    visa_cards + mastercard_cards + unionpay_cards + diners_cards + jcb_cards + discover_cards,
 )
-def test_returns_failed_on_valid_amex(value):
+def test_returns_failed_on_valid_amex(value: str):
+    """Test returns failed on valid amex."""
     assert isinstance(amex(value), ValidationFailure)
 
 
-@pytest.mark.parametrize('value', unionpay_cards)
-def test_returns_true_on_valid_unionpay(value):
+@pytest.mark.parametrize("value", unionpay_cards)
+def test_returns_true_on_valid_unionpay(value: str):
+    """Test returns true on valid unionpay."""
     assert unionpay(value)
 
 
 @pytest.mark.parametrize(
     "value",
-    visa_cards
-    + mastercard_cards
-    + amex_cards
-    + diners_cards
-    + jcb_cards
-    + discover_cards,
+    visa_cards + mastercard_cards + amex_cards + diners_cards + jcb_cards + discover_cards,
 )
-def test_returns_failed_on_valid_unionpay(value):
+def test_returns_failed_on_valid_unionpay(value: str):
+    """Test returns failed on valid unionpay."""
     assert isinstance(unionpay(value), ValidationFailure)
 
 
-@pytest.mark.parametrize('value', diners_cards)
-def test_returns_true_on_valid_diners(value):
+@pytest.mark.parametrize("value", diners_cards)
+def test_returns_true_on_valid_diners(value: str):
+    """Test returns true on valid diners."""
     assert diners(value)
 
 
 @pytest.mark.parametrize(
     "value",
-    visa_cards
-    + mastercard_cards
-    + amex_cards
-    + unionpay_cards
-    + jcb_cards
-    + discover_cards,
+    visa_cards + mastercard_cards + amex_cards + unionpay_cards + jcb_cards + discover_cards,
 )
-def test_returns_failed_on_valid_diners(value):
+def test_returns_failed_on_valid_diners(value: str):
+    """Test returns failed on valid diners."""
     assert isinstance(diners(value), ValidationFailure)
 
 
-@pytest.mark.parametrize('value', jcb_cards)
-def test_returns_true_on_valid_jcb(value):
+@pytest.mark.parametrize("value", jcb_cards)
+def test_returns_true_on_valid_jcb(value: str):
+    """Test returns true on valid jcb."""
     assert jcb(value)
 
 
 @pytest.mark.parametrize(
     "value",
-    visa_cards
-    + mastercard_cards
-    + amex_cards
-    + unionpay_cards
-    + diners_cards
-    + discover_cards,
+    visa_cards + mastercard_cards + amex_cards + unionpay_cards + diners_cards + discover_cards,
 )
-def test_returns_failed_on_valid_jcb(value):
+def test_returns_failed_on_valid_jcb(value: str):
+    """Test returns failed on valid jcb."""
     assert isinstance(jcb(value), ValidationFailure)
 
 
-@pytest.mark.parametrize('value', discover_cards)
-def test_returns_true_on_valid_discover(value):
+@pytest.mark.parametrize("value", discover_cards)
+def test_returns_true_on_valid_discover(value: str):
+    """Test returns true on valid discover."""
     assert discover(value)
 
 
 @pytest.mark.parametrize(
     "value",
-    visa_cards
-    + mastercard_cards
-    + amex_cards
-    + unionpay_cards
-    + diners_cards
-    + jcb_cards,
+    visa_cards + mastercard_cards + amex_cards + unionpay_cards + diners_cards + jcb_cards,
 )
-def test_returns_failed_on_valid_discover(value):
+def test_returns_failed_on_valid_discover(value: str):
+    """Test returns failed on valid discover."""
     assert isinstance(discover(value), ValidationFailure)