felipe/apparmor.d
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

tests(check): add a check to ensure all udev/data access are documented.

Browse source
This commit is contained in:
Alexandre Pujol 2025-08-11 16:18:58 +02:00
parent 175e2c3dc3
commit 616486d5ba
No known key found for this signature in database
GPG key ID: C5469996F0DF68EC
1 changed files with 13 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

16
tests/check.sh
View file

@ -108,6 +108,7 @@ _check() {
_check_trailing _check_trailing
_check_indentation _check_indentation
_check_vim _check_vim
_check_udev
# The following checks do not apply to commented lines # The following checks do not apply to commented lines
[[ "$line" =~ ^[[:space:]]*# ]] && continue [[ "$line" =~ ^[[:space:]]*# ]] && continue
@ -485,6 +486,15 @@ _res_vim() {
fi fi
} }
_check_udev() {
_is_enabled udev || return 0
if [[ "$line" == *"@{run}/udev/data/"* ]]; then
if [[ "$line" != *"#"* ]]; then
_err udev "$file:$line_number" "udev data path without a description comment"
fi
fi
}
check_sbin() { check_sbin() {
local file name jobs local file name jobs
mapfile -t sbin <tests/sbin.list mapfile -t sbin <tests/sbin.list
@ -531,7 +541,7 @@ check_profiles() {
jobs=0 jobs=0
WITH_CHECK=( WITH_CHECK=(
abstractions directory-mark equivalent too-wide useless transition tunables abstractions directory-mark equivalent too-wide useless transition tunables
abi include profile header tabs trailing indentation subprofiles vim abi include profile header tabs trailing indentation subprofiles vim udev
) )
for file in "${files[@]}"; do for file in "${files[@]}"; do
( (
@ -551,7 +561,7 @@ check_abstractions() {
jobs=0 jobs=0
WITH_CHECK=( WITH_CHECK=(
abstractions directory-mark equivalent too-wide tunables abstractions directory-mark equivalent too-wide tunables
abi include header tabs trailing indentation vim abi include header tabs trailing indentation vim udev
) )
for file in "${files[@]}"; do for file in "${files[@]}"; do
( (
@ -572,7 +582,7 @@ check_abstractions() {
jobs=0 jobs=0
WITH_CHECK=( WITH_CHECK=(
abstractions directory-mark equivalent too-wide tunables abstractions directory-mark equivalent too-wide tunables
header tabs trailing indentation vim header tabs trailing indentation vim udev
) )
for file in "${files[@]}"; do for file in "${files[@]}"; do
_check "$file" & _check "$file" &