RubyGems - aspect - Versions diffs - 0.0.1 → 0.0.2 - Mend

@@ -0,0 +1,72 @@
+<!DOCTYPE html>
+<html xmlns='http://www.w3.org/1999/xhtml'>
+  <head>
+    <title>Code coverage for Aspect</title>
+    <meta http-equiv="content-type" content="text/html; charset=utf-8" />
+    <script src='./assets/0.10.0/application.js' type='text/javascript'></script>
+    <link href='./assets/0.10.0/application.css' media='screen, projection, print' rel='stylesheet' type='text/css'>
+    <link rel="shortcut icon" type="image/png" href="./assets/0.10.0/favicon_green.png" />
+    <link rel="icon" type="image/png" href="./assets/0.10.0/favicon.png" />
+  </head>
+  <body>
+    <div id="loading">
+      <img src="./assets/0.10.0/loading.gif" alt="loading"/>
+    </div>
+    <div id="wrapper" style="display:none;">
+      <div class="timestamp">Generated <abbr class="timeago" title="2016-02-03T20:13:38-05:00">2016-02-03T20:13:38-05:00</abbr></div>
+      <ul class="group_tabs"></ul>
+      <div id="content">
+        <div class="file_list_container" id="AllFiles">
+  <h2>
+    <span class="group_name">All Files</span>
+    (<span class="covered_percent"><span class="green">100.0%</span></span>
+     covered at
+     <span class="covered_strength">
+       <span class="red">
+         0.0
+       </span>
+    </span> hits/line)
+  </h2>
+  <a name="AllFiles"></a>
+  <div>
+    <b>0</b> files in total.
+    <b>0.0</b> relevant lines.
+    <span class="green"><b>0.0</b> lines covered</span> and
+    <span class="red"><b>0.0</b> lines missed </span>
+  </div>
+  <table class="file_list">
+    <thead>
+      <tr>
+        <th>File</th>
+        <th>% covered</th>
+        <th>Lines</th>
+        <th>Relevant Lines</th>
+        <th>Lines covered</th>
+        <th>Lines missed</th>
+        <th>Avg. Hits / Line</th>
+      </tr>
+    </thead>
+    <tbody>
+    </tbody>
+  </table>
+</div>
+      </div>
+      <div id="footer">
+        Generated by <a href="http://github.com/colszowka/simplecov">simplecov</a> v0.11.1
+        and simplecov-html v0.10.0<br/>
+        using RSpec
+      </div>
+      <div class="source_files">
+      </div>
+    </div>
+  </body>
+</html>

data/spec/lib/aspect/has_attributes_spec.rb CHANGED Viewed

@@ -1,13 +1,19 @@
 require "spec_helper"
+class TestObject
+  include Aspect::HasAttributes
+  def initialize
+    @name = "foobar"
+  end
+end
 describe Aspect::HasAttributes do
   describe ".attribute" do
     context "when no options are given" do
       context "and no block is given" do
         let(:instance) do
-          instance_class = Class.new do
-            include Aspect::HasAttributes
+          instance_class = Class.new(TestObject) do
             attribute(:name)
           end
@@ -15,7 +21,7 @@ describe Aspect::HasAttributes do
         end
         it "should define a getter and a setter which uses the argument given to set the instance variable" do
-          expect(instance.name).to eq(nil)
+          expect(instance.name).to eq("foobar")
           instance.name = "foo"
           expect(instance.name).to eq("foo")
         end
@@ -23,9 +29,7 @@ describe Aspect::HasAttributes do
       context "and a block is given" do
         let(:instance) do
-          instance_class = Class.new do
-            include Aspect::HasAttributes
+          instance_class = Class.new(TestObject) do
             attribute(:name) { |value| value.to_s }
           end
@@ -33,7 +37,7 @@ describe Aspect::HasAttributes do
         end
         it "should define a getter and a setter which uses the return value of the block to set the instance variable" do
-          expect(instance.name).to eq(nil)
+          expect(instance.name).to eq("foobar")
           instance.name = 123
           expect(instance.name).to eq("123")
         end
@@ -43,25 +47,73 @@ describe Aspect::HasAttributes do
     context "when the :getter option is given" do
       context "and it's truthy" do
         let(:instance) do
-          instance_class = Class.new do
-            include Aspect::HasAttributes
+          instance_class = Class.new(TestObject) do
             attribute(:name, getter: true)
           end
           instance_class.new
         end
-        it "should define a query getter which uses the argument given to set the instance variable" do
-          expect(instance.name).to eq(nil)
+        it "should define the getter" do
+          expect(instance.name).to eq("foobar")
+        end
+        context "and it's a Hash" do
+          context "and the :method option is passed" do
+            let(:instance) do
+              instance_class = Class.new(TestObject) do
+                attribute(:name, getter: { method: :capitalize })
+              end
+              instance_class.new
+            end
+            it "define the getter which calls the given method on the instance variable" do
+              expect(instance.name).to eq("Foobar")
+            end
+          end
+          context "and the :instance_method option is passed" do
+            let(:instance) do
+              instance_class = Class.new(TestObject) do
+                attribute(:name, getter: { instance_method: :reversitalize })
+                def reversitalize(value)
+                  value.reverse.capitalize
+                end
+              end
+              instance_class.new
+            end
+            it "define the getter which calls the given instance method, passing the instance variable as the argument" do
+              expect(instance.name).to eq("Raboof")
+            end
+          end
+          context "and both the :method and :instance_method options are passed" do
+            let(:instance) do
+              instance_class = Class.new(TestObject) do
+                attribute(:name, getter: { method: :capitalize, instance_method: :exclamationize })
+                def exclamationize(value)
+                  "#{value}!"
+                end
+              end
+              instance_class.new
+            end
+            it "define the getter which calls the given instance method, passing the result of the method called on the instance variable as the argument" do
+              expect(instance.name).to eq("Foobar!")
+            end
+          end
         end
       end
       context "and it's falsey" do
         let(:instance) do
-          instance_class = Class.new do
-            include Aspect::HasAttributes
+          instance_class = Class.new(TestObject) do
             attribute(:name, getter: false)
           end
@@ -69,7 +121,7 @@ describe Aspect::HasAttributes do
         end
         it "should not define a getter" do
-          expect(instance.respond_to?(:name)).to eq(false)
+          expect(instance).not_to respond_to(:name)
         end
       end
     end
@@ -78,37 +130,90 @@ describe Aspect::HasAttributes do
       context "and no block is given" do
         context "and it's truthy" do
           let(:instance) do
-            instance_class = Class.new do
-              include Aspect::HasAttributes
+            instance_class = Class.new(TestObject) do
               attribute(:name, setter: true)
             end
             instance_class.new
           end
-          it "should define a query setter which uses the argument given to set the instance variable" do
-            expect(instance.name).to eq(nil)
+          it "should define a setter which uses the argument given to set the instance variable" do
+            expect(instance.name).to eq("foobar")
             instance.name = "foobar"
             expect(instance.name).to eq("foobar")
           end
+          context "and it's a Hash" do
+            context "and the :method option is passed" do
+              let(:instance) do
+                instance_class = Class.new(TestObject) do
+                  attribute(:name, setter: { method: :capitalize })
+                end
+                instance_class.new
+              end
+              it "should define the setter which calls the given method on the instance variable" do
+                expect(instance.name).to eq("foobar")
+                instance.name = "foobar"
+                expect(instance.name).to eq("Foobar")
+              end
+            end
+            context "and the :instance_method option is passed" do
+              let(:instance) do
+                instance_class = Class.new(TestObject) do
+                  attribute(:name, setter: { instance_method: :reversitalize })
+                  def reversitalize(value)
+                    value.reverse.capitalize
+                  end
+                end
+                instance_class.new
+              end
+              it "should define the setter which calls the given instance method, passing the instance variable as the argument" do
+                expect(instance.name).to eq("foobar")
+                instance.name = "foobar"
+                expect(instance.name).to eq("Raboof")
+              end
+            end
+            context "and both the :method and :instance_method options are passed" do
+              let(:instance) do
+                instance_class = Class.new(TestObject) do
+                  attribute(:name, setter: { method: :capitalize, instance_method: :exclamationize })
+                  def exclamationize(value)
+                    "#{value}!"
+                  end
+                end
+                instance_class.new
+              end
+              it "define the setter which calls the given instance method, passing the result of the method called on the instance variable as the argument" do
+                expect(instance.name).to eq("foobar")
+                instance.name = "foobar"
+                expect(instance.name).to eq("Foobar!")
+              end
+            end
+          end
         end
         context "and it's falsey" do
           let(:instance) do
-            instance_class = Class.new do
-              include Aspect::HasAttributes
-              attribute(:admin, query: false)
+            instance_class = Class.new(TestObject) do
+              attribute(:name, setter: false)
             end
             instance_class.new
           end
-          it "should define a getter and a setter which uses the return value of the block to set the instance variable" do
-            expect(instance.admin).to eq(nil)
-            instance.admin = "truthy"
-            expect(instance.admin).to eq("truthy")
+          it "should define a getter and not a setter" do
+            expect(instance).to respond_to(:name)
+            expect(instance).not_to respond_to(:name=)
           end
         end
       end
@@ -116,43 +221,90 @@ describe Aspect::HasAttributes do
       context "and a block is given" do
         context "and it's truthy" do
           let(:instance) do
-            instance_class = Class.new do
-              include Aspect::HasAttributes
-              attribute(:moderator, query: true)
-              attribute(:admin, query: true) { |value| @moderator && value }
+            instance_class = Class.new(TestObject) do
+              attribute(:name, setter: true) { |value| value.reverse }
             end
             instance_class.new
           end
-          it "should define a query getter and a query setter which uses the return value of the block to set the instance variable as a truthy value" do
-            expect(instance.moderator?).to eq(false)
-            expect(instance.admin?).to eq(false)
-            instance.admin = "truthy"
-            expect(instance.admin?).to eq(false)
-            instance.moderator = true
-            instance.admin = "truthy"
-            expect(instance.moderator?).to eq(true)
-            expect(instance.admin?).to eq(true)
+          it "should define a setter which uses the argument given to set the instance variable" do
+            expect(instance.name).to eq("foobar")
+            instance.name = "foobar"
+            expect(instance.name).to eq("raboof")
+          end
+          context "and it's a Hash" do
+            context "and the :method option is passed" do
+              let(:instance) do
+                instance_class = Class.new(TestObject) do
+                  attribute(:name, setter: { method: :capitalize })
+                end
+                instance_class.new
+              end
+              it "should define the setter which calls the given method on the instance variable" do
+                expect(instance.name).to eq("foobar")
+                instance.name = "foobar"
+                expect(instance.name).to eq("Foobar")
+              end
+            end
+            context "and the :instance_method option is passed" do
+              let(:instance) do
+                instance_class = Class.new(TestObject) do
+                  attribute(:name, setter: { instance_method: :reversitalize })
+                  def reversitalize(value)
+                    value.reverse.capitalize
+                  end
+                end
+                instance_class.new
+              end
+              it "should define the setter which calls the given instance method, passing the instance variable as the argument" do
+                expect(instance.name).to eq("foobar")
+                instance.name = "foobar"
+                expect(instance.name).to eq("Raboof")
+              end
+            end
+            context "and both the :method and :instance_method options are passed" do
+              let(:instance) do
+                instance_class = Class.new(TestObject) do
+                  attribute(:name, setter: { method: :capitalize, instance_method: :exclamationize })
+                  def exclamationize(value)
+                    "#{value}!"
+                  end
+                end
+                instance_class.new
+              end
+              it "define the setter which calls the given instance method, passing the result of the method called on the instance variable as the argument" do
+                expect(instance.name).to eq("foobar")
+                instance.name = "foobar"
+                expect(instance.name).to eq("Foobar!")
+              end
+            end
           end
         end
         context "and it's falsey" do
           let(:instance) do
-            instance_class = Class.new do
-              include Aspect::HasAttributes
-              attribute(:admin, query: false) { "return value" }
+            instance_class = Class.new(TestObject) do
+              attribute(:name, setter: false)
             end
             instance_class.new
           end
-          it "should define a getter and a setter which uses the return value of the block to set the instance variable" do
-            expect(instance.admin).to eq(nil)
-            instance.admin = "anything"
-            expect(instance.admin).to eq("return value")
+          it "should define a getter and not a setter" do
+            expect(instance).to respond_to(:name)
+            expect(instance).not_to respond_to(:name=)
           end
         end
       end
@@ -162,9 +314,7 @@ describe Aspect::HasAttributes do
       context "and no block is given" do
         context "and it's truthy" do
           let(:instance) do
-            instance_class = Class.new do
-              include Aspect::HasAttributes
+            instance_class = Class.new(TestObject) do
               attribute(:admin, query: true)
             end
@@ -180,9 +330,7 @@ describe Aspect::HasAttributes do
         context "and it's falsey" do
           let(:instance) do
-            instance_class = Class.new do
-              include Aspect::HasAttributes
+            instance_class = Class.new(TestObject) do
               attribute(:admin, query: false)
             end
@@ -200,9 +348,7 @@ describe Aspect::HasAttributes do
       context "and a block is given" do
         context "and it's truthy" do
           let(:instance) do
-            instance_class = Class.new do
-              include Aspect::HasAttributes
+            instance_class = Class.new(TestObject) do
               attribute(:moderator, query: true)
               attribute(:admin, query: true) { |value| @moderator && value }
             end
@@ -224,9 +370,7 @@ describe Aspect::HasAttributes do
         context "and it's falsey" do
           let(:instance) do
-            instance_class = Class.new do
-              include Aspect::HasAttributes
+            instance_class = Class.new(TestObject) do
               attribute(:admin, query: false) { "return value" }
             end
@@ -245,9 +389,7 @@ describe Aspect::HasAttributes do
   describe "#update_attributes" do
     let(:instance) do
-      class_instance = Class.new do
-        include Aspect::HasAttributes
+      class_instance = Class.new(TestObject) do
         attr_accessor :name
         attr_accessor :age
       end
@@ -262,4 +404,42 @@ describe Aspect::HasAttributes do
       expect(instance.age).to eq(123)
     end
   end
+  context "when included from the method", pending: true do
+    context "and the :method option is passed" do
+      context "and the values are not nil" do
+        let(:instance) do
+          class_instance = Class.new do
+            include Aspect::HasAttributes(method: { define: :atr, update: :mass_assign })
+          end
+          class_instance.new
+        end
+        it "should update the attributes on the instance" do
+          expect(instance.class).to respond_to("atr")
+          expect(instance.class).not_to respond_to("attribute")
+          expect(instance).to respond_to("mass_assign")
+          expect(instance).not_to respond_to("update_attributes")
+        end
+      end
+      context "and the values are nil" do
+        let(:instance) do
+          class_instance = Class.new do
+            include Aspect::HasAttributes(method: { define: nil, update: nil })
+          end
+          class_instance.new
+        end
+        it "should update the attributes on the instance" do
+          expect(instance.class).not_to respond_to("atr")
+          expect(instance.class).not_to respond_to("attribute")
+          expect(instance).not_to respond_to("mass_assign")
+          expect(instance).not_to respond_to("update_attributes")
+        end
+      end
+    end
+  end
 end