remove business_website como required! (#100)

* remove business_website como required! * Update tests/models/test_seller.py Co-authored-by: PedroRegisPOAR <pedroalencarregis@hotmail.com> * Update zoop_wrapper/models/seller.py Co-authored-by: PedroRegisPOAR <pedroalencarregis@hotmail.com> Co-authored-by: PedroRegisPOAR <pedroalencarregis@hotmail.com>
imobanco · Aug 24, 2020 · c998ef6 · c998ef6
1 parent ad04331
commit c998ef6
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 5 deletions.
diff --git a/tests/models/test_seller.py b/tests/models/test_seller.py
@@ -44,15 +44,20 @@ def test_business_required_fields(self):
                 "business_name",
                 "business_opening_date",
                 "business_phone",
-                "business_website",
                 "ein",
             },
             Seller.get_business_required_fields(),
         )
 
     def test_business_non_required_fields(self):
         self.assertIsSubSet(
-            {"owner", "business_description", "business_facebook", "business_twitter"},
+            {
+                "business_website",
+                "business_description",
+                "business_facebook",
+                "business_twitter",
+                "owner",
+            },
             Seller.get_business_non_required_fields(),
         )
 
@@ -99,7 +104,6 @@ def test_business_get_validation_fields(self):
                 "business_name",
                 "business_opening_date",
                 "business_phone",
-                "business_website",
             },
             instance.get_validation_fields(),
         )

diff --git a/zoop_wrapper/models/seller.py b/zoop_wrapper/models/seller.py
@@ -126,7 +126,13 @@ def get_business_non_required_fields(cls):
         fields = cls.get_non_required_fields()
         return fields.union(
             super().get_business_non_required_fields(),
-            {"owner", "business_description", "business_facebook", "business_twitter"},
+            {
+                "business_description",
+                "business_website",
+                "business_facebook",
+                "business_twitter",
+                "owner",
+            },
         )
 
     @classmethod
@@ -146,7 +152,6 @@ def get_business_required_fields(cls):
                 "business_name",
                 "business_opening_date",
                 "business_phone",
-                "business_website",
             },
         )