1
0
mirror of https://github.com/fwbuilder/fwbuilder synced 2026-05-01 22:57:33 +02:00

test: FWException is not a subclass of std::exception

This commit is contained in:
Sirius Bakke 2020-04-24 18:56:55 +02:00
parent 20ba0bccc8
commit ac9e6296c8
5 changed files with 82 additions and 76 deletions

View File

@ -42,6 +42,7 @@
#include "fwbuilder/Rule.h" #include "fwbuilder/Rule.h"
#include "fwbuilder/TagService.h" #include "fwbuilder/TagService.h"
#include "fwbuilder/Constants.h" #include "fwbuilder/Constants.h"
#include "fwbuilder/FWException.h"
#include <QTest> #include <QTest>
#include <QDebug> #include <QDebug>
@ -233,8 +234,8 @@ void ImporterTest::IOSImporterTest()
try { try {
imp->run(); imp->run();
} catch (const std::exception &e) { } catch (const FWException &e) {
QFAIL(std::string("Exception thrown: ").append(e.what()).data()); QFAIL(std::string("Exception thrown: ").append(e.toString()).data());
} }
imp->finalize(); imp->finalize();
@ -263,8 +264,8 @@ void ImporterTest::IPTImporterTest()
try { try {
imp->run(); imp->run();
} catch (const std::exception &e) { } catch (const FWException &e) {
QFAIL(std::string("Exception thrown: ").append(e.what()).data()); QFAIL(std::string("Exception thrown: ").append(e.toString()).data());
} }
imp->finalize(); imp->finalize();
@ -293,8 +294,8 @@ void ImporterTest::IPTImporterNoNatTest()
try { try {
imp->run(); imp->run();
} catch (const std::exception &e) { } catch (const FWException &e) {
QFAIL(std::string("Exception thrown: ").append(e.what()).data()); QFAIL(std::string("Exception thrown: ").append(e.toString()).data());
} }
imp->finalize(); imp->finalize();
@ -334,8 +335,8 @@ void ImporterTest::IPTImporterParseVersionsTest()
try { try {
imp->run(); imp->run();
} catch (const std::exception &e) { } catch (const FWException &e) {
QFAIL(std::string("Exception thrown: ").append(e.what()).data()); QFAIL(std::string("Exception thrown: ").append(e.toString()).data());
} }
imp->finalize(); imp->finalize();

View File

@ -26,6 +26,7 @@
#include "Inet6AddrMaskTest.h" #include "Inet6AddrMaskTest.h"
#include <fwbuilder/Inet6AddrMask.h> #include <fwbuilder/Inet6AddrMask.h>
#include <fwbuilder/uint128.h> #include <fwbuilder/uint128.h>
#include "fwbuilder/FWException.h"
#include <QTest> #include <QTest>
@ -136,8 +137,8 @@ void Inet6AddrMaskTest::testStringToInetAddrExceptions()
try { try {
new InetAddr(AF_INET6, "fe80::20c:29ff:fed2:cca1/64"); new InetAddr(AF_INET6, "fe80::20c:29ff:fed2:cca1/64");
} catch (const std::exception &e) { } catch (const FWException &e) {
QFAIL(std::string("Exception thrown: ").append(e.what()).data()); QFAIL(std::string("Exception thrown: ").append(e.toString()).data());
} }
QVERIFY_EXCEPTION_THROWN(new InetAddr(AF_INET6, "fe80::20c:29ff:fed2:cca1/200"), FWException); QVERIFY_EXCEPTION_THROWN(new InetAddr(AF_INET6, "fe80::20c:29ff:fed2:cca1/200"), FWException);
@ -146,8 +147,8 @@ void Inet6AddrMaskTest::testStringToInetAddrExceptions()
try { try {
new InetAddr(AF_INET6, 64); new InetAddr(AF_INET6, 64);
} catch (const std::exception &e) { } catch (const FWException &e) {
QFAIL(std::string("Exception thrown: ").append(e.what()).data()); QFAIL(std::string("Exception thrown: ").append(e.toString()).data());
} }
QVERIFY_EXCEPTION_THROWN(new InetAddr(AF_INET6, 256), FWException); QVERIFY_EXCEPTION_THROWN(new InetAddr(AF_INET6, 256), FWException);

View File

@ -25,6 +25,8 @@
#include "InetAddrMaskTest.h" #include "InetAddrMaskTest.h"
#include <fwbuilder/InetAddrMask.h> #include <fwbuilder/InetAddrMask.h>
#include "fwbuilder/FWException.h"
#include <QTest> #include <QTest>
using namespace libfwbuilder; using namespace libfwbuilder;
@ -107,14 +109,14 @@ void InetAddrMaskTest::testStringToInetAddrExceptions()
{ {
try { try {
new InetAddr("1.2.3.4"); new InetAddr("1.2.3.4");
} catch (const std::exception &e) { } catch (const FWException &e) {
QFAIL(std::string("Exception thrown: ").append(e.what()).data()); QFAIL(std::string("Exception thrown: ").append(e.toString()).data());
} }
try { try {
new InetAddr("1.2.3.4/24"); new InetAddr("1.2.3.4/24");
} catch (const std::exception &e) { } catch (const FWException &e) {
QFAIL(std::string("Exception thrown: ").append(e.what()).data()); QFAIL(std::string("Exception thrown: ").append(e.toString()).data());
} }
QVERIFY_EXCEPTION_THROWN(new InetAddr("1.2.3.4/40"), FWException); QVERIFY_EXCEPTION_THROWN(new InetAddr("1.2.3.4/40"), FWException);
@ -127,16 +129,16 @@ void InetAddrMaskTest::testStringToInetAddrExceptions()
QVERIFY_EXCEPTION_THROWN(new InetAddr(40), FWException); QVERIFY_EXCEPTION_THROWN(new InetAddr(40), FWException);
try { try {
new InetAddr(24); new InetAddr(24);
} catch (const std::exception &e) { } catch (const FWException &e) {
QFAIL(std::string("Exception thrown: ").append(e.what()).data()); QFAIL(std::string("Exception thrown: ").append(e.toString()).data());
} }
QVERIFY_EXCEPTION_THROWN(new InetAddr((char*)(nullptr)), FWException); QVERIFY_EXCEPTION_THROWN(new InetAddr((char*)(nullptr)), FWException);
try { try {
new InetAddr(0); new InetAddr(0);
} catch (const std::exception &e) { } catch (const FWException &e) {
QFAIL(std::string("Exception thrown: ").append(e.what()).data()); QFAIL(std::string("Exception thrown: ").append(e.toString()).data());
} }
} }

View File

@ -41,6 +41,7 @@
#include "fwbuilder/Rule.h" #include "fwbuilder/Rule.h"
#include "fwbuilder/TagService.h" #include "fwbuilder/TagService.h"
#include "fwbuilder/Constants.h" #include "fwbuilder/Constants.h"
#include "fwbuilder/FWException.h"
#include <QTest> #include <QTest>
#include <QDebug> #include <QDebug>
@ -216,8 +217,8 @@ void PFImporterTest::macrosTest()
try { try {
imp->run() ; imp->run() ;
} catch (const std::exception &e) { } catch (const FWException &e) {
QFAIL(std::string("Exception thrown: ").append(e.what()).data()); QFAIL(std::string("Exception thrown: ").append(e.toString()).data());
} }
imp->finalize(); imp->finalize();
@ -241,8 +242,8 @@ void PFImporterTest::hostsMatchTest()
try { try {
imp->run() ; imp->run() ;
} catch (const std::exception &e) { } catch (const FWException &e) {
QFAIL(std::string("Exception thrown: ").append(e.what()).data()); QFAIL(std::string("Exception thrown: ").append(e.toString()).data());
} }
imp->finalize(); imp->finalize();
@ -269,8 +270,8 @@ void PFImporterTest::blockReturnTest()
try { try {
imp->run() ; imp->run() ;
} catch (const std::exception &e) { } catch (const FWException &e) {
QFAIL(std::string("Exception thrown: ").append(e.what()).data()); QFAIL(std::string("Exception thrown: ").append(e.toString()).data());
} }
imp->finalize(); imp->finalize();
@ -297,8 +298,8 @@ void PFImporterTest::icmpMatchTest()
try { try {
imp->run() ; imp->run() ;
} catch (const std::exception &e) { } catch (const FWException &e) {
QFAIL(std::string("Exception thrown: ").append(e.what()).data()); QFAIL(std::string("Exception thrown: ").append(e.toString()).data());
} }
imp->finalize(); imp->finalize();
@ -325,8 +326,8 @@ void PFImporterTest::interfaceMatchTest()
try { try {
imp->run() ; imp->run() ;
} catch (const std::exception &e) { } catch (const FWException &e) {
QFAIL(std::string("Exception thrown: ").append(e.what()).data()); QFAIL(std::string("Exception thrown: ").append(e.toString()).data());
} }
imp->finalize(); imp->finalize();
@ -353,8 +354,8 @@ void PFImporterTest::portMatchTest()
try { try {
imp->run() ; imp->run() ;
} catch (const std::exception &e) { } catch (const FWException &e) {
QFAIL(std::string("Exception thrown: ").append(e.what()).data()); QFAIL(std::string("Exception thrown: ").append(e.toString()).data());
} }
imp->finalize(); imp->finalize();
@ -381,8 +382,8 @@ void PFImporterTest::setCommandsTest()
try { try {
imp->run() ; imp->run() ;
} catch (const std::exception &e) { } catch (const FWException &e) {
QFAIL(std::string("Exception thrown: ").append(e.what()).data()); QFAIL(std::string("Exception thrown: ").append(e.toString()).data());
} }
imp->finalize(); imp->finalize();
@ -409,8 +410,8 @@ void PFImporterTest::stateMatchTest()
try { try {
imp->run() ; imp->run() ;
} catch (const std::exception &e) { } catch (const FWException &e) {
QFAIL(std::string("Exception thrown: ").append(e.what()).data()); QFAIL(std::string("Exception thrown: ").append(e.toString()).data());
} }
imp->finalize(); imp->finalize();
@ -437,8 +438,8 @@ void PFImporterTest::tcpFlagsMatchTest()
try { try {
imp->run() ; imp->run() ;
} catch (const std::exception &e) { } catch (const FWException &e) {
QFAIL(std::string("Exception thrown: ").append(e.what()).data()); QFAIL(std::string("Exception thrown: ").append(e.toString()).data());
} }
imp->finalize(); imp->finalize();
@ -465,8 +466,8 @@ void PFImporterTest::natCommands()
try { try {
imp->run() ; imp->run() ;
} catch (const std::exception &e) { } catch (const FWException &e) {
QFAIL(std::string("Exception thrown: ").append(e.what()).data()); QFAIL(std::string("Exception thrown: ").append(e.toString()).data());
} }
imp->finalize(); imp->finalize();
@ -493,8 +494,8 @@ void PFImporterTest::rdrCommands()
try { try {
imp->run() ; imp->run() ;
} catch (const std::exception &e) { } catch (const FWException &e) {
QFAIL(std::string("Exception thrown: ").append(e.what()).data()); QFAIL(std::string("Exception thrown: ").append(e.toString()).data());
} }
imp->finalize(); imp->finalize();
@ -521,8 +522,8 @@ void PFImporterTest::setTimeoutCommands()
try { try {
imp->run() ; imp->run() ;
} catch (const std::exception &e) { } catch (const FWException &e) {
QFAIL(std::string("Exception thrown: ").append(e.what()).data()); QFAIL(std::string("Exception thrown: ").append(e.toString()).data());
} }
imp->finalize(); imp->finalize();
@ -547,8 +548,8 @@ void PFImporterTest::scrubCommandsOld()
try { try {
imp->run() ; imp->run() ;
} catch (const std::exception &e) { } catch (const FWException &e) {
QFAIL(std::string("Exception thrown: ").append(e.what()).data()); QFAIL(std::string("Exception thrown: ").append(e.toString()).data());
} }
imp->finalize(); imp->finalize();
@ -570,8 +571,8 @@ void PFImporterTest::scrubCommandsNew()
try { try {
imp->run() ; imp->run() ;
} catch (const std::exception &e) { } catch (const FWException &e) {
QFAIL(std::string("Exception thrown: ").append(e.what()).data()); QFAIL(std::string("Exception thrown: ").append(e.toString()).data());
} }
imp->finalize(); imp->finalize();
@ -593,8 +594,8 @@ void PFImporterTest::tableDefinitions()
try { try {
imp->run() ; imp->run() ;
} catch (const std::exception &e) { } catch (const FWException &e) {
QFAIL(std::string("Exception thrown: ").append(e.what()).data()); QFAIL(std::string("Exception thrown: ").append(e.toString()).data());
} }
imp->finalize(); imp->finalize();
@ -619,8 +620,8 @@ void PFImporterTest::userGroupMatches()
try { try {
imp->run() ; imp->run() ;
} catch (const std::exception &e) { } catch (const FWException &e) {
QFAIL(std::string("Exception thrown: ").append(e.what()).data()); QFAIL(std::string("Exception thrown: ").append(e.toString()).data());
} }
imp->finalize(); imp->finalize();
@ -647,8 +648,8 @@ void PFImporterTest::routeToTest()
try { try {
imp->run() ; imp->run() ;
} catch (const std::exception &e) { } catch (const FWException &e) {
QFAIL(std::string("Exception thrown: ").append(e.what()).data()); QFAIL(std::string("Exception thrown: ").append(e.toString()).data());
} }
imp->finalize(); imp->finalize();
@ -672,8 +673,8 @@ void PFImporterTest::routeTo47Test()
try { try {
imp->run() ; imp->run() ;
} catch (const std::exception &e) { } catch (const FWException &e) {
QFAIL(std::string("Exception thrown: ").append(e.what()).data()); QFAIL(std::string("Exception thrown: ").append(e.toString()).data());
} }
imp->finalize(); imp->finalize();

View File

@ -41,6 +41,7 @@
#include "fwbuilder/Rule.h" #include "fwbuilder/Rule.h"
#include "fwbuilder/TagService.h" #include "fwbuilder/TagService.h"
#include "fwbuilder/Constants.h" #include "fwbuilder/Constants.h"
#include "fwbuilder/FWException.h"
#include <QTest> #include <QTest>
#include <QDebug> #include <QDebug>
@ -215,8 +216,8 @@ void PIXImporterTest::PIX_6_Test()
try { try {
imp->run(); imp->run();
} catch (const std::exception &e) { } catch (const FWException &e) {
QFAIL(std::string("Exception thrown: ").append(e.what()).data()); QFAIL(std::string("Exception thrown: ").append(e.toString()).data());
} }
imp->finalize(); imp->finalize();
@ -239,8 +240,8 @@ void PIXImporterTest::PIX_7_Test()
try { try {
imp->run(); imp->run();
} catch (const std::exception &e) { } catch (const FWException &e) {
QFAIL(std::string("Exception thrown: ").append(e.what()).data()); QFAIL(std::string("Exception thrown: ").append(e.toString()).data());
} }
imp->finalize(); imp->finalize();
@ -263,8 +264,8 @@ void PIXImporterTest::PIX_7_NAT_Test()
try { try {
imp->run(); imp->run();
} catch (const std::exception &e) { } catch (const FWException &e) {
QFAIL(std::string("Exception thrown: ").append(e.what()).data()); QFAIL(std::string("Exception thrown: ").append(e.toString()).data());
} }
imp->finalize(); imp->finalize();
@ -287,8 +288,8 @@ void PIXImporterTest::ASA_8_0_Test()
try { try {
imp->run(); imp->run();
} catch (const std::exception &e) { } catch (const FWException &e) {
QFAIL(std::string("Exception thrown: ").append(e.what()).data()); QFAIL(std::string("Exception thrown: ").append(e.toString()).data());
} }
imp->finalize(); imp->finalize();
@ -311,8 +312,8 @@ void PIXImporterTest::ASA_8_3_Test()
try { try {
imp->run(); imp->run();
} catch (const std::exception &e) { } catch (const FWException &e) {
QFAIL(std::string("Exception thrown: ").append(e.what()).data()); QFAIL(std::string("Exception thrown: ").append(e.toString()).data());
} }
imp->finalize(); imp->finalize();
@ -335,8 +336,8 @@ void PIXImporterTest::ObjectsAndGroupsTest()
try { try {
imp->run(); imp->run();
} catch (const std::exception &e) { } catch (const FWException &e) {
QFAIL(std::string("Exception thrown: ").append(e.what()).data()); QFAIL(std::string("Exception thrown: ").append(e.toString()).data());
} }
imp->finalize(); imp->finalize();
@ -362,8 +363,8 @@ void PIXImporterTest::ACLObjectsAndGroupsTest()
try { try {
imp->run(); imp->run();
} catch (const std::exception &e) { } catch (const FWException &e) {
QFAIL(std::string("Exception thrown: ").append(e.what()).data()); QFAIL(std::string("Exception thrown: ").append(e.toString()).data());
} }
imp->finalize(); imp->finalize();
@ -389,8 +390,8 @@ void PIXImporterTest::ACLTest()
try { try {
imp->run(); imp->run();
} catch (const std::exception &e) { } catch (const FWException &e) {
QFAIL(std::string("Exception thrown: ").append(e.what()).data()); QFAIL(std::string("Exception thrown: ").append(e.toString()).data());
} }
imp->finalize(); imp->finalize();
@ -413,8 +414,8 @@ void PIXImporterTest::NamesTest()
try { try {
imp->run(); imp->run();
} catch (const std::exception &e) { } catch (const FWException &e) {
QFAIL(std::string("Exception thrown: ").append(e.what()).data()); QFAIL(std::string("Exception thrown: ").append(e.toString()).data());
} }
imp->finalize(); imp->finalize();
@ -435,8 +436,8 @@ void PIXImporterTest::FWSM_4_1_Test()
try { try {
imp->run(); imp->run();
} catch (const std::exception &e) { } catch (const FWException &e) {
QFAIL(std::string("Exception thrown: ").append(e.what()).data()); QFAIL(std::string("Exception thrown: ").append(e.toString()).data());
} }
imp->finalize(); imp->finalize();