synvert 0.0.3 → 0.0.4

data/spec/synvert/rewriter_spec.rb

@@ -7,10 +7,11 @@ module Synvert
       expect(rewriter.description).to eq 'this is description'
     end
 
-    it 'parses gem_spec' do
-      expect(Rewriter::GemSpec).to receive(:new).with('synvert', '1.0.0')
-      Rewriter.new 'description' do
-        gem_spec 'synvert', '1.0.0'
+    describe 'get / set' do
+      it 'set and get parameters' do
+        rewriter = Rewriter.new 'this is description' do; end
+        rewriter.set 'parameter', 'post', ':title, :description'
+        expect(rewriter.get 'parameter', 'post').to eq ':title, :description'
       end
     end
 

data/spec/synvert/snippets/factory_girl/syntax_methods_spec.rb

@@ -0,0 +1,154 @@
+require 'spec_helper'
+
+describe 'FactoryGirl uses short synax' do
+  before do
+    Synvert::Configuration.instance.set :path, '.'
+    allow_any_instance_of(Synvert::Rewriter::GemSpec).to receive(:match?).and_return(true)
+    rewriter_path = File.join(File.dirname(__FILE__), '../../../../lib/synvert/snippets/factory_girl/syntax_methods.rb')
+    @rewriter = eval(File.read(rewriter_path))
+  end
+
+  describe 'rspec', fakefs: true do
+    let(:spec_helper_content) {'''
+RSpec.configure do |config|
+  config.treat_symbols_as_metadata_keys_with_true_values = true
+  config.run_all_when_everything_filtered = true
+end
+    '''}
+    let(:spec_helper_rewritten_content) {'''
+RSpec.configure do |config|
+  config.include FactoryGirl::Syntax::Methods
+  config.treat_symbols_as_metadata_keys_with_true_values = true
+  config.run_all_when_everything_filtered = true
+end
+    '''}
+    let(:post_spec_content) {'''
+describe Post do
+  it "tests post" do
+    post1 = FactoryGirl.create(:post)
+    post2 = FactoryGirl.build(:post)
+    post_attributes = FactoryGirl.attributes_for(:post)
+    post3 = FactoryGirl.build_stubbed(:post)
+    posts1 = FactoryGirl.create_list(:post, 2)
+    posts2 = FactoryGirl.build_list(:post, 2)
+    posts3 = FactoryGirl.create_pair(:post)
+    posts4 = FactoryGirl.build_pair(:post)
+  end
+end
+    '''}
+    let(:post_spec_rewritten_content) {'''
+describe Post do
+  it "tests post" do
+    post1 = create(:post)
+    post2 = build(:post)
+    post_attributes = attributes_for(:post)
+    post3 = build_stubbed(:post)
+    posts1 = create_list(:post, 2)
+    posts2 = build_list(:post, 2)
+    posts3 = create_pair(:post)
+    posts4 = build_pair(:post)
+  end
+end
+    '''}
+
+    it 'process' do
+      FileUtils.mkdir 'spec'
+      FileUtils.mkdir 'spec/models'
+      File.write 'spec/spec_helper.rb', spec_helper_content
+      File.write 'spec/models/post_spec.rb', post_spec_content
+      @rewriter.process
+      expect(File.read 'spec/spec_helper.rb').to eq spec_helper_rewritten_content
+      expect(File.read 'spec/models/post_spec.rb').to eq post_spec_rewritten_content
+    end
+  end
+
+  describe 'test/unit', fakefs: true do
+    let(:test_helper_content) {'''
+class ActiveSupport::TestCase
+end
+    '''}
+    let(:test_helper_rewritten_content) {'''
+class ActiveSupport::TestCase
+  include FactoryGirl::Syntax::Methods
+end
+    '''}
+    let(:post_test_content) {'''
+test "post" do
+  post1 = FactoryGirl.create(:post)
+  post2 = FactoryGirl.build(:post)
+  post_attributes = FactoryGirl.attributes_for(:post)
+  post3 = FactoryGirl.build_stubbed(:post)
+  posts1 = FactoryGirl.create_list(:post, 2)
+  posts2 = FactoryGirl.build_list(:post, 2)
+  posts3 = FactoryGirl.create_pair(:post)
+  posts4 = FactoryGirl.build_pair(:post)
+end
+    '''}
+    let(:post_test_rewritten_content) {'''
+test "post" do
+  post1 = create(:post)
+  post2 = build(:post)
+  post_attributes = attributes_for(:post)
+  post3 = build_stubbed(:post)
+  posts1 = create_list(:post, 2)
+  posts2 = build_list(:post, 2)
+  posts3 = create_pair(:post)
+  posts4 = build_pair(:post)
+end
+    '''}
+
+    it 'process' do
+      FileUtils.mkdir 'test'
+      FileUtils.mkdir 'test/unit'
+      File.write 'test/test_helper.rb', test_helper_content
+      File.write 'test/unit/post_test.rb', post_test_content
+      @rewriter.process
+      expect(File.read 'test/test_helper.rb').to eq test_helper_rewritten_content
+      expect(File.read 'test/unit/post_test.rb').to eq post_test_rewritten_content
+    end
+  end
+
+  describe 'cucumber', fakefs: true do
+    let(:env_content) {'''
+require "cucumber/rails"
+    '''}
+    let(:env_rewritten_content) {'''
+require "cucumber/rails"
+World(FactoryGirl::Syntax::Methods)
+    '''}
+    let(:post_steps_content) {'''
+test "post" do
+  post1 = FactoryGirl.create(:post)
+  post2 = FactoryGirl.build(:post)
+  post_attributes = FactoryGirl.attributes_for(:post)
+  post3 = FactoryGirl.build_stubbed(:post)
+  posts1 = FactoryGirl.create_list(:post, 2)
+  posts2 = FactoryGirl.build_list(:post, 2)
+  posts3 = FactoryGirl.create_pair(:post)
+  posts4 = FactoryGirl.build_pair(:post)
+end
+    '''}
+    let(:post_steps_rewritten_content) {'''
+test "post" do
+  post1 = create(:post)
+  post2 = build(:post)
+  post_attributes = attributes_for(:post)
+  post3 = build_stubbed(:post)
+  posts1 = create_list(:post, 2)
+  posts2 = build_list(:post, 2)
+  posts3 = create_pair(:post)
+  posts4 = build_pair(:post)
+end
+    '''}
+
+    it 'process' do
+      FileUtils.mkdir_p 'features/support'
+      FileUtils.mkdir_p 'features/step_definitions'
+      File.write 'features/support/env.rb', env_content
+      File.write 'features/step_definitions/post_steps.rb', post_steps_content
+      @rewriter.process
+      expect(File.read 'features/support/env.rb').to eq env_rewritten_content
+      expect(File.read 'features/step_definitions/post_steps.rb').to eq post_steps_rewritten_content
+    end
+  end
+end

data/spec/synvert/snippets/rails/upgrade_3_2_to_4_0_spec.rb

@@ -0,0 +1,268 @@
+require 'spec_helper'
+
+describe 'Upgrade rails from 3.2 to 4.0' do
+  before do
+    Synvert::Configuration.instance.set :path, '.'
+    rewriter_path = File.join(File.dirname(__FILE__), '../../../../lib/synvert/snippets/rails/upgrade_3_2_to_4_0.rb')
+    @rewriter = eval(File.read(rewriter_path))
+    allow_any_instance_of(Synvert::Rewriter::GemSpec).to receive(:match?).and_return(true)
+    expect(SecureRandom).to receive(:hex).with(64).and_return("bf4f3f46924ecd9adcb6515681c78144545bba454420973a274d7021ff946b8ef043a95ca1a15a9d1b75f9fbdf85d1a3afaf22f4e3c2f3f78e24a0a188b581df")
+  end
+
+  describe 'with fakefs', fakefs: true do
+    let(:application_content) {'''
+if defined?(Bundler)
+  Bundler.require(*Rails.groups(:assets => %w(development test)))
+end
+module Synvert
+  class Application < Rails::Application
+    config.active_record.whitelist_attributes = true
+    config.middleware.insert_before(Rack::Lock, ActionDispatch::BestStandardsSupport)
+  end
+end
+    '''}
+    let(:application_rewritten_content) {'''
+Bundler.require(:default, Rails.env)
+module Synvert
+  class Application < Rails::Application
+  end
+end
+    '''}
+    let(:production_content) {'''
+Synvert::Application.configure do
+  config.cache_classes = true
+  config.active_record.identity_map = true
+  config.action_dispatch.best_standards_support = :builtin
+
+  ActionController::Base.page_cache_extension = "html"
+end
+    '''}
+    let(:production_rewritten_content) {'''
+Synvert::Application.configure do
+  config.eager_load = true
+  config.cache_classes = true
+
+  ActionController::Base.default_static_extension = "html"
+end
+    '''}
+    let(:development_content) {'''
+Synvert::Application.configure do
+  config.cache_classes = false
+end
+    '''}
+    let(:development_rewritten_content) {'''
+Synvert::Application.configure do
+  config.eager_load = false
+  config.cache_classes = false
+end
+    '''}
+    let(:test_content) {'''
+Synvert::Application.configure do
+  config.cache_classes = false
+end
+    '''}
+    let(:test_rewritten_content) {'''
+Synvert::Application.configure do
+  config.eager_load = false
+  config.cache_classes = false
+end
+    '''}
+    let(:wrap_parameters_content) {'''
+ActiveSupport.on_load(:action_controller) do
+  wrap_parameters format: [:json]
+end
+ActiveSupport.on_load(:active_record) do
+  self.include_root_in_json = false
+end
+    '''}
+    let(:wrap_parameters_rewritten_content) {'''
+ActiveSupport.on_load(:action_controller) do
+  wrap_parameters format: [:json]
+end
+    '''}
+    let(:secret_token_content) {'''
+Synvert::Application.config.secret_token = "0447aa931d42918bfb934750bb78257088fb671186b5d1b6f9fddf126fc8a14d34f1d045cefab3900751c3da121a8dd929aec9bafe975f1cabb48232b4002e4e"
+    '''}
+    let(:secret_token_rewritten_content) {'''
+Synvert::Application.config.secret_token = "0447aa931d42918bfb934750bb78257088fb671186b5d1b6f9fddf126fc8a14d34f1d045cefab3900751c3da121a8dd929aec9bafe975f1cabb48232b4002e4e"
+Synvert::Application.config.secret_key_base = "bf4f3f46924ecd9adcb6515681c78144545bba454420973a274d7021ff946b8ef043a95ca1a15a9d1b75f9fbdf85d1a3afaf22f4e3c2f3f78e24a0a188b581df"
+    '''}
+    let(:routes_content) {"""
+Synvert::Application.routes.draw do
+  get Rack::Utils.escape('こんにちは'), controller: 'welcome', action: 'index'
+  match '/' => 'root#index'
+  match 'new', to: 'episodes#new'
+end
+    """}
+    let(:routes_rewritten_content) {"""
+Synvert::Application.routes.draw do
+  get 'こんにちは', controller: 'welcome', action: 'index'
+  get '/' => 'root#index'
+  get 'new', to: 'episodes#new'
+end
+    """}
+    let(:post_model_content) {'''
+class Post < ActiveRecord::Base
+  scope :active, where(active: true)
+
+  attr_accessible :title, :description
+
+  def serialized_attrs
+    self.serialized_attributes
+  end
+
+  def active_users_by_email(email)
+    User.find_all_by_email_and_active(email, true)
+  end
+
+  def first_active_user_by_email(email)
+    User.find_by_email_and_active(email, true)
+  end
+
+  def last_active_user_by_email(email)
+    User.find_last_by_email_and_active(email, true)
+  end
+
+  def scoped_active_user_by_email(email)
+    User.scoped_by_email_and_active(email, true)
+  end
+end
+    '''}
+    let(:post_model_rewritten_content) {'''
+class Post < ActiveRecord::Base
+  scope :active, -> { where(active: true) }
+
+  def serialized_attrs
+    self.class.serialized_attributes
+  end
+
+  def active_users_by_email(email)
+    User.where(email: email, active: true)
+  end
+
+  def first_active_user_by_email(email)
+    User.where(email: email, active: true).first
+  end
+
+  def last_active_user_by_email(email)
+    User.where(email: email, active: true).last
+  end
+
+  def scoped_active_user_by_email(email)
+    User.where(email: email, active: true)
+  end
+end
+    '''}
+    let(:users_controller_content) {'''
+class UsersController < ApplicationController
+  def new
+    @user = User.find_or_initialize_by_login_and_email(params[:user][:login], params[:user][:email])
+  end
+
+  def create
+    @user = User.find_or_create_by_login_and_email(params[:user][:login], params[:user][:email])
+  end
+end
+    '''}
+    let(:users_controller_rewritten_content) {'''
+class UsersController < ApplicationController
+  def new
+    @user = User.find_or_initialize_by(login: params[:user][:login], email: params[:user][:email])
+  end
+
+  def create
+    @user = User.find_or_create_by(login: params[:user][:login], email: params[:user][:email])
+  end
+end
+    '''}
+    let(:posts_controller_content) {'''
+class PostsController < ApplicationController
+  def update
+    @post = Post.find(params[:id])
+    if @post.update_attributes params[:post]
+      redirect_to post_path(@post)
+    else
+      render :action => :edit
+    end
+  end
+end
+    '''}
+    let(:posts_controller_rewritten_content) {'''
+class PostsController < ApplicationController
+  def update
+    @post = Post.find(params[:id])
+    if @post.update_attributes post_params
+      redirect_to post_path(@post)
+    else
+      render :action => :edit
+    end
+  end
+
+  def post_params
+    params.require(:post).permit(:title, :description)
+  end
+end
+    '''}
+    let(:post_test_content) {'''
+require "test_helper"
+
+class PostTest < ActiveRecord::TestCase
+end
+    '''}
+    let(:post_test_rewritten_content) {'''
+require "test_helper"
+
+class PostTest < ActiveSupport::TestCase
+end
+    '''}
+    let(:test_helper_content) {'''
+class ActiveSupport::TestCase
+  def constants
+    [ActionController::Integration, ActionController::IntegrationTest, ActionController::PerformanceTest, ActionController::AbstractRequest,
+    ActionController::Request, ActionController::AbstractResponse, ActionController::Response, ActionController::Routing]
+  end
+end
+    '''}
+    let(:test_helper_rewritten_content) {'''
+class ActiveSupport::TestCase
+  def constants
+    [ActionDispatch::Integration, ActionDispatch::IntegrationTest, ActionDispatch::PerformanceTest, ActionDispatch::Request,
+    ActionDispatch::Request, ActionDispatch::Response, ActionDispatch::Response, ActionDispatch::Routing]
+  end
+end
+    '''}
+
+    it 'process' do
+      FileUtils.mkdir_p 'config/environments'
+      FileUtils.mkdir_p 'config/initializers'
+      FileUtils.mkdir_p 'app/models'
+      FileUtils.mkdir_p 'app/controllers'
+      FileUtils.mkdir_p 'test/unit'
+      File.write 'config/application.rb', application_content
+      File.write 'config/environments/production.rb', production_content
+      File.write 'config/environments/development.rb', development_content
+      File.write 'config/environments/test.rb', test_content
+      File.write 'config/initializers/wrap_parameters.rb', wrap_parameters_content
+      File.write 'config/initializers/secret_token.rb', secret_token_content
+      File.write 'config/routes.rb', routes_content
+      File.write 'app/models/post.rb', post_model_content
+      File.write 'app/controllers/users_controller.rb', users_controller_content
+      File.write 'app/controllers/posts_controller.rb', posts_controller_content
+      File.write 'test/unit/post_test.rb', post_test_content
+      File.write 'test/test_helper.rb', test_helper_content
+      @rewriter.process
+      expect(File.read 'config/application.rb').to eq application_rewritten_content
+      expect(File.read 'config/environments/production.rb').to eq production_rewritten_content
+      expect(File.read 'config/environments/development.rb').to eq development_rewritten_content
+      expect(File.read 'config/environments/test.rb').to eq test_rewritten_content
+      expect(File.read 'config/initializers/wrap_parameters.rb').to eq wrap_parameters_rewritten_content
+      expect(File.read 'config/initializers/secret_token.rb').to eq secret_token_rewritten_content
+      expect(File.read 'config/routes.rb').to eq routes_rewritten_content
+      expect(File.read 'app/models/post.rb').to eq post_model_rewritten_content
+      expect(File.read 'app/controllers/users_controller.rb').to eq users_controller_rewritten_content
+      expect(File.read 'app/controllers/posts_controller.rb').to eq posts_controller_rewritten_content
+      expect(File.read 'test/unit/post_test.rb').to eq post_test_rewritten_content
+      expect(File.read 'test/test_helper.rb').to eq test_helper_rewritten_content
+    end
+  end
+end

data/synvert.gemspec

@@ -19,8 +19,10 @@ Gem::Specification.new do |spec|
   spec.require_paths = ["lib"]
 
   spec.add_runtime_dependency "parser"
+  spec.add_runtime_dependency "activesupport"
 
   spec.add_development_dependency "bundler", "~> 1.3"
   spec.add_development_dependency "rake"
   spec.add_development_dependency "rspec"
+  spec.add_development_dependency "fakefs"
 end