Skip to content

Commit ec2ea92

Browse files
Merge pull request #10 from CatalinAnt/Almu
Almu
2 parents 0416cab + 6f2fbd0 commit ec2ea92

File tree

1 file changed

+117
-8
lines changed

1 file changed

+117
-8
lines changed

README.md

Lines changed: 117 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -23,19 +23,74 @@ In order to not put in 8 large images, I will insert the final screenshot, with
2323

2424
### Your own coverage tool
2525

26-
<The following is supposed to be repeated for each group member>
26+
#### Ayman Errahmouni
2727

28-
<Group member name>
28+
##### Function 1: simplify_path_v2
29+
30+
[Link the commit](https://github.com/CatalinAnt/algorithms-SEP-95/pull/2/commits/22ee6fa1df4785596c603af61a725c558973eb0b)
31+
32+
Screenshot of branch measurement (66%):<br>
33+
![image](image-7.png)
34+
35+
##### Function 2: insertion_sort
36+
37+
[Link to commit](https://github.com/CatalinAnt/algorithms-SEP-95/pull/2/commits/5dae7f28036f89b7f6ff673639a922dd714aff3e)
38+
39+
Screenshot of branch measurement (0%, was untested):<br>
40+
![alt text](image-8.png)
2941

30-
<Function 1 name>
42+
#### Catalin Antonescu
3143

32-
<Show a patch (diff) or a link to a commit made in your forked repository that shows the instrumented code to gather coverage measurements>
44+
##### Function 1: strong_password
3345

34-
<Provide a screenshot of the coverage results output by the instrumentation>
46+
Link to commit:
47+
[https://github.com/CatalinAnt/algorithms-SEP-95/commit/eaad6d32ecd73bb8fde876a4d4852cb522aea6f8](https://github.com/CatalinAnt/algorithms-SEP-95/commit/2b0b9187c1c040e4476b1ca14f2c2249273566b7)
48+
49+
Screenshot of branch measurement:
50+
![image](https://github.com/CatalinAnt/algorithms-SEP-95/assets/113595149/e718a47f-5ea0-412c-b250-25a193412164)
51+
52+
##### Function 2: rotate_image
53+
54+
Link to commit:(same as for the first one)
55+
[https://github.com/CatalinAnt/algorithms-SEP-95/commit/eaad6d32ecd73bb8fde876a4d4852cb522aea6f8](https://github.com/CatalinAnt/algorithms-SEP-95/commit/2b0b9187c1c040e4476b1ca14f2c2249273566b7)
56+
57+
Screenshot of branch measurement:
58+
![image](https://github.com/CatalinAnt/algorithms-SEP-95/assets/113595149/94eec9b6-3dd6-46e3-b087-40892eccc10e)
59+
60+
#### Abdullah Abdelkhalik
61+
62+
63+
pythagoras
64+
65+
https://github.com/CatalinAnt/algorithms-SEP-95/commit/5651abafebe8ae3a5ea63e74883bb991acf19303
66+
67+
![pythagoras_hits](https://github.com/CatalinAnt/algorithms-SEP-95/assets/114078193/0df1fa2b-2185-4b9f-ae65-5d969edb009b)
68+
69+
70+
first_unique_char
71+
72+
https://github.com/CatalinAnt/algorithms-SEP-95/commit/c16f26e952322b2c1729778a4141a57103ba7658
73+
74+
![first_unique_char_hits](https://github.com/CatalinAnt/algorithms-SEP-95/assets/114078193/10d7c45c-398e-4408-8f11-6771f51fa95c)
75+
76+
77+
#### Abdullah Abdelkhalik
78+
79+
80+
pythagoras
81+
82+
https://github.com/CatalinAnt/algorithms-SEP-95/commit/5651abafebe8ae3a5ea63e74883bb991acf19303
83+
84+
![pythagoras_hits](https://github.com/CatalinAnt/algorithms-SEP-95/assets/114078193/0df1fa2b-2185-4b9f-ae65-5d969edb009b)
85+
86+
87+
first_unique_char
88+
89+
https://github.com/CatalinAnt/algorithms-SEP-95/commit/c16f26e952322b2c1729778a4141a57103ba7658
90+
91+
![first_unique_char_hits](https://github.com/CatalinAnt/algorithms-SEP-95/assets/114078193/10d7c45c-398e-4408-8f11-6771f51fa95c)
3592

36-
<Function 2 name>
3793

38-
<Provide the same kind of information provided for Function 1>
3994

4095
#### Ayman Errahmouni
4196

@@ -188,7 +243,6 @@ New coverage:
188243

189244
For strong_password there was a 26% coverage improvement with the existing tool and 40% with manual measurement tool.
190245

191-
192246
Test 2:
193247

194248
In test_matrix:
@@ -230,6 +284,59 @@ https://github.com/CatalinAnt/algorithms-SEP-95/commit/5651abafebe8ae3a5ea63e748
230284

231285
The coverage is improved by 13%, the code only hit three out of five branches and only set up two examples. I added a case where there is no unique letter.
232286

287+
## Almuthana Almustafa
288+
289+
stoogsort in stoog_sort.py
290+
291+
[Link to the commit in the founction files](https://github.com/CatalinAnt/algorithms-SEP-95/commit/57b66879c6ae0f82712c55528f540dfdb3c3ddd3)
292+
293+
result:
294+
295+
![alt text](ts.png)
296+
297+
word_break in word_break.py
298+
299+
link to commit in founction:
300+
[Link to the commit in the founction files](https://github.com/CatalinAnt/algorithms-SEP-95/commit/57b66879c6ae0f82712c55528f540dfdb3c3ddd3 )
301+
302+
result:
303+
304+
![alt text](ts.png)
305+
306+
Almuthana Almustafa
307+
308+
Test 1:
309+
310+
stoogsort function
311+
312+
link to commit in test files:
313+
[Link to the commit in the test files](https://github.com/CatalinAnt/algorithms-SEP-95/commit/157de36fd4c373b67cd03e3b3713be9ba5cf0d97)
314+
315+
existing tool result before:
316+
317+
![alt text](ss2.png)
318+
319+
existing tool result after:
320+
321+
![alt text](ss.png)
322+
323+
The coverage increased by 89%, largely attributable to the implementation of new tests.
324+
325+
Test 2:
326+
327+
word_break function
328+
329+
[Link to the commit in the test files](https://github.com/CatalinAnt/algorithms-SEP-95/commit/157de36fd4c373b67cd03e3b3713be9ba5cf0d97)
330+
331+
existing tool result before:
332+
333+
![alt text](wk1.png)
334+
335+
existing tool result after:
336+
337+
![alt text](wk2.png)
338+
339+
The coverage improved by 86% due to the creation of new tests.
233340

234341
### Overall
235342

@@ -249,3 +356,5 @@ Due to the large size of the project, the percentage only went up by one percent
249356
<Write what each group member did>
250357

251358
Abdullah -> increased the coverage for two functions.
359+
Almuthana Almustafa -> Instrumentation was added to two functions, and test cases were created for them to improve coverage.
360+

0 commit comments

Comments
 (0)