repo_analyzer 1.4.0 → 1.6.0
Sign up to get free protection for your applications and to get access to all the features.
- checksums.yaml +4 -4
- data/CHANGELOG.md +12 -0
- data/Gemfile.lock +1 -1
- data/app/extractors/repo_analyzer/bundler_stats_extractor.rb +18 -2
- data/app/jobs/repo_analyzer/extract_project_info_job.rb +2 -2
- data/lib/repo_analyzer/version.rb +1 -1
- data/lib/tasks/repo_analyzer_tasks.rake +4 -2
- data/spec/extractors/bundler_stats_extractor_spec.rb +33 -259
- data/spec/jobs/repo_analyzer/extract_project_info_job_spec.rb +3 -2
- metadata +2 -5
- data/app/extractors/repo_analyzer/fasterer_extractor.rb +0 -34
- data/spec/extractors/fasterer_extractor_spec.rb +0 -45
checksums.yaml
CHANGED
|
@@ -1,7 +1,7 @@
|
|
|
1
1
|
---
|
|
2
2
|
SHA256:
|
|
3
|
-
metadata.gz:
|
|
4
|
-
data.tar.gz:
|
|
3
|
+
metadata.gz: b3ba0febd4ab0260ef41e02596f408eed4ce7841467be48339f95a6fe86ebe35
|
|
4
|
+
data.tar.gz: 58f19d2a5c0fc9708eedfbef74ed7930aabaf928679d8da5392a4533927dfb15
|
|
5
5
|
SHA512:
|
|
6
|
-
metadata.gz:
|
|
7
|
-
data.tar.gz:
|
|
6
|
+
metadata.gz: a14e02b846867e11384b3034b0d70076cbff20e005a32d0ca9d8eda05331276b79bcd931c0d48a66a6911da7ca861f77ef699aa5f9892f8cbb93eb4bbc2e9d89
|
|
7
|
+
data.tar.gz: c675a4d5e84b1624f1af32ec1abbaec1bb7daa3ce880d3c7731250257c77c7df02687c22a98858af208fc3a6b1d0ea38298ed1b74ac99b269abe23dd5fe884cf
|
data/CHANGELOG.md
CHANGED
|
@@ -1,6 +1,18 @@
|
|
|
1
1
|
# Change Log
|
|
2
2
|
All notable changes to this project will be documented in this file.
|
|
3
3
|
This project adheres to [Semantic Versioning](http://semver.org/).
|
|
4
|
+
|
|
5
|
+
### v1.6.0
|
|
6
|
+
#### Added
|
|
7
|
+
* Complete [bundler-stats](https://github.com/jmmastey/bundler-stats) extractor data with potential removals
|
|
8
|
+
|
|
9
|
+
### v1.5.0
|
|
10
|
+
#### Added
|
|
11
|
+
|
|
12
|
+
* Run analyze rake task with custom project path.
|
|
13
|
+
#### Fixed
|
|
14
|
+
|
|
15
|
+
* Remove [fasterer](https://github.com/DamirSvrtan/fasterer) extractor.
|
|
4
16
|
### v1.4.0
|
|
5
17
|
#### Added
|
|
6
18
|
|
data/Gemfile.lock
CHANGED
|
@@ -3,8 +3,24 @@ module RepoAnalyzer
|
|
|
3
3
|
private
|
|
4
4
|
|
|
5
5
|
def extracted_info
|
|
6
|
-
|
|
7
|
-
|
|
6
|
+
stats['gems'] = gems_info
|
|
7
|
+
stats
|
|
8
|
+
end
|
|
9
|
+
|
|
10
|
+
def gems_info
|
|
11
|
+
gem_names.inject([]) do |memo, gem_name|
|
|
12
|
+
result = `bundle exec bundle-stats show '#{gem_name}' -f json`
|
|
13
|
+
memo << JSON.parse(result)
|
|
14
|
+
memo
|
|
15
|
+
end
|
|
16
|
+
end
|
|
17
|
+
|
|
18
|
+
def gem_names
|
|
19
|
+
stats['gems'].map { |gem| gem['name'] }
|
|
20
|
+
end
|
|
21
|
+
|
|
22
|
+
def stats
|
|
23
|
+
@stats ||= JSON.parse(`bundle exec bundle-stats stats -f json`)
|
|
8
24
|
end
|
|
9
25
|
end
|
|
10
26
|
end
|
|
@@ -1,8 +1,8 @@
|
|
|
1
1
|
module RepoAnalyzer
|
|
2
2
|
class ExtractProjectInfoJob < ApplicationJob
|
|
3
|
-
def perform(repo_name)
|
|
3
|
+
def perform(repo_name, project_path)
|
|
4
4
|
project_info = {}
|
|
5
|
-
bridge = RepoAnalyzer::ProjectDataBridge.new(repo_name)
|
|
5
|
+
bridge = RepoAnalyzer::ProjectDataBridge.new(repo_name, project_path)
|
|
6
6
|
|
|
7
7
|
for_each_extractor do |extractor|
|
|
8
8
|
extracted_data = extractor.new(bridge).extract
|
|
@@ -1,7 +1,9 @@
|
|
|
1
1
|
namespace :repo_analyzer do
|
|
2
2
|
desc "Extract repo info and post to defined endpoint"
|
|
3
|
-
task :analyze, [:repo_name] => :environment do |_t, args|
|
|
4
|
-
project_info = RepoAnalyzer::ExtractProjectInfoJob.perform_now(
|
|
3
|
+
task :analyze, [:repo_name, :project_path] => :environment do |_t, args|
|
|
4
|
+
project_info = RepoAnalyzer::ExtractProjectInfoJob.perform_now(
|
|
5
|
+
args.repo_name, args.project_path
|
|
6
|
+
)
|
|
5
7
|
RepoAnalyzer::PostExtractedInfoJob.perform_now(args.repo_name, project_info)
|
|
6
8
|
end
|
|
7
9
|
end
|
|
@@ -2,6 +2,29 @@ require 'rails_helper'
|
|
|
2
2
|
|
|
3
3
|
describe RepoAnalyzer::BundlerStatsExtractor, repo_analyzer_extractor_helpers: true do
|
|
4
4
|
describe "#extract" do
|
|
5
|
+
let(:rspec_junit_formatter) do
|
|
6
|
+
{
|
|
7
|
+
"name": "rspec_junit_formatter",
|
|
8
|
+
"total_dependencies": 2,
|
|
9
|
+
"first_level_dependencies": 1,
|
|
10
|
+
"top_level_dependencies": {},
|
|
11
|
+
"transitive_dependencies": [
|
|
12
|
+
"rspec-core (>= 2, < 4, != 2.12.0)",
|
|
13
|
+
"rspec-support (~> 3.12.0)"
|
|
14
|
+
]
|
|
15
|
+
}
|
|
16
|
+
end
|
|
17
|
+
|
|
18
|
+
let(:sqlite) do
|
|
19
|
+
{
|
|
20
|
+
"name": "sqlite",
|
|
21
|
+
"total_dependencies": 0,
|
|
22
|
+
"first_level_dependencies": 0,
|
|
23
|
+
"top_level_dependencies": {},
|
|
24
|
+
"transitive_dependencies": []
|
|
25
|
+
}
|
|
26
|
+
end
|
|
27
|
+
|
|
5
28
|
let(:stats) do
|
|
6
29
|
{
|
|
7
30
|
"summary": {
|
|
@@ -11,265 +34,8 @@ describe RepoAnalyzer::BundlerStatsExtractor, repo_analyzer_extractor_helpers: t
|
|
|
11
34
|
"github": 0
|
|
12
35
|
},
|
|
13
36
|
"gems": [
|
|
14
|
-
|
|
15
|
-
|
|
16
|
-
"total_dependencies": 84,
|
|
17
|
-
"first_level_dependencies": 14,
|
|
18
|
-
"top_level_dependencies": {},
|
|
19
|
-
"transitive_dependencies": [
|
|
20
|
-
"brakeman (>= 0)",
|
|
21
|
-
"bundler-audit (>= 0)",
|
|
22
|
-
"bundler-stats (>= 0)",
|
|
23
|
-
"faraday-retry (>= 0)",
|
|
24
|
-
"fasterer (>= 0)",
|
|
25
|
-
"octokit (~> 4.0)",
|
|
26
|
-
"rails (>= 6.0)",
|
|
27
|
-
"rails_best_practices (>= 0)",
|
|
28
|
-
"reek (>= 0)",
|
|
29
|
-
"rubocop (~> 1.9)",
|
|
30
|
-
"rubocop-performance (>= 0)",
|
|
31
|
-
"rubocop-platanus (~> 0.2)",
|
|
32
|
-
"rubocop-rails (>= 0)",
|
|
33
|
-
"rubocop-rspec (~> 2.2)",
|
|
34
|
-
"bundler (>= 1.2.0, < 3)",
|
|
35
|
-
"thor (~> 1.0)",
|
|
36
|
-
"faraday (~> 2.0)",
|
|
37
|
-
"faraday-net_http (>= 2.0, < 3.1)",
|
|
38
|
-
"ruby2_keywords (>= 0.0.4)",
|
|
39
|
-
"colorize (~> 0.7)",
|
|
40
|
-
"ruby_parser (>= 3.19.1)",
|
|
41
|
-
"sexp_processor (~> 4.16)",
|
|
42
|
-
"sawyer (~> 0.9)",
|
|
43
|
-
"addressable (>= 2.3.5)",
|
|
44
|
-
"public_suffix (>= 2.0.2, < 6.0)",
|
|
45
|
-
"actioncable (= 6.1.7)",
|
|
46
|
-
"actionmailbox (= 6.1.7)",
|
|
47
|
-
"actionmailer (= 6.1.7)",
|
|
48
|
-
"actionpack (= 6.1.7)",
|
|
49
|
-
"actiontext (= 6.1.7)",
|
|
50
|
-
"actionview (= 6.1.7)",
|
|
51
|
-
"activejob (= 6.1.7)",
|
|
52
|
-
"activemodel (= 6.1.7)",
|
|
53
|
-
"activerecord (= 6.1.7)",
|
|
54
|
-
"activestorage (= 6.1.7)",
|
|
55
|
-
"activesupport (= 6.1.7)",
|
|
56
|
-
"railties (= 6.1.7)",
|
|
57
|
-
"sprockets-rails (>= 2.0.0)",
|
|
58
|
-
"nio4r (~> 2.0)",
|
|
59
|
-
"websocket-driver (>= 0.6.1)",
|
|
60
|
-
"rack (~> 2.0, >= 2.0.9)",
|
|
61
|
-
"rack-test (>= 0.6.3)",
|
|
62
|
-
"rails-dom-testing (~> 2.0)",
|
|
63
|
-
"rails-html-sanitizer (~> 1.0, >= 1.2.0)",
|
|
64
|
-
"builder (~> 3.1)",
|
|
65
|
-
"erubi (~> 1.4)",
|
|
66
|
-
"concurrent-ruby (~> 1.0, >= 1.0.2)",
|
|
67
|
-
"i18n (>= 1.6, < 2)",
|
|
68
|
-
"minitest (>= 5.1)",
|
|
69
|
-
"tzinfo (~> 2.0)",
|
|
70
|
-
"zeitwerk (~> 2.3)",
|
|
71
|
-
"nokogiri (>= 1.6)",
|
|
72
|
-
"mini_portile2 (~> 2.8.0)",
|
|
73
|
-
"racc (~> 1.4)",
|
|
74
|
-
"loofah (~> 2.3)",
|
|
75
|
-
"crass (~> 1.0.2)",
|
|
76
|
-
"websocket-extensions (>= 0.1.0)",
|
|
77
|
-
"mail (>= 2.7.1)",
|
|
78
|
-
"globalid (>= 0.3.6)",
|
|
79
|
-
"marcel (~> 1.0)",
|
|
80
|
-
"mini_mime (>= 1.1.0)",
|
|
81
|
-
"net-imap (>= 0)",
|
|
82
|
-
"net-pop (>= 0)",
|
|
83
|
-
"net-smtp (>= 0)",
|
|
84
|
-
"date (>= 0)",
|
|
85
|
-
"net-protocol (>= 0)",
|
|
86
|
-
"timeout (>= 0)",
|
|
87
|
-
"method_source (>= 0)",
|
|
88
|
-
"rake (>= 12.2)",
|
|
89
|
-
"sprockets (>= 3.0.0)",
|
|
90
|
-
"code_analyzer (~> 0.5.5)",
|
|
91
|
-
"erubis (>= 0)",
|
|
92
|
-
"json (>= 0)",
|
|
93
|
-
"require_all (~> 3.0)",
|
|
94
|
-
"ruby-progressbar (>= 0)",
|
|
95
|
-
"kwalify (~> 0.7.0)",
|
|
96
|
-
"parser (~> 3.2.0)",
|
|
97
|
-
"rainbow (>= 2.0, < 4.0)",
|
|
98
|
-
"rexml (~> 3.1)",
|
|
99
|
-
"ast (~> 2.4.1)",
|
|
100
|
-
"parallel (~> 1.10)",
|
|
101
|
-
"regexp_parser (>= 1.8, < 3.0)",
|
|
102
|
-
"rubocop-ast (>= 1.23.0, < 2.0)",
|
|
103
|
-
"unicode-display_width (>= 1.4.0, < 3.0)"
|
|
104
|
-
]
|
|
105
|
-
},
|
|
106
|
-
{
|
|
107
|
-
"name": "rspec-rails",
|
|
108
|
-
"total_dependencies": 28,
|
|
109
|
-
"first_level_dependencies": 7,
|
|
110
|
-
"top_level_dependencies": {},
|
|
111
|
-
"transitive_dependencies": [
|
|
112
|
-
"actionpack (>= 6.1)",
|
|
113
|
-
"activesupport (>= 6.1)",
|
|
114
|
-
"railties (>= 6.1)",
|
|
115
|
-
"rspec-core (~> 3.11)",
|
|
116
|
-
"rspec-expectations (~> 3.11)",
|
|
117
|
-
"rspec-mocks (~> 3.11)",
|
|
118
|
-
"rspec-support (~> 3.11)",
|
|
119
|
-
"actionview (= 6.1.7)",
|
|
120
|
-
"rack (~> 2.0, >= 2.0.9)",
|
|
121
|
-
"rack-test (>= 0.6.3)",
|
|
122
|
-
"rails-dom-testing (~> 2.0)",
|
|
123
|
-
"rails-html-sanitizer (~> 1.0, >= 1.2.0)",
|
|
124
|
-
"builder (~> 3.1)",
|
|
125
|
-
"erubi (~> 1.4)",
|
|
126
|
-
"concurrent-ruby (~> 1.0, >= 1.0.2)",
|
|
127
|
-
"i18n (>= 1.6, < 2)",
|
|
128
|
-
"minitest (>= 5.1)",
|
|
129
|
-
"tzinfo (~> 2.0)",
|
|
130
|
-
"zeitwerk (~> 2.3)",
|
|
131
|
-
"nokogiri (>= 1.6)",
|
|
132
|
-
"mini_portile2 (~> 2.8.0)",
|
|
133
|
-
"racc (~> 1.4)",
|
|
134
|
-
"loofah (~> 2.3)",
|
|
135
|
-
"crass (~> 1.0.2)",
|
|
136
|
-
"method_source (>= 0)",
|
|
137
|
-
"rake (>= 12.2)",
|
|
138
|
-
"thor (~> 1.0)",
|
|
139
|
-
"diff-lcs (>= 1.2.0, < 2.0)"
|
|
140
|
-
]
|
|
141
|
-
},
|
|
142
|
-
{
|
|
143
|
-
"name": "factory_bot_rails",
|
|
144
|
-
"total_dependencies": 24,
|
|
145
|
-
"first_level_dependencies": 2,
|
|
146
|
-
"top_level_dependencies": {},
|
|
147
|
-
"transitive_dependencies": [
|
|
148
|
-
"factory_bot (~> 6.2.0)",
|
|
149
|
-
"railties (>= 5.0.0)",
|
|
150
|
-
"activesupport (>= 5.0.0)",
|
|
151
|
-
"concurrent-ruby (~> 1.0, >= 1.0.2)",
|
|
152
|
-
"i18n (>= 1.6, < 2)",
|
|
153
|
-
"minitest (>= 5.1)",
|
|
154
|
-
"tzinfo (~> 2.0)",
|
|
155
|
-
"zeitwerk (~> 2.3)",
|
|
156
|
-
"actionpack (= 6.1.7)",
|
|
157
|
-
"method_source (>= 0)",
|
|
158
|
-
"rake (>= 12.2)",
|
|
159
|
-
"thor (~> 1.0)",
|
|
160
|
-
"actionview (= 6.1.7)",
|
|
161
|
-
"rack (~> 2.0, >= 2.0.9)",
|
|
162
|
-
"rack-test (>= 0.6.3)",
|
|
163
|
-
"rails-dom-testing (~> 2.0)",
|
|
164
|
-
"rails-html-sanitizer (~> 1.0, >= 1.2.0)",
|
|
165
|
-
"builder (~> 3.1)",
|
|
166
|
-
"erubi (~> 1.4)",
|
|
167
|
-
"nokogiri (>= 1.6)",
|
|
168
|
-
"mini_portile2 (~> 2.8.0)",
|
|
169
|
-
"racc (~> 1.4)",
|
|
170
|
-
"loofah (~> 2.3)",
|
|
171
|
-
"crass (~> 1.0.2)"
|
|
172
|
-
]
|
|
173
|
-
},
|
|
174
|
-
{
|
|
175
|
-
"name": "guard-rspec",
|
|
176
|
-
"total_dependencies": 21,
|
|
177
|
-
"first_level_dependencies": 3,
|
|
178
|
-
"top_level_dependencies": {},
|
|
179
|
-
"transitive_dependencies": [
|
|
180
|
-
"guard (~> 2.1)",
|
|
181
|
-
"guard-compat (~> 1.1)",
|
|
182
|
-
"rspec (>= 2.99.0, < 4.0)",
|
|
183
|
-
"formatador (>= 0.2.4)",
|
|
184
|
-
"listen (>= 2.7, < 4.0)",
|
|
185
|
-
"lumberjack (>= 1.0.12, < 2.0)",
|
|
186
|
-
"nenv (~> 0.1)",
|
|
187
|
-
"notiffany (~> 0.0)",
|
|
188
|
-
"pry (>= 0.13.0)",
|
|
189
|
-
"shellany (~> 0.0)",
|
|
190
|
-
"thor (>= 0.18.1)",
|
|
191
|
-
"rb-fsevent (~> 0.10, >= 0.10.3)",
|
|
192
|
-
"rb-inotify (~> 0.9, >= 0.9.10)",
|
|
193
|
-
"ffi (~> 1.0)",
|
|
194
|
-
"coderay (~> 1.1)",
|
|
195
|
-
"method_source (~> 1.0)",
|
|
196
|
-
"rspec-core (~> 3.12.0)",
|
|
197
|
-
"rspec-expectations (~> 3.12.0)",
|
|
198
|
-
"rspec-mocks (~> 3.12.0)",
|
|
199
|
-
"rspec-support (~> 3.12.0)",
|
|
200
|
-
"diff-lcs (>= 1.2.0, < 2.0)"
|
|
201
|
-
]
|
|
202
|
-
},
|
|
203
|
-
{
|
|
204
|
-
"name": "coveralls",
|
|
205
|
-
"total_dependencies": 8,
|
|
206
|
-
"first_level_dependencies": 5,
|
|
207
|
-
"top_level_dependencies": {},
|
|
208
|
-
"transitive_dependencies": [
|
|
209
|
-
"json (>= 1.8, < 3)",
|
|
210
|
-
"simplecov (~> 0.16.1)",
|
|
211
|
-
"term-ansicolor (~> 1.3)",
|
|
212
|
-
"thor (>= 0.19.4, < 2.0)",
|
|
213
|
-
"tins (~> 1.6)",
|
|
214
|
-
"docile (~> 1.1)",
|
|
215
|
-
"simplecov-html (~> 0.10.0)",
|
|
216
|
-
"sync (>= 0)"
|
|
217
|
-
]
|
|
218
|
-
},
|
|
219
|
-
{
|
|
220
|
-
"name": "pry-rails",
|
|
221
|
-
"total_dependencies": 3,
|
|
222
|
-
"first_level_dependencies": 1,
|
|
223
|
-
"top_level_dependencies": {},
|
|
224
|
-
"transitive_dependencies": [
|
|
225
|
-
"pry (>= 0.10.4)",
|
|
226
|
-
"coderay (~> 1.1)",
|
|
227
|
-
"method_source (~> 1.0)"
|
|
228
|
-
]
|
|
229
|
-
},
|
|
230
|
-
{
|
|
231
|
-
"name": "pry",
|
|
232
|
-
"total_dependencies": 2,
|
|
233
|
-
"first_level_dependencies": 2,
|
|
234
|
-
"top_level_dependencies": {
|
|
235
|
-
"guard": "guard (2.18.0)",
|
|
236
|
-
"guard-rspec": "guard-rspec (4.7.3)",
|
|
237
|
-
"pry-rails": "pry-rails (0.3.9)"
|
|
238
|
-
},
|
|
239
|
-
"transitive_dependencies": [
|
|
240
|
-
"coderay (~> 1.1)",
|
|
241
|
-
"method_source (~> 1.0)"
|
|
242
|
-
]
|
|
243
|
-
},
|
|
244
|
-
{
|
|
245
|
-
"name": "rspec_junit_formatter",
|
|
246
|
-
"total_dependencies": 2,
|
|
247
|
-
"first_level_dependencies": 1,
|
|
248
|
-
"top_level_dependencies": {},
|
|
249
|
-
"transitive_dependencies": [
|
|
250
|
-
"rspec-core (>= 2, < 4, != 2.12.0)",
|
|
251
|
-
"rspec-support (~> 3.12.0)"
|
|
252
|
-
]
|
|
253
|
-
},
|
|
254
|
-
{
|
|
255
|
-
"name": "bundler",
|
|
256
|
-
"total_dependencies": 0,
|
|
257
|
-
"first_level_dependencies": 0,
|
|
258
|
-
"top_level_dependencies": {
|
|
259
|
-
"bundler-audit": "bundler-audit (0.9.1)",
|
|
260
|
-
"bundler-stats": "bundler-stats (2.3.0)",
|
|
261
|
-
"rails": "rails (6.1.7)",
|
|
262
|
-
"repo_analyzer": "repo_analyzer (1.3.0)"
|
|
263
|
-
},
|
|
264
|
-
"transitive_dependencies": []
|
|
265
|
-
},
|
|
266
|
-
{
|
|
267
|
-
"name": "sqlite3",
|
|
268
|
-
"total_dependencies": 0,
|
|
269
|
-
"first_level_dependencies": 0,
|
|
270
|
-
"top_level_dependencies": {},
|
|
271
|
-
"transitive_dependencies": []
|
|
272
|
-
}
|
|
37
|
+
rspec_junit_formatter,
|
|
38
|
+
sqlite
|
|
273
39
|
]
|
|
274
40
|
}
|
|
275
41
|
end
|
|
@@ -278,6 +44,14 @@ describe RepoAnalyzer::BundlerStatsExtractor, repo_analyzer_extractor_helpers: t
|
|
|
278
44
|
allow(extractor).to receive(:`).with(
|
|
279
45
|
'bundle exec bundle-stats stats -f json'
|
|
280
46
|
).and_return(stats.to_json)
|
|
47
|
+
|
|
48
|
+
allow(extractor).to receive(:`).with(
|
|
49
|
+
"bundle exec bundle-stats show 'rspec_junit_formatter' -f json"
|
|
50
|
+
).and_return(rspec_junit_formatter.to_json)
|
|
51
|
+
|
|
52
|
+
allow(extractor).to receive(:`).with(
|
|
53
|
+
"bundle exec bundle-stats show 'sqlite' -f json"
|
|
54
|
+
).and_return(sqlite.to_json)
|
|
281
55
|
end
|
|
282
56
|
|
|
283
57
|
it { expect(extract[:bundler_stats_extractor]).to eq(stats.with_indifferent_access) }
|
|
@@ -2,6 +2,7 @@ require 'rails_helper'
|
|
|
2
2
|
|
|
3
3
|
describe RepoAnalyzer::ExtractProjectInfoJob, type: :job do
|
|
4
4
|
let(:repo_name) { "platanus/alisur-formulator" }
|
|
5
|
+
let(:project_path) { "spec/assets/test_project" }
|
|
5
6
|
|
|
6
7
|
let(:files_list) do
|
|
7
8
|
"app/extractors/repo_analyzer/project_versions_extractor.rb"
|
|
@@ -31,7 +32,7 @@ describe RepoAnalyzer::ExtractProjectInfoJob, type: :job do
|
|
|
31
32
|
let(:engine_root) { instance_double("Pathname", join: files_list) }
|
|
32
33
|
|
|
33
34
|
def perform_now
|
|
34
|
-
described_class.perform_now(repo_name)
|
|
35
|
+
described_class.perform_now(repo_name, project_path)
|
|
35
36
|
end
|
|
36
37
|
|
|
37
38
|
before do
|
|
@@ -44,7 +45,7 @@ describe RepoAnalyzer::ExtractProjectInfoJob, type: :job do
|
|
|
44
45
|
|
|
45
46
|
it do
|
|
46
47
|
perform_now
|
|
47
|
-
expect(RepoAnalyzer::ProjectDataBridge).to have_received(:new).with(repo_name).once
|
|
48
|
+
expect(RepoAnalyzer::ProjectDataBridge).to have_received(:new).with(repo_name, project_path).once
|
|
48
49
|
expect(RepoAnalyzer::ProjectVersionsExtractor).to have_received(:new).with(bridge).once
|
|
49
50
|
end
|
|
50
51
|
end
|
metadata
CHANGED
|
@@ -1,7 +1,7 @@
|
|
|
1
1
|
--- !ruby/object:Gem::Specification
|
|
2
2
|
name: repo_analyzer
|
|
3
3
|
version: !ruby/object:Gem::Version
|
|
4
|
-
version: 1.
|
|
4
|
+
version: 1.6.0
|
|
5
5
|
platform: ruby
|
|
6
6
|
authors:
|
|
7
7
|
- Platanus
|
|
@@ -9,7 +9,7 @@ authors:
|
|
|
9
9
|
autorequire:
|
|
10
10
|
bindir: exe
|
|
11
11
|
cert_chain: []
|
|
12
|
-
date: 2023-
|
|
12
|
+
date: 2023-06-21 00:00:00.000000000 Z
|
|
13
13
|
dependencies:
|
|
14
14
|
- !ruby/object:Gem::Dependency
|
|
15
15
|
name: bundler-audit
|
|
@@ -350,7 +350,6 @@ files:
|
|
|
350
350
|
- app/extractors/repo_analyzer/bundler_audit_extractor.rb
|
|
351
351
|
- app/extractors/repo_analyzer/bundler_stats_extractor.rb
|
|
352
352
|
- app/extractors/repo_analyzer/circleci_extractor.rb
|
|
353
|
-
- app/extractors/repo_analyzer/fasterer_extractor.rb
|
|
354
353
|
- app/extractors/repo_analyzer/github_extractor.rb
|
|
355
354
|
- app/extractors/repo_analyzer/power_types_extractor.rb
|
|
356
355
|
- app/extractors/repo_analyzer/project_config_extractor.rb
|
|
@@ -448,7 +447,6 @@ files:
|
|
|
448
447
|
- spec/extractors/bundler_audit_extractor_spec.rb
|
|
449
448
|
- spec/extractors/bundler_stats_extractor_spec.rb
|
|
450
449
|
- spec/extractors/circleci_extractor_spec.rb
|
|
451
|
-
- spec/extractors/fasterer_extractor_spec.rb
|
|
452
450
|
- spec/extractors/github_extractor_spec.rb
|
|
453
451
|
- spec/extractors/power_types_extractor_spec.rb
|
|
454
452
|
- spec/extractors/project_config_extractor_spec.rb
|
|
@@ -556,7 +554,6 @@ test_files:
|
|
|
556
554
|
- spec/extractors/bundler_audit_extractor_spec.rb
|
|
557
555
|
- spec/extractors/bundler_stats_extractor_spec.rb
|
|
558
556
|
- spec/extractors/circleci_extractor_spec.rb
|
|
559
|
-
- spec/extractors/fasterer_extractor_spec.rb
|
|
560
557
|
- spec/extractors/github_extractor_spec.rb
|
|
561
558
|
- spec/extractors/power_types_extractor_spec.rb
|
|
562
559
|
- spec/extractors/project_config_extractor_spec.rb
|
|
@@ -1,34 +0,0 @@
|
|
|
1
|
-
require "fasterer/file_traverser"
|
|
2
|
-
|
|
3
|
-
module RepoAnalyzer
|
|
4
|
-
class FastererExtractor < ProjectInfoExtractor
|
|
5
|
-
private
|
|
6
|
-
|
|
7
|
-
def extracted_info
|
|
8
|
-
{ speedups: fasterer_info }
|
|
9
|
-
end
|
|
10
|
-
|
|
11
|
-
def fasterer_info
|
|
12
|
-
@fasterer_info ||= fasterer_text.split("\n").inject([]) do |memo, row|
|
|
13
|
-
next memo if row.blank?
|
|
14
|
-
|
|
15
|
-
row_parts = row.split("\e")
|
|
16
|
-
file_path_and_line = row_parts.second.gsub('[0;31;49m', '')
|
|
17
|
-
file_path, line = file_path_and_line.split(':')
|
|
18
|
-
next memo if file_path.include?("files inspected")
|
|
19
|
-
|
|
20
|
-
message = row_parts.last.gsub('[0m ', '')
|
|
21
|
-
memo << {
|
|
22
|
-
file_path: file_path,
|
|
23
|
-
line: line,
|
|
24
|
-
message: message
|
|
25
|
-
}
|
|
26
|
-
end
|
|
27
|
-
end
|
|
28
|
-
|
|
29
|
-
def fasterer_text
|
|
30
|
-
file_traverser = Fasterer::FileTraverser.new(".")
|
|
31
|
-
OutputUtils.with_captured_stdout { file_traverser.traverse }
|
|
32
|
-
end
|
|
33
|
-
end
|
|
34
|
-
end
|
|
@@ -1,45 +0,0 @@
|
|
|
1
|
-
require 'rails_helper'
|
|
2
|
-
|
|
3
|
-
describe RepoAnalyzer::FastererExtractor, repo_analyzer_extractor_helpers: true do
|
|
4
|
-
describe "#extract" do
|
|
5
|
-
let(:fasterer_result_content) do
|
|
6
|
-
<<~TEXT
|
|
7
|
-
\e[0;31;49mapp/extractors/repo_analyzer/github_extractor.rb:60\e[0m Don't rescue NoMethodError, rather check with respond_to?.
|
|
8
|
-
|
|
9
|
-
\e[0;31;49mapp/jobs/repo_analyzer/extract_project_info_job.rb:17\e[0m Calling blocks with call is slower than yielding.
|
|
10
|
-
|
|
11
|
-
\e[0;31;49mspec/dummy/config/puma.rb:14\e[0m Hash#fetch with second argument is slower than Hash#fetch with block.
|
|
12
|
-
|
|
13
|
-
\e[0;32;49m75 files inspected\e[0m, \e[0;31;49m3 offenses detected\e[0m
|
|
14
|
-
TEXT
|
|
15
|
-
end
|
|
16
|
-
|
|
17
|
-
before do
|
|
18
|
-
allow(OutputUtils).to receive(:with_captured_stdout).and_return(fasterer_result_content)
|
|
19
|
-
end
|
|
20
|
-
|
|
21
|
-
let(:expected) do
|
|
22
|
-
{
|
|
23
|
-
speedups: [
|
|
24
|
-
{
|
|
25
|
-
file_path: "app/extractors/repo_analyzer/github_extractor.rb",
|
|
26
|
-
line: "60",
|
|
27
|
-
message: "Don't rescue NoMethodError, rather check with respond_to?."
|
|
28
|
-
},
|
|
29
|
-
{
|
|
30
|
-
file_path: "app/jobs/repo_analyzer/extract_project_info_job.rb",
|
|
31
|
-
line: "17",
|
|
32
|
-
message: "Calling blocks with call is slower than yielding."
|
|
33
|
-
},
|
|
34
|
-
{
|
|
35
|
-
file_path: "spec/dummy/config/puma.rb",
|
|
36
|
-
line: "14",
|
|
37
|
-
message: "Hash#fetch with second argument is slower than Hash#fetch with block."
|
|
38
|
-
}
|
|
39
|
-
]
|
|
40
|
-
}.with_indifferent_access
|
|
41
|
-
end
|
|
42
|
-
|
|
43
|
-
it { expect(extract[:fasterer_extractor]).to eq(expected) }
|
|
44
|
-
end
|
|
45
|
-
end
|