Athemis/doi2dataset
1
0
Fork 0
Code Issues Pull requests Projects Releases 3 Packages Wiki Activity Actions

fix: Remove redundant topic name extraction

Browse source
This commit is contained in:
Alexander Minges 2025-07-10 15:13:51 +02:00
parent d8036fea2f
commit 2686d310a0
Signed by: Athemis
SSH key fingerprint: SHA256:TUXshgulbwL+FRYvBNo54pCsI0auROsSEgSvueKbkZ4
1 changed files with 0 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

5
tests/test_fetch_doi_mock.py
View file

@ -97,11 +97,6 @@ def test_subject_mapper(fake_openalex_response):
# Extract topics from the OpenAlex response # Extract topics from the OpenAlex response
topics = fake_openalex_response.get("topics", []) topics = fake_openalex_response.get("topics", [])
# Convert topics to strings - we'll use display_name
topic_names = []
if topics:
topic_names = [topic.get("display_name") for topic in topics if topic.get("display_name")]
# Get subjects using the class method # Get subjects using the class method
subjects = SubjectMapper.get_subjects({"topics": topics}) subjects = SubjectMapper.get_subjects({"topics": topics})