From 6315a7a0046edaf333c0055227c6fc693438ae43 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jose=CC=81=20Mota?= <jose@josemota.net>
Date: Thu, 30 May 2013 15:03:23 +0100
Subject: [PATCH] Separate Query from Modifier.

---
 move-method/lib/after.rb                      |   0
 move-method/lib/before.rb                     |   0
 move-method/test/test.rb                      |   5 --
 separate-query-from-modifier/.DS_Store        | Bin 0 -> 6148 bytes
 .../Rakefile                                  |   0
 separate-query-from-modifier/lib/after.rb     |  62 ++++++++++++++++++
 separate-query-from-modifier/lib/before.rb    |  59 +++++++++++++++++
 separate-query-from-modifier/test/test.rb     |  31 +++++++++
 8 files changed, 152 insertions(+), 5 deletions(-)
 delete mode 100644 move-method/lib/after.rb
 delete mode 100644 move-method/lib/before.rb
 delete mode 100644 move-method/test/test.rb
 create mode 100644 separate-query-from-modifier/.DS_Store
 rename {move-method => separate-query-from-modifier}/Rakefile (100%)
 create mode 100644 separate-query-from-modifier/lib/after.rb
 create mode 100644 separate-query-from-modifier/lib/before.rb
 create mode 100644 separate-query-from-modifier/test/test.rb

diff --git a/move-method/lib/after.rb b/move-method/lib/after.rb
deleted file mode 100644
index e69de29..0000000
diff --git a/move-method/lib/before.rb b/move-method/lib/before.rb
deleted file mode 100644
index e69de29..0000000
diff --git a/move-method/test/test.rb b/move-method/test/test.rb
deleted file mode 100644
index a8f6974..0000000
--- a/move-method/test/test.rb
+++ /dev/null
@@ -1,5 +0,0 @@
-require 'minitest/spec'
-require 'minitest/autorun'
-
-require 'before' if ENV["BEFORE"]
-require 'after' unless ENV["BEFORE"]
diff --git a/separate-query-from-modifier/.DS_Store b/separate-query-from-modifier/.DS_Store
new file mode 100644
index 0000000000000000000000000000000000000000..5008ddfcf53c02e82d7eee2e57c38e5672ef89f6
GIT binary patch
literal 6148
zcmeH~Jr2S!425mzP>H1@V-^m;4Wg<&0T*E43hX&L&p$$qDprKhvt+--jT7}7np#A3
zem<@ulZcFPQ@L2!n>{z**<q8>++&mCkOWA81W14cNZ<zv;LbK1Poaz?KmsK2CSc!(
z0ynLxE!0092;Krf2c+FF_Fe*7ECH>lEfg7;MkzE(HCqgga^y>{tEnwC%0;vJ&^%eQ
zLs35+`xjp>T0<F0fCPF1$Cyrb|F7^5{eNG?83~ZUUlGt@xh*qZDeu<Z%US-OSsOPv
j)R!Z4KLME7ReXlK;d!wEw5GODWMKRea10D2@KpjYNUI8I

literal 0
HcmV?d00001

diff --git a/move-method/Rakefile b/separate-query-from-modifier/Rakefile
similarity index 100%
rename from move-method/Rakefile
rename to separate-query-from-modifier/Rakefile
diff --git a/separate-query-from-modifier/lib/after.rb b/separate-query-from-modifier/lib/after.rb
new file mode 100644
index 0000000..6fcdf11
--- /dev/null
+++ b/separate-query-from-modifier/lib/after.rb
@@ -0,0 +1,62 @@
+class Post
+  attr_reader :id, :title, :body, :created_at
+  def initialize id, title, body, created_at
+    @id         = id
+    @title      = title
+    @body       = body
+    @created_at = created_at
+    @published = false
+  end
+
+  def self.find id
+    # database operation to retrieve data. We'll simulate it for now.
+    post = POSTS.find { |post| post.id == id }
+  end
+
+  def self.unpublished
+    return POSTS.count { |post| !post.published? }
+  end
+
+  def publish
+    @published = true
+  end
+
+  def unpublish
+    @published = false
+  end
+
+  def published?
+    @published
+  end
+
+end
+
+### Sample data
+
+POSTS = [
+  Post.new(
+    1,
+    "Introduce Null Object Pattern",
+    "Post body should be here",
+    Time.new(2013,01,25)
+  ),
+  Post.new(
+    2,
+    "Introduce Assertion",
+    "Post body should be here",
+    Time.new(2012,02,26)
+  ),
+  Post.new(
+    3,
+    "Extract Method",
+    "Post body should be here",
+    Time.new(2014,01,27)
+  ),
+  Post.new(
+    4,
+    "Replace Type Code with Polymorphism",
+    "Post body should be here",
+    Time.new(2015,10,12)
+  )
+]
+
diff --git a/separate-query-from-modifier/lib/before.rb b/separate-query-from-modifier/lib/before.rb
new file mode 100644
index 0000000..1ae21c9
--- /dev/null
+++ b/separate-query-from-modifier/lib/before.rb
@@ -0,0 +1,59 @@
+class Post
+  attr_reader :id, :title, :body, :created_at
+  def initialize id, title, body, created_at
+    @id         = id
+    @title      = title
+    @body       = body
+    @created_at = created_at
+    @published = false
+  end
+
+  def self.find id
+    # database operation to retrieve data. We'll simulate it for now.
+    post = POSTS.find { |post| post.id == id }
+  end
+
+  def publish
+    @published = true
+    return POSTS.count { |post| !post.published? }
+  end
+
+  def unpublish
+    @published = false
+  end
+
+  def published?
+    @published
+  end
+
+end
+
+### Sample data
+
+POSTS = [
+  Post.new(
+    1,
+    "Introduce Null Object Pattern",
+    "Post body should be here",
+    Time.new(2013,01,25)
+  ),
+  Post.new(
+    2,
+    "Introduce Assertion",
+    "Post body should be here",
+    Time.new(2012,02,26)
+  ),
+  Post.new(
+    3,
+    "Extract Method",
+    "Post body should be here",
+    Time.new(2014,01,27)
+  ),
+  Post.new(
+    4,
+    "Replace Type Code with Polymorphism",
+    "Post body should be here",
+    Time.new(2015,10,12)
+  )
+]
+
diff --git a/separate-query-from-modifier/test/test.rb b/separate-query-from-modifier/test/test.rb
new file mode 100644
index 0000000..5b9190c
--- /dev/null
+++ b/separate-query-from-modifier/test/test.rb
@@ -0,0 +1,31 @@
+require 'minitest/autorun'
+require 'minitest/spec'
+
+require 'before' if ENV["BEFORE"]
+require 'after' unless ENV["BEFORE"]
+
+describe Post do
+  
+  before :each do
+    POSTS.each { |post| post.unpublish }
+  end
+
+  if ENV["BEFORE"]
+    it "is publishable and retrieves all unpublished posts count" do
+      post = Post.find(1)
+      post.publish.must_equal 3
+    end
+
+  else # AFTER
+
+    it "is publishable" do
+      post = Post.find(1)
+      post.publish.must_equal true
+    end
+
+    it "retrieves all unpublished posts count" do
+      Post.unpublished.must_equal 4
+    end
+
+  end
+end