From: Travis Rhoden <trhoden@redhat.com>
Date: Thu, 25 Jun 2015 22:57:12 +0000 (-0700)
Subject: [RM-11742] Add argparse tests for ceph-deploy osd
X-Git-Tag: v1.5.26~14^2~4
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=2a9a85ff8595a5e8cc7af2399f0a3e789503740b;p=ceph-deploy.git

[RM-11742] Add argparse tests for ceph-deploy osd

Signed-off-by: Travis Rhoden <trhoden@redhat.com>
---

diff --git a/ceph_deploy/tests/parser/test_osd.py b/ceph_deploy/tests/parser/test_osd.py
new file mode 100644
index 0000000..ddd35e6
--- /dev/null
+++ b/ceph_deploy/tests/parser/test_osd.py
@@ -0,0 +1,207 @@
+import pytest
+
+from ceph_deploy.cli import get_parser
+
+SUBCMDS_WITH_ARGS = ['list', 'create', 'prepare', 'activate']
+
+
+class TestParserOSD(object):
+
+    def setup(self):
+        self.parser = get_parser()
+
+    def test_osd_help(self, capsys):
+        with pytest.raises(SystemExit):
+            self.parser.parse_args('osd --help'.split())
+        out, err = capsys.readouterr()
+        assert 'usage: ceph-deploy osd' in out
+        assert 'positional arguments:' in out
+        assert 'optional arguments:' in out
+
+    @pytest.mark.parametrize('cmd', SUBCMDS_WITH_ARGS)
+    def test_osd_valid_subcommands_with_args(self, cmd):
+        self.parser.parse_args(['osd'] + ['%s' % cmd] + ['host1'])
+
+    def test_osd_invalid_subcommand(self, capsys):
+        with pytest.raises(SystemExit):
+            self.parser.parse_args('osd bork'.split())
+        out, err = capsys.readouterr()
+        assert 'invalid choice' in err
+
+    @pytest.mark.skipif(reason="http://tracker.ceph.com/issues/12151")
+    def test_osd_list_help(self, capsys):
+        with pytest.raises(SystemExit):
+            self.parser.parse_args('osd list --help'.split())
+        out, err = capsys.readouterr()
+        assert 'usage: ceph-deploy osd list' in out
+
+    def test_osd_list_host_required(self, capsys):
+        with pytest.raises(SystemExit):
+            self.parser.parse_args('osd list'.split())
+        out, err = capsys.readouterr()
+        assert 'too few arguments' in err
+
+    def test_osd_list_single_host(self):
+        args = self.parser.parse_args('osd list host1'.split())
+        assert args.disk[0][0] == 'host1'
+
+    def test_osd_list_multi_host(self):
+        hostnames = ['host1', 'host2', 'host3']
+        args = self.parser.parse_args('osd list'.split() + hostnames)
+        # args.disk is a list of tuples, and tuple[0] is the hostname
+        hosts = [x[0] for x in args.disk]
+        assert hosts == hostnames
+
+    @pytest.mark.skipif(reason="http://tracker.ceph.com/issues/12151")
+    def test_osd_create_help(self, capsys):
+        with pytest.raises(SystemExit):
+            self.parser.parse_args('osd create --help'.split())
+        out, err = capsys.readouterr()
+        assert 'usage: ceph-deploy osd create' in out
+
+    def test_osd_create_host_required(self, capsys):
+        with pytest.raises(SystemExit):
+            self.parser.parse_args('osd create'.split())
+        out, err = capsys.readouterr()
+        assert 'too few arguments' in err
+
+    def test_osd_create_single_host(self):
+        args = self.parser.parse_args('osd create host1:sdb'.split())
+        assert args.disk[0][0] == 'host1'
+
+    def test_osd_create_multi_host(self):
+        hostnames = ['host1', 'host2', 'host3']
+        args = self.parser.parse_args('osd create'.split() + [x + ":sdb" for x in hostnames])
+        # args.disk is a list of tuples, and tuple[0] is the hostname
+        hosts = [x[0] for x in args.disk]
+        assert hosts == hostnames
+
+    @pytest.mark.skipif(reason="http://tracker.ceph.com/issues/12168")
+    def test_osd_create_zap_default_false(self):
+        args = self.parser.parse_args('osd create host1:sdb'.split())
+        assert args.zap_disk is False
+
+    def test_osd_create_zap_true(self):
+        args = self.parser.parse_args('osd create --zap-disk host1:sdb'.split())
+        assert args.zap_disk is True
+
+    def test_osd_create_fstype_default_xfs(self):
+        args = self.parser.parse_args('osd create host1:sdb'.split())
+        assert args.fs_type == "xfs"
+
+    def test_osd_create_fstype_ext4(self):
+        args = self.parser.parse_args('osd create --fs-type ext4 host1:sdb'.split())
+        assert args.fs_type == "ext4"
+
+    @pytest.mark.skipif(reason="http://tracker.ceph.com/issues/12151")
+    def test_osd_create_fstype_invalid(self, capsys):
+        with pytest.raises(SystemExit):
+            self.parser.parse_args('osd create --fs-type bork host1:sdb'.split())
+        out, err = capsys.readouterr()
+        assert 'invalid choice' in err
+
+    @pytest.mark.skipif(reason="http://tracker.ceph.com/issues/12168")
+    def test_osd_create_dmcrypt_default_false(self):
+        args = self.parser.parse_args('osd create host1:sdb'.split())
+        assert args.dmcrypt is False
+
+    def test_osd_create_dmcrypt_true(self):
+        args = self.parser.parse_args('osd create --dmcrypt host1:sdb'.split())
+        assert args.dmcrypt is True
+
+    def test_osd_create_dmcrypt_key_dir_default(self):
+        args = self.parser.parse_args('osd create host1:sdb'.split())
+        assert args.dmcrypt_key_dir == "/etc/ceph/dmcrypt-keys"
+
+    def test_osd_create_dmcrypt_key_dir_custom(self):
+        args = self.parser.parse_args('osd create --dmcrypt --dmcrypt-key-dir /tmp/keys host1:sdb'.split())
+        assert args.dmcrypt_key_dir == "/tmp/keys"
+
+    @pytest.mark.skipif(reason="http://tracker.ceph.com/issues/12151")
+    def test_osd_prepare_help(self, capsys):
+        with pytest.raises(SystemExit):
+            self.parser.parse_args('osd prepare --help'.split())
+        out, err = capsys.readouterr()
+        assert 'usage: ceph-deploy osd prepare' in out
+
+    @pytest.mark.skipif(reason="http://tracker.ceph.com/issues/12168")
+    def test_osd_prepare_zap_default_false(self):
+        args = self.parser.parse_args('osd prepare host1:sdb'.split())
+        assert args.zap_disk is False
+
+    def test_osd_prepare_zap_true(self):
+        args = self.parser.parse_args('osd prepare --zap-disk host1:sdb'.split())
+        assert args.zap_disk is True
+
+    def test_osd_prepare_fstype_default_xfs(self):
+        args = self.parser.parse_args('osd prepare host1:sdb'.split())
+        assert args.fs_type == "xfs"
+
+    def test_osd_prepare_fstype_ext4(self):
+        args = self.parser.parse_args('osd prepare --fs-type ext4 host1:sdb'.split())
+        assert args.fs_type == "ext4"
+
+    @pytest.mark.skipif(reason="http://tracker.ceph.com/issues/12151")
+    def test_osd_prepare_fstype_invalid(self, capsys):
+        with pytest.raises(SystemExit):
+            self.parser.parse_args('osd prepare --fs-type bork host1:sdb'.split())
+        out, err = capsys.readouterr()
+        assert 'invalid choice' in err
+
+    @pytest.mark.skipif(reason="http://tracker.ceph.com/issues/12168")
+    def test_osd_prepare_dmcrypt_default_false(self):
+        args = self.parser.parse_args('osd prepare host1:sdb'.split())
+        assert args.dmcrypt is False
+
+    def test_osd_prepare_dmcrypt_true(self):
+        args = self.parser.parse_args('osd prepare --dmcrypt host1:sdb'.split())
+        assert args.dmcrypt is True
+
+    def test_osd_prepare_dmcrypt_key_dir_default(self):
+        args = self.parser.parse_args('osd prepare host1:sdb'.split())
+        assert args.dmcrypt_key_dir == "/etc/ceph/dmcrypt-keys"
+
+    def test_osd_prepare_dmcrypt_key_dir_custom(self):
+        args = self.parser.parse_args('osd prepare --dmcrypt --dmcrypt-key-dir /tmp/keys host1:sdb'.split())
+        assert args.dmcrypt_key_dir == "/tmp/keys"
+
+    def test_osd_prepare_host_required(self, capsys):
+        with pytest.raises(SystemExit):
+            self.parser.parse_args('osd prepare'.split())
+        out, err = capsys.readouterr()
+        assert 'too few arguments' in err
+
+    def test_osd_prepare_single_host(self):
+        args = self.parser.parse_args('osd prepare host1:sdb'.split())
+        assert args.disk[0][0] == 'host1'
+
+    def test_osd_prepare_multi_host(self):
+        hostnames = ['host1', 'host2', 'host3']
+        args = self.parser.parse_args('osd prepare'.split() + [x + ":sdb" for x in hostnames])
+        # args.disk is a list of tuples, and tuple[0] is the hostname
+        hosts = [x[0] for x in args.disk]
+        assert hosts == hostnames
+
+    @pytest.mark.skipif(reason="http://tracker.ceph.com/issues/12151")
+    def test_osd_activate_help(self, capsys):
+        with pytest.raises(SystemExit):
+            self.parser.parse_args('osd activate --help'.split())
+        out, err = capsys.readouterr()
+        assert 'usage: ceph-deploy osd activate' in out
+
+    def test_osd_activate_host_required(self, capsys):
+        with pytest.raises(SystemExit):
+            self.parser.parse_args('osd activate'.split())
+        out, err = capsys.readouterr()
+        assert 'too few arguments' in err
+
+    def test_osd_activate_single_host(self):
+        args = self.parser.parse_args('osd activate host1:sdb1'.split())
+        assert args.disk[0][0] == 'host1'
+
+    def test_osd_activate_multi_host(self):
+        hostnames = ['host1', 'host2', 'host3']
+        args = self.parser.parse_args('osd activate'.split() + [x + ":sdb1" for x in hostnames])
+        # args.disk is a list of tuples, and tuple[0] is the hostname
+        hosts = [x[0] for x in args.disk]
+        assert hosts == hostnames