Fix issue #5969 for modified_iterating-list

[orSolocate]

Type of Changes

	Type
✓	🐛 Bug fix
	✨ New feature
	🔨 Refactoring
	📜 Docs

Description

Fix program crash for modified_iterating-list when the list being iterated through is a function call.
The bug was fixed so that it will not occur with dictionaries and sets as well.
A regression test was added too.

Closes #5969

[coveralls]

Pull Request Test Coverage Report for Build 2044411171

• 3 of 3 (100.0%) changed or added relevant lines in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.0004%) to 94.185%

---

Totals
Change from base Build 2044356620:	0.0004%
Covered Lines:	15339
Relevant Lines:	16286

---

💛 - Coveralls

[Pierre-Sassoulas]

Thank you for the quick fix @orSolocate, we're going to release 2.13.1 shortly :)

[Pierre-Sassoulas]

What do you think of the line that lost coverage as infer_val != utils.safe_infer(iter_obj): seems to be always false ? Can we remove this check following your change ?

[orSolocate]

@Pierre-Sassoulas thanks for noticing that letting me know what the line was,
It shouldn't be removed. I added a new test case where this line is covered :D

[Pierre-Sassoulas]

This is released in 2.13.1 :) btw if you want to check the coverage details on coverall you need to login with your GitHub account: coveralls use it to not get rate limited for api calls.

[orSolocate]

@Pierre-Sassoulas - thanks for merging man! and for for the coveralls tip, now I could see coverage details of my next uploads :)

[DanielNoord]

Just want to check in and say thanks @orSolocate for the quick turnaround here :)

[orSolocate]

Thanks mate!