-# Copyright (c) 2006 Zmanda Inc. All Rights Reserved.
+# Copyright (c) 2007-2012 Zmanda, Inc. All Rights Reserved.
#
# This program is free software; you can redistribute it and/or modify it
# under the terms of the GNU General Public License version 2 as published
# with this program; if not, write to the Free Software Foundation, Inc.,
# 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
#
-# Contact information: Zmanda Inc, 505 N Mathlida Ave, Suite 120
-# Sunnyvale, CA 94085, USA, or: http://www.zmanda.com
+# Contact information: Zmanda Inc, 465 S. Mathilda Ave., Suite 300
+# Sunnyvale, CA 94086, USA, or: http://www.zmanda.com
-use Test::More qw(no_plan);
-use Amconfig;
+use Test::More tests => 224;
use strict;
+use warnings;
+use Data::Dumper;
use lib "@amperldir@";
+use Installcheck::Config;
use Amanda::Paths;
-use Amanda::Config qw( :init :getconf );
+use Amanda::Tests;
+use Amanda::Config qw( :init :getconf string_to_boolean amandaify_property_name );
+use Amanda::Debug;
my $testconf;
+my $config_overrides;
+
+Amanda::Debug::dbopen("installcheck");
+Installcheck::log_test_output();
+
+# utility function
+
+sub diag_config_errors {
+ my ($level, @errors) = Amanda::Config::config_errors();
+ for my $errmsg (@errors) {
+ diag $errmsg;
+ }
+}
##
# Try starting with no configuration at all
-ok(config_init(0, ''), "Initialize with no configuration");
+
+is(config_init(0, ''), $CFGERR_OK,
+ "Initialize with no configuration")
+ or diag_config_errors();
+
+config_uninit();
+$config_overrides = new_config_overrides(1);
+add_config_override($config_overrides, "tapedev", "null:TEST");
+set_config_overrides($config_overrides);
+
+is(config_init(0, undef), $CFGERR_OK,
+ "Initialize with no configuration, passing a NULL config name")
+ or diag_config_errors();
+
+is(getconf($CNF_TAPEDEV), "null:TEST",
+ "config overwrites work with null config");
+
+##
+# Check out error handling
+
+$testconf = Installcheck::Config->new();
+$testconf->add_param('label_new_tapes', '"xx"'); # a deprecated keyword -> warning
+$testconf->write();
+
+{
+ is(config_init($CONFIG_INIT_EXPLICIT_NAME, "TESTCONF"), $CFGERR_WARNINGS,
+ "Deprecated keyword generates a warning");
+ my ($error_level, @errors) = Amanda::Config::config_errors();
+ like($errors[0], qr/is deprecated/,
+ "config_get_errors returns the warning string");
+
+ Amanda::Config::config_clear_errors();
+ ($error_level, @errors) = Amanda::Config::config_errors();
+ is(scalar(@errors), 0, "config_clear_errors clears error list");
+}
+
+$testconf = Installcheck::Config->new();
+$testconf->add_param('invalid-param', 'random-value'); # a deprecated keyword -> warning
+$testconf->write();
+
+is(config_init($CONFIG_INIT_EXPLICIT_NAME, "NO-SUCH-CONFIGURATION"), $CFGERR_ERRORS,
+ "Non-existent config generates an error");
+
+is(config_init($CONFIG_INIT_EXPLICIT_NAME, "TESTCONF"), $CFGERR_ERRORS,
+ "Invalid keyword generates an error");
+
+##
+# try a client configuration
+
+# (note use of uppercase letters to test lower-casing of property names)
+$testconf = Installcheck::Config->new();
+$testconf->add_client_param('property', '"client-prop" "yep"');
+$testconf->add_client_param('property', 'priority "clIent-prop1" "foo"');
+$testconf->add_client_param('property', 'append "clieNt-prop" "bar"');
+$testconf->add_client_param('property', '"ANotHer_prOp" "baz"');
+$testconf->add_client_param('property', 'append "ANOTHER-prop" "boo"');
+$testconf->write();
+
+my $cfg_result = config_init($CONFIG_INIT_CLIENT, undef);
+is($cfg_result, $CFGERR_OK,
+ "Load test client configuration")
+ or diag_config_errors();
+
+is_deeply(getconf($CNF_PROPERTY), { "client-prop1" => { priority => 1,
+ append => 0,
+ values => [ "foo" ]},
+ "client-prop" => { priority => 0,
+ append => 1,
+ values => [ "yep", "bar" ] },
+ "another-prop" => { priority => 0,
+ append => 1,
+ values => [ "baz", "boo" ] }},
+ "Client PROPERTY parameter parsed correctly");
##
# Parse up a basic configuration
-# invent "large" values for CONFTYPE_AM64 and CONFTYPE_SIZE
-my $am64_num = '171801575472'; # 0xA000B000C000 / 1024
-my $size_t_num = '2147483647'; # 0x7fffffff
+# invent a "large" unsigned number, and make $size_t_num
+# depend on the length of size_t
+my $int64_num = '171801575472'; # 0xA000B000C000 / 1024
+my $size_t_num;
+if (Amanda::Tests::sizeof_size_t() > 4) {
+ $size_t_num = $int64_num;
+} else {
+ $size_t_num = '2147483647'; # 0x7fffffff
+}
-$testconf = Amconfig->new();
+$testconf = Installcheck::Config->new();
$testconf->add_param('reserve', '75');
$testconf->add_param('autoflush', 'yes');
+$testconf->add_param('usetimestamps', '0');
$testconf->add_param('tapedev', '"/dev/foo"');
-$testconf->add_param('bumpsize', $am64_num);
+$testconf->add_param('bumpsize', $int64_num);
$testconf->add_param('bumpmult', '1.4');
-$testconf->add_param('reserved-udp-port', '100,200');
+$testconf->add_param('reserved_udp-port', '100,200'); # note use of '-' and '_'
$testconf->add_param('device_output_buffer_size', $size_t_num);
$testconf->add_param('taperalgo', 'last');
$testconf->add_param('device_property', '"foo" "bar"');
-$testconf->add_param('device_property', '"blue" "car"');
+$testconf->add_param('device_property', '"blUE" "car" "tar"');
+$testconf->add_param('autolabel', 'non-amanda empty');
$testconf->add_param('displayunit', '"m"');
$testconf->add_param('debug_auth', '1');
$testconf->add_tapetype('mytapetype', [
'comment' => '"mine"',
'length' => '128 M',
+ 'part_size' => '100M',
+ 'part_cache_type' => 'disk',
+ 'part_cache_dir' => '"/usr/bin"',
+ 'part_cache_max_size' => '50M',
]);
-$testconf->add_dumptype('mydumptype', [
+$testconf->add_dumptype('mydump-type', [ # note dash
'comment' => '"mine"',
'priority' => 'high', # == 2
- 'bumpsize' => $am64_num,
+ 'bumpsize' => $int64_num,
'bumpmult' => 1.75,
'starttime' => 1829,
'holdingdisk' => 'required',
'include list' => '"bing" "ting"',
'include list append' => '"string" "fling"',
'include file optional' => '"rhyme"',
+ 'property' => '"prop" "erty"',
+ 'property' => '"DROP" "qwerty" "asdfg"',
+ 'estimate' => 'server calcsize client',
+ 'allow_split' => 'no',
+ 'allow_split' => 'no',
+]);
+$testconf->add_dumptype('second_dumptype', [ # note underscore
+ '' => 'mydump-type',
+ 'comment' => '"refers to mydump-type with a dash"',
]);
-$testconf->add_interface('inyoface', [
+$testconf->add_dumptype('third_dumptype', [
+ '' => 'second_dumptype',
+ 'comment' => '"refers to second_dumptype with an underscore"',
+ 'recovery-limit' => '"left" same-host "right"',
+]);
+$testconf->add_interface('ethernet', [
'comment' => '"mine"',
'use' => '100',
]);
-$testconf->add_interface('inherface', [
+$testconf->add_interface('nic', [
'comment' => '"empty"',
]);
$testconf->add_holdingdisk('hd1', [
$testconf->add_holdingdisk('hd2', [
'comment' => '"empty"',
]);
-$testconf->write();
+$testconf->add_application('my_app', [
+ 'comment' => '"my_app_comment"',
+ 'plugin' => '"amgtar"',
+]);
+$testconf->add_script('my_script', [
+ 'comment' => '"my_script_comment"',
+ 'plugin' => '"script-email"',
+ 'execute-on' => 'pre-host-backup, post-host-backup',
+ 'execute-where' => 'client',
+ 'property' => '"mailto" "amandabackup" "amanda"',
+]);
+$testconf->add_device('my_device', [
+ 'comment' => '"my device is mine, not yours"',
+ 'tapedev' => '"tape:/dev/nst0"',
+ 'device_property' => '"BLOCK_SIZE" "128k"',
+ 'device_property' => '"CoMmENT" "what up?"',
+]);
+$testconf->add_changer('my_changer', [
+ 'comment' => '"my changer is mine, not yours"',
+ 'tpchanger' => '"chg-foo"',
+ 'changerdev' => '"/dev/sg0"',
+ 'changerfile' => '"chg.state"',
+ 'property' => '"testprop" "testval"',
+ 'device_property' => '"testdprop" "testdval"',
+]);
+$testconf->add_interactivity('my_interactivity', [
+ 'comment' => '"my interactivity is mine, not yours"',
+ 'plugin' => '"MY-interactivity"',
+ 'property' => '"testprop" "testval"',
+]);
-my $cfg_ok = config_init($CONFIG_INIT_EXPLICIT_NAME, 'TESTCONF');
-ok($cfg_ok, "Load test configuration");
+$testconf->add_taperscan('my_taperscan', [
+ 'comment' => '"my taperscan is mine, not yours"',
+ 'plugin' => '"MY-taperscan"',
+ 'property' => '"testprop" "testval"',
+]);
-SKIP: {
- skip "error loading config", unless $cfg_ok;
-
- is(Amanda::Config::get_config_name(), "TESTCONF",
- "config_name set");
- is(Amanda::Config::get_config_dir(), "$CONFIG_DIR/TESTCONF",
- "config_dir set");
- is(Amanda::Config::get_config_filename(),
- "$CONFIG_DIR/TESTCONF/amanda.conf",
- "config_filename set");
-}
+$testconf->write();
-SKIP: { # global parameters
- skip "error loading config", unless $cfg_ok;
-
- is(getconf($CNF_RESERVE), 75,
- "integer global confparm");
- is(getconf($CNF_BUMPSIZE), $am64_num+0,
- "am64 global confparm");
- is(getconf($CNF_TAPEDEV), "/dev/foo",
- "string global confparm");
- is(getconf($CNF_DEVICE_OUTPUT_BUFFER_SIZE), $size_t_num+0,
- "size global confparm");
- ok(getconf($CNF_AUTOFLUSH),
- "boolean global confparm");
- is(getconf($CNF_TAPERALGO), $Amanda::Config::ALGO_LAST,
- "taperalgo global confparam");
- is_deeply([getconf($CNF_RESERVED_UDP_PORT)], [100,200],
- "intrange global confparm");
- is(getconf($CNF_DISPLAYUNIT), "M",
- "displayunit is correctly uppercased");
- is_deeply(getconf($CNF_DEVICE_PROPERTY),
- { "foo" => "bar", "blue" => "car" },
- "proplist global confparm");
-
- ok(getconf_seen($CNF_TAPEDEV),
- "'tapedev' parm was seen");
- ok(!getconf_seen($CNF_NETUSAGE),
- "'netusage' parm was not seen");
+$cfg_result = config_init($CONFIG_INIT_EXPLICIT_NAME, 'TESTCONF');
+if (!is($cfg_result, $CFGERR_OK,
+ "Load test configuration")) {
+ diag_config_errors();
+ die "aborting after config errors";
}
-SKIP: { # derived values
- skip "error loading config", unless $cfg_ok;
+is(Amanda::Config::get_config_name(), "TESTCONF",
+ "config_name set");
+is(Amanda::Config::get_config_dir(), "$CONFIG_DIR/TESTCONF",
+ "config_dir set");
+is(Amanda::Config::get_config_filename(),
+ "$CONFIG_DIR/TESTCONF/amanda.conf",
+ "config_filename set");
- is(Amanda::Config::getconf_unit_divisor(), 1024,
- "correct unit divisor (from displayunit -> KB)");
- ok($Amanda::Config::debug_auth,
- "debug_auth setting reflected in global variable");
- ok(!$Amanda::Config::debug_amandad,
- "debug_amandad defaults to false");
-}
+is(getconf($CNF_RESERVE), 75,
+ "integer global confparm");
+is(getconf($CNF_BUMPSIZE), $int64_num+0,
+ "int64 global confparm");
+is(getconf($CNF_TAPEDEV), "/dev/foo",
+ "string global confparm");
+is(getconf($CNF_DEVICE_OUTPUT_BUFFER_SIZE), $size_t_num+0,
+ "size global confparm");
+ok(getconf($CNF_AUTOFLUSH),
+ "boolean global confparm");
+is(getconf($CNF_USETIMESTAMPS), 0,
+ "boolean global confparm, passing an integer (0)");
+is(getconf($CNF_TAPERALGO), $Amanda::Config::ALGO_LAST,
+ "taperalgo global confparam");
+is_deeply([getconf($CNF_RESERVED_UDP_PORT)], [100,200],
+ "intrange global confparm");
+is(getconf($CNF_DISPLAYUNIT), "M",
+ "displayunit is correctly uppercased");
+is_deeply(getconf($CNF_DEVICE_PROPERTY),
+ { "foo" => { priority => 0, append => 0, values => ["bar"]},
+ "blue" => { priority => 0, append => 0,
+ values => ["car", "tar"]} },
+ "proplist global confparm");
+is_deeply(getconf($CNF_AUTOLABEL),
+ { template => undef, other_config => '',
+ non_amanda => 1, volume_error => '', empty => 1 },
+ "'autolabel non-amanda empty' represented correctly");
+ok(getconf_seen($CNF_TAPEDEV),
+ "'tapedev' parm was seen");
+ok(!getconf_seen($CNF_CHANGERFILE),
+ "'changerfile' parm was not seen");
-SKIP: { # tapetypes
- skip "error loading config", unless $cfg_ok;
- my $ttyp = lookup_tapetype("mytapetype");
- ok($ttyp, "found mytapetype");
- is(tapetype_getconf($ttyp, $TAPETYPE_COMMENT), 'mine',
- "tapetype comment");
- is(tapetype_getconf($ttyp, $TAPETYPE_LENGTH), 128 * 1024,
- "tapetype comment");
-
- ok(tapetype_seen($ttyp, $TAPETYPE_COMMENT),
- "tapetype comment was seen");
- ok(!tapetype_seen($ttyp, $TAPETYPE_LBL_TEMPL),
- "tapetype lbl_templ was not seen");
-
- is_deeply([ sort(+getconf_list("tapetype")) ],
- [ sort("mytapetype", "TEST-TAPE") ],
- "getconf_list lists all tapetypes");
-}
+is(Amanda::Config::getconf_unit_divisor(), 1024,
+ "correct unit divisor (from displayunit -> KB)");
+ok($Amanda::Config::debug_auth,
+ "debug_auth setting reflected in global variable");
+ok(!$Amanda::Config::debug_amandad,
+ "debug_amandad defaults to false");
-SKIP: { # dumptypes
- skip "error loading config", unless $cfg_ok;
-
- my $dtyp = lookup_dumptype("mydumptype");
- ok($dtyp, "found mydumptype");
- is(dumptype_getconf($dtyp, $DUMPTYPE_COMMENT), 'mine',
- "dumptype string");
- is(dumptype_getconf($dtyp, $DUMPTYPE_PRIORITY), 2,
- "dumptype priority");
- is(dumptype_getconf($dtyp, $DUMPTYPE_BUMPSIZE), $am64_num+0,
- "dumptype size");
- is(dumptype_getconf($dtyp, $DUMPTYPE_BUMPMULT), 1.75,
- "dumptype real");
- is(dumptype_getconf($dtyp, $DUMPTYPE_STARTTIME), 1829,
- "dumptype time");
- is(dumptype_getconf($dtyp, $DUMPTYPE_HOLDINGDISK), $HOLD_REQUIRED,
- "dumptype holdingdisk");
- is(dumptype_getconf($dtyp, $DUMPTYPE_COMPRESS), $COMP_BEST,
- "dumptype compress");
- is(dumptype_getconf($dtyp, $DUMPTYPE_ENCRYPT), $ENCRYPT_SERV_CUST,
- "dumptype encrypt");
- is(dumptype_getconf($dtyp, $DUMPTYPE_STRATEGY), $DS_INCRONLY,
- "dumptype strategy");
- is_deeply([dumptype_getconf($dtyp, $DUMPTYPE_COMPRATE)], [0.25, 0.75],
- "dumptype comprate");
- is_deeply(dumptype_getconf($dtyp, $DUMPTYPE_INCLUDE),
- { 'file' => [ 'rhyme' ],
- 'list' => [ 'bing', 'ting', 'string', 'fling' ],
- 'optional' => 1 },
- "dumptype include list");
- is_deeply(dumptype_getconf($dtyp, $DUMPTYPE_EXCLUDE),
- { 'file' => [ 'foolist' ],
- 'list' => [ 'foo', 'bar', 'true', 'star' ],
- 'optional' => 0 },
- "dumptype exclude list");
-
- ok(dumptype_seen($dtyp, $DUMPTYPE_EXCLUDE),
- "'exclude' parm was seen");
- ok(!dumptype_seen($dtyp, $DUMPTYPE_RECORD),
- "'record' parm was not seen");
-
- is_deeply([ sort(+getconf_list("dumptype")) ],
- [ sort(qw(
- mydumptype
- NO-COMPRESS COMPRESS-FAST COMPRESS-BEST COMPRESS-CUST
- SRVCOMPRESS BSD-AUTH KRB4-AUTH NO-RECORD NO-HOLD
- NO-FULL
- )) ],
- "getconf_list lists all dumptypes (including defaults)");
-}
+my $ttyp = lookup_tapetype("mytapetype");
+ok($ttyp, "found mytapetype");
+is(tapetype_getconf($ttyp, $TAPETYPE_COMMENT), 'mine',
+ "tapetype comment");
+is(tapetype_getconf($ttyp, $TAPETYPE_LENGTH), 128 * 1024,
+ "tapetype comment");
-SKIP: { # interfaces
- skip "error loading config" unless $cfg_ok;
- my $iface = lookup_interface("inyoface");
- ok($iface, "found inyoface");
- is(interface_name($iface), "inyoface",
- "interface knows its name");
- is(interface_getconf($iface, $INTER_COMMENT), 'mine',
- "interface comment");
- is(interface_getconf($iface, $INTER_MAXUSAGE), 100,
- "interface maxusage");
-
- $iface = lookup_interface("inherface");
- ok($iface, "found inherface");
- ok(interface_seen($iface, $INTER_COMMENT),
- "seen set for parameters that appeared");
- ok(!interface_seen($iface, $INTER_MAXUSAGE),
- "seen not set for parameters that did not appear");
-
- is_deeply([ sort(+getconf_list("interface")) ],
- [ sort('inyoface', 'inherface', 'default') ],
- "getconf_list lists all interfaces (in any order)");
-}
+ok(tapetype_seen($ttyp, $TAPETYPE_COMMENT),
+ "tapetype comment was seen");
+ok(!tapetype_seen($ttyp, $TAPETYPE_LBL_TEMPL),
+ "tapetype lbl_templ was not seen");
-SKIP: { # holdingdisks
- skip "error loading config" unless $cfg_ok;
- my $hdisk = lookup_holdingdisk("hd1");
- ok($hdisk, "found hd1");
- is(holdingdisk_name($hdisk), "hd1",
- "hd1 knows its name");
- is(holdingdisk_getconf($hdisk, $HOLDING_COMMENT), 'mine',
- "holdingdisk comment");
- is(holdingdisk_getconf($hdisk, $HOLDING_DISKDIR), '/mnt/hd1',
- "holdingdisk diskdir (directory)");
- is(holdingdisk_getconf($hdisk, $HOLDING_DISKSIZE), 100*1024,
- "holdingdisk disksize (use)");
- is(holdingdisk_getconf($hdisk, $HOLDING_CHUNKSIZE), 1024,
- "holdingdisk chunksize");
-
- $hdisk = lookup_holdingdisk("hd2");
- ok($hdisk, "found hd2");
- ok(holdingdisk_seen($hdisk, $HOLDING_COMMENT),
- "seen set for parameters that appeared");
- ok(!holdingdisk_seen($hdisk, $HOLDING_CHUNKSIZE),
- "seen not set for parameters that did not appear");
-
- # only holdingdisks have this linked-list structure
- # exposed
- $hdisk = getconf_holdingdisks();
- like(holdingdisk_name($hdisk), qr/hd[12]/,
- "one disk is first in list of holdingdisks");
- $hdisk = holdingdisk_next($hdisk);
- like(holdingdisk_name($hdisk), qr/hd[12]/,
- "another is second in list of holdingdisks");
- ok(!holdingdisk_next($hdisk),
- "no third holding disk");
-
- is_deeply([ sort(+getconf_list("holdingdisk")) ],
- [ sort('hd1', 'hd2') ],
- "getconf_list lists all holdingdisks (in any order)");
-}
+is(tapetype_getconf($ttyp, $TAPETYPE_PART_SIZE), 100*1024,
+ "tapetype part_size");
+is(tapetype_getconf($ttyp, $TAPETYPE_PART_CACHE_TYPE), $PART_CACHE_TYPE_DISK,
+ "tapetype part_cache_type");
+is(tapetype_getconf($ttyp, $TAPETYPE_PART_CACHE_DIR), "/usr/bin",
+ "tapetype part_cache_dir");
+is(tapetype_getconf($ttyp, $TAPETYPE_PART_CACHE_MAX_SIZE), 50*1024,
+ "tapetype part_cache_max_size");
+
+is_deeply([ sort(+getconf_list("tapetype")) ],
+ [ sort("mytapetype", "TEST-TAPE") ],
+ "getconf_list lists all tapetypes");
+
+my $dtyp = lookup_dumptype("mydump-type");
+ok($dtyp, "found mydump-type");
+is(dumptype_getconf($dtyp, $DUMPTYPE_COMMENT), 'mine',
+ "dumptype string");
+is(dumptype_getconf($dtyp, $DUMPTYPE_PRIORITY), 2,
+ "dumptype priority");
+is(dumptype_getconf($dtyp, $DUMPTYPE_BUMPSIZE), $int64_num+0,
+ "dumptype size");
+is(dumptype_getconf($dtyp, $DUMPTYPE_BUMPMULT), 1.75,
+ "dumptype real");
+is(dumptype_getconf($dtyp, $DUMPTYPE_STARTTIME), 1829,
+ "dumptype time");
+is(dumptype_getconf($dtyp, $DUMPTYPE_HOLDINGDISK), $HOLD_REQUIRED,
+ "dumptype holdingdisk");
+is(dumptype_getconf($dtyp, $DUMPTYPE_COMPRESS), $COMP_BEST,
+ "dumptype compress");
+is(dumptype_getconf($dtyp, $DUMPTYPE_ENCRYPT), $ENCRYPT_SERV_CUST,
+ "dumptype encrypt");
+is(dumptype_getconf($dtyp, $DUMPTYPE_STRATEGY), $DS_INCRONLY,
+ "dumptype strategy");
+is_deeply([dumptype_getconf($dtyp, $DUMPTYPE_COMPRATE)], [0.25, 0.75],
+ "dumptype comprate");
+is_deeply(dumptype_getconf($dtyp, $DUMPTYPE_INCLUDE),
+ { 'file' => [ 'rhyme' ],
+ 'list' => [ 'bing', 'ting', 'string', 'fling' ],
+ 'optional' => 1 },
+ "dumptype include list");
+is_deeply(dumptype_getconf($dtyp, $DUMPTYPE_EXCLUDE),
+ { 'file' => [ 'foolist' ],
+ 'list' => [ 'foo', 'bar', 'true', 'star' ],
+ 'optional' => 0 },
+ "dumptype exclude list");
+is_deeply(dumptype_getconf($dtyp, $DUMPTYPE_ESTIMATELIST),
+ [ $ES_SERVER, $ES_CALCSIZE, $ES_CLIENT ],
+ "dumptype estimate list");
+is_deeply(dumptype_getconf($dtyp, $DUMPTYPE_PROPERTY),
+ { "prop" => { priority => 0, append => 0, values => ["erty"]},
+ "drop" => { priority => 0, append => 0,
+ values => ["qwerty", "asdfg"] }},
+ "dumptype proplist");
+is_deeply(dumptype_getconf($dtyp, $DUMPTYPE_RECOVERY_LIMIT),
+ [],
+ "dumptype recovery limit with no limit specified => empty");
+
+ok(dumptype_seen($dtyp, $DUMPTYPE_EXCLUDE),
+ "'exclude' parm was seen");
+ok(!dumptype_seen($dtyp, $DUMPTYPE_RECORD),
+ "'record' parm was not seen");
+
+is_deeply([ sort(+getconf_list("dumptype")) ],
+ [ sort(qw(
+ mydump-type second_dumptype third_dumptype
+ NO-COMPRESS COMPRESS-FAST COMPRESS-BEST COMPRESS-CUST
+ SRVCOMPRESS BSD-AUTH BSDTCP-AUTH NO-RECORD NO-HOLD
+ NO-FULL
+ )) ],
+ "getconf_list lists all dumptypes (including defaults)");
+is(dumptype_getconf($dtyp, $DUMPTYPE_ALLOW_SPLIT), 0,
+ "dumptype allow_split");
+
+my $iface = lookup_interface("ethernet");
+ok($iface, "found ethernet");
+is(interface_name($iface), "ethernet",
+ "interface knows its name");
+is(interface_getconf($iface, $INTER_COMMENT), 'mine',
+ "interface comment");
+is(interface_getconf($iface, $INTER_MAXUSAGE), 100,
+ "interface maxusage");
+
+$iface = lookup_interface("nic");
+ok($iface, "found nic");
+ok(interface_seen($iface, $INTER_COMMENT),
+ "seen set for parameters that appeared");
+ok(!interface_seen($iface, $INTER_MAXUSAGE),
+ "seen not set for parameters that did not appear");
+
+is_deeply([ sort(+getconf_list("interface")) ],
+ [ sort('ethernet', 'nic', 'default') ],
+ "getconf_list lists all interfaces (in any order)");
+
+skip "error loading config", 13 unless $cfg_result == $CFGERR_OK;
+my $hdisk = lookup_holdingdisk("hd1");
+ok($hdisk, "found hd1");
+is(holdingdisk_name($hdisk), "hd1",
+ "hd1 knows its name");
+is(holdingdisk_getconf($hdisk, $HOLDING_COMMENT), 'mine',
+ "holdingdisk comment");
+is(holdingdisk_getconf($hdisk, $HOLDING_DISKDIR), '/mnt/hd1',
+ "holdingdisk diskdir (directory)");
+is(holdingdisk_getconf($hdisk, $HOLDING_DISKSIZE), 100*1024,
+ "holdingdisk disksize (use)");
+is(holdingdisk_getconf($hdisk, $HOLDING_CHUNKSIZE), 1024,
+ "holdingdisk chunksize");
+
+$hdisk = lookup_holdingdisk("hd2");
+ok($hdisk, "found hd2");
+ok(holdingdisk_seen($hdisk, $HOLDING_COMMENT),
+ "seen set for parameters that appeared");
+ok(!holdingdisk_seen($hdisk, $HOLDING_CHUNKSIZE),
+ "seen not set for parameters that did not appear");
+
+# only holdingdisks have this linked-list structure
+# exposed
+my $hdisklist = getconf($CNF_HOLDINGDISK);
+my $first_disk = @$hdisklist[0];
+$hdisk = lookup_holdingdisk($first_disk);
+like(holdingdisk_name($hdisk), qr/hd[12]/,
+ "one disk is first in list of holdingdisks");
+$hdisk = lookup_holdingdisk(@$hdisklist[1]);
+like(holdingdisk_name($hdisk), qr/hd[12]/,
+ "another is second in list of holdingdisks");
+ok($#$hdisklist == 1,
+ "no third holding disk");
+
+is_deeply([ sort(+getconf_list("holdingdisk")) ],
+ [ sort('hd1', 'hd2') ],
+ "getconf_list lists all holdingdisks (in any order)");
+
+skip "error loading config", 5 unless $cfg_result == $CFGERR_OK;
+my $app = lookup_application("my_app");
+ok($app, "found my_app");
+is(application_name($app), "my_app",
+ "my_app knows its name");
+is(application_getconf($app, $APPLICATION_COMMENT), 'my_app_comment',
+ "application comment");
+is(application_getconf($app, $APPLICATION_PLUGIN), 'amgtar',
+ "application plugin (amgtar)");
+
+is_deeply([ sort(+getconf_list("application-tool")) ],
+ [ sort("my_app") ],
+ "getconf_list lists all applications");
+# test backward compatibility
+is_deeply([ sort(+getconf_list("application")) ],
+ [ sort("my_app") ],
+ "getconf_list works for 'application-tool', too");
+
+my $sc = lookup_pp_script("my_script");
+ok($sc, "found my_script");
+is(pp_script_name($sc), "my_script",
+ "my_script knows its name");
+is(pp_script_getconf($sc, $PP_SCRIPT_COMMENT), 'my_script_comment',
+ "script comment");
+is(pp_script_getconf($sc, $PP_SCRIPT_PLUGIN), 'script-email',
+ "script plugin (script-email)");
+is(pp_script_getconf($sc, $PP_SCRIPT_EXECUTE_WHERE), $ES_CLIENT,
+ "script execute_where (client)");
+is(pp_script_getconf($sc, $PP_SCRIPT_EXECUTE_ON),
+ $EXECUTE_ON_PRE_HOST_BACKUP|$EXECUTE_ON_POST_HOST_BACKUP,
+ "script execute_on");
+
+is_deeply([ sort(+getconf_list("script")) ],
+ [ sort("my_script") ],
+ "getconf_list lists all script");
+
+is_deeply([ sort(+getconf_list("script-tool")) ],
+ [ sort("my_script") ],
+ "getconf_list works for 'script-tool', too");
+
+my $dc = lookup_device_config("my_device");
+ok($dc, "found my_device");
+is(device_config_name($dc), "my_device",
+ "my_device knows its name");
+is(device_config_getconf($dc, $DEVICE_CONFIG_COMMENT), 'my device is mine, not yours',
+ "device comment");
+is(device_config_getconf($dc, $DEVICE_CONFIG_TAPEDEV), 'tape:/dev/nst0',
+ "device tapedev");
+# TODO do we really need all of this equipment for device properties?
+is_deeply(device_config_getconf($dc, $DEVICE_CONFIG_DEVICE_PROPERTY),
+ { "block-size" => { 'priority' => 0, 'values' => ["128k"], 'append' => 0 },
+ "comment" => { 'priority' => 0, 'values' => ["what up?"], 'append' => 0 }, },
+ "device config proplist");
+
+is_deeply([ sort(+getconf_list("device")) ],
+ [ sort("my_device") ],
+ "getconf_list lists all devices");
+
+skip "error loading config", 7 unless $cfg_result == $CFGERR_OK;
+$dc = lookup_changer_config("my_changer");
+ok($dc, "found my_changer");
+is(changer_config_name($dc), "my_changer",
+ "my_changer knows its name");
+is(changer_config_getconf($dc, $CHANGER_CONFIG_COMMENT), 'my changer is mine, not yours',
+ "changer comment");
+is(changer_config_getconf($dc, $CHANGER_CONFIG_CHANGERDEV), '/dev/sg0',
+ "changer tapedev");
+is_deeply(changer_config_getconf($dc, $CHANGER_CONFIG_PROPERTY),
+ { 'testprop' => {
+ 'priority' => 0,
+ 'values' => [ 'testval' ],
+ 'append' => 0,
+ }
+ }, "changer properties represented correctly");
+
+is_deeply(changer_config_getconf($dc, $CHANGER_CONFIG_DEVICE_PROPERTY),
+ { 'testdprop' => {
+ 'priority' => 0,
+ 'values' => [ 'testdval' ],
+ 'append' => 0,
+ }
+ }, "changer device properties represented correctly");
+
+is_deeply([ sort(+getconf_list("changer")) ],
+ [ sort("my_changer") ],
+ "getconf_list lists all changers");
+
+$dc = lookup_interactivity("my_interactivity");
+ok($dc, "found my_interactivity");
+is(interactivity_name($dc), "my_interactivity",
+ "my_interactivity knows its name");
+is(interactivity_getconf($dc, $INTERACTIVITY_COMMENT), 'my interactivity is mine, not yours',
+ "interactivity comment");
+is(interactivity_getconf($dc, $INTERACTIVITY_PLUGIN), 'MY-interactivity',
+ "interactivity plugin");
+is_deeply(interactivity_getconf($dc, $INTERACTIVITY_PROPERTY),
+ { 'testprop' => {
+ 'priority' => 0,
+ 'values' => [ 'testval' ],
+ 'append' => 0,
+ }
+ }, "interactivity properties represented correctly");
+
+is_deeply([ sort(+getconf_list("interactivity")) ],
+ [ sort("my_interactivity") ],
+ "getconf_list lists all interactivity");
+
+$dc = lookup_taperscan("my_taperscan");
+ok($dc, "found my_taperscan");
+is(taperscan_name($dc), "my_taperscan",
+ "my_taperscan knows its name");
+is(taperscan_getconf($dc, $TAPERSCAN_COMMENT), 'my taperscan is mine, not yours',
+ "taperscan comment");
+is(taperscan_getconf($dc, $TAPERSCAN_PLUGIN), 'MY-taperscan',
+ "taperscan plugin");
+is_deeply(taperscan_getconf($dc, $TAPERSCAN_PROPERTY),
+ { 'testprop' => {
+ 'priority' => 0,
+ 'values' => [ 'testval' ],
+ 'append' => 0,
+ }
+ }, "taperscan properties represented correctly");
+
+is_deeply([ sort(+getconf_list("taperscan")) ],
+ [ sort("my_taperscan") ],
+ "getconf_list lists all taperscan");
+
+
+##
+# Test config overwrites (using the config from above)
+
+config_uninit();
+$config_overrides = new_config_overrides(1); # note estimate is too small
+add_config_override($config_overrides, "tapedev", "null:TEST");
+add_config_override($config_overrides, "tpchanger", "chg-test");
+add_config_override_opt($config_overrides, "org=KAOS");
+set_config_overrides($config_overrides);
+config_init($CONFIG_INIT_EXPLICIT_NAME, 'TESTCONF');
+
+is(getconf($CNF_TAPEDEV), "null:TEST",
+ "config overwrites work with real config");
+is(getconf($CNF_ORG), "KAOS",
+ "add_config_override_opt parsed correctly");
+
+# introduce an error
+config_uninit();
+$config_overrides = new_config_overrides(1);
+add_config_override($config_overrides, "bogusparam", "foo");
+set_config_overrides($config_overrides);
+config_init($CONFIG_INIT_EXPLICIT_NAME, 'TESTCONF');
+
+my ($error_level, @errors) = Amanda::Config::config_errors();
+is($error_level, $CFGERR_ERRORS, "bogus config overwrite flagged as an error");
##
# Test configuration dumping
my $pid = open(my $kid, "-|");
die "Can't fork: $!" unless defined($pid);
if (!$pid) {
- Amanda::Config::dump_configuration();
+ Amanda::Config::dump_configuration(1, 0);
exit 1;
}
-my $dump = join'', <$kid>;
+my $dump_first_line = <$kid>;
+my $dump = join'', $dump_first_line, <$kid>;
close $kid;
+waitpid $pid, 0;
my $fn = Amanda::Config::get_config_filename();
-like($dump, qr/AMANDA CONFIGURATION FROM FILE "$fn"/,
+my $dump_filename = $dump_first_line;
+chomp $dump_filename;
+$dump_filename =~ s/^# AMANDA CONFIGURATION FROM FILE "//g;
+$dump_filename =~ s/":$//g;
+is($dump_filename, $fn,
"config filename is included correctly");
-like($dump, qr/DEVICE_PROPERTY\s+"foo" "bar"\n/i,
- "DEVICE_PROPERTY appears in dump output");
+like($dump, qr/DEVICE-PROPERTY\s+"foo" "bar"\n/i,
+ "DEVICE-PROPERTY appears in dump output");
-like($dump, qr/AMRECOVER_CHECK_LABEL\s+(yes|no)/i,
- "AMRECOVER_CHECK_LABEL has a trailing space");
+like($dump, qr/AMRECOVER-CHECK-LABEL\s+(yes|no)/i,
+ "AMRECOVER-CHECK-LABEL has a trailing space");
-like($dump, qr/AMRECOVER_CHECK_LABEL\s+(yes|no)/i,
- "AMRECOVER_CHECK_LABEL has a trailing space");
+like($dump, qr/AMRECOVER-CHECK-LABEL\s+(yes|no)/i,
+ "AMRECOVER-CHECK-LABEL has a trailing space");
like($dump, qr/EXCLUDE\s+LIST "foo" "bar" "true" "star"/i,
"EXCLUDE LIST is in the dump");
"INCLUDE LIST is in the dump");
like($dump, qr/INCLUDE\s+FILE OPTIONAL "rhyme"/i,
"INCLUDE FILE is in the dump");
+like($dump, qr/RECOVERY-LIMIT.*SAME-HOST/i,
+ "RECOVERY-LIST is in the dump");
+
+##
+# Test nested definitions inside a dumptype
+
+$testconf = Installcheck::Config->new();
+$testconf->add_dumptype('nested_stuff', [
+ 'comment' => '"contains a nested application, pp_script"',
+ 'application' => '{
+ comment "my app"
+ plugin "amfun"
+}',
+ 'script' => '{
+ comment "my script"
+ plugin "ppfun"
+}',
+]);
+
+$testconf->write();
+
+$cfg_result = config_init($CONFIG_INIT_EXPLICIT_NAME, "TESTCONF");
+is($cfg_result, $CFGERR_OK,
+ "parsing nested config loaded")
+ or diag_config_errors();
+SKIP: {
+ skip "error loading config", 8 unless $cfg_result == $CFGERR_OK;
+
+ my $dtyp = lookup_dumptype("nested_stuff");
+ ok($dtyp, "found nested_stuff");
+
+ my $appname = dumptype_getconf($dtyp, $DUMPTYPE_APPLICATION);
+ like($appname, qr/^custom\(/,
+ "DUMPTYPE_APPLICATION is the generated name of an application subsection");
+
+ my $app = lookup_application($appname);
+ ok($app, ".. and that name leads to an application object");
+ is(application_getconf($app, $APPLICATION_COMMENT), "my app",
+ ".. that has the right comment");
+
+ my $sc = dumptype_getconf($dtyp, $DUMPTYPE_SCRIPTLIST);
+ ok(ref($sc) eq 'ARRAY' && @$sc == 1, "DUMPTYPE_SCRIPTLIST returns a 1-element list");
+ like($sc->[0], qr/^custom\(/,
+ ".. and the first element is the generated name of a script subsection");
+
+ $sc = lookup_pp_script($sc->[0]);
+ ok($sc, ".. and that name leads to a pp_script object");
+ is(pp_script_getconf($sc, $PP_SCRIPT_COMMENT), "my script",
+ ".. that has the right comment");
+}
##
# Explore a quirk of exinclude parsing. Only the last
# We may want to change this, but we should do so intentionally.
# This is also tested by the 'amgetconf' installcheck.
-$testconf = Amconfig->new();
-$testconf->add_dumptype('mydumptype', [
+$testconf = Installcheck::Config->new();
+$testconf->add_dumptype('mydump-type', [
'exclude list' => '"foo" "bar"',
'exclude list optional append' => '"true" "star"',
'exclude list append' => '"true" "star"',
]);
$testconf->write();
-$cfg_ok = config_init($CONFIG_INIT_EXPLICIT_NAME, "TESTCONF");
+$cfg_result = config_init($CONFIG_INIT_EXPLICIT_NAME, "TESTCONF");
+is($cfg_result, $CFGERR_OK,
+ "first exinclude parsing config loaded")
+ or diag_config_errors();
SKIP: {
- skip "error loading config", unless $cfg_ok;
+ skip "error loading config", 2 unless $cfg_result == $CFGERR_OK;
- my $dtyp = lookup_dumptype("mydumptype");
- ok($dtyp, "found mydumptype");
+ my $dtyp = lookup_dumptype("mydump-type");
+ ok($dtyp, "found mydump-type");
is(dumptype_getconf($dtyp, $DUMPTYPE_EXCLUDE)->{'optional'}, 0,
"'optional' has no effect when not on the last occurrence");
}
-$testconf = Amconfig->new();
-$testconf->add_dumptype('mydumptype', [
- 'exclude file' => '"foo" "bar"',
- 'exclude file optional append' => '"true" "star"',
- 'exclude list append' => '"true" "star"',
+##
+# Check out recovery-limit parsing
+
+$testconf = Installcheck::Config->new();
+$testconf->add_param('recovery-limit', '"foo" "bar"');
+$testconf->add_dumptype('rl1', [
+ 'recovery-limit' => 'same-host server',
+]);
+$testconf->add_dumptype('rl2', [
+ 'recovery-limit' => '"somehost"',
+]);
+$testconf->add_dumptype('rl3', [
+ 'recovery-limit' => 'same-host server "somehost"',
+]);
+$testconf->add_dumptype('rl4', [
+ 'recovery-limit' => '"foohost" same-host',
]);
$testconf->write();
-$cfg_ok = config_init($CONFIG_INIT_EXPLICIT_NAME, "TESTCONF");
+$cfg_result = config_init($CONFIG_INIT_EXPLICIT_NAME, "TESTCONF");
+is($cfg_result, $CFGERR_OK,
+ "recovery-limit config loaded")
+ or diag_config_errors();
SKIP: {
- skip "error loading config", unless $cfg_ok;
+ skip "error loading config", 5 unless $cfg_result == $CFGERR_OK;
+ my $dtyp;
- my $dtyp = lookup_dumptype("mydumptype");
- ok($dtyp, "found mydumptype");
- is(dumptype_getconf($dtyp, $DUMPTYPE_EXCLUDE)->{'optional'}, 0,
- "'optional' has no effect when not on the last occurrence of 'file'");
+ is_deeply(getconf($CNF_RECOVERY_LIMIT),
+ [ 'foo', 'bar' ],
+ "global recovery-limit parameter");
+
+ $dtyp = lookup_dumptype("rl1");
+ is_deeply(dumptype_getconf($dtyp, $DUMPTYPE_RECOVERY_LIMIT),
+ [ "SAMEHOST-SAMEHOST-SAMEHOST", "SERVER-SERVER-SERVER" ],
+ "same-host => undef in list");
+
+ $dtyp = lookup_dumptype("rl2");
+ is_deeply(dumptype_getconf($dtyp, $DUMPTYPE_RECOVERY_LIMIT),
+ [ "somehost" ],
+ "hostname => match pattern");
+
+ $dtyp = lookup_dumptype("rl3");
+ is_deeply(dumptype_getconf($dtyp, $DUMPTYPE_RECOVERY_LIMIT),
+ [ "SAMEHOST-SAMEHOST-SAMEHOST", "SERVER-SERVER-SERVER", "somehost" ],
+ "hostname and same-host parsed correctly");
+
+ $dtyp = lookup_dumptype("rl4");
+ is_deeply(dumptype_getconf($dtyp, $DUMPTYPE_RECOVERY_LIMIT),
+ [ "SAMEHOST-SAMEHOST-SAMEHOST", "foohost" ], # note that the order is an implementation detail
+ ".. even if same-host comes last");
+}
+
+##
+# Check out dump-limit parsing
+
+$testconf = Installcheck::Config->new();
+$testconf->add_dumptype('dl1', [
+ 'dump-limit' => 'same-host',
+]);
+$testconf->add_dumptype('dl2', [
+ 'dump-limit' => 'server',
+]);
+$testconf->add_dumptype('dl3', [
+ 'dump-limit' => 'same-host server',
+]);
+$testconf->write();
+
+$cfg_result = config_init($CONFIG_INIT_EXPLICIT_NAME, "TESTCONF");
+is($cfg_result, $CFGERR_OK,
+ "dump-limit config loaded")
+ or diag_config_errors();
+SKIP: {
+ skip "error loading config", 5 unless $cfg_result == $CFGERR_OK;
+ my $dtyp;
+
+ $dtyp = lookup_dumptype("dl1");
+ is_deeply(dumptype_getconf($dtyp, $DUMPTYPE_DUMP_LIMIT),
+ [ "SAMEHOST-SAMEHOST-SAMEHOST" ],
+ "same-host => \"SAMEHOST-SAMEHOST-SAMEHOST\" in list");
+
+ $dtyp = lookup_dumptype("dl2");
+ is_deeply(dumptype_getconf($dtyp, $DUMPTYPE_DUMP_LIMIT),
+ [ "SERVER-SERVER-SERVER" ],
+ "server => \"SERVER-SERVER-SERVER\" in list");
+
+ $dtyp = lookup_dumptype("dl3");
+ is_deeply(dumptype_getconf($dtyp, $DUMPTYPE_DUMP_LIMIT),
+ [ "SAMEHOST-SAMEHOST-SAMEHOST", "SERVER-SERVER-SERVER" ],
+ "same-host and server");
+}
+
+$testconf->add_dumptype('dl4', [
+ 'dump-limit' => 'same-host server "somehost"',
+]);
+$testconf->write();
+$cfg_result = config_init($CONFIG_INIT_EXPLICIT_NAME, "TESTCONF");
+isnt($cfg_result, $CFGERR_OK,
+ "dump-limit do not accept hostname");
+
+##
+# Try an autolabel with a template and 'any'
+
+$testconf = Installcheck::Config->new();
+$testconf->add_param('autolabel', '"FOO%%%BAR" any');
+$testconf->write();
+
+$cfg_result = config_init($CONFIG_INIT_EXPLICIT_NAME, "TESTCONF");
+is($cfg_result, $CFGERR_OK,
+ "first exinclude parsing config loaded")
+ or diag_config_errors();
+SKIP: {
+ skip "error loading config", 1 unless $cfg_result == $CFGERR_OK;
+ is_deeply(getconf($CNF_AUTOLABEL),
+ { template => "FOO%%%BAR", other_config => 1,
+ non_amanda => 1, volume_error => 1, empty => 1 },
+ "'autolabel \"FOO%%%BAR\" any' represented correctly");
+}
+
+##
+# Check out where quoting is and is not required.
+
+$testconf = Installcheck::Config->new();
+
+# make sure an unquoted tapetype is OK
+$testconf->add_param('tapetype', 'TEST-TAPE'); # unquoted (Installcheck::Config uses quoted)
+
+# strings can optionally be quoted
+$testconf->add_param('dumporder', '"STSTST"');
+
+# enumerations (e.g., taperalgo) must not be quoted; implicitly tested above
+
+# definitions
+$testconf->add_dumptype('"parent"', [ # note quotes
+ 'bumpsize' => '10240',
+]);
+$testconf->add_dumptype('child', [
+ '' => '"parent"', # note quotes
+]);
+$testconf->add_dumptype('child2', [
+ '' => 'parent',
+]);
+$testconf->write();
+
+$cfg_result = config_init($CONFIG_INIT_EXPLICIT_NAME, "TESTCONF");
+is($cfg_result, $CFGERR_OK,
+ "parsed config to test strings vs. identifiers")
+ or diag_config_errors();
+SKIP: {
+ skip "error loading config", 3 unless $cfg_result == $CFGERR_OK;
+
+ my $dtyp = lookup_dumptype("parent");
+ ok($dtyp, "found parent");
+ $dtyp = lookup_dumptype("child");
+ ok($dtyp, "found child");
+ is(dumptype_getconf($dtyp, $DUMPTYPE_BUMPSIZE), 10240,
+ "child dumptype correctly inherited bumpsize");
}
-# TODO:
-# overwrites
-# inheritance
-# more init
+##
+# Explore a quirk of read_int_or_str parsing.
+
+$testconf = Installcheck::Config->new();
+$testconf->add_dumptype('mydump-type1', [
+ 'client_port' => '12345',
+]);
+$testconf->add_dumptype('mydump-type2', [
+ 'client_port' => '"newamanda"',
+]);
+$testconf->add_dumptype('mydump-type3', [
+ 'client_port' => '"67890"',
+]);
+$testconf->write();
+
+$cfg_result = config_init($CONFIG_INIT_EXPLICIT_NAME, "TESTCONF");
+is($cfg_result, $CFGERR_OK,
+ "read_int_or_str parsing config loaded")
+ or diag_config_errors();
+SKIP: {
+ skip "error loading config", 6 unless $cfg_result == $CFGERR_OK;
+
+ my $dtyp = lookup_dumptype("mydump-type1");
+ ok($dtyp, "found mydump-type1");
+ is(dumptype_getconf($dtyp, $DUMPTYPE_CLIENT_PORT), "12345",
+ "client_port set to 12345");
+
+ $dtyp = lookup_dumptype("mydump-type2");
+ ok($dtyp, "found mydump-type1");
+ is(dumptype_getconf($dtyp, $DUMPTYPE_CLIENT_PORT), "newamanda",
+ "client_port set to \"newamanda\"");
+
+ $dtyp = lookup_dumptype("mydump-type3");
+ ok($dtyp, "found mydump-type1");
+ is(dumptype_getconf($dtyp, $DUMPTYPE_CLIENT_PORT), "67890",
+ "client_port set to \"67890\"");
+}
+
+##
+# Check property inheritance
+
+$testconf = Installcheck::Config->new();
+$testconf->add_application('app1', [
+ 'property' => '"prop1" "val1"'
+]);
+$testconf->add_application('app2', [
+ 'property' => 'append "prop2" "val2"'
+]);
+$testconf->add_application('app3', [
+ 'property' => '"prop3" "val3"'
+]);
+$testconf->add_application('app1a', [
+ 'property' => '"prop4" "val4"',
+ 'property' => '"prop1" "val1a"',
+ 'app1' => undef
+]);
+$testconf->add_application('app2a', [
+ 'property' => '"prop5" "val5"',
+ 'property' => '"prop2" "val2a"',
+ 'app2' => undef
+]);
+$testconf->add_application('app3a', [
+ 'property' => '"prop6" "val6"',
+ 'app3' => undef,
+ 'property' => '"prop7" "val7"'
+]);
+$testconf->add_application('app1b', [
+ 'property' => '"prop4" "val4"',
+ 'property' => '"prop1" "val1a"',
+ 'app1' => undef,
+ 'property' => '"prop1" "val1b"',
+]);
+$testconf->add_application('app2b', [
+ 'property' => '"prop5" "val5"',
+ 'property' => '"prop2" "val2a"',
+ 'app2' => undef,
+ 'property' => 'append "prop2" "val2b"',
+]);
+$testconf->write();
+
+$cfg_result = config_init($CONFIG_INIT_EXPLICIT_NAME, "TESTCONF");
+is($cfg_result, $CFGERR_OK,
+ "application properties inheritance")
+ or diag_config_errors();
+SKIP: {
+ skip "error loading config", 15 unless $cfg_result == $CFGERR_OK;
+
+ my $app = lookup_application("app1a");
+ ok($app, "found app1a");
+ is(application_name($app), "app1a",
+ "app1a knows its name");
+ my $prop = application_getconf($app, $APPLICATION_PROPERTY);
+ is_deeply($prop, { "prop4" => { priority => 0,
+ append => 0,
+ values => [ "val4" ]},
+ "prop1" => { priority => 0,
+ append => 0,
+ values => [ "val1" ] }},
+ "PROPERTY parameter of app1a parsed correctly");
+
+ $app = lookup_application("app2a");
+ ok($app, "found app2a");
+ is(application_name($app), "app2a",
+ "app2a knows its name");
+ $prop = application_getconf($app, $APPLICATION_PROPERTY);
+ is_deeply($prop, { "prop5" => { priority => 0,
+ append => 0,
+ values => [ "val5" ]},
+ "prop2" => { priority => 0,
+ append => 0,
+ values => [ "val2a", "val2" ] }},
+ "PROPERTY parameter of app2a parsed correctly");
+
+ $app = lookup_application("app3a");
+ ok($app, "found app3a");
+ is(application_name($app), "app3a",
+ "app3a knows its name");
+ $prop = application_getconf($app, $APPLICATION_PROPERTY);
+ is_deeply($prop, { "prop3" => { priority => 0,
+ append => 0,
+ values => [ "val3" ]},
+ "prop6" => { priority => 0,
+ append => 0,
+ values => [ "val6" ] },
+ "prop7" => { priority => 0,
+ append => 0,
+ values => [ "val7" ] }},
+ "PROPERTY parameter of app3a parsed correctly");
+
+ $app = lookup_application("app1b");
+ ok($app, "found app1b");
+ is(application_name($app), "app1b",
+ "app1b knows its name");
+ $prop = application_getconf($app, $APPLICATION_PROPERTY);
+ is_deeply($prop, { "prop4" => { priority => 0,
+ append => 0,
+ values => [ "val4" ]},
+ "prop1" => { priority => 0,
+ append => 0,
+ values => [ "val1b" ] }},
+ "PROPERTY parameter of app1b parsed correctly");
+
+ $app = lookup_application("app2b");
+ ok($app, "found app2b");
+ is(application_name($app), "app2b",
+ "app2b knows its name");
+ $prop = application_getconf($app, $APPLICATION_PROPERTY);
+ is_deeply($prop, { "prop5" => { priority => 0,
+ append => 0,
+ values => [ "val5" ]},
+ "prop2" => { priority => 0,
+ append => 1,
+ values => [ "val2a", "val2", "val2b" ] }},
+ "PROPERTY parameter of app2b parsed correctly");
+}
+
+
+##
+# Check getconf_byname and getconf_byname_strs
+
+$testconf = Installcheck::Config->new();
+$testconf->add_param('tapedev', '"thats a funny name"');
+$testconf->add_application('app1', [
+ 'comment' => '"one"',
+]);
+$testconf->add_script('scr1', [
+ 'comment' => '"one"',
+]);
+# check old names, too
+$testconf->add_text(<<EOF);
+define application-tool "app2" {
+ comment "two"
+}
+EOF
+$testconf->add_text(<<EOF);
+define script-tool "scr2" {
+ comment "two"
+}
+EOF
+$testconf->write();
+
+$cfg_result = config_init($CONFIG_INIT_EXPLICIT_NAME, "TESTCONF");
+is($cfg_result, $CFGERR_OK,
+ "getconf_byname")
+ or diag_config_errors();
+SKIP: {
+ skip "error loading config", 7 unless $cfg_result == $CFGERR_OK;
+
+ is(getconf_byname("Tapedev"), "thats a funny name",
+ "getconf_byname for global param");
+ is_deeply([ getconf_byname_strs("Tapedev", 1) ],
+ [ "\"thats a funny name\"" ],
+ "getconf_byname_strs for global param with quotes");
+ is_deeply([ getconf_byname_strs("Tapedev", 0) ],
+ [ "thats a funny name" ],
+ "getconf_byname_strs for global param without quotes");
+
+ # test * and *-tool (the old name)
+ is(getconf_byname("application-tool:app1:comment"), "one",
+ "getconf_byname for appplication-tool param");
+ is(getconf_byname("application:app2:comment"), "two",
+ "getconf_byname for application param");
+ is(getconf_byname("script-tool:scr1:comment"), "one",
+ "getconf_byname for appplication-tool param");
+ is(getconf_byname("script:scr2:comment"), "two",
+ "getconf_byname for script param");
+}
+
+my @boolean_vals = (
+ {'val' => '1', 'expected' => 1},
+ {'val' => '0', 'expected' => 0},
+ {'val' => 't', 'expected' => 1},
+ {'val' => 'true', 'expected' => 1},
+ {'val' => 'f', 'expected' => 0},
+ {'val' => 'false', 'expected' => 0},
+ {'val' => 'y', 'expected' => 1},
+ {'val' => 'yes', 'expected' => 1},
+ {'val' => 'n', 'expected' => 0},
+ {'val' => 'no', 'expected' => 0},
+ {'val' => 'on', 'expected' => 1},
+ {'val' => 'off', 'expected' => 0},
+ {'val' => 'oFf', 'expected' => 0},
+ {'val' => 'foo', 'expected' => undef},
+ );
+
+for my $bv (@boolean_vals) {
+ is(string_to_boolean($bv->{'val'}), $bv->{'expected'},
+ "string_to_boolean('$bv->{'val'}') is right");
+}
+
+my @prop_names = (
+ {'val' => '', 'expected' => ''},
+ {'val' => 'prop-name', 'expected' => 'prop-name'},
+ {'val' => 'PRoP-NaME', 'expected' => 'prop-name'},
+ {'val' => 'prop_name', 'expected' => 'prop-name'},
+ {'val' => 'FaNCy_ProP', 'expected' => 'fancy-prop'},
+ {'val' => '_under_', 'expected' => '-under-'},
+ {'val' => '-dash-', 'expected' => '-dash-'},
+ {'val' => '-', 'expected' => '-'},
+ {'val' => '_', 'expected' => '-'},
+ );
+
+for my $pn (@prop_names) {
+ is(amandaify_property_name($pn->{'val'}), $pn->{'expected'},
+ "amandaify_property_name('$pn->{'val'}') is right");
+}
+
+$testconf = Installcheck::Config->new();
+$testconf->add_param('property', '"PrOP_nAme" "VALUE"');
+$testconf->write();
+config_init($CONFIG_INIT_EXPLICIT_NAME, "TESTCONF");
+my $properties = getconf($CNF_PROPERTY);
+
+@prop_names = (
+ {'val' => 'prop-name'},
+ {'val' => 'PRoP-NaME'},
+ {'val' => 'prop_name'},
+ {'val' => 'PROP_NAME'},
+ {'val' => 'PRoP-NaME'},
+ {'val' => 'prop_name'},
+ );
+
+for my $pn (@prop_names) {
+ is_deeply($properties->{$pn->{'val'}}->{values}, [ "VALUE" ]);
+}
+
+$testconf = Installcheck::Config->new();
+$testconf->add_client_config_param('amdump-server', '"amdump.localhost"');
+$testconf->add_client_config_param('index-server', '"index.localhost"');
+$testconf->add_client_config_param('tape-server', '"tape.localhost"');
+$testconf->write();
+config_init($CONFIG_INIT_CLIENT | $CONFIG_INIT_EXPLICIT_NAME, "TESTCONF");
+my $amdump_server = getconf($CNF_AMDUMP_SERVER);
+is ($amdump_server, "amdump.localhost", "amdump-server is \"amdump.localhost\"");
+my $index_server = getconf($CNF_INDEX_SERVER);
+is ($index_server, "index.localhost", "index-server is \"index.localhost\"");
+my $tape_server = getconf($CNF_TAPE_SERVER);
+is ($tape_server, "tape.localhost", "amdump is \"tape.localhost\"");
+
projects / debian / amanda / blobdiff