java-checkstyle 0.0.2
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- checksums.yaml +7 -0
- data/.codeclimate.yml +2 -0
- data/.gitignore +17 -0
- data/.rspec +2 -0
- data/.travis.yml +9 -0
- data/Gemfile +10 -0
- data/LICENSE.txt +22 -0
- data/README.md +47 -0
- data/Rakefile +6 -0
- data/java-checkstyle.gemspec +27 -0
- data/lib/plugins/pre_commit/checks/checkstyle.rb +71 -0
- data/lib/plugins/pre_commit/message/extractor.rb +27 -0
- data/lib/plugins/pre_commit/message/formatter.rb +38 -0
- data/lib/plugins/pre_commit/support/path.rb +18 -0
- data/lib/pre-commit/checkstyle/version.rb +16 -0
- data/lib/pre-commit/support/checkstyle/checkstyle-5.7-all.jar +0 -0
- data/lib/pre-commit/support/checkstyle/checkstyle-6.11-all.jar +0 -0
- data/lib/pre-commit/support/checkstyle/google_checks.xml +206 -0
- data/lib/pre-commit/support/checkstyle/sun_checks.xml +177 -0
- data/spec/fixtures/bad.java +5 -0
- data/spec/fixtures/bad2.java +5 -0
- data/spec/fixtures/good.java +22 -0
- data/spec/fixtures/google_checks.xml +206 -0
- data/spec/plugins/pre_commit/checks/checkstyle_spec.rb +53 -0
- data/spec/plugins/pre_commit/message/extractor_spec.rb +59 -0
- data/spec/plugins/pre_commit/output/formatter.rb +0 -0
- data/spec/spec_helper.rb +22 -0
- metadata +164 -0
@@ -0,0 +1,59 @@
|
|
1
|
+
require 'spec_helper'
|
2
|
+
require 'plugins/pre_commit/message/extractor'
|
3
|
+
##
|
4
|
+
# Tests for PreCommit::Message::Extractor
|
5
|
+
describe PreCommit::Message::Extractor do
|
6
|
+
let(:extractor) { PreCommit::Message::Extractor.new }
|
7
|
+
let(:output) {
|
8
|
+
'<?xml version="1.0" encoding="UTF-8"?>
|
9
|
+
<checkstyle version="6.11">
|
10
|
+
<file name="/Users/cristianoliveira/work/java-checkstyle/spec/fixtures/bad.java">
|
11
|
+
<error line="1" severity="error" message="some error message" source="com.puppycrawl.tools.checkstyle.checks.javadoc.JavadocTypeCheck"/>
|
12
|
+
<error line="11" column= "1 " severity= "warning" message="some error message" source="com.puppycrawl.tools.checkstyle.checks.design.HideUtilityClassConstructorCheck"/>
|
13
|
+
</file>
|
14
|
+
<file name="/Users/cristianoliveira/work/java-checkstyle/spec/fixtures/bad2.java">
|
15
|
+
<error line="2 " column= "3 " severity= "error" message="some error message" source= "com.puppycrawl.tools.checkstyle.checks.javadoc.JavadocMethodCheck "/>
|
16
|
+
<error line="2 " column= "27 " severity= "error" message="some error message" source= "com.puppycrawl.tools.checkstyle.checks.FinalParametersCheck "/>
|
17
|
+
</file>
|
18
|
+
</checkstyle>
|
19
|
+
Checkstyle ends with 4 errors.' }
|
20
|
+
|
21
|
+
it "should return empty file for nil output" do
|
22
|
+
output = nil
|
23
|
+
result = extractor.extract output
|
24
|
+
|
25
|
+
expect(result['checkstyle']['file']).to be_empty
|
26
|
+
end
|
27
|
+
|
28
|
+
it "should extract files" do
|
29
|
+
result = extractor.extract output
|
30
|
+
expect(result['checkstyle']['file'].size).to eq 2
|
31
|
+
end
|
32
|
+
|
33
|
+
it "should extract errors" do
|
34
|
+
result = extractor.extract output
|
35
|
+
expect(result['checkstyle']['file'][0]['error'].size).to eq 2
|
36
|
+
end
|
37
|
+
|
38
|
+
it "should extract error details" do
|
39
|
+
expected = [
|
40
|
+
{
|
41
|
+
"line"=>"1",
|
42
|
+
"severity"=>"error",
|
43
|
+
"message"=>"some error message",
|
44
|
+
"source"=>"com.puppycrawl.tools.checkstyle.checks.javadoc.JavadocTypeCheck"
|
45
|
+
},
|
46
|
+
{
|
47
|
+
"line" => "11",
|
48
|
+
"column" => "1 ",
|
49
|
+
"severity" => "warning",
|
50
|
+
"message" => "some error message",
|
51
|
+
"source" => "com.puppycrawl.tools.checkstyle.checks.design.HideUtilityClassConstructorCheck"
|
52
|
+
}
|
53
|
+
]
|
54
|
+
|
55
|
+
result = extractor.extract output
|
56
|
+
errors = result['checkstyle']['file'][0]['error']
|
57
|
+
expect(errors).to eq expected
|
58
|
+
end
|
59
|
+
end
|
File without changes
|
data/spec/spec_helper.rb
ADDED
@@ -0,0 +1,22 @@
|
|
1
|
+
require "codeclimate-test-reporter"
|
2
|
+
CodeClimate::TestReporter.start
|
3
|
+
|
4
|
+
$LOAD_PATH.unshift File.expand_path('../../lib', __FILE__)
|
5
|
+
|
6
|
+
require 'pre-commit'
|
7
|
+
|
8
|
+
RSpec.configure do |config|
|
9
|
+
config.treat_symbols_as_metadata_keys_with_true_values = true
|
10
|
+
config.run_all_when_everything_filtered = true
|
11
|
+
config.filter_run :focus
|
12
|
+
|
13
|
+
# Run specs in random order to surface order dependencies. If you find an
|
14
|
+
# order dependency and want to debug it, you can fix the order by providing
|
15
|
+
# the seed, which is printed after each run.
|
16
|
+
# --seed 1234
|
17
|
+
config.order = 'random'
|
18
|
+
end
|
19
|
+
|
20
|
+
def fixture_file(file)
|
21
|
+
File.expand_path("../fixtures/#{file}", __FILE__)
|
22
|
+
end
|
metadata
ADDED
@@ -0,0 +1,164 @@
|
|
1
|
+
--- !ruby/object:Gem::Specification
|
2
|
+
name: java-checkstyle
|
3
|
+
version: !ruby/object:Gem::Version
|
4
|
+
version: 0.0.2
|
5
|
+
platform: ruby
|
6
|
+
authors:
|
7
|
+
- Allen Madsen
|
8
|
+
- Cristian Oliveira
|
9
|
+
- Alex Rocha
|
10
|
+
autorequire:
|
11
|
+
bindir: bin
|
12
|
+
cert_chain: []
|
13
|
+
date: 2015-10-11 00:00:00.000000000 Z
|
14
|
+
dependencies:
|
15
|
+
- !ruby/object:Gem::Dependency
|
16
|
+
name: pre-commit
|
17
|
+
requirement: !ruby/object:Gem::Requirement
|
18
|
+
requirements:
|
19
|
+
- - "~>"
|
20
|
+
- !ruby/object:Gem::Version
|
21
|
+
version: '0.16'
|
22
|
+
type: :runtime
|
23
|
+
prerelease: false
|
24
|
+
version_requirements: !ruby/object:Gem::Requirement
|
25
|
+
requirements:
|
26
|
+
- - "~>"
|
27
|
+
- !ruby/object:Gem::Version
|
28
|
+
version: '0.16'
|
29
|
+
- !ruby/object:Gem::Dependency
|
30
|
+
name: crack
|
31
|
+
requirement: !ruby/object:Gem::Requirement
|
32
|
+
requirements:
|
33
|
+
- - "~>"
|
34
|
+
- !ruby/object:Gem::Version
|
35
|
+
version: 0.4.2
|
36
|
+
type: :runtime
|
37
|
+
prerelease: false
|
38
|
+
version_requirements: !ruby/object:Gem::Requirement
|
39
|
+
requirements:
|
40
|
+
- - "~>"
|
41
|
+
- !ruby/object:Gem::Version
|
42
|
+
version: 0.4.2
|
43
|
+
- !ruby/object:Gem::Dependency
|
44
|
+
name: bundler
|
45
|
+
requirement: !ruby/object:Gem::Requirement
|
46
|
+
requirements:
|
47
|
+
- - "~>"
|
48
|
+
- !ruby/object:Gem::Version
|
49
|
+
version: '1.5'
|
50
|
+
type: :development
|
51
|
+
prerelease: false
|
52
|
+
version_requirements: !ruby/object:Gem::Requirement
|
53
|
+
requirements:
|
54
|
+
- - "~>"
|
55
|
+
- !ruby/object:Gem::Version
|
56
|
+
version: '1.5'
|
57
|
+
- !ruby/object:Gem::Dependency
|
58
|
+
name: rake
|
59
|
+
requirement: !ruby/object:Gem::Requirement
|
60
|
+
requirements:
|
61
|
+
- - "~>"
|
62
|
+
- !ruby/object:Gem::Version
|
63
|
+
version: '10.4'
|
64
|
+
- - ">="
|
65
|
+
- !ruby/object:Gem::Version
|
66
|
+
version: 10.4.2
|
67
|
+
type: :development
|
68
|
+
prerelease: false
|
69
|
+
version_requirements: !ruby/object:Gem::Requirement
|
70
|
+
requirements:
|
71
|
+
- - "~>"
|
72
|
+
- !ruby/object:Gem::Version
|
73
|
+
version: '10.4'
|
74
|
+
- - ">="
|
75
|
+
- !ruby/object:Gem::Version
|
76
|
+
version: 10.4.2
|
77
|
+
- !ruby/object:Gem::Dependency
|
78
|
+
name: rspec
|
79
|
+
requirement: !ruby/object:Gem::Requirement
|
80
|
+
requirements:
|
81
|
+
- - "~>"
|
82
|
+
- !ruby/object:Gem::Version
|
83
|
+
version: '3.3'
|
84
|
+
- - ">="
|
85
|
+
- !ruby/object:Gem::Version
|
86
|
+
version: 3.3.2
|
87
|
+
type: :development
|
88
|
+
prerelease: false
|
89
|
+
version_requirements: !ruby/object:Gem::Requirement
|
90
|
+
requirements:
|
91
|
+
- - "~>"
|
92
|
+
- !ruby/object:Gem::Version
|
93
|
+
version: '3.3'
|
94
|
+
- - ">="
|
95
|
+
- !ruby/object:Gem::Version
|
96
|
+
version: 3.3.2
|
97
|
+
description: Checkstyle linter plugin for pre-commit. Useful for linting Java code.
|
98
|
+
email:
|
99
|
+
- blatyo@gmail.com
|
100
|
+
- contato@cristianoliveira.com.br
|
101
|
+
executables: []
|
102
|
+
extensions: []
|
103
|
+
extra_rdoc_files: []
|
104
|
+
files:
|
105
|
+
- ".codeclimate.yml"
|
106
|
+
- ".gitignore"
|
107
|
+
- ".rspec"
|
108
|
+
- ".travis.yml"
|
109
|
+
- Gemfile
|
110
|
+
- LICENSE.txt
|
111
|
+
- README.md
|
112
|
+
- Rakefile
|
113
|
+
- java-checkstyle.gemspec
|
114
|
+
- lib/plugins/pre_commit/checks/checkstyle.rb
|
115
|
+
- lib/plugins/pre_commit/message/extractor.rb
|
116
|
+
- lib/plugins/pre_commit/message/formatter.rb
|
117
|
+
- lib/plugins/pre_commit/support/path.rb
|
118
|
+
- lib/pre-commit/checkstyle/version.rb
|
119
|
+
- lib/pre-commit/support/checkstyle/checkstyle-5.7-all.jar
|
120
|
+
- lib/pre-commit/support/checkstyle/checkstyle-6.11-all.jar
|
121
|
+
- lib/pre-commit/support/checkstyle/google_checks.xml
|
122
|
+
- lib/pre-commit/support/checkstyle/sun_checks.xml
|
123
|
+
- spec/fixtures/bad.java
|
124
|
+
- spec/fixtures/bad2.java
|
125
|
+
- spec/fixtures/good.java
|
126
|
+
- spec/fixtures/google_checks.xml
|
127
|
+
- spec/plugins/pre_commit/checks/checkstyle_spec.rb
|
128
|
+
- spec/plugins/pre_commit/message/extractor_spec.rb
|
129
|
+
- spec/plugins/pre_commit/output/formatter.rb
|
130
|
+
- spec/spec_helper.rb
|
131
|
+
homepage: https://github.com/CristianOliveiraDaRosa/java-checkstyle
|
132
|
+
licenses:
|
133
|
+
- MIT
|
134
|
+
metadata: {}
|
135
|
+
post_install_message:
|
136
|
+
rdoc_options: []
|
137
|
+
require_paths:
|
138
|
+
- lib
|
139
|
+
required_ruby_version: !ruby/object:Gem::Requirement
|
140
|
+
requirements:
|
141
|
+
- - ">="
|
142
|
+
- !ruby/object:Gem::Version
|
143
|
+
version: '0'
|
144
|
+
required_rubygems_version: !ruby/object:Gem::Requirement
|
145
|
+
requirements:
|
146
|
+
- - ">="
|
147
|
+
- !ruby/object:Gem::Version
|
148
|
+
version: '0'
|
149
|
+
requirements: []
|
150
|
+
rubyforge_project:
|
151
|
+
rubygems_version: 2.4.8
|
152
|
+
signing_key:
|
153
|
+
specification_version: 4
|
154
|
+
summary: Checkstyle linter plugin for pre-commit
|
155
|
+
test_files:
|
156
|
+
- spec/fixtures/bad.java
|
157
|
+
- spec/fixtures/bad2.java
|
158
|
+
- spec/fixtures/good.java
|
159
|
+
- spec/fixtures/google_checks.xml
|
160
|
+
- spec/plugins/pre_commit/checks/checkstyle_spec.rb
|
161
|
+
- spec/plugins/pre_commit/message/extractor_spec.rb
|
162
|
+
- spec/plugins/pre_commit/output/formatter.rb
|
163
|
+
- spec/spec_helper.rb
|
164
|
+
has_rdoc:
|