antwort 0.0.12

data/spec/builder/partial_spec.rb

@@ -0,0 +1,87 @@
+require 'spec_helper'
+
+describe Antwort::PartialBuilder do
+
+  before :all do
+    Dir.chdir(fixtures_root)
+  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')
+      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')
+      end
+
+      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-->' }
+
+      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')
+      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)
+      end
+
+      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
+        h = {
+          "{% &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)
+        end
+      end
+
+      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)
+      end
+    end
+  end
+end

data/spec/builder/style_spec.rb

@@ -0,0 +1,66 @@
+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') }
+
+    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
+      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') }
+
+    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
+        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'
+        })
+      end
+      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
+      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'})
+    end
+  end
+
+  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

data/spec/cli/helpers_spec.rb

@@ -0,0 +1,60 @@
+require 'spec_helper'
+
+class Dummy
+end
+
+describe Antwort::CLIHelpers do
+
+  before :each do
+    @helper = Dummy.new
+    @helper.extend(Antwort::CLIHelpers)
+  end
+
+  before :all do
+    Dir.chdir(fixtures_root)
+  end
+
+  describe 'CLI Helpers' do
+
+    it '#built_emails' do
+      b = @helper.built_emails
+      expect(b).to include('demo-123456')
+      expect(b).not_to include('2-no-layout')
+    end
+
+    it '#available_emails - lists all available emails' do
+      a = @helper.available_emails
+      expect(a).to include('1-demo')
+      expect(a).to include('2-no-layout')
+      expect(a).to include('3-no-title')
+    end
+
+    it '#images_dir - returns images asset path by EMAIL_ID' do
+      expect(@helper.images_dir('foo')).to eq './assets/images/foo'
+    end
+
+    it '#count_files' do
+      expect(@helper.count_files './assets/images/shared').to be(2)
+      expect(@helper.count_files './assets/images/1-demo').to be(1)
+    end
+
+    it '#last_build_by_id' do
+      expect(@helper.last_build_by_id 'foo').to eq('foo-1')
+      expect(@helper.last_build_by_id 'demo').to eq('demo-123457')
+      expect(@helper.last_build_by_id 'demo').not_to eq('demo-123456')
+      expect(@helper.last_build_by_id 'demo').not_to eq('demo-bar-123')
+    end
+
+    it '#list_folders' do
+      a = @helper.list_folders '.'
+      expect(a).to include('assets')
+      expect(a).to include('build')
+      expect(a).to include('data')
+      expect(a).to include('emails')
+      expect(a).to include('views')
+      expect(a).not_to include('support')
+      expect(a).not_to include('lib')
+    end
+
+  end
+end

data/spec/cli/upload_spec.rb

@@ -0,0 +1,47 @@
+require 'spec_helper'
+
+describe Antwort::CLI::Upload do
+
+  subject { described_class.new('newsletter') }
+
+  before :all do
+    Dir.chdir(fixtures_root)
+  end
+
+  before(:each) do
+    allow($stdout).to receive(:write)
+    Fog.mock!
+    allow_any_instance_of(Thor::Actions).to receive(:yes?).and_return(true)
+  end
+
+  after(:each) {  Fog.unmock! }
+
+  describe '#upload' do
+    before :each do
+      allow(Dir).to receive(:foreach)
+    end
+
+    it 'cleans S3 directory' do
+      expect(subject).to receive(:clean_directory!)
+      subject.upload
+    end
+  end
+
+  describe '#connection' do
+    it 'returns S3 connection' do
+      expect(subject.connection).to be_a(Fog::Storage::AWS::Mock)
+    end
+  end
+
+  describe '#directory' do
+    it 'responds to #directory' do
+      expect(subject).to respond_to(:directory)
+    end
+  end
+
+  describe '#clean_directory!' do
+    it 'responds to #clean_directory!' do
+      expect(subject).to respond_to(:clean_directory!)
+    end
+  end
+end

data/spec/cli_spec.rb

@@ -0,0 +1,46 @@
+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 :each do
+      allow($stdout).to receive(:write)
+    end
+
+    context 'user confirms upload' do
+      before :each do
+        allow_any_instance_of(Thor::Actions).to receive(:yes?).and_return(true)
+      end
+
+      it 'uploads mail' do
+        expect(subject).to receive(:upload)
+        output
+      end
+
+      it 'messages success' do
+        expect(output).to include('create')
+      end
+    end
+
+    context 'user denies upload' do
+      before :each do
+        allow_any_instance_of(Thor::Actions).to receive(:yes?).and_return(false)
+      end
+
+      it 'does not upload mail' do
+        expect(subject).not_to receive(:upload_mail)
+        output
+      end
+
+      it 'messages cancelation' do
+        expect(output).to include('aborted')
+      end
+    end
+  end
+end

data/spec/fixtures/build/demo-123456/build.html

@@ -0,0 +1,7 @@
+<!doctype html>
+<html lang="en">
+  <head></head>
+  <body>
+    <h1>Build</h1>
+  </body>
+</html>

data/spec/fixtures/build/demo-123457/build.html

@@ -0,0 +1,7 @@
+<!doctype html>
+<html lang="en">
+  <head></head>
+  <body>
+    <h1>Build</h1>
+  </body>
+</html>

data/spec/fixtures/build/demo-bar-123/build.html

@@ -0,0 +1,7 @@
+<!doctype html>
+<html lang="en">
+  <head></head>
+  <body>
+    <h1>Build</h1>
+  </body>
+</html>

data/spec/fixtures/build/foo-1/build.html

@@ -0,0 +1,7 @@
+<!doctype html>
+<html lang="en">
+  <head></head>
+  <body>
+    <h1>Build</h1>
+  </body>
+</html>

data/spec/fixtures/data/1-demo.yml

@@ -0,0 +1,3 @@
+foo:
+  - title: 'foo'
+  - title: 'bar'

data/spec/fixtures/emails/1-demo/index.html.erb

@@ -0,0 +1,9 @@
+---
+title: Email One
+---
+
+<h1>Hello One</h1>
+
+<% @foo.each do |f| %>
+  <%= f[:title] %>
+<% end %>

data/spec/fixtures/emails/2-no-layout/index.html.erb

@@ -0,0 +1,6 @@
+---
+title: Email Two
+layout: false
+---
+
+<h1>Hello Two</h1>

data/spec/fixtures/emails/3-no-title/index.html.erb

@@ -0,0 +1 @@
+<h1>Hello Three</h1>

data/spec/fixtures/views/404.html.erb

@@ -0,0 +1 @@
+404 error, template not found

data/spec/fixtures/views/index.html.erb

@@ -0,0 +1,14 @@
+
+  <% if @pages.length == 0 %>
+    No emails found
+  <% else %>
+    <ul>
+      <% @pages.each do |page| %>
+      <li>
+        <a href="template/<%= page[:path] %>">
+          <%= page[:title] %>
+        </a>
+      </li>
+      <% end %>
+    </ul>
+  <% end %>

data/spec/fixtures/views/layout.erb

@@ -0,0 +1,8 @@
+<html>
+  <head>
+    <title><%= @title || "Untitled" %></title>
+  </head>
+  <body>
+    <%= yield %>
+  </body>
+</html>

data/spec/fixtures/views/server.erb

@@ -0,0 +1,5 @@
+<html>
+  <body>
+    <%= yield %>
+  </body>
+</html>

data/spec/server_spec.rb

@@ -0,0 +1,54 @@
+require 'spec_helper'
+
+describe 'Antwort Server' do
+
+  def app
+    Antwort::Server
+  end
+
+  describe "Index page" do
+    it "is valid" do
+      get '/'
+      expect(last_response.status).to eq(200)
+    end
+
+    it "lists available templates by title, includes link" do
+      get '/'
+      expect(last_response.body).to include('Email One')
+      expect(last_response.body).to include('Email Two')
+      expect(last_response.body).to include('<a href="template/1-demo">')
+      expect(last_response.body).to include('<a href="template/2-no-layout">')
+    end
+  end
+
+  describe "Template show action" do
+
+    it "shows email preview" do
+      get '/template/1-demo'
+      expect(last_response.body).to include('<h1>Hello One</h1>')
+      expect(last_response.body).to include('<title>Email One</title>')
+    end
+
+    it "loads corresponding data yaml file" do
+      get '/template/1-demo'
+      expect(last_response.body).to include('foo')
+      expect(last_response.body).to include('bar')
+    end
+
+    it "respects layout:false metadata attribute" do
+      get '/template/2-no-layout'
+      expect(last_response.body).not_to include('<body>')
+    end
+
+    it "has default title if none found" do
+      get 'template/3-no-title'
+      expect(last_response.body).to include('Untitled')
+    end
+
+    it "shows 404 error if template not found" do
+      get '/template/does-not-exist'
+      expect(last_response.status).to eq(404)
+    end
+  end
+
+end