Skip to content

Commit 71c4027

Browse files
committed
fix(Findings): createTime and updateTime in NoteOptions
1 parent a9e6d32 commit 71c4027

File tree

2 files changed

+60
-4
lines changed

2 files changed

+60
-4
lines changed

ibm_scc/findings_v1.py

Lines changed: 32 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -230,6 +230,8 @@ def create_note(self,
230230
reported_by: 'Reporter',
231231
*,
232232
related_url: List['ApiNoteRelatedUrl'] = None,
233+
create_time: datetime = None,
234+
update_time: datetime = None,
233235
shared: bool = None,
234236
finding: 'FindingType' = None,
235237
kpi: 'KpiType' = None,
@@ -262,6 +264,10 @@ def create_note(self,
262264
:param str id: The ID of the note.
263265
:param Reporter reported_by: The entity reporting a note.
264266
:param List[ApiNoteRelatedUrl] related_url: (optional)
267+
:param datetime create_time: (optional) Output only. The time this note was
268+
created. This field can be used as a filter in list requests.
269+
:param datetime update_time: (optional) Output only. The time this note was
270+
last updated. This field can be used as a filter in list requests.
265271
:param bool shared: (optional) True if this note can be shared by multiple
266272
accounts.
267273
:param FindingType finding: (optional) FindingType provides details about a
@@ -293,6 +299,10 @@ def create_note(self,
293299
reported_by = convert_model(reported_by)
294300
if related_url is not None:
295301
related_url = [convert_model(x) for x in related_url]
302+
if create_time is not None:
303+
create_time = datetime_to_string(create_time)
304+
if update_time is not None:
305+
update_time = datetime_to_string(update_time)
296306
if finding is not None:
297307
finding = convert_model(finding)
298308
if kpi is not None:
@@ -316,6 +326,8 @@ def create_note(self,
316326
'id': id,
317327
'reported_by': reported_by,
318328
'related_url': related_url,
329+
'create_time': create_time,
330+
'update_time': update_time,
319331
'shared': shared,
320332
'finding': finding,
321333
'kpi': kpi,
@@ -462,6 +474,8 @@ def update_note(self,
462474
reported_by: 'Reporter',
463475
*,
464476
related_url: List['ApiNoteRelatedUrl'] = None,
477+
create_time: datetime = None,
478+
update_time: datetime = None,
465479
shared: bool = None,
466480
finding: 'FindingType' = None,
467481
kpi: 'KpiType' = None,
@@ -493,6 +507,10 @@ def update_note(self,
493507
:param str id: The ID of the note.
494508
:param Reporter reported_by: The entity reporting a note.
495509
:param List[ApiNoteRelatedUrl] related_url: (optional)
510+
:param datetime create_time: (optional) Output only. The time this note was
511+
created. This field can be used as a filter in list requests.
512+
:param datetime update_time: (optional) Output only. The time this note was
513+
last updated. This field can be used as a filter in list requests.
496514
:param bool shared: (optional) True if this note can be shared by multiple
497515
accounts.
498516
:param FindingType finding: (optional) FindingType provides details about a
@@ -526,6 +544,10 @@ def update_note(self,
526544
reported_by = convert_model(reported_by)
527545
if related_url is not None:
528546
related_url = [convert_model(x) for x in related_url]
547+
if create_time is not None:
548+
create_time = datetime_to_string(create_time)
549+
if update_time is not None:
550+
update_time = datetime_to_string(update_time)
529551
if finding is not None:
530552
finding = convert_model(finding)
531553
if kpi is not None:
@@ -549,6 +571,8 @@ def update_note(self,
549571
'id': id,
550572
'reported_by': reported_by,
551573
'related_url': related_url,
574+
'create_time': create_time,
575+
'update_time': update_time,
552576
'shared': shared,
553577
'finding': finding,
554578
'kpi': kpi,
@@ -2682,6 +2706,10 @@ def __init__(self,
26822706
:param str id: The ID of the note.
26832707
:param Reporter reported_by: The entity reporting a note.
26842708
:param List[ApiNoteRelatedUrl] related_url: (optional)
2709+
:param datetime create_time: (optional) Output only. The time this note was
2710+
created. This field can be used as a filter in list requests.
2711+
:param datetime update_time: (optional) Output only. The time this note was
2712+
last updated. This field can be used as a filter in list requests.
26852713
:param bool shared: (optional) True if this note can be shared by multiple
26862714
accounts.
26872715
:param FindingType finding: (optional) FindingType provides details about a
@@ -2764,10 +2792,10 @@ def to_dict(self) -> Dict:
27642792
_dict['kind'] = self.kind
27652793
if hasattr(self, 'related_url') and self.related_url is not None:
27662794
_dict['related_url'] = [x.to_dict() for x in self.related_url]
2767-
if hasattr(self, 'create_time') and getattr(self, 'create_time') is not None:
2768-
_dict['create_time'] = datetime_to_string(getattr(self, 'create_time'))
2769-
if hasattr(self, 'update_time') and getattr(self, 'update_time') is not None:
2770-
_dict['update_time'] = datetime_to_string(getattr(self, 'update_time'))
2795+
if hasattr(self, 'create_time') and self.create_time is not None:
2796+
_dict['create_time'] = datetime_to_string(self.create_time)
2797+
if hasattr(self, 'update_time') and self.update_time is not None:
2798+
_dict['update_time'] = datetime_to_string(self.update_time)
27712799
if hasattr(self, 'id') and self.id is not None:
27722800
_dict['id'] = self.id
27732801
if hasattr(self, 'shared') and self.shared is not None:

test/unit/test_findings_v1.py

Lines changed: 28 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -487,6 +487,8 @@ def test_create_note_all_params(self):
487487
id = 'testString'
488488
reported_by = reporter_model
489489
related_url = [api_note_related_url_model]
490+
create_time = string_to_datetime('2019-01-01T12:00:00.000Z')
491+
update_time = string_to_datetime('2019-01-01T12:00:00.000Z')
490492
shared = True
491493
finding = finding_type_model
492494
kpi = kpi_type_model
@@ -503,6 +505,8 @@ def test_create_note_all_params(self):
503505
id,
504506
reported_by,
505507
related_url=related_url,
508+
create_time=create_time,
509+
update_time=update_time,
506510
shared=shared,
507511
finding=finding,
508512
kpi=kpi,
@@ -523,6 +527,8 @@ def test_create_note_all_params(self):
523527
assert req_body['id'] == 'testString'
524528
assert req_body['reported_by'] == reporter_model
525529
assert req_body['related_url'] == [api_note_related_url_model]
530+
assert req_body['create_time'] == "2019-01-01T12:00:00Z"
531+
assert req_body['update_time'] == "2019-01-01T12:00:00Z"
526532
assert req_body['shared'] == True
527533
assert req_body['finding'] == finding_type_model
528534
assert req_body['kpi'] == kpi_type_model
@@ -617,6 +623,8 @@ def test_create_note_required_params(self):
617623
id = 'testString'
618624
reported_by = reporter_model
619625
related_url = [api_note_related_url_model]
626+
create_time = string_to_datetime('2019-01-01T12:00:00.000Z')
627+
update_time = string_to_datetime('2019-01-01T12:00:00.000Z')
620628
shared = True
621629
finding = finding_type_model
622630
kpi = kpi_type_model
@@ -632,6 +640,8 @@ def test_create_note_required_params(self):
632640
id,
633641
reported_by,
634642
related_url=related_url,
643+
create_time=create_time,
644+
update_time=update_time,
635645
shared=shared,
636646
finding=finding,
637647
kpi=kpi,
@@ -651,6 +661,8 @@ def test_create_note_required_params(self):
651661
assert req_body['id'] == 'testString'
652662
assert req_body['reported_by'] == reporter_model
653663
assert req_body['related_url'] == [api_note_related_url_model]
664+
assert req_body['create_time'] == "2019-01-01T12:00:00Z"
665+
assert req_body['update_time'] == "2019-01-01T12:00:00Z"
654666
assert req_body['shared'] == True
655667
assert req_body['finding'] == finding_type_model
656668
assert req_body['kpi'] == kpi_type_model
@@ -745,6 +757,8 @@ def test_create_note_value_error(self):
745757
id = 'testString'
746758
reported_by = reporter_model
747759
related_url = [api_note_related_url_model]
760+
create_time = string_to_datetime('2019-01-01T12:00:00.000Z')
761+
update_time = string_to_datetime('2019-01-01T12:00:00.000Z')
748762
shared = True
749763
finding = finding_type_model
750764
kpi = kpi_type_model
@@ -1138,6 +1152,8 @@ def test_update_note_all_params(self):
11381152
id = 'testString'
11391153
reported_by = reporter_model
11401154
related_url = [api_note_related_url_model]
1155+
create_time = string_to_datetime('2019-01-01T12:00:00.000Z')
1156+
update_time = string_to_datetime('2019-01-01T12:00:00.000Z')
11411157
shared = True
11421158
finding = finding_type_model
11431159
kpi = kpi_type_model
@@ -1155,6 +1171,8 @@ def test_update_note_all_params(self):
11551171
id,
11561172
reported_by,
11571173
related_url=related_url,
1174+
create_time=create_time,
1175+
update_time=update_time,
11581176
shared=shared,
11591177
finding=finding,
11601178
kpi=kpi,
@@ -1175,6 +1193,8 @@ def test_update_note_all_params(self):
11751193
assert req_body['id'] == 'testString'
11761194
assert req_body['reported_by'] == reporter_model
11771195
assert req_body['related_url'] == [api_note_related_url_model]
1196+
assert req_body['create_time'] == "2019-01-01T12:00:00Z"
1197+
assert req_body['update_time'] == "2019-01-01T12:00:00Z"
11781198
assert req_body['shared'] == True
11791199
assert req_body['finding'] == finding_type_model
11801200
assert req_body['kpi'] == kpi_type_model
@@ -1270,6 +1290,8 @@ def test_update_note_required_params(self):
12701290
id = 'testString'
12711291
reported_by = reporter_model
12721292
related_url = [api_note_related_url_model]
1293+
create_time = string_to_datetime('2019-01-01T12:00:00.000Z')
1294+
update_time = string_to_datetime('2019-01-01T12:00:00.000Z')
12731295
shared = True
12741296
finding = finding_type_model
12751297
kpi = kpi_type_model
@@ -1286,6 +1308,8 @@ def test_update_note_required_params(self):
12861308
id,
12871309
reported_by,
12881310
related_url=related_url,
1311+
create_time=create_time,
1312+
update_time=update_time,
12891313
shared=shared,
12901314
finding=finding,
12911315
kpi=kpi,
@@ -1305,6 +1329,8 @@ def test_update_note_required_params(self):
13051329
assert req_body['id'] == 'testString'
13061330
assert req_body['reported_by'] == reporter_model
13071331
assert req_body['related_url'] == [api_note_related_url_model]
1332+
assert req_body['create_time'] == "2019-01-01T12:00:00Z"
1333+
assert req_body['update_time'] == "2019-01-01T12:00:00Z"
13081334
assert req_body['shared'] == True
13091335
assert req_body['finding'] == finding_type_model
13101336
assert req_body['kpi'] == kpi_type_model
@@ -1400,6 +1426,8 @@ def test_update_note_value_error(self):
14001426
id = 'testString'
14011427
reported_by = reporter_model
14021428
related_url = [api_note_related_url_model]
1429+
create_time = string_to_datetime('2019-01-01T12:00:00.000Z')
1430+
update_time = string_to_datetime('2019-01-01T12:00:00.000Z')
14031431
shared = True
14041432
finding = finding_type_model
14051433
kpi = kpi_type_model

0 commit comments

Comments
 (0)