Re: [PATCH 3/4] rteval: Convert CpuList class to a module

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



út 12. 12. 2023 v 22:46 odesílatel John Kacur <jkacur@xxxxxxxxxx> napsal:
>
> You don't have to do it in this patch, but some builtin testing would be
> nice under this
> if __name__ == "__main__":
>

Yeah that's definitely a good idea, moving the cpulist logic into a
separate module will allow for more elegant unit tests directly in the
file.

In the future, we might also consider moving tests into separate files
using some kind of unit test library like pytest [1] or the builtin
unit test [2] module. If I understand it correctly, currently you have
to run tests manually for each module that has them, and you also have
to check the results manually; changing that to a more systematic and
automated approach should make development easier and prevent bugs.

[1] - https://docs.pytest.org/en/7.4.x/
[2] - https://docs.python.org/3/library/unittest.html

Tomas






[Index of Archives]     [RT Stable]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]

  Powered by Linux