Fix a cloud GQL api change for repoTopicsByNamespaceAndRepository (…

…now returns a list of `topic` rather than a single `topics` list).
splitgraph · Aug 18, 2021 · 8dfa862 · 8dfa862
1 parent 6eb8c92
commit 8dfa862
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/splitgraph/cloud/__init__.py b/splitgraph/cloud/__init__.py
@@ -197,7 +197,7 @@ def get_headers():
       repository
       repoTopicsByNamespaceAndRepository {
         nodes {
-          topics
+          topic
         }
       }
       repoProfileByNamespaceAndRepository {

diff --git a/splitgraph/cloud/models.py b/splitgraph/cloud/models.py
@@ -83,7 +83,7 @@ class RepositoriesYAML(BaseModel):
 class MetadataResponse(BaseModel):
     class RepoTopicsResponse(BaseModel):
         class RepositoryTopics(BaseModel):
-            topics: List[str]
+            topic: str
 
         nodes: List[RepositoryTopics]
 
@@ -116,9 +116,7 @@ def to_metadata(self) -> Metadata:
         return Metadata(
             readme=Metadata.Readme(text=profile.readme),
             description=profile.description,
-            topics=[
-                t for node in self.repoTopicsByNamespaceAndRepository.nodes for t in node.topics
-            ],
+            topics=[node.topic for node in self.repoTopicsByNamespaceAndRepository.nodes],
             sources=profile.sources,
             license=profile.license,
             extra_metadata=profile.metadata.upstream_metadata if profile.metadata else None,

diff --git a/test/splitgraph/commandline/http_fixtures.py b/test/splitgraph/commandline/http_fixtures.py
@@ -170,7 +170,9 @@ def _gql_callback(request, uri, response_headers):
         _somerepo_2 = {
             "namespace": "otheruser",
             "repository": "somerepo_2",
-            "repoTopicsByNamespaceAndRepository": {"nodes": [{"topics": ["topic_1", "topic_2"]}]},
+            "repoTopicsByNamespaceAndRepository": {
+                "nodes": [{"topic": "topic_1"}, {"topic": "topic_2"}]
+            },
             "repoProfileByNamespaceAndRepository": {
                 "description": "Repository Description 2",
                 "license": None,
-Original file line number
+Diff line change
@@ Expand Up / @@ -197,7 +197,7 @@ def get_headers(): @@
           repository
           repoTopicsByNamespaceAndRepository {
             nodes {
-              topics
+              topic
             }
           }
           repoProfileByNamespaceAndRepository {
@@ Expand Down @@