guacamole 0.2.0 → 0.3.0

data/lib/rails/generators/rspec/callbacks/templates/callbacks_spec.rb.tt

@@ -0,0 +1,7 @@
+require 'spec_helper'
+
+<% module_namespacing do -%>
+describe <%= class_name %>Callbacks do
+  pending "add some examples to (or delete) #{__FILE__}"
+end
+<% end -%>

data/lib/rails/generators/test_unit/callbacks/callbacks_generator.rb

@@ -0,0 +1,13 @@
+# -*- encoding : utf-8 -*-
+
+require 'rails/generators/guacamole_generator'
+
+module TestUnit
+  module Generators
+    class CallbacksGenerator < ::Guacamole::Generators::Base
+      def create_test_file
+        template 'callbacks_test.rb.tt', File.join('test/callbacks', class_path, "#{file_name}_callbacks_test.rb")
+      end
+    end
+  end
+end

data/lib/rails/generators/test_unit/callbacks/templates/callbacks_test.rb.tt

@@ -0,0 +1,9 @@
+require 'test_helper'
+
+<% module_namespacing do -%>
+class <%= class_name %>CallbacksTest < ActiveSupport::TestCase
+  # test "the truth" do
+  #   assert true
+  # end
+end
+<% end -%>

data/lib/rails/generators/test_unit/collection/collection_generator.rb

@@ -0,0 +1,13 @@
+# -*- encoding : utf-8 -*-
+
+require 'rails/generators/guacamole_generator'
+
+module TestUnit
+  module Generators
+    class CollectionGenerator < ::Guacamole::Generators::Base
+      def create_test_file
+        template 'collection_test.rb.tt', File.join('test/collections', class_path, "#{file_name}_collection_test.rb")
+      end
+    end
+  end
+end

data/lib/rails/generators/test_unit/collection/templates/collection_test.rb.tt

@@ -0,0 +1,10 @@
+require 'test_helper'
+
+<% module_namespacing do -%>
+class <%= class_name %>CollectionTest < ActiveSupport::TestCase
+  # test "the truth" do
+  #   assert true
+  # end
+end
+<% end -%>
+

data/spec/acceptance/aql_spec.rb

@@ -3,18 +3,6 @@
 require 'guacamole'
 require 'acceptance/spec_helper'
 
-class Pony
-  include Guacamole::Model
-
-  attribute :name, String
-  attribute :color, String
-  attribute :type, Array[String]
-end
-
-class PoniesCollection
-  include Guacamole::Collection
-end
-
 describe 'BasicAQLSupport' do
   subject { PoniesCollection }
 

data/spec/acceptance/basic_spec.rb

@@ -2,29 +2,6 @@
 require 'guacamole'
 require 'acceptance/spec_helper'
 
-class Comment
-  include Guacamole::Model
-
-  attribute :text, String
-end
-
-class Article
-  include Guacamole::Model
-
-  attribute :title, String
-  attribute :comments, Array[Comment]
-
-  validates :title, presence: true
-end
-
-class ArticlesCollection
-  include Guacamole::Collection
-
-  map do
-    embeds :comments
-  end
-end
-
 describe 'ModelBasics' do
 
   describe Article do
@@ -45,9 +22,9 @@ describe 'ModelBasics' do
     end
 
     it 'should validate its attributes' do
-      expect(subject.valid?).to be_false
+      expect(subject.valid?).to be false
       subject.title = 'The Legend of Zelda'
-      expect(subject.valid?).to be_true
+      expect(subject.valid?).to be true
     end
 
     it 'should know its model name' do
@@ -100,6 +77,20 @@ describe 'CollectionBasics' do
       expect(result.title).to eq 'Disturbed'
     end
 
+    context 'delete documents' do
+      it 'should delete a document by key' do
+        deleted_key = subject.delete(some_article.key)
+
+        expect { subject.by_key(deleted_key) }.to raise_error(Ashikawa::Core::DocumentNotFoundException)
+      end
+
+      it 'should delete a document by model' do
+        deleted_key = subject.delete(some_article)
+
+        expect { subject.by_key(deleted_key) }.to raise_error(Ashikawa::Core::DocumentNotFoundException)
+      end
+    end
+
     it 'should allow to nest models' do
       article_with_comments = Fabricate(:article_with_two_comments)
       found_article = subject.by_key(article_with_comments.key)

data/spec/acceptance/callbacks_spec.rb

@@ -0,0 +1,181 @@
+# -*- encoding : utf-8 -*-
+
+require 'guacamole'
+require 'acceptance/spec_helper'
+
+require 'bcrypt'
+
+class SecurePonyCallbacks
+  include Guacamole::Callbacks
+
+  # Those will be triggered by the collection
+  before_create :encrypt_password
+  after_create  :throw_welcome_party
+  around_create :log_create_time
+
+  # Those will be triggered by the model
+  before_validate :generate_token
+  after_validate  :remove_safety_switch
+  around_validate :log_validate_time
+
+  def encrypt_password
+    object.encrypted_password = BCrypt::Password.create(object.password)
+  end
+
+  def throw_welcome_party
+    Party.throw!
+  end
+
+  def log_create_time
+    TimingLogger.log_time 'before_create'
+    yield
+    TimingLogger.log_time 'after_create'
+  end
+
+  def generate_token
+    object.token = SecureRandom.hex
+  end
+
+  def remove_safety_switch
+    object.safety_switch = :removed
+  end
+
+  def log_validate_time
+    TimingLogger.log_time 'before_validate'
+    yield
+    TimingLogger.log_time 'after_validate'
+  end
+end
+
+class SecurePony
+  include Guacamole::Model
+
+  callbacks :secure_pony_callbacks
+
+  attribute :name, String
+  attribute :token, String
+  attribute :encrypted_password, String
+
+  # define a virtual attribute
+  attr_accessor :password, :safety_switch
+end
+
+class SecurePoniesCollection
+  include Guacamole::Collection
+end
+
+class Party
+  def self.throw!; end
+end
+
+class TimingLogger
+  def self.log_time(*args); end
+end
+
+describe 'CallbacksSpec' do
+  subject { SecurePonyCallbacks }
+
+  let(:pinkie_pie) { SecurePony.new name: 'Pinkie Pie', password: 'cupcakes' }
+
+  before do
+    allow(TimingLogger).to receive(:log_time)
+  end
+
+  describe 'collection based callbacks' do
+    let(:collection) { SecurePoniesCollection }
+
+    it 'should fire the before create callback' do
+      collection.save pinkie_pie
+
+      expect(BCrypt::Password.new(pinkie_pie.encrypted_password)).to eq pinkie_pie.password
+    end
+
+    it 'should fire the after create callback' do
+      expect(Party).to receive(:throw!)
+
+      collection.save pinkie_pie
+    end
+
+    it 'should fire the around create callback' do
+      expect(TimingLogger).to receive(:log_time).with('before_create').ordered
+      expect(TimingLogger).to receive(:log_time).with('after_create').ordered
+
+      collection.save pinkie_pie
+    end
+
+    context 'fill time stamp attributes' do
+      let(:past) { 'Oct 26 1955 01:21'.to_datetime }
+      let(:now)  { 'Oct 26 1985 01:21'.to_datetime }
+
+      it 'should fill created_at and updated_at on create' do
+        Timecop.freeze(now) do
+          collection.save pinkie_pie
+        end
+
+        expect(pinkie_pie.created_at).to eq now
+        expect(pinkie_pie.updated_at).to eq now
+      end
+
+      it 'should update updated_at on update' do
+        collection.save pinkie_pie
+        pinkie_pie.name = 'Pinkie Pie - Updated'
+
+        Timecop.freeze(now) do
+          collection.save pinkie_pie
+        end
+
+        expect(pinkie_pie.updated_at).to eq now
+      end
+
+      context 'with the default callback' do
+        let(:pinkie_pie) { Fabricate.build(:pony, name: 'Pinkie Pie') }
+        let(:collection) { PoniesCollection }
+
+        it 'should fill created_at and updated_at on create' do
+          Timecop.freeze(now) do
+            collection.save pinkie_pie
+          end
+
+          expect(pinkie_pie.created_at).to eq now
+          expect(pinkie_pie.updated_at).to eq now
+        end
+
+        it 'should update updated_at on update' do
+          Timecop.freeze(past) do
+            collection.save pinkie_pie
+          end
+
+          pinkie_pie.color = 'pink'
+
+          Timecop.freeze(now) do
+            collection.save pinkie_pie
+          end
+
+          expect(pinkie_pie.created_at).to eq past
+          expect(pinkie_pie.updated_at).to eq now
+        end
+      end
+    end
+  end
+
+  describe 'model based callbacks' do
+    it 'should fire the before validate callback' do
+      pinkie_pie.valid?
+
+      expect(pinkie_pie.token).not_to be_nil
+    end
+
+    it 'should fire the after validate callback' do
+      pinkie_pie.valid?
+
+      expect(pinkie_pie.safety_switch).to be_truthy
+    end
+
+    it 'should fire the around validate callback' do
+      expect(TimingLogger).to receive(:log_time).with('before_validate')
+      expect(TimingLogger).to receive(:log_time).with('after_validate')
+
+      pinkie_pie.valid?
+    end
+  end
+end

data/spec/acceptance/config/guacamole.yml

@@ -8,4 +8,4 @@ test:
   port: 8529
   password: ''
   username: ''
-  database: '_system'
+  database: 'guacamole_acceptance_test'

data/spec/acceptance/spec_helper.rb

@@ -8,12 +8,8 @@ require 'fabrication'
 require 'faker'
 require 'logging'
 require 'ashikawa-core'
-
-begin
-  require 'debugger'
-rescue LoadError
-  puts "Debugger is not available. Maybe you're Travis."
-end
+require 'pry'
+require 'timecop'
 
 # This is required to remove the deprecation warning introduced in this commit:
 # https://github.com/svenfuchs/i18n/commit/3b6e56e06fd70f6e4507996b017238505e66608c.
@@ -50,6 +46,12 @@ Guacamole.configure do |config|
   config.load File.join(File.dirname(__FILE__), 'config', 'guacamole.yml')
 end
 
+begin
+  Guacamole.configuration.database.create
+rescue Ashikawa::Core::ClientError
+  # Database already exists, don't worry
+end
+
 RSpec.configure do |config|
   config.expect_with :rspec do |c|
     c.syntax = :expect
@@ -63,3 +65,19 @@ RSpec.configure do |config|
     Guacamole.configuration.database.truncate
   end
 end
+
+# Collections for fabricator models
+
+require 'fabricators/article'
+
+class ArticlesCollection
+  include Guacamole::Collection
+
+  map do
+    embeds :comments
+  end
+end
+
+class PoniesCollection
+  include Guacamole::Collection
+end

data/spec/fabricators/article.rb

@@ -0,0 +1,12 @@
+# -*- encoding: utf-8 -*-
+
+require 'fabricators/comment'
+
+class Article
+  include Guacamole::Model
+
+  attribute :title, String
+  attribute :comments, Array[Comment]
+
+  validates :title, presence: true
+end

data/spec/fabricators/comment.rb

@@ -0,0 +1,7 @@
+# -*- encoding: utf-8 -*-
+
+class Comment
+  include Guacamole::Model
+
+  attribute :text, String
+end

data/spec/fabricators/pony.rb

@@ -1,7 +1,9 @@
 # -*- encoding: utf-8 -*-
 
-Fabricator(:pony) do
-  name  { Faker::Name.name }
-  color { %w(purple pink blue yellow green white).sample }
-  type  { %w(Pegasus Earthpony Unicorn).sample }
+class Pony
+  include Guacamole::Model
+
+  attribute :name, String
+  attribute :color, String
+  attribute :type, Array[String]
 end

data/spec/fabricators/pony_fabricator.rb

@@ -0,0 +1,7 @@
+# -*- encoding: utf-8 -*-
+
+Fabricator(:pony) do
+  name  { Faker::Name.name }
+  color { %w(purple pink blue yellow green white).sample }
+  type  { %w(Pegasus Earthpony Unicorn).sample }
+end

data/spec/spec_helper.rb

@@ -2,12 +2,13 @@
 $LOAD_PATH.unshift(File.dirname(__FILE__))
 $LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
 
-begin
-  require 'debugger'
-rescue LoadError
-  puts "Debugger is not available. Maybe you're Travis."
+if ENV['CODECLIMATE_REPO_TOKEN']
+  require 'codeclimate-test-reporter'
+  CodeClimate::TestReporter.start
 end
 
+require 'rspec/its'
+
 RSpec.configure do |config|
   config.expect_with :rspec do |c|
     c.syntax = :expect