Skip to content

Commit

Permalink
set definition uri obligatory to false - update test document percent…
Browse files Browse the repository at this point in the history
… complete because removing definition uri from obligatory changes det percent completed (#356)
  • Loading branch information
tilen1976 authored Jul 1, 2024
1 parent a5ccb4e commit 778b88a
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/datadoc/frontend/fields/display_variables.py
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,7 @@ class VariableIdentifiers(str, Enum):
identifier=VariableIdentifiers.DEFINITION_URI.value,
display_name="Definition URI",
description="Oppgi lenke (URI) til tilhørende variabel i VarDef.",
obligatory=True,
obligatory=False,
),
VariableIdentifiers.DIRECT_PERSON_IDENTIFYING: MetadataCheckboxField(
identifier=VariableIdentifiers.DIRECT_PERSON_IDENTIFYING.value,
Expand Down
2 changes: 1 addition & 1 deletion tests/backend/test_datadoc_metadata.py
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,7 @@ def test_metadata_document_percent_complete(metadata: DataDocMetadata):
metadata.dataset = document.dataset # type: ignore [assignment]
metadata.variables = document.variables # type: ignore [assignment]

assert metadata.percent_complete == 11 # noqa: PLR2004
assert metadata.percent_complete == 12 # noqa: PLR2004


def test_write_metadata_document(
Expand Down

0 comments on commit 778b88a

Please sign in to comment.