Skip to content

Ql4ql: Quality query tagging. #19931

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

Merged
michaelnebel merged 10 commits into from
Jul 17, 2025
Merged

Ql4ql: Quality query tagging. #19931

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
cce1774
Ql4Ql: Re-factor the ql/mising-security-metadata query.
michaelnebel Jun 30, 2025
c46b528
Ql4Ql: Add some quality tag testcases.
michaelnebel Jun 30, 2025
e00b535
Ql4Ql: Add a check for quality tag consistency.
michaelnebel Jun 30, 2025
60a1d02
Ql4Ql: Add MissingQualityMetadata test.
michaelnebel Jun 30, 2025
af1c4e0
Ql4Ql: Share the definition of TestFile between multiple tests.
michaelnebel Jul 1, 2025
f58064e
Ql4Ql: Address review comments.
michaelnebel Jul 2, 2025
b79e2dd
Ql4Ql: Add some more quality tag testcases.
michaelnebel Jul 3, 2025
f810e17
Ql4Ql: Address review comments and update expected test output.
michaelnebel Jul 3, 2025
aefd941
Java/Javascript: Fix violations.
michaelnebel Jul 3, 2025
11c4a63
Quality tags: Clarify the quality sub-category tagging policy.
michaelnebel Jul 3, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Ql4Ql: Address review comments and update expected test output.
  • Loading branch information
@michaelnebel
michaelnebel committed Jul 3, 2025
commit f810e17d9ee6956fe74bac548853aa3fa806e345
23 changes: 14 additions & 9 deletions ql/ql/src/queries/style/MissingQualityMetadata.ql
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -34,14 +34,19 @@ where
msg =
"This query file has incorrect top-level categorisation. It should have exactly one top-level category, either `@tags maintainability` or `@tags reliability`."
or
maintainabilitySubCategory(doc) and
not doc.getAQueryTag() = "maintainability" and
msg =
"This query file has a sub-category of maintainability but is missing the `@tags maintainability` tag."
or
reliabilitySubCategory(doc) and
not doc.getAQueryTag() = "reliability" and
msg =
"This query file has a sub-category of reliability but is missing the `@tags reliability` tag."
correctTopLevelCategorisation(doc) and
(
doc.getAQueryTag() = "reliability" and
not reliabilitySubCategory(doc) and
maintainabilitySubCategory(doc) and
msg =
"This query file has a sub-category of maintainability but has the `@tags reliability` tag."
or
doc.getAQueryTag() = "maintainability" and
not maintainabilitySubCategory(doc) and
reliabilitySubCategory(doc) and
msg =
"This query file has a sub-category of reliability but has the `@tags maintainability` tag."
)
)
select doc, msg
6 changes: 2 additions & 4 deletions ql/ql/test/queries/style/MissingQualityMetadata/MissingQualityMetadata.expected
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,4 @@
| testcases/BadQualityMaintainabilityWrongToplevel.ql:1:1:11:3 | QueryDoc | This query file has a sub-category of reliability but is missing the `@tags reliability` tag. |
| testcases/BadQualityMaintainabilityWrongToplevel.ql:1:1:11:3 | QueryDoc | This query file has a sub-category of reliability but has the `@tags maintainability` tag. |
| testcases/BadQualityMultipleTopLevel.ql:1:1:11:3 | QueryDoc | This query file has incorrect top-level categorisation. It should have exactly one top-level category, either `@tags maintainability` or `@tags reliability`. |
| testcases/BadQualityNoToplevel.ql:1:1:10:3 | QueryDoc | This query file has incorrect top-level categorisation. It should have exactly one top-level category, either `@tags maintainability` or `@tags reliability`. |
| testcases/BadQualityReliabilityWrongToplevel.ql:1:1:11:3 | QueryDoc | This query file has a sub-category of maintainability but is missing the `@tags maintainability` tag. |
| testcases/GoodQualityMaintainabilityWithCrossSub.ql:1:1:12:3 | QueryDoc | This query file has a sub-category of reliability but is missing the `@tags reliability` tag. |
| testcases/GoodQualityReliabilityWithCrossSub.ql:1:1:12:3 | QueryDoc | This query file has a sub-category of maintainability but is missing the `@tags maintainability` tag. |
| testcases/BadQualityReliabilityWrongToplevel.ql:1:1:11:3 | QueryDoc | This query file has a sub-category of maintainability but has the `@tags reliability` tag. |