Sort attrs of Traits and update docs

Author: marselester

README.rst

@@ -514,21 +514,3 @@ Please report all issues with this code using the `GitHub issue tracker
 If you are having an issue with a MaxMind service that is not specific to the
 client API, please contact `MaxMind support
 <https://www.maxmind.com/en/support>`_ for assistance.
-
-Testing
--------
-
-Get the repository and its submodule.
-
-.. code-block:: bash
-
-    $ git clone https://github.com/maxmind/GeoIP2-python.git
-    $ cd ./GeoIP2-python/
-    $ git submodule update --init --recursive
-
-Install and run ``tox``.
-
-.. code-block:: bash
-
-    $ pip install tox
-    $ tox

geoip2/records.py

@@ -634,14 +634,6 @@ class Traits(Record):
 
       :type: str
 
-    .. attribute:: is_anycast
-
-      This is true if the IP address is anycast.
-      This attribute is available from the Country, City Plus, Insights
-      web services and the Country, City, Enterprise databases.
-
-      :type: bool
-
     .. attribute:: is_anonymous
 
       This is true if the IP address belongs to any sort of anonymous network.
@@ -673,6 +665,15 @@ class Traits(Record):
 
       :type: bool
 
+    .. attribute:: is_anycast
+
+      This returns true if the IP address belongs to an
+      `anycast network <https://en.wikipedia.org/wiki/Anycast>`_.
+      This is available for the GeoIP2 Country, City Plus, and Insights
+      web services and the GeoIP2 Country, City, and Enterprise databases.
+
+      :type: bool
+
     .. attribute:: is_hosting_provider
 
       This is true if the IP address belongs to a hosting or VPN provider
@@ -823,18 +824,18 @@ class Traits(Record):
     autonomous_system_organization: Optional[str]
     connection_type: Optional[str]
     domain: Optional[str]
-    is_anycast: bool
+    ip_address: Optional[str]
     is_anonymous: bool
     is_anonymous_proxy: bool
     is_anonymous_vpn: bool
+    is_anycast: bool
     is_hosting_provider: bool
     is_legitimate_proxy: bool
     is_public_proxy: bool
     is_residential_proxy: bool
     is_satellite_provider: bool
     is_tor_exit_node: bool
     isp: Optional[str]
-    ip_address: Optional[str]
     mobile_country_code: Optional[str]
     mobile_network_code: Optional[str]
     organization: Optional[str]
@@ -850,7 +851,6 @@ def __init__(
         autonomous_system_organization: Optional[str] = None,
         connection_type: Optional[str] = None,
         domain: Optional[str] = None,
-        is_anycast: bool = False,
         is_anonymous: bool = False,
         is_anonymous_proxy: bool = False,
         is_anonymous_vpn: bool = False,
@@ -870,16 +870,17 @@ def __init__(
         user_type: Optional[str] = None,
         mobile_country_code: Optional[str] = None,
         mobile_network_code: Optional[str] = None,
+        is_anycast: bool = False,
         **_,
     ) -> None:
         self.autonomous_system_number = autonomous_system_number
         self.autonomous_system_organization = autonomous_system_organization
         self.connection_type = connection_type
         self.domain = domain
-        self.is_anycast = is_anycast
         self.is_anonymous = is_anonymous
         self.is_anonymous_proxy = is_anonymous_proxy
         self.is_anonymous_vpn = is_anonymous_vpn
+        self.is_anycast = is_anycast
         self.is_hosting_provider = is_hosting_provider
         self.is_legitimate_proxy = is_legitimate_proxy
         self.is_public_proxy = is_public_proxy

tests/models_test.py

@@ -75,10 +75,10 @@ def test_insights_full(self) -> None:
                 "connection_type": "Cable/DSL",
                 "domain": "example.com",
                 "ip_address": "1.2.3.4",
-                "is_anycast": True,
                 "is_anonymous": True,
                 "is_anonymous_proxy": True,
                 "is_anonymous_vpn": True,
+                "is_anycast": True,
                 "is_hosting_provider": True,
                 "is_public_proxy": True,
                 "is_residential_proxy": True,
@@ -192,10 +192,10 @@ def test_insights_full(self) -> None:
         self.assertIs(model.registered_country.is_in_european_union, False)
         self.assertIs(model.represented_country.is_in_european_union, True)
 
-        self.assertIs(model.traits.is_anycast, True)
         self.assertIs(model.traits.is_anonymous, True)
         self.assertIs(model.traits.is_anonymous_proxy, True)
         self.assertIs(model.traits.is_anonymous_vpn, True)
+        self.assertIs(model.traits.is_anycast, True)
         self.assertIs(model.traits.is_hosting_provider, True)
         self.assertIs(model.traits.is_public_proxy, True)
         self.assertIs(model.traits.is_residential_proxy, True)
@@ -325,14 +325,14 @@ def test_city_full(self) -> None:
             "registered_country name is correct",
         )
         self.assertEqual(
-            model.traits.is_anycast,
+            model.traits.is_anonymous_proxy,
             False,
-            "traits is_anycast returns False by default",
+            "traits is_anonymous_proxy returns False by default",
         )
         self.assertEqual(
-            model.traits.is_anonymous_proxy,
+            model.traits.is_anycast,
             False,
-            "traits is_anonymous_proxy returns False by default",
+            "traits is_anycast returns False by default",
         )
         self.assertEqual(
             model.traits.is_satellite_provider,