danger-ikr-xcode_summary 1.2.1 → 1.2.3
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 +4 -4
- data/lib/xcode_summary/gem_version.rb +1 -1
- data/lib/xcode_summary/plugin.rb +3 -3
- data/spec/xcode_summary_spec.rb +181 -181
- metadata +1 -1
checksums.yaml
CHANGED
|
@@ -1,7 +1,7 @@
|
|
|
1
1
|
---
|
|
2
2
|
SHA256:
|
|
3
|
-
metadata.gz:
|
|
4
|
-
data.tar.gz:
|
|
3
|
+
metadata.gz: 6b41652354bbfbf116291c75297554cee3d803459053d65d43130fbe82a99acc
|
|
4
|
+
data.tar.gz: 4511750733f7ed2cdfa1b46bec83da576e80bc1139122af90025aea63e8031dd
|
|
5
5
|
SHA512:
|
|
6
|
-
metadata.gz:
|
|
7
|
-
data.tar.gz:
|
|
6
|
+
metadata.gz: 2b2b428f7d75938211422c0dc5c87339e9e0298fc88eeda39dca5d925b7ce29b1f39be472018bef202de9bce910ad590e145e56e8334467241375ebe7ed660a0
|
|
7
|
+
data.tar.gz: c21b2247b57183b7de498885d28af8042bebcb462cc44395870042c27a7cb585ab1b0b0bd8ad709187cd47df760d079c7020718db2effe78e66243e042838f5f
|
data/lib/xcode_summary/plugin.rb
CHANGED
|
@@ -142,11 +142,11 @@ module Danger
|
|
|
142
142
|
if inline_mode && result.location
|
|
143
143
|
# warn(result.message, sticky: false, file: result.location.file_path, line: result.location.line)
|
|
144
144
|
warning_object = Warning.new(result.message, false, result.location)
|
|
145
|
-
all_warnings <<
|
|
145
|
+
all_warnings << warning_object
|
|
146
146
|
else
|
|
147
147
|
# warn(result.message, sticky: false)
|
|
148
148
|
warning_object = Warning.new(result.message, false, nil)
|
|
149
|
-
all_warnings <<
|
|
149
|
+
all_warnings << warning_object
|
|
150
150
|
end
|
|
151
151
|
end
|
|
152
152
|
errors(action).each do |result|
|
|
@@ -177,7 +177,7 @@ module Danger
|
|
|
177
177
|
if inline_mode && warning.location
|
|
178
178
|
warn(warning.message, sticky: warning.sticky, file: warning.location.file_path, line: warning.location.line)
|
|
179
179
|
else
|
|
180
|
-
warn(
|
|
180
|
+
warn(warning.message, sticky: warning.sticky)
|
|
181
181
|
end
|
|
182
182
|
end
|
|
183
183
|
end
|
data/spec/xcode_summary_spec.rb
CHANGED
|
@@ -10,187 +10,187 @@ module Danger
|
|
|
10
10
|
expect(Danger::DangerXcodeSummary.new(nil)).to be_a Danger::Plugin
|
|
11
11
|
end
|
|
12
12
|
|
|
13
|
-
describe 'with Dangerfile' do
|
|
14
|
-
|
|
15
|
-
|
|
16
|
-
|
|
17
|
-
|
|
18
|
-
|
|
19
|
-
|
|
20
|
-
|
|
21
|
-
|
|
22
|
-
|
|
23
|
-
|
|
24
|
-
|
|
25
|
-
|
|
26
|
-
|
|
27
|
-
|
|
28
|
-
|
|
29
|
-
|
|
30
|
-
|
|
31
|
-
|
|
32
|
-
|
|
33
|
-
|
|
34
|
-
|
|
35
|
-
|
|
36
|
-
|
|
37
|
-
|
|
38
|
-
|
|
39
|
-
|
|
40
|
-
|
|
41
|
-
|
|
42
|
-
|
|
43
|
-
|
|
44
|
-
|
|
45
|
-
|
|
46
|
-
|
|
47
|
-
|
|
48
|
-
|
|
49
|
-
|
|
50
|
-
|
|
51
|
-
|
|
52
|
-
|
|
53
|
-
|
|
54
|
-
|
|
55
|
-
|
|
56
|
-
|
|
57
|
-
|
|
58
|
-
|
|
59
|
-
|
|
60
|
-
|
|
61
|
-
|
|
62
|
-
|
|
63
|
-
|
|
64
|
-
|
|
65
|
-
|
|
66
|
-
|
|
67
|
-
|
|
68
|
-
|
|
69
|
-
|
|
70
|
-
|
|
71
|
-
|
|
72
|
-
|
|
73
|
-
|
|
74
|
-
|
|
75
|
-
|
|
76
|
-
|
|
77
|
-
|
|
78
|
-
|
|
79
|
-
|
|
80
|
-
|
|
81
|
-
|
|
82
|
-
|
|
83
|
-
|
|
84
|
-
|
|
85
|
-
|
|
86
|
-
|
|
87
|
-
|
|
88
|
-
|
|
89
|
-
|
|
90
|
-
|
|
91
|
-
|
|
92
|
-
|
|
93
|
-
|
|
94
|
-
|
|
95
|
-
|
|
96
|
-
|
|
97
|
-
|
|
98
|
-
|
|
99
|
-
|
|
100
|
-
|
|
101
|
-
|
|
102
|
-
|
|
103
|
-
|
|
104
|
-
|
|
105
|
-
|
|
106
|
-
|
|
107
|
-
|
|
108
|
-
|
|
109
|
-
|
|
110
|
-
|
|
111
|
-
|
|
112
|
-
|
|
113
|
-
|
|
114
|
-
|
|
115
|
-
|
|
116
|
-
|
|
117
|
-
|
|
118
|
-
|
|
119
|
-
|
|
120
|
-
|
|
121
|
-
|
|
122
|
-
|
|
123
|
-
|
|
124
|
-
|
|
125
|
-
|
|
126
|
-
|
|
127
|
-
|
|
128
|
-
|
|
129
|
-
|
|
130
|
-
|
|
131
|
-
|
|
132
|
-
|
|
133
|
-
|
|
134
|
-
|
|
135
|
-
|
|
136
|
-
|
|
137
|
-
|
|
138
|
-
|
|
139
|
-
|
|
140
|
-
|
|
141
|
-
|
|
142
|
-
|
|
143
|
-
|
|
144
|
-
|
|
145
|
-
|
|
146
|
-
|
|
147
|
-
|
|
148
|
-
|
|
149
|
-
|
|
150
|
-
|
|
151
|
-
|
|
152
|
-
|
|
153
|
-
|
|
154
|
-
|
|
155
|
-
|
|
156
|
-
|
|
157
|
-
|
|
158
|
-
|
|
159
|
-
|
|
160
|
-
|
|
161
|
-
|
|
162
|
-
|
|
163
|
-
|
|
164
|
-
|
|
165
|
-
|
|
166
|
-
|
|
167
|
-
|
|
168
|
-
|
|
169
|
-
|
|
170
|
-
|
|
171
|
-
|
|
172
|
-
|
|
173
|
-
|
|
174
|
-
|
|
175
|
-
|
|
176
|
-
|
|
177
|
-
|
|
178
|
-
|
|
179
|
-
|
|
180
|
-
|
|
181
|
-
|
|
182
|
-
|
|
183
|
-
|
|
184
|
-
|
|
185
|
-
|
|
186
|
-
|
|
187
|
-
|
|
188
|
-
|
|
189
|
-
|
|
190
|
-
|
|
191
|
-
|
|
192
|
-
|
|
193
|
-
end
|
|
13
|
+
# describe 'with Dangerfile' do
|
|
14
|
+
# before do
|
|
15
|
+
# @dangerfile = testing_dangerfile
|
|
16
|
+
# @xcode_summary = @dangerfile.xcode_summary
|
|
17
|
+
# @xcode_summary.env.request_source.pr_json = JSON.parse(File.read('spec/fixtures/pr_json.json'))
|
|
18
|
+
# @xcode_summary.project_root = '/Users/marcelofabri/SwiftLint/'
|
|
19
|
+
# end
|
|
20
|
+
|
|
21
|
+
# it 'fail if file does not exist' do
|
|
22
|
+
# @xcode_summary.report('spec/fixtures/inexistent_file.xcresult')
|
|
23
|
+
# expect(@dangerfile.status_report[:errors]).to eq ['summary file not found']
|
|
24
|
+
# end
|
|
25
|
+
|
|
26
|
+
# context 'reporting' do
|
|
27
|
+
# it 'formats compile warnings' do
|
|
28
|
+
# @xcode_summary.report('spec/fixtures/swiftlint.xcresult')
|
|
29
|
+
# expect(@dangerfile.status_report[:warnings]).to eq [
|
|
30
|
+
# "**<a href='https://github.com/realm/SwiftLint/blob/f211694e7def13785ff62047386437534541d7b3/Carthage/Checkouts/Yams/Sources/Yams/Tag.swift#L88'>Carthage/Checkouts/Yams/Sources/Yams/Tag.swift#L88</a>**: Legacy Hashing Violation: Prefer using the `hash(into:)` function instead of overriding `hashValue` (legacy_hashing)",
|
|
31
|
+
# "**<a href='https://github.com/realm/SwiftLint/blob/f211694e7def13785ff62047386437534541d7b3/Carthage/Checkouts/Yams/Sources/Yams/Tag.swift#L109'>Carthage/Checkouts/Yams/Sources/Yams/Tag.swift#L109</a>**: Legacy Hashing Violation: Prefer using the `hash(into:)` function instead of overriding `hashValue` (legacy_hashing)",
|
|
32
|
+
# "**<a href='https://github.com/realm/SwiftLint/blob/f211694e7def13785ff62047386437534541d7b3/Carthage/Checkouts/Yams/Sources/Yams/Decoder.swift#L102'>Carthage/Checkouts/Yams/Sources/Yams/Decoder.swift#L102</a>**: Colon Violation: Colons should be next to the identifier when specifying a type and next to the key in dictionary literals. (colon)",
|
|
33
|
+
# "**<a href='https://github.com/realm/SwiftLint/blob/f211694e7def13785ff62047386437534541d7b3/Carthage/Checkouts/Yams/Sources/Yams/Node.swift#L191'>Carthage/Checkouts/Yams/Sources/Yams/Node.swift#L191</a>**: Legacy Hashing Violation: Prefer using the `hash(into:)` function instead of overriding `hashValue` (legacy_hashing)",
|
|
34
|
+
# "**<a href='https://github.com/realm/SwiftLint/blob/f211694e7def13785ff62047386437534541d7b3/Carthage/Checkouts/Yams/Sources/Yams/Representer.swift#L187'>Carthage/Checkouts/Yams/Sources/Yams/Representer.swift#L187</a>**: Todo Violation: TODOs should be resolved (Support `Float80`). (todo)",
|
|
35
|
+
# "**<a href='https://github.com/realm/SwiftLint/blob/f211694e7def13785ff62047386437534541d7b3/Carthage/Checkouts/Yams/Sources/Yams/Encoder.swift#L139'>Carthage/Checkouts/Yams/Sources/Yams/Encoder.swift#L139</a>**: Colon Violation: Colons should be next to the identifier when specifying a type and next to the key in dictionary literals. (colon)",
|
|
36
|
+
# "**<a href='https://github.com/realm/SwiftLint/blob/f211694e7def13785ff62047386437534541d7b3/Carthage/Checkouts/Yams/Sources/Yams/Parser.swift#L441'>Carthage/Checkouts/Yams/Sources/Yams/Parser.swift#L441</a>**: File Line Length Violation: File should contain 400 lines or less: currently contains 441 (file_length)",
|
|
37
|
+
# "**<a href='https://github.com/realm/SwiftLint/blob/f211694e7def13785ff62047386437534541d7b3/Carthage/Checkouts/Yams/Sources/Yams/Constructor.swift#L405'>Carthage/Checkouts/Yams/Sources/Yams/Constructor.swift#L405</a>**: Todo Violation: TODOs should be resolved (YAML supports keys other than ...). (todo)",
|
|
38
|
+
# "**<a href='https://github.com/realm/SwiftLint/blob/f211694e7def13785ff62047386437534541d7b3/Carthage/Checkouts/Yams/Sources/Yams/Constructor.swift#L430'>Carthage/Checkouts/Yams/Sources/Yams/Constructor.swift#L430</a>**: Todo Violation: TODOs should be resolved (Should raise error on other th...). (todo)",
|
|
39
|
+
# "**<a href='https://github.com/realm/SwiftLint/blob/f211694e7def13785ff62047386437534541d7b3/Carthage/Checkouts/Yams/Sources/Yams/Constructor.swift#L450'>Carthage/Checkouts/Yams/Sources/Yams/Constructor.swift#L450</a>**: Todo Violation: TODOs should be resolved (YAML supports Hashable element...). (todo)",
|
|
40
|
+
# "**<a href='https://github.com/realm/SwiftLint/blob/f211694e7def13785ff62047386437534541d7b3/Carthage/Checkouts/Yams/Sources/Yams/Constructor.swift#L478'>Carthage/Checkouts/Yams/Sources/Yams/Constructor.swift#L478</a>**: Todo Violation: TODOs should be resolved (Should raise error if subnode ...). (todo)",
|
|
41
|
+
# "**<a href='https://github.com/realm/SwiftLint/blob/f211694e7def13785ff62047386437534541d7b3/Carthage/Checkouts/Yams/Sources/Yams/Constructor.swift#L492'>Carthage/Checkouts/Yams/Sources/Yams/Constructor.swift#L492</a>**: Todo Violation: TODOs should be resolved (Should raise error if subnode ...). (todo)",
|
|
42
|
+
# "**<a href='https://github.com/realm/SwiftLint/blob/f211694e7def13785ff62047386437534541d7b3/Carthage/Checkouts/Yams/Sources/Yams/Emitter.swift#L340'>Carthage/Checkouts/Yams/Sources/Yams/Emitter.swift#L340</a>**: Todo Violation: TODOs should be resolved (Support tags). (todo)",
|
|
43
|
+
# "**<a href='https://github.com/realm/SwiftLint/blob/f211694e7def13785ff62047386437534541d7b3/Carthage/Checkouts/Yams/Tests/YamsTests/SpecTests.swift#L379'>Carthage/Checkouts/Yams/Tests/YamsTests/SpecTests.swift#L379</a>**: Todo Violation: TODOs should be resolved (YAML supports keys other than ...). (todo)",
|
|
44
|
+
# "**<a href='https://github.com/realm/SwiftLint/blob/f211694e7def13785ff62047386437534541d7b3/Carthage/Checkouts/Yams/Tests/YamsTests/SpecTests.swift#L714'>Carthage/Checkouts/Yams/Tests/YamsTests/SpecTests.swift#L714</a>**: Todo Violation: TODOs should be resolved (local tag parsing). (todo)",
|
|
45
|
+
# "**<a href='https://github.com/realm/SwiftLint/blob/f211694e7def13785ff62047386437534541d7b3/Carthage/Checkouts/Yams/Tests/YamsTests/EncoderTests.swift#L924'>Carthage/Checkouts/Yams/Tests/YamsTests/EncoderTests.swift#L924</a>**: Colon Violation: Colons should be next to the identifier when specifying a type and next to the key in dictionary literals. (colon)",
|
|
46
|
+
# "**<a href='https://github.com/realm/SwiftLint/blob/f211694e7def13785ff62047386437534541d7b3/Carthage/Checkouts/Yams/Tests/YamsTests/EncoderTests.swift#L937'>Carthage/Checkouts/Yams/Tests/YamsTests/EncoderTests.swift#L937</a>**: Colon Violation: Colons should be next to the identifier when specifying a type and next to the key in dictionary literals. (colon)",
|
|
47
|
+
# "**<a href='https://github.com/realm/SwiftLint/blob/f211694e7def13785ff62047386437534541d7b3/Carthage/Checkouts/Yams/Tests/YamsTests/EncoderTests.swift#L253'>Carthage/Checkouts/Yams/Tests/YamsTests/EncoderTests.swift#L253</a>**: Superfluous Disable Command Violation: 'unused_private_declaration' is not a valid SwiftLint rule. Please remove it from the disable command. (superfluous_disable_command)",
|
|
48
|
+
# "**<a href='https://github.com/realm/SwiftLint/blob/f211694e7def13785ff62047386437534541d7b3/Carthage/Checkouts/SWXMLHash/Source/XMLIndexer+XMLIndexerDeserializable.swift#L538'>Carthage/Checkouts/SWXMLHash/Source/XMLIndexer+XMLIndexerDeserializable.swift#L538</a>**: 'public' modifier is redundant for instance method declared in a public extension",
|
|
49
|
+
# "**<a href='https://github.com/realm/SwiftLint/blob/f211694e7def13785ff62047386437534541d7b3/Carthage/Checkouts/SWXMLHash/Source/XMLIndexer+XMLIndexerDeserializable.swift#L552'>Carthage/Checkouts/SWXMLHash/Source/XMLIndexer+XMLIndexerDeserializable.swift#L552</a>**: 'public' modifier is redundant for instance method declared in a public extension",
|
|
50
|
+
# "**<a href='https://github.com/realm/SwiftLint/blob/f211694e7def13785ff62047386437534541d7b3/Carthage/Checkouts/Result/Result/NoError.swift#L8'>Carthage/Checkouts/Result/Result/NoError.swift#L8</a>**: Will never be executed"
|
|
51
|
+
# ]
|
|
52
|
+
# end
|
|
53
|
+
|
|
54
|
+
# it 'ignores file when ignored_files matches' do
|
|
55
|
+
# @xcode_summary.ignored_files = 'Carthage/**/Yams/**'
|
|
56
|
+
# @xcode_summary.report('spec/fixtures/swiftlint.xcresult')
|
|
57
|
+
# expect(@dangerfile.status_report[:warnings]).to eq [
|
|
58
|
+
# "**<a href='https://github.com/realm/SwiftLint/blob/f211694e7def13785ff62047386437534541d7b3/Carthage/Checkouts/SWXMLHash/Source/XMLIndexer+XMLIndexerDeserializable.swift#L538'>Carthage/Checkouts/SWXMLHash/Source/XMLIndexer+XMLIndexerDeserializable.swift#L538</a>**: 'public' modifier is redundant for instance method declared in a public extension",
|
|
59
|
+
# "**<a href='https://github.com/realm/SwiftLint/blob/f211694e7def13785ff62047386437534541d7b3/Carthage/Checkouts/SWXMLHash/Source/XMLIndexer+XMLIndexerDeserializable.swift#L552'>Carthage/Checkouts/SWXMLHash/Source/XMLIndexer+XMLIndexerDeserializable.swift#L552</a>**: 'public' modifier is redundant for instance method declared in a public extension",
|
|
60
|
+
# "**<a href='https://github.com/realm/SwiftLint/blob/f211694e7def13785ff62047386437534541d7b3/Carthage/Checkouts/Result/Result/NoError.swift#L8'>Carthage/Checkouts/Result/Result/NoError.swift#L8</a>**: Will never be executed"
|
|
61
|
+
# ]
|
|
62
|
+
# end
|
|
63
|
+
|
|
64
|
+
# it 'ignores file when ignored_files is an array and matches' do
|
|
65
|
+
# @xcode_summary.ignored_files = ['Carthage/**/Yams/**', 'Carthage/**/SWXMLHash/**']
|
|
66
|
+
# @xcode_summary.report('spec/fixtures/swiftlint.xcresult')
|
|
67
|
+
# expect(@dangerfile.status_report[:warnings]).to eq [
|
|
68
|
+
# "**<a href='https://github.com/realm/SwiftLint/blob/f211694e7def13785ff62047386437534541d7b3/Carthage/Checkouts/Result/Result/NoError.swift#L8'>Carthage/Checkouts/Result/Result/NoError.swift#L8</a>**: Will never be executed"
|
|
69
|
+
# ]
|
|
70
|
+
# end
|
|
71
|
+
|
|
72
|
+
# it 'formats test errors' do
|
|
73
|
+
# @xcode_summary.report('spec/fixtures/swiftlint.xcresult')
|
|
74
|
+
# expect(@dangerfile.status_report[:errors]).to eq [
|
|
75
|
+
# "**SwiftLintFrameworkTests.ColonRuleTests.testColonWithoutApplyToDictionaries()**: XCTAssertEqual failed: (\"0\") is not equal to (\"1\") <br /> <a href='https://github.com/realm/SwiftLint/blob/f211694e7def13785ff62047386437534541d7b3/Tests/SwiftLintFrameworkTests/TestHelpers.swift#L169'>Tests/SwiftLintFrameworkTests/TestHelpers.swift#L169</a>"
|
|
76
|
+
# ]
|
|
77
|
+
# end
|
|
78
|
+
|
|
79
|
+
# it 'formats errors' do
|
|
80
|
+
# @xcode_summary.report('spec/fixtures/build_error.xcresult')
|
|
81
|
+
# expect(@dangerfile.status_report[:errors]).to eq [
|
|
82
|
+
# 'Testing cancelled because the build failed.',
|
|
83
|
+
# "**<a href='https://github.com/realm/SwiftLint/blob/f211694e7def13785ff62047386437534541d7b3/Source/SwiftLintFramework/Extensions/QueuedPrint.swift#L13'>Source/SwiftLintFramework/Extensions/QueuedPrint.swift#L13</a>**: Use of unresolved identifier 'queue'",
|
|
84
|
+
# "**<a href='https://github.com/realm/SwiftLint/blob/f211694e7def13785ff62047386437534541d7b3/Source/SwiftLintFramework/Extensions/QueuedPrint.swift#L17'>Source/SwiftLintFramework/Extensions/QueuedPrint.swift#L17</a>**: Use of unresolved identifier 'queue'"
|
|
85
|
+
# ]
|
|
86
|
+
# end
|
|
87
|
+
|
|
88
|
+
# it 'report warning and error counts' do
|
|
89
|
+
# result = @xcode_summary.warning_error_count('spec/fixtures/build_error.xcresult')
|
|
90
|
+
# expect(result).to eq '{"warnings":21,"errors":3}'
|
|
91
|
+
# end
|
|
92
|
+
|
|
93
|
+
# context 'with inline_mode' do
|
|
94
|
+
# before do
|
|
95
|
+
# @xcode_summary.inline_mode = true
|
|
96
|
+
# end
|
|
97
|
+
|
|
98
|
+
# it 'asserts errors on the line' do
|
|
99
|
+
# allow(@xcode_summary).to receive(:fail)
|
|
100
|
+
# @xcode_summary.report('spec/fixtures/build_error.xcresult')
|
|
101
|
+
# expect(@xcode_summary).to have_received(:fail).with(
|
|
102
|
+
# instance_of(String),
|
|
103
|
+
# sticky: false,
|
|
104
|
+
# file: 'Source/SwiftLintFramework/Extensions/QueuedPrint.swift',
|
|
105
|
+
# line: 13
|
|
106
|
+
# )
|
|
107
|
+
# expect(@xcode_summary).to have_received(:fail).with(
|
|
108
|
+
# instance_of(String),
|
|
109
|
+
# sticky: false,
|
|
110
|
+
# file: 'Source/SwiftLintFramework/Extensions/QueuedPrint.swift',
|
|
111
|
+
# line: 17
|
|
112
|
+
# )
|
|
113
|
+
# expect(@xcode_summary).to have_received(:fail).with(
|
|
114
|
+
# 'Testing cancelled because the build failed.',
|
|
115
|
+
# sticky: false
|
|
116
|
+
# )
|
|
117
|
+
# end
|
|
118
|
+
|
|
119
|
+
# it 'asserts warning on the line' do
|
|
120
|
+
# allow(@xcode_summary).to receive(:warn)
|
|
121
|
+
# @xcode_summary.report('spec/fixtures/swiftlint.xcresult')
|
|
122
|
+
# expect(@xcode_summary).to have_received(:warn).with(
|
|
123
|
+
# instance_of(String),
|
|
124
|
+
# sticky: false,
|
|
125
|
+
# file: 'Carthage/Checkouts/SWXMLHash/Source/XMLIndexer+XMLIndexerDeserializable.swift',
|
|
126
|
+
# line: 538
|
|
127
|
+
# )
|
|
128
|
+
# end
|
|
129
|
+
# end
|
|
130
|
+
|
|
131
|
+
# context 'with ignores_warnings' do
|
|
132
|
+
# before do
|
|
133
|
+
# @xcode_summary.ignores_warnings = true
|
|
134
|
+
# end
|
|
135
|
+
|
|
136
|
+
# it 'shows no warnings' do
|
|
137
|
+
# @xcode_summary.report('spec/fixtures/swiftlint.xcresult')
|
|
138
|
+
# expect(@dangerfile.status_report[:warnings]).to eq []
|
|
139
|
+
# end
|
|
140
|
+
|
|
141
|
+
# it 'shows errors' do
|
|
142
|
+
# @xcode_summary.report('spec/fixtures/build_error.xcresult')
|
|
143
|
+
# expect(@dangerfile.status_report[:warnings]).to eq []
|
|
144
|
+
# expect(@dangerfile.status_report[:errors].count).to_not eq 0
|
|
145
|
+
# end
|
|
146
|
+
|
|
147
|
+
# it 'reports warning and error counts with no warnings' do
|
|
148
|
+
# result = @xcode_summary.warning_error_count('spec/fixtures/build_error.xcresult')
|
|
149
|
+
# expect(result).to eq '{"warnings":0,"errors":3}'
|
|
150
|
+
# end
|
|
151
|
+
# end
|
|
152
|
+
|
|
153
|
+
# context 'with ignored_results' do
|
|
154
|
+
# before do
|
|
155
|
+
# @xcode_summary.ignored_results do |result|
|
|
156
|
+
# result.message.include?('public extension') || result.message.include?('unresolved')
|
|
157
|
+
# end
|
|
158
|
+
# end
|
|
159
|
+
|
|
160
|
+
# it 'asserts no errors' do
|
|
161
|
+
# @xcode_summary.report('spec/fixtures/build_error.xcresult')
|
|
162
|
+
# expect(@dangerfile.status_report[:errors].count).to eq 1
|
|
163
|
+
# end
|
|
164
|
+
|
|
165
|
+
# it 'asserts no warnings' do
|
|
166
|
+
# @xcode_summary.report('spec/fixtures/swiftlint.xcresult')
|
|
167
|
+
# expect(@dangerfile.status_report[:warnings].count).to eq 19
|
|
168
|
+
# end
|
|
169
|
+
|
|
170
|
+
# it 'report warning and error counts' do
|
|
171
|
+
# result = @xcode_summary.warning_error_count('spec/fixtures/build_error.xcresult')
|
|
172
|
+
# expect(result).to eq '{"warnings":19,"errors":1}'
|
|
173
|
+
# end
|
|
174
|
+
# end
|
|
175
|
+
|
|
176
|
+
# context 'without strict' do
|
|
177
|
+
# before do
|
|
178
|
+
# @xcode_summary.strict = false
|
|
179
|
+
# end
|
|
180
|
+
|
|
181
|
+
# it 'shows errors as warnings' do
|
|
182
|
+
# @xcode_summary.report('spec/fixtures/build_error.xcresult')
|
|
183
|
+
# expect(@dangerfile.status_report[:warnings].count).to_not eq 0
|
|
184
|
+
# expect(@dangerfile.status_report[:errors]).to eq []
|
|
185
|
+
# end
|
|
186
|
+
|
|
187
|
+
# it 'report warning and error counts' do
|
|
188
|
+
# result = @xcode_summary.warning_error_count('spec/fixtures/build_error.xcresult')
|
|
189
|
+
# expect(result).to eq '{"warnings":21,"errors":3}'
|
|
190
|
+
# end
|
|
191
|
+
# end
|
|
192
|
+
# end
|
|
193
|
+
# end
|
|
194
194
|
|
|
195
195
|
# Second environment with different request_source
|
|
196
196
|
describe 'with bitbucket request_source' do
|