start adding some rspec tests

Author: whoward

.gitignore

@@ -23,3 +23,4 @@
 # bower files
 vendor/assets/bower_components
 database.yml
+.rspec

Gemfile

@@ -49,4 +49,15 @@ gem 'bootstrap-kaminari-views'
 # backends for mint api
 gem 'state_machine'
 gem 'chronic'
-gem 'selenium-webdriver'
+gem 'selenium-webdriver'
+
+group :test do
+  gem 'rspec-rails'
+  gem 'factory_girl_rails'
+  gem 'database_cleaner'
+end
+
+group :development do
+  gem 'pry'
+  gem 'pry-remote'
+end

Gemfile.lock

@@ -39,15 +39,23 @@ GEM
     childprocess (0.5.5)
       ffi (~> 1.0, >= 1.0.11)
     chronic (0.10.2)
+    coderay (1.1.0)
     coffee-rails (4.0.1)
       coffee-script (>= 2.2.0)
       railties (>= 4.0.0, < 5.0)
     coffee-script (2.3.0)
       coffee-script-source
       execjs
     coffee-script-source (1.8.0)
+    database_cleaner (1.3.0)
+    diff-lcs (1.2.5)
     erubis (2.7.0)
     execjs (2.2.1)
+    factory_girl (4.4.0)
+      activesupport (>= 3.0.0)
+    factory_girl_rails (4.4.1)
+      factory_girl (~> 4.4.0)
+      railties (>= 3.0.0)
     ffi (1.9.5)
     figaro (1.0.0)
       thor (~> 0.14)
@@ -64,12 +72,20 @@ GEM
       activesupport (>= 3.0.0)
     mail (2.6.1)
       mime-types (>= 1.16, < 3)
+    method_source (0.8.2)
     mime-types (2.3)
     minitest (5.4.2)
     multi_json (1.10.1)
     pg (0.17.1)
     polyamorous (1.1.0)
       activerecord (>= 3.0)
+    pry (0.10.1)
+      coderay (~> 1.1.0)
+      method_source (~> 0.8.1)
+      slop (~> 3.4)
+    pry-remote (0.1.8)
+      pry (~> 0.9)
+      slop (~> 3.0)
     rack (1.5.2)
     rack-test (0.6.2)
       rack (>= 1.0)
@@ -95,6 +111,22 @@ GEM
       activesupport (>= 3.0)
       i18n
       polyamorous (~> 1.1)
+    rspec-core (3.1.5)
+      rspec-support (~> 3.1.0)
+    rspec-expectations (3.1.2)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.1.0)
+    rspec-mocks (3.1.2)
+      rspec-support (~> 3.1.0)
+    rspec-rails (3.1.0)
+      actionpack (>= 3.0)
+      activesupport (>= 3.0)
+      railties (>= 3.0)
+      rspec-core (~> 3.1.0)
+      rspec-expectations (~> 3.1.0)
+      rspec-mocks (~> 3.1.0)
+      rspec-support (~> 3.1.0)
+    rspec-support (3.1.1)
     rubyzip (1.1.6)
     sass (3.2.19)
     sass-rails (4.0.3)
@@ -107,6 +139,7 @@ GEM
       multi_json (~> 1.0)
       rubyzip (~> 1.0)
       websocket (~> 1.0)
+    slop (3.6.0)
     sprockets (2.11.0)
       hike (~> 1.2)
       multi_json (~> 1.0)
@@ -145,13 +178,18 @@ DEPENDENCIES
   bower-rails
   chronic
   coffee-rails
+  database_cleaner
+  factory_girl_rails
   figaro
   haml
   jquery-rails
   kaminari
   pg
+  pry
+  pry-remote
   rails (= 4.1.6)
   ransack
+  rspec-rails
   sass-rails (~> 4.0.3)
   selenium-webdriver
   state_machine

app/models/category.rb

@@ -5,7 +5,6 @@ class Category < ActiveRecord::Base
 
   ALLOWED_ROOTS = %w(Income Expense Transfers).freeze
 
-  # TODO: fixturize this data
   def self.income
     @income ||= roots.where(name: "Income").first
   end
@@ -26,11 +25,21 @@ def self.transfer_to
     @transfer_to ||= where(parent_id: transfers, name: "Transfer To").first
   end
 
+  def self.invalidate_cache!
+    @income = nil
+    @expense = nil
+    @transfers = nil
+    @transfer_from = nil
+    @transfer_to = nil
+  end
+
   scope :budgeted, -> { where("budgeted_cents > 0") }
   scope :unbudgeted, -> { where("budgeted_cents is null or budgeted_cents <= 0") }
 
   validates_inclusion_of :name, in: ALLOWED_ROOTS, if: :root?
 
+  after_create { Category.invalidate_cache! }
+
   def can_edit?
     can_destroy?
   end

db/seeds.rb

@@ -5,3 +5,10 @@
 #
 #   cities = City.create([{ name: 'Chicago' }, { name: 'Copenhagen' }])
 #   Mayor.create(name: 'Emanuel', city: cities.first)
+
+income = Category.find_or_create_by(name: "Income")
+expense = Category.find_or_create_by(name: "Expense")
+transfers = Category.find_or_create_by(name: "Transfers")
+
+Category.find_or_create_by(name: "Transfer From", parent_id: transfers.id)
+Category.find_or_create_by(name: "Transfer To", parent_id: transfers.id)

spec/factories/accounts.rb

@@ -0,0 +1,8 @@
+
+FactoryGirl.define do
+
+  factory :account do
+    name "MyAccount"
+  end
+
+end

spec/factories/category.rb

@@ -0,0 +1,16 @@
+
+FactoryGirl.define do
+
+  factory :category do
+    parent_id { Category.expense.id }
+    name "Restaurant"
+  end
+
+  factory :expense_category, parent: :category do
+  end
+
+  factory :income_category, parent: :category do
+    parent_id { Category.income.id }
+  end
+
+end

spec/factories/transactions.rb

@@ -0,0 +1,12 @@
+
+FactoryGirl.define do
+
+  factory :transaction do
+    account
+    category
+    cents 100
+    date { Date.today }
+    description "Tim Hortons"
+  end
+
+end

spec/models/account_spec.rb

@@ -0,0 +1,5 @@
+require 'rails_helper'
+
+describe Account do
+  # nothing to do yet
+end

spec/models/category_spec.rb

@@ -0,0 +1,11 @@
+require 'rails_helper'
+
+describe Category do
+
+  describe "validations" do
+    it "can only be Income, Expense or Transfer when it is a root category" do
+      expect(build(:category, parent: nil, name: "Foo")).not_to be_valid
+    end
+  end
+
+end

spec/models/transaction_spec.rb

@@ -0,0 +1,27 @@
+require 'rails_helper'
+
+describe Transaction do
+
+  describe "validation" do
+    it "requires an account" do
+      expect(build(:transaction, account: nil)).not_to be_valid
+    end
+
+    it "requires a category" do
+      expect(build(:transaction, category: nil)).not_to be_valid
+    end
+
+    it "requires the amount" do
+      expect(build(:transaction, cents: nil)).not_to be_valid
+    end
+
+    it "requires the date" do
+      expect(build(:transaction, date: nil)).not_to be_valid
+    end
+
+    it "requires a description" do
+      expect(build(:transaction, description: nil)).not_to be_valid
+    end
+  end
+
+end

spec/rails_helper.rb

@@ -0,0 +1,67 @@
+# This file is copied to spec/ when you run 'rails generate rspec:install'
+ENV["RAILS_ENV"] ||= 'test'
+require 'spec_helper'
+require File.expand_path("../../config/environment", __FILE__)
+require 'rspec/rails'
+require Rails.root.join("db", "seeds")
+
+require 'pry'
+
+# Add additional requires below this line. Rails is not loaded until this point!
+
+# Requires supporting ruby files with custom matchers and macros, etc, in
+# spec/support/ and its subdirectories. Files matching `spec/**/*_spec.rb` are
+# run as spec files by default. This means that files in spec/support that end
+# in _spec.rb will both be required and run as specs, causing the specs to be
+# run twice. It is recommended that you do not name files matching this glob to
+# end with _spec.rb. You can configure this pattern with the --pattern
+# option on the command line or in ~/.rspec, .rspec or `.rspec-local`.
+#
+# The following line is provided for convenience purposes. It has the downside
+# of increasing the boot-up time by auto-requiring all files in the support
+# directory. Alternatively, in the individual `*_spec.rb` files, manually
+# require only the support files necessary.
+#
+Dir[Rails.root.join("spec/support/**/*.rb")].each { |f| require f }
+
+# Checks for pending migrations before tests are run.
+# If you are not using ActiveRecord, you can remove this line.
+ActiveRecord::Migration.maintain_test_schema!
+
+RSpec.configure do |config|
+  # If you're not using ActiveRecord, or you'd prefer not to run each of your
+  # examples within a transaction, remove the following line or assign false
+  # instead of true.
+  config.use_transactional_fixtures = true
+
+  # RSpec Rails can automatically mix in different behaviours to your tests
+  # based on their file location, for example enabling you to call `get` and
+  # `post` in specs under `spec/controllers`.
+  #
+  # You can disable this behaviour by removing the line below, and instead
+  # explicitly tag your specs with their type, e.g.:
+  #
+  #     RSpec.describe UsersController, :type => :controller do
+  #       # ...
+  #     end
+  #
+  # The different available types are documented in the features, such as in
+  # https://relishapp.com/rspec/rspec-rails/docs
+  config.infer_spec_type_from_file_location!
+
+  # Include FactoryGirl helpers
+  config.include FactoryGirl::Syntax::Methods
+
+  # Before running the suite build all factory objects and ensure they are #valid?
+  config.before(:suite) do
+    FactoryGirl.lint
+  end
+
+  config.before(:each) do
+    DatabaseCleaner.strategy = :transaction
+  end
+
+  config.after(:each) do
+    DatabaseCleaner.clean
+  end
+end