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

ponytail 0.0.2 → 0.0.3

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