Fix errors found after running tests

* Make passing "private" in when saving Twitter User data optional (because it's not in the downloaded archive of data)
* Cope with the fact some idiot (me) decided to pass either a dict *or* a boolean from a method.

For #229

ditto/twitter/fetch/savers.py

@@ -59,14 +59,16 @@ def save_user(self, user, fetch_time, download_avatar=True):
             "raw": raw_json,
             "screen_name": user["screen_name"],
             "name": user["name"],
-            "is_private": user["protected"],
             "is_verified": user["verified"],
             "profile_image_url_https": user["profile_image_url_https"],
         }
 
         # When ingesting tweets there are lots of fields the 'user' element
         # doesn't have, compared to the API:
 
+        if "protected" in user:
+            defaults["is_private"] = user["protected"]
+
         if "created_at" in user:
             defaults["created_at"] = self._api_time_to_datetime(user["created_at"])
 

ditto/twitter/models.py

@@ -70,7 +70,11 @@ def save(self, *args, **kwargs):
 
             # It would be nice to make this more visible, but not sure how to
             # given we don't have access to a request at this point.
-            if "success" in result and result["success"] is False:
+            if (
+                type(result) is dict
+                and "success" in result
+                and result["success"] is False
+            ):
                 if "messages" in result:
                     messages = ", ".join(result["messages"])
                 else: