metainspector 5.2.0 → 5.2.1

Sign up to get free protection for your applications and to get access to all the features.
Files changed (9) hide show
  1. checksums.yaml +4 -4
  2. data/lib/meta_inspector/parsers/texts.rb +2 -1
  3. data/lib/meta_inspector/version.rb +1 -1
  4. data/meta_inspector.gemspec +0 -2
  5. data/spec/fixtures/empty_metatag_description.response +22 -0
  6. data/spec/meta_inspector/texts_spec.rb +5 -0
  7. data/spec/spec_helper.rb +3 -0
  8. metadata +3 -31
  9. data/Guardfile +0 -5
checksums.yaml CHANGED
@@ -1,7 +1,7 @@
1
1
  ---
2
2
  SHA1:
3
- metadata.gz: 4d6981fee7a5b3c01045c6e152d20fb2c788c549
4
- data.tar.gz: 4f0ff717747c9b1bdc5abae2eae1696bc4138033
3
+ metadata.gz: ee91e2584212fcc3855df7194995c540f270b93d
4
+ data.tar.gz: be340bfaf27f1711f41f4aa2bdede5d9ed13dcf1
5
5
  SHA512:
6
- metadata.gz: 782597ff87e03b177d73302efb5dcfa0713e7a27367fe93b43579960e89824caa8331a88c95d96410597d03ba0849fa4c4cf3fcf2caeb02d63bf2aa606aa4c08
7
- data.tar.gz: 268e91699c34b9649e6441450ccb244fc1b49c6c06a3c1e6787f73fa04332b8172886228ce8a20ad714c727ee45cfa2665308aaa79b33eeef3ded7ac01225722
6
+ metadata.gz: ad094ae13e5ee7aeb2b31bb11fbe77e6970b64ee8a075db4dff76ec2a0161f6bbf958c6e0910d4893e59838d4ec4efb6c534fa6b31ef6f3005d4350fec29b11e
7
+ data.tar.gz: bd2f433f1c9f992b0ef93eab38157ebe235a3989307a043eabc7d0da20596905503ae967b148661b1b3cc4ac5aab76fe268e16ed9c2404fe293e7f02fee2c550
data/lib/meta_inspector/parsers/texts.rb CHANGED
@@ -18,7 +18,8 @@ module MetaInspector
18
18
  # and if not present will guess by looking at the first paragraph
19
19
  # with more than 120 characters
20
20
  def description
21
- meta['description'] || secondary_description
21
+ return meta['description'] unless meta['description'].nil? || meta['description'].empty?
22
+ secondary_description
22
23
  end
23
24
 
24
25
  private
data/lib/meta_inspector/version.rb CHANGED
@@ -1,3 +1,3 @@
1
1
  module MetaInspector
2
- VERSION = '5.2.0'
2
+ VERSION = '5.2.1'
3
3
  end
data/meta_inspector.gemspec CHANGED
@@ -30,7 +30,5 @@ Gem::Specification.new do |gem|
30
30
  gem.add_development_dependency 'awesome_print'
31
31
  gem.add_development_dependency 'rake', '~> 10.1.0'
32
32
  gem.add_development_dependency 'pry'
33
- gem.add_development_dependency 'guard'
34
- gem.add_development_dependency 'guard-rspec'
35
33
  gem.add_development_dependency 'rubocop'
36
34
  end
data/spec/fixtures/empty_metatag_description.response ADDED
@@ -0,0 +1,22 @@
1
+ HTTP/1.1 200 OK
2
+ Date: Fri, 29 Apr 2016 00:27:28 GMT
3
+ Content-Type: text/html; charset=utf-8
4
+ Connection: keep-alive
5
+ X-Powered-By: Express
6
+ Cache-Control: public, max-age=0
7
+ Vary: Accept-Encoding
8
+ Via: 1.1 vegur
9
+ Server: cloudflare-nginx
10
+ CF-RAY: 29aea0e05c720938-DFW
11
+
12
+ <!DOCTYPE html>
13
+ <html>
14
+ <head>
15
+ <meta charset="utf-8"/>
16
+ <title>Disrupting DeAcero, un ejemplo de innovación Disruptive Angels</title>
17
+ <meta name="description" content=""/>
18
+ </head>
19
+ <body>
20
+ <p>Vivimos en una época en la que el término 'Innovación' esta siendo usado a placer por organizaciones de todo tipo. Hace algunos meses en una reunión de trabajo alguien mencionó:"La innovación esta siendo usada como todo aquello que las empresas no saben dónde poner". Mejor no lo pudo haber dicho. Llevamos más de dos años de estar colaborando cercanamente con directores de innovación de decenas de empresas, participando en comisiones industriales enfocadas a la innovación y haciendo conexiones laborales internacionales con expertos en materias de innovación (particularmente innovación abierta), y después de todo, la gran mayoría de las empresas no tienen claro lo que (por lo menos dentro de su organización) es innovación. </p>
21
+ </body>
22
+ </html>
data/spec/meta_inspector/texts_spec.rb CHANGED
@@ -61,5 +61,10 @@ describe MetaInspector do
61
61
  page = MetaInspector.new('http://theonion-no-description.com')
62
62
  expect(page.description).to eq("SAN FRANCISCO—In a move expected to revolutionize the mobile device industry, Apple launched its fastest and most powerful iPhone to date Tuesday, an innovative new model that can only be seen by the company's hippest and most dedicated customers. This is secondary text picked up because of a missing meta description.")
63
63
  end
64
+
65
+ it 'should find first paragraph if meta description is empty' do
66
+ expect(MetaInspector.new('http://example.com/empty-meta-description').description)
67
+ .to eq("Vivimos en una época en la que el término 'Innovación' esta siendo usado a placer por organizaciones de todo tipo. Hace algunos meses en una reunión de trabajo alguien mencionó:\"La innovación esta siendo usada como todo aquello que las empresas no saben dónde poner\". Mejor no lo pudo haber dicho. Llevamos más de dos años de estar colaborando cercanamente con directores de innovación de decenas de empresas, participando en comisiones industriales enfocadas a la innovación y haciendo conexiones laborales internacionales con expertos en materias de innovación (particularmente innovación abierta), y después de todo, la gran mayoría de las empresas no tienen claro lo que (por lo menos dentro de su organización) es innovación. ")
68
+ end
64
69
  end
65
70
  end
data/spec/spec_helper.rb CHANGED
@@ -112,3 +112,6 @@ FakeWeb.register_uri(:get, "http://example.com/~", :response => fixture_file("ex
112
112
 
113
113
  # Example to test correct encoding
114
114
  FakeWeb.register_uri(:get, "http://example-rtl.com/", :response => fixture_file("encoding.response"))
115
+
116
+ # Example used to test empty description metatags
117
+ FakeWeb.register_uri(:get, "http://example.com/empty-meta-description", :response => fixture_file("empty_metatag_description.response"))
metadata CHANGED
@@ -1,14 +1,14 @@
1
1
  --- !ruby/object:Gem::Specification
2
2
  name: metainspector
3
3
  version: !ruby/object:Gem::Version
4
- version: 5.2.0
4
+ version: 5.2.1
5
5
  platform: ruby
6
6
  authors:
7
7
  - Jaime Iniesta
8
8
  autorequire:
9
9
  bindir: bin
10
10
  cert_chain: []
11
- date: 2016-04-21 00:00:00.000000000 Z
11
+ date: 2016-04-30 00:00:00.000000000 Z
12
12
  dependencies:
13
13
  - !ruby/object:Gem::Dependency
14
14
  name: nokogiri
@@ -220,34 +220,6 @@ dependencies:
220
220
  - - ">="
221
221
  - !ruby/object:Gem::Version
222
222
  version: '0'
223
- - !ruby/object:Gem::Dependency
224
- name: guard
225
- requirement: !ruby/object:Gem::Requirement
226
- requirements:
227
- - - ">="
228
- - !ruby/object:Gem::Version
229
- version: '0'
230
- type: :development
231
- prerelease: false
232
- version_requirements: !ruby/object:Gem::Requirement
233
- requirements:
234
- - - ">="
235
- - !ruby/object:Gem::Version
236
- version: '0'
237
- - !ruby/object:Gem::Dependency
238
- name: guard-rspec
239
- requirement: !ruby/object:Gem::Requirement
240
- requirements:
241
- - - ">="
242
- - !ruby/object:Gem::Version
243
- version: '0'
244
- type: :development
245
- prerelease: false
246
- version_requirements: !ruby/object:Gem::Requirement
247
- requirements:
248
- - - ">="
249
- - !ruby/object:Gem::Version
250
- version: '0'
251
223
  - !ruby/object:Gem::Dependency
252
224
  name: rubocop
253
225
  requirement: !ruby/object:Gem::Requirement
@@ -276,7 +248,6 @@ files:
276
248
  - ".travis.yml"
277
249
  - CHANGELOG.md
278
250
  - Gemfile
279
- - Guardfile
280
251
  - MIT-LICENSE
281
252
  - README.md
282
253
  - Rakefile
@@ -309,6 +280,7 @@ files:
309
280
  - spec/fixtures/charset_000.response
310
281
  - spec/fixtures/charset_001.response
311
282
  - spec/fixtures/charset_002.response
283
+ - spec/fixtures/empty_metatag_description.response
312
284
  - spec/fixtures/empty_page.response
313
285
  - spec/fixtures/encoding.response
314
286
  - spec/fixtures/example.response
data/Guardfile DELETED
@@ -1,5 +0,0 @@
1
- guard :rspec, cmd: 'bundle exec rspec' do
2
- watch(%r{^spec/.+_spec\.rb$})
3
- watch(%r{^lib/meta_inspector/(.+)\.rb$}) { |m| "spec/#{m[1]}_spec.rb" }
4
- watch('spec/spec_helper.rb') { 'spec' }
5
- end