Skip to content

test: cover glossary CSV typed relations across unit, IT, Playwright #27740

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
harshach wants to merge 5 commits into
base: main
Choose a base branch
from

test(glossary-csv): address Copilot review feedback on typed-relation…

5d12c9f
Select commit
Loading
Failed to load commit list.
Open

test: cover glossary CSV typed relations across unit, IT, Playwright #27740

test(glossary-csv): address Copilot review feedback on typed-relation…
5d12c9f
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / [open-metadata-ui] SonarCloud Code Analysis succeeded Apr 29, 2026 in 6m 17s

Quality Gate passed

Issues
1072 New issues
0 Accepted issues

Measures
17 Security Hotspots
0.0% Coverage on New Code
3.8% Duplication on New Code

See analysis details on SonarQube Cloud

Annotations

Check failure on line 161 in src/components/OntologyExplorer/OntologyExplorer.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 23 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3KvkIzMMAbgOav56OY&open=AZ3KvkIzMMAbgOav56OY&pullRequest=27740

Check warning on line 71 in src/components/Explore/EntitySummaryPanel/CommonEntitySummaryInfo/CommonEntitySummaryInfo.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Extract this nested ternary operation into an independent statement. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3KviuUMMAbgOav56MQ&open=AZ3KviuUMMAbgOav56MQ&pullRequest=27740

Check warning on line 23 in src/utils/JSONSchemaFormUtils.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer `String#replaceAll()` over `String#replace()`. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3KvmX4MMAbgOav56Sj&open=AZ3KvmX4MMAbgOav56Sj&pullRequest=27740

Check warning on line 295 in src/utils/CommonUtils.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Unexpected negated condition. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3Kvm2AMMAbgOav56Tk&open=AZ3Kvm2AMMAbgOav56Tk&pullRequest=27740

Check warning on line 975 in src/components/DataContract/ODCSImportModal/ODCSImportModal.component.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Do not use Array index in keys 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3Kvj2nMMAbgOav56N1&open=AZ3Kvj2nMMAbgOav56N1&pullRequest=27740

Check warning on line 248 in src/components/common/NavigationBlocker/NavigationBlocker.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer `globalThis` over `window`. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3KvffDMMAbgOav56Hu&open=AZ3KvffDMMAbgOav56Hu&pullRequest=27740

Check warning on line 83 in src/components/WorkflowDefinitions/WorkflowBuilder/forms/QueryBuilderSection.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer `String#replaceAll()` over `String#replace()`. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3KvhqtMMAbgOav56Kp&open=AZ3KvhqtMMAbgOav56Kp&pullRequest=27740

Check warning on line 73 in src/components/BlockEditor/Extensions/File/FileNode.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer `Number.parseInt` over `parseInt`. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3KveJ-MMAbgOav56Ff&open=AZ3KveJ-MMAbgOav56Ff&pullRequest=27740

Check warning on line 225 in src/components/DataQuality/AddDataQualityTest/components/AddTestSuitePipeline.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Unexpected negated condition. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3KvgjfMMAbgOav56I8&open=AZ3KvgjfMMAbgOav56I8&pullRequest=27740

Check warning on line 285 in src/components/MyData/CustomizableComponents/CustomiseLandingPageHeader/CustomiseLandingPageHeader.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Use <input type="button">, <input type="image">, <input type="reset">, <input type="submit">, or <button> instead of the "button" role to ensure accessibility across all devices. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3KvgzpMMAbgOav56JV&open=AZ3KvgzpMMAbgOav56JV&pullRequest=27740

Check warning on line 102 in src/components/common/NavigationBlocker/NavigationBlocker.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer `globalThis` over `window`. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3KvffDMMAbgOav56Hd&open=AZ3KvffDMMAbgOav56Hd&pullRequest=27740

Check warning on line 133 in src/hooks/useGridEditController.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer `childNode.remove()` over `parentNode.removeChild(childNode)`. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3KvnsaMMAbgOav56Vd&open=AZ3KvnsaMMAbgOav56Vd&pullRequest=27740

Check warning on line 34 in src/components/ActivityFeed/ActivityFeedList/TaskListV1.component.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

'isFullWidth' PropType is defined but prop is never used 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3KvhXjMMAbgOav56KH&open=AZ3KvhXjMMAbgOav56KH&pullRequest=27740

Check failure on line 250 in src/components/SettingsSso/SettingsSso.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 21 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3KvjJyMMAbgOav56M4&open=AZ3KvjJyMMAbgOav56M4&pullRequest=27740

Check warning on line 134 in src/pages/AddQueryPage/AddQueryPage.component.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Handle this exception or don't catch it at all. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3KvlYvMMAbgOav56Qn&open=AZ3KvlYvMMAbgOav56Qn&pullRequest=27740

Check warning on line 114 in src/components/WorkflowDefinitions/WorkflowBuilder/NodeConfigSidebar.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Unexpected negated condition. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3KvhurMMAbgOav56K1&open=AZ3KvhurMMAbgOav56K1&pullRequest=27740

Check warning on line 36 in src/hooks/useClipBoard.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer `childNode.remove()` over `parentNode.removeChild(childNode)`. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3KvnmpMMAbgOav56VM&open=AZ3KvnmpMMAbgOav56VM&pullRequest=27740

Check warning on line 102 in src/rest/suggestionsAPI.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

This assertion is unnecessary since it does not change the type of the expression. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3Kvn0jMMAbgOav56V-&open=AZ3Kvn0jMMAbgOav56V-&pullRequest=27740

Check warning on line 160 in src/components/common/Form/JSONSchema/JsonSchemaWidgets/QueryBuilderWidget/QueryBuilderWidget.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Unexpected negated condition. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3KvfSeMMAbgOav56HI&open=AZ3KvfSeMMAbgOav56HI&pullRequest=27740

Check warning on line 213 in src/components/Glossary/GlossaryHeader/GlossaryHeader.component.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Unexpected negated condition. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3KvijlMMAbgOav56MD&open=AZ3KvijlMMAbgOav56MD&pullRequest=27740

Check failure on line 281 in src/utils/BlockEditorUtils.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer `.dataset` over `setAttribute(…)`. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3KvmUlMMAbgOav56Sf&open=AZ3KvmUlMMAbgOav56Sf&pullRequest=27740

Check failure on line 315 in src/utils/SchedularUtils.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer `throw error` over `return Promise.reject(error)`. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3KvnGAMMAbgOav56UG&open=AZ3KvnGAMMAbgOav56UG&pullRequest=27740

Check warning on line 87 in src/components/common/PopOverCard/EntityPopOverCard.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Handle this exception or don't catch it at all. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3KvgC0MMAbgOav56IZ&open=AZ3KvgC0MMAbgOav56IZ&pullRequest=27740

Check failure on line 716 in src/components/common/Table/TableV2.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Refactor this code to not nest functions more than 4 levels deep. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3KvfBIMMAbgOav56Gx&open=AZ3KvfBIMMAbgOav56Gx&pullRequest=27740

Check warning on line 71 in src/utils/ApplicationUtils.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

`filteredRow` should be a `Set`, and use `filteredRow.has()` to check existence or non-existence. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3KvnOVMMAbgOav56Ua&open=AZ3KvnOVMMAbgOav56Ua&pullRequest=27740
View more details on SonarQubeCloud