RubyGems - ponytail - Versions diffs - 0.0.2 → 0.0.3 - Mend

ponytail 0.0.2 → 0.0.3

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (25) hide show

checksums.yaml +4 -4
data/.coveralls.yml +1 -0
data/.travis.yml +1 -0
data/Gemfile +4 -0
data/README.md +3 -0
data/app/assets/javascripts/ponytail/migrations.js +13 -1
data/app/assets/stylesheets/ponytail/application.css +30 -0
data/app/controllers/ponytail/migrations_controller.rb +3 -1
data/app/views/layouts/ponytail/application.html.erb +1 -3
data/app/views/ponytail/migrations/_new_errors.html.erb +10 -0
data/app/views/ponytail/migrations/_notice.html.erb +6 -1
data/app/views/ponytail/migrations/_table.html.erb +7 -0
data/app/views/ponytail/migrations/index.html.erb +10 -8
data/app/views/ponytail/migrations/new.html.erb +8 -2
data/lib/ponytail/migration.rb +7 -3
data/lib/ponytail/schema.rb +22 -0
data/lib/ponytail/version.rb +1 -1
data/lib/ponytail.rb +1 -0
data/spec/controllers/migrations_controller_spec.rb +92 -0
data/spec/ponytail/migration_spec.rb +71 -0
data/spec/ponytail/schema_spec.rb +23 -0
data/spec/rails/mapper_spec.rb +17 -0
data/spec/spec_helper.rb +20 -0
metadata +14 -4
data/spec/migration_spec.rb +0 -21

checksums.yaml CHANGED Viewed

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 3abe40fddc7790c3c4df41e86756bff646adb10a
-  data.tar.gz: 0b5606b7198008fee75171c6c62926b6fc2bbc71
+  metadata.gz: 78aeabe758e879167178c1b3ab68880f43561b76
+  data.tar.gz: ba0bc6eff510a36202b96d31c4d8a5ad3cf37462
 SHA512:
-  metadata.gz: ed5138f2740020d332fcacd9ca100046b01fb7573face47cb000768db84d307d5cdb9ab08a0d7cf97fc47a680b39ba072bae20aefab98c97998404ae689155a7
-  data.tar.gz: dae7e9101eb822be5f73c9a9917bdccca4648eaf5f573b7e587aea6182e07180141b217d87a34fc0de373a668a57c7359690c300568f1ec919395278794ad050
+  metadata.gz: 69f19add1e33a3f1ea7a073be5f41fc672ca3db2ba1566368217d2b758812c6bf3c6732c6e045ac4b8e7a68e5ee03f5a3b1b4a933efecd9656988f06bc81a208
+  data.tar.gz: 54ae11479d174006e6bcd0bcaa7273167bea066cbf3e650f9a8aeee66602cb736bb9000e8367e86a94b1cdb0fc419b3251d429149f14349cc7b7cbbc31516a9a

data/.coveralls.yml ADDED Viewed

	@@ -0,0 +1 @@
1	+ service_name: travis-ci

data/.travis.yml CHANGED Viewed

@@ -5,5 +5,6 @@ branches:
   only:
     - master
     - develop
+    - feature/travis
 notifications:
   email: false

data/Gemfile CHANGED Viewed

@@ -2,3 +2,7 @@ source 'https://rubygems.org'
 # Specify your gem's dependencies in ponytail.gemspec
 gemspec
+group :test do
+  gem 'coveralls', require: false
+end

data/README.md CHANGED Viewed

@@ -1,6 +1,9 @@
 # Ponytail
+[![Gem Version](https://badge.fury.io/rb/ponytail.png)](http://badge.fury.io/rb/ponytail)
 [![Build Status](https://travis-ci.org/sinsoku/ponytail.png?branch=develop)](https://travis-ci.org/sinsoku/ponytail)
+[![Coverage Status](https://coveralls.io/repos/sinsoku/ponytail/badge.png?branch=develop)](https://coveralls.io/r/sinsoku/ponytail?branch=develop)
+[![Code Climate](https://codeclimate.com/github/sinsoku/ponytail.png)](https://codeclimate.com/github/sinsoku/ponytail)
 Ponytail is a Rails engine that shows the migrations.

data/app/assets/javascripts/ponytail/migrations.js CHANGED Viewed

@@ -11,10 +11,22 @@ function onClick(elems, func) {
   });
 }
-function setupMigrations() {
+function toggleMigrationRawContent() {
   var elems = document.querySelectorAll(".pt_migration .pt_header");
   onClick(elems, function() {
     var content = this.nextElementSibling;
     content.style.display = content.style.display === "" ? "block" : "";
   });
 }
+function closeNotice() {
+  var elems = document.querySelectorAll(".pt_close_notice");
+  onClick(elems, function() {
+    this.parentElement.style.display = "none";
+  });
+}
+function setupMigrations() {
+  toggleMigrationRawContent();
+  closeNotice();
+}

data/app/assets/stylesheets/ponytail/application.css CHANGED Viewed

@@ -1,3 +1,11 @@
+/* index */
+.pt_index {
+  display: -moz-box;
+  display: -webkit-box;
+  -moz-box-orient: vertical;
+  -webkit-box-orient: vertical;
+}
 .pt_actions {
   margin: 5px;
 }
@@ -7,6 +15,7 @@
   margin: 0 2px;
 }
+/* migration */
 .pt_migration .pt_name {
   font-size: large;
   font-weight: bold;
@@ -15,3 +24,24 @@
 .pt_migration .pt_raw_content {
   display: none;
 }
+/* notice */
+.pt_notice {
+  display: -moz-box;
+  display: -webkit-box;
+  border-radius: 5px;
+  margin-bottom: 5px;
+  padding: 5px;
+  background: limegreen;
+  color: white;
+}
+.pt_notice_message {
+  -moz-box-flex: 1;
+  -webkit-box-flex: 1;
+}
+/* errors */
+.pt_new_errors ul {
+  margin: 3px 0;
+}

data/app/controllers/ponytail/migrations_controller.rb CHANGED Viewed

@@ -9,6 +9,7 @@ module Ponytail
     def new
       @migration = Migration.new
+      @schema = Schema.new
     end
     def create
@@ -17,7 +18,8 @@ module Ponytail
       if @migration.save
         redirect_to :migrations, notice: 'Migration was successfully created.'
       else
-        render action: 'new'
+        @schema = Schema.new
+        render action: :new
       end
     end

data/app/views/layouts/ponytail/application.html.erb CHANGED Viewed

@@ -7,8 +7,6 @@
   <%= csrf_meta_tags %>
 </head>
 <body>
-<%= yield %>
+  <%= yield %>
 </body>
 </html>

data/app/views/ponytail/migrations/_new_errors.html.erb ADDED Viewed

@@ -0,0 +1,10 @@
+<% if migration.errors.any? %>
+  <div class="pt_new_errors">
+    <span><%= pluralize(migration.errors.count, "error") %></span>
+    <ul>
+      <% migration.errors.full_messages.each do |msg| %>
+        <li><%= msg %></li>
+      <% end %>
+    </ul>
+  </div>
+<% end %>

data/app/views/ponytail/migrations/_notice.html.erb CHANGED Viewed

@@ -1 +1,6 @@
-<span class="pt_notice"><%= notice %></span>
+<% if notice %>
+  <div class="pt_notice">
+    <div class="pt_notice_message"><%= notice %></div>
+    <div class="pt_close_notice"><a href="#">x</a></div>
+  </div>
+<% end %>

data/app/views/ponytail/migrations/_table.html.erb ADDED Viewed

@@ -0,0 +1,7 @@
+<div class="pt_table"><%= table.name %></div>
+<div>
+  <% table.columns.each do |column| %>
+    <span>t.<%= column.type %></span>
+    <span><%= column.name %></span>
+<% end %>
+</div>

data/app/views/ponytail/migrations/index.html.erb CHANGED Viewed

@@ -1,13 +1,15 @@
-<%= render 'notice' %>
+<div class="pt_index">
+  <%= render 'notice' %>
-<% @migrations.each do |migration| %>
-  <%= render 'migration', migration: migration, current: @current_version %>
-<% end %>
+  <% @migrations.each do |migration| %>
+    <%= render 'migration', migration: migration, current: @current_version %>
+  <% end %>
-<div class="pt_actions">
-  <%= button_to 'New', new_migration_path, class: 'pt_new', method: :get, disabled: Ponytail::Migration.check_pending? %>
-  <%= button_to 'Migrate', migrate_migrations_path, class: 'pt_migrate', disabled: Ponytail::Migration.check_pending? %>
-  <%= button_to 'Rollback', rollback_migrations_path, class: 'pt_rallback', disabled: Ponytail::Migration.check_pending? %>
+  <div class="pt_actions">
+    <%= button_to 'New', new_migration_path, class: 'pt_new', method: :get, disabled: Ponytail::Migration.check_pending? %>
+    <%= button_to 'Migrate', migrate_migrations_path, class: 'pt_migrate', disabled: Ponytail::Migration.check_pending? %>
+    <%= button_to 'Rollback', rollback_migrations_path, class: 'pt_rallback', disabled: Ponytail::Migration.check_pending? %>
+  </div>
 </div>
 <script type='text/javascript'>

data/app/views/ponytail/migrations/new.html.erb CHANGED Viewed

@@ -1,6 +1,6 @@
-<%= render 'notice' %>
 <%= form_for @migration, url: migrations_path do |f| %>
+  <%= render 'new_errors', migration: @migration %>
   <%= f.label :name %>
   <%= f.text_field :name %>
@@ -10,3 +10,9 @@
   <%= f.submit 'Submit' %>
 <% end %>
+<%= button_to 'Cancel', migrations_path, method: :get %>
+<% @schema.tables.each do |t| %>
+  <%= render 'table', table: t %>
+<% end %>

data/lib/ponytail/migration.rb CHANGED Viewed

@@ -3,9 +3,12 @@ module Ponytail
     include ActiveModel::Model
     attr_accessor :name, :filename, :version, :raw_content
+    validates :name, presence: true
+    validates :raw_content, presence: true
     class << self
       def check_pending?
-        Rails.application.config.middleware.include?(ActiveRecord::Migration::CheckPending)
+        ActiveRecord::VERSION::MAJOR >= 4 && Rails.application.config.middleware.include?(ActiveRecord::Migration::CheckPending)
       end
       def all
@@ -34,8 +37,9 @@ module Ponytail
     def save
       if valid?
-        next_migration_filename = "#{Migration.migrations_path}/#{Migration.next_version}_#{name.underscore}.rb"
-        open(next_migration_filename, 'w').write(raw_content)
+        next_filename = "#{Migration.migrations_path}/#{Migration.next_version}_#{name.underscore}.rb"
+        open(next_filename, 'w').write(raw_content)
+        true
       else
         false
       end

data/lib/ponytail/schema.rb ADDED Viewed

@@ -0,0 +1,22 @@
+module Ponytail
+  class Schema
+    def tables
+      ActiveRecord::Base.connection.tables.sort.map { |t| Table.new(t) }
+    end
+  end
+  class Table
+    attr_reader :name
+    def initialize(name=nil)
+      @name = name
+    end
+    def columns
+      @columns ||= ActiveRecord::Base.connection.columns(name)
+    end
+    def ==(other)
+      name == other.name
+    end
+  end
+end

data/lib/ponytail/version.rb CHANGED Viewed

@@ -1,3 +1,3 @@
 module Ponytail
-  VERSION = "0.0.2"
+  VERSION = "0.0.3"
 end

data/lib/ponytail.rb CHANGED Viewed

@@ -1,3 +1,4 @@
 require 'ponytail/engine'
 require 'ponytail/migration'
+require 'ponytail/schema'
 require 'rails/mapper'

data/spec/controllers/migrations_controller_spec.rb ADDED Viewed

@@ -0,0 +1,92 @@
+require 'spec_helper'
+module Ponytail
+  describe MigrationsController do
+    let(:controller) { MigrationsController.new }
+    describe "#index" do
+      before do
+        Migration.stub(:all)
+        Migration.stub(:current_version)
+        controller.index
+      end
+      it { expect(controller).to be_instance_variable_defined(:@migrations) }
+      it { expect(controller).to be_instance_variable_defined(:@current_version) }
+    end
+    describe "#new" do
+       before do
+         Migration.stub(:new)
+         controller.new
+       end
+       it { expect(controller).to be_instance_variable_defined(:@migration) }
+    end
+    describe "#create" do
+      context "save was succeed." do
+        before do
+          @migration = Migration.new
+          @migration.stub(save: true)
+          Migration.stub(new: @migration)
+          controller.stub(migraion_params: {})
+          controller.should_receive(:redirect_to).with(:migrations, notice: 'Migration was successfully created.')
+          controller.create
+        end
+        it "redirect to migrations, and notice" do end
+        it { expect(controller).to be_instance_variable_defined(:@migration) }
+      end
+      context "save was failed." do
+        before do
+          @migration = Migration.new
+          @migration.stub(save: false)
+          Migration.stub(new: @migration)
+          controller.stub(migraion_params: {})
+          controller.should_receive(:render).with(action: :new)
+          controller.create
+        end
+        it "redirect to migrations, and notice" do end
+        it { expect(controller).to be_instance_variable_defined(:@migration) }
+      end
+    end
+    describe "#migrate" do
+      context "migrate was succeed." do
+        before do
+          Migration.stub(migrate: true)
+          controller.should_receive(:redirect_to).with(:migrations, notice: 'Migrate was succeed.')
+          controller.migrate
+        end
+        it "redirect to migrations, and notice" do end
+      end
+      context "migrate was failed." do
+        before do
+          Migration.stub(migrate: false)
+          controller.should_receive(:redirect_to).with(:migrations, notice: 'Migrate was failed.')
+          controller.migrate
+        end
+        it "redirect to migrations, and notice" do end
+      end
+    end
+    describe "#rollback" do
+      context "rollback was succeed." do
+        before do
+          Migration.stub(rollback: true)
+          controller.should_receive(:redirect_to).with(:migrations, notice: 'Rollback was succeed.')
+          controller.rollback
+        end
+        it "redirect to migrations, and notice" do end
+      end
+      context "rollback was failed." do
+        before do
+          Migration.stub(rollback: false)
+          controller.should_receive(:redirect_to).with(:migrations, notice: 'Rollback was failed.')
+          controller.rollback
+        end
+        it "redirect to migrations, and notice" do end
+      end
+    end
+  end
+end

data/spec/ponytail/migration_spec.rb ADDED Viewed

@@ -0,0 +1,71 @@
+require 'spec_helper'
+module Ponytail
+  describe Migration do
+    describe ".all" do
+      before do
+        proxy = Struct.new(:proxy, :name, :filename, :version)
+        ActiveRecord::Migrator.stub(migrations: [proxy.new('CreateUsers', '001_create_users', 1)])
+      end
+      subject { Migration.all }
+      its(:first) { should be_a_kind_of Migration }
+    end
+    describe ".migrate" do
+      before do
+        Migration.stub(:migrations_paths)
+        ActiveRecord::Migrator.should_receive(:migrate)
+        Migration.migrate
+      end
+      it "" do end
+    end
+    describe ".rollback" do
+      before do
+        Migration.stub(:migrations_paths)
+        ActiveRecord::Migrator.should_receive(:rollback)
+        Migration.rollback
+      end
+      it "" do end
+    end
+    describe ".next_version" do
+      before do
+        migration = Migration.new(version: 1)
+        Migration.stub(all: [migration])
+        ActiveRecord::Migration.should_receive(:next_migration_number)
+        Migration.next_version
+      end
+      it "" do end
+    end
+    describe "#initialize" do
+      subject { Migration.new(name: 'CreateUsers', filename: '001_create_users.rb', version: 1) }
+      its(:name) { should eq 'CreateUsers' }
+      its(:filename) { should eq '001_create_users.rb' }
+      its(:version) { should eq 1 }
+    end
+    describe "#save" do
+      context "migration is valid" do
+        before do
+          Migration.stub(migrations_path: 'db/migrate', next_version: '001')
+          @migration = Migration.new(name: 'CreateBooks')
+          @migration.stub(valid?: true)
+          @migration.stub_chain(:open, :write)
+          @migration.should_receive(:open).with('db/migrate/001_create_books.rb', 'w')
+        end
+        subject { @migration.save }
+        it { should be_true }
+      end
+      context "migration is invalid" do
+        before do
+          @migration = Migration.new(name: 'CreateBooks')
+          @migration.stub(valid?: false)
+        end
+        subject { @migration.save }
+        it { should be_false }
+      end
+    end
+  end
+end

data/spec/ponytail/schema_spec.rb ADDED Viewed

@@ -0,0 +1,23 @@
+require 'spec_helper'
+module Ponytail
+  describe Schema do
+    let(:schema) { Schema.new }
+    let(:table_names) { [:users, :books] }
+    before do
+      ActiveRecord::Base.stub_chain(:connection, :tables).and_return(table_names)
+    end
+    it { expect(schema.tables).to include(Table.new(:users)) }
+  end
+  describe Table do
+    describe "#columns" do
+      let(:table) { Table.new }
+      let(:expected_columns) { [] }
+      before do
+        ActiveRecord::Base.stub_chain(:connection, :columns).and_return(expected_columns)
+      end
+      it { expect(table.columns).to eq expected_columns }
+    end
+  end
+end

data/spec/rails/mapper_spec.rb ADDED Viewed

@@ -0,0 +1,17 @@
+module ActionDispatch::Routing
+  describe Mapper do
+    describe "#mount_ponytail" do
+      let(:mapper) do
+        dummy = Class.new { def resources_path_names; end }
+        Mapper.new(dummy.new)
+      end
+      it "should receive resources" do
+        mapper.should_receive(:resources) { |*args, &block| block.call }
+        mapper.should_receive(:collection) { |*args, &block| block.call }
+        mapper.should_receive(:post).twice
+        mapper.mount_ponytail
+      end
+    end
+  end
+end

data/spec/spec_helper.rb CHANGED Viewed

@@ -1,4 +1,24 @@
+if ENV["TRAVIS"]
+  require 'simplecov'
+  require 'coveralls'
+  SimpleCov.formatter = SimpleCov::Formatter::MultiFormatter[
+    SimpleCov::Formatter::HTMLFormatter,
+    Coveralls::SimpleCov::Formatter
+  ]
+  SimpleCov.start do
+    add_group 'Controllers', 'app/controllers'
+    add_group 'Lib', 'lib'
+    add_filter 'spec'
+    add_filter 'vendor'
+  end
+end
 $LOAD_PATH.unshift File.expand_path('../../lib', __FILE__)
 require 'active_record'
+require 'action_controller'
 require 'rails'
 require 'ponytail'
+app_dir = File.expand_path('../../app', __FILE__)
+Dir.glob("#{app_dir}/controllers/**/*.rb").each { |f| require f }

metadata CHANGED Viewed

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ponytail
 version: !ruby/object:Gem::Version
-  version: 0.0.2
+  version: 0.0.3
 platform: ruby
 authors:
 - sinsoku
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2013-12-04 00:00:00.000000000 Z
+date: 2013-12-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -73,6 +73,7 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- .coveralls.yml
 - .gitignore
 - .rspec
 - .travis.yml
@@ -86,18 +87,24 @@ files:
 - app/controllers/ponytail/migrations_controller.rb
 - app/views/layouts/ponytail/application.html.erb
 - app/views/ponytail/migrations/_migration.html.erb
+- app/views/ponytail/migrations/_new_errors.html.erb
 - app/views/ponytail/migrations/_notice.html.erb
+- app/views/ponytail/migrations/_table.html.erb
 - app/views/ponytail/migrations/index.html.erb
 - app/views/ponytail/migrations/new.html.erb
 - config/routes.rb
 - lib/ponytail.rb
 - lib/ponytail/engine.rb
 - lib/ponytail/migration.rb
+- lib/ponytail/schema.rb
 - lib/ponytail/version.rb
 - lib/rails/mapper.rb
 - ponytail.gemspec
-- spec/migration_spec.rb
+- spec/controllers/migrations_controller_spec.rb
+- spec/ponytail/migration_spec.rb
+- spec/ponytail/schema_spec.rb
 - spec/ponytail_spec.rb
+- spec/rails/mapper_spec.rb
 - spec/spec_helper.rb
 homepage: https://github.com/sinsoku/ponytail
 licenses:
@@ -124,6 +131,9 @@ signing_key:
 specification_version: 4
 summary: Ponytail is a Rails engine that shows the migrations.
 test_files:
-- spec/migration_spec.rb
+- spec/controllers/migrations_controller_spec.rb
+- spec/ponytail/migration_spec.rb
+- spec/ponytail/schema_spec.rb
 - spec/ponytail_spec.rb
+- spec/rails/mapper_spec.rb
 - spec/spec_helper.rb

data/spec/migration_spec.rb DELETED Viewed

@@ -1,21 +0,0 @@
-require 'spec_helper'
-module Ponytail
-  describe Migration do
-    describe ".all" do
-      before do
-        proxy = Struct.new(:proxy, :name, :filename, :version)
-        ActiveRecord::Migrator.stub(migrations: [proxy.new('CreateUsers', '001_create_users', 1)])
-      end
-      subject { Migration.all }
-      its(:first) { should be_a_kind_of Migration }
-    end
-    describe "#initialize" do
-      subject { Migration.new(name: 'CreateUsers', filename: '001_create_users.rb', version: 1) }
-      its(:name) { should eq 'CreateUsers' }
-      its(:filename) { should eq '001_create_users.rb' }
-      its(:version) { should eq 1 }
-    end
-  end
-end