db/pgbackrest
1
0
Fork 0
mirror of https://github.com/pgbackrest/pgbackrest.git synced 2025-11-06 08:49:29 +02:00
Code Issues Releases Activity

Remove extraneous calls to cfgOptionValid() in unit tests.

Browse Source
This commit is contained in:
David Steele
2020-10-19 14:19:28 -04:00
parent 7d069a2b91
commit 303b9b17e7
2 changed files with 0 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
test/src/module/command/repoTest.c
View File

@@ -138,7 +138,6 @@ testRun(void)
// Recurse // Recurse
// ------------------------------------------------------------------------------------------------------------------------- // -------------------------------------------------------------------------------------------------------------------------
cfgOptionValidSet(cfgOptRecurse, true);
cfgOptionSet(cfgOptRecurse, cfgSourceParam, VARBOOL(true)); cfgOptionSet(cfgOptRecurse, cfgSourceParam, VARBOOL(true));
output = bufNew(0); output = bufNew(0);
@@ -148,7 +147,6 @@ testRun(void)
// Filter // Filter
// ------------------------------------------------------------------------------------------------------------------------- // -------------------------------------------------------------------------------------------------------------------------
cfgOptionValidSet(cfgOptFilter, true);
cfgOptionSet(cfgOptFilter, cfgSourceParam, VARSTRDEF("^aaa$")); cfgOptionSet(cfgOptFilter, cfgSourceParam, VARSTRDEF("^aaa$"));
output = bufNew(0); output = bufNew(0);

4
test/src/module/storage/remoteTest.c
View File

@@ -36,13 +36,10 @@ testRun(void)
// Set type since we'll be running local and remote tests here // Set type since we'll be running local and remote tests here
cfgOptionSet(cfgOptRemoteType, cfgSourceParam, VARSTRDEF("repo")); cfgOptionSet(cfgOptRemoteType, cfgSourceParam, VARSTRDEF("repo"));
cfgOptionValidSet(cfgOptRemoteType, true);
// Set pg settings so we can run both db and backup remotes // Set pg settings so we can run both db and backup remotes
cfgOptionSet(cfgOptPgHost, cfgSourceParam, VARSTRDEF("localhost")); cfgOptionSet(cfgOptPgHost, cfgSourceParam, VARSTRDEF("localhost"));
cfgOptionValidSet(cfgOptPgHost, true);
cfgOptionSet(cfgOptPgPath, cfgSourceParam, VARSTR(strNewFmt("%s/pg", testPath()))); cfgOptionSet(cfgOptPgPath, cfgSourceParam, VARSTR(strNewFmt("%s/pg", testPath())));
cfgOptionValidSet(cfgOptPgPath, true);
// Start a protocol server to test the remote protocol // Start a protocol server to test the remote protocol
Buffer *serverRead = bufNew(8192); Buffer *serverRead = bufNew(8192);
@@ -215,7 +212,6 @@ testRun(void)
// Do these tests against pg for coverage. We're not really going to get a pg remote here because the remote for this host // Do these tests against pg for coverage. We're not really going to get a pg remote here because the remote for this host
// id has already been created. This will just test that the pg storage is selected to provide coverage. // id has already been created. This will just test that the pg storage is selected to provide coverage.
cfgOptionSet(cfgOptRemoteType, cfgSourceParam, VARSTRDEF("pg")); cfgOptionSet(cfgOptRemoteType, cfgSourceParam, VARSTRDEF("pg"));
cfgOptionValidSet(cfgOptRemoteType, true);
paramList = varLstNew(); paramList = varLstNew();
varLstAdd(paramList, varNewStrZ(hrnReplaceKey("{[path]}/repo/test"))); varLstAdd(paramList, varNewStrZ(hrnReplaceKey("{[path]}/repo/test")));