[V0/V1 Migration] Migrate /v1/bulk/properties/{direction} to /v2/node (#280)

Based on a [github
search](https://github.com/search?q=org%3Adatacommonsorg+BulkProperties&type=code)
it looks like this endpoint is only used by the old Python client
(except some legacy docsite references), so this PR migrates the Python
client usage.

This is a pretty straightforward migration to /v2/node, which has a very
similar structure for fetching properties, so there was little change in
response processing. I also updated tests and added an additional test
case.

Based on Apigee, in the past month there's been 480 requests to this
endpoint:
* 440 (not set) 
* 39 datcom-core
* 1 me (for testing)

This PR also bumps the version for typing_extensions for compatibility

Author: n-h-diaz

datacommons/node.py

@@ -17,6 +17,7 @@
 from typing import Dict, List
 
 from datacommons.requests import _post
+from datacommons.utils import _get_arrow
 from datacommons.utils import _get_direction
 
 
@@ -31,12 +32,10 @@ def properties(nodes: List[str], is_out: bool = True) -> Dict[str, List[str]]:
       A dict keyed by node DCID, with the values being a list of properties
       for the queried node.
   """
-  resp = _post(f'/v1/bulk/properties/{_get_direction(is_out)}', {
-      'nodes': nodes,
-  })
+  resp = _post('/v2/node', {'nodes': nodes, 'property': _get_arrow(is_out)})
   result = {}
-  for item in resp.get('data', []):
-    node, properties = item['node'], item.get('properties', [])
+  for node, item in resp.get('data', {}).items():
+    properties = item.get('properties', [])
     result[node] = properties
   return result
 

datacommons/test/node_test.py

@@ -24,17 +24,74 @@ class TestProperties(unittest.TestCase):
   def test_with_data(self, _post):
 
     def side_effect(path, data):
-      if path == "/v1/bulk/properties/out" and data == {"nodes": ["City"]}:
+      if path == "/v2/node" and data == {
+          "nodes": ["City", "Count_Person", "foo"],
+          "property": "->"
+      }:
         return {
-            "data": [{
-                "node": "City",
-                "properties": ["name", "provenance", "subClassOf", "typeOf"]
-            }]
+            "data": {
+                "City": {
+                    "properties": [
+                        "name", "provenance", "subClassOf", "typeOf"
+                    ]
+                },
+                "Count_Person": {
+                    "properties": [
+                        "description", "measuredProperty", "memberOf", "name",
+                        "populationType", "provenance", "statType", "typeOf"
+                    ]
+                },
+                "foo": {}
+            }
+        }
+
+    _post.side_effect = side_effect
+    response = datacommons.properties(["City", "Count_Person", "foo"])
+    assert response == {
+        "City": ["name", "provenance", "subClassOf", "typeOf"],
+        "Count_Person": [
+            "description", "measuredProperty", "memberOf", "name",
+            "populationType", "provenance", "statType", "typeOf"
+        ],
+        "foo": []
+    }
+
+  @patch("datacommons.node._post")
+  def test_with_direction(self, _post):
+
+    def side_effect(path, data):
+      if path == "/v2/node" and data == {
+          "nodes": ["City", "Count_Person", "foo"],
+          "property": "<-"
+      }:
+        return {
+            "data": {
+                "City": {
+                    "properties": [
+                        "placeType", "rangeIncludes", "schoolLocationType",
+                        "typeOf"
+                    ]
+                },
+                "Count_Person": {
+                    "properties": [
+                        "measurementDenominator", "outputProperty",
+                        "relevantVariable"
+                    ]
+                },
+                "foo": {}
+            }
         }
 
     _post.side_effect = side_effect
-    response = datacommons.properties(["City"])
-    assert response == {"City": ["name", "provenance", "subClassOf", "typeOf"]}
+    response = datacommons.properties(["City", "Count_Person", "foo"],
+                                      is_out=False)
+    assert response == {
+        "City": ["placeType", "rangeIncludes", "schoolLocationType", "typeOf"],
+        "Count_Person": [
+            "measurementDenominator", "outputProperty", "relevantVariable"
+        ],
+        "foo": []
+    }
 
 
 class TestPropertyValues(unittest.TestCase):

datacommons/utils.py

@@ -124,3 +124,14 @@ def _format_expand_payload(payload, new_key, must_exist=[]):
 
 def _get_direction(out: bool):
   return "out" if out else "in"
+
+
+def _get_arrow(out: bool):
+  """Returns the arrow syntax for an arc direction.
+
+  Args:
+      out: Whether the arc direction is out.
+  Returns:
+      The corresponding arrow syntax.
+  """
+  return "->" if out else "<-"

requirements.txt

@@ -3,6 +3,6 @@ mock
 pandas
 pytest
 requests==2.32.0
-typing_extensions==4.7.1
+typing_extensions==4.12.2
 yapf==0.40.2
 pydantic>=2.11