Skip to content

Commit 1b13baa

Browse files
committed
update f string
2 parents 1210ac7 + f9e3462 commit 1b13baa

File tree

3 files changed

+13
-25
lines changed

3 files changed

+13
-25
lines changed

nmdc_server/api.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1270,7 +1270,7 @@ def create_github_issue(submission: schemas_submission.SubmissionMetadataSchema,
12701270
f"Has data been generated: {data_generated}",
12711271
f"PI name: {pi_name}",
12721272
f"PI orcid: {pi_orcid}",
1273-
f"Status: {submission.status}",
1273+
f"Status: {SubmissionStatusEnum['SubmittedPendingReview'].title}",
12741274
f"Data types: {omics_processing_types}",
12751275
f"Sample type: {sample_types}",
12761276
f"Number of samples: {num_samples}",

nmdc_server/models.py

Lines changed: 11 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,10 @@
11
import enum
22
from datetime import datetime
3+
from importlib import resources
34
from typing import Any, Dict, Iterator, List, Optional, Type, Union
45
from uuid import uuid4
56

6-
from nmdc_schema.nmdc_data import get_nmdc_schema_definition
7+
from linkml_runtime.utils.schemaview import SchemaView
78
from sqlalchemy import (
89
BigInteger,
910
Boolean,
@@ -37,8 +38,15 @@
3738

3839
def get_submission_status_enum():
3940
"""Get SubmissionStatusEnum from the NMDC schema definition."""
40-
schema = get_nmdc_schema_definition()
41-
return schema.enums["SubmissionStatusEnum"].permissible_values
41+
permissible_values = {}
42+
submission_schema_files = resources.files("nmdc_schema")
43+
schema_path = submission_schema_files / "nmdc_materialized_patterns.yaml"
44+
sv = SchemaView(str(schema_path))
45+
for enum_name, enum_definition in sv.all_enums().items():
46+
if enum_name == "SubmissionStatusEnum":
47+
permissible_values = enum_definition.permissible_values
48+
break
49+
return permissible_values
4250

4351

4452
SubmissionStatusEnum = get_submission_status_enum()

tests/test_submission.py

Lines changed: 1 addition & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -3,21 +3,13 @@
33
from datetime import datetime, timedelta
44

55
import pytest
6-
from nmdc_schema.nmdc_data import get_nmdc_schema_definition
76
from sqlalchemy.orm.session import Session
87
from starlette.testclient import TestClient
98

109
from nmdc_server import fakes
11-
from nmdc_server.models import SubmissionEditorRole, SubmissionRole
10+
from nmdc_server.models import SubmissionEditorRole, SubmissionRole, get_submission_status_enum
1211
from nmdc_server.schemas_submission import SubmissionMetadataSchema, SubmissionMetadataSchemaPatch
1312

14-
15-
def get_submission_status_enum():
16-
"""Get SubmissionStatusEnum from the NMDC schema definition."""
17-
schema = get_nmdc_schema_definition()
18-
return schema.enums["SubmissionStatusEnum"].permissible_values
19-
20-
2113
SubmissionStatusEnum = get_submission_status_enum()
2214

2315

@@ -55,18 +47,6 @@ def test_get_metadata_submissions_mixs_as_non_admin(
5547
assert response.status_code == 403
5648

5749

58-
def test_print_submission_enum(db: Session, client: TestClient, logged_in_user):
59-
submission = fakes.MetadataSubmissionFactory(
60-
author=logged_in_user, author_orcid=logged_in_user.orcid
61-
)
62-
print("Available submission status enum values:")
63-
print(SubmissionStatusEnum)
64-
print("\nSpecific enum value string")
65-
print(SubmissionStatusEnum["InProgress"].title)
66-
67-
assert submission
68-
69-
7050
def test_get_metadata_submissions_mixs_as_admin(
7151
db: Session, client: TestClient, logged_in_admin_user
7252
):

0 commit comments

Comments
 (0)