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

S2050. Extra Scope: CI Tools #24

Merged
merged 36 commits into from
Dec 3, 2020
Merged

Conversation

nickrgarner
Copy link
Contributor

This portion of the project focuses on setting up the SQLFE repository for future open-source contributions from 3rd party developers. The aim is to create a working config for Travis CI to run tests in the sqlfe.junit package on each commit and pull request and to get coverage data reported from CodeCov.

Summary of changes:

  • Added pom.xml configuration to wrap the project in a Maven build to allow for compile and testing from command line. Maven will also manage project dependencies as defined in pom.xml.
  • Created .travis.yml configuration to specify the test conditions on Travis CI. This includes the script to pipe coverage information to CodeCov via JaCoCo library.
  • Edited AbstractTest.java to default to using MySQL for testDAO, as Travis CI does not support OracleDB.

Future work:

  • There is currently a bug preventing Travis CI from connecting to the MySQL database during testing. This is preventing the test suite from completing and is blocking the CodeCov output. The most likely cause is missing MySQL setup code in .travis.yml. An example of this behavior can be seen in this Travis build output.

Contributors:

  1. Jack Macdonald (@jmmacdo4)
  2. Nick Garner (@nickrgarner)
  3. Sumitosh Pal (@sith2594 )
  4. Abhishek Gupta (@abhishekgupta930)

nickrgarner and others added 30 commits October 30, 2020 13:53
@wagnerpj42 wagnerpj42 merged commit 92765cd into wagnerpj42:CItools Dec 3, 2020
@wagnerpj42
Copy link
Owner

Maven and Travis CI changes from NCSU class project S2050 merged into CITools branch; additional work needed to remove any remaining issues before merge with main branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants