Skip to content

Fix multiple display additional test 2758#2761

Open
prakhyatox wants to merge 6 commits intodevfrom
fix-multiple-display-additional-test-2758
Open

Fix multiple display additional test 2758#2761
prakhyatox wants to merge 6 commits intodevfrom
fix-multiple-display-additional-test-2758

Conversation

@prakhyatox
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 5, 2026

Coverage Report for Vitest Coverage

Status Category Percentage Covered / Total
🔵 Lines 96.23% (🎯 90%) 38808 / 40326
🔵 Statements 96.23% (🎯 90%) 38808 / 40326
🔵 Functions 90.64% (🎯 90%) 1861 / 2053
🔵 Branches 95.6% (🎯 90%) 3654 / 3822
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
src/main.js 97.33% 100% 50% 97.33%
src/components/Editor/AdditionalInformation/EditAdditionalInfo.vue 99.8% 97.91% 94.44% 99.8%
src/components/Records/Record/AdditionalInfo.vue 100% 100% 100% 100%
src/components/Records/Record/AdditionalInfo/DatasetArray.vue 97.72% 91.66% 77.77% 97.72%
src/utils/stringUtils.js 96.66% 91.66% 100% 96.66%
Generated in workflow #6971 for commit 6a6ce0b by the Vitest Coverage Report Action

@allysonlister
Copy link
Copy Markdown
Contributor

allysonlister commented May 5, 2026

This is working as expected with regards to obsolete and duplicate tests. Just one question to ask Mark in today's meeting though - should the URL provided be the endpoint (as it currently is, e.g. https://api.fairsharing.org/test_descriptions/ft_f1_m_idgloballyunique/api) or the test ID (e.g. https://ostrails.github.io/assessment-component-metadata-records/test/FT_F1_M_IdGloballyUnique.ttl)

I'll get back to you shortly regarding this.

@allysonlister
Copy link
Copy Markdown
Contributor

@knirirr has put the answer at https://github.com/FAIRsharing/FAIRsharing-API/issues/1572#issuecomment-4379057010 (we need to use dcterms:identifier instead of the endpoint URL) thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants