30
Critique This! A Simple Qt Dock Widget which enables the whole widget to be 'folded' to use less screen space
(programming.dev)
I'm working on a project that needs lots of toolbars on screen at once, even though not all of them will be used at the same time. So, I'm modelling this 'foldable' dock widget after what I remember Photoshop panels used to be like.
It's a work in progress, but would like to hear constructive suggestions.
https://blocks.programming.dev/0101100101/42c5d67f86c049baa3500aa38e439f8a
Recommend against the ternary expression.
coveragemight not detect the two code blocks. Would also not be able to apply# pragma: no covercomment to a code block that isn't important enough to justify testing it. Often usedo nothingcode blocks.self.float_button.setIcon(self.icon_dock if is_floating else self.icon_float)lets say while testing something goes wrong and trying to debug what happened.Will not see the value that gets past into
self.float_button.setIconI like your idea of having the param be keyword only. Makes it more readable.
If you branch coverage tool can't handle branches on the same line I would suggest you use a different one! Does it handle
if foo or bar?Uhm, yes you will? Just step into the function.
self.float_button.setIconis a Python wrapper around a C++ library. Might not be possible to, or want to, step into the function.if foo or baris part of a if-else condition. The else portion needs# pragma: no coveror coverage may complain.Well... the else condition (
bar) needs to be covered. I haven't used branch coverage tools in Python but in any sane language you cover the actual semantics, not the lines. It shouldn't make any difference if you write your code on one line, or with ternary expressions, or whatever.What matters is what the package coverage can do and what limitations or nuances it has. Have to work with what we have. We are lucky to have coverage. Especially within a subprocess and multiprocessing workers
We are dealing with Python and coverage, not some theoretical situation or circumstances.
Was assuming measuring branch coverage. In which case, my advice is coming from experience