Skip to content

Commit

Permalink
Fix: one more test case
Browse files Browse the repository at this point in the history
Fix: remove additional alias

Fix: pynteny
  • Loading branch information
lwasser committed Mar 9, 2024
1 parent 0eb0a1d commit 114e8ef
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 5 deletions.
4 changes: 1 addition & 3 deletions src/pyosmeta/models.py
Original file line number Diff line number Diff line change
Expand Up @@ -220,9 +220,7 @@ class ReviewModel(BaseModel):
version_accepted: str | None = None
date_accepted: str | None = Field(
default=None,
validation_alias=AliasChoices(
"date_accepted_(month/day/year)", "Date accepted", "date_accepted"
),
validation_alias=AliasChoices("Date accepted", "date_accepted"),
)
created_at: str = None
updated_at: str = None
Expand Down
2 changes: 0 additions & 2 deletions src/pyosmeta/parse_issues.py
Original file line number Diff line number Diff line change
Expand Up @@ -146,8 +146,6 @@ def parse_issue_header(
for issue in issues:
# Return issue comment as cleaned list + package name
pkg_name, body_data = self.comment_to_list(issue)
if pkg_name.lower() == "pynteny":
print("halt")
if not pkg_name:
continue

Expand Down
4 changes: 4 additions & 0 deletions tests/unit/test_utils_clean.py
Original file line number Diff line number Diff line change
Expand Up @@ -66,6 +66,10 @@ def test_clean_name(input_name, expected_output):
"input_dict, expected_output",
[
# Test case where key starts with "date_accepted"
(
{"date_accepted_(month-day-year)": "2024-03-07"},
{"date_accepted": "2024-03-07"},
),
(
{"date_accepted_(month/day/year)": "2024/03/07"},
{"date_accepted": "2024/03/07"},
Expand Down

0 comments on commit 114e8ef

Please sign in to comment.