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

Adding test cases #65

Merged
merged 1 commit into from
Jun 14, 2015
Merged

Adding test cases #65

merged 1 commit into from
Jun 14, 2015

Conversation

tomasbjerre
Copy link
Contributor

  • For FindBugs, Checkstyle, PMD, CPD, CPPLint, CSSLint, PEP8, PerlCritic, PyLint, PyFlakes, XmlLint, Zptlint and Resharper
  • Adding new test util to ease writing test cases
  • Using latest Findbugs and FB-contrib messages
  • Letting Resharper parser report files, even if they dont exist on filesystem. The parser should parse and only parse.
  • Letting CSSLint support both lint and csslint format. It was implemented to support only lint. Would be better to have a Lint type but this makes us backwards compatible. We should probably also add a lint type.
  • Removing the static block from TypeDescriptor. For it to work, the parser descriptors has to load after TypeDescriptor and that is not guaranteed.
  • These tests increases code coverage from 26.6% to 59.4%. They make code more flexible as their input is report-files and config, output is data structures exposed to GUI. They make it clear what input is actually expected by the parsers.
    New release? #61

@tomasbjerre tomasbjerre mentioned this pull request Jun 14, 2015
Closed
@tomasbjerre
Adding test cases #61
597569a 
* For FindBugs, Checkstyle, PMD, CPD, CPPLint, CSSLint, PEP8, PerlCritic, PyLint, PyFlakes, XmlLint, Zptlint and Resharper
* Adding new test util to ease writing test cases
* Using latest Findbugs and FB-contrib messages
* Letting Resharper parser report files, even if they dont exist on filesystem. The parser should parse and only parse.
* Letting CSSLint support both lint and csslint format. It was implemented to support only lint. Would be better to have a Lint type but this makes us backwards compatible. We should probably also add a lint type.
* Removing the static block from TypeDescriptor. For it to work, the parser descriptors has to load after TypeDescriptor and that is not guaranteed.
* These tests increases code coverage from 26.6% to 59.4%. They make code more flexible as their input is report-files and config, output is data structures exposed to GUI. They make it clear what input is actually expected by the parsers.
* Adding Travis CI, it gives more control of build process and can build for all supported JDK:s.
tomasbjerre added a commit that referenced this pull request Jun 14, 2015
@tomasbjerre
Merge pull request #65 from tomasbjerre/tests
8902ad1 
Adding test cases
@tomasbjerre tomasbjerre merged commit 8902ad1 into jenkinsci:master Jun 14, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@tomasbjerre