db_nazi 0.0.1

data/.gitignore

@@ -0,0 +1 @@
+/Gemfile.lock

data/.travis.yml

@@ -0,0 +1,9 @@
+language: ruby
+bundler_args: --without dev
+rvm:
+  - 1.9.2
+  - 1.9.3
+  - jruby-19mode
+  - rbx-19mode
+  - ruby-head
+  - jruby-head

data/CHANGELOG

@@ -0,0 +1,3 @@
+== 0.0.1 2012-06-23
+
+ * Hi.

data/Gemfile

@@ -0,0 +1,2 @@
+source :rubygems
+gemspec

data/LICENSE

@@ -0,0 +1,20 @@
+Copyright (c) George Ogata
+
+Permission is hereby granted, free of charge, to any person obtaining
+a copy of this software and associated documentation files (the
+"Software"), to deal in the Software without restriction, including
+without limitation the rights to use, copy, modify, merge, publish,
+distribute, sublicense, and/or sell copies of the Software, and to
+permit persons to whom the Software is furnished to do so, subject to
+the following conditions:
+
+The above copyright notice and this permission notice shall be
+included in all copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE
+LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
+OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
+WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

data/README.markdown

@@ -0,0 +1,87 @@
+## DB Nazi
+
+Encourages good DB practices in Active Record migrations.
+
+## What?
+
+Active Record makes schema changes wonderfully easy, but due to some unfortunate
+defaults, it can be easy to forget to adhere to some basic database best
+practices, such as restricting columns to be non-nullable, or setting meaningful
+varchar limits. DB Nazi forces you to be explicit about these things so you
+can't simply forget.
+
+It might take another 2 seconds to type `null: true`, but it may save you hours
+resolving integrity issues down the line!
+
+### Nullability
+
+If `DBNazi.require_nullability` is set to `true` (the default), you must specify
+a `:null` option for all columns.
+
+    add_column :users, :awesome, :boolean              # raises DBNazi::NullabilityRequired
+    add_column :users, :awesome, :boolean, null: true  # ok
+
+### Varchar limits
+
+If `DBNazi.require_varchar_limits` is set to `true` (the default), you must
+specify a `:limit` option for all `:string` columns.
+
+    add_column :users, :name, :string              # raises DBNazi::VarcharLimitRequired
+    add_column :users, :name, :string, limit: 100  # ok
+
+### Index uniqueness
+
+If `DBNazi.require_index_uniqueness` is set the `true` (the default), you must
+specify a `:unique` option for all indexes.
+
+    add_index :users, :email                 # raises DBNazi::IndexUniquenessRequired
+    add_index :users, :email, unique: false  # ok
+
+## Usage
+
+Since this tool is about enforcing developer discipline, I suggest including
+this only in the `:development` group in your `Gemfile`.
+
+    group :development do
+      gem 'db_nazi'
+    end
+
+If you have an established project, you probably don't want to lay the hard line
+on all your existing migrations. You can do this by specifying a minimum
+migration version in `config/environments/development.rb`:
+
+    DBNazi.from_version = 20120623000000
+
+This means "only be a nazi from migration 20120623000000 onwards."
+
+If you're using a migration written by a 3rd party, such as a generator you're
+using, I recommend editing it to conform to the rules above. After all, perhaps
+the 3rd party forgot a 'NOT NULL' or two.
+
+If you're *really* sure you want to subvert DB Nazi for whatever reason, you may
+do so like this:
+
+    class BeAJerk < ActiveRecord::Migration
+      no_nazi
+      ...
+    end
+
+Or just for a given block like this:
+
+    DBNazi.disable do
+      ...
+    end
+
+But no soup for you!
+
+## Contributing
+
+ * [Bug reports](https://github.com/oggy/db_nazi/issues)
+ * [Source](https://github.com/oggy/db_nazi)
+ * Patches: Fork on Github, send pull request.
+   * Include tests where practical.
+   * Leave the version alone, or bump it in a separate commit.
+
+## Copyright
+
+Copyright (c) George Ogata. See LICENSE for details.

data/Rakefile

@@ -0,0 +1,6 @@
+require 'ritual'
+
+task :ci do
+  sh 'bundle exec testrb test/unit'
+  sh 'bundle exec testrb test/integration'
+end

data/db_nazi.gemspec

@@ -0,0 +1,25 @@
+# -*- encoding: utf-8 -*-
+$:.unshift File.expand_path('lib', File.dirname(__FILE__))
+require 'db_nazi/version'
+
+Gem::Specification.new do |gem|
+  gem.name          = 'db_nazi'
+  gem.version       = DBNazi::VERSION
+  gem.authors       = ['George Ogata']
+  gem.email         = ['george.ogata@gmail.com']
+  gem.license       = 'MIT'
+  gem.description   = ""
+  gem.summary       = "Encourage good DB practices in ActiveRecord migrations."
+  gem.homepage      = ""
+
+  gem.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
+  gem.files         = `git ls-files`.split("\n")
+  gem.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
+
+  gem.add_runtime_dependency 'activerecord', '~> 3.2.6'
+  gem.add_development_dependency 'ritual', '~> 0.4.1'
+  gem.add_development_dependency 'temporaries', '~> 0.2.0'
+  gem.add_development_dependency 'looksee', '~> 0.2.0'
+  gem.add_development_dependency 'debugger', '~> 1.1.3'
+  gem.add_development_dependency 'sqlite3', '~> 1.3.6'
+end

data/lib/db_nazi/abstract_adapter.rb

@@ -0,0 +1,31 @@
+module DBNazi
+  module AbstractAdapter
+    def add_column(table_name, column_name, type, options = {})
+      if DBNazi.enabled?(:require_nullability)
+        options.key?(:null) or
+          raise NullabilityRequired, "[db_nazi] :null parameter required"
+      end
+      if DBNazi.enabled?(:require_varchar_limits)
+        # AR calls #to_sym on type, so do the same here.
+        type.to_sym == :string && !options.key?(:limit) and
+          raise VarcharLimitRequired, "[db_nazi] string column requires :limit parameter"
+      end
+      super
+    end
+
+    def add_index(table_name, column_name, options = {})
+      if DBNazi.enabled?(:require_index_uniqueness)
+        options.key?(:unique) or
+          raise IndexUniquenessRequired, "[db_nazi] :unique parameter required"
+      end
+    end
+
+    def create_table(name, *)
+      if name.to_s == ActiveRecord::Migrator.schema_migrations_table_name.to_s
+        DBNazi.disable { super }
+      else
+        super
+      end
+    end
+  end
+end

data/lib/db_nazi/migration.rb

@@ -0,0 +1,21 @@
+module DBNazi
+  module Migration
+    def self.included(base)
+      base.extend ClassMethods
+    end
+
+    def nazi_disabled?
+      self.class.nazi_disabled?
+    end
+
+    module ClassMethods
+      def no_nazi
+        @nazi_disabled = true
+      end
+
+      def nazi_disabled?
+        @nazi_disabled
+      end
+    end
+  end
+end

data/lib/db_nazi/migration_proxy.rb

@@ -0,0 +1,17 @@
+module DBNazi
+  module MigrationProxy
+    def self.included(base)
+      base.class_eval do
+        alias migrate_without_db_nazi migrate
+        alias migrate migrate_with_db_nazi
+      end
+    end
+
+    def migrate_with_db_nazi(direction)
+      action = DBNazi.enabled_for_migration?(migration, version) ? :enable : :disable
+      DBNazi.send(action) do
+        migrate_without_db_nazi(direction)
+      end
+    end
+  end
+end

data/lib/db_nazi/table_definition.rb

@@ -0,0 +1,23 @@
+module DBNazi
+  module TableDefinition
+    def self.included(base)
+      base.class_eval do
+        alias column_without_db_nazi column
+        alias column column_with_db_nazi
+      end
+    end
+
+    def column_with_db_nazi(name, type, options = {})
+      if DBNazi.enabled?(:require_nullability) && type != :primary_key
+        options.key?(:null) or
+          raise NullabilityRequired, "[db_nazi] :null parameter required"
+      end
+      if DBNazi.enabled?(:require_varchar_limits)
+        # AR calls #to_sym on type, so do the same here.
+        type.to_sym == :string && !options.key?(:limit) and
+          raise VarcharLimitRequired, "[db_nazi] string column requires :limit parameter"
+      end
+      column_without_db_nazi(name, type, options)
+    end
+  end
+end

data/lib/db_nazi/version.rb

@@ -0,0 +1,11 @@
+module DBNazi
+  VERSION = [0, 0, 1]
+
+  class << VERSION
+    include Comparable
+
+    def to_s
+      join('.')
+    end
+  end
+end

data/lib/db_nazi.rb

@@ -0,0 +1,66 @@
+module DBNazi
+  autoload :AbstractAdapter, 'db_nazi/abstract_adapter'
+  autoload :Migration, 'db_nazi/migration'
+  autoload :MigrationProxy, 'db_nazi/migration_proxy'
+  autoload :TableDefinition, 'db_nazi/table_definition'
+  autoload :VERSION, 'db_nazi/version'
+
+  ArgumentError = Class.new(::ArgumentError)
+  NullabilityRequired = Class.new(ArgumentError)
+  VarcharLimitRequired = Class.new(ArgumentError)
+  IndexUniquenessRequired = Class.new(ArgumentError)
+
+  class << self
+    attr_accessor :enabled
+    attr_accessor :from_version
+    attr_accessor :require_nullability
+    attr_accessor :require_varchar_limits
+    attr_accessor :require_index_uniqueness
+
+    def enabled?(setting)
+      @enabled && send(setting)
+    end
+
+    def enable
+      original_enabled = @enabled
+      @enabled = true
+      begin
+        yield
+      ensure
+        @enabled = original_enabled
+      end
+    end
+
+    def disable
+      original_enabled = @enabled
+      @enabled = false
+      begin
+        yield
+      ensure
+        @enabled = original_enabled
+      end
+    end
+
+    def enabled_for_migration?(migration, version)
+      return false if !@enabled
+      return false if migration.nazi_disabled?
+      return false if DBNazi.from_version && DBNazi.from_version > version
+      true
+    end
+
+    def reset
+      self.enabled = true
+      self.from_version = nil
+      self.require_nullability      = true
+      self.require_varchar_limits   = true
+      self.require_index_uniqueness = true
+    end
+  end
+
+  reset
+end
+
+ActiveRecord::ConnectionAdapters::AbstractAdapter.__send__ :include, DBNazi::AbstractAdapter
+ActiveRecord::ConnectionAdapters::TableDefinition.__send__ :include, DBNazi::TableDefinition
+ActiveRecord::Migration.__send__ :include, DBNazi::Migration
+ActiveRecord::MigrationProxy.__send__ :include, DBNazi::MigrationProxy

data/test/database.yml

@@ -0,0 +1,2 @@
+sqlite3:
+  database: ':memory:'

data/test/integration/test_integration.rb

@@ -0,0 +1,76 @@
+require_relative '../test_helper'
+
+describe DBNazi do
+  use_database
+  use_temporary_directory "#{ROOT}/test/tmp"
+
+  # Migrations use Kernel.puts. Lame.
+  out = Class.new { def write(*) end; def flush(*) end }.new
+  use_global_value :stdout, out
+
+  before do
+    DBNazi.reset
+    @original_pwd = Dir.pwd
+    Dir.chdir "#{ROOT}/test/tmp"
+  end
+
+  after do
+    Dir.chdir @original_pwd
+  end
+
+  before do
+    DBNazi.reset
+  end
+
+  it "raises an error when there is a careless migration" do
+    create_bad_migration(1)
+    run_migrations
+    errors.size.must_equal 1
+  end
+
+  it "only applies nazism to selected migrations" do
+    create_bad_migration(1)
+    create_good_migration(2)
+    DBNazi.from_version = 2
+    run_migrations
+    errors.size.must_equal 0
+  end
+
+  def create_bad_migration(version)
+    create_migration(version, :bad)
+  end
+
+  def create_good_migration(version)
+    create_migration(version, :good)
+  end
+
+  def create_migration(version, good_or_bad)
+    write_file "#{version}_test_migration_#{version}.rb", <<-EOS
+      |class TestMigration#{version} < ActiveRecord::Migration
+      |  def up
+      |    create_table 'table_#{version}' do |t|
+      |      t.boolean :test_column #{good_or_bad == :good ? ', null: true' : ''}
+      |    end
+      |  end
+      |end
+    EOS
+  end
+
+  def run_migrations
+    ActiveRecord::Migrator.migrate('.', nil)
+  rescue StandardError => e
+    if e.message.include?('[db_nazi]')
+      errors << e
+    else
+      raise
+    end
+  end
+
+  def errors
+    @errors ||= []
+  end
+
+  def write_file(name, content)
+    open(name, 'w') { |f| f.print content.gsub(/^ *\|/, '') }
+  end
+end

data/test/test_helper.rb

@@ -0,0 +1,46 @@
+$:.unshift File.expand_path('../lib', File.dirname(__FILE__))
+ROOT = File.expand_path('..', File.dirname(__FILE__))
+
+require 'minitest/spec'
+require 'yaml'
+require 'active_record'
+require 'temporaries'
+require 'debugger'
+require 'db_nazi'
+
+ADAPTER = ENV['DBNAZI_ADAPTER'] || 'sqlite3'
+CONNECTION = YAML.load_file("#{ROOT}/test/database.yml")[ADAPTER].merge(adapter: ADAPTER)
+ActiveRecord::Base.establish_connection(CONNECTION)
+
+MiniTest::Spec.class_eval do
+  def recreate_database
+    drop_database
+    case ADAPTER
+    when 'sqlite3'
+      ActiveRecord::Base.establish_connection(CONNECTION)
+    when 'mysql2', 'postgresql'
+      ActiveRecord::Base.connection.create_database 'db_nazi_test'
+    else
+      raise "can't create database for #{ADAPTER}"
+    end
+  end
+
+  def drop_database
+    case ADAPTER
+    when 'sqlite3'
+    when 'mysql2', 'postgresql'
+      ActiveRecord::Base.connection.drop_database 'db_nazi_test'
+    else
+      raise "can't drop database for #{ADAPTER}"
+    end
+  end
+
+  def connection
+    ActiveRecord::Base.connection
+  end
+
+  def self.use_database
+    before { recreate_database }
+    after { drop_database }
+  end
+end

data/test/unit/db_nazi/test_abstract_adapter.rb

@@ -0,0 +1,99 @@
+require_relative '../../test_helper'
+
+describe DBNazi::AbstractAdapter do
+  use_database
+
+  before do
+    DBNazi.reset
+    connection.create_table 'test_table'
+  end
+
+  describe "nullability" do
+    describe "when it is required" do
+      use_attribute_value DBNazi, :require_nullability, true
+
+      it "raises a DBNazi::NullabilityRequired if :null is not specified when adding a column" do
+        lambda do
+          connection.add_column 'test_table', 'test_column', :boolean
+        end.must_raise(DBNazi::NullabilityRequired)
+      end
+
+      it "does not raise a DBNazi::NullabilityRequired if :null is true when adding a column" do
+        connection.add_column 'test_table', 'test_column', :boolean, null: true
+      end
+
+      it "does not raise a DBNazi::NullabilityRequired if :null is false when adding a column" do
+        connection.add_column 'test_table', 'test_column', :boolean, null: false, default: false
+      end
+    end
+
+    describe "when it is not required" do
+      use_attribute_value DBNazi, :require_nullability, false
+
+      it "does not raise a DBNazi::NullabilityRequired if :null is not specified when adding a column" do
+        connection.add_column 'test_table', 'test_column', :boolean
+      end
+    end
+  end
+
+  describe "varchar limits" do
+    describe "when they are required" do
+      use_attribute_value DBNazi, :require_varchar_limits, true
+
+      it "raises a DBNazi::VarcharLimitRequired if :limit is not specified for a :string column" do
+        lambda do
+          connection.add_column 'test_table', 'test_column', :string, null: true
+        end.must_raise(DBNazi::VarcharLimitRequired)
+      end
+
+      it "does not raise a DBNazi::VarcharLimitRequired if :limit is specified for a :string column" do
+        connection.add_column 'test_table', 'test_column', :string, limit: 255, null: true
+      end
+    end
+
+    describe "when they are not required" do
+      use_attribute_value DBNazi, :require_varchar_limits, false
+
+      it "does not raise a DBNazi::VarcharLimitRequired if :limit is not specified for a :string column" do
+        connection.add_column 'test_table', 'test_column', :string, null: true
+      end
+    end
+  end
+
+  describe "index uniqueness" do
+    before do
+      connection.add_column 'test_table', 'test_column', :boolean, null: true
+    end
+
+    describe "when it is required" do
+      use_attribute_value DBNazi, :require_index_uniqueness, true
+
+      it "raises a DBNazi::IndexUniquenessRequired if :unique is not specified for an index" do
+        lambda do
+          connection.add_index 'test_table', 'test_column'
+        end.must_raise(DBNazi::IndexUniquenessRequired)
+      end
+
+      it "does not raise a DBNazi::IndexUniquenessRequired if :unique is true for an index" do
+        connection.add_index 'test_table', 'test_column', unique: true
+      end
+
+      it "does not raise a DBNazi::IndexUniquenessRequired if :unique is false for an index" do
+        connection.add_index 'test_table', 'test_column', unique: false
+      end
+    end
+
+    describe "when it is not required" do
+      use_attribute_value DBNazi, :require_index_uniqueness, false
+
+      it "does not raise a DBNazi::IndexUniquenessRequired if :unique is not specified for an index" do
+        connection.add_index 'test_table', 'test_column'
+      end
+    end
+  end
+
+  it "does not prevent construction of the schema migrations table" do
+    # AR doesn't specify a varchar limit for the version column. Lame.
+    connection.initialize_schema_migrations_table
+  end
+end

data/test/unit/db_nazi/test_migration.rb

@@ -0,0 +1,10 @@
+require_relative '../../test_helper'
+
+describe DBNazi::Migration do
+  describe "#nazi_disabled?" do
+    it "is true if no_nazi was called on the migration class" do
+      klass = Class.new(ActiveRecord::Migration) { no_nazi }
+      klass.new.nazi_disabled?.must_equal true
+    end
+  end
+end

data/test/unit/db_nazi/test_migration_proxy.rb

@@ -0,0 +1,68 @@
+require_relative '../../test_helper'
+
+describe DBNazi::MigrationProxy do
+  use_database
+  use_temporary_directory "#{ROOT}/test/tmp"
+
+  # Migrations use Kernel.puts. Lame.
+  out = Class.new { def write(*) end; def flush(*) end }.new
+  use_global_value :stdout, out
+
+  before do
+    @original_pwd = Dir.pwd
+    Dir.chdir "#{ROOT}/test/tmp"
+  end
+
+  after do
+    Dir.chdir @original_pwd
+  end
+
+  before do
+    @migration_proxy = ActiveRecord::MigrationProxy.new("test_migration", 10, "test_migration.rb", nil)
+    @migration_class = Class.new(ActiveRecord::Migration) do
+      attr_reader :enabled
+
+      def up
+        @enabled = DBNazi.enabled
+      end
+    end
+    migration = @migration = @migration_class.new
+    @migration_proxy.singleton_class.send(:define_method, :migration) { migration }
+  end
+
+  describe "#migrate" do
+    it "disables DBNazi when no_nazi is used" do
+      @migration_class.no_nazi
+      @migration_proxy.migrate(:up)
+      @migration.enabled.must_equal false
+    end
+
+    describe "when DBNazi.from_version is set" do
+      it "disables DBNazi if this migration comes before the minimum version" do
+        with_attribute_value DBNazi, :from_version, 11 do
+          @migration_proxy.migrate(:up)
+          @migration.enabled.must_equal false
+        end
+      end
+
+      it "does not disable DBNazi if this migration is the minimum version" do
+        with_attribute_value DBNazi, :from_version, 10 do
+          @migration_proxy.migrate(:up)
+          @migration.enabled.must_equal true
+        end
+      end
+
+      it "does not disable DBNazi if this migration comes after the minimum version" do
+        with_attribute_value DBNazi, :from_version, 9 do
+          @migration_proxy.migrate(:up)
+          @migration.enabled.must_equal true
+        end
+      end
+    end
+
+    it "enables DBNazi otherwise" do
+      @migration_proxy.migrate(:up)
+      @migration.enabled.must_equal true
+    end
+  end
+end

data/test/unit/db_nazi/test_table_definition.rb

@@ -0,0 +1,75 @@
+require_relative '../../test_helper'
+
+describe DBNazi::TableDefinition do
+  use_database
+
+  before do
+    DBNazi.reset
+  end
+
+  describe "nullability" do
+    describe "when it is required" do
+      use_attribute_value DBNazi, :require_nullability, true
+
+      it "raises a DBNazi::NullabilityRequired if :null is not specified when adding a column" do
+        connection.create_table 'test_table' do |t|
+          lambda do
+            t.column 'test_column', :boolean
+          end.must_raise(DBNazi::NullabilityRequired)
+        end
+      end
+
+      it "does not raise a DBNazi::NullabilityRequired if :null is true when adding a column" do
+        connection.create_table 'test_table' do |t|
+          t.column 'test_column', :boolean, null: true
+        end
+      end
+
+      it "does not raise a DBNazi::NullabilityRequired if :null is false when adding a column" do
+        connection.create_table 'test_table' do |t|
+          t.column 'test_column', :boolean, null: false, default: false
+        end
+      end
+    end
+
+    describe "when it is not required" do
+      use_attribute_value DBNazi, :require_nullability, false
+
+      it "does not raise a DBNazi::NullabilityRequired if :null is not specified when adding a column" do
+        connection.create_table 'test_table' do |t|
+          t.column 'test_column', :boolean
+        end
+      end
+    end
+  end
+
+  describe "varchar limits" do
+    describe "when they are required" do
+      use_attribute_value DBNazi, :require_varchar_limits, true
+
+      it "raises a DBNazi::VarcharLimitRequired if :limit is not specified for a :string column" do
+        connection.create_table 'test_table' do |t|
+          lambda do
+            t.column 'test_column', :string, null: true
+          end.must_raise(DBNazi::VarcharLimitRequired)
+        end
+      end
+
+      it "does not raise a DBNazi::VarcharLimitRequired if :limit is specified for a :string column" do
+        connection.create_table 'test_table' do |t|
+          t.column 'test_column', :string, limit: 255, null: true
+        end
+      end
+    end
+
+    describe "when they are not required" do
+      use_attribute_value DBNazi, :require_varchar_limits, false
+
+      it "does not raise a DBNazi::VarcharLimitRequired if :limit is not specified for a :string column" do
+        connection.create_table 'test_table' do |t|
+          t.column 'test_column', :string, null: true
+        end
+      end
+    end
+  end
+end

data/test/unit/test_db_nazi.rb

@@ -0,0 +1,51 @@
+require_relative '../test_helper'
+
+describe DBNazi do
+  before do
+    DBNazi.reset
+  end
+
+  describe ".enable" do
+    it "should turn on all feature flags for the duration of the block" do
+      DBNazi.enabled = false
+      DBNazi.enable do
+        DBNazi.enabled?(:require_nullability).must_equal true
+        DBNazi.enabled?(:require_varchar_limits).must_equal true
+        DBNazi.enabled?(:require_index_uniqueness).must_equal true
+      end
+      DBNazi.enabled?(:require_nullability).must_equal false
+      DBNazi.enabled?(:require_varchar_limits).must_equal false
+      DBNazi.enabled?(:require_index_uniqueness).must_equal false
+    end
+
+    it "should restore a flag to true if it was already enabled" do
+      DBNazi.enabled = true
+      DBNazi.enable do
+        DBNazi.enabled?(:require_nullability).must_equal true
+      end
+      DBNazi.enabled?(:require_nullability).must_equal true
+    end
+  end
+
+  describe ".disable" do
+    it "should turn off all feature flags for the duration of the block" do
+      DBNazi.enabled = true
+      DBNazi.disable do
+        DBNazi.enabled?(:require_nullability).must_equal false
+        DBNazi.enabled?(:require_varchar_limits).must_equal false
+        DBNazi.enabled?(:require_index_uniqueness).must_equal false
+      end
+      DBNazi.enabled?(:require_nullability).must_equal true
+      DBNazi.enabled?(:require_varchar_limits).must_equal true
+      DBNazi.enabled?(:require_index_uniqueness).must_equal true
+    end
+
+    it "should restore a flag to false if it was already disabled" do
+      DBNazi.enabled = false
+      DBNazi.disable do
+        DBNazi.enabled?(:require_nullability).must_equal false
+      end
+      DBNazi.enabled?(:require_nullability).must_equal false
+    end
+  end
+end

metadata

@@ -0,0 +1,178 @@
+--- !ruby/object:Gem::Specification
+name: db_nazi
+version: !ruby/object:Gem::Version
+  version: 0.0.1
+  prerelease: 
+platform: ruby
+authors:
+- George Ogata
+autorequire: 
+bindir: bin
+cert_chain: []
+date: 2012-06-23 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: activerecord
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 3.2.6
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 3.2.6
+- !ruby/object:Gem::Dependency
+  name: ritual
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 0.4.1
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 0.4.1
+- !ruby/object:Gem::Dependency
+  name: temporaries
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 0.2.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 0.2.0
+- !ruby/object:Gem::Dependency
+  name: looksee
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 0.2.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 0.2.0
+- !ruby/object:Gem::Dependency
+  name: debugger
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 1.1.3
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 1.1.3
+- !ruby/object:Gem::Dependency
+  name: sqlite3
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 1.3.6
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 1.3.6
+description: ''
+email:
+- george.ogata@gmail.com
+executables: []
+extensions: []
+extra_rdoc_files: []
+files:
+- .gitignore
+- .travis.yml
+- CHANGELOG
+- Gemfile
+- LICENSE
+- README.markdown
+- Rakefile
+- db_nazi.gemspec
+- lib/db_nazi.rb
+- lib/db_nazi/abstract_adapter.rb
+- lib/db_nazi/migration.rb
+- lib/db_nazi/migration_proxy.rb
+- lib/db_nazi/table_definition.rb
+- lib/db_nazi/version.rb
+- test/database.yml
+- test/integration/test_integration.rb
+- test/test_helper.rb
+- test/unit/db_nazi/test_abstract_adapter.rb
+- test/unit/db_nazi/test_migration.rb
+- test/unit/db_nazi/test_migration_proxy.rb
+- test/unit/db_nazi/test_table_definition.rb
+- test/unit/test_db_nazi.rb
+homepage: ''
+licenses:
+- MIT
+post_install_message: 
+rdoc_options: []
+require_paths:
+- lib
+required_ruby_version: !ruby/object:Gem::Requirement
+  none: false
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
+      segments:
+      - 0
+      hash: 2002143158366431765
+required_rubygems_version: !ruby/object:Gem::Requirement
+  none: false
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
+      segments:
+      - 0
+      hash: 2002143158366431765
+requirements: []
+rubyforge_project: 
+rubygems_version: 1.8.24
+signing_key: 
+specification_version: 3
+summary: Encourage good DB practices in ActiveRecord migrations.
+test_files:
+- test/database.yml
+- test/integration/test_integration.rb
+- test/test_helper.rb
+- test/unit/db_nazi/test_abstract_adapter.rb
+- test/unit/db_nazi/test_migration.rb
+- test/unit/db_nazi/test_migration_proxy.rb
+- test/unit/db_nazi/test_table_definition.rb
+- test/unit/test_db_nazi.rb