/mandos/release

To get this branch, use:
bzr branch http://bzr.recompile.se/loggerhead/mandos/release

« back to all changes in this revision

Viewing changes to mandos-ctl

  • Committer: Teddy Hogeborn
  • Date: 2019-03-12 20:24:58 UTC
  • mto: This revision was merged to the branch mainline in revision 382.
  • Revision ID: teddy@recompile.se-20190312202458-14d4f7bvc61kwp2l
mandos-ctl: Refactor

* mandos-ctl (milliseconds_to_string): Move to
  "PrintTableCmd.TableOfClients.milliseconds_to_string".
  (PrintTableCmd.TableOfClients.valuetostring): Change to a class
                                                method to be able to
                                                call
                                         cls.milliseconds_to_string().
  (TestCmd.other_client): Change "ApprovalDuration" to a more odd
                          value to compensate for the removal of
                          "Test_milliseconds_to_string", which
                          contained a test for this value.  All users
                          changed.
  (Test_milliseconds_to_string): Remove.

Show diffs side-by-side

added added

removed removed

Lines of Context:
93
93
    if options.debug:
94
94
        log.setLevel(logging.DEBUG)
95
95
 
96
 
    bus = dbus.SystemBus()
97
 
 
98
 
    mandos_dbus_object = get_mandos_dbus_object(bus)
99
 
 
100
 
    mandos_serv = dbus.Interface(
101
 
        mandos_dbus_object, dbus_interface=server_dbus_interface)
 
96
    try:
 
97
        bus = dbus.SystemBus()
 
98
        log.debug("D-Bus: Connect to: (busname=%r, path=%r)",
 
99
                  dbus_busname, server_dbus_path)
 
100
        mandos_dbus_objc = bus.get_object(dbus_busname,
 
101
                                          server_dbus_path)
 
102
    except dbus.exceptions.DBusException:
 
103
        log.critical("Could not connect to Mandos server")
 
104
        sys.exit(1)
 
105
 
 
106
    mandos_serv = dbus.Interface(mandos_dbus_objc,
 
107
                                 dbus_interface=server_dbus_interface)
102
108
    mandos_serv_object_manager = dbus.Interface(
103
 
        mandos_dbus_object, dbus_interface=dbus.OBJECT_MANAGER_IFACE)
104
 
 
105
 
    managed_objects = get_managed_objects(mandos_serv_object_manager)
106
 
 
107
 
    all_clients = {}
108
 
    for path, ifs_and_props in managed_objects.items():
109
 
        try:
110
 
            all_clients[path] = ifs_and_props[client_dbus_interface]
111
 
        except KeyError:
112
 
            pass
113
 
 
114
 
    # Compile dict of (clientpath: properties) to process
 
109
        mandos_dbus_objc, dbus_interface=dbus.OBJECT_MANAGER_IFACE)
 
110
 
 
111
    # Filter out log message from dbus module
 
112
    dbus_logger = logging.getLogger("dbus.proxies")
 
113
    class NullFilter(logging.Filter):
 
114
        def filter(self, record):
 
115
            return False
 
116
    dbus_filter = NullFilter()
 
117
    try:
 
118
        dbus_logger.addFilter(dbus_filter)
 
119
        log.debug("D-Bus: %s:%s:%s.GetManagedObjects()", dbus_busname,
 
120
                  server_dbus_path, dbus.OBJECT_MANAGER_IFACE)
 
121
        mandos_clients = {path: ifs_and_props[client_dbus_interface]
 
122
                          for path, ifs_and_props in
 
123
                          mandos_serv_object_manager
 
124
                          .GetManagedObjects().items()
 
125
                          if client_dbus_interface in ifs_and_props}
 
126
    except dbus.exceptions.DBusException as e:
 
127
        log.critical("Failed to access Mandos server through D-Bus:"
 
128
                     "\n%s", e)
 
129
        sys.exit(1)
 
130
    finally:
 
131
        # restore dbus logger
 
132
        dbus_logger.removeFilter(dbus_filter)
 
133
 
 
134
    # Compile dict of (clients: properties) to process
 
135
    clients = {}
 
136
 
115
137
    if not clientnames:
116
 
        clients = all_clients
 
138
        clients = {objpath: properties
 
139
                   for objpath, properties in mandos_clients.items()}
117
140
    else:
118
 
        clients = {}
119
141
        for name in clientnames:
120
 
            for objpath, properties in all_clients.items():
 
142
            for objpath, properties in mandos_clients.items():
121
143
                if properties["Name"] == name:
122
144
                    clients[objpath] = properties
123
145
                    break
125
147
                log.critical("Client not found on server: %r", name)
126
148
                sys.exit(1)
127
149
 
 
150
    # Run all commands on clients
128
151
    commands = commands_from_options(options)
129
 
 
130
152
    for command in commands:
131
153
        command.run(clients, bus, mandos_serv)
132
154
 
232
254
    >>> rfc3339_duration_to_delta("")
233
255
    Traceback (most recent call last):
234
256
    ...
235
 
    ValueError: Invalid RFC 3339 duration: ""
 
257
    ValueError: Invalid RFC 3339 duration: u''
236
258
    >>> # Must start with "P":
237
259
    >>> rfc3339_duration_to_delta("1D")
238
260
    Traceback (most recent call last):
239
261
    ...
240
 
    ValueError: Invalid RFC 3339 duration: "1D"
 
262
    ValueError: Invalid RFC 3339 duration: u'1D'
241
263
    >>> # Must use correct order
242
264
    >>> rfc3339_duration_to_delta("PT1S2M")
243
265
    Traceback (most recent call last):
244
266
    ...
245
 
    ValueError: Invalid RFC 3339 duration: "PT1S2M"
 
267
    ValueError: Invalid RFC 3339 duration: u'PT1S2M'
246
268
    >>> # Time needs time marker
247
269
    >>> rfc3339_duration_to_delta("P1H2S")
248
270
    Traceback (most recent call last):
249
271
    ...
250
 
    ValueError: Invalid RFC 3339 duration: "P1H2S"
 
272
    ValueError: Invalid RFC 3339 duration: u'P1H2S'
251
273
    >>> # Weeks can not be combined with anything else
252
274
    >>> rfc3339_duration_to_delta("P1D2W")
253
275
    Traceback (most recent call last):
254
276
    ...
255
 
    ValueError: Invalid RFC 3339 duration: "P1D2W"
 
277
    ValueError: Invalid RFC 3339 duration: u'P1D2W'
256
278
    >>> rfc3339_duration_to_delta("P2W2H")
257
279
    Traceback (most recent call last):
258
280
    ...
259
 
    ValueError: Invalid RFC 3339 duration: "P2W2H"
 
281
    ValueError: Invalid RFC 3339 duration: u'P2W2H'
260
282
    """
261
283
 
262
284
    # Parsing an RFC 3339 duration with regular expressions is not
333
355
                break
334
356
        else:
335
357
            # No currently valid tokens were found
336
 
            raise ValueError("Invalid RFC 3339 duration: \"{}\""
 
358
            raise ValueError("Invalid RFC 3339 duration: {!r}"
337
359
                             .format(duration))
338
360
    # End token found
339
361
    return value
424
446
        options.remove = True
425
447
 
426
448
 
427
 
def get_mandos_dbus_object(bus):
428
 
    log.debug("D-Bus: Connect to: (busname=%r, path=%r)",
429
 
              dbus_busname, server_dbus_path)
430
 
    with if_dbus_exception_log_with_exception_and_exit(
431
 
            "Could not connect to Mandos server: %s"):
432
 
        mandos_dbus_object = bus.get_object(dbus_busname,
433
 
                                            server_dbus_path)
434
 
    return mandos_dbus_object
435
 
 
436
 
 
437
 
@contextlib.contextmanager
438
 
def if_dbus_exception_log_with_exception_and_exit(*args, **kwargs):
439
 
    try:
440
 
        yield
441
 
    except dbus.exceptions.DBusException as e:
442
 
        log.critical(*(args + (e,)), **kwargs)
443
 
        sys.exit(1)
444
 
 
445
 
 
446
 
def get_managed_objects(object_manager):
447
 
    log.debug("D-Bus: %s:%s:%s.GetManagedObjects()", dbus_busname,
448
 
              server_dbus_path, dbus.OBJECT_MANAGER_IFACE)
449
 
    with if_dbus_exception_log_with_exception_and_exit(
450
 
            "Failed to access Mandos server through D-Bus:\n%s"):
451
 
        with SilenceLogger("dbus.proxies"):
452
 
            managed_objects = object_manager.GetManagedObjects()
453
 
    return managed_objects
454
 
 
455
 
 
456
 
class SilenceLogger(object):
457
 
    "Simple context manager to silence a particular logger"
458
 
    def __init__(self, loggername):
459
 
        self.logger = logging.getLogger(loggername)
460
 
 
461
 
    def __enter__(self):
462
 
        self.logger.addFilter(self.nullfilter)
463
 
        return self
464
 
 
465
 
    class NullFilter(logging.Filter):
466
 
        def filter(self, record):
467
 
            return False
468
 
 
469
 
    nullfilter = NullFilter()
470
 
 
471
 
    def __exit__(self, exc_type, exc_val, exc_tb):
472
 
        self.logger.removeFilter(self.nullfilter)
473
 
 
474
 
 
475
449
def commands_from_options(options):
476
450
 
477
451
    commands = []
478
452
 
479
453
    if options.is_enabled:
480
 
        commands.append(command.IsEnabled())
 
454
        commands.append(IsEnabledCmd())
481
455
 
482
456
    if options.approve:
483
 
        commands.append(command.Approve())
 
457
        commands.append(ApproveCmd())
484
458
 
485
459
    if options.deny:
486
 
        commands.append(command.Deny())
 
460
        commands.append(DenyCmd())
487
461
 
488
462
    if options.remove:
489
 
        commands.append(command.Remove())
 
463
        commands.append(RemoveCmd())
490
464
 
491
465
    if options.dump_json:
492
 
        commands.append(command.DumpJSON())
 
466
        commands.append(DumpJSONCmd())
493
467
 
494
468
    if options.enable:
495
 
        commands.append(command.Enable())
 
469
        commands.append(EnableCmd())
496
470
 
497
471
    if options.disable:
498
 
        commands.append(command.Disable())
 
472
        commands.append(DisableCmd())
499
473
 
500
474
    if options.bump_timeout:
501
 
        commands.append(command.BumpTimeout())
 
475
        commands.append(BumpTimeoutCmd())
502
476
 
503
477
    if options.start_checker:
504
 
        commands.append(command.StartChecker())
 
478
        commands.append(StartCheckerCmd())
505
479
 
506
480
    if options.stop_checker:
507
 
        commands.append(command.StopChecker())
 
481
        commands.append(StopCheckerCmd())
508
482
 
509
483
    if options.approved_by_default is not None:
510
484
        if options.approved_by_default:
511
 
            commands.append(command.ApproveByDefault())
 
485
            commands.append(ApproveByDefaultCmd())
512
486
        else:
513
 
            commands.append(command.DenyByDefault())
 
487
            commands.append(DenyByDefaultCmd())
514
488
 
515
489
    if options.checker is not None:
516
 
        commands.append(command.SetChecker(options.checker))
 
490
        commands.append(SetCheckerCmd(options.checker))
517
491
 
518
492
    if options.host is not None:
519
 
        commands.append(command.SetHost(options.host))
 
493
        commands.append(SetHostCmd(options.host))
520
494
 
521
495
    if options.secret is not None:
522
 
        commands.append(command.SetSecret(options.secret))
 
496
        commands.append(SetSecretCmd(options.secret))
523
497
 
524
498
    if options.timeout is not None:
525
 
        commands.append(command.SetTimeout(options.timeout))
 
499
        commands.append(SetTimeoutCmd(options.timeout))
526
500
 
527
501
    if options.extended_timeout:
528
502
        commands.append(
529
 
            command.SetExtendedTimeout(options.extended_timeout))
 
503
            SetExtendedTimeoutCmd(options.extended_timeout))
530
504
 
531
505
    if options.interval is not None:
532
 
        commands.append(command.SetInterval(options.interval))
 
506
        commands.append(SetIntervalCmd(options.interval))
533
507
 
534
508
    if options.approval_delay is not None:
535
 
        commands.append(
536
 
            command.SetApprovalDelay(options.approval_delay))
 
509
        commands.append(SetApprovalDelayCmd(options.approval_delay))
537
510
 
538
511
    if options.approval_duration is not None:
539
512
        commands.append(
540
 
            command.SetApprovalDuration(options.approval_duration))
 
513
            SetApprovalDurationCmd(options.approval_duration))
541
514
 
542
515
    # If no command option has been given, show table of clients,
543
516
    # optionally verbosely
544
517
    if not commands:
545
 
        commands.append(command.PrintTable(verbose=options.verbose))
 
518
        commands.append(PrintTableCmd(verbose=options.verbose))
546
519
 
547
520
    return commands
548
521
 
549
522
 
550
 
class command(object):
551
 
    """A namespace for command classes"""
552
 
 
553
 
    class Base(object):
554
 
        """Abstract base class for commands"""
555
 
        def run(self, clients, bus=None, mandos=None):
556
 
            """Normal commands should implement run_on_one_client(),
557
 
but commands which want to operate on all clients at the same time can
558
 
override this run() method instead.
559
 
"""
560
 
            self.mandos = mandos
561
 
            for clientpath, properties in clients.items():
562
 
                log.debug("D-Bus: Connect to: (busname=%r, path=%r)",
563
 
                          dbus_busname, str(clientpath))
564
 
                client = bus.get_object(dbus_busname, clientpath)
565
 
                self.run_on_one_client(client, properties)
566
 
 
567
 
 
568
 
    class IsEnabled(Base):
569
 
        def run(self, clients, bus=None, mandos=None):
570
 
            client, properties = next(iter(clients.items()))
571
 
            if self.is_enabled(client, properties):
572
 
                sys.exit(0)
573
 
            sys.exit(1)
574
 
        def is_enabled(self, client, properties):
575
 
            return properties["Enabled"]
576
 
 
577
 
 
578
 
    class Approve(Base):
579
 
        def run_on_one_client(self, client, properties):
580
 
            log.debug("D-Bus: %s:%s:%s.Approve(True)", dbus_busname,
581
 
                      client.__dbus_object_path__,
582
 
                      client_dbus_interface)
583
 
            client.Approve(dbus.Boolean(True),
584
 
                           dbus_interface=client_dbus_interface)
585
 
 
586
 
 
587
 
    class Deny(Base):
588
 
        def run_on_one_client(self, client, properties):
589
 
            log.debug("D-Bus: %s:%s:%s.Approve(False)", dbus_busname,
590
 
                      client.__dbus_object_path__,
591
 
                      client_dbus_interface)
592
 
            client.Approve(dbus.Boolean(False),
593
 
                           dbus_interface=client_dbus_interface)
594
 
 
595
 
 
596
 
    class Remove(Base):
597
 
        def run_on_one_client(self, client, properties):
598
 
            log.debug("D-Bus: %s:%s:%s.RemoveClient(%r)",
599
 
                      dbus_busname, server_dbus_path,
600
 
                      server_dbus_interface,
601
 
                      str(client.__dbus_object_path__))
602
 
            self.mandos.RemoveClient(client.__dbus_object_path__)
603
 
 
604
 
 
605
 
    class Output(Base):
606
 
        """Abstract class for commands outputting client details"""
607
 
        all_keywords = ("Name", "Enabled", "Timeout", "LastCheckedOK",
608
 
                        "Created", "Interval", "Host", "KeyID",
609
 
                        "Fingerprint", "CheckerRunning",
610
 
                        "LastEnabled", "ApprovalPending",
611
 
                        "ApprovedByDefault", "LastApprovalRequest",
612
 
                        "ApprovalDelay", "ApprovalDuration",
613
 
                        "Checker", "ExtendedTimeout", "Expires",
614
 
                        "LastCheckerStatus")
615
 
 
616
 
        def run(self, clients, bus=None, mandos=None):
617
 
            print(self.output(clients.values()))
618
 
 
619
 
        def output(self, clients):
620
 
            raise NotImplementedError()
621
 
 
622
 
 
623
 
    class DumpJSON(Output):
624
 
        def output(self, clients):
625
 
            data = {client["Name"]:
626
 
                    {key: self.dbus_boolean_to_bool(client[key])
627
 
                     for key in self.all_keywords}
628
 
                    for client in clients}
629
 
            return json.dumps(data, indent=4, separators=(',', ': '))
 
523
class Command(object):
 
524
    """Abstract class for commands"""
 
525
    def run(self, clients, bus=None, mandos=None):
 
526
        """Normal commands should implement run_on_one_client(), but
 
527
        commands which want to operate on all clients at the same time
 
528
        can override this run() method instead."""
 
529
        self.mandos = mandos
 
530
        for clientpath, properties in clients.items():
 
531
            log.debug("D-Bus: Connect to: (busname=%r, path=%r)",
 
532
                      dbus_busname, str(clientpath))
 
533
            client = bus.get_object(dbus_busname, clientpath)
 
534
            self.run_on_one_client(client, properties)
 
535
 
 
536
 
 
537
class IsEnabledCmd(Command):
 
538
    def run(self, clients, bus=None, mandos=None):
 
539
        client, properties = next(iter(clients.items()))
 
540
        if self.is_enabled(client, properties):
 
541
            sys.exit(0)
 
542
        sys.exit(1)
 
543
    def is_enabled(self, client, properties):
 
544
        return properties["Enabled"]
 
545
 
 
546
 
 
547
class ApproveCmd(Command):
 
548
    def run_on_one_client(self, client, properties):
 
549
        log.debug("D-Bus: %s:%s:%s.Approve(True)", dbus_busname,
 
550
                  client.__dbus_object_path__, client_dbus_interface)
 
551
        client.Approve(dbus.Boolean(True),
 
552
                       dbus_interface=client_dbus_interface)
 
553
 
 
554
 
 
555
class DenyCmd(Command):
 
556
    def run_on_one_client(self, client, properties):
 
557
        log.debug("D-Bus: %s:%s:%s.Approve(False)", dbus_busname,
 
558
                  client.__dbus_object_path__, client_dbus_interface)
 
559
        client.Approve(dbus.Boolean(False),
 
560
                       dbus_interface=client_dbus_interface)
 
561
 
 
562
 
 
563
class RemoveCmd(Command):
 
564
    def run_on_one_client(self, client, properties):
 
565
        log.debug("D-Bus: %s:%s:%s.RemoveClient(%r)", dbus_busname,
 
566
                  server_dbus_path, server_dbus_interface,
 
567
                  str(client.__dbus_object_path__))
 
568
        self.mandos.RemoveClient(client.__dbus_object_path__)
 
569
 
 
570
 
 
571
class PrintCmd(Command):
 
572
    """Abstract class for commands printing client details"""
 
573
    all_keywords = ("Name", "Enabled", "Timeout", "LastCheckedOK",
 
574
                    "Created", "Interval", "Host", "KeyID",
 
575
                    "Fingerprint", "CheckerRunning", "LastEnabled",
 
576
                    "ApprovalPending", "ApprovedByDefault",
 
577
                    "LastApprovalRequest", "ApprovalDelay",
 
578
                    "ApprovalDuration", "Checker", "ExtendedTimeout",
 
579
                    "Expires", "LastCheckerStatus")
 
580
    def run(self, clients, bus=None, mandos=None):
 
581
        print(self.output(clients.values()))
 
582
    def output(self, clients):
 
583
        raise NotImplementedError()
 
584
 
 
585
 
 
586
class DumpJSONCmd(PrintCmd):
 
587
    def output(self, clients):
 
588
        data = {client["Name"]:
 
589
                {key: self.dbus_boolean_to_bool(client[key])
 
590
                 for key in self.all_keywords}
 
591
                for client in clients.values()}
 
592
        return json.dumps(data, indent=4, separators=(',', ': '))
 
593
    @staticmethod
 
594
    def dbus_boolean_to_bool(value):
 
595
        if isinstance(value, dbus.Boolean):
 
596
            value = bool(value)
 
597
        return value
 
598
 
 
599
 
 
600
class PrintTableCmd(PrintCmd):
 
601
    def __init__(self, verbose=False):
 
602
        self.verbose = verbose
 
603
 
 
604
    def output(self, clients):
 
605
        default_keywords = ("Name", "Enabled", "Timeout",
 
606
                            "LastCheckedOK")
 
607
        keywords = default_keywords
 
608
        if self.verbose:
 
609
            keywords = self.all_keywords
 
610
        return str(self.TableOfClients(clients, keywords))
 
611
 
 
612
    class TableOfClients(object):
 
613
        tableheaders = {
 
614
            "Name": "Name",
 
615
            "Enabled": "Enabled",
 
616
            "Timeout": "Timeout",
 
617
            "LastCheckedOK": "Last Successful Check",
 
618
            "LastApprovalRequest": "Last Approval Request",
 
619
            "Created": "Created",
 
620
            "Interval": "Interval",
 
621
            "Host": "Host",
 
622
            "Fingerprint": "Fingerprint",
 
623
            "KeyID": "Key ID",
 
624
            "CheckerRunning": "Check Is Running",
 
625
            "LastEnabled": "Last Enabled",
 
626
            "ApprovalPending": "Approval Is Pending",
 
627
            "ApprovedByDefault": "Approved By Default",
 
628
            "ApprovalDelay": "Approval Delay",
 
629
            "ApprovalDuration": "Approval Duration",
 
630
            "Checker": "Checker",
 
631
            "ExtendedTimeout": "Extended Timeout",
 
632
            "Expires": "Expires",
 
633
            "LastCheckerStatus": "Last Checker Status",
 
634
        }
 
635
 
 
636
        def __init__(self, clients, keywords, tableheaders=None):
 
637
            self.clients = clients
 
638
            self.keywords = keywords
 
639
            if tableheaders is not None:
 
640
                self.tableheaders = tableheaders
 
641
 
 
642
        def __str__(self):
 
643
            return "\n".join(self.rows())
 
644
 
 
645
        if sys.version_info.major == 2:
 
646
            __unicode__ = __str__
 
647
            def __str__(self):
 
648
                return str(self).encode(locale.getpreferredencoding())
 
649
 
 
650
        def rows(self):
 
651
            format_string = self.row_formatting_string()
 
652
            rows = [self.header_line(format_string)]
 
653
            rows.extend(self.client_line(client, format_string)
 
654
                        for client in self.clients)
 
655
            return rows
 
656
 
 
657
        def row_formatting_string(self):
 
658
            "Format string used to format table rows"
 
659
            return " ".join("{{{key}:{width}}}".format(
 
660
                width=max(len(self.tableheaders[key]),
 
661
                          *(len(self.string_from_client(client, key))
 
662
                            for client in self.clients)),
 
663
                key=key)
 
664
                            for key in self.keywords)
 
665
 
 
666
        def string_from_client(self, client, key):
 
667
            return self.valuetostring(client[key], key)
 
668
 
 
669
        @classmethod
 
670
        def valuetostring(cls, value, keyword):
 
671
            if isinstance(value, dbus.Boolean):
 
672
                return "Yes" if value else "No"
 
673
            if keyword in ("Timeout", "Interval", "ApprovalDelay",
 
674
                           "ApprovalDuration", "ExtendedTimeout"):
 
675
                return cls.milliseconds_to_string(value)
 
676
            return str(value)
 
677
 
 
678
        def header_line(self, format_string):
 
679
            return format_string.format(**self.tableheaders)
 
680
 
 
681
        def client_line(self, client, format_string):
 
682
            return format_string.format(
 
683
                **{key: self.string_from_client(client, key)
 
684
                   for key in self.keywords})
630
685
 
631
686
        @staticmethod
632
 
        def dbus_boolean_to_bool(value):
633
 
            if isinstance(value, dbus.Boolean):
634
 
                value = bool(value)
635
 
            return value
636
 
 
637
 
 
638
 
    class PrintTable(Output):
639
 
        def __init__(self, verbose=False):
640
 
            self.verbose = verbose
641
 
 
642
 
        def output(self, clients):
643
 
            default_keywords = ("Name", "Enabled", "Timeout",
644
 
                                "LastCheckedOK")
645
 
            keywords = default_keywords
646
 
            if self.verbose:
647
 
                keywords = self.all_keywords
648
 
            return str(self.TableOfClients(clients, keywords))
649
 
 
650
 
        class TableOfClients(object):
651
 
            tableheaders = {
652
 
                "Name": "Name",
653
 
                "Enabled": "Enabled",
654
 
                "Timeout": "Timeout",
655
 
                "LastCheckedOK": "Last Successful Check",
656
 
                "LastApprovalRequest": "Last Approval Request",
657
 
                "Created": "Created",
658
 
                "Interval": "Interval",
659
 
                "Host": "Host",
660
 
                "Fingerprint": "Fingerprint",
661
 
                "KeyID": "Key ID",
662
 
                "CheckerRunning": "Check Is Running",
663
 
                "LastEnabled": "Last Enabled",
664
 
                "ApprovalPending": "Approval Is Pending",
665
 
                "ApprovedByDefault": "Approved By Default",
666
 
                "ApprovalDelay": "Approval Delay",
667
 
                "ApprovalDuration": "Approval Duration",
668
 
                "Checker": "Checker",
669
 
                "ExtendedTimeout": "Extended Timeout",
670
 
                "Expires": "Expires",
671
 
                "LastCheckerStatus": "Last Checker Status",
672
 
            }
673
 
 
674
 
            def __init__(self, clients, keywords):
675
 
                self.clients = clients
676
 
                self.keywords = keywords
677
 
 
678
 
            def __str__(self):
679
 
                return "\n".join(self.rows())
680
 
 
681
 
            if sys.version_info.major == 2:
682
 
                __unicode__ = __str__
683
 
                def __str__(self):
684
 
                    return str(self).encode(
685
 
                        locale.getpreferredencoding())
686
 
 
687
 
            def rows(self):
688
 
                format_string = self.row_formatting_string()
689
 
                rows = [self.header_line(format_string)]
690
 
                rows.extend(self.client_line(client, format_string)
691
 
                            for client in self.clients)
692
 
                return rows
693
 
 
694
 
            def row_formatting_string(self):
695
 
                "Format string used to format table rows"
696
 
                return " ".join("{{{key}:{width}}}".format(
697
 
                    width=max(len(self.tableheaders[key]),
698
 
                              *(len(self.string_from_client(client,
699
 
                                                            key))
700
 
                                for client in self.clients)),
701
 
                    key=key)
702
 
                                for key in self.keywords)
703
 
 
704
 
            def string_from_client(self, client, key):
705
 
                return self.valuetostring(client[key], key)
706
 
 
707
 
            @classmethod
708
 
            def valuetostring(cls, value, keyword):
709
 
                if isinstance(value, dbus.Boolean):
710
 
                    return "Yes" if value else "No"
711
 
                if keyword in ("Timeout", "Interval", "ApprovalDelay",
712
 
                               "ApprovalDuration", "ExtendedTimeout"):
713
 
                    return cls.milliseconds_to_string(value)
714
 
                return str(value)
715
 
 
716
 
            def header_line(self, format_string):
717
 
                return format_string.format(**self.tableheaders)
718
 
 
719
 
            def client_line(self, client, format_string):
720
 
                return format_string.format(
721
 
                    **{key: self.string_from_client(client, key)
722
 
                       for key in self.keywords})
723
 
 
724
 
            @staticmethod
725
 
            def milliseconds_to_string(ms):
726
 
                td = datetime.timedelta(0, 0, 0, ms)
727
 
                return ("{days}{hours:02}:{minutes:02}:{seconds:02}"
728
 
                        .format(days="{}T".format(td.days)
729
 
                                if td.days else "",
730
 
                                hours=td.seconds // 3600,
731
 
                                minutes=(td.seconds % 3600) // 60,
732
 
                                seconds=td.seconds % 60))
733
 
 
734
 
 
735
 
    class Property(Base):
736
 
        "Abstract class for Actions for setting one client property"
737
 
 
738
 
        def run_on_one_client(self, client, properties):
739
 
            """Set the Client's D-Bus property"""
740
 
            log.debug("D-Bus: %s:%s:%s.Set(%r, %r, %r)", dbus_busname,
741
 
                      client.__dbus_object_path__,
742
 
                      dbus.PROPERTIES_IFACE, client_dbus_interface,
743
 
                      self.propname, self.value_to_set
744
 
                      if not isinstance(self.value_to_set,
745
 
                                        dbus.Boolean)
746
 
                      else bool(self.value_to_set))
747
 
            client.Set(client_dbus_interface, self.propname,
748
 
                       self.value_to_set,
749
 
                       dbus_interface=dbus.PROPERTIES_IFACE)
750
 
 
751
 
        @property
752
 
        def propname(self):
753
 
            raise NotImplementedError()
754
 
 
755
 
 
756
 
    class Enable(Property):
757
 
        propname = "Enabled"
758
 
        value_to_set = dbus.Boolean(True)
759
 
 
760
 
 
761
 
    class Disable(Property):
762
 
        propname = "Enabled"
763
 
        value_to_set = dbus.Boolean(False)
764
 
 
765
 
 
766
 
    class BumpTimeout(Property):
767
 
        propname = "LastCheckedOK"
768
 
        value_to_set = ""
769
 
 
770
 
 
771
 
    class StartChecker(Property):
772
 
        propname = "CheckerRunning"
773
 
        value_to_set = dbus.Boolean(True)
774
 
 
775
 
 
776
 
    class StopChecker(Property):
777
 
        propname = "CheckerRunning"
778
 
        value_to_set = dbus.Boolean(False)
779
 
 
780
 
 
781
 
    class ApproveByDefault(Property):
782
 
        propname = "ApprovedByDefault"
783
 
        value_to_set = dbus.Boolean(True)
784
 
 
785
 
 
786
 
    class DenyByDefault(Property):
787
 
        propname = "ApprovedByDefault"
788
 
        value_to_set = dbus.Boolean(False)
789
 
 
790
 
 
791
 
    class PropertyValue(Property):
792
 
        "Abstract class for Property recieving a value as argument"
793
 
        def __init__(self, value):
794
 
            self.value_to_set = value
795
 
 
796
 
 
797
 
    class SetChecker(PropertyValue):
798
 
        propname = "Checker"
799
 
 
800
 
 
801
 
    class SetHost(PropertyValue):
802
 
        propname = "Host"
803
 
 
804
 
 
805
 
    class SetSecret(PropertyValue):
806
 
        propname = "Secret"
807
 
 
808
 
        @property
809
 
        def value_to_set(self):
810
 
            return self._vts
811
 
 
812
 
        @value_to_set.setter
813
 
        def value_to_set(self, value):
814
 
            """When setting, read data from supplied file object"""
815
 
            self._vts = value.read()
816
 
            value.close()
817
 
 
818
 
 
819
 
    class MillisecondsPropertyValueArgument(PropertyValue):
820
 
        """Abstract class for PropertyValue taking a value argument as
 
687
        def milliseconds_to_string(ms):
 
688
            td = datetime.timedelta(0, 0, 0, ms)
 
689
            return ("{days}{hours:02}:{minutes:02}:{seconds:02}"
 
690
                    .format(days="{}T".format(td.days)
 
691
                            if td.days else "",
 
692
                            hours=td.seconds // 3600,
 
693
                            minutes=(td.seconds % 3600) // 60,
 
694
                            seconds=td.seconds % 60))
 
695
 
 
696
 
 
697
class PropertyCmd(Command):
 
698
    """Abstract class for Actions for setting one client property"""
 
699
    def run_on_one_client(self, client, properties):
 
700
        """Set the Client's D-Bus property"""
 
701
        log.debug("D-Bus: %s:%s:%s.Set(%r, %r, %r)", dbus_busname,
 
702
                  client.__dbus_object_path__,
 
703
                  dbus.PROPERTIES_IFACE, client_dbus_interface,
 
704
                  self.propname, self.value_to_set
 
705
                  if not isinstance(self.value_to_set, dbus.Boolean)
 
706
                  else bool(self.value_to_set))
 
707
        client.Set(client_dbus_interface, self.propname,
 
708
                   self.value_to_set,
 
709
                   dbus_interface=dbus.PROPERTIES_IFACE)
 
710
    @property
 
711
    def propname(self):
 
712
        raise NotImplementedError()
 
713
 
 
714
 
 
715
class EnableCmd(PropertyCmd):
 
716
    propname = "Enabled"
 
717
    value_to_set = dbus.Boolean(True)
 
718
 
 
719
 
 
720
class DisableCmd(PropertyCmd):
 
721
    propname = "Enabled"
 
722
    value_to_set = dbus.Boolean(False)
 
723
 
 
724
 
 
725
class BumpTimeoutCmd(PropertyCmd):
 
726
    propname = "LastCheckedOK"
 
727
    value_to_set = ""
 
728
 
 
729
 
 
730
class StartCheckerCmd(PropertyCmd):
 
731
    propname = "CheckerRunning"
 
732
    value_to_set = dbus.Boolean(True)
 
733
 
 
734
 
 
735
class StopCheckerCmd(PropertyCmd):
 
736
    propname = "CheckerRunning"
 
737
    value_to_set = dbus.Boolean(False)
 
738
 
 
739
 
 
740
class ApproveByDefaultCmd(PropertyCmd):
 
741
    propname = "ApprovedByDefault"
 
742
    value_to_set = dbus.Boolean(True)
 
743
 
 
744
 
 
745
class DenyByDefaultCmd(PropertyCmd):
 
746
    propname = "ApprovedByDefault"
 
747
    value_to_set = dbus.Boolean(False)
 
748
 
 
749
 
 
750
class PropertyValueCmd(PropertyCmd):
 
751
    """Abstract class for PropertyCmd recieving a value as argument"""
 
752
    def __init__(self, value):
 
753
        self.value_to_set = value
 
754
 
 
755
 
 
756
class SetCheckerCmd(PropertyValueCmd):
 
757
    propname = "Checker"
 
758
 
 
759
 
 
760
class SetHostCmd(PropertyValueCmd):
 
761
    propname = "Host"
 
762
 
 
763
 
 
764
class SetSecretCmd(PropertyValueCmd):
 
765
    propname = "Secret"
 
766
    @property
 
767
    def value_to_set(self):
 
768
        return self._vts
 
769
    @value_to_set.setter
 
770
    def value_to_set(self, value):
 
771
        """When setting, read data from supplied file object"""
 
772
        self._vts = value.read()
 
773
        value.close()
 
774
 
 
775
 
 
776
class MillisecondsPropertyValueArgumentCmd(PropertyValueCmd):
 
777
    """Abstract class for PropertyValueCmd taking a value argument as
821
778
a datetime.timedelta() but should store it as milliseconds."""
822
 
 
823
 
        @property
824
 
        def value_to_set(self):
825
 
            return self._vts
826
 
 
827
 
        @value_to_set.setter
828
 
        def value_to_set(self, value):
829
 
            "When setting, convert value from a datetime.timedelta"
830
 
            self._vts = int(round(value.total_seconds() * 1000))
831
 
 
832
 
 
833
 
    class SetTimeout(MillisecondsPropertyValueArgument):
834
 
        propname = "Timeout"
835
 
 
836
 
 
837
 
    class SetExtendedTimeout(MillisecondsPropertyValueArgument):
838
 
        propname = "ExtendedTimeout"
839
 
 
840
 
 
841
 
    class SetInterval(MillisecondsPropertyValueArgument):
842
 
        propname = "Interval"
843
 
 
844
 
 
845
 
    class SetApprovalDelay(MillisecondsPropertyValueArgument):
846
 
        propname = "ApprovalDelay"
847
 
 
848
 
 
849
 
    class SetApprovalDuration(MillisecondsPropertyValueArgument):
850
 
        propname = "ApprovalDuration"
 
779
    @property
 
780
    def value_to_set(self):
 
781
        return self._vts
 
782
    @value_to_set.setter
 
783
    def value_to_set(self, value):
 
784
        """When setting, convert value from a datetime.timedelta"""
 
785
        self._vts = int(round(value.total_seconds() * 1000))
 
786
 
 
787
 
 
788
class SetTimeoutCmd(MillisecondsPropertyValueArgumentCmd):
 
789
    propname = "Timeout"
 
790
 
 
791
 
 
792
class SetExtendedTimeoutCmd(MillisecondsPropertyValueArgumentCmd):
 
793
    propname = "ExtendedTimeout"
 
794
 
 
795
 
 
796
class SetIntervalCmd(MillisecondsPropertyValueArgumentCmd):
 
797
    propname = "Interval"
 
798
 
 
799
 
 
800
class SetApprovalDelayCmd(MillisecondsPropertyValueArgumentCmd):
 
801
    propname = "ApprovalDelay"
 
802
 
 
803
 
 
804
class SetApprovalDurationCmd(MillisecondsPropertyValueArgumentCmd):
 
805
    propname = "ApprovalDuration"
851
806
 
852
807
 
853
808
 
854
 
class TestCaseWithAssertLogs(unittest.TestCase):
855
 
    """unittest.TestCase.assertLogs only exists in Python 3.4"""
856
 
 
857
 
    if not hasattr(unittest.TestCase, "assertLogs"):
858
 
        @contextlib.contextmanager
859
 
        def assertLogs(self, logger, level=logging.INFO):
860
 
            capturing_handler = self.CapturingLevelHandler(level)
861
 
            old_level = logger.level
862
 
            old_propagate = logger.propagate
863
 
            logger.addHandler(capturing_handler)
864
 
            logger.setLevel(level)
865
 
            logger.propagate = False
866
 
            try:
867
 
                yield capturing_handler.watcher
868
 
            finally:
869
 
                logger.propagate = old_propagate
870
 
                logger.removeHandler(capturing_handler)
871
 
                logger.setLevel(old_level)
872
 
            self.assertGreater(len(capturing_handler.watcher.records),
873
 
                               0)
874
 
 
875
 
        class CapturingLevelHandler(logging.Handler):
876
 
            def __init__(self, level, *args, **kwargs):
877
 
                logging.Handler.__init__(self, *args, **kwargs)
878
 
                self.watcher = self.LoggingWatcher([], [])
879
 
            def emit(self, record):
880
 
                self.watcher.records.append(record)
881
 
                self.watcher.output.append(self.format(record))
882
 
 
883
 
            LoggingWatcher = collections.namedtuple("LoggingWatcher",
884
 
                                                    ("records",
885
 
                                                     "output"))
886
 
 
887
 
 
888
 
class Test_string_to_delta(TestCaseWithAssertLogs):
 
809
class Test_string_to_delta(unittest.TestCase):
889
810
    def test_handles_basic_rfc3339(self):
890
811
        self.assertEqual(string_to_delta("PT0S"),
891
812
                         datetime.timedelta())
895
816
                         datetime.timedelta(0, 1))
896
817
        self.assertEqual(string_to_delta("PT2H"),
897
818
                         datetime.timedelta(0, 7200))
898
 
 
899
819
    def test_falls_back_to_pre_1_6_1_with_warning(self):
900
 
        with self.assertLogs(log, logging.WARNING):
901
 
            value = string_to_delta("2h")
 
820
        # assertLogs only exists in Python 3.4
 
821
        if hasattr(self, "assertLogs"):
 
822
            with self.assertLogs(log, logging.WARNING):
 
823
                value = string_to_delta("2h")
 
824
        else:
 
825
            class WarningFilter(logging.Filter):
 
826
                """Don't show, but record the presence of, warnings"""
 
827
                def filter(self, record):
 
828
                    is_warning = record.levelno >= logging.WARNING
 
829
                    self.found = is_warning or getattr(self, "found",
 
830
                                                       False)
 
831
                    return not is_warning
 
832
            warning_filter = WarningFilter()
 
833
            log.addFilter(warning_filter)
 
834
            try:
 
835
                value = string_to_delta("2h")
 
836
            finally:
 
837
                log.removeFilter(warning_filter)
 
838
            self.assertTrue(getattr(warning_filter, "found", False))
902
839
        self.assertEqual(value, datetime.timedelta(0, 7200))
903
840
 
904
841
 
905
842
class Test_check_option_syntax(unittest.TestCase):
906
 
    def setUp(self):
907
 
        self.parser = argparse.ArgumentParser()
908
 
        add_command_line_options(self.parser)
909
 
 
910
 
    def test_actions_requires_client_or_all(self):
911
 
        for action, value in self.actions.items():
912
 
            options = self.parser.parse_args()
913
 
            setattr(options, action, value)
914
 
            with self.assertParseError():
915
 
                self.check_option_syntax(options)
916
 
 
917
843
    # This mostly corresponds to the definition from has_actions() in
918
844
    # check_option_syntax()
919
845
    actions = {
940
866
        "deny": True,
941
867
    }
942
868
 
 
869
    def setUp(self):
 
870
        self.parser = argparse.ArgumentParser()
 
871
        add_command_line_options(self.parser)
 
872
 
943
873
    @contextlib.contextmanager
944
874
    def assertParseError(self):
945
875
        with self.assertRaises(SystemExit) as e:
946
 
            with self.redirect_stderr_to_devnull():
 
876
            with self.temporarily_suppress_stderr():
947
877
                yield
948
878
        # Exit code from argparse is guaranteed to be "2".  Reference:
949
879
        # https://docs.python.org/3/library
952
882
 
953
883
    @staticmethod
954
884
    @contextlib.contextmanager
955
 
    def redirect_stderr_to_devnull():
 
885
    def temporarily_suppress_stderr():
956
886
        null = os.open(os.path.devnull, os.O_RDWR)
957
887
        stderrcopy = os.dup(sys.stderr.fileno())
958
888
        os.dup2(null, sys.stderr.fileno())
967
897
    def check_option_syntax(self, options):
968
898
        check_option_syntax(self.parser, options)
969
899
 
970
 
    def test_actions_all_conflicts_with_verbose(self):
 
900
    def test_actions_requires_client_or_all(self):
971
901
        for action, value in self.actions.items():
972
902
            options = self.parser.parse_args()
973
903
            setattr(options, action, value)
974
 
            options.all = True
975
 
            options.verbose = True
976
904
            with self.assertParseError():
977
905
                self.check_option_syntax(options)
978
906
 
979
 
    def test_actions_with_client_conflicts_with_verbose(self):
 
907
    def test_actions_conflicts_with_verbose(self):
980
908
        for action, value in self.actions.items():
981
909
            options = self.parser.parse_args()
982
910
            setattr(options, action, value)
983
911
            options.verbose = True
984
 
            options.client = ["foo"]
985
912
            with self.assertParseError():
986
913
                self.check_option_syntax(options)
987
914
 
1013
940
            options.all = True
1014
941
            self.check_option_syntax(options)
1015
942
 
1016
 
    def test_any_action_is_ok_with_one_client(self):
1017
 
        for action, value in self.actions.items():
1018
 
            options = self.parser.parse_args()
1019
 
            setattr(options, action, value)
1020
 
            options.client = ["foo"]
 
943
    def test_is_enabled_fails_without_client(self):
 
944
        options = self.parser.parse_args()
 
945
        options.is_enabled = True
 
946
        with self.assertParseError():
1021
947
            self.check_option_syntax(options)
1022
948
 
1023
 
    def test_one_client_with_all_actions_except_is_enabled(self):
 
949
    def test_is_enabled_works_with_one_client(self):
1024
950
        options = self.parser.parse_args()
1025
 
        for action, value in self.actions.items():
1026
 
            if action == "is_enabled":
1027
 
                continue
1028
 
            setattr(options, action, value)
 
951
        options.is_enabled = True
1029
952
        options.client = ["foo"]
1030
953
        self.check_option_syntax(options)
1031
954
 
1032
 
    def test_two_clients_with_all_actions_except_is_enabled(self):
1033
 
        options = self.parser.parse_args()
1034
 
        for action, value in self.actions.items():
1035
 
            if action == "is_enabled":
1036
 
                continue
1037
 
            setattr(options, action, value)
1038
 
        options.client = ["foo", "barbar"]
1039
 
        self.check_option_syntax(options)
1040
 
 
1041
 
    def test_two_clients_are_ok_with_actions_except_is_enabled(self):
1042
 
        for action, value in self.actions.items():
1043
 
            if action == "is_enabled":
1044
 
                continue
1045
 
            options = self.parser.parse_args()
1046
 
            setattr(options, action, value)
1047
 
            options.client = ["foo", "barbar"]
1048
 
            self.check_option_syntax(options)
1049
 
 
1050
 
    def test_is_enabled_fails_without_client(self):
1051
 
        options = self.parser.parse_args()
1052
 
        options.is_enabled = True
1053
 
        with self.assertParseError():
1054
 
            self.check_option_syntax(options)
1055
 
 
1056
955
    def test_is_enabled_fails_with_two_clients(self):
1057
956
        options = self.parser.parse_args()
1058
957
        options.is_enabled = True
1072
971
                self.check_option_syntax(options)
1073
972
 
1074
973
 
1075
 
class Test_get_mandos_dbus_object(TestCaseWithAssertLogs):
1076
 
    def test_calls_and_returns_get_object_on_bus(self):
1077
 
        class MockBus(object):
1078
 
            called = False
1079
 
            def get_object(mockbus_self, busname, dbus_path):
1080
 
                # Note that "self" is still the testcase instance,
1081
 
                # this MockBus instance is in "mockbus_self".
1082
 
                self.assertEqual(busname, dbus_busname)
1083
 
                self.assertEqual(dbus_path, server_dbus_path)
1084
 
                mockbus_self.called = True
1085
 
                return mockbus_self
1086
 
 
1087
 
        mockbus = get_mandos_dbus_object(bus=MockBus())
1088
 
        self.assertIsInstance(mockbus, MockBus)
1089
 
        self.assertTrue(mockbus.called)
1090
 
 
1091
 
    def test_logs_and_exits_on_dbus_error(self):
1092
 
        class MockBusFailing(object):
1093
 
            def get_object(self, busname, dbus_path):
1094
 
                raise dbus.exceptions.DBusException("Test")
1095
 
 
1096
 
        with self.assertLogs(log, logging.CRITICAL):
1097
 
            with self.assertRaises(SystemExit) as e:
1098
 
                bus = get_mandos_dbus_object(bus=MockBusFailing())
1099
 
 
1100
 
        if isinstance(e.exception.code, int):
1101
 
            self.assertNotEqual(e.exception.code, 0)
1102
 
        else:
1103
 
            self.assertIsNotNone(e.exception.code)
1104
 
 
1105
 
 
1106
 
class Test_get_managed_objects(TestCaseWithAssertLogs):
1107
 
    def test_calls_and_returns_GetManagedObjects(self):
1108
 
        managed_objects = {"/clients/foo": { "Name": "foo"}}
1109
 
        class MockObjectManager(object):
1110
 
            def GetManagedObjects(self):
1111
 
                return managed_objects
1112
 
        retval = get_managed_objects(MockObjectManager())
1113
 
        self.assertDictEqual(managed_objects, retval)
1114
 
 
1115
 
    def test_logs_and_exits_on_dbus_error(self):
1116
 
        dbus_logger = logging.getLogger("dbus.proxies")
1117
 
 
1118
 
        class MockObjectManagerFailing(object):
1119
 
            def GetManagedObjects(self):
1120
 
                dbus_logger.error("Test")
1121
 
                raise dbus.exceptions.DBusException("Test")
1122
 
 
1123
 
        class CountingHandler(logging.Handler):
1124
 
            count = 0
1125
 
            def emit(self, record):
1126
 
                self.count += 1
1127
 
 
1128
 
        counting_handler = CountingHandler()
1129
 
 
1130
 
        dbus_logger.addHandler(counting_handler)
1131
 
 
1132
 
        try:
1133
 
            with self.assertLogs(log, logging.CRITICAL) as watcher:
1134
 
                with self.assertRaises(SystemExit) as e:
1135
 
                    get_managed_objects(MockObjectManagerFailing())
1136
 
        finally:
1137
 
            dbus_logger.removeFilter(counting_handler)
1138
 
 
1139
 
        # Make sure the dbus logger was suppressed
1140
 
        self.assertEqual(counting_handler.count, 0)
1141
 
 
1142
 
        # Test that the dbus_logger still works
1143
 
        with self.assertLogs(dbus_logger, logging.ERROR):
1144
 
            dbus_logger.error("Test")
1145
 
 
1146
 
        if isinstance(e.exception.code, int):
1147
 
            self.assertNotEqual(e.exception.code, 0)
1148
 
        else:
1149
 
            self.assertIsNotNone(e.exception.code)
1150
 
 
1151
 
 
1152
 
class Test_commands_from_options(unittest.TestCase):
 
974
class Test_command_from_options(unittest.TestCase):
1153
975
    def setUp(self):
1154
976
        self.parser = argparse.ArgumentParser()
1155
977
        add_command_line_options(self.parser)
1156
 
 
1157
 
    def test_is_enabled(self):
1158
 
        self.assert_command_from_args(["--is-enabled", "foo"],
1159
 
                                      command.IsEnabled)
1160
 
 
1161
978
    def assert_command_from_args(self, args, command_cls,
1162
979
                                 **cmd_attrs):
1163
980
        """Assert that parsing ARGS should result in an instance of
1170
987
        self.assertIsInstance(command, command_cls)
1171
988
        for key, value in cmd_attrs.items():
1172
989
            self.assertEqual(getattr(command, key), value)
1173
 
 
1174
 
    def test_is_enabled_short(self):
1175
 
        self.assert_command_from_args(["-V", "foo"],
1176
 
                                      command.IsEnabled)
1177
 
 
1178
 
    def test_approve(self):
1179
 
        self.assert_command_from_args(["--approve", "foo"],
1180
 
                                      command.Approve)
1181
 
 
1182
 
    def test_approve_short(self):
1183
 
        self.assert_command_from_args(["-A", "foo"], command.Approve)
1184
 
 
1185
 
    def test_deny(self):
1186
 
        self.assert_command_from_args(["--deny", "foo"], command.Deny)
1187
 
 
1188
 
    def test_deny_short(self):
1189
 
        self.assert_command_from_args(["-D", "foo"], command.Deny)
1190
 
 
1191
 
    def test_remove(self):
1192
 
        self.assert_command_from_args(["--remove", "foo"],
1193
 
                                      command.Remove)
1194
 
 
1195
 
    def test_deny_before_remove(self):
1196
 
        options = self.parser.parse_args(["--deny", "--remove",
1197
 
                                          "foo"])
1198
 
        check_option_syntax(self.parser, options)
1199
 
        commands = commands_from_options(options)
1200
 
        self.assertEqual(len(commands), 2)
1201
 
        self.assertIsInstance(commands[0], command.Deny)
1202
 
        self.assertIsInstance(commands[1], command.Remove)
1203
 
 
1204
 
    def test_deny_before_remove_reversed(self):
1205
 
        options = self.parser.parse_args(["--remove", "--deny",
1206
 
                                          "--all"])
1207
 
        check_option_syntax(self.parser, options)
1208
 
        commands = commands_from_options(options)
1209
 
        self.assertEqual(len(commands), 2)
1210
 
        self.assertIsInstance(commands[0], command.Deny)
1211
 
        self.assertIsInstance(commands[1], command.Remove)
1212
 
 
1213
 
    def test_remove_short(self):
1214
 
        self.assert_command_from_args(["-r", "foo"], command.Remove)
1215
 
 
1216
 
    def test_dump_json(self):
1217
 
        self.assert_command_from_args(["--dump-json"],
1218
 
                                      command.DumpJSON)
 
990
    def test_print_table(self):
 
991
        self.assert_command_from_args([], PrintTableCmd,
 
992
                                      verbose=False)
 
993
 
 
994
    def test_print_table_verbose(self):
 
995
        self.assert_command_from_args(["--verbose"], PrintTableCmd,
 
996
                                      verbose=True)
 
997
 
 
998
    def test_print_table_verbose_short(self):
 
999
        self.assert_command_from_args(["-v"], PrintTableCmd,
 
1000
                                      verbose=True)
1219
1001
 
1220
1002
    def test_enable(self):
1221
 
        self.assert_command_from_args(["--enable", "foo"],
1222
 
                                      command.Enable)
 
1003
        self.assert_command_from_args(["--enable", "foo"], EnableCmd)
1223
1004
 
1224
1005
    def test_enable_short(self):
1225
 
        self.assert_command_from_args(["-e", "foo"], command.Enable)
 
1006
        self.assert_command_from_args(["-e", "foo"], EnableCmd)
1226
1007
 
1227
1008
    def test_disable(self):
1228
1009
        self.assert_command_from_args(["--disable", "foo"],
1229
 
                                      command.Disable)
 
1010
                                      DisableCmd)
1230
1011
 
1231
1012
    def test_disable_short(self):
1232
 
        self.assert_command_from_args(["-d", "foo"], command.Disable)
 
1013
        self.assert_command_from_args(["-d", "foo"], DisableCmd)
1233
1014
 
1234
1015
    def test_bump_timeout(self):
1235
1016
        self.assert_command_from_args(["--bump-timeout", "foo"],
1236
 
                                      command.BumpTimeout)
 
1017
                                      BumpTimeoutCmd)
1237
1018
 
1238
1019
    def test_bump_timeout_short(self):
1239
 
        self.assert_command_from_args(["-b", "foo"],
1240
 
                                      command.BumpTimeout)
 
1020
        self.assert_command_from_args(["-b", "foo"], BumpTimeoutCmd)
1241
1021
 
1242
1022
    def test_start_checker(self):
1243
1023
        self.assert_command_from_args(["--start-checker", "foo"],
1244
 
                                      command.StartChecker)
 
1024
                                      StartCheckerCmd)
1245
1025
 
1246
1026
    def test_stop_checker(self):
1247
1027
        self.assert_command_from_args(["--stop-checker", "foo"],
1248
 
                                      command.StopChecker)
1249
 
 
1250
 
    def test_approve_by_default(self):
1251
 
        self.assert_command_from_args(["--approve-by-default", "foo"],
1252
 
                                      command.ApproveByDefault)
1253
 
 
1254
 
    def test_deny_by_default(self):
1255
 
        self.assert_command_from_args(["--deny-by-default", "foo"],
1256
 
                                      command.DenyByDefault)
 
1028
                                      StopCheckerCmd)
 
1029
 
 
1030
    def test_remove(self):
 
1031
        self.assert_command_from_args(["--remove", "foo"],
 
1032
                                      RemoveCmd)
 
1033
 
 
1034
    def test_remove_short(self):
 
1035
        self.assert_command_from_args(["-r", "foo"], RemoveCmd)
1257
1036
 
1258
1037
    def test_checker(self):
1259
1038
        self.assert_command_from_args(["--checker", ":", "foo"],
1260
 
                                      command.SetChecker,
1261
 
                                      value_to_set=":")
 
1039
                                      SetCheckerCmd, value_to_set=":")
1262
1040
 
1263
1041
    def test_checker_empty(self):
1264
1042
        self.assert_command_from_args(["--checker", "", "foo"],
1265
 
                                      command.SetChecker,
1266
 
                                      value_to_set="")
 
1043
                                      SetCheckerCmd, value_to_set="")
1267
1044
 
1268
1045
    def test_checker_short(self):
1269
1046
        self.assert_command_from_args(["-c", ":", "foo"],
1270
 
                                      command.SetChecker,
1271
 
                                      value_to_set=":")
1272
 
 
1273
 
    def test_host(self):
1274
 
        self.assert_command_from_args(["--host", "foo.example.org",
1275
 
                                       "foo"], command.SetHost,
1276
 
                                      value_to_set="foo.example.org")
1277
 
 
1278
 
    def test_host_short(self):
1279
 
        self.assert_command_from_args(["-H", "foo.example.org",
1280
 
                                       "foo"], command.SetHost,
1281
 
                                      value_to_set="foo.example.org")
1282
 
 
1283
 
    def test_secret_devnull(self):
1284
 
        self.assert_command_from_args(["--secret", os.path.devnull,
1285
 
                                       "foo"], command.SetSecret,
1286
 
                                      value_to_set=b"")
1287
 
 
1288
 
    def test_secret_tempfile(self):
1289
 
        with tempfile.NamedTemporaryFile(mode="r+b") as f:
1290
 
            value = b"secret\0xyzzy\nbar"
1291
 
            f.write(value)
1292
 
            f.seek(0)
1293
 
            self.assert_command_from_args(["--secret", f.name,
1294
 
                                           "foo"], command.SetSecret,
1295
 
                                          value_to_set=value)
1296
 
 
1297
 
    def test_secret_devnull_short(self):
1298
 
        self.assert_command_from_args(["-s", os.path.devnull, "foo"],
1299
 
                                      command.SetSecret,
1300
 
                                      value_to_set=b"")
1301
 
 
1302
 
    def test_secret_tempfile_short(self):
1303
 
        with tempfile.NamedTemporaryFile(mode="r+b") as f:
1304
 
            value = b"secret\0xyzzy\nbar"
1305
 
            f.write(value)
1306
 
            f.seek(0)
1307
 
            self.assert_command_from_args(["-s", f.name, "foo"],
1308
 
                                          command.SetSecret,
1309
 
                                          value_to_set=value)
 
1047
                                      SetCheckerCmd, value_to_set=":")
1310
1048
 
1311
1049
    def test_timeout(self):
1312
1050
        self.assert_command_from_args(["--timeout", "PT5M", "foo"],
1313
 
                                      command.SetTimeout,
 
1051
                                      SetTimeoutCmd,
1314
1052
                                      value_to_set=300000)
1315
1053
 
1316
1054
    def test_timeout_short(self):
1317
1055
        self.assert_command_from_args(["-t", "PT5M", "foo"],
1318
 
                                      command.SetTimeout,
 
1056
                                      SetTimeoutCmd,
1319
1057
                                      value_to_set=300000)
1320
1058
 
1321
1059
    def test_extended_timeout(self):
1322
1060
        self.assert_command_from_args(["--extended-timeout", "PT15M",
1323
1061
                                       "foo"],
1324
 
                                      command.SetExtendedTimeout,
 
1062
                                      SetExtendedTimeoutCmd,
1325
1063
                                      value_to_set=900000)
1326
1064
 
1327
1065
    def test_interval(self):
1328
1066
        self.assert_command_from_args(["--interval", "PT2M", "foo"],
1329
 
                                      command.SetInterval,
 
1067
                                      SetIntervalCmd,
1330
1068
                                      value_to_set=120000)
1331
1069
 
1332
1070
    def test_interval_short(self):
1333
1071
        self.assert_command_from_args(["-i", "PT2M", "foo"],
1334
 
                                      command.SetInterval,
 
1072
                                      SetIntervalCmd,
1335
1073
                                      value_to_set=120000)
1336
1074
 
 
1075
    def test_approve_by_default(self):
 
1076
        self.assert_command_from_args(["--approve-by-default", "foo"],
 
1077
                                      ApproveByDefaultCmd)
 
1078
 
 
1079
    def test_deny_by_default(self):
 
1080
        self.assert_command_from_args(["--deny-by-default", "foo"],
 
1081
                                      DenyByDefaultCmd)
 
1082
 
1337
1083
    def test_approval_delay(self):
1338
1084
        self.assert_command_from_args(["--approval-delay", "PT30S",
1339
 
                                       "foo"],
1340
 
                                      command.SetApprovalDelay,
 
1085
                                       "foo"], SetApprovalDelayCmd,
1341
1086
                                      value_to_set=30000)
1342
1087
 
1343
1088
    def test_approval_duration(self):
1344
1089
        self.assert_command_from_args(["--approval-duration", "PT1S",
1345
 
                                       "foo"],
1346
 
                                      command.SetApprovalDuration,
 
1090
                                       "foo"], SetApprovalDurationCmd,
1347
1091
                                      value_to_set=1000)
1348
1092
 
1349
 
    def test_print_table(self):
1350
 
        self.assert_command_from_args([], command.PrintTable,
1351
 
                                      verbose=False)
1352
 
 
1353
 
    def test_print_table_verbose(self):
1354
 
        self.assert_command_from_args(["--verbose"],
1355
 
                                      command.PrintTable,
1356
 
                                      verbose=True)
1357
 
 
1358
 
    def test_print_table_verbose_short(self):
1359
 
        self.assert_command_from_args(["-v"], command.PrintTable,
1360
 
                                      verbose=True)
1361
 
 
1362
 
 
1363
 
class TestCommand(unittest.TestCase):
 
1093
    def test_host(self):
 
1094
        self.assert_command_from_args(["--host", "foo.example.org",
 
1095
                                       "foo"], SetHostCmd,
 
1096
                                      value_to_set="foo.example.org")
 
1097
 
 
1098
    def test_host_short(self):
 
1099
        self.assert_command_from_args(["-H", "foo.example.org",
 
1100
                                       "foo"], SetHostCmd,
 
1101
                                      value_to_set="foo.example.org")
 
1102
 
 
1103
    def test_secret_devnull(self):
 
1104
        self.assert_command_from_args(["--secret", os.path.devnull,
 
1105
                                       "foo"], SetSecretCmd,
 
1106
                                      value_to_set=b"")
 
1107
 
 
1108
    def test_secret_tempfile(self):
 
1109
        with tempfile.NamedTemporaryFile(mode="r+b") as f:
 
1110
            value = b"secret\0xyzzy\nbar"
 
1111
            f.write(value)
 
1112
            f.seek(0)
 
1113
            self.assert_command_from_args(["--secret", f.name,
 
1114
                                           "foo"], SetSecretCmd,
 
1115
                                          value_to_set=value)
 
1116
 
 
1117
    def test_secret_devnull_short(self):
 
1118
        self.assert_command_from_args(["-s", os.path.devnull, "foo"],
 
1119
                                      SetSecretCmd, value_to_set=b"")
 
1120
 
 
1121
    def test_secret_tempfile_short(self):
 
1122
        with tempfile.NamedTemporaryFile(mode="r+b") as f:
 
1123
            value = b"secret\0xyzzy\nbar"
 
1124
            f.write(value)
 
1125
            f.seek(0)
 
1126
            self.assert_command_from_args(["-s", f.name, "foo"],
 
1127
                                          SetSecretCmd,
 
1128
                                          value_to_set=value)
 
1129
 
 
1130
    def test_approve(self):
 
1131
        self.assert_command_from_args(["--approve", "foo"],
 
1132
                                      ApproveCmd)
 
1133
 
 
1134
    def test_approve_short(self):
 
1135
        self.assert_command_from_args(["-A", "foo"], ApproveCmd)
 
1136
 
 
1137
    def test_deny(self):
 
1138
        self.assert_command_from_args(["--deny", "foo"], DenyCmd)
 
1139
 
 
1140
    def test_deny_short(self):
 
1141
        self.assert_command_from_args(["-D", "foo"], DenyCmd)
 
1142
 
 
1143
    def test_dump_json(self):
 
1144
        self.assert_command_from_args(["--dump-json"], DumpJSONCmd)
 
1145
 
 
1146
    def test_is_enabled(self):
 
1147
        self.assert_command_from_args(["--is-enabled", "foo"],
 
1148
                                      IsEnabledCmd)
 
1149
 
 
1150
    def test_is_enabled_short(self):
 
1151
        self.assert_command_from_args(["-V", "foo"], IsEnabledCmd)
 
1152
 
 
1153
    def test_deny_before_remove(self):
 
1154
        options = self.parser.parse_args(["--deny", "--remove",
 
1155
                                          "foo"])
 
1156
        check_option_syntax(self.parser, options)
 
1157
        commands = commands_from_options(options)
 
1158
        self.assertEqual(len(commands), 2)
 
1159
        self.assertIsInstance(commands[0], DenyCmd)
 
1160
        self.assertIsInstance(commands[1], RemoveCmd)
 
1161
 
 
1162
    def test_deny_before_remove_reversed(self):
 
1163
        options = self.parser.parse_args(["--remove", "--deny",
 
1164
                                          "--all"])
 
1165
        check_option_syntax(self.parser, options)
 
1166
        commands = commands_from_options(options)
 
1167
        self.assertEqual(len(commands), 2)
 
1168
        self.assertIsInstance(commands[0], DenyCmd)
 
1169
        self.assertIsInstance(commands[1], RemoveCmd)
 
1170
 
 
1171
 
 
1172
class TestCmd(unittest.TestCase):
1364
1173
    """Abstract class for tests of command classes"""
1365
 
 
1366
1174
    def setUp(self):
1367
1175
        testcase = self
1368
1176
        class MockClient(object):
1440
1248
                ("/clients/barbar", self.other_client.attributes),
1441
1249
            ])
1442
1250
        self.one_client = {"/clients/foo": self.client.attributes}
1443
 
 
1444
1251
    @property
1445
1252
    def bus(self):
1446
1253
        class Bus(object):
1448
1255
            def get_object(client_bus_name, path):
1449
1256
                self.assertEqual(client_bus_name, dbus_busname)
1450
1257
                return {
1451
 
                    # Note: "self" here is the TestCmd instance, not
1452
 
                    # the Bus instance, since this is a static method!
1453
1258
                    "/clients/foo": self.client,
1454
1259
                    "/clients/barbar": self.other_client,
1455
1260
                }[path]
1456
1261
        return Bus()
1457
1262
 
1458
1263
 
1459
 
class TestBaseCommands(TestCommand):
1460
 
 
 
1264
class TestIsEnabledCmd(TestCmd):
1461
1265
    def test_is_enabled(self):
1462
 
        self.assertTrue(all(command.IsEnabled().is_enabled(client,
 
1266
        self.assertTrue(all(IsEnabledCmd().is_enabled(client,
1463
1267
                                                      properties)
1464
1268
                            for client, properties
1465
1269
                            in self.clients.items()))
1466
 
 
1467
1270
    def test_is_enabled_run_exits_successfully(self):
1468
1271
        with self.assertRaises(SystemExit) as e:
1469
 
            command.IsEnabled().run(self.one_client)
 
1272
            IsEnabledCmd().run(self.one_client)
1470
1273
        if e.exception.code is not None:
1471
1274
            self.assertEqual(e.exception.code, 0)
1472
1275
        else:
1473
1276
            self.assertIsNone(e.exception.code)
1474
 
 
1475
1277
    def test_is_enabled_run_exits_with_failure(self):
1476
1278
        self.client.attributes["Enabled"] = dbus.Boolean(False)
1477
1279
        with self.assertRaises(SystemExit) as e:
1478
 
            command.IsEnabled().run(self.one_client)
 
1280
            IsEnabledCmd().run(self.one_client)
1479
1281
        if isinstance(e.exception.code, int):
1480
1282
            self.assertNotEqual(e.exception.code, 0)
1481
1283
        else:
1482
1284
            self.assertIsNotNone(e.exception.code)
1483
1285
 
 
1286
 
 
1287
class TestApproveCmd(TestCmd):
1484
1288
    def test_approve(self):
1485
 
        command.Approve().run(self.clients, self.bus)
 
1289
        ApproveCmd().run(self.clients, self.bus)
1486
1290
        for clientpath in self.clients:
1487
1291
            client = self.bus.get_object(dbus_busname, clientpath)
1488
1292
            self.assertIn(("Approve", (True, client_dbus_interface)),
1489
1293
                          client.calls)
1490
1294
 
 
1295
 
 
1296
class TestDenyCmd(TestCmd):
1491
1297
    def test_deny(self):
1492
 
        command.Deny().run(self.clients, self.bus)
 
1298
        DenyCmd().run(self.clients, self.bus)
1493
1299
        for clientpath in self.clients:
1494
1300
            client = self.bus.get_object(dbus_busname, clientpath)
1495
1301
            self.assertIn(("Approve", (False, client_dbus_interface)),
1496
1302
                          client.calls)
1497
1303
 
 
1304
class TestRemoveCmd(TestCmd):
1498
1305
    def test_remove(self):
1499
1306
        class MockMandos(object):
1500
1307
            def __init__(self):
1502
1309
            def RemoveClient(self, dbus_path):
1503
1310
                self.calls.append(("RemoveClient", (dbus_path,)))
1504
1311
        mandos = MockMandos()
1505
 
        super(TestBaseCommands, self).setUp()
1506
 
        command.Remove().run(self.clients, self.bus, mandos)
 
1312
        super(TestRemoveCmd, self).setUp()
 
1313
        RemoveCmd().run(self.clients, self.bus, mandos)
1507
1314
        self.assertEqual(len(mandos.calls), 2)
1508
1315
        for clientpath in self.clients:
1509
1316
            self.assertIn(("RemoveClient", (clientpath,)),
1510
1317
                          mandos.calls)
1511
1318
 
1512
 
    expected_json = {
1513
 
        "foo": {
1514
 
            "Name": "foo",
1515
 
            "KeyID": ("92ed150794387c03ce684574b1139a65"
1516
 
                      "94a34f895daaaf09fd8ea90a27cddb12"),
1517
 
            "Host": "foo.example.org",
1518
 
            "Enabled": True,
1519
 
            "Timeout": 300000,
1520
 
            "LastCheckedOK": "2019-02-03T00:00:00",
1521
 
            "Created": "2019-01-02T00:00:00",
1522
 
            "Interval": 120000,
1523
 
            "Fingerprint": ("778827225BA7DE539C5A"
1524
 
                            "7CFA59CFF7CDBD9A5920"),
1525
 
            "CheckerRunning": False,
1526
 
            "LastEnabled": "2019-01-03T00:00:00",
1527
 
            "ApprovalPending": False,
1528
 
            "ApprovedByDefault": True,
1529
 
            "LastApprovalRequest": "",
1530
 
            "ApprovalDelay": 0,
1531
 
            "ApprovalDuration": 1000,
1532
 
            "Checker": "fping -q -- %(host)s",
1533
 
            "ExtendedTimeout": 900000,
1534
 
            "Expires": "2019-02-04T00:00:00",
1535
 
            "LastCheckerStatus": 0,
1536
 
        },
1537
 
        "barbar": {
1538
 
            "Name": "barbar",
1539
 
            "KeyID": ("0558568eedd67d622f5c83b35a115f79"
1540
 
                      "6ab612cff5ad227247e46c2b020f441c"),
1541
 
            "Host": "192.0.2.3",
1542
 
            "Enabled": True,
1543
 
            "Timeout": 300000,
1544
 
            "LastCheckedOK": "2019-02-04T00:00:00",
1545
 
            "Created": "2019-01-03T00:00:00",
1546
 
            "Interval": 120000,
1547
 
            "Fingerprint": ("3E393AEAEFB84C7E89E2"
1548
 
                            "F547B3A107558FCA3A27"),
1549
 
            "CheckerRunning": True,
1550
 
            "LastEnabled": "2019-01-04T00:00:00",
1551
 
            "ApprovalPending": False,
1552
 
            "ApprovedByDefault": False,
1553
 
            "LastApprovalRequest": "2019-01-03T00:00:00",
1554
 
            "ApprovalDelay": 30000,
1555
 
            "ApprovalDuration": 93785000,
1556
 
            "Checker": ":",
1557
 
            "ExtendedTimeout": 900000,
1558
 
            "Expires": "2019-02-05T00:00:00",
1559
 
            "LastCheckerStatus": -2,
1560
 
        },
1561
 
    }
1562
1319
 
1563
 
    def test_DumpJSON_normal(self):
1564
 
        output = command.DumpJSON().output(self.clients.values())
1565
 
        json_data = json.loads(output)
 
1320
class TestDumpJSONCmd(TestCmd):
 
1321
    def setUp(self):
 
1322
        self.expected_json = {
 
1323
            "foo": {
 
1324
                "Name": "foo",
 
1325
                "KeyID": ("92ed150794387c03ce684574b1139a65"
 
1326
                          "94a34f895daaaf09fd8ea90a27cddb12"),
 
1327
                "Host": "foo.example.org",
 
1328
                "Enabled": True,
 
1329
                "Timeout": 300000,
 
1330
                "LastCheckedOK": "2019-02-03T00:00:00",
 
1331
                "Created": "2019-01-02T00:00:00",
 
1332
                "Interval": 120000,
 
1333
                "Fingerprint": ("778827225BA7DE539C5A"
 
1334
                                "7CFA59CFF7CDBD9A5920"),
 
1335
                "CheckerRunning": False,
 
1336
                "LastEnabled": "2019-01-03T00:00:00",
 
1337
                "ApprovalPending": False,
 
1338
                "ApprovedByDefault": True,
 
1339
                "LastApprovalRequest": "",
 
1340
                "ApprovalDelay": 0,
 
1341
                "ApprovalDuration": 1000,
 
1342
                "Checker": "fping -q -- %(host)s",
 
1343
                "ExtendedTimeout": 900000,
 
1344
                "Expires": "2019-02-04T00:00:00",
 
1345
                "LastCheckerStatus": 0,
 
1346
            },
 
1347
            "barbar": {
 
1348
                "Name": "barbar",
 
1349
                "KeyID": ("0558568eedd67d622f5c83b35a115f79"
 
1350
                          "6ab612cff5ad227247e46c2b020f441c"),
 
1351
                "Host": "192.0.2.3",
 
1352
                "Enabled": True,
 
1353
                "Timeout": 300000,
 
1354
                "LastCheckedOK": "2019-02-04T00:00:00",
 
1355
                "Created": "2019-01-03T00:00:00",
 
1356
                "Interval": 120000,
 
1357
                "Fingerprint": ("3E393AEAEFB84C7E89E2"
 
1358
                                "F547B3A107558FCA3A27"),
 
1359
                "CheckerRunning": True,
 
1360
                "LastEnabled": "2019-01-04T00:00:00",
 
1361
                "ApprovalPending": False,
 
1362
                "ApprovedByDefault": False,
 
1363
                "LastApprovalRequest": "2019-01-03T00:00:00",
 
1364
                "ApprovalDelay": 30000,
 
1365
                "ApprovalDuration": 93785000,
 
1366
                "Checker": ":",
 
1367
                "ExtendedTimeout": 900000,
 
1368
                "Expires": "2019-02-05T00:00:00",
 
1369
                "LastCheckerStatus": -2,
 
1370
            },
 
1371
        }
 
1372
        return super(TestDumpJSONCmd, self).setUp()
 
1373
    def test_normal(self):
 
1374
        json_data = json.loads(DumpJSONCmd().output(self.clients))
1566
1375
        self.assertDictEqual(json_data, self.expected_json)
1567
 
 
1568
 
    def test_DumpJSON_one_client(self):
1569
 
        output = command.DumpJSON().output(self.one_client.values())
1570
 
        json_data = json.loads(output)
 
1376
    def test_one_client(self):
 
1377
        clients = self.one_client
 
1378
        json_data = json.loads(DumpJSONCmd().output(clients))
1571
1379
        expected_json = {"foo": self.expected_json["foo"]}
1572
1380
        self.assertDictEqual(json_data, expected_json)
1573
1381
 
1574
 
    def test_PrintTable_normal(self):
1575
 
        output = command.PrintTable().output(self.clients.values())
 
1382
 
 
1383
class TestPrintTableCmd(TestCmd):
 
1384
    def test_normal(self):
 
1385
        output = PrintTableCmd().output(self.clients.values())
1576
1386
        expected_output = "\n".join((
1577
1387
            "Name   Enabled Timeout  Last Successful Check",
1578
1388
            "foo    Yes     00:05:00 2019-02-03T00:00:00  ",
1579
1389
            "barbar Yes     00:05:00 2019-02-04T00:00:00  ",
1580
1390
        ))
1581
1391
        self.assertEqual(output, expected_output)
1582
 
 
1583
 
    def test_PrintTable_verbose(self):
1584
 
        output = command.PrintTable(verbose=True).output(
 
1392
    def test_verbose(self):
 
1393
        output = PrintTableCmd(verbose=True).output(
1585
1394
            self.clients.values())
1586
1395
        columns = (
1587
1396
            (
1674
1483
                                            for rows in columns)
1675
1484
                                    for line in range(num_lines))
1676
1485
        self.assertEqual(output, expected_output)
1677
 
 
1678
 
    def test_PrintTable_one_client(self):
1679
 
        output = command.PrintTable().output(self.one_client.values())
1680
 
        expected_output = "\n".join((
1681
 
            "Name Enabled Timeout  Last Successful Check",
1682
 
            "foo  Yes     00:05:00 2019-02-03T00:00:00  ",
1683
 
        ))
 
1486
    def test_one_client(self):
 
1487
        output = PrintTableCmd().output(self.one_client.values())
 
1488
        expected_output = """
 
1489
Name Enabled Timeout  Last Successful Check
 
1490
foo  Yes     00:05:00 2019-02-03T00:00:00  
 
1491
"""[1:-1]
1684
1492
        self.assertEqual(output, expected_output)
1685
1493
 
1686
1494
 
1687
 
class TestPropertyCmd(TestCommand):
1688
 
    """Abstract class for tests of command.Property classes"""
 
1495
class Unique(object):
 
1496
    """Class for objects which exist only to be unique objects, since
 
1497
unittest.mock.sentinel only exists in Python 3.3"""
 
1498
 
 
1499
 
 
1500
class TestPropertyCmd(TestCmd):
 
1501
    """Abstract class for tests of PropertyCmd classes"""
1689
1502
    def runTest(self):
1690
1503
        if not hasattr(self, "command"):
1691
1504
            return
1696
1509
            for clientpath in self.clients:
1697
1510
                client = self.bus.get_object(dbus_busname, clientpath)
1698
1511
                old_value = client.attributes[self.propname]
1699
 
                self.assertNotIsInstance(old_value, self.Unique)
1700
 
                client.attributes[self.propname] = self.Unique()
 
1512
                self.assertNotIsInstance(old_value, Unique)
 
1513
                client.attributes[self.propname] = Unique()
1701
1514
            self.run_command(value_to_set, self.clients)
1702
1515
            for clientpath in self.clients:
1703
1516
                client = self.bus.get_object(dbus_busname, clientpath)
1704
1517
                value = client.attributes[self.propname]
1705
 
                self.assertNotIsInstance(value, self.Unique)
 
1518
                self.assertNotIsInstance(value, Unique)
1706
1519
                self.assertEqual(value, value_to_get)
1707
 
 
1708
 
    class Unique(object):
1709
 
        """Class for objects which exist only to be unique objects,
1710
 
since unittest.mock.sentinel only exists in Python 3.3"""
1711
 
 
1712
1520
    def run_command(self, value, clients):
1713
1521
        self.command().run(clients, self.bus)
1714
1522
 
1715
1523
 
1716
 
class TestEnableCmd(TestPropertyCmd):
1717
 
    command = command.Enable
1718
 
    propname = "Enabled"
1719
 
    values_to_set = [dbus.Boolean(True)]
1720
 
 
1721
 
 
1722
 
class TestDisableCmd(TestPropertyCmd):
1723
 
    command = command.Disable
1724
 
    propname = "Enabled"
1725
 
    values_to_set = [dbus.Boolean(False)]
 
1524
class TestEnableCmd(TestCmd):
 
1525
    def test_enable(self):
 
1526
        for clientpath in self.clients:
 
1527
            client = self.bus.get_object(dbus_busname, clientpath)
 
1528
            client.attributes["Enabled"] = False
 
1529
 
 
1530
        EnableCmd().run(self.clients, self.bus)
 
1531
 
 
1532
        for clientpath in self.clients:
 
1533
            client = self.bus.get_object(dbus_busname, clientpath)
 
1534
            self.assertTrue(client.attributes["Enabled"])
 
1535
 
 
1536
 
 
1537
class TestDisableCmd(TestCmd):
 
1538
    def test_disable(self):
 
1539
        DisableCmd().run(self.clients, self.bus)
 
1540
        for clientpath in self.clients:
 
1541
            client = self.bus.get_object(dbus_busname, clientpath)
 
1542
            self.assertFalse(client.attributes["Enabled"])
1726
1543
 
1727
1544
 
1728
1545
class TestBumpTimeoutCmd(TestPropertyCmd):
1729
 
    command = command.BumpTimeout
 
1546
    command = BumpTimeoutCmd
1730
1547
    propname = "LastCheckedOK"
1731
1548
    values_to_set = [""]
1732
1549
 
1733
1550
 
1734
1551
class TestStartCheckerCmd(TestPropertyCmd):
1735
 
    command = command.StartChecker
 
1552
    command = StartCheckerCmd
1736
1553
    propname = "CheckerRunning"
1737
1554
    values_to_set = [dbus.Boolean(True)]
1738
1555
 
1739
1556
 
1740
1557
class TestStopCheckerCmd(TestPropertyCmd):
1741
 
    command = command.StopChecker
 
1558
    command = StopCheckerCmd
1742
1559
    propname = "CheckerRunning"
1743
1560
    values_to_set = [dbus.Boolean(False)]
1744
1561
 
1745
1562
 
1746
1563
class TestApproveByDefaultCmd(TestPropertyCmd):
1747
 
    command = command.ApproveByDefault
 
1564
    command = ApproveByDefaultCmd
1748
1565
    propname = "ApprovedByDefault"
1749
1566
    values_to_set = [dbus.Boolean(True)]
1750
1567
 
1751
1568
 
1752
1569
class TestDenyByDefaultCmd(TestPropertyCmd):
1753
 
    command = command.DenyByDefault
 
1570
    command = DenyByDefaultCmd
1754
1571
    propname = "ApprovedByDefault"
1755
1572
    values_to_set = [dbus.Boolean(False)]
1756
1573
 
1757
1574
 
1758
1575
class TestPropertyValueCmd(TestPropertyCmd):
1759
1576
    """Abstract class for tests of PropertyValueCmd classes"""
1760
 
 
1761
1577
    def runTest(self):
1762
1578
        if type(self) is TestPropertyValueCmd:
1763
1579
            return
1764
1580
        return super(TestPropertyValueCmd, self).runTest()
1765
 
 
1766
1581
    def run_command(self, value, clients):
1767
1582
        self.command(value).run(clients, self.bus)
1768
1583
 
1769
1584
 
1770
1585
class TestSetCheckerCmd(TestPropertyValueCmd):
1771
 
    command = command.SetChecker
 
1586
    command = SetCheckerCmd
1772
1587
    propname = "Checker"
1773
1588
    values_to_set = ["", ":", "fping -q -- %s"]
1774
1589
 
1775
1590
 
1776
1591
class TestSetHostCmd(TestPropertyValueCmd):
1777
 
    command = command.SetHost
 
1592
    command = SetHostCmd
1778
1593
    propname = "Host"
1779
1594
    values_to_set = ["192.0.2.3", "foo.example.org"]
1780
1595
 
1781
1596
 
1782
1597
class TestSetSecretCmd(TestPropertyValueCmd):
1783
 
    command = command.SetSecret
 
1598
    command = SetSecretCmd
1784
1599
    propname = "Secret"
1785
1600
    values_to_set = [io.BytesIO(b""),
1786
1601
                     io.BytesIO(b"secret\0xyzzy\nbar")]
1788
1603
 
1789
1604
 
1790
1605
class TestSetTimeoutCmd(TestPropertyValueCmd):
1791
 
    command = command.SetTimeout
 
1606
    command = SetTimeoutCmd
1792
1607
    propname = "Timeout"
1793
1608
    values_to_set = [datetime.timedelta(),
1794
1609
                     datetime.timedelta(minutes=5),
1799
1614
 
1800
1615
 
1801
1616
class TestSetExtendedTimeoutCmd(TestPropertyValueCmd):
1802
 
    command = command.SetExtendedTimeout
 
1617
    command = SetExtendedTimeoutCmd
1803
1618
    propname = "ExtendedTimeout"
1804
1619
    values_to_set = [datetime.timedelta(),
1805
1620
                     datetime.timedelta(minutes=5),
1810
1625
 
1811
1626
 
1812
1627
class TestSetIntervalCmd(TestPropertyValueCmd):
1813
 
    command = command.SetInterval
 
1628
    command = SetIntervalCmd
1814
1629
    propname = "Interval"
1815
1630
    values_to_set = [datetime.timedelta(),
1816
1631
                     datetime.timedelta(minutes=5),
1821
1636
 
1822
1637
 
1823
1638
class TestSetApprovalDelayCmd(TestPropertyValueCmd):
1824
 
    command = command.SetApprovalDelay
 
1639
    command = SetApprovalDelayCmd
1825
1640
    propname = "ApprovalDelay"
1826
1641
    values_to_set = [datetime.timedelta(),
1827
1642
                     datetime.timedelta(minutes=5),
1832
1647
 
1833
1648
 
1834
1649
class TestSetApprovalDurationCmd(TestPropertyValueCmd):
1835
 
    command = command.SetApprovalDuration
 
1650
    command = SetApprovalDurationCmd
1836
1651
    propname = "ApprovalDuration"
1837
1652
    values_to_set = [datetime.timedelta(),
1838
1653
                     datetime.timedelta(minutes=5),
1860
1675
    return tests
1861
1676
 
1862
1677
if __name__ == "__main__":
1863
 
    try:
1864
 
        if should_only_run_tests():
1865
 
            # Call using ./tdd-python-script --check [--verbose]
1866
 
            unittest.main()
1867
 
        else:
1868
 
            main()
1869
 
    finally:
1870
 
        logging.shutdown()
 
1678
    if should_only_run_tests():
 
1679
        # Call using ./tdd-python-script --check [--verbose]
 
1680
        unittest.main()
 
1681
    else:
 
1682
        main()