chiark / gitweb /
remove no longer working udevd-test program
[elogind.git] / test / udev-test.pl
old mode 100644 (file)
new mode 100755 (executable)
index 52c5c8d..44bc483
@@ -9,7 +9,7 @@
 # Every test is driven by its own temporary config file.
 # This program prepares the environment, creates the config and calls udev.
 #
-# udev reads the config, looks at the provided sysfs and
+# udev parses the rules, looks at the provided sysfs and
 # first creates and then removes the device node.
 # After creation and removal the result is checked against the
 # expected value and the result is printed.
 use warnings;
 use strict;
 
-my $PWD = $ENV{PWD};
-my $sysfs     = "sys/";
-my $udev_bin  = "../udev";
-my $udev_root = "udev-root/"; # !!! directory will be removed !!!
-my $udev_db   = ".udevdb";
-my $main_conf = "udev-test.conf";
-my $conf_tmp  = "udev-test.rules";
+my $PWD                = $ENV{PWD};
+my $sysfs      = "sys/";
+my $udev_bin   = "../udev";
+my $udev_root  = "udev-root/"; # !!! directory will be removed !!!
+my $udev_db    = ".udevdb";
+my $udev_conf  = "udev-test.conf";
+my $udev_rules = "udev-test.rules";
 
 # uncomment following line to run udev with valgrind.
 # Should make this a runtime option to the script someday...
@@ -38,9 +38,9 @@ my @tests = (
                subsys          => "block",
                devpath         => "/block/sda",
                exp_name        => "boot_disk" ,
-               conf            => <<EOF
-BUS="scsi", SYSFS{vendor}="IBM-ESXS", NAME="boot_disk%n"
-KERNEL="ttyUSB0", NAME="visor"
+               rules           => <<EOF
+BUS=="scsi", SYSFS{vendor}=="IBM-ESXS", NAME="boot_disk%n"
+KERNEL=="ttyUSB0", NAME="visor"
 EOF
        },
        {
@@ -48,8 +48,8 @@ EOF
                subsys          => "block",
                devpath         => "/block/sda/sda1",
                exp_name        => "boot_disk1" ,
-               conf            => <<EOF
-BUS="scsi", SYSFS{vendor}="IBM-ESXS", NAME="boot_disk%n"
+               rules           => <<EOF
+BUS=="scsi", SYSFS{vendor}=="IBM-ESXS", NAME="boot_disk%n"
 EOF
        },
        {
@@ -57,11 +57,11 @@ EOF
                subsys          => "block",
                devpath         => "/block/sda/sda1",
                exp_name        => "boot_disk1" ,
-               conf            => <<EOF
-BUS="scsi", SYSFS{vendor}="?IBM-ESXS", NAME="boot_disk%n-1"
-BUS="scsi", SYSFS{vendor}="IBM-ESXS?", NAME="boot_disk%n-2"
-BUS="scsi", SYSFS{vendor}="IBM-ES??", NAME="boot_disk%n"
-BUS="scsi", SYSFS{vendor}="IBM-ESXSS", NAME="boot_disk%n-3"
+               rules           => <<EOF
+BUS=="scsi", SYSFS{vendor}=="?IBM-ESXS", NAME="boot_disk%n-1"
+BUS=="scsi", SYSFS{vendor}=="IBM-ESXS?", NAME="boot_disk%n-2"
+BUS=="scsi", SYSFS{vendor}=="IBM-ES??", NAME="boot_disk%n"
+BUS=="scsi", SYSFS{vendor}=="IBM-ESXSS", NAME="boot_disk%n-3"
 EOF
        },
        {
@@ -69,9 +69,9 @@ EOF
                subsys          => "block",
                devpath         => "/block/sda/sda1",
                exp_name        => "boot_disk1" ,
-               conf            => <<EOF
-BUS="scsi", SYSFS{vendor}="IBM-ESXS", SYSFS{model}="ST336605LW   !#", NAME="boot_diskX%n"
-BUS="scsi", SYSFS{vendor}="IBM-ESXS", SYSFS{model}="ST336605LW    !#", NAME="boot_disk%n"
+               rules           => <<EOF
+BUS=="scsi", SYSFS{vendor}=="IBM-ESXS", SYSFS{model}=="ST336605LW   !#", NAME="boot_diskX%n"
+BUS=="scsi", SYSFS{vendor}=="IBM-ESXS", SYSFS{model}=="ST336605LW    !#", NAME="boot_disk%n"
 EOF
        },
        {
@@ -79,9 +79,9 @@ EOF
                subsys          => "block",
                devpath         => "/block/sda/sda1",
                exp_name        => "boot_disk1" ,
-               conf            => <<EOF
-BUS="scsi", SYSFS{vendor}="IBM-ESXS", SYSFS{model}="ST336605LW    !#", SYSFS{scsi_level}="4", SYSFS{rev}="B245", SYSFS{type}="2", SYSFS{queue_depth}="32", NAME="boot_diskXX%n"
-BUS="scsi", SYSFS{vendor}="IBM-ESXS", SYSFS{model}="ST336605LW    !#", SYSFS{scsi_level}="4", SYSFS{rev}="B245", SYSFS{type}="0", NAME="boot_disk%n"
+               rules           => <<EOF
+BUS=="scsi", SYSFS{vendor}=="IBM-ESXS", SYSFS{model}=="ST336605LW    !#", SYSFS{scsi_level}=="4", SYSFS{rev}=="B245", SYSFS{type}=="2", SYSFS{queue_depth}=="32", NAME="boot_diskXX%n"
+BUS=="scsi", SYSFS{vendor}=="IBM-ESXS", SYSFS{model}=="ST336605LW    !#", SYSFS{scsi_level}=="4", SYSFS{rev}=="B245", SYSFS{type}=="0", NAME="boot_disk%n"
 EOF
        },
        {
@@ -89,8 +89,8 @@ EOF
                subsys          => "tty",
                devpath         => "/class/tty/ttyUSB0",
                exp_name        => "visor/0" ,
-               conf            => <<EOF
-KERNEL="ttyUSB*", NAME="visor/%n"
+               rules           => <<EOF
+KERNEL=="ttyUSB*", NAME="visor/%n"
 EOF
        },
        {
@@ -98,9 +98,9 @@ EOF
                subsys          => "tty",
                devpath         => "/class/tty/ttyUSB0",
                exp_name        => "visor/0" ,
-               conf            => <<EOF
-KERNEL="*USB1", NAME="bad"
-KERNEL="*USB0", NAME="visor/%n"
+               rules           => <<EOF
+KERNEL=="*USB1", NAME="bad"
+KERNEL=="*USB0", NAME="visor/%n"
 EOF
        },
        {
@@ -108,10 +108,10 @@ EOF
                subsys          => "tty",
                devpath         => "/class/tty/ttyUSB0",
                exp_name        => "visor/0" ,
-               conf            => <<EOF
-KERNEL="ttyUSB??*", NAME="visor/%n-1"
-KERNEL="ttyUSB??", NAME="visor/%n-2"
-KERNEL="ttyUSB?", NAME="visor/%n"
+               rules           => <<EOF
+KERNEL=="ttyUSB??*", NAME="visor/%n-1"
+KERNEL=="ttyUSB??", NAME="visor/%n-2"
+KERNEL=="ttyUSB?", NAME="visor/%n"
 EOF
        },
        {
@@ -119,10 +119,10 @@ EOF
                subsys          => "tty",
                devpath         => "/class/tty/ttyUSB0",
                exp_name        => "visor/0" ,
-               conf            => <<EOF
-KERNEL="ttyUSB[A-Z]*", NAME="visor/%n-1"
-KERNEL="ttyUSB?[0-9]", NAME="visor/%n-2"
-KERNEL="ttyUSB[0-9]*", NAME="visor/%n"
+               rules           => <<EOF
+KERNEL=="ttyUSB[A-Z]*", NAME="visor/%n-1"
+KERNEL=="ttyUSB?[0-9]", NAME="visor/%n-2"
+KERNEL=="ttyUSB[0-9]*", NAME="visor/%n"
 EOF
        },
        {
@@ -130,8 +130,8 @@ EOF
                subsys          => "tty",
                devpath         => "/class/tty/ttyUSB0",
                exp_name        => "visor" ,
-               conf            => <<EOF
-KERNEL="ttyUSB0", NAME="visor"
+               rules           => <<EOF
+KERNEL=="ttyUSB0", NAME="visor"
 EOF
        },
        {
@@ -139,9 +139,9 @@ EOF
                subsys          => "tty",
                devpath         => "/class/tty/ttyUSB0",
                exp_name        => "visor" ,
-               conf            => <<EOF
+               rules           => <<EOF
 # this is a comment
-KERNEL="ttyUSB0", NAME="visor"
+KERNEL=="ttyUSB0", NAME="visor"
 
 EOF
        },
@@ -150,9 +150,9 @@ EOF
                subsys          => "tty",
                devpath         => "/class/tty/ttyUSB0",
                exp_name        => "visor" ,
-               conf            => <<EOF
+               rules           => <<EOF
  # this is a comment with whitespace before the comment 
-KERNEL="ttyUSB0", NAME="visor"
+KERNEL=="ttyUSB0", NAME="visor"
 
 EOF
        },
@@ -161,12 +161,12 @@ EOF
                subsys          => "tty",
                devpath         => "/class/tty/ttyUSB0",
                exp_name        => "whitespace" ,
-               conf            => <<EOF
+               rules           => <<EOF
 
  
 
  # this is a comment with whitespace before the comment 
-KERNEL="ttyUSB0", NAME="whitespace"
+KERNEL=="ttyUSB0", NAME="whitespace"
 
  
 
@@ -177,9 +177,9 @@ EOF
                subsys          => "tty",
                devpath         => "/class/tty/ttyUSB0",
                exp_name        => "visor" ,
-               conf            => <<EOF
+               rules           => <<EOF
 
-KERNEL="ttyUSB0", NAME="visor"
+KERNEL=="ttyUSB0", NAME="visor"
 
 EOF
        },
@@ -188,8 +188,8 @@ EOF
                subsys          => "tty",
                devpath         => "/class/tty/ttyUSB0",
                exp_name        => "visor" ,
-               conf            => <<EOF
-KERNEL="ttyUSB0", \\
+               rules           => <<EOF
+KERNEL=="ttyUSB0", \\
 NAME="visor"
 
 EOF
@@ -199,8 +199,8 @@ EOF
                subsys          => "tty",
                devpath         => "/class/tty/ttyUSB0",
                exp_name        => "aaa",
-               conf            => <<EOF
-KERNEL="ttyUSB0", PROGRAM="/bin/echo -e \\101", RESULT="A", NAME="aaa"
+               rules           => <<EOF
+KERNEL=="ttyUSB0", PROGRAM=="/bin/echo -e \\101", RESULT=="A", NAME="aaa"
 EOF
        },
        {
@@ -208,7 +208,7 @@ EOF
                subsys          => "tty",
                devpath         => "/class/tty/ttyUSB0",
                exp_name        => "visor" ,
-               conf            => <<EOF
+               rules           => <<EOF
 
 #
 \\
@@ -217,7 +217,7 @@ EOF
 
 #\\
 
-KERNEL="ttyUSB0", \\
+KERNEL=="ttyUSB0", \\
        NAME="visor"
 
 EOF
@@ -227,8 +227,8 @@ EOF
                subsys          => "tty",
                devpath         => "/class/tty/ttyUSB0",
                exp_name        => "sub/direct/ory/visor" ,
-               conf            => <<EOF
-KERNEL="ttyUSB0", NAME="sub/direct/ory/visor"
+               rules           => <<EOF
+KERNEL=="ttyUSB0", NAME="sub/direct/ory/visor"
 EOF
        },
        {
@@ -236,35 +236,46 @@ EOF
                subsys          => "block",
                devpath         => "/block/sda/sda3",
                exp_name        => "first_disk3" ,
-               conf            => <<EOF
-BUS="scsi", ID="0:0:0:0", NAME="first_disk%n"
+               rules           => <<EOF
+BUS=="scsi", ID=="0:0:0:0", NAME="first_disk%n"
 EOF
        },
        {
-               desc            => "test NAME substitution chars",
+               desc            => "test substitution chars",
                subsys          => "block",
                devpath         => "/block/sda/sda3",
                exp_name        => "Major:8:minor:3:kernelnumber:3:bus:0:0:0:0" ,
-               conf            => <<EOF
-BUS="scsi", ID="0:0:0:0", NAME="Major:%M:minor:%m:kernelnumber:%n:bus:%b"
+               rules           => <<EOF
+BUS=="scsi", ID=="0:0:0:0", NAME="Major:%M:minor:%m:kernelnumber:%n:bus:%b"
 EOF
        },
        {
-               desc            => "test NAME substitution chars (with length limit)",
+               desc            => "test substitution chars (with length limit)",
                subsys          => "block",
                devpath         => "/block/sda/sda3",
                exp_name        => "M8-m3-n3-b0:0-sIBM" ,
-               conf            => <<EOF
-BUS="scsi", ID="0:0:0:0", NAME="M%M-m%m-n%n-b%3b-s%3s{vendor}"
+               rules           => <<EOF
+BUS=="scsi", ID=="0:0:0:0", NAME="M%M-m%m-n%n-b%3b-s%3s{vendor}"
 EOF
        },
        {
-               desc            => "old style SYSFS_ attribute",
+               desc            => "import of shell-value file",
                subsys          => "block",
                devpath         => "/block/sda",
-               exp_name        => "good" ,
-               conf            => <<EOF
-BUS="scsi", SYSFS_vendor="IBM-ESXS", NAME="good"
+               exp_name        => "subdir/sys/node" ,
+               rules           => <<EOF
+BUS=="scsi", IMPORT{file}="test.all", NAME="subdir%E{SYSFSDIR}/node"
+KERNEL=="ttyUSB0", NAME="visor"
+EOF
+       },
+       {
+               desc            => "import of shell-value returned from program",
+               subsys          => "block",
+               devpath         => "/block/sda",
+               exp_name        => "node12345678",
+               rules           => <<EOF
+BUS=="scsi", IMPORT="/bin/echo -e \' TEST_KEY=12345678  \\n  TEST_key2=98765 \'", NAME="node\$env{TEST_KEY}"
+KERNEL=="ttyUSB0", NAME="visor"
 EOF
        },
        {
@@ -272,9 +283,9 @@ EOF
                subsys          => "block",
                devpath         => "/block/sda",
                exp_name        => "disk-IBM-ESXS-sda" ,
-               conf            => <<EOF
-BUS="scsi", SYSFS{vendor}="IBM-ESXS", NAME="disk-%s{vendor}-%k"
-KERNEL="ttyUSB0", NAME="visor"
+               rules           => <<EOF
+BUS=="scsi", SYSFS{vendor}=="IBM-ESXS", NAME="disk-%s{vendor}-%k"
+KERNEL=="ttyUSB0", NAME="visor"
 EOF
        },
        {
@@ -282,12 +293,12 @@ EOF
                subsys          => "block",
                devpath         => "/block/sda/sda3",
                exp_name        => "special-device-3" ,
-               conf            => <<EOF
-BUS="scsi", PROGRAM="/bin/echo -n special-device", RESULT="-special-*", NAME="%c-1-%n"
-BUS="scsi", PROGRAM="/bin/echo -n special-device", RESULT="special--*", NAME="%c-2-%n"
-BUS="scsi", PROGRAM="/bin/echo -n special-device", RESULT="special-device-", NAME="%c-3-%n"
-BUS="scsi", PROGRAM="/bin/echo -n special-device", RESULT="special-devic", NAME="%c-4-%n"
-BUS="scsi", PROGRAM="/bin/echo -n special-device", RESULT="special-*", NAME="%c-%n"
+               rules           => <<EOF
+BUS=="scsi", PROGRAM=="/bin/echo -n special-device", RESULT=="-special-*", NAME="%c-1-%n"
+BUS=="scsi", PROGRAM=="/bin/echo -n special-device", RESULT=="special--*", NAME="%c-2-%n"
+BUS=="scsi", PROGRAM=="/bin/echo -n special-device", RESULT=="special-device-", NAME="%c-3-%n"
+BUS=="scsi", PROGRAM=="/bin/echo -n special-device", RESULT=="special-devic", NAME="%c-4-%n"
+BUS=="scsi", PROGRAM=="/bin/echo -n special-device", RESULT=="special-*", NAME="%c-%n"
 EOF
        },
        {
@@ -295,8 +306,8 @@ EOF
                subsys          => "block",
                devpath         => "/block/sda/sda3",
                exp_name        => "subsys_block" ,
-               conf            => <<EOF
-BUS="scsi", PROGRAM="/bin/echo", RESULT="block", NAME="subsys_block"
+               rules           => <<EOF
+BUS=="scsi", PROGRAM=="/bin/echo", RESULT=="block", NAME="subsys_block"
 EOF
        },
        {
@@ -304,8 +315,8 @@ EOF
                subsys          => "block",
                devpath         => "/block/sda/sda3",
                exp_name        => "newline_removed" ,
-               conf            => <<EOF
-BUS="scsi", PROGRAM="/bin/echo test", RESULT="test", NAME="newline_removed"
+               rules           => <<EOF
+BUS=="scsi", PROGRAM=="/bin/echo test", RESULT=="test", NAME="newline_removed"
 EOF
        },
        {
@@ -313,8 +324,8 @@ EOF
                subsys          => "block",
                devpath         => "/block/sda/sda3",
                exp_name        => "test-0:0:0:0" ,
-               conf            => <<EOF
-BUS="scsi", PROGRAM="/bin/echo -n test-%b", RESULT="test-0:0*", NAME="%c"
+               rules           => <<EOF
+BUS=="scsi", PROGRAM=="/bin/echo -n test-%b", RESULT=="test-0:0*", NAME="%c"
 EOF
        },
        {
@@ -322,8 +333,8 @@ EOF
                subsys          => "block",
                devpath         => "/block/sda/sda3",
                exp_name        => "escape-3" ,
-               conf            => <<EOF
-BUS="scsi", PROGRAM="/bin/echo -n escape-%%n", KERNEL="sda3", NAME="%c"
+               rules           => <<EOF
+BUS=="scsi", PROGRAM=="/bin/echo -n escape-%%n", KERNEL=="sda3", NAME="%c"
 EOF
        },
        {
@@ -331,8 +342,8 @@ EOF
                subsys          => "block",
                devpath         => "/block/sda/sda3",
                exp_name        => "foo9" ,
-               conf            => <<EOF
-BUS="scsi", PROGRAM="/bin/echo -n foo3 foo4 foo5 foo6 foo7 foo8 foo9", KERNEL="sda3", NAME="%c{7}"
+               rules           => <<EOF
+BUS=="scsi", PROGRAM=="/bin/echo -n foo3 foo4 foo5 foo6 foo7 foo8 foo9", KERNEL=="sda3", NAME="%c{7}"
 EOF
        },
        {
@@ -340,8 +351,8 @@ EOF
                subsys          => "block",
                devpath         => "/block/sda/sda3",
                exp_name        => "bar9" ,
-               conf            => <<EOF
-BUS="scsi", PROGRAM="/bin/sh -c 'echo foo3 foo4 foo5 foo6 foo7 foo8 foo9 | sed  s/foo9/bar9/'", KERNEL="sda3", NAME="%c{7}"
+               rules           => <<EOF
+BUS=="scsi", PROGRAM=="/bin/sh -c 'echo foo3 foo4 foo5 foo6 foo7 foo8 foo9 | sed  s/foo9/bar9/'", KERNEL=="sda3", NAME="%c{7}"
 EOF
        },
        {
@@ -349,8 +360,8 @@ EOF
                subsys          => "block",
                devpath         => "/block/sda/sda3",
                exp_name        => "foo7" ,
-               conf            => <<EOF
-BUS="scsi", PROGRAM="/bin/echo -n 'foo3 foo4'   'foo5   foo6   foo7 foo8'", KERNEL="sda3", NAME="%c{5}"
+               rules           => <<EOF
+BUS=="scsi", PROGRAM=="/bin/echo -n 'foo3 foo4'   'foo5   foo6   foo7 foo8'", KERNEL=="sda3", NAME="%c{5}"
 EOF
        },
        {
@@ -358,8 +369,8 @@ EOF
                subsys          => "block",
                devpath         => "/block/sda/sda3",
                exp_name        => "my-foo9" ,
-               conf            => <<EOF
-BUS="scsi", PROGRAM="/bin/echo -n foo3 foo4 foo5 foo6 foo7 foo8 foo9", KERNEL="sda3", NAME="my-%c{7}"
+               rules           => <<EOF
+BUS=="scsi", PROGRAM=="/bin/echo -n foo3 foo4 foo5 foo6 foo7 foo8 foo9", KERNEL=="sda3", NAME="my-%c{7}"
 EOF
        },
        {
@@ -367,8 +378,53 @@ EOF
                subsys          => "block",
                devpath         => "/block/sda/sda3",
                exp_name        => "my-foo8" ,
-               conf            => <<EOF
-BUS="scsi", PROGRAM="/bin/echo -n foo3 foo4 foo5 foo6 foo7 foo8 foo9", KERNEL="sda3", NAME="my-%c{6}"
+               rules           => <<EOF
+BUS=="scsi", PROGRAM=="/bin/echo -n foo3 foo4 foo5 foo6 foo7 foo8 foo9", KERNEL=="sda3", NAME="my-%c{6}"
+EOF
+       },
+       {
+               desc            => "test substitution by variable name",
+               subsys          => "block",
+               devpath         => "/block/sda/sda3",
+               exp_name        => "Major:8-minor:3-kernelnumber:3-bus:0:0:0:0" ,
+               rules           => <<EOF
+BUS=="scsi", ID=="0:0:0:0", NAME="Major:\$major-minor:\$minor-kernelnumber:\$number-bus:\$id"
+EOF
+       },
+       {
+               desc            => "test substitution by variable name 2",
+               subsys          => "block",
+               devpath         => "/block/sda/sda3",
+               exp_name        => "Major:8-minor:3-kernelnumber:3-bus:0:0:0:0" ,
+               rules           => <<EOF
+BUS=="scsi", ID=="0:0:0:0", DEVPATH="*/sda/*", NAME="Major:\$major-minor:%m-kernelnumber:\$number-bus:%b"
+EOF
+       },
+       {
+               desc            => "test substitution by variable name 3",
+               subsys          => "block",
+               devpath         => "/block/sda/sda3",
+               exp_name        => "830:0:0:03" ,
+               rules           => <<EOF
+BUS=="scsi", ID=="0:0:0:0", DEVPATH="*/sda/*", NAME="%M%m%b%n"
+EOF
+       },
+       {
+               desc            => "test substitution by variable name 4",
+               subsys          => "block",
+               devpath         => "/block/sda/sda3",
+               exp_name        => "833" ,
+               rules           => <<EOF
+BUS=="scsi", ID=="0:0:0:0", DEVPATH="*/sda/*", NAME="\$major\$minor\$number"
+EOF
+       },
+       {
+               desc            => "test substitution by variable name 5",
+               subsys          => "block",
+               devpath         => "/block/sda/sda3",
+               exp_name        => "8330:0:0:0" ,
+               rules           => <<EOF
+BUS=="scsi", ID=="0:0:0:0", DEVPATH="*/sda/*", NAME="\$major%m%n\$id"
 EOF
        },
        {
@@ -376,9 +432,9 @@ EOF
                subsys          => "tty",
                devpath         => "/class/tty/console",
                exp_name        => "TTY" ,
-               conf            => <<EOF
-BUS="scsi", PROGRAM="/bin/echo -n foo", RESULT="foo", NAME="foo"
-KERNEL="console", NAME="TTY"
+               rules           => <<EOF
+BUS=="scsi", PROGRAM=="/bin/echo -n foo", RESULT=="foo", NAME="foo"
+KERNEL=="console", NAME="TTY"
 EOF
        },
        {
@@ -386,9 +442,9 @@ EOF
                subsys          => "tty",
                devpath         => "/class/tty/console",
                exp_name        => "foo" ,
-               conf            => <<EOF
-PROGRAM="/bin/echo -n foo", RESULT="foo", NAME="foo"
-KERNEL="console", NAME="TTY"
+               rules           => <<EOF
+PROGRAM=="/bin/echo -n foo", RESULT=="foo", NAME="foo"
+KERNEL=="console", NAME="TTY"
 EOF
        },
        {
@@ -396,9 +452,9 @@ EOF
                subsys          => "tty",
                devpath         => "/class/tty/console",
                exp_name        => "TTY" ,
-               conf            => <<EOF
-BUS="foo", SYSFS{dev}="5:1", NAME="foo"
-KERNEL="console", NAME="TTY"
+               rules           => <<EOF
+BUS=="foo", SYSFS{dev}=="5:1", NAME="foo"
+KERNEL=="console", NAME="TTY"
 EOF
        },
        {
@@ -406,9 +462,9 @@ EOF
                subsys          => "tty",
                devpath         => "/class/tty/console",
                exp_name        => "foo" ,
-               conf            => <<EOF
-SYSFS{dev}="5:1", NAME="foo"
-KERNEL="console", NAME="TTY"
+               rules           => <<EOF
+SYSFS{dev}=="5:1", NAME="foo"
+KERNEL=="console", NAME="TTY"
 EOF
        },
        {
@@ -416,10 +472,10 @@ EOF
                subsys          => "block",
                devpath         => "/block/sda",
                exp_name        => "scsi-0:0:0:0" ,
-               conf            => <<EOF
-BUS="usb", PROGRAM="/bin/echo -n usb-%b", NAME="%c"
-BUS="scsi", PROGRAM="/bin/echo -n scsi-%b", NAME="%c"
-BUS="foo", PROGRAM="/bin/echo -n foo-%b", NAME="%c"
+               rules           => <<EOF
+BUS=="usb", PROGRAM=="/bin/echo -n usb-%b", NAME="%c"
+BUS=="scsi", PROGRAM=="/bin/echo -n scsi-%b", NAME="%c"
+BUS=="foo", PROGRAM=="/bin/echo -n foo-%b", NAME="%c"
 EOF
        },
        {
@@ -427,8 +483,9 @@ EOF
                subsys          => "block",
                devpath         => "/block/sda",
                exp_name        => "boot_disk15" ,
-               conf            => <<EOF
-BUS="scsi", SYSFS{vendor}="IBM-ESXS", NAME{all_partitions}="boot_disk"
+               exp_majorminor  => "8:15",
+               rules           => <<EOF
+BUS=="scsi", SYSFS{vendor}=="IBM-ESXS", NAME{all_partitions}="boot_disk"
 EOF
        },
        {
@@ -436,8 +493,8 @@ EOF
                subsys          => "tty",
                devpath         => "/class/tty/ttyUSB0",
                exp_name        => "visor" ,
-               conf            => <<EOF
-SYSFS{idProduct}="2008", NAME="visor"
+               rules           => <<EOF
+SYSFS{idProduct}=="2008", NAME="visor"
 EOF
        },
        {
@@ -445,9 +502,9 @@ EOF
                subsys          => "block",
                devpath         => "/block/rd!c0d0",
                exp_name        => "rd/c0d0" ,
-               conf            => <<EOF
-BUS="scsi", NAME="%k"
-KERNEL="ttyUSB0", NAME="visor"
+               rules           => <<EOF
+BUS=="scsi", NAME="%k"
+KERNEL=="ttyUSB0", NAME="visor"
 EOF
        },
        {
@@ -455,8 +512,8 @@ EOF
                subsys          => "block",
                devpath         => "/block/rd!c0d0",
                exp_name        => "rd/c0d0" ,
-               conf            => <<EOF
-KERNEL="ttyUSB0", NAME="visor"
+               rules           => <<EOF
+KERNEL=="ttyUSB0", NAME="visor"
 EOF
        },
        {
@@ -464,9 +521,9 @@ EOF
                subsys          => "block",
                devpath         => "/block/cciss!c0d0/cciss!c0d0p1",
                exp_name        => "cciss/c0d0p1" ,
-               conf            => <<EOF
-BUS="scsi", NAME="%k"
-KERNEL="ttyUSB0", NAME="visor"
+               rules           => <<EOF
+BUS=="scsi", NAME="%k"
+KERNEL=="ttyUSB0", NAME="visor"
 EOF
        },
        {
@@ -474,12 +531,12 @@ EOF
                subsys          => "block",
                devpath         => "/block/sda",
                exp_name        => "scsi-0:0:0:0",
-               conf            => <<EOF
-BUS="usb", ID="0:0:0:0", NAME="not-scsi"
-BUS="scsi", ID="0:0:0:1", NAME="no-match"
-BUS="scsi", ID=":0", NAME="short-id"
-BUS="scsi", ID="/0:0:0:0", NAME="no-match"
-BUS="scsi", ID="0:0:0:0", NAME="scsi-0:0:0:0"
+               rules           => <<EOF
+BUS=="usb", ID=="0:0:0:0", NAME="not-scsi"
+BUS=="scsi", ID=="0:0:0:1", NAME="no-match"
+BUS=="scsi", ID==":0", NAME="short-id"
+BUS=="scsi", ID=="/0:0:0:0", NAME="no-match"
+BUS=="scsi", ID=="0:0:0:0", NAME="scsi-0:0:0:0"
 EOF
        },
        {
@@ -487,12 +544,12 @@ EOF
                subsys          => "block",
                devpath         => "/block/sda",
                exp_name        => "scsi-0:0:0:0",
-               conf            => <<EOF
-BUS="scsi", ID="*:1", NAME="no-match"
-BUS="scsi", ID="*:0:1", NAME="no-match"
-BUS="scsi", ID="*:0:0:1", NAME="no-match"
-BUS="scsi", ID="*", NAME="scsi-0:0:0:0"
-BUS="scsi", ID="0:0:0:0", NAME="bad"
+               rules           => <<EOF
+BUS=="scsi", ID=="*:1", NAME="no-match"
+BUS=="scsi", ID=="*:0:1", NAME="no-match"
+BUS=="scsi", ID=="*:0:0:1", NAME="no-match"
+BUS=="scsi", ID=="*", NAME="scsi-0:0:0:0"
+BUS=="scsi", ID=="0:0:0:0", NAME="bad"
 EOF
        },
        {
@@ -500,9 +557,9 @@ EOF
                subsys          => "block",
                devpath         => "/block/sda",
                exp_name        => "scsi-0:0:0:0",
-               conf            => <<EOF
-BUS="scsi", ID="*:0", NAME="scsi-0:0:0:0"
-BUS="scsi", ID="0:0:0:0", NAME="bad"
+               rules           => <<EOF
+BUS=="scsi", ID=="*:0", NAME="scsi-0:0:0:0"
+BUS=="scsi", ID=="0:0:0:0", NAME="bad"
 EOF
        },
        {
@@ -510,9 +567,9 @@ EOF
                subsys          => "block",
                devpath         => "/block/sda",
                exp_name        => "scsi-0:0:0:0",
-               conf            => <<EOF
-BUS="scsi", ID="*:0:0:0", NAME="scsi-0:0:0:0"
-BUS="scsi", ID="0:0:0:0", NAME="bad"
+               rules           => <<EOF
+BUS=="scsi", ID=="*:0:0:0", NAME="scsi-0:0:0:0"
+BUS=="scsi", ID=="0:0:0:0", NAME="bad"
 EOF
        },
        {
@@ -520,8 +577,8 @@ EOF
                subsys          => "block",
                devpath         => "/block/sda",
                exp_name        => "ignored",
-               conf            => <<EOF
-BUS="scsi", SYSFS{whitespace_test}="WHITE  SPACE", NAME="ignored"
+               rules           => <<EOF
+BUS=="scsi", SYSFS{whitespace_test}=="WHITE  SPACE", NAME="ignored"
 EOF
        },
        {
@@ -529,9 +586,9 @@ EOF
                subsys          => "block",
                devpath         => "/block/sda",
                exp_name        => "matched-with-space",
-               conf            => <<EOF
-BUS="scsi", SYSFS{whitespace_test}="WHITE  SPACE ", NAME="wrong-to-ignore"
-BUS="scsi", SYSFS{whitespace_test}="WHITE  SPACE   ", NAME="matched-with-space"
+               rules           => <<EOF
+BUS=="scsi", SYSFS{whitespace_test}=="WHITE  SPACE ", NAME="wrong-to-ignore"
+BUS=="scsi", SYSFS{whitespace_test}=="WHITE  SPACE   ", NAME="matched-with-space"
 EOF
        },
        {
@@ -540,8 +597,8 @@ EOF
                devpath         => "/class/tty/tty33",
                exp_name        => "tty33",
                exp_perms       => "0:0:0660",
-               conf            => <<EOF
-KERNEL="tty33", NAME="tty33", OWNER="bad", GROUP="name"
+               rules           => <<EOF
+KERNEL=="tty33", NAME="tty33", OWNER="bad", GROUP="name"
 EOF
        },
        {
@@ -550,8 +607,8 @@ EOF
                devpath         => "/block/sda",
                exp_name        => "node",
                exp_perms       => "5000::0660",
-               conf            => <<EOF
-BUS="scsi", KERNEL="sda", NAME="node", OWNER="5000"
+               rules           => <<EOF
+BUS=="scsi", KERNEL=="sda", NAME="node", OWNER="5000"
 EOF
        },
        {
@@ -560,8 +617,38 @@ EOF
                devpath         => "/block/sda",
                exp_name        => "node",
                exp_perms       => ":100:0660",
-               conf            => <<EOF
-BUS="scsi", KERNEL="sda", NAME="node", GROUP="100"
+               rules           => <<EOF
+BUS=="scsi", KERNEL=="sda", NAME="node", GROUP="100"
+EOF
+       },
+       {
+               desc            => "textual user id",
+               subsys          => "block",
+               devpath         => "/block/sda",
+               exp_name        => "node",
+               exp_perms       => "nobody::0660",
+               rules           => <<EOF
+BUS=="scsi", KERNEL=="sda", NAME="node", OWNER="nobody"
+EOF
+       },
+       {
+               desc            => "textual group id",
+               subsys          => "block",
+               devpath         => "/block/sda",
+               exp_name        => "node",
+               exp_perms       => ":daemon:0660",
+               rules           => <<EOF
+BUS=="scsi", KERNEL=="sda", NAME="node", GROUP="daemon"
+EOF
+       },
+       {
+               desc            => "textual user/group id",
+               subsys          => "block",
+               devpath         => "/block/sda",
+               exp_name        => "node",
+               exp_perms       => "root:mail:0660",
+               rules           => <<EOF
+BUS=="scsi", KERNEL=="sda", NAME="node", OWNER="root", GROUP="mail"
 EOF
        },
        {
@@ -570,8 +657,8 @@ EOF
                devpath         => "/block/sda",
                exp_name        => "node",
                exp_perms       => "::0777",
-               conf            => <<EOF
-BUS="scsi", KERNEL="sda", NAME="node", MODE="0777"
+               rules           => <<EOF
+BUS=="scsi", KERNEL=="sda", NAME="node", MODE="0777"
 EOF
        },
        {
@@ -580,8 +667,8 @@ EOF
                devpath         => "/block/sda",
                exp_name        => "node",
                exp_perms       => "5000:100:0777",
-               conf            => <<EOF
-BUS="scsi", KERNEL="sda", NAME="node", OWNER="5000", GROUP="100", MODE="0777"
+               rules           => <<EOF
+BUS=="scsi", KERNEL=="sda", NAME="node", OWNER="5000", GROUP="100", MODE="0777"
 EOF
        },
        {
@@ -590,8 +677,8 @@ EOF
                devpath         => "/class/tty/ttyUSB0",
                exp_name        => "ttyUSB0",
                exp_perms       => "5000::",
-               conf            => <<EOF
-KERNEL="ttyUSB[0-9]*", NAME="ttyUSB%n", OWNER="5000"
+               rules           => <<EOF
+KERNEL=="ttyUSB[0-9]*", NAME="ttyUSB%n", OWNER="5000"
 EOF
        },
        {
@@ -600,8 +687,8 @@ EOF
                devpath         => "/class/tty/ttyUSB0",
                exp_name        => "ttyUSB0",
                exp_perms       => ":100:0660",
-               conf            => <<EOF
-KERNEL="ttyUSB[0-9]*", NAME="ttyUSB%n", GROUP="100"
+               rules           => <<EOF
+KERNEL=="ttyUSB[0-9]*", NAME="ttyUSB%n", GROUP="100"
 EOF
        },
        {
@@ -610,8 +697,8 @@ EOF
                devpath         => "/class/tty/ttyUSB0",
                exp_name        => "ttyUSB0",
                exp_perms       => "::0060",
-               conf            => <<EOF
-KERNEL="ttyUSB[0-9]*", NAME="ttyUSB%n", MODE="0060"
+               rules           => <<EOF
+KERNEL=="ttyUSB[0-9]*", NAME="ttyUSB%n", MODE="0060"
 EOF
        },
        {
@@ -620,8 +707,8 @@ EOF
                devpath         => "/class/tty/ttyUSB0",
                exp_name        => "ttyUSB0",
                exp_perms       => "5000:100:0777",
-               conf            => <<EOF
-KERNEL="ttyUSB[0-9]*", NAME="ttyUSB%n", OWNER="5000", GROUP="100", MODE="0777"
+               rules           => <<EOF
+KERNEL=="ttyUSB[0-9]*", NAME="ttyUSB%n", OWNER="5000", GROUP="100", MODE="0777"
 EOF
        },
        {
@@ -630,10 +717,10 @@ EOF
                devpath         => "/class/tty/ttyUSB0",
                exp_name        => "ttyUSB0",
                exp_perms       => "5000:100:0777",
-               conf            => <<EOF
-KERNEL="ttyUSB[0-9]*", OWNER="5000", GROUP="100", MODE="0777"
-KERNEL="ttyUSX[0-9]*", OWNER="5001", GROUP="101", MODE="0444"
-KERNEL="ttyUSB[0-9]*", NAME="ttyUSB%n"
+               rules           => <<EOF
+KERNEL=="ttyUSB[0-9]*", OWNER="5000", GROUP="100", MODE="0777"
+KERNEL=="ttyUSX[0-9]*", OWNER="5001", GROUP="101", MODE="0444"
+KERNEL=="ttyUSB[0-9]*", NAME="ttyUSB%n"
 EOF
        },
        {
@@ -642,12 +729,12 @@ EOF
                devpath         => "/class/tty/ttyUSB0",
                exp_name        => "ttyUSB0",
                exp_perms       => "3000:4000:0777",
-               conf            => <<EOF
-SUBSYSTEM="tty", OWNER="3000"
-SUBSYSTEM="tty", GROUP="4000"
-SUBSYSTEM="tty", MODE="0777"
-KERNEL="ttyUSX[0-9]*", OWNER="5001", GROUP="101", MODE="0444"
-KERNEL="ttyUSB[0-9]*", NAME="ttyUSB%n"
+               rules           => <<EOF
+SUBSYSTEM=="tty", OWNER="3000"
+SUBSYSTEM=="tty", GROUP="4000"
+SUBSYSTEM=="tty", MODE="0777"
+KERNEL=="ttyUSX[0-9]*", OWNER="5001", GROUP="101", MODE="0444"
+KERNEL=="ttyUSB[0-9]*", NAME="ttyUSB%n"
 EOF
        },
        {
@@ -656,12 +743,12 @@ EOF
                devpath         => "/class/tty/ttyUSB0",
                exp_name        => "ttyUSB0",
                exp_perms       => "3000:8000:0777",
-               conf            => <<EOF
-SUBSYSTEM="tty", OWNER="3000"
-SUBSYSTEM="tty", GROUP="4000"
-SUBSYSTEM="tty", MODE="0777"
-KERNEL="ttyUSX[0-9]*", OWNER="5001", GROUP="101", MODE="0444"
-KERNEL="ttyUSB[0-9]*", NAME="ttyUSB%n", GROUP="8000"
+               rules           => <<EOF
+SUBSYSTEM=="tty", OWNER="3000"
+SUBSYSTEM=="tty", GROUP="4000"
+SUBSYSTEM=="tty", MODE="0777"
+KERNEL=="ttyUSX[0-9]*", OWNER="5001", GROUP="101", MODE="0444"
+KERNEL=="ttyUSB[0-9]*", NAME="ttyUSB%n", GROUP="8000"
 EOF
        },
        {
@@ -670,8 +757,8 @@ EOF
                devpath         => "/block/sda",
                exp_name        => "node",
                exp_majorminor  => "8:0",
-               conf            => <<EOF
-BUS="scsi", KERNEL="sda", NAME="node"
+               rules           => <<EOF
+BUS=="scsi", KERNEL=="sda", NAME="node"
 EOF
        },
        {
@@ -680,8 +767,8 @@ EOF
                devpath         => "/class/i2c-dev/i2c-300",
                exp_name        => "node",
                exp_majorminor  => "89:300",
-               conf            => <<EOF
-KERNEL="i2c-300", NAME="node"
+               rules           => <<EOF
+KERNEL=="i2c-300", NAME="node"
 EOF
        },
        {
@@ -690,8 +777,8 @@ EOF
                devpath         => "/class/i2c-dev/i2c-fake1",
                exp_name        => "node",
                exp_majorminor  => "4095:1",
-               conf            => <<EOF
-KERNEL="i2c-fake1", NAME="node"
+               rules           => <<EOF
+KERNEL=="i2c-fake1", NAME="node"
 EOF
        },
        {
@@ -700,8 +787,8 @@ EOF
                devpath         => "/class/i2c-dev/i2c-fake2",
                exp_name        => "node",
                exp_majorminor  => "4094:89999",
-               conf            => <<EOF
-KERNEL="i2c-fake2", NAME="node"
+               rules           => <<EOF
+KERNEL=="i2c-fake2", NAME="node"
 EOF
        },
        {
@@ -710,8 +797,19 @@ EOF
                devpath         => "/class/tty/ttyUSB0",
                exp_name        => "symlink2-ttyUSB0",
                exp_target      => "ttyUSB0",
-               conf            => <<EOF
-KERNEL="ttyUSB[0-9]*", NAME="ttyUSB%n", SYMLINK="symlink1-%n symlink2-%k symlink3-%b"
+               rules           => <<EOF
+KERNEL=="ttyUSB[0-9]*", NAME="ttyUSB%n", SYMLINK="symlink1-%n symlink2-%k symlink3-%b"
+EOF
+       },
+       {
+               desc            => "multiple symlinks with a lot of s p a c e s",
+               subsys          => "tty",
+               devpath         => "/class/tty/ttyUSB0",
+               exp_name        => "one",
+               not_exp_name    => " ",
+               exp_target      => "ttyUSB0",
+               rules           => <<EOF
+KERNEL=="ttyUSB[0-9]*", NAME="ttyUSB%n", SYMLINK="  one     two        "
 EOF
        },
        {
@@ -720,8 +818,8 @@ EOF
                devpath         => "/class/tty/ttyUSB0",
                exp_name        => "visor0",
                exp_target      => "ttyUSB0",
-               conf            => <<EOF
-KERNEL="ttyUSB[0-9]*", NAME="ttyUSB%n", SYMLINK="visor%n"
+               rules           => <<EOF
+KERNEL=="ttyUSB[0-9]*", NAME="ttyUSB%n", SYMLINK="visor%n"
 EOF
        },
        {
@@ -730,8 +828,8 @@ EOF
                devpath         => "/block/sda/sda2",
                exp_name        => "1/2/symlink" ,
                exp_target      => "a/b/node",
-               conf            => <<EOF
-BUS="scsi", SYSFS{vendor}="IBM-ESXS", NAME="1/2/a/b/node", SYMLINK="1/2/symlink"
+               rules           => <<EOF
+BUS=="scsi", SYSFS{vendor}=="IBM-ESXS", NAME="1/2/a/b/node", SYMLINK="1/2/symlink"
 EOF
        },
        {
@@ -740,8 +838,8 @@ EOF
                devpath         => "/block/sda/sda2",
                exp_name        => "1/2/c/d/symlink" ,
                exp_target      => "../../a/b/node",
-               conf            => <<EOF
-BUS="scsi", SYSFS{vendor}="IBM-ESXS", NAME="1/2/a/b/node", SYMLINK="1/2/c/d/symlink"
+               rules           => <<EOF
+BUS=="scsi", SYSFS{vendor}=="IBM-ESXS", NAME="1/2/a/b/node", SYMLINK="1/2/c/d/symlink"
 EOF
        },
        {
@@ -750,8 +848,8 @@ EOF
                devpath         => "/class/tty/ttyUSB0",
                exp_name        => "second-0" ,
                exp_target      => "visor" ,
-               conf            => <<EOF
-KERNEL="ttyUSB0", NAME="visor", SYMLINK="first-%n second-%n third-%n"
+               rules           => <<EOF
+KERNEL=="ttyUSB0", NAME="visor", SYMLINK="first-%n second-%n third-%n"
 EOF
        },
        {
@@ -760,10 +858,10 @@ EOF
                devpath         => "/block/sda",
                exp_name        => "symlink-only2",
                exp_target      => "link",
-               conf            => <<EOF
-BUS="scsi", KERNEL="sda", SYMLINK="symlink-only1"
-BUS="scsi", KERNEL="sda", SYMLINK="symlink-only2"
-BUS="scsi", KERNEL="sda", NAME="link", SYMLINK="symlink0"
+               rules           => <<EOF
+BUS=="scsi", KERNEL=="sda", SYMLINK+="symlink-only1"
+BUS=="scsi", KERNEL=="sda", SYMLINK+="symlink-only2"
+BUS=="scsi", KERNEL=="sda", NAME="link", SYMLINK+="symlink0"
 EOF
        },
        {
@@ -774,8 +872,8 @@ EOF
                exp_target      => "link",
                exp_add_error   => "yes",
                exp_rem_error   => "yes",
-               conf            => <<EOF
-BUS="scsi", KERNEL="sda", NAME="link", SYMLINK="."
+               rules           => <<EOF
+BUS=="scsi", KERNEL=="sda", NAME="link", SYMLINK+="."
 EOF
        },
        {
@@ -785,9 +883,9 @@ EOF
                exp_name        => "link",
                exp_target      => "link",
                exp_rem_error   => "yes",
-               option          => "clear",
-               conf            => <<EOF
-KERNEL="tty0", NAME="link", SYMLINK="link"
+               option          => "clean",
+               rules           => <<EOF
+KERNEL=="tty0", NAME="link", SYMLINK+="link"
 EOF
        },
        {
@@ -796,8 +894,8 @@ EOF
                devpath         => "/class/tty/ttyUSB0",
                exp_name        => "symlink0",
                exp_target      => "ttyUSB0",
-               conf            => <<EOF
-KERNEL="ttyUSB[0-9]*", NAME="ttyUSB%n", SYMLINK="symlink%n"
+               rules           => <<EOF
+KERNEL=="ttyUSB[0-9]*", NAME="ttyUSB%n", SYMLINK+="symlink%n"
 EOF
        },
        {
@@ -806,8 +904,8 @@ EOF
                devpath         => "/class/tty/ttyUSB0",
                exp_name        => "symlink-ttyUSB0",
                exp_target      => "ttyUSB0",
-               conf            => <<EOF
-KERNEL="ttyUSB[0-9]*", NAME="ttyUSB%n", SYMLINK="symlink-%k"
+               rules           => <<EOF
+KERNEL=="ttyUSB[0-9]*", NAME="ttyUSB%n", SYMLINK+="symlink-%k"
 EOF
        },
        {
@@ -816,8 +914,8 @@ EOF
                devpath         => "/class/tty/ttyUSB0",
                exp_name        => "major-188:0",
                exp_target      => "ttyUSB0",
-               conf            => <<EOF
-KERNEL="ttyUSB[0-9]*", NAME="ttyUSB%n", SYMLINK="major-%M:%m"
+               rules           => <<EOF
+KERNEL=="ttyUSB[0-9]*", NAME="ttyUSB%n", SYMLINK+="major-%M:%m"
 EOF
        },
        {
@@ -826,8 +924,8 @@ EOF
                devpath         => "/block/sda",
                exp_name        => "symlink-0:0:0:0",
                exp_target      => "node",
-               conf            => <<EOF
-BUS="scsi", KERNEL="sda", NAME="node", SYMLINK="symlink-%b"
+               rules           => <<EOF
+BUS=="scsi", KERNEL=="sda", NAME="node", SYMLINK+="symlink-%b"
 EOF
        },
        {
@@ -836,8 +934,8 @@ EOF
                devpath         => "/class/tty/ttyUSB0",
                exp_name        => "test",
                exp_target      => "ttyUSB0",
-               conf            => <<EOF
-KERNEL="ttyUSB[0-9]*", PROGRAM="/bin/echo test" NAME="ttyUSB%n", SYMLINK="%c"
+               rules           => <<EOF
+KERNEL=="ttyUSB[0-9]*", PROGRAM=="/bin/echo test" NAME="ttyUSB%n", SYMLINK+="%c"
 EOF
        },
        {
@@ -846,8 +944,8 @@ EOF
                devpath         => "/class/tty/ttyUSB0",
                exp_name        => "test",
                exp_target      => "ttyUSB0",
-               conf            => <<EOF
-KERNEL="ttyUSB[0-9]*", PROGRAM="/bin/echo symlink test this" NAME="ttyUSB%n", SYMLINK="%c{2}"
+               rules           => <<EOF
+KERNEL=="ttyUSB[0-9]*", PROGRAM=="/bin/echo symlink test this" NAME="ttyUSB%n", SYMLINK+="%c{2}"
 EOF
        },
        {
@@ -856,8 +954,8 @@ EOF
                devpath         => "/class/tty/ttyUSB0",
                exp_name        => "this",
                exp_target      => "ttyUSB0",
-               conf            => <<EOF
-KERNEL="ttyUSB[0-9]*", PROGRAM="/bin/echo symlink test this" NAME="ttyUSB%n", SYMLINK="%c{2+}"
+               rules           => <<EOF
+KERNEL=="ttyUSB[0-9]*", PROGRAM=="/bin/echo symlink test this" NAME="ttyUSB%n", SYMLINK+="%c{2+}"
 EOF
        },
        {
@@ -866,9 +964,9 @@ EOF
                devpath         => "/block/sda",
                exp_name        => "test",
                exp_target      => "link",
-               conf            => <<EOF
-BUS="scsi", KERNEL="sda", PROGRAM="/bin/echo link test this" SYMLINK="%c{2+}"
-BUS="scsi", KERNEL="sda", NAME="link", SYMLINK="symlink0"
+               rules           => <<EOF
+BUS=="scsi", KERNEL=="sda", PROGRAM=="/bin/echo link test this" SYMLINK+="%c{2+}"
+BUS=="scsi", KERNEL=="sda", NAME="link", SYMLINK+="symlink0"
 EOF
        },
        {
@@ -877,8 +975,8 @@ EOF
                devpath         => "/class/tty/ttyUSB0",
                exp_name        => "188:0",
                exp_target      => "ttyUSB0",
-               conf            => <<EOF
-KERNEL="ttyUSB[0-9]*", NAME="ttyUSB%n", SYMLINK="%s{dev}"
+               rules           => <<EOF
+KERNEL=="ttyUSB[0-9]*", NAME="ttyUSB%n", SYMLINK+="%s{dev}"
 EOF
        },
        {
@@ -887,8 +985,8 @@ EOF
                devpath         => "/class/tty/ttyUSB0",
                exp_name        => "188",
                exp_target      => "ttyUSB0",
-               conf            => <<EOF
-KERNEL="ttyUSB[0-9]*", NAME="ttyUSB%n", SYMLINK="%3s{dev}"
+               rules           => <<EOF
+KERNEL=="ttyUSB[0-9]*", NAME="ttyUSB%n", SYMLINK+="%3s{dev}"
 EOF
        },
        {
@@ -897,8 +995,8 @@ EOF
                devpath         => "/class/tty/ttyUSB0",
                exp_name        => "percent%sign",
                exp_target      => "ttyUSB0",
-               conf            => <<EOF
-KERNEL="ttyUSB[0-9]*", NAME="ttyUSB%n", SYMLINK="percent%%sign"
+               rules           => <<EOF
+KERNEL=="ttyUSB[0-9]*", NAME="ttyUSB%n", SYMLINK+="percent%%sign"
 EOF
        },
        {
@@ -907,8 +1005,8 @@ EOF
                devpath         => "/class/tty/ttyUSB0",
                exp_name        => "%ttyUSB0_name",
                exp_target      => "ttyUSB0",
-               conf            => <<EOF
-KERNEL="ttyUSB[0-9]*", NAME="ttyUSB%n", SYMLINK="%%%k_name"
+               rules           => <<EOF
+KERNEL=="ttyUSB[0-9]*", NAME="ttyUSB%n", SYMLINK+="%%%k_name"
 EOF
        },
        {
@@ -917,8 +1015,8 @@ EOF
                devpath         => "/block/sda/sda3",
                exp_name        => "link1",
                exp_target      => "node",
-               conf            => <<EOF
-BUS="scsi", PROGRAM="/bin/echo -n node link1 link2", RESULT="node *", NAME="%c{1}", SYMLINK="%c{2} %c{3}"
+               rules           => <<EOF
+BUS=="scsi", PROGRAM=="/bin/echo -n node link1 link2", RESULT=="node *", NAME="%c{1}", SYMLINK+="%c{2} %c{3}"
 EOF
        },
        {
@@ -927,8 +1025,8 @@ EOF
                devpath         => "/block/sda/sda3",
                exp_name        => "link4",
                exp_target      => "node",
-               conf            => <<EOF
-BUS="scsi", PROGRAM="/bin/echo -n node link1 link2 link3 link4", RESULT="node *", NAME="%c{1}", SYMLINK="%c{2+}"
+               rules           => <<EOF
+BUS=="scsi", PROGRAM=="/bin/echo -n node link1 link2 link3 link4", RESULT=="node *", NAME="%c{1}", SYMLINK+="%c{2+}"
 EOF
        },
        {
@@ -936,8 +1034,8 @@ EOF
                subsys          => "block",
                devpath         => "/block/sda",
                exp_name        => "cdrom",
-               conf            => <<EOF
-KERNEL="sda", NAME="cdrom%e"
+               rules           => <<EOF
+KERNEL=="sda", NAME="cdrom%e"
 EOF
        },
        {
@@ -946,8 +1044,8 @@ EOF
                devpath         => "/block/sda",
                exp_name        => "cdrom",
                option          => "keep",
-               conf            => <<EOF
-KERNEL="sda", NAME="cdrom%e"
+               rules           => <<EOF
+KERNEL=="sda", NAME="cdrom%e"
 EOF
        },
        {
@@ -956,8 +1054,8 @@ EOF
                devpath         => "/block/sda/sda1",
                exp_name        => "enum",
                option          => "keep",
-               conf            => <<EOF
-KERNEL="sda1", NAME="enum%e"
+               rules           => <<EOF
+KERNEL=="sda1", NAME="enum%e"
 EOF
        },
        {
@@ -966,8 +1064,8 @@ EOF
                devpath         => "/block/sda/sda2",
                exp_name        => "cdrom1",
                option          => "keep",
-               conf            => <<EOF
-KERNEL="sda2", NAME="cdrom%e"
+               rules           => <<EOF
+KERNEL=="sda2", NAME="cdrom%e"
 EOF
        },
        {
@@ -976,8 +1074,8 @@ EOF
                devpath         => "/block/sda/sda3",
                exp_name        => "enum1",
                option          => "keep",
-               conf            => <<EOF
-KERNEL="sda3", NAME="enum%e"
+               rules           => <<EOF
+KERNEL=="sda3", NAME="enum%e"
 EOF
        },
        {
@@ -985,9 +1083,9 @@ EOF
                subsys          => "block",
                devpath         => "/block/sda/sda4",
                exp_name        => "cdrom2",
-               option          => "clear",
-               conf            => <<EOF
-KERNEL="sda4", NAME="cdrom%e"
+               option          => "clean",
+               rules           => <<EOF
+KERNEL=="sda4", NAME="cdrom%e"
 EOF
        },
        {
@@ -995,18 +1093,19 @@ EOF
                subsys          => "block",
                devpath         => "/block/sda",
                exp_name        => "cdrom",
-               conf            => <<EOF
-KERNEL="sda", NAME="cdrom%e"
+               rules           => <<EOF
+KERNEL=="sda", NAME="cdrom%e"
 EOF
        },
        {
                desc            => "ignore rule test",
                subsys          => "block",
                devpath         => "/block/sda",
-               exp_name        => "node",
+               exp_name        => "nothing",
+               not_exp_name    => "node",
                exp_add_error   => "yes",
-               conf            => <<EOF
-BUS="scsi", KERNEL="sda", NAME="node", OPTIONS="ignore"
+               rules           => <<EOF
+BUS=="scsi", KERNEL=="sda", NAME="node", OPTIONS="ignore"
 EOF
        },
        {
@@ -1014,9 +1113,9 @@ EOF
                subsys          => "block",
                devpath         => "/block/sda",
                exp_name        => "node6",
-               conf            => <<EOF
-SUBSYSTEM="block", OPTIONS="all_partitions"
-BUS="scsi", KERNEL="sda", NAME="node"
+               rules           => <<EOF
+SUBSYSTEM=="block", OPTIONS="all_partitions"
+BUS=="scsi", KERNEL=="sda", NAME="node"
 EOF
        },
        {
@@ -1025,9 +1124,9 @@ EOF
                devpath         => "/block/sda/sda1",
                exp_name        => "node6",
                exp_add_error   => "yes",
-               conf            => <<EOF
-SUBSYSTEM="block", OPTIONS="all_partitions"
-BUS="scsi", KERNEL="sda", NAME="node"
+               rules           => <<EOF
+SUBSYSTEM=="block", OPTIONS="all_partitions"
+BUS=="scsi", KERNEL=="sda", NAME="node"
 EOF
        },
        {
@@ -1036,8 +1135,8 @@ EOF
                devpath         => "/block/sda",
                exp_name        => "node",
                exp_rem_error   => "yes",
-               conf            => <<EOF
-BUS="scsi", KERNEL="sda", NAME="node", OPTIONS="ignore_remove"
+               rules           => <<EOF
+BUS=="scsi", KERNEL=="sda", NAME="node", OPTIONS="ignore_remove"
 EOF
        },
        {
@@ -1046,9 +1145,9 @@ EOF
                devpath         => "/block/sda",
                exp_name        => "node14",
                exp_rem_error   => "yes",
-               option          => "clear",
-               conf            => <<EOF
-BUS="scsi", KERNEL="sda", NAME="node", OPTIONS="ignore_remove, all_partitions"
+               option          => "clean",
+               rules           => <<EOF
+BUS=="scsi", KERNEL=="sda", NAME="node", OPTIONS="ignore_remove, all_partitions"
 EOF
        },
        {
@@ -1056,10 +1155,10 @@ EOF
                subsys          => "block",
                devpath         => "/block/sda",
                exp_name        => "node",
-               conf            => <<EOF
-BUS="scsi", KERNEL="sda", NAME="should_not_match", SUBSYSTEM="vc"
-BUS="scsi", KERNEL="sda", NAME="node", SUBSYSTEM="block"
-BUS="scsi", KERNEL="sda", NAME="should_not_match2", SUBSYSTEM="vc"
+               rules           => <<EOF
+BUS=="scsi", KERNEL=="sda", NAME="should_not_match", SUBSYSTEM=="vc"
+BUS=="scsi", KERNEL=="sda", NAME="node", SUBSYSTEM=="block"
+BUS=="scsi", KERNEL=="sda", NAME="should_not_match2", SUBSYSTEM=="vc"
 EOF
        },
        {
@@ -1067,9 +1166,9 @@ EOF
                subsys          => "block",
                devpath         => "/block/sda",
                exp_name        => "node",
-               conf            => <<EOF
-BUS="scsi", KERNEL="sda", NAME="should_not_match", DRIVER="sd-wrong"
-BUS="scsi", KERNEL="sda", NAME="node", DRIVER="sd"
+               rules           => <<EOF
+BUS=="scsi", KERNEL=="sda", NAME="should_not_match", DRIVER=="sd-wrong"
+BUS=="scsi", KERNEL=="sda", NAME="node", DRIVER=="sd"
 EOF
        },
        {
@@ -1077,8 +1176,8 @@ EOF
                subsys          => "block",
                devpath         => "/block/sda",
                exp_name        => "node",
-               conf            => <<EOF
-BUS="scsi", KERNEL="sda", PROGRAM="/usr/bin/test -b %N" NAME="node"
+               rules           => <<EOF
+BUS=="scsi", KERNEL=="sda", PROGRAM=="/usr/bin/test -b %N" NAME="node"
 EOF
        },
        {
@@ -1086,8 +1185,8 @@ EOF
                subsys          => "block",
                devpath         => "/block/sda",
                exp_name        => "sda",
-               conf            => <<EOF
-BUS="scsi", KERNEL="sda", PROGRAM="/bin/echo %p", RESULT="/block/sda" NAME="%k"
+               rules           => <<EOF
+BUS=="scsi", KERNEL=="sda", PROGRAM=="/bin/echo %p", RESULT=="/block/sda" NAME="%k"
 EOF
        },
        {
@@ -1096,8 +1195,8 @@ EOF
                devpath         => "/block/sda",
                exp_name        => "main_device",
                option          => "keep",
-               conf            => <<EOF
-BUS="scsi", KERNEL="sda", NAME="main_device"
+               rules           => <<EOF
+BUS=="scsi", KERNEL=="sda", NAME="main_device"
 EOF
        },
        {
@@ -1106,8 +1205,8 @@ EOF
                devpath         => "/block/sda/sda1",
                exp_name        => "main_device-part-1",
                option          => "clean",
-               conf            => <<EOF
-BUS="scsi", KERNEL="sda1", NAME="%P-part-1"
+               rules           => <<EOF
+BUS=="scsi", KERNEL=="sda1", NAME="%P-part-1"
 EOF
        },
        {
@@ -1115,8 +1214,8 @@ EOF
                subsys          => "block",
                devpath         => "/block/sda/sda1",
                exp_name        => "start-udev-root-end",
-               conf            => <<EOF
-BUS="scsi", KERNEL="sda1", NAME="start-%r-end"
+               rules           => <<EOF
+BUS=="scsi", KERNEL=="sda1", NAME="start-%r-end"
 EOF
        },
        {
@@ -1124,29 +1223,351 @@ EOF
                subsys          => "block",
                devpath         => "/block/sda/sda1",
                exp_name        => "last",
-               conf            => <<EOF
-BUS="scsi", KERNEL="sda1", SYMLINK="last", OPTIONS="last_rule"
-BUS="scsi", KERNEL="sda1", NAME="very-last"
+               rules           => <<EOF
+BUS=="scsi", KERNEL=="sda1", SYMLINK+="last", OPTIONS="last_rule"
+BUS=="scsi", KERNEL=="sda1", NAME="very-last"
+EOF
+       },
+       {
+               desc            => "negation KERNEL!=",
+               subsys          => "block",
+               devpath         => "/block/sda/sda1",
+               exp_name        => "match",
+               rules           => <<EOF
+BUS=="scsi", KERNEL!="sda1", NAME="matches-but-is-negated"
+BUS=="scsi", KERNEL!="xsda1", NAME="match"
+BUS=="scsi", KERNEL=="sda1", NAME="wrong"
+EOF
+       },
+       {
+               desc            => "negation SUBSYSTEM!=",
+               subsys          => "block",
+               devpath         => "/block/sda/sda1",
+               exp_name        => "not-anything",
+               rules           => <<EOF
+BUS=="scsi", SUBSYSTEM=="block", KERNEL!="sda1", NAME="matches-but-is-negated"
+BUS=="scsi", SUBSYSTEM!="anything", NAME="not-anything"
+BUS=="scsi", KERNEL=="sda1", NAME="wrong"
+EOF
+       },
+       {
+               desc            => "negation PROGRAM!= exit code",
+               subsys          => "block",
+               devpath         => "/block/sda/sda1",
+               exp_name        => "nonzero-program",
+               rules           => <<EOF
+KERNEL=="sda1", PROGRAM!="/bin/false", NAME="nonzero-program"
+BUS=="scsi", KERNEL=="sda1", NAME="wrong"
+EOF
+       },
+       {
+               desc            => "test for whitespace between the operator",
+               subsys          => "block",
+               devpath         => "/block/sda/sda1",
+               exp_name        => "true",
+               rules           => <<EOF
+KERNEL   ==   "sda1"     ,    NAME   =    "true"
+BUS=="scsi", KERNEL=="sda1", NAME="wrong"
+EOF
+       },
+       {
+               desc            => "ENV{} test",
+               subsys          => "block",
+               devpath         => "/block/sda/sda1",
+               exp_name        => "true",
+               rules           => <<EOF
+BUS=="scsi", KERNEL=="sda1", ENV{ENV_KEY_TEST}=="go", NAME="wrong"
+BUS=="scsi", KERNEL=="sda1", ENV{ENV_KEY_TEST}=="test", NAME="true"
+BUS=="scsi", KERNEL=="sda1", ENV{ENV_KEY_TEST}=="bad", NAME="bad"
+EOF
+       },
+       {
+               desc            => "ENV{} test",
+               subsys          => "block",
+               devpath         => "/block/sda/sda1",
+               exp_name        => "true",
+               rules           => <<EOF
+BUS=="scsi", KERNEL=="sda1", ENV{ENV_KEY_TEST}=="go", NAME="wrong"
+BUS=="scsi", KERNEL=="sda1", ENV{ENV_KEY_TEST}=="yes", ENV{ACTION}=="add", ENV{DEVPATH}=="/block/sda/sdax1", NAME="no"
+BUS=="scsi", KERNEL=="sda1", ENV{ENV_KEY_TEST}=="test", ENV{ACTION}=="add", ENV{DEVPATH}=="/block/sda/sda1", NAME="true"
+BUS=="scsi", KERNEL=="sda1", ENV{ENV_KEY_TEST}=="bad", NAME="bad"
+EOF
+       },
+       {
+               desc            => "ENV{} test (assign)",
+               subsys          => "block",
+               devpath         => "/block/sda/sda1",
+               exp_name        => "true",
+               rules           => <<EOF
+BUS=="scsi", KERNEL=="sda1", ENV{ASSIGN}="true"
+BUS=="scsi", KERNEL=="sda1", ENV{ASSIGN}=="yes", NAME="no"
+BUS=="scsi", KERNEL=="sda1", ENV{ASSIGN}=="true", NAME="true"
+BUS=="scsi", KERNEL=="sda1", NAME="bad"
+EOF
+       },
+       {
+               desc            => "ENV{} test (assign2)",
+               subsys          => "block",
+               devpath         => "/block/sda/sda1",
+               exp_name        => "part",
+               rules           => <<EOF
+SUBSYSTEM=="block", KERNEL="*[0-9]", ENV{PARTITION}="true", ENV{MAINDEVICE}="false"
+SUBSYSTEM=="block", KERNEL="*[!0-9]", ENV{PARTITION}="false", ENV{MAINDEVICE}="true"
+ENV{MAINDEVICE}=="true", NAME="disk"
+ENV{PARTITION}=="true", NAME="part"
+NAME="bad"
+EOF
+       },
+       {
+               desc            => "untrusted string sanitize",
+               subsys          => "block",
+               devpath         => "/block/sda/sda1",
+               exp_name        => "sane",
+               rules           => <<EOF
+BUS=="scsi", KERNEL=="sda1", PROGRAM=="/bin/echo -e name; (/sbin/badprogram)", RESULT=="name_ _/sbin/badprogram_", NAME="sane"
+EOF
+       },
+       {
+               desc            => "untrusted string sanitize (don't replace utf8)",
+               subsys          => "block",
+               devpath         => "/block/sda/sda1",
+               exp_name        => "uber",
+               rules           => <<EOF
+BUS=="scsi", KERNEL=="sda1", PROGRAM=="/bin/echo -e \\xc3\\xbcber" RESULT=="\xc3\xbcber", NAME="uber"
+EOF
+       },
+       {
+               desc            => "untrusted string sanitize (replace invalid utf8)",
+               subsys          => "block",
+               devpath         => "/block/sda/sda1",
+               exp_name        => "replaced",
+               rules           => <<EOF
+BUS=="scsi", KERNEL=="sda1", PROGRAM=="/bin/echo -e \\xef\\xe8garbage", RESULT=="__garbage", NAME="replaced"
+EOF
+       },
+       {
+               desc            => "read sysfs value from device down in the chain",
+               subsys          => "block",
+               devpath         => "/class/tty/ttyUSB0",
+               exp_name        => "serial-0000:00:09.0",
+               rules           => <<EOF
+KERNEL=="ttyUSB*", NAME="serial-%s{serial}"
+EOF
+       },
+       {
+               desc            => "match against empty key string",
+               subsys          => "block",
+               devpath         => "/block/sda",
+               exp_name        => "ok",
+               rules           => <<EOF
+KERNEL=="sda", SYSFS{nothing}!="", NAME="not-1-ok"
+KERNEL=="sda", SYSFS{nothing}=="", NAME="not-2-ok"
+KERNEL=="sda", SYSFS{vendor}!="", NAME="ok"
+KERNEL=="sda", SYSFS{vendor}=="", NAME="not-3-ok"
+EOF
+       },
+       {
+               desc            => "check ACTION value",
+               subsys          => "block",
+               devpath         => "/block/sda",
+               exp_name        => "ok",
+               rules           => <<EOF
+ACTION=="unknown", KERNEL=="sda", NAME="unknown-not-ok"
+ACTION=="add", KERNEL=="sda", NAME="ok"
+EOF
+       },
+       {
+               desc            => "apply NAME only once",
+               subsys          => "block",
+               devpath         => "/block/sda",
+               exp_name        => "link",
+               exp_target      => "ok",
+               rules           => <<EOF
+KERNEL=="sda", NAME="ok"
+KERNEL=="sda", NAME="not-ok"
+KERNEL=="sda", SYMLINK+="link"
+EOF
+       },
+       {
+               desc            => "test RUN key",
+               subsys          => "block",
+               devpath         => "/block/sda",
+               exp_name        => "testsymlink",
+               exp_target      => "ok",
+               exp_rem_error   => "yes",
+               option          => "clean",
+               rules           => <<EOF
+KERNEL=="sda", NAME="ok", RUN+="/bin/ln -s ok %r/testsymlink"
+KERNEL=="sda", NAME="not-ok"
+EOF
+       },
+       {
+               desc            => "test RUN key and DEVNAME",
+               subsys          => "block",
+               devpath         => "/block/sda",
+               exp_name        => "testsymlink",
+               exp_target      => "ok",
+               exp_rem_error   => "yes",
+               option          => "clean",
+               rules           => <<EOF
+KERNEL=="sda", NAME="ok", RUN+="/bin/sh -c 'ln -s `basename \$\$DEVNAME` %r/testsymlink'"
+KERNEL=="sda", NAME="not-ok"
+EOF
+       },
+       {
+               desc            => "test RUN key remove",
+               subsys          => "block",
+               devpath         => "/block/sda",
+               exp_name        => "testsymlink2",
+               exp_target      => "ok2",
+               rules           => <<EOF
+KERNEL=="sda", NAME="ok2", RUN+="/bin/ln -s ok2 %r/testsymlink2"
+KERNEL=="sda", ACTION=="remove", RUN+="/bin/rm -f %r/testsymlink2"
+KERNEL=="sda", NAME="not-ok2"
+EOF
+       },
+       {
+               desc            => "final assignment",
+               subsys          => "block",
+               devpath         => "/block/sda",
+               exp_name        => "ok",
+               exp_perms       => "root:nobody:0640",
+               rules           => <<EOF
+KERNEL=="sda", GROUP:="nobody"
+KERNEL=="sda", GROUP="not-ok", MODE="0640", NAME="ok"
+EOF
+       },
+       {
+               desc            => "final assignment",
+               subsys          => "block",
+               devpath         => "/block/sda",
+               exp_name        => "ok",
+               exp_perms       => "root:nobody:0640",
+               rules           => <<EOF
+KERNEL=="sda", GROUP:="nobody"
+SUBSYSTEM=="block", MODE:="640"
+KERNEL=="sda", GROUP="not-ok", MODE="0666", NAME="ok"
+EOF
+       },
+       {
+               desc            => "env substitution",
+               subsys          => "block",
+               devpath         => "/block/sda",
+               exp_name        => "node-add-me",
+               rules           => <<EOF
+KERNEL=="sda", MODE="0666", NAME="node-\$env{ACTION}-me"
+EOF
+       },
+       {
+               desc            => "reset list to current value",
+               subsys          => "tty",
+               devpath         => "/class/tty/ttyUSB0",
+               exp_name        => "three",
+               not_exp_name    => "two",
+               exp_target      => "node",
+               rules           => <<EOF
+KERNEL=="ttyUSB[0-9]*", SYMLINK+="one"
+KERNEL=="ttyUSB[0-9]*", SYMLINK+="two"
+KERNEL=="ttyUSB[0-9]*", SYMLINK="three"
+KERNEL=="ttyUSB[0-9]*", NAME="node"
+EOF
+       },
+       {
+               desc            => "test empty NAME",
+               subsys          => "tty",
+               devpath         => "/class/tty/ttyUSB0",
+               exp_name        => "node",
+               not_exp_name    => "wrong",
+               exp_add_error   => "yes",
+               rules           => <<EOF
+KERNEL=="ttyUSB[0-9]*", NAME=""
+KERNEL=="ttyUSB[0-9]*", NAME="wrong"
+EOF
+       },
+       {
+               desc            => "test empty NAME 2",
+               subsys          => "tty",
+               devpath         => "/class/tty/ttyUSB0",
+               exp_name        => "right",
+               rules           => <<EOF
+KERNEL=="ttyUSB[0-9]*", NAME="right"
+KERNEL=="ttyUSB[0-9]*", NAME=""
+KERNEL=="ttyUSB[0-9]*", NAME="wrong"
+EOF
+       },
+       {
+               desc            => "test multi matches",
+               subsys          => "tty",
+               devpath         => "/class/tty/ttyUSB0",
+               exp_name        => "right",
+               rules           => <<EOF
+KERNEL=="ttyUSB*|nothing", NAME="right"
+KERNEL=="ttyUSB*", NAME="wrong"
+EOF
+       },
+       {
+               desc            => "test multi matches 2",
+               subsys          => "tty",
+               devpath         => "/class/tty/ttyUSB0",
+               exp_name        => "right",
+               rules           => <<EOF
+KERNEL=="dontknow*|*nothing", NAME="nomatch"
+KERNEL=="dontknow*|ttyUSB*|nothing*", NAME="right"
+KERNEL=="ttyUSB*", NAME="wrong"
+EOF
+       },
+       {
+               desc            => "IMPORT parent test sequence 1/2 (keep)",
+               subsys          => "block",
+               devpath         => "/block/sda",
+               exp_name        => "parent",
+               option          => "keep",
+               rules           => <<EOF
+KERNEL=="sda", IMPORT="/bin/echo -e \'PARENT_KEY=parent_right\\nWRONG_PARENT_KEY=parent_wrong'"
+KERNEL=="sda", NAME="parent"
+EOF
+       },
+       {
+               desc            => "IMPORT parent test sequence 2/2 (keep)",
+               subsys          => "block",
+               devpath         => "/block/sda/sda1",
+               exp_name        => "parentenv-parent_right",
+               option          => "clean",
+               rules           => <<EOF
+KERNEL=="sda1", IMPORT{parent}="PARENT*", NAME="parentenv-\$env{PARENT_KEY}\$env{WRONG_PARENT_KEY}"
+EOF
+       },
+       {
+               desc            => "GOTO test",
+               subsys          => "block",
+               devpath         => "/block/sda/sda1",
+               exp_name        => "right",
+               rules           => <<EOF
+KERNEL=="sda1", GOTO="TEST"
+KERNEL=="sda1", NAME="wrong"
+KERNEL=="sda1", NAME="", LABEL="NO"
+KERNEL=="sda1", NAME="right", LABEL="TEST"
+KERNEL=="sda1", NAME="wrong2"
 EOF
        },
 );
 
 # set env
-$ENV{UDEV_TEST} = "yes";
+$ENV{ENV_KEY_TEST} = "test";
 $ENV{SYSFS_PATH} = $sysfs;
-$ENV{UDEV_CONFIG_FILE} = $main_conf;
+$ENV{UDEV_CONFIG_FILE} = $udev_conf;
 $ENV{UDEV_NO_DEVD} = "yes";
 $ENV{UDEV_NO_HOTPLUGD} = "yes";
 
 
 sub udev {
-       my ($action, $subsys, $devpath, $config) = @_;
+       my ($action, $subsys, $devpath, $rules) = @_;
 
        $ENV{DEVPATH} = $devpath;
 
-       # create temporary config
-       open CONF, ">$conf_tmp" || die "unable to create config file: $conf_tmp";
-       print CONF $$config;
+       # create temporary rules
+       open CONF, ">$udev_rules" || die "unable to create rules file: $udev_rules";
+       print CONF $$rules;
        close CONF;
 
        $ENV{ACTION} = $action;
@@ -1156,36 +1577,50 @@ sub udev {
 my $error = 0;
 
 sub permissions_test {
-       my($config, $uid, $gid, $mode) = @_;
+       my($rules, $uid, $gid, $mode) = @_;
 
        my $wrong = 0;
-       $config->{exp_perms} =~ m/^(.*):(.*):(.*)$/;
+       my $userid;
+       my $groupid;
+
+       $rules->{exp_perms} =~ m/^(.*):(.*):(.*)$/;
        if ($1 ne "") {
-               if ($uid != $1) { $wrong = 1; };
+               if (defined(getpwnam($1))) {
+                       $userid = int(getpwnam($1));
+               } else {
+                       $userid = $1;
+               }
+               if ($uid != $userid) { $wrong = 1; }
        }
        if ($2 ne "") {
-               if ($gid != $2) { $wrong = 1; };
+               if (defined(getgrnam($2))) {
+                       $groupid = int(getgrnam($2));
+               } else {
+                       $groupid = $2;
+               }
+               if ($gid != $groupid) { $wrong = 1; }
        }
        if ($3 ne "") {
                if (($mode & 07777) != oct($3)) { $wrong = 1; };
        }
        if ($wrong == 0) {
-               print "permissions: ok    ";
+               print "permissions: ok\n";
        } else {
-               printf "expected permissions are: %i:%i:%#o\n", $1, $2, oct($3);
-               printf "created permissions are : %i:%i:%#o\n", $uid, $gid, $mode & 07777;
+               printf "  expected permissions are: %s:%s:%#o\n", $1, $2, oct($3);
+               printf "  created permissions are : %i:%i:%#o\n", $uid, $gid, $mode & 07777;
+               print "permissions: error\n";
                $error++;
        }
 }
 
 sub major_minor_test {
-       my($config, $rdev) = @_;
+       my($rules, $rdev) = @_;
 
        my $major = ($rdev >> 8) & 0xfff;
        my $minor = ($rdev & 0xff) | (($rdev >> 12) & 0xfff00);
        my $wrong = 0;
 
-       $config->{exp_majorminor} =~ m/^(.*):(.*)$/;
+       $rules->{exp_majorminor} =~ m/^(.*):(.*)$/;
        if ($1 ne "") {
                if ($major != $1) { $wrong = 1; };
        }
@@ -1193,106 +1628,118 @@ sub major_minor_test {
                if ($minor != $2) { $wrong = 1; };
        }
        if ($wrong == 0) {
-               print "major:minor: ok    ";
+               print "major:minor: ok\n";
        } else {
-               printf "expected major:minor is: %i:%i\n", $1, $2;
-               printf "created major:minor is : %i:%i\n", $major, $minor;
-               print "major:minor: error    ";
+               printf "  expected major:minor is: %i:%i\n", $1, $2;
+               printf "  created major:minor is : %i:%i\n", $major, $minor;
+               print "major:minor: error\n";
                $error++;
        }
 }
 
 sub symlink_test {
-       my ($config) = @_;
+       my ($rules) = @_;
 
-       my $output = `ls -l $PWD/$udev_root$config->{exp_name}`;
+       my $output = `ls -l $PWD/$udev_root$rules->{exp_name}`;
 
        if ($output =~ m/(.*)-> (.*)/) {
-               if ($2 eq $config->{exp_target}) {
-                       print "symlink: ok    ";
+               if ($2 eq $rules->{exp_target}) {
+                       print "symlink:     ok\n";
                } else {
-                       print "expected symlink from: \'$config->{exp_name}\' to \'$config->{exp_target}\'\n";
-                       print "created symlink from: \'$config->{exp_name}\' to \'$2\'\n";
-                       if ($config->{exp_add_error}) {
-                               print "as expected    ";
+                       print "  expected symlink from: \'$rules->{exp_name}\' to \'$rules->{exp_target}\'\n";
+                       print "  created symlink from: \'$rules->{exp_name}\' to \'$2\'\n";
+                       print "symlink: error";
+                       if ($rules->{exp_add_error}) {
+                               print " as expected\n";
                        } else {
+                               print "\n";
                                $error++;
                        }
                }
        } else {
-               print "expected symlink from: \'$config->{exp_name}\' to \'$config->{exp_target}\'\n";
-               print "symlink: not created ";
-               if ($config->{exp_add_error}) {
-                       print "as expected    ";
+               print "  expected symlink from: \'$rules->{exp_name}\' to \'$rules->{exp_target}\'\n";
+               print "symlink:     not created";
+               if ($rules->{exp_add_error}) {
+                       print " as expected\n";
                } else {
+                       print "\n";
                        $error++;
                }
        }
 }
 
 sub run_test {
-       my ($config, $number) = @_;
+       my ($rules, $number) = @_;
 
-       print "TEST $number: $config->{desc}\n";
+       print "TEST $number: $rules->{desc}\n";
 
-       if ($config->{exp_target}) {
-               print "device \'$config->{devpath}\' expecting symlink '$config->{exp_name}' to node \'$config->{exp_target}\'\n";
+       if ($rules->{exp_target}) {
+               print "device \'$rules->{devpath}\' expecting symlink '$rules->{exp_name}' to node \'$rules->{exp_target}\'\n";
        } else {
-               print "device \'$config->{devpath}\' expecting node \'$config->{exp_name}\'\n";
+               print "device \'$rules->{devpath}\' expecting node \'$rules->{exp_name}\'\n";
        }
 
 
-       udev("add", $config->{subsys}, $config->{devpath}, \$config->{conf});
-       if ((-e "$PWD/$udev_root$config->{exp_name}") ||
-           (-l "$PWD/$udev_root$config->{exp_name}")) {
+       udev("add", $rules->{subsys}, $rules->{devpath}, \$rules->{rules});
+       if ((-e "$PWD/$udev_root$rules->{exp_name}") ||
+           (-l "$PWD/$udev_root$rules->{exp_name}")) {
 
                my ($dev, $ino, $mode, $nlink, $uid, $gid, $rdev, $size,
-                   $atime, $mtime, $ctime, $blksize, $blocks) = stat("$PWD/$udev_root$config->{exp_name}");
+                   $atime, $mtime, $ctime, $blksize, $blocks) = stat("$PWD/$udev_root$rules->{exp_name}");
 
-               if (defined($config->{exp_perms})) {
-                       permissions_test($config, $uid, $gid, $mode);
+               if (defined($rules->{not_exp_name})) {
+                       if ((-e "$PWD/$udev_root$rules->{not_exp_name}") ||
+                           (-l "$PWD/$udev_root$rules->{not_exp_name}")) {
+                               print "nonexistent: error \'$rules->{not_exp_name}\' not expected to be there\n";
+                               $error++
+                       }
+               }
+               if (defined($rules->{exp_perms})) {
+                       permissions_test($rules, $uid, $gid, $mode);
                }
-               if (defined($config->{exp_majorminor})) {
-                       major_minor_test($config, $rdev);
+               if (defined($rules->{exp_majorminor})) {
+                       major_minor_test($rules, $rdev);
                }
-               if (defined($config->{exp_target})) {
-                       symlink_test($config);
+               if (defined($rules->{exp_target})) {
+                       symlink_test($rules);
                }
-               print "add: ok    ";
+               print "add:         ok\n";
        } else {
-               print "add: error ";
-               if ($config->{exp_add_error}) {
-                       print "as expected    ";
+               print "add:         error";
+               if ($rules->{exp_add_error}) {
+                       print " as expected\n";
                } else {
-                       print "\n\n";
+                       print "\n";
                        system("tree $udev_root");
                        print "\n";
                        $error++;
                }
        }
 
-       if (defined($config->{option}) && $config->{option} eq "keep") {
+       if (defined($rules->{option}) && $rules->{option} eq "keep") {
                print "\n\n";
                return;
        }
 
-       udev("remove", $config->{subsys}, $config->{devpath}, \$config->{conf});
-       if ((-e "$PWD/$udev_root$config->{exp_name}") ||
-           (-l "$PWD/$udev_root$config->{exp_name}")) {
-               print "remove: error ";
-               if ($config->{exp_rem_error}) {
-                       print "as expected\n\n";
+       udev("remove", $rules->{subsys}, $rules->{devpath}, \$rules->{rules});
+       if ((-e "$PWD/$udev_root$rules->{exp_name}") ||
+           (-l "$PWD/$udev_root$rules->{exp_name}")) {
+               print "remove:      error";
+               if ($rules->{exp_rem_error}) {
+                       print " as expected\n";
                } else {
-                       print "\n\n";
+                       print "\n";
                        system("tree $udev_root");
                        print "\n";
                        $error++;
                }
        } else {
-               print "remove: ok\n\n";
+               print "remove:      ok\n";
        }
 
-       if (defined($config->{option}) && $config->{option} eq "clear") {
+       print "\n";
+
+       if (defined($rules->{option}) && $rules->{option} eq "clean") {
                system("rm -rf $udev_db");
                system("rm -rf $udev_root");
                mkdir($udev_root) || die "unable to create udev_root: $udev_root\n";
@@ -1311,11 +1758,11 @@ if (!($<==0)) {
 system("rm -rf $udev_root");
 mkdir($udev_root) || die "unable to create udev_root: $udev_root\n";
 
-# create initial config file
-open CONF, ">$main_conf" || die "unable to create config file: $main_conf";
+# create config file
+open CONF, ">$udev_conf" || die "unable to create config file: $udev_conf";
 print CONF "udev_root=\"$udev_root\"\n";
 print CONF "udev_db=\"$udev_db\"\n";
-print CONF "udev_rules=\"$conf_tmp\"\n";
+print CONF "udev_rules=\"$udev_rules\"\n";
 close CONF;
 
 my $test_num = 1;
@@ -1334,8 +1781,8 @@ if ($ARGV[0]) {
        # test all
        print "\nudev-test will run ".($#tests + 1)." tests:\n\n";
 
-       foreach my $config (@tests) {
-               run_test($config, $test_num);
+       foreach my $rules (@tests) {
+               run_test($rules, $test_num);
                $test_num++;
        }
 }
@@ -1345,6 +1792,6 @@ print "$error errors occured\n\n";
 # cleanup
 system("rm -rf $udev_db");
 system("rm -rf $udev_root");
-unlink($conf_tmp);
-unlink($main_conf);
+unlink($udev_rules);
+unlink($udev_conf);