Skip to content

Commit e8d81d3

Browse files
mgwozdz-uniconCopilot
andcommitted
fix: test_documents should expect output on all levels for tags
Co-authored-by: Copilot <[email protected]>
1 parent ae1afb1 commit e8d81d3

1 file changed

Lines changed: 46 additions & 7 deletions

File tree

openedx/core/djangoapps/content/search/tests/test_documents.py

Lines changed: 46 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -229,6 +229,9 @@ def test_problem_block(self):
229229
"tags": {
230230
"taxonomy": ["Difficulty"],
231231
"level0": ["Difficulty > Easy"],
232+
"level1": [],
233+
"level2": [],
234+
"level3": [],
232235
},
233236
}
234237

@@ -276,6 +279,7 @@ def test_html_block(self):
276279
"level0": ["Difficulty > Normal", "Subject > Hypertext", "Subject > Linguistics"],
277280
"level1": ["Subject > Hypertext > Jump Links", "Subject > Linguistics > Asian Languages"],
278281
"level2": ["Subject > Linguistics > Asian Languages > Chinese"],
282+
"level3": [],
279283
},
280284
}
281285

@@ -285,7 +289,9 @@ def test_video_block_untagged(self):
285289
"""
286290
block_usage_key = self.toy_course_key.make_usage_key("video", "Welcome")
287291
block = self.store.get_item(block_usage_key)
288-
doc = searchable_doc_for_course_block(block)
292+
doc = {}
293+
doc.update(searchable_doc_for_course_block(block))
294+
doc.update(searchable_doc_tags(block.usage_key))
289295
assert doc == {
290296
"id": "block-v1edxtoy2012_falltypevideoblockwelcome-0c9fd626",
291297
"type": "course_block",
@@ -307,7 +313,13 @@ def test_video_block_untagged(self):
307313
],
308314
"content": {},
309315
"modified": self.created_date.timestamp(),
310-
# This video has no tags.
316+
"tags": {
317+
"taxonomy": [],
318+
"level0": [],
319+
"level1": [],
320+
"level2": [],
321+
"level3": [],
322+
},
311323
}
312324

313325
def test_html_library_block(self):
@@ -346,6 +358,9 @@ def test_html_library_block(self):
346358
"tags": {
347359
"taxonomy": ["Difficulty"],
348360
"level0": ["Difficulty > Normal"],
361+
"level1": [],
362+
"level2": [],
363+
"level3": [],
349364
},
350365
"publish_status": "never",
351366
}
@@ -385,6 +400,9 @@ def test_html_published_library_block(self):
385400
"tags": {
386401
"taxonomy": ["Difficulty"],
387402
"level0": ["Difficulty > Normal"],
403+
"level1": [],
404+
"level2": [],
405+
"level3": [],
388406
},
389407
'published': {'display_name': 'Text'},
390408
"publish_status": "published",
@@ -427,6 +445,9 @@ def test_html_published_library_block(self):
427445
"tags": {
428446
"taxonomy": ["Difficulty"],
429447
"level0": ["Difficulty > Normal"],
448+
"level1": [],
449+
"level2": [],
450+
"level3": [],
430451
},
431452
"published": {"display_name": "Text"},
432453
"publish_status": "published",
@@ -467,6 +488,9 @@ def test_html_published_library_block(self):
467488
"tags": {
468489
"taxonomy": ["Difficulty"],
469490
"level0": ["Difficulty > Normal"],
491+
"level1": [],
492+
"level2": [],
493+
"level3": [],
470494
},
471495
"published": {
472496
"display_name": "Text 2",
@@ -506,7 +530,10 @@ def test_collection_with_library(self):
506530
"modified": 1680674828.0,
507531
'tags': {
508532
'taxonomy': ['Difficulty'],
509-
'level0': ['Difficulty > Normal']
533+
'level0': ['Difficulty > Normal'],
534+
'level1': [],
535+
'level2': [],
536+
'level3': [],
510537
},
511538
"published": {
512539
"num_children": 0
@@ -535,7 +562,10 @@ def test_collection_with_published_library(self):
535562
"modified": 1680674828.0,
536563
'tags': {
537564
'taxonomy': ['Difficulty'],
538-
'level0': ['Difficulty > Normal']
565+
'level0': ['Difficulty > Normal'],
566+
'level1': [],
567+
'level2': [],
568+
'level3': [],
539569
},
540570
"published": {
541571
"num_children": 1
@@ -579,7 +609,10 @@ def test_draft_container(self, container_name, container_slug, container_type, d
579609
"last_published": None,
580610
"tags": {
581611
"taxonomy": ["Difficulty"],
582-
"level0": ["Difficulty > Normal"]
612+
"level0": ["Difficulty > Normal"],
613+
"level1": [],
614+
"level2": [],
615+
"level3": [],
583616
},
584617
# "published" is not set since we haven't published it yet
585618
}
@@ -627,7 +660,10 @@ def test_published_container(self):
627660
"last_published": 1680674828.0,
628661
"tags": {
629662
"taxonomy": ["Difficulty"],
630-
"level0": ["Difficulty > Normal"]
663+
"level0": ["Difficulty > Normal"],
664+
"level1": [],
665+
"level2": [],
666+
"level3": [],
631667
},
632668
"published": {
633669
"num_children": 1,
@@ -701,7 +737,10 @@ def test_published_container_with_changes(self):
701737
"last_published": 1680674828.0,
702738
"tags": {
703739
"taxonomy": ["Difficulty"],
704-
"level0": ["Difficulty > Normal"]
740+
"level0": ["Difficulty > Normal"],
741+
"level1": [],
742+
"level2": [],
743+
"level3": [],
705744
},
706745
"published": {
707746
"num_children": 1,

0 commit comments

Comments
 (0)