Bug 5865 - confusing and disorganised structure in autotests directory
Summary: confusing and disorganised structure in autotests directory
Status: NEW
Alias: None
Product: ThinLinc
Classification: Unclassified
Component: Automatic tests (show other bugs)
Version: trunk
Hardware: PC Unknown
: P2 Normal
Target Milestone: MediumPrio
Assignee: Karl Mikaelsson
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-05-02 13:40 CEST by Pierre Ossman
Modified: 2021-10-19 12:44 CEST (History)
1 user (show)

See Also:
Acceptance Criteria:


Attachments

Description Pierre Ossman cendio 2016-05-02 13:40:52 CEST
The autotests directory is currently a bit of a confusing mess. The large scale system tests are mixed with the smaller unit tests without a very clear separation between the two. We should really do something about this to make things clearer.

We could consider moving the unit tests to their respective modules, rather than collecting them in one place. That could encourage more active use during development, rather than treating the automatic tests as something entirely separate.
Comment 1 Pierre Ossman cendio 2016-05-03 10:57:18 CEST
We should be able to move a few tests at a time and not have to move everything all at once. So the first step could be to move a few of them and see how that goes.
Comment 2 Karl Mikaelsson cendio 2016-05-11 09:25:13 CEST
I've got a git branch where I've started working on moving a couple of unit tests to the source tree:
http://git.cendio.se/cgit/~derfian/thinlinc.git/log/?h=local-unittests

They are currently accompanied by a Jenkins job that runs the tests.
https://jenkins.lkpg.cendio.se/job/thinlinc-python-unittests/default/

I've used nosetests as a test runner to provide xunit output files and coverage information that is compatible with the Jenkins plugins we're currently using.
Comment 3 Pierre Ossman cendio 2021-10-19 12:44:29 CEST
This is a lot better now in that all unit test have moved to their respective modules. We still have a few mixed things left though, so a last round of cleanup is needed.

Note You need to log in before you can comment on or make changes to this bug.