Code quality: update Ruff Rules (Bandit, S)

[furqan463]

Relates issue: #352

Adds ruff lint rules Bandit "S".

[furqan463]

After replacing assert with if raise in src, test coverage has declined.

[furqan463]

@mgovers a number of TODOs are present in pandapower_converter regarding creating unit tests, although unit tests are already created for these methods. Should these TODOs be removed?

[mgovers]

apologies for the merge mess i caused. there was a merge conflict in the uv.lock and i messed up during the manual merge. i've force-pushed to undo the bugged change and now it's in a good state again.

@figueroa1395 and/or @nitbharambe can one of you do a final review? I don't want to sign off cause i did some changes myself as well.

[figueroa1395]

Hello @furqan463, I gave an additional review as requested by @mgovers. Most comments are either very minor or questions from my side. This should be easy to resolve and be good to go soon.

Thanks again for the contribution!

[figueroa1395]

@mgovers a number of TODOs are present in pandapower_converter regarding creating unit tests, although unit tests are already created for these methods. Should these TODOs be removed?

@furqan463 I just noticed we never replied to this message. I see indeed some (if not all) the unit tests comments no longer apply. Can you also remove the ones that have already been added?