Skip to content

Commit 76d3c72

Browse files
garethgreenawaydwoz
authored andcommitted
fixses and a skip added to test_mac_shadow.py test_mac_softwareupdate.py
1 parent 7ab204f commit 76d3c72

File tree

2 files changed

+5
-4
lines changed

2 files changed

+5
-4
lines changed

tests/pytests/functional/modules/test_mac_shadow.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ def test_get_account_created(shadow, accounts):
4949
"""
5050
# Correct Functionality
5151
text_date = shadow.get_account_created(accounts.created)
52-
assert text_date.data != "Invalid Timestamp"
52+
assert text_date != "Invalid Timestamp"
5353
obj_date = datetime.datetime.strptime(text_date, "%Y-%m-%d %H:%M:%S")
5454
assert isinstance(obj_date, datetime.date)
5555

tests/pytests/functional/modules/test_mac_softwareupdate.py

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ def softwareupdate(modules):
2222
def _setup_teardown_vars(softwareupdate):
2323
IGNORED_LIST = softwareupdate.list_ignored()
2424

25-
SCHEDULE = softwareupdate.schedule()
25+
SCHEDULE = softwareupdate.schedule_enabled()
2626

2727
CATALOG = softwareupdate.get_catalog()
2828

@@ -35,7 +35,7 @@ def _setup_teardown_vars(softwareupdate):
3535
else:
3636
softwareupdate.reset_ignored()
3737

38-
softwareupdate.schedule(SCHEDULE)
38+
softwareupdate.schedule_enable(SCHEDULE)
3939

4040
if CATALOG == "Default":
4141
softwareupdate.reset_catalog()
@@ -53,6 +53,7 @@ def test_list_available(softwareupdate):
5353
assert isinstance(ret, dict)
5454

5555

56+
@pytest.mark.skip(reason="Ignore removed from latest OS X.")
5657
def test_ignore(softwareupdate):
5758
"""
5859
Test softwareupdate.ignore
@@ -95,7 +96,7 @@ def test_schedule(softwareupdate):
9596

9697
# Test disable in case it was already enabled
9798
ret = softwareupdate.schedule_enable(False)
98-
assert ret
99+
assert not ret
99100

100101
ret = softwareupdate.schedule_enabled()
101102
assert not ret

0 commit comments

Comments
 (0)