Author: dab
Date: Sun Apr 12 22:11:29 2020
New Revision: 359838
URL: https://svnweb.freebsd.org/changeset/base/359838

Log:
  MFC r359717:
  
  Add a basic test for nvmecontrol
  
  I recently made some bug fixes in nvmecontrol. It occurred to me that
  since nvmecontrol lacks any kyua tests, I should convert the informal
  testing I did into a more formal automated test. The test in this
  change should be considered just a starting point; it is neither
  complete nor thorough. While converting the test to ATF/kyua, I
  discovered a small bug in nvmecontrol; the nvmecontrol devlist command
  would always exit with an unsuccessful status. So I included the fix
  for that, too, so that the test won't fail.
  
  Sponsored by: Dell EMC Isilon

Added:
  stable/12/sbin/nvmecontrol/tests/
     - copied from r359717, head/sbin/nvmecontrol/tests/
Modified:
  stable/12/etc/mtree/BSD.tests.dist
  stable/12/sbin/nvmecontrol/Makefile
  stable/12/sbin/nvmecontrol/devlist.c
Directory Properties:
  stable/12/   (props changed)

Modified: stable/12/etc/mtree/BSD.tests.dist
==============================================================================
--- stable/12/etc/mtree/BSD.tests.dist  Sun Apr 12 21:23:51 2020        
(r359837)
+++ stable/12/etc/mtree/BSD.tests.dist  Sun Apr 12 22:11:29 2020        
(r359838)
@@ -432,6 +432,8 @@
         ..
         mdconfig
         ..
+        nvmecontrol
+        ..
         pfctl
             files
             ..

Modified: stable/12/sbin/nvmecontrol/Makefile
==============================================================================
--- stable/12/sbin/nvmecontrol/Makefile Sun Apr 12 21:23:51 2020        
(r359837)
+++ stable/12/sbin/nvmecontrol/Makefile Sun Apr 12 22:11:29 2020        
(r359838)
@@ -1,5 +1,7 @@
 # $FreeBSD$
 
+.include <src.opts.mk>
+
 PACKAGE=runtime
 PROG=  nvmecontrol
 SRCS=  comnd.c nvmecontrol.c
@@ -11,6 +13,8 @@ MAN=  nvmecontrol.8
 LDFLAGS+= -rdynamic
 LIBADD+= util
 SUBDIR=        modules
+HAS_TESTS=
+SUBDIR.${MK_TESTS}+= tests
 
 .PATH: ${SRCTOP}/sys/dev/nvme
 

Modified: stable/12/sbin/nvmecontrol/devlist.c
==============================================================================
--- stable/12/sbin/nvmecontrol/devlist.c        Sun Apr 12 21:23:51 2020        
(r359837)
+++ stable/12/sbin/nvmecontrol/devlist.c        Sun Apr 12 22:11:29 2020        
(r359838)
@@ -122,8 +122,10 @@ devlist(const struct cmd *f, int argc, char *argv[])
                close(fd);
        }
 
-       if (found == 0)
+       if (found == 0) {
                printf("No NVMe controllers found.\n");
+               exit(1);
+       }
 
-       exit(1);
+       exit(0);
 }
_______________________________________________
[email protected] mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "[email protected]"

Reply via email to