Skip to content

Commit

Permalink
Merge pull request #303 from linkml/linkml-issue-1942
Browse files Browse the repository at this point in the history
Fixed CURIE validation to be conformant.
  • Loading branch information
cmungall authored Feb 27, 2024
2 parents 5fa3115 + 27b9158 commit 7c311d9
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 5 deletions.
2 changes: 0 additions & 2 deletions linkml_runtime/utils/metamodelcore.py
Original file line number Diff line number Diff line change
Expand Up @@ -177,8 +177,6 @@ def ns_ln(cls, v: str) -> Optional[Tuple[str, str]]:
if not NCName.is_valid(prefix):
return None
reference = curie_split[1]
if not cls.term_name.match(reference):
return None
return prefix, reference

@classmethod
Expand Down
5 changes: 2 additions & 3 deletions tests/test_utils/test_metamodelcore.py
Original file line number Diff line number Diff line change
Expand Up @@ -66,12 +66,11 @@ def test_curie(self):
self.assertTrue(Curie.is_valid(':type'))
self.assertTrue(Curie.is_valid('WIKIDATA_PROPERTY:P854'))
self.assertTrue(Curie.is_valid('WIKIDATA.PROPERTY:P854'))
self.assertTrue(Curie.is_valid('CL:0000001'))
with self.assertRaises(ValueError):
Curie("1df:type")
self.assertFalse(Curie.is_valid('1df:type'))
with self.assertRaises(ValueError):
Curie("rdf:17")
self.assertFalse(Curie.is_valid('rdf:17'))
self.assertTrue(Curie.is_valid('rdf:17'))
nsm = Namespaces(Graph())
self.assertEqual(RDF.type, Curie("rdf:type").as_uri(nsm))
self.assertIsNone(Curie("ex:foo").as_uri(nsm))
Expand Down

0 comments on commit 7c311d9

Please sign in to comment.