6136c1
From 134a415c2d690e57d0f1add23e900e60bcef4627 Mon Sep 17 00:00:00 2001
6136c1
From: Martin Wilck <mwilck@suse.com>
6136c1
Date: Tue, 24 Apr 2018 18:09:50 +0200
6136c1
Subject: [PATCH] test/udev-test.pl: Make some tests a little harder
6136c1
6136c1
Add some rules that make it a bit harder to pass, mainly the
6136c1
non-existence checks.
6136c1
6136c1
(cherry picked from commit 06d4d4e24e7d0b51120b165e540d278842e8b1a3)
6136c1
6136c1
Related: #1642728
6136c1
---
6136c1
 test/udev-test.pl | 13 +++++++++----
6136c1
 1 file changed, 9 insertions(+), 4 deletions(-)
6136c1
6136c1
diff --git a/test/udev-test.pl b/test/udev-test.pl
6136c1
index a9c2dd95f1..7465b5859e 100755
6136c1
--- a/test/udev-test.pl
6136c1
+++ b/test/udev-test.pl
6136c1
@@ -1358,7 +1358,7 @@ EOF
6136c1
                                 exp_links       => ["sda-part-1"],
6136c1
                         }],
6136c1
                 rules           => <
6136c1
-SUBSYSTEMS=="scsi", KERNEL=="sda1", SYMLINK+="%P-part-1"
6136c1
+SUBSYSTEMS=="scsi", KERNEL=="sda1", SYMLINK+="%P-part-%n"
6136c1
 EOF
6136c1
         },
6136c1
         {
6136c1
@@ -1486,6 +1486,7 @@ SUBSYSTEMS=="scsi", KERNEL=="sda1", ENV{ASSIGN}="true"
6136c1
 SUBSYSTEMS=="scsi", KERNEL=="sda1", ENV{ASSIGN}="absolutely-\$env{ASSIGN}"
6136c1
 SUBSYSTEMS=="scsi", KERNEL=="sda1", SYMLINK+="before"
6136c1
 SUBSYSTEMS=="scsi", KERNEL=="sda1", ENV{ASSIGN}=="yes", SYMLINK+="no"
6136c1
+SUBSYSTEMS=="scsi", KERNEL=="sda1", ENV{ASSIGN}=="true", SYMLINK+="bad"
6136c1
 SUBSYSTEMS=="scsi", KERNEL=="sda1", ENV{ASSIGN}=="absolutely-true", SYMLINK+="true"
6136c1
 EOF
6136c1
         },
6136c1
@@ -1497,6 +1498,11 @@ EOF
6136c1
                                 exp_links       => ["part"],
6136c1
                                 not_exp_links   => ["disk"],
6136c1
                         },
6136c1
+                        {
6136c1
+                                devpath         => "/devices/pci0000:00/0000:00:1f.2/host0/target0:0:0/0:0:0:0/block/sda",
6136c1
+                                exp_links       => ["disk"],
6136c1
+                                not_exp_links   => ["part"],
6136c1
+                        },
6136c1
                     ],
6136c1
                 rules           => <
6136c1
 SUBSYSTEM=="block", KERNEL=="*[0-9]", ENV{PARTITION}="true", ENV{MAINDEVICE}="false"
6136c1
@@ -1588,7 +1594,7 @@ EOF
6136c1
                         }],
6136c1
                 rules           => <
6136c1
 KERNEL=="sda", GROUP:="tty"
6136c1
-KERNEL=="sda", GROUP="not-ok", MODE="0640", SYMLINK+="ok"
6136c1
+KERNEL=="sda", GROUP="root", MODE="0640", SYMLINK+="ok"
6136c1
 EOF
6136c1
         },
6136c1
         {
6136c1
@@ -1602,7 +1608,7 @@ EOF
6136c1
                 rules           => <
6136c1
 KERNEL=="sda", GROUP:="tty"
6136c1
 SUBSYSTEM=="block", MODE:="640"
6136c1
-KERNEL=="sda", GROUP="not-ok", MODE="0666", SYMLINK+="ok"
6136c1
+KERNEL=="sda", GROUP="root", MODE="0666", SYMLINK+="ok"
6136c1
 EOF
6136c1
         },
6136c1
         {
6136c1
@@ -1983,7 +1989,6 @@ EOF
6136c1
                         {
6136c1
                                 devpath         => "/devices/pci0000:00/0000:00:1f.2/host0/target0:0:0/0:0:0:0/block/sda",
6136c1
                                 exp_perms       => "1:1:0400",
6136c1
-                                exp_rem_error   => "yes",
6136c1
                         }],
6136c1
                 rules           => <
6136c1
 KERNEL=="sda", MODE="666"