svn commit: r315320 - in head: etc/mtree lib/libcam lib/libcam/tests
Ngie Cooper
ngie at FreeBSD.org
Wed Mar 15 18:00:56 UTC 2017
Author: ngie
Date: Wed Mar 15 18:00:54 2017
New Revision: 315320
URL: https://svnweb.freebsd.org/changeset/base/315320
Log:
Start adding basic tests for cam(3)
This change contains several negative and positive tests for:
- cam_open_device
- cam_close_device
- cam_getccb
- cam_freeccb
This also contains a test for the failure case noted in bug 217649,
i.e., O_RDWR must be specified because pass(4) requires it.
This test unfortunately cannot assume that cam-capable devices are
present, so the user must explicitly provide a device via
`test_suites.FreeBSD.cam_test_device`. In the future, a test kernel
module might be shipped, or ctl(4) might be used, as a test device
when testing out libcam, which will allow the tests to do away with
having to specify an explicit test device.
Reviewed by: asomers, ken (earlier diff)
MFC after: 1 week
Sponsored by: Dell EMC Isilon
Differential Revision: D9928
Added:
head/lib/libcam/tests/
head/lib/libcam/tests/Makefile (contents, props changed)
head/lib/libcam/tests/libcam_test.c (contents, props changed)
Modified:
head/etc/mtree/BSD.tests.dist
head/lib/libcam/Makefile
Modified: head/etc/mtree/BSD.tests.dist
==============================================================================
--- head/etc/mtree/BSD.tests.dist Wed Mar 15 17:54:47 2017 (r315319)
+++ head/etc/mtree/BSD.tests.dist Wed Mar 15 18:00:54 2017 (r315320)
@@ -310,6 +310,8 @@
ttyio
..
..
+ libcam
+ ..
libcasper
services
cap_dns
Modified: head/lib/libcam/Makefile
==============================================================================
--- head/lib/libcam/Makefile Wed Mar 15 17:54:47 2017 (r315319)
+++ head/lib/libcam/Makefile Wed Mar 15 18:00:54 2017 (r315320)
@@ -43,4 +43,8 @@ CFLAGS+= -I${.CURDIR} -I${SRCTOP}/sys
SHLIB_MAJOR= 7
+.include <src.opts.mk>
+
+SUBDIR.${MK_TESTS}+= tests
+
.include <bsd.lib.mk>
Added: head/lib/libcam/tests/Makefile
==============================================================================
--- /dev/null 00:00:00 1970 (empty, because file is newly added)
+++ head/lib/libcam/tests/Makefile Wed Mar 15 18:00:54 2017 (r315320)
@@ -0,0 +1,9 @@
+# $FreeBSD$
+
+ATF_TESTS_C+= libcam_test
+
+LIBADD+= cam
+
+WARNS?= 6
+
+.include <bsd.test.mk>
Added: head/lib/libcam/tests/libcam_test.c
==============================================================================
--- /dev/null 00:00:00 1970 (empty, because file is newly added)
+++ head/lib/libcam/tests/libcam_test.c Wed Mar 15 18:00:54 2017 (r315320)
@@ -0,0 +1,217 @@
+/*-
+ * Copyright (c) 2017 Ngie Cooper <ngie at freebsd.org>
+ * 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.
+ * 2. Redistributions in binary form must reproduce the above copyright
+ * notice, this list of conditions and the following disclaimer in the
+ * documentation and/or other materials provided with the distribution.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE AUTHOR AND CONTRIBUTORS ``AS IS'' AND
+ * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
+ * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
+ * ARE DISCLAIMED. IN NO EVENT SHALL THE AUTHOR OR CONTRIBUTORS BE LIABLE
+ * FOR ANY DIRECT, INDIRECT, INCIDENTAL, 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 DAMAGE.
+ *
+ * $FreeBSD$
+ */
+
+#include <errno.h>
+#include <fcntl.h>
+#include <stdio.h>
+#include <camlib.h>
+
+#include <atf-c.h>
+
+static const char *
+get_cam_test_device(const atf_tc_t *tc)
+{
+ const char *cam_test_device;
+
+ cam_test_device = atf_tc_get_config_var(tc, "cam_test_device");
+
+ return (cam_test_device);
+}
+
+static void
+cam_clear_error(void)
+{
+
+ strcpy(cam_errbuf, "");
+}
+
+static bool
+cam_has_error(void)
+{
+
+ return (strlen(cam_errbuf) != 0);
+}
+
+ATF_TC(cam_open_device_negative_test_O_RDONLY);
+ATF_TC_HEAD(cam_open_device_negative_test_O_RDONLY, tc)
+{
+
+ atf_tc_set_md_var(tc, "descr",
+ "test that cam_open_device(`cam_device`, O_RDONLY) fails to open "
+ "the underlying pass(4) device (bug 217649)");
+ atf_tc_set_md_var(tc, "require.config", "cam_test_device");
+ atf_tc_set_md_var(tc, "require.user", "root");
+}
+
+ATF_TC_BODY(cam_open_device_negative_test_O_RDONLY, tc)
+{
+ const char *cam_test_device;
+
+ cam_test_device = get_cam_test_device(tc);
+
+ cam_clear_error();
+ ATF_CHECK(cam_open_device(cam_test_device, O_RDONLY) == NULL);
+ ATF_REQUIRE(cam_has_error());
+}
+
+ATF_TC(cam_open_device_negative_test_nonexistent);
+ATF_TC_HEAD(cam_open_device_negative_test_nonexistent, tc)
+{
+
+ atf_tc_set_md_var(tc, "require.user", "root");
+}
+
+ATF_TC_BODY(cam_open_device_negative_test_nonexistent, tc)
+{
+
+ cam_clear_error();
+ ATF_REQUIRE(cam_open_device("/nonexistent", O_RDWR) == NULL);
+ ATF_REQUIRE(cam_has_error());
+}
+
+ATF_TC(cam_open_device_negative_test_unprivileged);
+ATF_TC_HEAD(cam_open_device_negative_test_unprivileged, tc)
+{
+
+ atf_tc_set_md_var(tc, "require.config", "cam_test_device");
+ atf_tc_set_md_var(tc, "require.user", "unprivileged");
+}
+
+ATF_TC_BODY(cam_open_device_negative_test_unprivileged, tc)
+{
+ const char *cam_test_device;
+
+ cam_test_device = get_cam_test_device(tc);
+
+ cam_clear_error();
+ ATF_CHECK(cam_open_device(cam_test_device, O_RDONLY) == NULL);
+ ATF_REQUIRE(cam_has_error());
+
+ cam_clear_error();
+ ATF_CHECK(cam_open_device(cam_test_device, O_RDWR) == NULL);
+ ATF_REQUIRE(cam_has_error());
+}
+
+ATF_TC(cam_open_device_positive_test);
+ATF_TC_HEAD(cam_open_device_positive_test, tc)
+{
+
+ atf_tc_set_md_var(tc, "require.config", "cam_test_device");
+ atf_tc_set_md_var(tc, "require.user", "root");
+}
+
+ATF_TC_BODY(cam_open_device_positive_test, tc)
+{
+ struct cam_device *cam_dev;
+ const char *cam_test_device;
+
+ cam_test_device = get_cam_test_device(tc);
+
+ cam_clear_error();
+ cam_dev = cam_open_device(cam_test_device, O_RDWR);
+ ATF_CHECK_MSG(cam_dev != NULL, "cam_open_device failed: %s",
+ cam_errbuf);
+ ATF_REQUIRE(!cam_has_error());
+ cam_close_device(cam_dev);
+}
+
+ATF_TC(cam_close_device_negative_test_NULL);
+ATF_TC_HEAD(cam_close_device_negative_test_NULL, tc)
+{
+
+ atf_tc_set_md_var(tc, "descr",
+ "test that cam_close_device(NULL) succeeds without error");
+ atf_tc_set_md_var(tc, "require.user", "root");
+}
+
+ATF_TC_BODY(cam_close_device_negative_test_NULL, tc)
+{
+
+ cam_clear_error();
+ cam_close_device(NULL);
+ ATF_REQUIRE(!cam_has_error());
+}
+
+ATF_TC(cam_getccb_positive_test);
+ATF_TC_HEAD(cam_getccb_positive_test, tc)
+{
+
+ atf_tc_set_md_var(tc, "require.config", "cam_test_device");
+ atf_tc_set_md_var(tc, "require.user", "root");
+}
+
+ATF_TC_BODY(cam_getccb_positive_test, tc)
+{
+ union ccb *cam_ccb;
+ struct cam_device *cam_dev;
+ const char *cam_test_device;
+
+ cam_test_device = get_cam_test_device(tc);
+
+ cam_clear_error();
+ cam_dev = cam_open_device(cam_test_device, O_RDWR);
+ ATF_CHECK_MSG(cam_dev != NULL, "cam_open_device failed: %s",
+ cam_errbuf);
+ ATF_REQUIRE(!cam_has_error());
+ cam_ccb = cam_getccb(cam_dev);
+ ATF_CHECK_MSG(cam_ccb != NULL, "get_camccb failed: %s", cam_errbuf);
+ ATF_REQUIRE(!cam_has_error());
+ cam_freeccb(cam_ccb);
+ cam_close_device(cam_dev);
+}
+
+ATF_TC(cam_freeccb_negative_test_NULL);
+ATF_TC_HEAD(cam_freeccb_negative_test_NULL, tc)
+{
+
+ atf_tc_set_md_var(tc, "descr",
+ "test that cam_freeccb(NULL) succeeds without error");
+ atf_tc_set_md_var(tc, "require.user", "root");
+}
+
+ATF_TC_BODY(cam_freeccb_negative_test_NULL, tc)
+{
+
+ cam_clear_error();
+ cam_freeccb(NULL);
+ ATF_REQUIRE(!cam_has_error());
+}
+
+ATF_TP_ADD_TCS(tp)
+{
+
+ ATF_TP_ADD_TC(tp, cam_open_device_negative_test_O_RDONLY);
+ ATF_TP_ADD_TC(tp, cam_open_device_negative_test_nonexistent);
+ ATF_TP_ADD_TC(tp, cam_open_device_negative_test_unprivileged);
+ ATF_TP_ADD_TC(tp, cam_open_device_positive_test);
+ ATF_TP_ADD_TC(tp, cam_close_device_negative_test_NULL);
+ ATF_TP_ADD_TC(tp, cam_getccb_positive_test);
+ ATF_TP_ADD_TC(tp, cam_freeccb_negative_test_NULL);
+
+ return (atf_no_error());
+}
More information about the svn-src-head
mailing list