Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix 834 and Add Unit Tests (dependent on #780) #849

Open
wants to merge 42 commits into
base: master
Choose a base branch
from
Open

Fix 834 and Add Unit Tests (dependent on #780) #849

wants to merge 42 commits into from

Commits on Jul 26, 2015

  1. preparation for unit test integration 

    create a static library of all apmplanner classes except of main, moved qgroundcontrol.pro to apmplanner_core.pro to keep the build process the same as before
    @mrpilot2
    mrpilot2 committed Jul 26, 2015
    Configuration menu
    Copy the full SHA
    240a190 View commit details
    Browse the repository at this point in the history

  2. prepared new unit test setup for apmplanner 

    removed old project configuration and source files from qgcunittest.pro because that is now part of the new common apmplanner-core library

added a test for the qt test framwork to make sure the new setup works as expected
    @mrpilot2
    mrpilot2 committed Jul 26, 2015
    Configuration menu
    Copy the full SHA
    1ab35c9 View commit details
    Browse the repository at this point in the history

  3. integrate new unit test setup into project

    @mrpilot2
    mrpilot2 committed Jul 26, 2015
    Configuration menu
    Copy the full SHA
    1bd66f5 View commit details
    Browse the repository at this point in the history

  4. created VersionComparator to compare two firmware or application 

    versions, copied from ApmFirmwareConfig.cc

added unit tests for version comparator
    @mrpilot2
    mrpilot2 committed Jul 26, 2015
    Configuration menu
    Copy the full SHA
    0256031 View commit details
    Browse the repository at this point in the history

  5. unit test exposed a bug when comparing release candidates 

    in ApmFirmwareConfig.cc

This bug was already fixed in AutoUpdateCheck.cc
commit 8de0f15
    @mrpilot2
    mrpilot2 committed Jul 26, 2015
    Configuration menu
    Copy the full SHA
    ab161c9 View commit details
    Browse the repository at this point in the history

  6. use new VersionComparator in ApmFirmwareConfig and AutoUpdateCheck

    @mrpilot2
    mrpilot2 committed Jul 26, 2015
    Configuration menu
    Copy the full SHA
    9346665 View commit details
    Browse the repository at this point in the history

  7. modified unit test for VersionComparator: add letter 'v' in random tests

    @mrpilot2
    mrpilot2 committed Jul 26, 2015
    Configuration menu
    Copy the full SHA
    00d909f View commit details
    Browse the repository at this point in the history

  8. VersionComparator: captureCount() returns always the number of groups… 

    … defined in the regex,

therefore it is always 2 and the if statement is useless
    @mrpilot2
    mrpilot2 committed Jul 26, 2015
    Configuration menu
    Copy the full SHA
    26035bb View commit details
    Browse the repository at this point in the history

Commits on Jul 29, 2015

  1. VersionComparator: use better regular expression to split complete ve… 

    …rsion string in one place

fixed uninitialized variable warning
    @mrpilot2
    mrpilot2 committed Jul 29, 2015
    Configuration menu
    Copy the full SHA
    81b1762 View commit details
    Browse the repository at this point in the history

  2. VersionComparator: extract method that splits string into parts

    @mrpilot2
    mrpilot2 committed Jul 29, 2015
    Configuration menu
    Copy the full SHA
    c8b0256 View commit details
    Browse the repository at this point in the history

Commits on Aug 1, 2015

  1. VersionComparator: fixed regular expression to capture multiple digit… 

    …s per group, allow muliple digits in build number
    @mrpilot2
    mrpilot2 committed Aug 1, 2015
    Configuration menu
    Copy the full SHA
    0331d7b View commit details
    Browse the repository at this point in the history

  2. project file cleanup

    @mrpilot2
    mrpilot2 committed Aug 1, 2015
    Configuration menu
    Copy the full SHA
    bcdbccc View commit details
    Browse the repository at this point in the history

  3. VersionComparator: test that invalid new version is not newer than cu… 

    …rrent version
    @mrpilot2
    mrpilot2 committed Aug 1, 2015
    Configuration menu
    Copy the full SHA
    e7c7da9 View commit details
    Browse the repository at this point in the history

  4. VersionComparator: added documentation

    @mrpilot2
    mrpilot2 committed Aug 1, 2015
    Configuration menu
    Copy the full SHA
    ca85668 View commit details
    Browse the repository at this point in the history

  5. Merge branch 'unit_test_integration'

    @mrpilot2
    mrpilot2 committed Aug 1, 2015
    Configuration menu
    Copy the full SHA
    acb2fff View commit details
    Browse the repository at this point in the history

Commits on Oct 10, 2015

  1. Merge remote-tracking branch 'upstream/master' 

    Conflicts:
	qgroundcontrol.pro
    @mrpilot2
    mrpilot2 committed Oct 10, 2015
    Configuration menu
    Copy the full SHA
    7eafe3b View commit details
    Browse the repository at this point in the history

Commits on Oct 24, 2015

  1. Merge remote-tracking branch 'upstream/master'

    @mrpilot2
    mrpilot2 committed Oct 24, 2015
    Configuration menu
    Copy the full SHA
    1a7ac04 View commit details
    Browse the repository at this point in the history

  2. Map Widget: Fixes #691 - dragging multiple waypoints 

    whenever a waypoint is dragged, go through the list of waypoints and find all other selected waypoints - shift these waypoints by the same lat/long difference as the initial waypoint was moved
    @mrpilot2
    mrpilot2 committed Oct 24, 2015
    Configuration menu
    Copy the full SHA
    17e9c48 View commit details
    Browse the repository at this point in the history

  3. Map Widget: refactoring - removed old unused code

    @mrpilot2
    mrpilot2 committed Oct 24, 2015
    Configuration menu
    Copy the full SHA
    30966f9 View commit details
    Browse the repository at this point in the history

  4. Mission Widget: refactoring - remove duplicate code when blocking/unb… 

    …locking SpinBox signals
    @mrpilot2
    mrpilot2 committed Oct 24, 2015
    Configuration menu
    Copy the full SHA
    c5563d2 View commit details
    Browse the repository at this point in the history

  5. Mission Widget: refactoring - remove unused variable

    @mrpilot2
    mrpilot2 committed Oct 24, 2015
    Configuration menu
    Copy the full SHA
    cf0b1c5 View commit details
    Browse the repository at this point in the history

  6. Mission Widget: refactoring - merge child items that are searched for… 

    … SpinBox in one list to fix duplicate for loop
    @mrpilot2
    mrpilot2 committed Oct 24, 2015
    Configuration menu
    Copy the full SHA
    774a659 View commit details
    Browse the repository at this point in the history

Commits on Nov 4, 2015

  1. Merge remote-tracking branch 'upstream/master'

    @mrpilot2
    mrpilot2 committed Nov 4, 2015
    Configuration menu
    Copy the full SHA
    d51e13d View commit details
    Browse the repository at this point in the history

Commits on Nov 28, 2015

  1. Merge branch 'master' of github.com:diydrones/apm_planner 

    Conflicts:
	qgroundcontrol.pro
    @mrpilot2
    mrpilot2 committed Nov 28, 2015
    Configuration menu
    Copy the full SHA
    be1bb40 View commit details
    Browse the repository at this point in the history

  2. Fix compiler warning initalization sequence (issue #40)

    @mrpilot2
    mrpilot2 committed Nov 28, 2015
    Configuration menu
    Copy the full SHA
    1e469b4 View commit details
    Browse the repository at this point in the history

  3. Fix compiler warning unused but set variable and unused variable (issue 

    #40)

unused variable
    @mrpilot2
    mrpilot2 committed Nov 28, 2015
    Configuration menu
    Copy the full SHA
    c6171e0 View commit details
    Browse the repository at this point in the history

  4. Fix compiler warning: suggest explicit braces to avoid ambiguous else… 

    … (issue #40)

Verbose flag is always true and debug flag always false - therefore removed these flags, if another verbosity level is needed the logging system should be used
    @mrpilot2
    mrpilot2 committed Nov 28, 2015
    Configuration menu
    Copy the full SHA
    6c503a1 View commit details
    Browse the repository at this point in the history

  5. Fix compiler warning: ignored qualifiers (issue #40)

    @mrpilot2
    mrpilot2 committed Nov 28, 2015
    Configuration menu
    Copy the full SHA
    752e600 View commit details
    Browse the repository at this point in the history

  6. Fix compiler warnings: case value 38 not in enumerated type QVariant:… 

    …:Type and comparison between enum QVariant::Type and enum QMetaType::Type

according to Qt documentation QVariant::type() should be interpreted as QMetaType::Type
    @mrpilot2
    mrpilot2 committed Nov 28, 2015
    Configuration menu
    Copy the full SHA
    a4e7771 View commit details
    Browse the repository at this point in the history

  7. Fix compiler warning: deleting object of polymorphic class type which… 

    … has non-virtual destructor (issue #40)
    @mrpilot2
    mrpilot2 committed Nov 28, 2015
    Configuration menu
    Copy the full SHA
    639ba35 View commit details
    Browse the repository at this point in the history

  8. Fix compiler warning: enumeration value not handled in switch (issue #40 

    )
    @mrpilot2
    mrpilot2 committed Nov 28, 2015
    Configuration menu
    Copy the full SHA
    f5b642f View commit details
    Browse the repository at this point in the history

  9. Fix compiler warning: unused parameter in libs (issue #40) 

    Mark unused parameters as unused
    @mrpilot2
    mrpilot2 committed Nov 28, 2015
    Configuration menu
    Copy the full SHA
    7407df2 View commit details
    Browse the repository at this point in the history

  10. Fix compiler warning: unused parameter (issue #40) 

    Mark parameters as not used
    @mrpilot2
    mrpilot2 committed Nov 28, 2015
    Configuration menu
    Copy the full SHA
    6ed7079 View commit details
    Browse the repository at this point in the history

  11. Fix compiler warning: unused parameter (issue #40) 

    Completely remove functions that are empty or not used
    @mrpilot2
    mrpilot2 committed Nov 28, 2015
    Configuration menu
    Copy the full SHA
    c931194 View commit details
    Browse the repository at this point in the history

  12. Fix compiler warnings: removed set but unused variables (issue #40)

    @mrpilot2
    mrpilot2 committed Nov 28, 2015
    Configuration menu
    Copy the full SHA
    0f2e894 View commit details
    Browse the repository at this point in the history

  13. Fix compiler warnings: Qt Ui Invalid z-order assignment (issue #40)

    @mrpilot2
    mrpilot2 committed Nov 28, 2015
    Configuration menu
    Copy the full SHA
    c16ca9b View commit details
    Browse the repository at this point in the history

  14. Merge branch 'fix_compiler_warnings'

    @mrpilot2
    mrpilot2 committed Nov 28, 2015
    Configuration menu
    Copy the full SHA
    dec9bf6 View commit details
    Browse the repository at this point in the history

Commits on Dec 5, 2015

  1. enable coverage analysis for unit test on linux

    @mrpilot2
    mrpilot2 committed Dec 5, 2015
    Configuration menu
    Copy the full SHA
    9dc3211 View commit details
    Browse the repository at this point in the history

  2. Merge branch 'master' of github.com:mrpilot2/apm_planner

    @mrpilot2
    mrpilot2 committed Dec 5, 2015
    Configuration menu
    Copy the full SHA
    10c078e View commit details
    Browse the repository at this point in the history

  3. link againt gcov for unit test coverage and fix compiler flag

    @mrpilot2
    mrpilot2 committed Dec 5, 2015
    Configuration menu
    Copy the full SHA
    3d0e506 View commit details
    Browse the repository at this point in the history

Commits on Dec 7, 2015

  1. Audio output: add unit test that reproduces issue #834

    @mrpilot2
    mrpilot2 committed Dec 7, 2015
    Configuration menu
    Copy the full SHA
    1151b68 View commit details
    Browse the repository at this point in the history

  2. Audio: Fix #834: Revert "Audio: Fixes #812 Port to flite2" 

    This reverts commit 5f0890a.
    @mrpilot2
    mrpilot2 committed Dec 7, 2015
    Configuration menu
    Copy the full SHA
    dc60229 View commit details
    Browse the repository at this point in the history