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.

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

Improve Travis CI testing #39

Closed
skliper opened this issue Jan 7, 2020 · 1 comment · Fixed by #40 or #58
Closed

Improve Travis CI testing #39

skliper opened this issue Jan 7, 2020 · 1 comment · Fixed by #40 or #58
Assignees

Comments

@skliper
Copy link
Contributor

skliper commented Jan 7, 2020

Is your feature request related to a problem? Please describe.
Travis CI currently just does cppcheck and default build on the bundle

Describe the solution you'd like
Parallel test the following:

  • BUILDTYPE=release OMIT_DEPRECATED=true STRICT_NO_WARNINGS=true
  • BUILDTYPE=release OMIT_DEPRECATED=false(default) STRICT_NO_WARNINGS=true
  • BUILDTYPE=debug(default) OMIT_DEPRECATED=true STRICT_NO_WARNINGS=true
  • BUILDTYPE=debug(default) OMIT_DEPRECATED=false(default) STRICT_NO_WARNINGS=true

And do the following tests:

  • Add document generation (eventually enforce no warnings)
  • Build with ENABLE_UNIT_TESTS=TRUE SIMULATION=native
  • Run unit tests (eventually enforce coverage)
  • Execute cFE and pipe output to file
  • Send reset cmd from cmdutil
  • Confirm execution output lacks error/warning/fail messages

Describe alternatives you've considered
None

Additional context
Eventually add nightly or triggered local builds for other environments (pc-RTEMS, vxworks on a modern platform)
Eventually add automated build verification (trade CI vs local)

Requester Info
Jacob Hageman - NASA/GSFC

@jphickey
Copy link
Contributor

jphickey commented Jan 8, 2020

Recommend that this be accomplished by using alternate sample_defs directories (under different names. of course) for the alternate configs. This should also ideally cover different CFE config options (e.g. TIME server/client) as some TIME options were broken in the past and failed to compile with these options turned on.

This allows the baseline config which is suggested as a starting point for new users to remain simple and straightforward while allowing the CI config(s) to be as complex as it needs to be to cover all these various options.

skliper added a commit to skliper/cFS that referenced this issue Jan 8, 2020
skliper added a commit to skliper/cFS that referenced this issue Jan 8, 2020
skliper added a commit to skliper/cFS that referenced this issue Jan 8, 2020
skliper added a commit that referenced this issue Jan 15, 2020
skliper added a commit to skliper/cFS that referenced this issue Jan 21, 2020
skliper added a commit to skliper/cFS that referenced this issue Jan 22, 2020
skliper added a commit to skliper/cFS that referenced this issue Jan 28, 2020
astrogeco pushed a commit that referenced this issue Feb 12, 2020
astrogeco pushed a commit that referenced this issue Feb 13, 2020
astrogeco pushed a commit that referenced this issue Feb 26, 2020
astrogeco pushed a commit that referenced this issue Feb 28, 2020
astrogeco pushed a commit that referenced this issue Mar 2, 2020
skliper added a commit to skliper/cFS that referenced this issue Mar 17, 2020
astrogeco pushed a commit that referenced this issue Mar 18, 2020
astrogeco added a commit that referenced this issue Mar 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants