Skip to content

Commit 91e1458

Browse files
committed
writing a test and integrating the middle name
1 parent 114bef1 commit 91e1458

File tree

2 files changed

+23
-1
lines changed

2 files changed

+23
-1
lines changed

bids2openminds/main.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ def create_openminds_person(full_name):
2121
family_name = person.last
2222

2323
if person.middle:
24-
alternate_names.append(person.full_name)
24+
given_name = f"{given_name} {person.middle}"
2525

2626
if person.nickname:
2727
alternate_names.append(person.nickname)

test/test_person.py

+22
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,22 @@
1+
# test for create_openminds_person function in the main
2+
import pytest
3+
from bids2openminds.main import create_openminds_person
4+
import openminds.latest.core as omcore
5+
6+
# Test data: (full_name, given_name, family_name)
7+
example_names = [("John Ronald Reuel Tolkien", "John Ronald Reuel", "Tolkien"),
8+
("Bilbo Baggins", "Bilbo", "Baggins")]
9+
10+
11+
@pytest.mark.parametrize("full_name,given_name,family_name", example_names)
12+
def test_create_openminds_person(full_name, given_name, family_name):
13+
openminds_person_object = omcore.Person(given_name=given_name,
14+
family_name=family_name)
15+
bids2openminds_person_object = create_openminds_person(full_name)
16+
assert openminds_person_object.given_name == bids2openminds_person_object.given_name, \
17+
f"Given names don't match for input '{full_name}'"
18+
assert openminds_person_object.family_name == bids2openminds_person_object.family_name, \
19+
f"Family names don't match for input '{full_name}'"
20+
assert openminds_person_object.type_ == bids2openminds_person_object.type_, \
21+
f"Person types don't match for input '{full_name}'"
22+
# assert openminds_person_object == bids2openminds_person_object

0 commit comments

Comments
 (0)