Skip to content
This repository was archived by the owner on Jul 3, 2024. It is now read-only.

Interface for QTimer #4

Open
Fettpet opened this issue Jan 19, 2022 · 0 comments
Open

Interface for QTimer #4

Fettpet opened this issue Jan 19, 2022 · 0 comments

Comments

@Fettpet
Copy link

Fettpet commented Jan 19, 2022

The PlanRunner used QTimer. This is not testable with unittests. QTimer isn't usable for unit test, since the timeout signal is private. Instead we should implement a interface, which encapsulate the QTimer.
@Fettpet Fettpet mentioned this issue Jan 19, 2022
Open
Fettpet referenced this issue in Fettpet/IntervalTimer Jan 19, 2022
Chore: Add simple CI for windows and linux
e7ffa39 
Issue: #4
Fettpet referenced this issue in Fettpet/IntervalTimer Jan 19, 2022
Feature: Add Timer interface
67fd52d 
* it has a start function and a timeout signal

Ticket: #4
Fettpet referenced this issue in Fettpet/IntervalTimer Jan 19, 2022
Feature: Add a implementation for timer which uses QTimer
e3384b8 
Ticket: #4
Fettpet referenced this issue in Fettpet/IntervalTimer Jan 19, 2022
Test: Add timeout test for timer
e31a770 
* refactor the QCoreApplication out into main test file
* remove the QCoreApplication in PlanRunner
Ticket: #4
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Fettpet