Commit 5ec9a555 authored by Marcin Siodelski's avatar Marcin Siodelski

[#451,!227] Added unit tests for createBinding/createTriplet methods.

As a result of a review.
parent c737fe95
......@@ -87,7 +87,7 @@ MySqlConfigBackendImpl::~MySqlConfigBackendImpl() {
MySqlConfigBackendImpl::createBinding(const Triplet<uint32_t>& triplet) const {
MySqlConfigBackendImpl::createBinding(const Triplet<uint32_t>& triplet) {
if (triplet.unspecified()) {
return (MySqlBinding::createNull());
......@@ -95,7 +95,7 @@ MySqlConfigBackendImpl::createBinding(const Triplet<uint32_t>& triplet) const {
MySqlConfigBackendImpl::createTriplet(const MySqlBindingPtr& binding) const {
MySqlConfigBackendImpl::createTriplet(const MySqlBindingPtr& binding) {
if (!binding) {
isc_throw(Unexpected, "MySQL configuration backend internal error: "
"binding pointer is NULL when creating a triplet value");
......@@ -112,7 +112,7 @@ public:
/// @return Pointer to a null binding if the triplet is "unspecified" or
/// a pointer to a binding representing 32-bit unsigned integer value
/// otherwise.
db::MySqlBindingPtr createBinding(const Triplet<uint32_t>& triplet) const;
static db::MySqlBindingPtr createBinding(const Triplet<uint32_t>& triplet);
/// @brief Creates @c Triplet object from MySQL binding.
......@@ -123,7 +123,7 @@ public:
/// @throw isc::Unexpected if the provided binding pointer is NULL.
/// @throw isc::InvalidOperation if the binding does not represent
/// a 32-bit unsigned integer.
Triplet<uint32_t> createTriplet(const db::MySqlBindingPtr& binding) const;
static Triplet<uint32_t> createTriplet(const db::MySqlBindingPtr& binding);
/// @brief Returns server tags associated with the particular selector.
......@@ -2,7 +2,7 @@ SUBDIRS = .
AM_CPPFLAGS = -I$(top_builddir)/src/lib -I$(top_srcdir)/src/lib
AM_CPPFLAGS += -I$(top_builddir)/src/hooks/dhcp/mysql_cb -I$(top_srcdir)/src/hooks/dhcp/mysql_cb
AM_CPPFLAGS += -DINSTALL_PROG=\"$(abs_top_srcdir)/install-sh\"
......@@ -25,6 +25,7 @@ TESTS += mysql_cb_unittests
mysql_cb_unittests_SOURCES =
mysql_cb_unittests_SOURCES +=
mysql_cb_unittests_SOURCES +=
mysql_cb_unittests_SOURCES +=
// Copyright (C) 2019 Internet Systems Consortium, Inc. ("ISC")
// This Source Code Form is subject to the terms of the Mozilla Public
// License, v. 2.0. If a copy of the MPL was not distributed with this
// file, You can obtain one at
#include <config.h>
#include <mysql_cb_impl.h>
#include <gtest/gtest.h>
using namespace isc::db;
using namespace isc::dhcp;
namespace {
// Test that the MySQL binding can be created from a triplet.
TEST(MySqlConfigBackendImplTest, createBindingFromTriplet) {
// Create a binding from an unspecified triplet.
auto binding = MySqlConfigBackendImpl::createBinding(Triplet<uint32_t>());
// The binding pointer should be non-null but the type of the binding
// should be null.
// This time create a triplet encapsulating a number.
binding = MySqlConfigBackendImpl::createBinding(Triplet<uint32_t>(123));
// The binding type should be non-null.
ASSERT_EQ(MYSQL_TYPE_LONG, binding->getType());
// Check that the correct value was stored in the binding.
EXPECT_EQ(123, binding->getInteger<uint32_t>());
// Test that the triplet can be created from the MySQL binding.
TEST(MySqlConfigBackendImplTest, createTripletFromBinding) {
// Create a binding encapsulating a number and try to create a triplet
// from this binding.
MySqlBindingPtr binding = MySqlBinding::createInteger<uint32_t>(234);
Triplet<uint32_t> triplet;
ASSERT_NO_THROW(triplet = MySqlConfigBackendImpl::createTriplet(binding));
// The triplet should be specified and equal to the value stored in the
// binding.
EXPECT_EQ(234, triplet.get());
// Create a null binding and then use it to create a triplet.
binding = MySqlBinding::createNull();
ASSERT_NO_THROW(triplet = MySqlConfigBackendImpl::createTriplet(binding));
// This time the triplet should be unspecified.
// Finally, make sure that the null binding pointer is rejected.
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment