antwort 0.0.12 → 1.0.0

data/spec/builder/partial_spec.rb

@@ -1,87 +1,78 @@
 require 'spec_helper'
 
 describe Antwort::PartialBuilder do
+  let(:builder) { Antwort::PartialBuilder.new(email: '1-demo', id: '123', 'css-style': 'expanded') }
 
-  before :all do
-    Dir.chdir(fixtures_root)
+  describe 'Builds' do
+    it 'builds HTML'
+    it 'inlines CSS'
   end
 
-  before :each do
-    allow($stdout).to receive(:write) # Ignore warnings
-    @builder = Antwort::PartialBuilder.new({email: '1-demo', id: '123', 'css-style': 'expanded'})
-  end
-
-  describe "Builds" do
-    it "builds HTML"
-    it "inlines CSS"
-  end
-
-  describe "Helpers" do
-    describe "adjusts filename as necssary (make sure it ends with .html)" do
-
-      it "always starts with _" do
-        expect(@builder.adjust_filename('foo.erb')).to eq('_foo.html')
-        expect(@builder.adjust_filename('foo.html')).to eq('_foo.html')
-        expect(@builder.adjust_filename('foo.html.erb')).to eq('_foo.html')
+  describe 'Helpers' do
+    describe 'adjusts filename as necssary (make sure it ends with .html)' do
+      it 'always starts with _' do
+        expect(builder.adjust_filename('foo.erb')).to eq('_foo.html')
+        expect(builder.adjust_filename('foo.html')).to eq('_foo.html')
+        expect(builder.adjust_filename('foo.html.erb')).to eq('_foo.html')
       end
 
-      it "always ends with .html" do
-        expect(@builder.adjust_filename('_foo.html.erb')).to eq('_foo.html')
-        expect(@builder.adjust_filename('_foo.erb')).to eq('_foo.html')
-        expect(@builder.adjust_filename('_foo.html')).to eq('_foo.html')
+      it 'always ends with .html' do
+        expect(builder.adjust_filename('_foo.html.erb')).to eq('_foo.html')
+        expect(builder.adjust_filename('_foo.erb')).to eq('_foo.html')
+        expect(builder.adjust_filename('_foo.html')).to eq('_foo.html')
       end
 
-      it "uses template name instead of _index" do
-        expect(@builder.adjust_filename('index.html.erb')).to eq('_1-demo.html')
+      it 'uses template name instead of _index' do
+        expect(builder.adjust_filename('index.html.erb')).to eq('_1-demo.html')
       end
     end
 
-    describe "save from Nokogiri" do
-      let (:start) { '<div id="valid-dom-tree">' }
-      let (:ende)  {'</div><!--/#valid-dom-tree-->' }
+    describe 'save from Nokogiri' do
+      let(:start) { '<div id="valid-dom-tree">' }
+      let(:ende)  { '</div><!--/#valid-dom-tree-->' }
 
-      it "can add nokogiri wrapper" do
-        expect(@builder.send(:add_nokogiri_wrapper, 'foo')).to eq(start + 'foo' + ende)
+      it 'can add nokogiri wrapper' do
+        expect(builder.send(:add_nokogiri_wrapper, 'foo')).to eq(start + 'foo' + ende)
       end
 
-      it "can remove nokogiri wrapper" do
-        expect(@builder.send(:remove_nokogiri_wrapper, start + 'foo' + ende)).to eq('foo')
-        expect(@builder.send(:remove_nokogiri_wrapper, start + 'foo' + "</div>  <!--/#valid-dom-tree-->")).to eq('foo')
-        expect(@builder.send(:remove_nokogiri_wrapper, start + 'foo' + "</div>\n<!--/#valid-dom-tree-->")).to eq('foo')
+      it 'can remove nokogiri wrapper' do
+        expect(builder.send(:remove_nokogiri_wrapper, start + 'foo' + ende)).to eq('foo')
+        expect(builder.send(:remove_nokogiri_wrapper, start + 'foo' + '</div>  <!--/#valid-dom-tree-->')).to eq('foo')
+        expect(builder.send(:remove_nokogiri_wrapper, start + 'foo' + "</div>\n<!--/#valid-dom-tree-->")).to eq('foo')
       end
 
       # TODO: These specs don't work because PartialBuilder cannot find matching CSS
 
-      it "adds wrapper before inlining", skip: true do
-        @builder.inline('foo')
-        expect(@builder).to receive(:add_nokogiri_wrapper)
+      it 'adds wrapper before inlining', skip: true do
+        builder.inline('foo')
+        expect(builder).to receive(:add_nokogiri_wrapper)
       end
 
-      it "removes wrapper after inlining", skip: true do
-        @builder.inline('foo')
-        expect(@builder).to receive(:remove_nokogiri_wrapper)
+      it 'removes wrapper after inlining', skip: true do
+        builder.inline('foo')
+        expect(builder).to receive(:remove_nokogiri_wrapper)
       end
     end
 
-    describe "Clean up" do
-      it "can convert logic html entities back to operators" do
+    describe 'Clean up' do
+      it 'can convert logic html entities back to operators' do
         h = {
-          "{% &lt;= %}" => "{% <= %}",  # only convert operators within logic
-          "{{ &lt;= }}" => "{{ <= }}",  # leave operators in content
-          "{% &gt;= %}" => "{% >= %}",
-          "{{ &gt;= }}" => "{{ >= }}",
-          " &amp;&amp; " => " && "
+          '{% &lt;= %}' => '{% <= %}',  # only convert operators within logic
+          '{{ &lt;= }}' => '{{ <= }}',  # leave operators in content
+          '{% &gt;= %}' => '{% >= %}',
+          '{{ &gt;= }}' => '{{ >= }}',
+          ' &amp;&amp; ' => ' && '
         }
         h.each do |key, value|
-          expect(@builder.cleanup_logic(key)).to eq(value)
+          expect(builder.cleanup_logic(key)).to eq(value)
         end
       end
 
-      it "can restore variables in links" do
+      it 'can restore variables in links' do
         a = '<a href="%7B%7B%20hello%20%7D%7D">world</a>'
         b = '<a href="{{ hello }}">world</a>'
-        expect(@builder.restore_variables_in_links(a)).to eq(b)
+        expect(builder.restore_variables_in_links(a)).to eq(b)
       end
     end
   end
-end
+end

data/spec/builder/style_spec.rb

@@ -1,66 +1,57 @@
-require "spec_helper"
+require 'spec_helper'
 
 describe Antwort::Style do
+  describe 'on Initialize' do
+    let(:style) { Antwort::Style.new('font-weight:bold;font-size:24px;line-height:28px;font-family:Helvetica, Arial, sans-serif;color:#111111;color:#2CA4D8') }
 
-  describe "on Initialize" do
-    let (:style) { Antwort::Style.new('font-weight:bold;font-size:24px;line-height:28px;font-family:Helvetica, Arial, sans-serif;color:#111111;color:#2CA4D8') }
-
-    it "remembers all keys" do
-      expect(style.keys).to eq(['font-weight','font-size','line-height','font-family','color'])
+    it 'remembers all keys' do
+      expect(style.keys).to eq(['font-weight', 'font-size', 'line-height', 'font-family', 'color'])
     end
-    it "remembers duplicate keys" do
+    it 'remembers duplicate keys' do
       expect(style.duplicate_keys).to eq(['color'])
     end
   end
 
-  describe "Readable Attributes" do
-    let (:style) { Antwort::Style.new('font-size:24px;color:black;color:red') }
+  describe 'Readable Attributes' do
+    let(:style) { Antwort::Style.new('font-size:24px;color:black;color:red') }
 
-    context "not flattened" do
-      it "can be accessed as a hash" do
-        expect(style.original).to eq([{'font-size' => '24px'}, {'color'=>'black'}, {'color'=>'red'}])
+    context 'not flattened' do
+      it 'can be accessed as a hash' do
+        expect(style.original).to eq([{ 'font-size' => '24px' }, { 'color' => 'black' }, { 'color' => 'red' }])
       end
-      it "can be accessed as a string" do
+      it 'can be accessed as a string' do
         expect(style.original_str).to eq('font-size:24px;color:black;color:red')
       end
     end
 
-    context "flattened" do
-      it "can be accessed as a hash" do
-        expect(style.flattened).to eq({
-          'font-size' => '24px',
-          'color' => 'red'
-        })
+    context 'flattened' do
+      it 'can be accessed as a hash' do
+        expect(style.flattened).to eq('font-size' => '24px', 'color' => 'red')
       end
-      it "can be accessed as a string" do
+      it 'can be accessed as a string' do
         expect(style.flattened_str).to eq('font-size:24px;color:red')
       end
     end
 
-    it "has a duplicates? method" do
+    it 'has a duplicates? method' do
       expect(Antwort::Style.new('color:red;color:black').duplicates?).to eq(true)
       expect(Antwort::Style.new('color:red;font-size:12px').duplicates?).to eq(false)
     end
   end
 
-  describe "Code Formatting" do
-    it "trims leading and trailing white space from styles" do
-      style =  Antwort::Style.new('color:red; color: black;border :none;')
-      expect(style.flattened).to eq({
-        'color' => 'black',
-        'border' => 'none'})
-      expect(style.flattened).not_to eq({
-        ' color' => ' black',
-        'border ' => 'none'})
+  describe 'Code Formatting' do
+    it 'trims leading and trailing white space from styles' do
+      style = Antwort::Style.new('color:red; color: black;border :none;')
+      expect(style.flattened).to eq('color' => 'black', 'border' => 'none')
+      expect(style.flattened).not_to eq(' color' => ' black', 'border ' => 'none')
     end
   end
 
-  describe "Internal Helpers" do
-    it "can push styles hash back into sting" do
+  describe 'Internal Helpers' do
+    it 'can push styles hash back into sting' do
       hash = { 'font-size' => '14px', 'font-family' => 'Helvetica' }
       style = Antwort::Style.new
       expect(style.send(:hash_to_str, hash)).to eq('font-size:14px;font-family:Helvetica')
     end
   end
-
-end
+end

data/spec/{cli_spec.rb → cli/cli_spec.rb}

@@ -1,19 +1,18 @@
 require 'spec_helper'
 
 describe Antwort::CLI do
-
-  before :all do
-    Dir.chdir(fixtures_root)
-  end
-
   describe '#upload' do
-    let(:output) { capture(:stdout) { subject.upload('newsletter') } }
+    before :all do
+      Fog.mock!
+    end
 
-    before :each do
-      allow($stdout).to receive(:write)
+    after :all do
+      Fog.unmock!
     end
 
-    context 'user confirms upload' do
+    let(:output) { capture(:stdout) { subject.upload('1-demo') } }
+
+    context 'user confirms upload', skip: true do
       before :each do
         allow_any_instance_of(Thor::Actions).to receive(:yes?).and_return(true)
       end

data/spec/cli/send_spec.rb

@@ -0,0 +1,80 @@
+require 'spec_helper'
+
+describe Antwort::CLI::Send do
+  after :each do
+    Mail::TestMailer.deliveries.clear
+  end
+
+  describe 'Send' do
+    it 'can send emails' do
+      Antwort::CLI::Send.new('demo-20160101').send
+      expect(Mail::TestMailer.deliveries.length).to eq(1)
+    end
+  end
+
+  describe 'Defaults' do
+    before :each do
+      @a = Antwort::CLI::Send.new('demo-20160101')
+      @a.send
+      @sent = Mail::TestMailer.deliveries.last
+    end
+
+    it 'has a default text body' do
+      expect(@sent.text_part.body).to eq('This is plain text')
+    end
+
+    describe 'set by environment variables' do
+      it 'from' do
+        expect(@sent.from.length).to eq(1)
+        expect(@sent.from.first).to eq('TEST_ENV_SEND_FROM')
+      end
+
+      it 'to' do
+        expect(@sent.to.length).to eq(1)
+        expect(@sent.to.first).to eq('TEST_ENV_SEND_TO')
+      end
+    end
+
+    it "uses to build <title> prefixed with '[Test] '" do
+      expect(@sent.subject).to eq('[Test] Fixture Demo')
+    end
+
+    it 'has an HTML body, that is pulled from build index.html' do
+      # comes back as empty string in testing?
+      # expect(@sent.html_part.body.to_s).not_to eq ''
+      # puts @sent.to_s.inspect
+
+      contents = File.open('build/demo-20160101/demo.html').read
+      expect(@a.html_body).to eq(contents)
+    end
+  end
+
+  describe 'Optional Arguments' do
+    before :each do
+      Antwort::CLI::Send.new(
+        'demo-20160101',
+        from: 'CUSTOM_FROM',
+        recipient: 'CUSTOM_TO',
+        subject: 'MY SUBJECT'
+      ).send
+      @sent = Mail::TestMailer.deliveries.last
+    end
+
+    it 'accepts a sender argument' do
+      expect(@sent.from.first).to eq('CUSTOM_FROM')
+    end
+
+    it 'accepts one recipient' do
+      expect(@sent.to).to include('CUSTOM_TO')
+    end
+
+    it 'accepts multiple recipients' do
+      Antwort::CLI::Send.new('demo-20160101', recipient: 'one, two').send
+      expect(Mail::TestMailer.deliveries.last.to).to eq(%w(one two))
+    end
+
+    it 'accepts a title argument' do
+      expect(@sent.subject).to eq('MY SUBJECT')
+    end
+  end
+end

data/spec/cli/upload_spec.rb

@@ -1,27 +1,26 @@
 require 'spec_helper'
 
 describe Antwort::CLI::Upload do
-
-  subject { described_class.new('newsletter') }
+  subject { Antwort::CLI::Upload.new('1-demo') }
 
   before :all do
-    Dir.chdir(fixtures_root)
+    Fog.mock!
   end
 
-  before(:each) do
-    allow($stdout).to receive(:write)
-    Fog.mock!
-    allow_any_instance_of(Thor::Actions).to receive(:yes?).and_return(true)
+  after :all do
+    Fog.unmock!
   end
 
-  after(:each) {  Fog.unmock! }
+  before :each do
+    allow_any_instance_of(Thor::Actions).to receive(:yes?).and_return(true)
+  end
 
   describe '#upload' do
     before :each do
       allow(Dir).to receive(:foreach)
     end
 
-    it 'cleans S3 directory' do
+    xit 'cleans S3 directory before upload' do
       expect(subject).to receive(:clean_directory!)
       subject.upload
     end
@@ -44,4 +43,13 @@ describe Antwort::CLI::Upload do
       expect(subject).to respond_to(:clean_directory!)
     end
   end
+
+  describe('#upload_path') do
+    it 'retuns AWS path (excludes assets/images)' do
+      expect(subject.upload_path('foo.png')).to eq('1-demo/foo.png')
+    end
+  end
+
+  it 'uploads files to AWS'
+  it 'allows user to abort upload'
 end

data/spec/email/build_spec.rb

@@ -0,0 +1,13 @@
+require 'spec_helper'
+
+describe Antwort::Build do
+  describe 'Attributes' do
+    it 'has a timestamp id'
+    it 'accepts single or many emails'
+  end
+
+  describe 'Builds' do
+    it 'can build emails'
+    it 'can build partials'
+  end
+end

data/spec/email/collection_spec.rb

@@ -0,0 +1,56 @@
+require 'spec_helper'
+
+describe Antwort::EmailCollection do
+  let(:collection) { Antwort::EmailCollection.new }
+  let(:fixtures_email_list) { ['1-demo', '2-no-layout', '3-no-title', '4-custom-layout', 'demo', 'no-images'] }
+
+  describe 'has a templates attribute' do
+    it 'is an array' do
+      expect(collection.templates).to be_kind_of(Array)
+    end
+
+    it 'holds EmailTemplates' do
+      expect(collection.templates.first).to be_kind_of(Antwort::EmailTemplate)
+    end
+
+    it 'loads emails by directory names' do
+      result = fixtures_email_list
+      expect(collection.list).to eq(result)
+    end
+
+    describe 'filters' do
+      it 'excludes non-email folders' do
+        expect(collection.list).not_to include('.')
+        expect(collection.list).not_to include('..')
+      end
+
+      it 'excludes the shared folder' do
+        expect(collection.list).not_to include('shared')
+      end
+    end
+
+    describe 'API' do
+      describe '`#empty?`' do
+        context 'has templates' do
+          it 'returns false' do
+            expect(collection.empty?).to be false
+          end
+        end
+        context 'has no templates' do
+          it 'returns true' do
+            Dir.chdir("#{fixtures_root}/emails")
+            c = Antwort::EmailCollection.new
+            expect(c.empty?).to be true
+
+            # Go back or randomized specs fail
+            Dir.chdir(fixtures_root)
+          end
+        end
+      end
+
+      it '#total' do
+        expect(collection.total).to eq fixtures_email_list.length
+      end
+    end
+  end
+end