[qemu-server.git] / test / run_config2command_tests.pl

#!/usr/bin/perl

use strict;
use warnings;

use lib qw(..);

use Test::More;
use Test::MockModule;
use Socket qw(AF_INET AF_INET6);

use PVE::Tools qw(file_get_contents file_set_contents run_command);
use PVE::INotify;
use PVE::SysFSTools;

use PVE::QemuConfig;
use PVE::QemuServer;
use PVE::QemuServer::Monitor;
use PVE::QemuServer::Machine;
use PVE::QemuServer::CPUConfig;

my $base_env = {
    storage_config => {
	ids => {
	    local => {
		content => {
		    images => 1,
		},
		path => '/var/lib/vz',
		type => 'dir',
		shared => 0,
	    },
	    'btrfs-store' => {
		content => {
		    images => 1,
		},
		path => '/butter/bread',
		type => 'btrfs',
	    },
	    'cifs-store' => {
		shared => 1,
		path => '/mnt/pve/cifs-store',
		username => 'guest',
		server => '127.0.0.42',
		type => 'cifs',
		share => 'CIFShare',
		content => {
		    images => 1
		},
	    },
	    'rbd-store' => {
		monhost => '127.0.0.42,127.0.0.21,::1',
		fsid => 'fc4181a6-56eb-4f68-b452-8ba1f381ca2a',
		content => {
		    images => 1
		},
		type => 'rbd',
		pool => 'cpool',
		username => 'admin',
		shared => 1
	    },
	    'local-lvm' => {
		vgname => 'pve',
		bwlimit => 'restore=1024',
		type => 'lvmthin',
		thinpool => 'data',
		content => {
		    images => 1,
		}
	    }
	}
    },
    vmid => 8006,
    real_qemu_version => PVE::QemuServer::kvm_user_version(), # not yet mocked
};

my $pci_devs = [
    "0000:00:43.1",
    "0000:00:f4.0",
    "0000:00:ff.1",
    "0000:0f:f2.0",
    "0000:d0:13.0",
    "0000:d0:15.1",
    "0000:d0:17.0",
    "0000:f0:42.0",
    "0000:f0:43.0",
    "0000:f0:43.1",
    "1234:f0:43.1",
];

my $current_test; # = {
#   description => 'Test description', # if available
#   qemu_version => '2.12',
#   host_arch => 'HOST_ARCH',
#   expected_error => 'error message',
#   expected_warning => 'warning message',
#   config => { config hash },
#   expected => [ expected outcome cmd line array ],
# };

# use the config description to allow changing environment, fields are:
#   TEST: A single line describing the test, gets outputted
#   QEMU_VERSION: \d+\.\d+(\.\d+)? (defaults to current version)
#   HOST_ARCH: x86_64 | aarch64 (default to x86_64, to make tests stable)
#   EXPECT_ERROR: <error message> For negative tests
# all fields are optional
sub parse_test($) {
    my ($config_fn) = @_;

    $current_test = {}; # reset

    my $fake_config_fn ="$config_fn/qemu-server/8006.conf";
    my $config_raw = file_get_contents($config_fn);
    my $config = PVE::QemuServer::parse_vm_config($fake_config_fn, $config_raw);

    $current_test->{config} = $config;

    my $description = $config->{description} // '';

    while ($description =~ /^\h*(.*?)\h*$/gm) {
	my $line = $1;
	next if !$line || $line =~ /^#/;
	$line =~ s/^\s+//;
	$line =~ s/\s+$//;

	if ($line =~ /^TEST:\s*(.*)\s*$/) {
	    $current_test->{description} = "$1";
	} elsif ($line =~ /^QEMU_VERSION:\s*(.*)\s*$/) {
	    $current_test->{qemu_version} = "$1";
	} elsif ($line =~ /^HOST_ARCH:\s*(.*)\s*$/) {
	    $current_test->{host_arch} = "$1";
	} elsif ($line =~ /^EXPECT_ERROR:\s*(.*)\s*$/) {
	    $current_test->{expect_error} = "$1";
	} elsif ($line =~ /^EXPECT_WARN(?:ING)?:\s*(.*)\s*$/) {
	    $current_test->{expect_warning} = "$1";
	}
    }

    $config_fn =~ /([^\/]+)$/;
    my $testname = "$1";
    if (my $desc = $current_test->{description}) {
	$testname = "'$testname' - $desc";
    }
    $current_test->{testname} = $testname;
}

sub get_test_qemu_version {
    $current_test->{qemu_version} // $base_env->{real_qemu_version} // '2.12';
}

my $qemu_server_module;
$qemu_server_module = Test::MockModule->new('PVE::QemuServer');
$qemu_server_module->mock(
    kvm_user_version => sub {
	return get_test_qemu_version();
    },
    kvm_version => sub {
	return get_test_qemu_version();
    },
    kernel_has_vhost_net => sub {
	return 1; # TODO: make this per-test configurable?
    },
    get_host_arch => sub() {
	return $current_test->{host_arch} // 'x86_64';
    },
    get_initiator_name => sub {
	return 'iqn.1993-08.org.debian:01:aabbccddeeff';
    }
);

my $qemu_server_config;
$qemu_server_config = Test::MockModule->new('PVE::QemuConfig');
$qemu_server_config->mock(
    load_config => sub {
	my ($class, $vmid, $node) = @_;

	return $current_test->{config};
    },
);

my $pve_common_tools;
$pve_common_tools = Test::MockModule->new('PVE::Tools');
$pve_common_tools->mock(
    next_vnc_port => sub {
	my ($family, $address) = @_;

	return '5900';
    },
    next_spice_port => sub {
	my ($family, $address) = @_;

	return '61000';
    },
    getaddrinfo_all => sub {
	my ($hostname, @opts) = @_;
	die "need stable hostname" if $hostname ne 'localhost';
	return (
	    {
		addr => Socket::pack_sockaddr_in(0, Socket::INADDR_LOOPBACK),
		family => AF_INET, # IPv4
		protocol => 6,
		socktype => 1,
	    },
	);
    },
);

my $pve_cpuconfig;
$pve_cpuconfig = Test::MockModule->new('PVE::QemuServer::CPUConfig');
$pve_cpuconfig->mock(
    load_custom_model_conf => sub {
	# mock custom CPU model config
	return PVE::QemuServer::CPUConfig->parse_config("cpu-models.conf",
<<EOF

# "qemu64" is also a default CPU, used here to test that this doesn't matter
cpu-model: qemu64
    reported-model athlon
    flags +aes;+avx;-kvm_pv_unhalt
    hv-vendor-id testvend
    phys-bits 40

cpu-model: alldefault

EOF
	)
    },
);

my $pve_common_network;
$pve_common_network = Test::MockModule->new('PVE::Network');
$pve_common_network->mock(
    read_bridge_mtu => sub {
	return 1500;
    },
);


my $pve_common_inotify;
$pve_common_inotify = Test::MockModule->new('PVE::INotify');
$pve_common_inotify->mock(
    nodename => sub {
	return 'localhost';
    },
);

my $pve_common_sysfstools;
$pve_common_sysfstools = Test::MockModule->new('PVE::SysFSTools');
$pve_common_sysfstools->mock(
    lspci => sub {
	my ($filter, $verbose) = @_;

	return [
	    map { { id => $_ } }
	    grep {
		!defined($filter)
		|| (!ref($filter) && $_ =~ m/^(0000:)?\Q$filter\E/)
		|| (ref($filter) eq 'CODE' && $filter->({ id => $_ }))
	    } sort @$pci_devs
	];
    },
);

my $qemu_monitor_module;
$qemu_monitor_module = Test::MockModule->new('PVE::QemuServer::Monitor');
$qemu_monitor_module->mock(
    mon_cmd => sub {
	my ($vmid, $cmd) = @_;

	die "invalid vmid: $vmid (expected: $base_env->{vmid})"
	    if $vmid != $base_env->{vmid};

	if ($cmd eq 'query-version') {
	    my $ver = get_test_qemu_version();
	    $ver =~ m/(\d+)\.(\d+)(?:\.(\d+))?/;
	    return {
		qemu => {
		    major => $1,
		    minor => $2,
		    micro => $3
		}
	    }
	}

	die "unexpected QMP command: '$cmd'";
    },
);
$qemu_monitor_module->mock('qmp_cmd', \&qmp_cmd);

sub diff($$) {
    my ($a, $b) = @_;
    return if $a eq $b;

    my ($ra, $wa) = POSIX::pipe();
    my ($rb, $wb) = POSIX::pipe();
    my $ha = IO::Handle->new_from_fd($wa, 'w');
    my $hb = IO::Handle->new_from_fd($wb, 'w');

    open my $diffproc, '-|', 'diff', '-up', "/proc/self/fd/$ra", "/proc/self/fd/$rb" ## no critic
	or die "failed to run program 'diff': $!";
    POSIX::close($ra);
    POSIX::close($rb);

    open my $f1, '<', \$a;
    open my $f2, '<', \$b;
    my ($line1, $line2);
    do {
	$ha->print($line1) if defined($line1 = <$f1>);
	$hb->print($line2) if defined($line2 = <$f2>);
    } while (defined($line1 // $line2));
    close $f1;
    close $f2;
    close $ha;
    close $hb;

    local $/ = undef;
    my $diff = <$diffproc>;
    close $diffproc;
    die "files differ:\n$diff";
}

$SIG{__WARN__} = sub {
    my $warning = shift;
    chomp $warning;
    if (my $warn_expect = $current_test->{expect_warning}) {
	if ($warn_expect ne $warning) {
	    fail($current_test->{testname});
	    note("warning does not match expected error: '$warning' != '$warn_expect'");
	} else {
	    note("got expected warning '$warning'");
	    return;
	}
    }

    fail($current_test->{testname});
    note("got unexpected warning '$warning'");
};

sub do_test($) {
    my ($config_fn) = @_;

    die "no such input test config: $config_fn\n" if ! -f $config_fn;

    parse_test $config_fn;

    my $testname = $current_test->{testname};

    my ($vmid, $storecfg) = $base_env->@{qw(vmid storage_config)};

    my $cmdline = eval { PVE::QemuServer::vm_commandline($storecfg, $vmid) };
    my $err = $@;

    if (my $err_expect = $current_test->{expect_error}) {
	if (!$err) {
	    fail("$testname");
	    note("did NOT get any error, but expected error: $err_expect");
	    return;
	}
	chomp $err;
	if ($err ne $err_expect) {
	    fail("$testname");
	    note("error does not match expected error: '$err' !~ '$err_expect'");
	} else {
	    pass("$testname");
	}
	return;
    } elsif ($err) {
	fail("$testname");
	note("got unexpected error: $err");
	return;
    }

    # check if QEMU version set correctly and test version_cmp
    (my $qemu_major = get_test_qemu_version()) =~ s/\..*$//;
    die "runs_at_least_qemu_version returned false, maybe error in version_cmp?"
	if !PVE::QemuServer::Machine::runs_at_least_qemu_version($vmid, $qemu_major);

    $cmdline =~ s/ -/ \\\n  -/g; # same as qm showcmd --pretty
    $cmdline .= "\n";

    my $cmd_fn = "$config_fn.cmd";

    if (-f $cmd_fn) {
	my $cmdline_expected = file_get_contents($cmd_fn);

	my $cmd_expected = [ split /\s*\\?\n\s*/, $cmdline_expected ];
	my $cmd = [ split /\s*\\?\n\s*/, $cmdline ];

	# uncomment for easier debugging
	#file_set_contents("$cmd_fn.tmp", $cmdline);

	my $exp = join("\n", @$cmd_expected);
	my $got = join("\n", @$cmd);
	eval { diff($exp, $got) };
	if (my $err = $@) {
	    fail("$testname");
	    note($err);
	} else {
	    pass("$testname");
	}
    } else {
	file_set_contents($cmd_fn, $cmdline);
    }
}

print "testing config to command stabillity\n";

# exec tests
if (my $file = shift) {
    do_test $file;
} else {
    while (my $file = <cfg2cmd/*.conf>) {
	do_test $file;
    }
}

done_testing();
Commit	Line	Data
7b963e57 TL	1	#!/usr/bin/perl
	2
	3	use strict;
	4	use warnings;
	5
	6	use lib qw(..);
	7
	8	use Test::More;
	9	use Test::MockModule;
38277afc	10	use Socket qw(AF_INET AF_INET6);
7b963e57 TL	11
7b963e57 TL	12	use PVE::Tools qw(file_get_contents file_set_contents run_command);
38277afc TL	13	use PVE::INotify;
	14	use PVE::SysFSTools;
	15
7b963e57 TL	16	use PVE::QemuConfig;
7b963e57 TL	17	use PVE::QemuServer;
6db4c69e SR	18	use PVE::QemuServer::Monitor;
6db4c69e SR	19	use PVE::QemuServer::Machine;
5bc08470	20	use PVE::QemuServer::CPUConfig;
7b963e57 TL	21
	22	my $base_env = {
	23	storage_config => {
	24	ids => {
	25	local => {
	26	content => {
	27	images => 1,
	28	},
	29	path => '/var/lib/vz',
	30	type => 'dir',
	31	shared => 0,
	32	},
5620282f TL	33	'btrfs-store' => {
	34	content => {
	35	images => 1,
	36	},
	37	path => '/butter/bread',
	38	type => 'btrfs',
	39	},
7b963e57 TL	40	'cifs-store' => {
	41	shared => 1,
	42	path => '/mnt/pve/cifs-store',
	43	username => 'guest',
	44	server => '127.0.0.42',
	45	type => 'cifs',
	46	share => 'CIFShare',
	47	content => {
	48	images => 1
	49	},
	50	},
	51	'rbd-store' => {
	52	monhost => '127.0.0.42,127.0.0.21,::1',
188eb9c3	53	fsid => 'fc4181a6-56eb-4f68-b452-8ba1f381ca2a',
7b963e57 TL	54	content => {
	55	images => 1
	56	},
	57	type => 'rbd',
	58	pool => 'cpool',
	59	username => 'admin',
	60	shared => 1
	61	},
	62	'local-lvm' => {
	63	vgname => 'pve',
	64	bwlimit => 'restore=1024',
	65	type => 'lvmthin',
	66	thinpool => 'data',
	67	content => {
	68	images => 1,
	69	}
	70	}
	71	}
	72	},
	73	vmid => 8006,
	74	real_qemu_version => PVE::QemuServer::kvm_user_version(), # not yet mocked
	75	};
	76
0360faad DC	77	my $pci_devs = [
	78	"0000:00:43.1",
	79	"0000:00:f4.0",
	80	"0000:00:ff.1",
	81	"0000:0f:f2.0",
	82	"0000:d0:13.0",
	83	"0000:d0:15.1",
	84	"0000:d0:17.0",
	85	"0000:f0:42.0",
	86	"0000:f0:43.0",
	87	"0000:f0:43.1",
	88	"1234:f0:43.1",
	89	];
	90
7b963e57 TL	91	my $current_test; # = {
	92	# description => 'Test description', # if available
	93	# qemu_version => '2.12',
	94	# host_arch => 'HOST_ARCH',
1e27bda1 TL	95	# expected_error => 'error message',
1e27bda1 TL	96	# expected_warning => 'warning message',
7b963e57 TL	97	# config => { config hash },
	98	# expected => [ expected outcome cmd line array ],
	99	# };
	100
	101	# use the config description to allow changing environment, fields are:
	102	# TEST: A single line describing the test, gets outputted
	103	# QEMU_VERSION: \d+\.\d+(\.\d+)? (defaults to current version)
	104	# HOST_ARCH: x86_64 \| aarch64 (default to x86_64, to make tests stable)
1e27bda1	105	# EXPECT_ERROR: <error message> For negative tests
7b963e57 TL	106	# all fields are optional
	107	sub parse_test($) {
	108	my ($config_fn) = @_;
	109
	110	$current_test = {}; # reset
	111
	112	my $fake_config_fn ="$config_fn/qemu-server/8006.conf";
	113	my $config_raw = file_get_contents($config_fn);
	114	my $config = PVE::QemuServer::parse_vm_config($fake_config_fn, $config_raw);
	115
	116	$current_test->{config} = $config;
	117
	118	my $description = $config->{description} // '';
	119
	120	while ($description =~ /^\h(.?)\h*$/gm) {
	121	my $line = $1;
	122	next if !$line \|\| $line =~ /^#/;
	123	$line =~ s/^\s+//;
	124	$line =~ s/\s+$//;
	125
	126	if ($line =~ /^TEST:\s(.)\s*$/) {
	127	$current_test->{description} = "$1";
	128	} elsif ($line =~ /^QEMU_VERSION:\s(.)\s*$/) {
	129	$current_test->{qemu_version} = "$1";
	130	} elsif ($line =~ /^HOST_ARCH:\s(.)\s*$/) {
	131	$current_test->{host_arch} = "$1";
a546da03 TL	132	} elsif ($line =~ /^EXPECT_ERROR:\s(.)\s*$/) {
a546da03 TL	133	$current_test->{expect_error} = "$1";
1e27bda1 TL	134	} elsif ($line =~ /^EXPECT_WARN(?:ING)?:\s(.)\s*$/) {
1e27bda1 TL	135	$current_test->{expect_warning} = "$1";
7b963e57 TL	136	}
7b963e57 TL	137	}
90d96715 TL	138
	139	$config_fn =~ /([^\/]+)$/;
	140	my $testname = "$1";
	141	if (my $desc = $current_test->{description}) {
	142	$testname = "'$testname' - $desc";
	143	}
	144	$current_test->{testname} = $testname;
7b963e57 TL	145	}
7b963e57 TL	146
6db4c69e SR	147	sub get_test_qemu_version {
	148	$current_test->{qemu_version} // $base_env->{real_qemu_version} // '2.12';
	149	}
	150
7b963e57 TL	151	my $qemu_server_module;
	152	$qemu_server_module = Test::MockModule->new('PVE::QemuServer');
	153	$qemu_server_module->mock(
	154	kvm_user_version => sub {
6db4c69e	155	return get_test_qemu_version();
7b963e57	156	},
66c53994	157	kvm_version => sub {
6db4c69e	158	return get_test_qemu_version();
66c53994	159	},
db70021b TL	160	kernel_has_vhost_net => sub {
	161	return 1; # TODO: make this per-test configurable?
	162	},
7b963e57 TL	163	get_host_arch => sub() {
	164	return $current_test->{host_arch} // 'x86_64';
	165	},
a7740135 WB	166	get_initiator_name => sub {
	167	return 'iqn.1993-08.org.debian:01:aabbccddeeff';
	168	}
7b963e57 TL	169	);
	170
	171	my $qemu_server_config;
	172	$qemu_server_config = Test::MockModule->new('PVE::QemuConfig');
	173	$qemu_server_config->mock(
	174	load_config => sub {
	175	my ($class, $vmid, $node) = @_;
	176
	177	return $current_test->{config};
	178	},
	179	);
	180
ff9e7dc1 TL	181	my $pve_common_tools;
	182	$pve_common_tools = Test::MockModule->new('PVE::Tools');
	183	$pve_common_tools->mock(
	184	next_vnc_port => sub {
	185	my ($family, $address) = @_;
	186
	187	return '5900';
	188	},
	189	next_spice_port => sub {
	190	my ($family, $address) = @_;
	191
	192	return '61000';
	193	},
38277afc TL	194	getaddrinfo_all => sub {
	195	my ($hostname, @opts) = @_;
	196	die "need stable hostname" if $hostname ne 'localhost';
	197	return (
	198	{
	199	addr => Socket::pack_sockaddr_in(0, Socket::INADDR_LOOPBACK),
	200	family => AF_INET, # IPv4
	201	protocol => 6,
	202	socktype => 1,
	203	},
	204	);
	205	},
	206	);
	207
5bc08470 SR	208	my $pve_cpuconfig;
	209	$pve_cpuconfig = Test::MockModule->new('PVE::QemuServer::CPUConfig');
	210	$pve_cpuconfig->mock(
	211	load_custom_model_conf => sub {
	212	# mock custom CPU model config
	213	return PVE::QemuServer::CPUConfig->parse_config("cpu-models.conf",
	214	<<EOF
	215
	216	# "qemu64" is also a default CPU, used here to test that this doesn't matter
	217	cpu-model: qemu64
	218	reported-model athlon
	219	flags +aes;+avx;-kvm_pv_unhalt
	220	hv-vendor-id testvend
	221	phys-bits 40
	222
	223	cpu-model: alldefault
	224
	225	EOF
	226	)
	227	},
	228	);
	229
121e3400 FG	230	my $pve_common_network;
	231	$pve_common_network = Test::MockModule->new('PVE::Network');
	232	$pve_common_network->mock(
	233	read_bridge_mtu => sub {
	234	return 1500;
	235	},
	236	);
	237
	238
38277afc TL	239	my $pve_common_inotify;
	240	$pve_common_inotify = Test::MockModule->new('PVE::INotify');
	241	$pve_common_inotify->mock(
	242	nodename => sub {
	243	return 'localhost';
	244	},
ff9e7dc1 TL	245	);
ff9e7dc1 TL	246
0360faad DC	247	my $pve_common_sysfstools;
	248	$pve_common_sysfstools = Test::MockModule->new('PVE::SysFSTools');
	249	$pve_common_sysfstools->mock(
	250	lspci => sub {
	251	my ($filter, $verbose) = @_;
	252
	253	return [
	254	map { { id => $_ } }
	255	grep {
	256	!defined($filter)
	257	\|\| (!ref($filter) && $_ =~ m/^(0000:)?\Q$filter\E/)
	258	\|\| (ref($filter) eq 'CODE' && $filter->({ id => $_ }))
	259	} sort @$pci_devs
	260	];
	261	},
	262	);
	263
6db4c69e SR	264	my $qemu_monitor_module;
	265	$qemu_monitor_module = Test::MockModule->new('PVE::QemuServer::Monitor');
	266	$qemu_monitor_module->mock(
	267	mon_cmd => sub {
	268	my ($vmid, $cmd) = @_;
	269
	270	die "invalid vmid: $vmid (expected: $base_env->{vmid})"
	271	if $vmid != $base_env->{vmid};
	272
	273	if ($cmd eq 'query-version') {
	274	my $ver = get_test_qemu_version();
	275	$ver =~ m/(\d+)\.(\d+)(?:\.(\d+))?/;
	276	return {
	277	qemu => {
	278	major => $1,
	279	minor => $2,
	280	micro => $3
	281	}
	282	}
	283	}
	284
	285	die "unexpected QMP command: '$cmd'";
	286	},
	287	);
	288	$qemu_monitor_module->mock('qmp_cmd', \&qmp_cmd);
	289
3dc780c6 WB	290	sub diff($$) {
	291	my ($a, $b) = @_;
	292	return if $a eq $b;
	293
	294	my ($ra, $wa) = POSIX::pipe();
	295	my ($rb, $wb) = POSIX::pipe();
	296	my $ha = IO::Handle->new_from_fd($wa, 'w');
	297	my $hb = IO::Handle->new_from_fd($wb, 'w');
	298
808a65b5	299	open my $diffproc, '-\|', 'diff', '-up', "/proc/self/fd/$ra", "/proc/self/fd/$rb" ## no critic
3dc780c6 WB	300	or die "failed to run program 'diff': $!";
	301	POSIX::close($ra);
	302	POSIX::close($rb);
	303
	304	open my $f1, '<', \$a;
	305	open my $f2, '<', \$b;
	306	my ($line1, $line2);
	307	do {
	308	$ha->print($line1) if defined($line1 = <$f1>);
	309	$hb->print($line2) if defined($line2 = <$f2>);
	310	} while (defined($line1 // $line2));
	311	close $f1;
	312	close $f2;
	313	close $ha;
	314	close $hb;
	315
	316	local $/ = undef;
	317	my $diff = <$diffproc>;
	318	close $diffproc;
	319	die "files differ:\n$diff";
	320	}
	321
1e27bda1 TL	322	$SIG{__WARN__} = sub {
	323	my $warning = shift;
	324	chomp $warning;
	325	if (my $warn_expect = $current_test->{expect_warning}) {
	326	if ($warn_expect ne $warning) {
	327	fail($current_test->{testname});
	328	note("warning does not match expected error: '$warning' != '$warn_expect'");
	329	} else {
	330	note("got expected warning '$warning'");
	331	return;
	332	}
	333	}
	334
1e27bda1	335	fail($current_test->{testname});
3eb2f3eb	336	note("got unexpected warning '$warning'");
1e27bda1 TL	337	};
1e27bda1 TL	338
7b963e57 TL	339	sub do_test($) {
	340	my ($config_fn) = @_;
	341
	342	die "no such input test config: $config_fn\n" if ! -f $config_fn;
	343
	344	parse_test $config_fn;
	345
90d96715	346	my $testname = $current_test->{testname};
7b963e57 TL	347
	348	my ($vmid, $storecfg) = $base_env->@{qw(vmid storage_config)};
	349
a546da03 TL	350	my $cmdline = eval { PVE::QemuServer::vm_commandline($storecfg, $vmid) };
	351	my $err = $@;
	352
	353	if (my $err_expect = $current_test->{expect_error}) {
	354	if (!$err) {
	355	fail("$testname");
	356	note("did NOT get any error, but expected error: $err_expect");
	357	return;
	358	}
	359	chomp $err;
ac0077cc	360	if ($err ne $err_expect) {
a546da03 TL	361	fail("$testname");
	362	note("error does not match expected error: '$err' !~ '$err_expect'");
	363	} else {
	364	pass("$testname");
	365	}
	366	return;
	367	} elsif ($err) {
	368	fail("$testname");
	369	note("got unexpected error: $err");
	370	return;
	371	}
7b963e57	372
6db4c69e SR	373	# check if QEMU version set correctly and test version_cmp
	374	(my $qemu_major = get_test_qemu_version()) =~ s/\..*$//;
	375	die "runs_at_least_qemu_version returned false, maybe error in version_cmp?"
	376	if !PVE::QemuServer::Machine::runs_at_least_qemu_version($vmid, $qemu_major);
	377
7b963e57 TL	378	$cmdline =~ s/ -/ \\\n -/g; # same as qm showcmd --pretty
	379	$cmdline .= "\n";
	380
	381	my $cmd_fn = "$config_fn.cmd";
	382
	383	if (-f $cmd_fn) {
	384	my $cmdline_expected = file_get_contents($cmd_fn);
	385
69f4bd34 TL	386	my $cmd_expected = [ split /\s\\?\n\s/, $cmdline_expected ];
69f4bd34 TL	387	my $cmd = [ split /\s\\?\n\s/, $cmdline ];
7b963e57	388
8b26544e	389	# uncomment for easier debugging
7b963e57 TL	390	#file_set_contents("$cmd_fn.tmp", $cmdline);
7b963e57 TL	391
3dc780c6 WB	392	my $exp = join("\n", @$cmd_expected);
3dc780c6 WB	393	my $got = join("\n", @$cmd);
82562200 TL	394	eval { diff($exp, $got) };
	395	if (my $err = $@) {
	396	fail("$testname");
	397	note($err);
	398	} else {
	399	pass("$testname");
	400	}
7b963e57 TL	401	} else {
	402	file_set_contents($cmd_fn, $cmdline);
	403	}
	404	}
	405
	406	print "testing config to command stabillity\n";
	407
	408	# exec tests
	409	if (my $file = shift) {
	410	do_test $file;
	411	} else {
ce11958a	412	while (my $file = <cfg2cmd/*.conf>) {
7b963e57 TL	413	do_test $file;
	414	}
	415	}
	416
	417	done_testing();