From 706774bf611d936a20c71d2567cc7eed79c39a6a Mon Sep 17 00:00:00 2001
From: FireFart <FireFart@gmail.com>
Date: Tue, 22 Oct 2013 22:39:23 +0200
Subject: [PATCH] Add detection for all-in-one-seo-pack Closes issue #332

---
 .../collections/wp_plugins/detectable.rb      |  4 +++
 .../collections/wp_plugins/detectable_spec.rb | 36 ++++++++++++++++++-
 2 files changed, 39 insertions(+), 1 deletion(-)

diff --git a/lib/common/collections/wp_plugins/detectable.rb b/lib/common/collections/wp_plugins/detectable.rb
index 1c8a5df2..5a3e0ef9 100644
--- a/lib/common/collections/wp_plugins/detectable.rb
+++ b/lib/common/collections/wp_plugins/detectable.rb
@@ -64,6 +64,10 @@ class WpPlugins < WpItems
       wp_plugins.add('wp-super-cache') if body =~ /wp-super-cache/i
       wp_plugins.add('w3-total-cache') if body =~ /w3 total cache/i
 
+      if body =~ /<!-- all in one seo pack ([^\s]+)/i
+        wp_plugins.add('all-in-one-seo-pack', version: $1)
+      end
+
       wp_plugins
     end
 
diff --git a/spec/lib/common/collections/wp_plugins/detectable_spec.rb b/spec/lib/common/collections/wp_plugins/detectable_spec.rb
index 9d7bf2fe..99f6da38 100644
--- a/spec/lib/common/collections/wp_plugins/detectable_spec.rb
+++ b/spec/lib/common/collections/wp_plugins/detectable_spec.rb
@@ -63,7 +63,41 @@ describe 'WpPlugins::Detectable' do
   end
 
   describe '::from_content' do
-    # TODO
+    context 'when no body' do
+      it 'returns an empty WpPlugins' do
+        stub_request(:get, url).to_return(status: 200, body: '')
+        subject.send(:from_content, wp_target).should == subject.new
+      end
+    end
+
+    context 'when body' do
+      @body = ''
+      let(:expected) { subject.new(wp_target) }
+
+      after :each do
+        stub_request(:get, url).to_return(status: 200, body: @body)
+        stub_request(:get, /readme\.txt/i).to_return(status: 404)
+        subject.send(:from_content, wp_target).should == expected
+      end
+
+        it 'returns the w3-total-cache' do
+          @body = 'w3 total cache'
+          expected.add('w3-total-cache')
+        end
+      context 'when wp-super-cache detected' do
+        it 'returns the wp-super-cache' do
+          @body = 'wp-super-cache'
+          expected.add('wp-super-cache')
+        end
+      end
+
+      context 'when all-in-one-seo-pack detected' do
+        it 'returns the all-in-one-seo-pack' do
+          @body = '<!-- All in One SEO Pack 2.0.3.1 by Michael Torbert of Semper Fi Web Design[300,342] -->'
+          expected.add('all-in-one-seo-pack', version: '2.0.3.1')
+        end
+      end
+    end
   end
 
   describe '::passive_detection' do