test: fix banlistmodel unit build breakages

Author: johnny9

qml/models/banlistmodel.h

@@ -6,6 +6,7 @@
 #define BITCOIN_QML_MODELS_BANLISTMODEL_H
 
 #include <net_types.h>
+#include <netaddress.h>
 
 #include <QAbstractListModel>
 #include <QList>

test/CMakeLists.txt

@@ -35,6 +35,7 @@ target_include_directories(bitcoinqml_unit_tests
   PRIVATE
     ${CMAKE_CURRENT_SOURCE_DIR}/..
     ${CMAKE_CURRENT_SOURCE_DIR}/../bitcoin/src
+    ${CMAKE_CURRENT_SOURCE_DIR}/../bitcoin/src/univalue/include
 )
 
 target_link_libraries(bitcoinqml_unit_tests

test/test_banlistmodel.cpp

@@ -6,15 +6,13 @@
 
 #include <QSet>
 
-#include <qml/models/banlistmodel.h>
 #include <test/mocks/mocknode.h>
+#include <qml/models/banlistmodel.h>
 #include <util/translation.h>
 
-#include <gmock/gmock.h>
 #include <netbase.h>
 
 #include <map>
-#include <optional>
 #include <stdexcept>
 #include <utility>
 
@@ -23,11 +21,11 @@ const TranslateFn G_TRANSLATION_FUN{nullptr};
 namespace {
 CSubNet ParseSubnet(const std::string& subnet)
 {
-    const std::optional<CSubNet> parsed = LookupSubNet(subnet, /*allow_lookup=*/false);
-    if (!parsed.has_value()) {
+    CSubNet parsed = LookupSubNet(subnet);
+    if (!parsed.IsValid()) {
         throw std::runtime_error("failed to parse subnet test fixture");
     }
-    return *parsed;
+    return parsed;
 }
 
 CBanEntry MakeBanEntry(int64_t ban_until)