Skip to content

Commit 920335f

Browse files
beachdwellerbeachdweller
committed
feature : add test__collect_longrepr__compare_contents()
1 parent d2ca9f8 commit 920335f

2 files changed

Lines changed: 54 additions & 32 deletions

File tree

‎tests/sample_report.json‎

Lines changed: 32 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -37,12 +37,12 @@
3737
"message": ";) 8) ;D :) ;P"
3838
}
3939
],
40-
"longrepr": ";P ;D ;) 8) :)"
40+
"longrepr": "longrepr100 ;P ;D ;) 8) :)"
4141
},
4242
"teardown": {
4343
"duration": "8) :) ;P ;) ;D",
4444
"outcome": "8) :) ;D ;) ;P",
45-
"longrepr": ":) ;D 8) ;) ;P"
45+
"longrepr": "longrepr110 :) ;D 8) ;) ;P"
4646
}
4747
},
4848
{
@@ -71,12 +71,12 @@
7171
"message": ";D ;P ;) :) 8)"
7272
}
7373
],
74-
"longrepr": ";D ;P :) 8) ;)"
74+
"longrepr": "longrepr200 ;D ;P :) 8) ;)"
7575
},
7676
"teardown": {
7777
"duration": "8) ;P :) ;D ;)",
7878
"outcome": ";P ;D 8) :) ;)",
79-
"longrepr": ";) :) ;D ;P 8)"
79+
"longrepr": "longrepr210 ;) :) ;D ;P 8)"
8080
}
8181
},
8282
{
@@ -105,12 +105,12 @@
105105
"message": "8) :) ;) ;D ;P"
106106
}
107107
],
108-
"longrepr": ";D 8) ;P :) ;)"
108+
"longrepr": "longrepr300 ;D 8) ;P :) ;)"
109109
},
110110
"teardown": {
111111
"duration": "8) ;) ;P ;D :)",
112112
"outcome": "8) ;D :) ;P ;)",
113-
"longrepr": ";P :) ;) ;D 8)"
113+
"longrepr": "longrepr310 ;P :) ;) ;D 8)"
114114
}
115115
},
116116
{
@@ -139,12 +139,12 @@
139139
"message": ":) ;) ;D ;P 8)"
140140
}
141141
],
142-
"longrepr": ";) 8) :) ;D ;P"
142+
"longrepr": "longrepr400 ;) 8) :) ;D ;P"
143143
},
144144
"teardown": {
145145
"duration": ";D 8) ;P ;) :)",
146146
"outcome": ";D 8) :) ;P ;)",
147-
"longrepr": ";) ;P :) 8) ;D"
147+
"longrepr": "longrepr410;) ;P :) 8) ;D"
148148
}
149149
},
150150
{
@@ -162,17 +162,17 @@
162162
"setup": {
163163
"duration": "8) :) ;D ;) ;P",
164164
"outcome": "8) ;P ;) ;D :)",
165-
"longrepr": "8) ;D :) ;) ;P"
165+
"longrepr": "longrepr500 8) ;D :) ;) ;P"
166166
},
167167
"call": {
168168
"duration": ";) :) ;P 8) ;D",
169169
"outcome": ";) ;P 8) :) ;D",
170-
"longrepr": ";) 8) :) ;D ;P"
170+
"longrepr": "longrepr510;) 8) :) ;D ;P"
171171
},
172172
"teardown": {
173173
"duration": ";) 8) ;P :) ;D",
174174
"outcome": "8) :) ;) ;D ;P",
175-
"longrepr": ";D 8) :) ;) ;P"
175+
"longrepr": "longrepr520 ;D 8) :) ;) ;P"
176176
}
177177
},
178178
{
@@ -190,17 +190,17 @@
190190
"setup": {
191191
"duration": ":) ;) ;D 8) ;P",
192192
"outcome": ";P 8) ;D :) ;)",
193-
"longrepr": "8) :) ;D ;) ;P"
193+
"longrepr": "longrepr600 8) :) ;D ;) ;P"
194194
},
195195
"call": {
196196
"duration": "8) ;D ;) ;P :)",
197197
"outcome": ";) ;D :) ;P 8)",
198-
"longrepr": ";P :) 8) ;D ;)"
198+
"longrepr": "longrepr610 ;P :) 8) ;D ;)"
199199
},
200200
"teardown": {
201201
"duration": "8) :) ;P ;) ;D",
202202
"outcome": "8) ;D ;) :) ;P",
203-
"longrepr": "8) ;P :) ;) ;D"
203+
"longrepr": "longrepr620 8) ;P :) ;) ;D"
204204
}
205205
},
206206
{
@@ -218,17 +218,17 @@
218218
"setup": {
219219
"duration": ";P ;) :) 8) ;D",
220220
"outcome": ":) ;D 8) ;P ;)",
221-
"longrepr": ";) ;P :) 8) ;D"
221+
"longrepr": "longrepr700 ;) ;P :) 8) ;D"
222222
},
223223
"call": {
224224
"duration": "8) ;D ;) :) ;P",
225225
"outcome": ";) :) ;P ;D 8)",
226-
"longrepr": ";P ;D :) ;) 8)"
226+
"longrepr": "longrepr710 ;P ;D :) ;) 8)"
227227
},
228228
"teardown": {
229229
"duration": ";P :) ;) ;D 8)",
230230
"outcome": ";P ;D ;) 8) :)",
231-
"longrepr": ";D :) ;P 8) ;)"
231+
"longrepr": "longrepr720 ;D :) ;P 8) ;)"
232232
}
233233
},
234234
{
@@ -246,17 +246,17 @@
246246
"setup": {
247247
"duration": ";D ;P :) ;) 8)",
248248
"outcome": ";D ;P :) ;) 8)",
249-
"longrepr": ";) 8) ;D ;P :)"
249+
"longrepr": "longrepr800 ;) 8) ;D ;P :)"
250250
},
251251
"call": {
252252
"duration": ";) ;D 8) ;P :)",
253253
"outcome": "8) ;) ;D ;P :)",
254-
"longrepr": ";) ;P 8) ;D :)"
254+
"longrepr": "longrepr810;) ;P 8) ;D :)"
255255
},
256256
"teardown": {
257257
"duration": "8) ;D :) ;P ;)",
258258
"outcome": ";P 8) ;) ;D :)",
259-
"longrepr": "8) ;P ;D :) ;)"
259+
"longrepr": "longrepr820 8) ;P ;D :) ;)"
260260
}
261261
},
262262
{
@@ -274,17 +274,17 @@
274274
"setup": {
275275
"duration": "8) ;D ;) :) ;P",
276276
"outcome": ";D ;) 8) :) ;P",
277-
"longrepr": ":) ;) ;P 8) ;D"
277+
"longrepr": "longrepr900 :) ;) ;P 8) ;D"
278278
},
279279
"call": {
280280
"duration": ";) ;D ;P 8) :)",
281281
"outcome": "8) ;) :) ;D ;P",
282-
"longrepr": ";P ;D 8) ;) :)"
282+
"longrepr": "longrepr910 ;P ;D 8) ;) :)"
283283
},
284284
"teardown": {
285285
"duration": ":) 8) ;P ;) ;D",
286286
"outcome": "8) ;D :) ;P ;)",
287-
"longrepr": ";) ;P ;D :) 8)"
287+
"longrepr": "longrepr920 ;) ;P ;D :) 8)"
288288
}
289289
},
290290
{
@@ -302,17 +302,17 @@
302302
"setup": {
303303
"duration": ";D 8) :) ;) ;P",
304304
"outcome": ";P ;D 8) ;) :)",
305-
"longrepr": ";D ;P 8) ;) :)"
305+
"longrepr": "longrepr1000 ;D ;P 8) ;) :)"
306306
},
307307
"call": {
308308
"duration": ";) ;P 8) ;D :)",
309309
"outcome": ";) ;P ;D 8) :)",
310-
"longrepr": "8) ;D ;P ;) :)"
310+
"longrepr": "longrepr1010 8) ;D ;P ;) :)"
311311
},
312312
"teardown": {
313313
"duration": ";P :) 8) ;D ;)",
314314
"outcome": ";P 8) :) ;) ;D",
315-
"longrepr": ";) ;D 8) ;P :)"
315+
"longrepr": "longrepr1020 ;) ;D 8) ;P :)"
316316
}
317317
},
318318
{
@@ -330,17 +330,17 @@
330330
"setup": {
331331
"duration": ";D ;P :) ;) 8)",
332332
"outcome": "8) :) ;D ;) ;P",
333-
"longrepr": ";D 8) ;) ;P :)"
333+
"longrepr": "longrepr1100 ;D 8) ;) ;P :)"
334334
},
335335
"call": {
336336
"duration": ":) ;D ;) 8) ;P",
337337
"outcome": ":) ;P ;) ;D 8)",
338-
"longrepr": ":) ;) ;D 8) ;P"
338+
"longrepr": "longrepr1110 :) ;) ;D 8) ;P"
339339
},
340340
"teardown": {
341341
"duration": ";D ;) ;P :) 8)",
342342
"outcome": "8) ;D ;P ;) :)",
343-
"longrepr": ";) ;D 8) :) ;P"
343+
"longrepr": "longrepr1120;) ;D 8) :) ;P"
344344
}
345345
},
346346
{
@@ -358,17 +358,17 @@
358358
"setup": {
359359
"duration": ";D :) ;) 8) ;P",
360360
"outcome": ";) ;D 8) :) ;P",
361-
"longrepr": "8) ;P ;) :) ;D"
361+
"longrepr": "longrepr1200 8) ;P ;) :) ;D"
362362
},
363363
"call": {
364364
"duration": "8) ;) ;P ;D :)",
365365
"outcome": ":) ;) 8) ;P ;D",
366-
"longrepr": "8) ;) ;D :) ;P"
366+
"longrepr": "longrepr1210 8) ;) ;D :) ;P"
367367
},
368368
"teardown": {
369369
"duration": ";P 8) ;) ;D :)",
370370
"outcome": ":) ;) ;D 8) ;P",
371-
"longrepr": ";D 8) ;) ;P :)"
371+
"longrepr": "longrepr1220 ;D 8) ;) ;P :)"
372372
}
373373
}
374374
]

‎tests/test_ai_tutor.py‎

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -626,5 +626,27 @@ def test__collect_longrepr__has_list_items_len(collect_longrepr_result:List[str]
626626
assert s, "Expected non-empty string, got empty string."
627627

628628

629+
def test__collect_longrepr__compare_contents(collect_longrepr_result:List[str]):
630+
markers = 'longrepr100 longrepr110'.split()
631+
markers += 'longrepr200 longrepr210'.split()
632+
markers += 'longrepr300 longrepr310'.split()
633+
markers += 'longrepr400 longrepr410'.split()
634+
markers += 'longrepr500 longrepr510 longrepr520'.split()
635+
markers += 'longrepr600 longrepr610 longrepr620'.split()
636+
markers += 'longrepr700 longrepr710 longrepr720'.split()
637+
markers += 'longrepr800 longrepr810 longrepr820'.split()
638+
markers += 'longrepr900 longrepr910 longrepr920'.split()
639+
markers += 'longrepr1000 longrepr1010 longrepr1020'.split()
640+
markers += 'longrepr1100 longrepr1110 longrepr1120'.split()
641+
markers += 'longrepr1200 longrepr1210 longrepr1220'.split()
642+
643+
for s in collect_longrepr_result:
644+
for marker in markers:
645+
if marker in s:
646+
markers.remove(marker)
647+
648+
assert not markers, f"Expected all markers to be found, but missing: {markers}."
649+
650+
629651
if '__main__' == __name__:
630652
pytest.main([__file__])

0 commit comments

Comments
 (0)