ghost_in_the_post 0.0.1

data/spec/dummy/tmp/cache/assets/sprockets/v3.0/23mzeUAO-pIzGJ1n84Dbg25udWeSiBcd6e_QsDihWQc.cache

@@ -0,0 +1,3 @@
+[o:Set:
+@hash{
+I"environment-version:ETTI"environment-paths;TTI"Lprocessors:type=application/javascript&file_type=application/javascript;TTI"1file-digest://app/assets/javascripts/test.js;TTI"Zprocessors:type=application/javascript&file_type=application/javascript&pipeline=self;TT

data/spec/dummy/tmp/cache/assets/sprockets/v3.0/6544SZAu2zCeDrdro4h3izdKNXVvHzstwz12UxeiJoA.cache

@@ -0,0 +1,2 @@
+[o:Set:
+@hash{	I"environment-version:ETTI"environment-paths;TTI"Zprocessors:type=application/javascript&file_type=application/javascript&pipeline=self;TTI"1file-digest://app/assets/javascripts/test.js;TT

data/spec/dummy/tmp/cache/assets/sprockets/v3.0/BMI0IB92xLhvUPcr_K0JuSk8zczmrEEbJCd3qwHauhQ.cache

@@ -0,0 +1 @@
+"%�W\�yho9S���ƫ˥��)|�~���(���

data/spec/dummy/tmp/cache/assets/sprockets/v3.0/IFLV6zTaBXU1ccodbD3QJ4xUIOEix_Fbb7j8wsxrHw8.cache

@@ -0,0 +1 @@
+I"
�app/assets/javascripts/test.js?type=application/javascript&pipeline=self&id=b64c38812a66f3714232b5f6a440aa0d755642ad933e0bc59df8594afcc9d899:ET

data/spec/dummy/tmp/cache/assets/sprockets/v3.0/O9gjtBITAGlfzUwc2DT5N0GW2bo2b2yQ64gnQljLogM.cache

@@ -0,0 +1 @@
+I"
�app/assets/javascripts/application.js?type=application/javascript&id=d60a1828a97cefde6fd8ada124f7d6b9e3754a5b8606cadb78146ea03d465ae7:ET

data/spec/dummy/tmp/cache/assets/sprockets/v3.0/OZ04c215C4Euu3scfhmxmte0o_tZOVWTqrv61cpy7H4.cache

@@ -0,0 +1 @@
+I"
�app/assets/javascripts/application.js?type=application/javascript&pipeline=self&id=0cb0379e0f5747f3fad310dacef102654555c5334351880dc80f112d7e2b86f0:ET

data/spec/dummy/tmp/cache/assets/sprockets/v3.0/_Cw52w24AWZEbxLmduyWBMA6Y1OmJiEm1VNn7pY4rtk.cache

@@ -0,0 +1,2 @@
+[o:Set:
+@hash{	I"environment-version:ETTI"environment-paths;TTI"Zprocessors:type=application/javascript&file_type=application/javascript&pipeline=self;TTI"1file-digest://app/assets/javascripts/test.js;TT

data/spec/dummy/tmp/cache/assets/sprockets/v3.0/eN_ueL7m-D9xkEZpaLKwCxdrNzyz-MW9jcdJ5TdF9j8.cache

@@ -0,0 +1 @@
+"%�

data/spec/dummy/tmp/cache/assets/sprockets/v3.0/efKfhZKxLC7ERXlMb5CQMqYwbvWS-A--pH-0fqzvRiQ.cache

@@ -0,0 +1 @@
+I"
~app/assets/javascripts/test.js?type=application/javascript&id=07c03e9c18a6664cd589766bf1c9c5fabfac303b890045debcb704e109e4fff7:ET

data/spec/dummy/tmp/cache/assets/sprockets/v3.0/goEv0RKoifagsjFSdEHY-23E9ybQn_t1HHEZq7PlKQI.cache

@@ -0,0 +1 @@
+I"
�app/assets/javascripts/test.js?type=application/javascript&pipeline=self&id=963de54cc08f25dea0c490c740550aad17af6c3a62ca371ac3db8ebe89bc4fd2:ET

data/spec/dummy/tmp/cache/assets/sprockets/v3.0/n6_6RjvPX2rCv5dytbSnnDrGwAj1ZcPfninuNUeP_iE.cache

@@ -0,0 +1 @@
+I"
�app/assets/javascripts/application.js?type=application/javascript&pipeline=self&id=06950834cef83fe6dbc7a7c85bc9e4f615f55d9b8dbd85655e0c987022c23724:ET

data/spec/dummy/tmp/cache/assets/sprockets/v3.0/nXo-w2o4zQBM_uSUnniuP53t5erBbSZPecMED9YnzQI.cache

@@ -0,0 +1 @@
+"%yŜ�{���F�5��/_��XJ��QQ-� �

data/spec/dummy/tmp/cache/assets/sprockets/v3.0/oB1njyQ78MOMRf_5TvuYzHleeI8eke5-FDIIVl7qas0.cache

@@ -0,0 +1 @@
+"%6���%i���,}����Y� ���.��>m�

data/spec/dummy/tmp/cache/assets/sprockets/v3.0/y98HqYF9nxOk8t_j3BKv9Nc4LuSAEzhER7rCCvOM6xI.cache

@@ -0,0 +1 @@
+I"
�app/assets/javascripts/application.js?type=application/javascript&id=d5e992511ffbb70655162f9a39194f63a0c1fc96e34031c4d657819ef0485a81:ET

data/spec/lib/ghost_in_the_post/automatic_spec.rb

@@ -0,0 +1,42 @@
+require "spec_helper"
+require 'mail'
+
+module GhostInThePost
+  describe Automatic do
+    class AutoMailer < BaseMailer
+      include Automatic
+    end
+
+    let(:email) { Mail.new(to: "foo@example.com", from: "me@example.com") }
+    let(:instance) { AutoMailer.new(email) }
+
+    describe "#include_script" do
+
+      it "should add to include scripts" do
+        instance.include_script "test.js"
+        expect(instance.included_scripts).to eq(["test.js"])
+      end
+    end
+
+    describe "#mail" do
+      it "should return an email" do
+        expect(instance.mail).to be_kind_of(Mail::Message)
+      end
+
+      it "should return an email extended with GhostOnCommand and GhostOnDelivery" do
+        expect(email).to receive(:extend).with(GhostOnCommand).and_call_original
+        expect(email).to receive(:extend).with(GhostOnDelivery).and_call_original
+        mail = instance.mail
+        expect(mail).to be_kind_of(GhostOnCommand)
+        expect(mail).to be_kind_of(GhostOnDelivery)
+      end
+
+      it "should have set the emails included scripts" do
+        email.extend GhostOnCommand
+        expect(email).to receive(:included_scripts=).with(nil).and_call_original
+        instance.mail
+      end
+    end
+
+  end
+end

data/spec/lib/ghost_in_the_post/ghost_on_command_spec.rb

@@ -0,0 +1,26 @@
+require "spec_helper"
+require 'mail'
+
+module GhostInThePost
+  describe GhostOnCommand do
+    let(:instance) { SomeMailer.new(Mail.new(to: "foo@example.com", from: "me@example.com")) }
+    let(:email) {instance.mail}
+
+    describe "#ghost" do
+      it "Should return a Message" do
+        expect(email.ghost).to be_a(Mail::Message)
+      end
+      it "should create a MailGhost" do
+        email.included_scripts = ["test"]
+        expect(MailGhost).to receive(:new).with(email, ["test"]).and_call_original
+        email.ghost
+      end
+      it "should apply included scripts to the mail ghost" do
+        expect(email).to receive(:included_scripts).and_call_original
+        email.ghost
+      end
+    end
+
+  end
+end
+

data/spec/lib/ghost_in_the_post/mail_ghost_spec.rb

@@ -0,0 +1,73 @@
+require 'spec_helper'
+require 'mail'
+
+module GhostInThePost
+  describe MailGhost do
+    let(:email) { Mail.new }
+    let(:included_scripts) { ["application.js"] }
+    subject(:ghost) { MailGhost.new(email, included_scripts) }
+
+    it "takes an email and options" do
+      expect(ghost.email).to eq(email)
+      expect(ghost.included_scripts).to eq(included_scripts)
+    end
+
+    context "with an plaintext email" do
+      let(:email) do
+        Mail.new do
+          body "Hello world"
+        end
+      end
+
+      it "returns the email without doing anything" do
+        expect(ghost.execute).to eq(email)
+        expect(email.html_part).to be_nil
+        expect(email.body.decoded).to eq("Hello world")
+      end
+
+      it "works fine when given nil included_scripts" do
+        ghost = MailGhost.new(email, nil)
+        expect { ghost.execute }.to_not raise_error
+        expect(ghost.included_scripts).to eq([])
+      end
+    end
+
+    context "with an HTML email" do
+      let(:html) { "<h1>Hello world!</h1>" }
+      let(:email) do
+        html_string = html
+        Mail.new do
+          content_type 'text/html; charset=UTF-8'
+          body html_string
+        end
+      end
+
+      it "adjusts the html part using Roadie" do
+        document = double "A document", transform: "transformed HTML"
+        expect(PhantomTransform).to receive(:new).with(html, included_scripts).and_return document
+        ghost.execute
+        expect(email.body.decoded).to eq("transformed HTML")
+      end
+    end
+
+    context "with an multipart email" do
+      let(:html) { "<h1>Hello world!</h1>" }
+      let(:email) do
+        html_string = html
+        Mail.new do
+          text_part { body "Hello world" }
+          html_part { body html_string }
+        end
+      end
+
+      it "adjusts the html part using Roadie" do
+        document = double "A document", transform: "transformed HTML"
+        expect(PhantomTransform).to receive(:new).with(html, included_scripts).and_return document
+        ghost.execute
+        expect(email.html_part.body.decoded).to eq("transformed HTML")
+      end
+    end
+
+  end
+end
+

data/spec/lib/ghost_in_the_post/mailer_spec.rb

@@ -0,0 +1,38 @@
+require "spec_helper"
+require 'mail'
+
+module GhostInThePost
+  describe Mailer do
+    let(:email) { Mail.new(to: "foo@example.com", from: "me@example.com") }
+    let(:instance) { AutoMailer.new(email) }
+
+    describe "#include_script" do
+
+      it "should add to include scripts" do
+        instance.include_script "test.js"
+        expect(instance.included_scripts).to eq(["test.js"])
+      end
+    end
+
+    describe "#mail" do
+      it "should return an email" do
+        expect(instance.mail).to be_kind_of(Mail::Message)
+      end
+
+      it "should return an email extended with GhostOnCommand and GhostOnDelivery" do
+        expect(email).to receive(:extend).with(GhostOnCommand).and_call_original
+        expect(email).to receive(:extend).with(GhostOnDelivery).and_call_original
+        mail = instance.mail
+        expect(mail).to be_kind_of(GhostOnCommand)
+        expect(mail).to be_kind_of(GhostOnDelivery)
+      end
+
+      it "should have set the emails included scripts" do
+        email.extend GhostOnCommand
+        expect(email).to receive(:included_scripts=).with(nil).and_call_original
+        instance.mail
+      end
+    end
+
+  end
+end

data/spec/lib/ghost_in_the_post/phantom_transform_spec.rb

@@ -0,0 +1,134 @@
+require "spec_helper"
+
+module GhostInThePost
+  describe PhantomTransform do
+    let(:html){"<html><head><script></script></head><body><div id='test'></div></body></html>"}
+    let(:js){"(function(){document.getElementById('test').innerHTML='complete';})()"}
+    let(:included_scripts){["application.js"]}
+    subject {PhantomTransform.new(html, included_scripts)}
+
+    before :each do
+      GhostInThePost.config = {phantomjs_path: "this/is/path"}
+      allow_message_expectations_on_nil
+    end
+
+    describe "#initialize" do
+      it "should set html" do
+        pt = PhantomTransform.new(html, included_scripts)
+        expect(pt.instance_variable_get(:@html)).to eq(html)
+      end
+      it "should set included_scripts" do
+        pt = PhantomTransform.new(html, included_scripts)
+        expect(pt.instance_variable_get(:@included_scripts)).to eq(included_scripts)
+      end
+      it "should default included_scripts to an array" do
+        pt = PhantomTransform.new(html, nil)
+        expect(pt.instance_variable_get(:@included_scripts)).to eq([])
+      end
+    end
+
+    describe "#transform" do
+
+      before :each do
+        allow(IO).to receive(:popen)
+      end
+
+      context "with included_scripts" do
+        let(:path) {"/this/is/path"}
+
+        before :each do
+          allow(IO).to receive(:popen)
+          allow(Rails.application).to receive(:assets){
+            {
+              "application.js": "asset string"
+            }
+          }
+          allow_any_instance_of(Tempfile).to receive(:path){path}
+        end
+
+        it "should call transform_with_injections" do
+          expect(subject).to receive(:transform_with_injections)
+          subject.transform
+        end
+
+        it "should create a temp file, write to it and unlink it" do
+          file = Object.new
+          expect(subject).to receive(:injectable_scripts).twice{included_scripts}
+          expect(subject).to receive(:find_asset_in_pipeline).once.with(included_scripts.first){""}
+          expect(Tempfile).to receive(:new).with(['inject', '.js']){file}
+          allow(file).to receive(:path){path}
+          expect(file).to receive(:write)
+          expect(file).to receive(:close)
+          expect(file).to receive(:unlink)
+          subject.transform
+        end
+
+        it "should call IO.popen with arguments" do
+          expect(IO).to receive(:popen).with([
+            GhostInThePost.phantomjs_path, 
+            GhostInThePost::PhantomTransform::PHANTOMJS_SCRIPT, 
+            html,
+            GhostInThePost.remove_js_tags,
+            path
+          ])
+          subject.transform
+        end
+
+        it "should return the result of IO.popen with arguments" do
+          allow(IO).to receive(:popen) {"this is the end"}
+          expect(subject.transform).to eq("this is the end")
+        end
+
+        it "should return the html if there was an error" do
+          file = Object.new
+          allow(Tempfile).to receive(:new).with(['inject', '.js']){file}
+          allow(file).to receive(:path){path}
+          allow(file).to receive(:write)
+          allow(file).to receive(:close)
+          allow(file).to receive(:unlink)
+          allow(IO).to receive(:popen) {raise ArgumentError}
+          expect(subject.transform).to eq(html)
+        end
+
+        it "should unlink file even if there was an error" do
+          file = Object.new
+          allow(Tempfile).to receive(:new).with(['inject', '.js']){file}
+          allow(file).to receive(:path){path}
+          allow(file).to receive(:write)
+          allow(file).to receive(:close)
+          allow(file).to receive(:unlink)
+          allow(IO).to receive(:popen) {raise ArgumentError}
+          subject.transform
+        end                                               
+
+      end
+
+      context "without included_scripts" do
+        let(:pt){PhantomTransform.new(html)}
+
+        it "should call simple_transform" do
+          expect(pt).to receive(:simple_transform)
+          pt.transform
+        end
+
+        it "should call IO.popen with arguments" do
+          expect(IO).to receive(:popen).with([
+            GhostInThePost.phantomjs_path, 
+            GhostInThePost::PhantomTransform::PHANTOMJS_SCRIPT, 
+            html,
+            GhostInThePost.remove_js_tags
+          ])
+          pt.transform
+        end
+
+        it "should return the result of IO.popen with arguments" do
+          allow(IO).to receive(:popen) {"this is the end"}
+          expect(pt.transform).to eq("this is the end")
+        end
+
+      end
+    end
+
+  end
+end
+

data/spec/lib/ghost_in_the_post_spec.rb

@@ -0,0 +1,60 @@
+require "spec_helper"
+
+describe GhostInThePost do
+
+  before :each do
+    #reload so that variables are set to initials values
+    quiet_load 'ghost_in_the_post.rb'
+  end
+  
+  describe "#create=" do
+    it "should complain if you add unknown keys" do
+      expect do
+        GhostInThePost.config = {bad_key: "blah"} 
+      end.to raise_error(ArgumentError)
+    end
+    it "should set phantomjs_path" do
+      GhostInThePost.config = {phantomjs_path: "test_path"}
+      expect(GhostInThePost.phantomjs_path).to eq("test_path")
+    end
+    it "should raise ArgumentError if not path was set for phantom" do
+      expect do
+        GhostInThePost.config = {} 
+      end.to raise_error(ArgumentError)
+    end
+    it "should set includes" do
+      GhostInThePost.config = {phantomjs_path: "test_path", includes: ["test"]}
+      expect(GhostInThePost.includes).to eq(["test"])
+    end
+    it "should force includes to an array" do
+      GhostInThePost.config = {phantomjs_path: "test_path", includes: "test"}
+      expect(GhostInThePost.includes).to eq(["test"])
+    end
+    it "should default includes to an empty array" do
+      GhostInThePost.config = {phantomjs_path: "test_path"}
+      expect(GhostInThePost.includes).to eq([])
+    end
+    it "should set remove_js_tags" do
+      GhostInThePost.config = {phantomjs_path: "test_path", remove_js_tags: false}
+      expect(GhostInThePost.remove_js_tags).to eq(false)
+    end
+    it "should default remove_js_tags to true" do
+      GhostInThePost.config = {phantomjs_path: "test_path"}
+      expect(GhostInThePost.remove_js_tags).to eq(true)
+    end
+  end
+
+  describe "#phantomjs_path" do
+    it "should return phantomjs_path" do
+      GhostInThePost.config = {phantomjs_path: "test_path"}
+      expect(GhostInThePost.phantomjs_path).to eq("test_path")
+    end
+    it "should raise and ArgumentError if the path is not set" do
+      expect do
+        GhostInThePost.phantomjs_path
+      end.to raise_error(ArgumentError)
+    end
+  end
+
+end
+