ensure unique account names

app/models/plutus/account.rb

@@ -36,6 +36,7 @@ class Account < ActiveRecord::Base
     has_many :debit_transactions, :through => :debit_amounts, :source => :transaction
 
     validates_presence_of :type, :name
+    validates_uniqueness_of :name
 
     # The trial balance of all accounts in the system. This should always equal zero,
     # otherwise there is an error in the system.

spec/factories/account_factory.rb

@@ -1,31 +1,35 @@
 FactoryGirl.define do
   factory :account, :class => Plutus::Account do |account|
-    account.name 'factory name'
+    account.name
     account.contra false
   end
 
   factory :asset, :class => Plutus::Asset do |account|
-    account.name 'factory name'
+    account.name
     account.contra false
   end
 
   factory :equity, :class => Plutus::Equity do |account|
-    account.name 'factory name'
+    account.name
     account.contra false
   end
 
   factory :expense, :class => Plutus::Expense do |account|
-    account.name 'factory name'
+    account.name
     account.contra false
   end
 
   factory :liability, :class => Plutus::Liability do |account|
-    account.name 'factory name'
+    account.name
     account.contra false
   end
 
   factory :revenue, :class => Plutus::Revenue do |account|
-    account.name 'factory name'
+    account.name
     account.contra false
   end
+
+  sequence :name do |n|
+    "Factory Name #{n}"
+  end
 end

spec/models/account_spec.rb

@@ -8,6 +8,13 @@ module Plutus
       account.should_not be_valid
     end
 
+    it "should be unique per name" do
+      FactoryGirl.create(:account, :name => "Test1", :type => "Plutus::Asset")
+      account = FactoryGirl.build(:account, :name => "Test1", :type => "Plutus::Asset")
+      account.should_not be_valid
+      account.errors[:name].should == ["has already been taken"]
+    end
+
     it "should not have a balance method" do
       lambda{Account.balance}.should raise_error(NoMethodError)
     end