_get_foreign_key_setting() takes an optional DB handle

This fixes test failures with newer DBD::SQLite where it would die with "database is locked"

Change-Id: Ic636691af42b2462be4a53d2e236cbc2a1b9d3ce
Add XML::Dumper as an optional prerequisite

Change-Id: Id66a138da8900c8d7baba6a9b9e950ea61e80b8e
avoid an uninitialized value warning from $type

Change-Id: I8506b2c6a56443eef7f6aaf962ff632f81351420
avoid an uninitialized value warning with $value->{operator}

Change-Id: I2fb31ab14f0c7bd1d38a0de351872f751e3db25a
avoid uninitialized value warning with members of @$val

Change-Id: Ie9d9edee53c343b0576e3278854a8b7243f5b545
extract function to tell whether an element of the list is a null value

also avoids an uninitialized value warning with elements of the list

Change-Id: If99d404cd80ee0276cff821d384dedac6a5ff30c
use the default handle instead of making a new one

Avoids a "database is locked" error in newer DBD::SQLite

Change-Id: I43d22d11e74ed0b2c8c718a4625b4323a6113e64
put quotes around empty strings, too

Change-Id: I83e3ef6a1a01892c814e2cf0da36e8bd088cd948
Don't overwrite URT's metaDB dump file in the repo when 98_ur_update.t runs

Previous changes are making the test overwrite
t/URT/DataSource/Meta.sqlite3-dump and cause the test to fail.  This makes
the MetaDB dump get copied to /tmp when the test starts, and read/write
to this copy instead of the file in the repo.

Change-Id: I8cd3eda1707f3e4a135cfec5e04306937333471d
avoid an uninitialized value warning later when $constraint_name is used as a hash key

Change-Id: Ic0a38ff86dd01c8240779e60ad9c8ba075601baa
avoid a warning about re-declared my variables $class and $params

Change-Id: I007f9651d7b11e1d9af81e84f173f7e1949de2a7
remove warning "test_hashref() called too early to check prototype"

Change-Id: Ifa7188fd4beb3ac2acb192b54160d98c1de8fc71
fixup with 552c8ad0b21f61ef3e97d95eb9689d1754d58af3

Change-Id: I6ab3c1f798c890d58ce261b1fa32e24a0e67a5ae
suppress expected error message about datasource failing to sync

Change-Id: I54b3691c35a2ce2e23badfe0572ff6e5d036eadd
avoid uninitialized value warnings in 04_rdbms_retriable_operation.t

Change-Id: I30900bb8745b82476894719d53102b806db58962
Sort the BoolExpr's param keys when displaying

94b_flatten_reframe.t stringifies BXs to see if two are the same.  Since
newer Perls return hash keys in random order, we need to sort them to
ensure two BXs with the same params and values will stringify the same.

Change-Id: Idca10b6ddd4653107bbea3bac2f6fb4d81414940
avoid an uninitialized value warning inside _create_data_file

_create_data_file() is called before $files_written was getting initialized to 0

Change-Id: I861a6edafd348e03a5df738dadaca6cdcb1a1afd
Extract function to return the proper order property definition keys should be processed in

This fixes 84b_implied_properties.t by ensuring the 'has' section is always
processed before has_optional

Change-Id: Ifa3baf2c3da24d83c88a39d9d61e88e811b13ae6
Switch to id comparison instead of stringify for more reliable results

Change-Id: I6b85d357ddc16a40fa06335bb3682154223bc809
Remove unnecessary cruft

This test was apparently copied from another that did DB access.  This test
doesn't require any database to function.

Change-Id: I569ff2f61cb3a39bd56d58e1625fa98a717229fd