rang 0.1.1

data/spec/generators/install_spec.rb

@@ -0,0 +1,40 @@
+require 'spec_helper'
+require 'rails/generators'
+require_relative '../../lib/rails/generators/rang/install/install_generator.rb'
+
+describe Rang::Generators::InstallGenerator do
+
+  let(:klass) { Rang::Generators::InstallGenerator }
+  let(:generator) { klass.new }
+
+  it "has a description" do
+    expect(klass.desc.empty?).to be_falsey
+  end
+
+  describe "#install_generator" do
+    let(:expected_config) do
+      {
+        patch_sprockets_to_use_html_extension: true,
+        disable_html_precompilation: true,
+        frontend_assets_directory: "frontend"
+      }
+    end
+
+    it "installs the generator" do
+      expect(generator).to receive(:template).with("rang.rb", "config/initializers/rang.rb")
+      generator.install_initializer
+    end
+
+    it "installs a generator with appropriate config" do
+      expect(generator).to receive(:template) do |file, path|
+        eval(IO.read(File.join(klass.source_root, file)))
+        expected_config.each do |key, value|
+          expect(Rang::Config.send(key)).to eq value
+        end
+      end
+
+      generator.install_initializer
+    end
+  end
+
+end

data/spec/generators/teaspoon_install_spec.rb

@@ -0,0 +1,123 @@
+require "spec_helper"
+require "rails/generators"
+require_relative "../../lib/rails/generators/rang/teaspoon/install/install_generator.rb"
+
+describe Rang::Teaspoon::Generators::InstallGenerator do
+
+  let(:klass) { Rang::Teaspoon::Generators::InstallGenerator }
+  let(:generator) { klass.new }
+
+  it "has a description" do
+    expect(klass.desc.empty?).to be_falsey
+  end
+
+  describe "#add_teaspoon_to_gemfile" do
+    it "Adds gems to gemfile" do
+      expect(generator).to receive(:gem_group).with(:development, :test).and_yield
+      expect(generator).to receive(:gem).with("teaspoon", an_instance_of(String))
+      expect(generator).to receive(:gem).with("phantomjs", an_instance_of(String))
+      generator.add_teaspoon_to_gemfile
+    end
+  end
+
+  describe "#add_angular_mocks_to_gemfile" do
+    it "Runs bower:add generator with angular-mocks" do
+      expect(generator).to receive(:generate)
+        .with("rang:bower:add", "angular-mocks '~> 1.2.0' quiet")
+      generator.add_angular_mocks_to_gemfile
+    end
+  end
+
+  describe "#teaspoon_install" do
+    it "runs teaspoon:install" do
+      expect(generator).to receive(:generate).with("teaspoon:install")
+      generator.teaspoon_install
+    end
+  end
+
+  describe "#add_teaspoon_assets_hack" do
+    it "injects fix into teaspoon_env" do
+      expect(generator).to receive(:inject_into_file)
+        .with("spec/teaspoon_env.rb", {before: "Teaspoon.configure do |config|"}) do |a, b, &block|
+          expect(block.call).to be_a String
+        end
+      generator.add_teaspoon_assets_hack
+    end
+  end
+
+  describe "#add_matcher" do
+    it "adds test matcher to teaspoon_env" do
+      expect(generator).to receive(:inject_into_file) do |filename, regex, &block|
+        expect(filename).to eq "spec/teaspoon_env.rb"
+        expect(block.call).to be_a String
+      end
+      generator.add_matcher
+    end
+
+    it "uses a line regex that matches the right line" do
+      expect(generator).to receive(:inject_into_file) do |filename, regex, &block|
+        regex = regex[:after]
+        expect("#suite.matcher = \"{spec/javascripts,app/assets}/**/*[_.]spec.{js,js.coffee,coffee}\"" =~ regex).to be_truthy
+      end
+      generator.add_matcher
+    end
+
+    it "uses a line regex that doesn't match the similar line later on" do
+      expect(generator).to receive(:inject_into_file) do |filename, regex, &block|
+        regex = regex[:after]
+        expect("  #config.suite :targeted do |suite|\n#  suite.matcher = \"test/javascripts/targeted/*_test.{js,js.coffee,coffee}\"\n  #end" =~ regex ).to be_falsey
+      end
+      generator.add_matcher
+    end
+  end
+
+  describe "#require_angular_mocks" do
+    it "adds angular-mocks require to test_helper" do
+      expect(generator).to receive(:inject_into_file) do |filename, regex, &block|
+        expect(filename).to eq "spec/javascripts/spec_helper.js"
+        expect(block.call).to eq "//= require angular\n//= require angular-mocks\n"
+      end
+      generator.require_angular_mocks
+    end
+
+    it "uses a line regex that matches the right line" do
+      expect(generator).to receive(:inject_into_file) do |filename, regex, &block|
+        regex = regex[:before]
+        expect("//= require application" =~ regex).to be_truthy
+      end
+      generator.require_angular_mocks
+    end
+  end
+
+  describe "#inform_of_actions" do
+    it "informs the user of some things" do
+      expect(generator).to receive(:say).at_least(1).times
+      generator.inform_of_actions
+    end
+  end
+
+  describe "#inform_about_better_errors" do
+    context "when better_errors is in the Gemfile" do
+      before do
+        allow(Gem::Specification).to receive(:find_all_by_name).with('better_errors').and_return([true])
+      end
+
+      it "warns the user" do
+        expect(generator).to receive(:say).at_least(1).times
+        generator.inform_about_better_errors
+      end
+    end
+
+    context "when better_errors is not in the Gemfile" do
+      before do
+        allow(Gem::Specification).to receive(:find_all_by_name).with('better_errors').and_return([])
+      end
+
+      it "does not speak" do
+        expect(generator).to_not receive(:say)
+        generator.inform_about_better_errors
+      end
+    end
+  end
+
+end

data/spec/helpers_spec.rb

@@ -0,0 +1,64 @@
+require 'spec_helper'
+
+class ContextFake
+  def return_self(*arguments)
+    self
+  end
+
+  def escape_javascript(string)
+    string
+  end
+
+  alias_method :assets, :return_self
+  alias_method :logical_paths, :return_self
+  alias_method :assets, :return_self
+  alias_method :environment, :return_self
+  alias_method :find_asset, :return_self
+  alias_method :depend_on_asset, :return_self
+end
+
+describe Rang::Helpers do
+
+  describe '#templates' do
+    let(:context) { ContextFake.new }
+
+    it 'calls given block for each template path' do
+      allow(context).to receive(:logical_paths).and_return(['one.html','two.html','three.html', 'invalid.jpg'])
+      expect do |b|
+        Rang::Helpers.templates(context, &b)
+      end.to yield_control.exactly(3).times
+    end
+
+    it 'calls given block with valid slimmed path' do
+      allow(context).to receive(:logical_paths).and_return(['one.html'])
+      expect do |b|
+        Rang::Helpers.templates(context, &b)
+      end.to yield_with_args('one.html', anything)
+    end
+
+    it 'calls given block with template string' do
+      template_string = '<p>html</p>'
+      allow(context).to receive(:logical_paths).and_return(['one.html'])
+      allow(context).to receive(:find_asset).and_return(template_string)
+      expect do |b|
+        Rang::Helpers.templates(context, &b)
+      end.to yield_with_args(anything, Rang::Helpers.escape_javascript(template_string))
+    end
+
+    it 'calls given block with escaped template string' do
+      template_string = '<p>html</p>'
+      allow(context).to receive(:logical_paths).and_return(['one.html'])
+      allow(Rang::Helpers).to receive(:escape_javascript).and_return(template_string)
+      expect do |b|
+        Rang::Helpers.templates(context, &b)
+      end.to yield_with_args(anything, template_string)
+    end
+
+    it 'calls depend_on_asset for each template' do
+      template_path = 'one.html'
+      allow(context).to receive(:logical_paths).and_return([template_path, 'invalid.jpg'])
+      expect(context).to receive(:depend_on_asset).with(template_path)
+      Rang::Helpers.templates(context) {}
+    end
+  end
+end

data/spec/patcher_spec.rb

@@ -0,0 +1,103 @@
+require "spec_helper"
+
+describe Rang::Patcher do
+
+  def enable_config(prop, value=true)
+    allow(Rang::Config).to receive(prop).and_return(value)
+  end
+
+  describe "#patch_sprockets_to_use_html_extension!" do
+    let(:trail) { double() }
+
+    before do
+      enable_config(:patch_sprockets_to_use_html_extension)
+      allow(trail).to receive(:alias_extension)
+      allow(Rails.application.assets).to receive(:send).and_return(trail)
+      allow(Rails.application.assets).to receive(:instance_eval)
+      allow(Rails.application.assets).to receive(:register_mime_type)
+    end
+
+    it "patches sprockets trail with slim->html alias" do
+      expect(Rails.application.assets).to receive(:send).with(:trail).and_return(trail)
+      expect(trail).to receive(:alias_extension).with(".slim", ".html")
+      Rang::Patcher.patch!
+    end
+
+    it "patches assets to change #extension_for_mime_type behaviour" do
+      expect(Rails.application.assets).to receive(:instance_eval).and_call_original
+      allow(Rails.application.assets).to receive(:register_mime_type).and_call_original
+      Rang::Patcher.patch!
+    end
+
+    it "registers the text/html mime type to .html" do
+      allow(Rails.application.assets).to receive(:instance_eval).and_call_original
+      allow(Rails.application.assets).to receive(:register_mime_type).and_call_original
+      Rang::Patcher.patch!
+      expect(Rails.application.assets.extension_for_mime_type("text/html")).to eq ".html"
+    end
+  end
+
+  describe "#disable_html_precompilation!" do
+    before do
+      enable_config(:disable_html_precompilation)
+    end
+
+    it "Changes assets.precompile to ignore html files" do
+      expect(Rails.application.config.assets).to receive(:precompile=) do |proc, regex|
+        expect(proc.call("asset.html", "app/assets")).to be_falsey
+      end
+      Rang::Patcher.patch!
+    end
+  end
+
+  describe "#add_frontend_assets_directory!" do
+    before do
+      enable_config(:frontend_assets_directory, "frontend")
+    end
+
+    it "adds /frontend to assets directories" do
+      Rang::Patcher.patch!
+      expect(Rails.application.config.assets.paths).to include "#{Rails.root}/app/assets/frontend"
+    end
+  end
+
+  describe "#register_slim_as_assets_engine!" do
+    context "when slim gem is present" do
+      before do
+        allow(Rang::Util).to receive(:gem_present?).and_return(true)
+      end
+
+      it "registers slim as an assets engine" do
+        expect(Rails.application.assets).to receive(:register_engine)
+          .with(".slim", ::Slim::Template)
+        Rang::Patcher.patch!
+      end
+    end
+
+    context "when slim gem is not present" do
+      before do
+        allow(Rang::Util).to receive(:gem_present?).and_return(false)
+      end
+
+      it "does not register slim as an assets engine" do
+        expect(Rails.application.assets).to_not receive(:register_engine)
+          .with(".slim", ::Slim::Template)
+        Rang::Patcher.patch!
+      end
+    end
+  end
+
+  describe "#remove_angular_delims_from_slim!" do
+
+    before do
+      allow(Rang::Util).to receive(:gem_present?).and_return(true)
+      enable_config(:remove_angular_delims_from_slim, "frontend")
+    end
+
+    it "should change attr_delims in slim default_config" do
+      expect(Slim::Engine).to receive(:set_default_options).with({attr_delims: {'(' => ')', '[' => ']'}})
+      Rang::Patcher.patch!
+    end
+  end
+
+end

data/spec/rails_app/.gitignore

@@ -0,0 +1,16 @@
+# See https://help.github.com/articles/ignoring-files for more about ignoring files.
+#
+# If you find yourself ignoring temporary files generated by your text editor
+# or operating system, you probably want to add a global ignore instead:
+#   git config --global core.excludesfile '~/.gitignore_global'
+
+# Ignore bundler config.
+/.bundle
+
+# Ignore the default SQLite database.
+/db/*.sqlite3
+/db/*.sqlite3-journal
+
+# Ignore all logfiles and tempfiles.
+/log/*.log
+/tmp

data/spec/rails_app/Gemfile

@@ -0,0 +1,41 @@
+source 'https://rubygems.org'
+
+
+# Bundle edge Rails instead: gem 'rails', github: 'rails/rails'
+gem 'rails', '4.1.4'
+# Use sqlite3 as the database for Active Record
+gem 'sqlite3'
+# Use SCSS for stylesheets
+gem 'sass-rails', '~> 4.0.3'
+# Use Uglifier as compressor for JavaScript assets
+gem 'uglifier', '>= 1.3.0'
+# Use CoffeeScript for .js.coffee assets and views
+gem 'coffee-rails', '~> 4.0.0'
+# See https://github.com/sstephenson/execjs#readme for more supported runtimes
+# gem 'therubyracer',  platforms: :ruby
+
+# Use jquery as the JavaScript library
+gem 'jquery-rails'
+# Turbolinks makes following links in your web application faster. Read more: https://github.com/rails/turbolinks
+gem 'turbolinks'
+# Build JSON APIs with ease. Read more: https://github.com/rails/jbuilder
+gem 'jbuilder', '~> 2.0'
+# bundle exec rake doc:rails generates the API under doc/api.
+gem 'sdoc', '~> 0.4.0',          group: :doc
+
+# Spring speeds up development by keeping your application running in the background. Read more: https://github.com/rails/spring
+gem 'spring',        group: :development
+
+# Use ActiveModel has_secure_password
+# gem 'bcrypt', '~> 3.1.7'
+
+# Use unicorn as the app server
+# gem 'unicorn'
+
+# Use Capistrano for deployment
+# gem 'capistrano-rails', group: :development
+
+# Use debugger
+# gem 'debugger', group: [:development, :test]
+
+gem 'rang', path: '../../'

data/spec/rails_app/README.rdoc

@@ -0,0 +1,28 @@
+== README
+
+This README would normally document whatever steps are necessary to get the
+application up and running.
+
+Things you may want to cover:
+
+* Ruby version
+
+* System dependencies
+
+* Configuration
+
+* Database creation
+
+* Database initialization
+
+* How to run the test suite
+
+* Services (job queues, cache servers, search engines, etc.)
+
+* Deployment instructions
+
+* ...
+
+
+Please feel free to use a different markup language if you do not plan to run
+<tt>rake doc:app</tt>.

data/spec/rails_app/Rakefile

@@ -0,0 +1,6 @@
+# Add your own tasks in files placed in lib/tasks ending in .rake,
+# for example lib/tasks/capistrano.rake, and they will automatically be available to Rake.
+
+require File.expand_path('../config/application', __FILE__)
+
+Rails.application.load_tasks

data/spec/rails_app/app/assets/javascripts/application.js

@@ -0,0 +1,16 @@
+// This is a manifest file that'll be compiled into application.js, which will include all the files
+// listed below.
+//
+// Any JavaScript/Coffee file within this directory, lib/assets/javascripts, vendor/assets/javascripts,
+// or vendor/assets/javascripts of plugins, if any, can be referenced here using a relative path.
+//
+// It's not advisable to add code directly here, but if you do, it'll appear at the bottom of the
+// compiled file.
+//
+// Read Sprockets README (https://github.com/sstephenson/sprockets#sprockets-directives) for details
+// about supported directives.
+//
+//= require jquery
+//= require jquery_ujs
+//= require turbolinks
+//= require_tree .