Fisheye on application/unittests/tst_canbus.h

application (bugfix/DEN-16597-UI-BN-S114): DEN-1714 : Implementation...

Behrouz NematiPour — Tue, 21 Jan 2020 19:03:16 -0800

Behrouz NematiPour committed 805119c on branch master, 2020_EOY_DEMO and 274 more in application

DEN-1714 : Implementation

- Added cppcheck output.

- Added Linux signal handler to quit the application gracefully.

- Added manual coco validation. // coco begin validated:

· /cppcheck.err

(-3, +3)

| History

| Source

| Diff

· /cppcheck.log

(-23, +30)

| History

| Source

| Diff

· /denali.pro

(-0, +13)

| History

| Source

| Diff

· /denali.pro.user

(-274, +368)

| History

| Source

| Diff

· /main.cpp

(-1, +27)

| History

| Source

| Diff

· /sources/applicationcontroller.cpp

(-0, +7)

| History

| Source

| Diff

· /sources/canbus/caninterface.cpp

(-0, +7)

| History

| Source

| Diff

· /sources/canbus/caninterface.h

(-0, +2)

| History

| Source

| Diff

· /sources/canbus/frameinterface.cpp

(-0, +7)

| History

| Source

| Diff

· /sources/canbus/frameinterface.h

(-0, +2)

| History

| Source

| Diff

· /sources/canbus/messageacknowmodel.cpp

(-0, +8)

| History

| Source

| Diff

· /sources/canbus/messageacknowmodel.h

(-0, +5)

| History

| Source

| Diff

· /sources/canbus/messagedispatcher.cpp

(-16, +14)

| History

| Source

| Diff

· /sources/canbus/messagedispatcher.h

(-6, +10)

| History

| Source

| Diff

· /sources/gui/guicontroller.cpp

(-1, +26)

| History

| Source

| Diff

… 11 more files in changeset.

application (bugfix/DEN-16597-UI-BN-S114): DEN-1714 : Implementation...

Behrouz NematiPour — Mon, 20 Jan 2020 18:34:02 -0800

Behrouz NematiPour committed 1b8d8fd on branch master, 2020_EOY_DEMO and 274 more in application

DEN-1714 : Implementation

- Fixed the CanBus unit tests by the new sequence number added to the messages.

- Removed none implemented classes because it was coming always as the low coverage.

· /denali.pro	(-6, +6)	\| History	\| Source	\| Diff
· /sources/canbus/messageinterpreter.cpp	(-0, +1)	\| History	\| Source	\| Diff
· /unittests/tst_canbus.cpp	(-40, +48)	\| History	\| Source	\| Diff
· /unittests/tst_canbus.h	(-0, +9)	\| History	\| Source	\| Diff

application (bugfix/DEN-16597-UI-BN-S114): DEN-1714 : Implementation

Behrouz NematiPour — Tue, 14 Jan 2020 23:30:03 -0800

Behrouz NematiPour committed e58b907 on branch master, 2020_EOY_DEMO and 274 more in application

DEN-1714 : Implementation

· /denali.pro

(-0, +6)

| History

| Source

| Diff

· /denali.pro.user

(-4, +4)

| History

| Source

| Diff

· /main.cpp

(-2, +2)

| History

| Source

| Diff

· /scripts/run.sh

(-0, +2)

| History

| Source

| Diff

· /sources/canbus/caninterface.cpp

(-7, +42)

| History

| Source

| Diff

· /sources/canbus/caninterface.h

(-1, +10)

| History

| Source

| Diff

· /sources/canbus/messagebuilder.cpp

(-14, +49)

| History

| Source

| Diff

· /sources/canbus/messagebuilder.h

(-4, +6)

| History

| Source

| Diff

· /sources/canbus/messagedispatcher.cpp

(-17, +57)

| History

| Source

| Diff

· /sources/canbus/messagedispatcher.h

(-0, +18)

| History

| Source

| Diff

· /sources/canbus/messageglobals.h

(-3, +10)

| History

| Source

| Diff

· /sources/canbus/messageinterpreter.cpp

(-7, +6)

| History

| Source

| Diff

· /sources/canbus/messageinterpreter.h

(-4, +4)

| History

| Source

| Diff

· /sources/gui/guiglobals.h

(-1, +3)

| History

| Source

| Diff

· /sources/storage/logger.cpp

(-0, +1)

| History

| Source

| Diff

… 4 more files in changeset.

application (bugfix/DEN-16597-UI-BN-S114): Merge branch 'master' into DEN-1090_MessageHandler

Behrouz NematiPour — Thu, 09 Jan 2020 11:08:09 -0800

Behrouz NematiPour committed c933552 on branch master, 2020_EOY_DEMO and 276 more in application

Merge branch 'master' into DEN-1090_MessageHandler

· /main.cpp

(-1, +1)

| History

| Source

| Diff

· /sources/applicationcontroller.cpp

(-1, +1)

| History

| Source

| Diff

· /sources/applicationcontroller.h

(-1, +1)

| History

| Source

| Diff

· /sources/canbus/caninterface.cpp

(-1, +1)

| History

| Source

| Diff

· /sources/canbus/caninterface.h

(-1, +1)

| History

| Source

| Diff

· /sources/gui/guicontroller.cpp

(-1, +1)

| History

| Source

| Diff

· /sources/gui/guicontroller.h

(-1, +1)

| History

| Source

| Diff

· /sources/gui/guiglobals.cpp

(-1, +1)

| History

| Source

| Diff

· /sources/gui/guiglobals.h

(-1, +1)

| History

| Source

| Diff

· /sources/gui/guiview.cpp

(-1, +1)

| History

| Source

| Diff

· /sources/gui/guiview.h

(-1, +1)

| History

| Source

| Diff

· /sources/gui/qml/components/ModalDialog.qml

(-1, +1)

| History

| Source

| Diff

· /sources/gui/qml/components/TouchRect.qml

(-1, +1)

| History

| Source

| Diff

· /sources/gui/qml/main.qml

(-1, +1)

| History

| Source

| Diff

· /sources/gui/qml/pages/SettingsHome.qml

(-1, +1)

| History

| Source

| Diff

… 5 more files in changeset.

application (bugfix/DEN-16597-UI-BN-S114): DEN-1617 : Implementation - Alarm Logging and Acknowledgment...

Behrouz NematiPour — Wed, 08 Jan 2020 23:39:49 -0800

Behrouz NematiPour committed e02b2d4 on branch master, 2020_EOY_DEMO and 275 more in application

DEN-1617 : Implementation - Alarm Logging and Acknowledgment

- Added a unit test project for miscellaneous test which is not covered on feature based unit tests.

- Added a simple unit test to test the coverage.

· /denali.pro	(-0, +12)	\| History	\| Source	\| Diff
· /denali.pro.user	(-3, +97)	\| History	\| Source	\| Diff
· /sources/applicationcontroller.cpp	(-1, +0)	\| History	\| Source	\| Diff
· /sources/storage/filehandler.cpp	(-1, +7)	\| History	\| Source	\| Diff
· /sources/utility/types.h	(-2, +4)	\| History	\| Source	\| Diff
· /unittests/tst_canbus.cpp	(-0, +6)	\| History	\| Source	\| Diff
· /unittests/tst_canbus.h	(-0, +2)	\| History	\| Source	\| Diff
· /unittests/tst_utilities.cpp	(-0, +51)	\| History	\| Source
· /unittests/tst_utilities.h	(-0, +50)	\| History	\| Source

application (bugfix/DEN-16597-UI-BN-S114): DEN-1617 : Implementation - Alarm Logging and Acknowledgment...

Behrouz NematiPour — Wed, 08 Jan 2020 03:35:24 -0800

Behrouz NematiPour committed b9c5b0b on branch master, 2020_EOY_DEMO and 275 more in application

DEN-1617 : Implementation - Alarm Logging and Acknowledgment

- Added unit tests for logging

- Added unit tests for Alarm message.

- Tested the logging and export with more than 150MB log and it took ~10 sec to copy so added some feature to disable USB/Export buttons during export.

- Instead of using QtCuncorrent which is asynchronous moved logger to a thread to take advantage of the Signal/Slot queue between threads to have events logged in order.

- Move USB/Export buttons on Treatment Screen.

- Moved export from file handler into Logger.

· /denali.pro

(-0, +12)

| History

| Source

| Diff

· /denali.pro.user

(-158, +252)

| History

| Source

| Diff

· /main.cpp

(-1, +1)

| History

| Source

| Diff

· /sources/applicationcontroller.cpp

(-5, +13)

| History

| Source

| Diff

· /sources/applicationcontroller.h

(-1, +2)

| History

| Source

| Diff

· /sources/gui/guicontroller.cpp

(-0, +14)

| History

| Source

| Diff

· /sources/gui/guicontroller.h

(-0, +3)

| History

| Source

| Diff

· /sources/gui/guiview.cpp

(-0, +13)

| History

| Source

| Diff

· /sources/gui/guiview.h

(-0, +3)

| History

| Source

| Diff

· /sources/gui/qml/components/ExportButton.qml

(-2, +6)

| History

| Source

| Diff

· /sources/gui/qml/components/USBButton.qml

(-0, +6)

| History

| Source

| Diff

· /sources/gui/qml/pages/TreatmentHome.qml

(-21, +0)

| History

| Source

| Diff

· /sources/gui/qml/pages/TreatmentStart.qml

(-0, +19)

| History

| Source

| Diff

· /sources/storage/filehandler.cpp

(-18, +1)

| History

| Source

| Diff

· /sources/storage/filehandler.h

(-6, +2)

| History

| Source

| Diff

… 8 more files in changeset.

application (bugfix/DEN-16597-UI-BN-S114): DEN-1090 : UI Message Handler / Error Handler...

Behrouz NematiPour — Thu, 19 Dec 2019 03:37:24 -0800

Behrouz NematiPour committed 5610552 on branch master, 2020_EOY_DEMO and 276 more in application

DEN-1090 : UI Message Handler / Error Handler

- Trying a solution to have separate unit tests for each feature.

- Changed the unittests class to tst_canbus since this is the unit test for implemented canbus messages and interaction.

- removed .pro.user from gitignore since denali.pro.user contains configurations for local unit tests setup and need to be cloned for the new ones.

- Fixed issues which has been found during the unit tests.

· /.gitignore	(-1, +0)	\| History	\| Source	\| Diff
· /denali.pro	(-7, +9)	\| History	\| Source	\| Diff
· /denali.pro.user	(-0, +737)	\| History	\| Source
· /main.cpp	(-2, +2)	\| History	\| Source	\| Diff
· /sources/canbus/caninterface.cpp	(-0, +1)	\| History	\| Source	\| Diff
· /sources/canbus/caninterface.h	(-2, +2)	\| History	\| Source	\| Diff
· /sources/canbus/frameinterface.h	(-2, +2)	\| History	\| Source	\| Diff
· /sources/canbus/messagebuilder.cpp	(-7, +7)	\| History	\| Source	\| Diff
· /sources/canbus/messagedispatcher.h	(-0, +6)	\| History	\| Source	\| Diff
· /sources/canbus/messageinterpreter.cpp	(-0, +1)	\| History	\| Source	\| Diff
· /sources/utility/format.cpp	(-0, +7)	\| History	\| Source	\| Diff
· /sources/utility/format.h	(-3, +3)	\| History	\| Source	\| Diff
· /unittests/tst_canbus.cpp	(-0, +277)	\| History	\| Source
· /unittests/tst_canbus.h	(-0, +62)	\| History	\| Source
· /unittests/unittests.cpp	(-136, +0)	\| History	\| Source	\| Diff
· /unittests/unittests.h	(-51, +0)	\| History	\| Source	\| Diff