Details
Details
- Reviewers
stikonas cjlcarvalho - Commits
- R16:7af8045f3263: Add unit test for Device
Diff Detail
Diff Detail
- Repository
- R16 KPMCore
- Branch
- testdevice
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 14234 Build 14252: arc lint + arc unit
test/testdevice.cpp | ||
---|---|---|
80 | I think I have some USB devices that actually return empty string... |
test/testdevice.cpp | ||
---|---|---|
80 | You need to check if QList is empty before calling first() |
test/testdevice.cpp | ||
---|---|---|
80 | Why don't you check if any device from the list has the empty name instead of checking only the first one? |
Comment Actions
I guesss it's alright to go in for now.
Although, it's a bit ugly that we are relying on devices present on tester's system... But anything else would be much harder.