Date: Wed, 13 Apr 2016 16:28:59 +0000 (UTC) From: Alan Somers <asomers@FreeBSD.org> To: src-committers@freebsd.org, svn-src-projects@freebsd.org Subject: svn commit: r297928 - in projects/zfsd/head: etc/mtree lib/libdevdctl/tests Message-ID: <201604131628.u3DGSxYO007665@repo.freebsd.org>
next in thread | raw e-mail | index | archive | help
Author: asomers Date: Wed Apr 13 16:28:59 2016 New Revision: 297928 URL: https://svnweb.freebsd.org/changeset/base/297928 Log: Add some unit tests for the existing behavior of libdevdctl. Like zfsd's unit tests, these are written in googletest and will not be built on a system that does not have devel/googletest installed. Sponsored by: Spectra Logic Corp Added: projects/zfsd/head/lib/libdevdctl/tests/ projects/zfsd/head/lib/libdevdctl/tests/Makefile (contents, props changed) projects/zfsd/head/lib/libdevdctl/tests/libdevdctl_unittest.cc (contents, props changed) Modified: projects/zfsd/head/etc/mtree/BSD.tests.dist Modified: projects/zfsd/head/etc/mtree/BSD.tests.dist ============================================================================== --- projects/zfsd/head/etc/mtree/BSD.tests.dist Wed Apr 13 16:19:50 2016 (r297927) +++ projects/zfsd/head/etc/mtree/BSD.tests.dist Wed Apr 13 16:28:59 2016 (r297928) @@ -302,6 +302,8 @@ .. libcrypt .. + libdevdctl + .. libmp .. libnv Added: projects/zfsd/head/lib/libdevdctl/tests/Makefile ============================================================================== --- /dev/null 00:00:00 1970 (empty, because file is newly added) +++ projects/zfsd/head/lib/libdevdctl/tests/Makefile Wed Apr 13 16:28:59 2016 (r297928) @@ -0,0 +1,21 @@ +# $FreeBSD$ + +TESTSDIR= ${TESTSBASE}/lib/libdevdctl + +.PATH: ${.CURDIR}/.. + +PLAIN_TESTS_CXX= libdevdctl_unittest + +SRCS.libdevdctl_unittest+= event_factory.cc \ + libdevdctl_unittest.cc \ + event.cc exception.cc \ + guid.cc +CFLAGS.libdevdctl_unittest+= -I ${LOCALBASE}/include -D_THREAD_SAFE -pthread +DPADD.libdevdctl_unittest+= ${LIBDEVDCTL} +LDADD.libdevdctl_unittest+= -L ${LOCALBASE}/lib -D_THREAD_SAFE -pthread -lgtest -lgtest_main + +# Googletest options +LOCALBASE?= /usr/local + +WARNS?= 3 +.include <bsd.test.mk> Added: projects/zfsd/head/lib/libdevdctl/tests/libdevdctl_unittest.cc ============================================================================== --- /dev/null 00:00:00 1970 (empty, because file is newly added) +++ projects/zfsd/head/lib/libdevdctl/tests/libdevdctl_unittest.cc Wed Apr 13 16:28:59 2016 (r297928) @@ -0,0 +1,113 @@ +/*- + * Copyright (c) 2016 Spectra Logic Corporation + * All rights reserved. + * + * Redistribution and use in source and binary forms, with or without + * modification, are permitted provided that the following conditions + * are met: + * 1. Redistributions of source code must retain the above copyright + * notice, this list of conditions, and the following disclaimer, + * without modification. + * 2. Redistributions in binary form must reproduce at minimum a disclaimer + * substantially similar to the "NO WARRANTY" disclaimer below + * ("Disclaimer") and any redistribution must be conditioned upon + * including a substantially similar Disclaimer requirement for further + * binary redistribution. + * + * NO WARRANTY + * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS + * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT + * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTIBILITY AND FITNESS FOR + * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT + * HOLDERS OR CONTRIBUTORS BE LIABLE FOR SPECIAL, EXEMPLARY, OR CONSEQUENTIAL + * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS + * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) + * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, + * STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING + * IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE + * POSSIBILITY OF SUCH DAMAGES. + * + * Authors: Alan Somers (Spectra Logic Corporation) + */ + +#include <gtest/gtest.h> + +#include <list> +#include <map> +#include <string> + +#include <devdctl/guid.h> +#include <devdctl/event.h> +#include <devdctl/event_factory.h> + +using namespace DevdCtl; +using namespace std; +using namespace testing; + +#define NUM_ELEMENTS(x) (sizeof(x) / sizeof(*x)) + +class IsDiskDevTest : public TestWithParam<pair<bool, const char*> >{ +protected: + virtual void SetUp() + { + m_factory = new EventFactory(); + } + + virtual void TearDown() + { + if (m_ev) delete m_ev; + if (m_factory) delete m_factory; + } + + EventFactory *m_factory; + Event *m_ev; + static EventFactory::Record s_registry[]; +}; + +DevdCtl::EventFactory::Record IsDiskDevTest::s_registry[] = { + { Event::NOTIFY, "DEVFS", &DevfsEvent::Builder } +}; + +TEST_P(IsDiskDevTest, TestIsDiskDev) { + pair<bool, const char*> param = GetParam(); + DevfsEvent *devfs_ev; + + m_factory->UpdateRegistry(s_registry, NUM_ELEMENTS(s_registry)); + string evString(param.second); + m_ev = Event::CreateEvent(*m_factory, evString); + devfs_ev = dynamic_cast<DevfsEvent*>(m_ev); + ASSERT_NE(nullptr, devfs_ev); + EXPECT_EQ(param.first, devfs_ev->IsDiskDev()); +} + +INSTANTIATE_TEST_CASE_P(IsDiskDevTestInstantiation, IsDiskDevTest, Values( + pair<bool, const char*>(true, + "!system=DEVFS subsystem=CDEV type=CREATE cdev=da6\n"), + pair<bool, const char*>(false, + "!system=DEVFS subsystem=CDEV type=CREATE cdev=cuau0\n"), + pair<bool, const char*>(true, + "!system=DEVFS subsystem=CDEV type=CREATE cdev=ada6\n"), + pair<bool, const char*>(true, + "!system=DEVFS subsystem=CDEV type=CREATE cdev=da6p1\n"), + pair<bool, const char*>(true, + "!system=DEVFS subsystem=CDEV type=CREATE cdev=ada6p1\n"), + pair<bool, const char*>(true, + "!system=DEVFS subsystem=CDEV type=CREATE cdev=da6s0p1\n"), + pair<bool, const char*>(true, + "!system=DEVFS subsystem=CDEV type=CREATE cdev=ada6s0p1\n"), + /* + * Test physical path nodes. These are currently all set to false since + * physical path nodes are implemented with symlinks, and most CAM and + * ZFS operations can't use symlinked device nodes + */ + /* A SpectraBSD-style physical path node*/ + pair<bool, const char*>(false, + "!system=DEVFS subsystem=CDEV type=CREATE cdev=enc@50030480019f53fd/elmtype@array_device/slot@18/da\n"), + pair<bool, const char*>(false, + "!system=DEVFS subsystem=CDEV type=CREATE cdev=enc@50030480019f53fd/elmtype@array_device/slot@18/pass\n"), + /* A FreeBSD-style physical path node */ + pair<bool, const char*>(true, + "!system=DEVFS subsystem=CDEV type=CREATE cdev=enc@n50030480019f53fd/type@0/slot@18/elmdesc@ArrayDevice18/da6\n"), + pair<bool, const char*>(false, + "!system=DEVFS subsystem=CDEV type=CREATE cdev=enc@n50030480019f53fd/type@0/slot@18/elmdesc@ArrayDevice18/pass6\n")); +);
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201604131628.u3DGSxYO007665>