From deedf5036d90336c4b3b2097564c33422bbee47d Mon Sep 17 00:00:00 2001
From: Rene Saarsoo <nene@triin.net>
Date: Thu, 1 Aug 2013 08:58:22 +0300
Subject: [PATCH] Extract (required) specs to separate file.

---
 spec/aggregator_attributes_spec.rb | 43 -----------------------
 spec/aggregator_required_spec.rb   | 55 ++++++++++++++++++++++++++++++
 2 files changed, 55 insertions(+), 43 deletions(-)
 create mode 100644 spec/aggregator_required_spec.rb

diff --git a/spec/aggregator_attributes_spec.rb b/spec/aggregator_attributes_spec.rb
index 7ed79cff..1b8ff4fd 100644
--- a/spec/aggregator_attributes_spec.rb
+++ b/spec/aggregator_attributes_spec.rb
@@ -72,49 +72,6 @@ describe JsDuck::Aggregator do
     end
   end
 
-  describe "a normal config option" do
-    before do
-      @doc = parse_member(<<-EOS)
-        /**
-         * @cfg foo Something
-         */
-      EOS
-    end
-    it "is not required by default" do
-      @doc[:required].should_not == true
-    end
-  end
-
-  describe "a config option labeled as required" do
-    before do
-      @doc = parse_member(<<-EOS)
-        /**
-         * @cfg foo (required) Something
-         */
-      EOS
-    end
-    it "has required flag set to true" do
-      @doc[:required].should == true
-    end
-  end
-
-  describe "a class with @cfg (required)" do
-    before do
-      @doc = parse(<<-EOS)["MyClass"]
-        /**
-         * @class MyClass
-         * @cfg foo (required)
-         */
-      EOS
-    end
-    it "doesn't become a required class" do
-      @doc[:required].should_not == true
-    end
-    it "contains required config" do
-      @doc[:members][0][:required].should == true
-    end
-  end
-
   describe "member with @deprecated" do
     before do
       @deprecated = parse_member(<<-EOS)[:deprecated]
diff --git a/spec/aggregator_required_spec.rb b/spec/aggregator_required_spec.rb
new file mode 100644
index 00000000..b110f7fe
--- /dev/null
+++ b/spec/aggregator_required_spec.rb
@@ -0,0 +1,55 @@
+require "mini_parser"
+
+describe JsDuck::Aggregator do
+  def parse(string)
+    Helper::MiniParser.parse(string)
+  end
+
+  def parse_member(string)
+    parse(string)["global"][:members][0]
+  end
+
+  describe "a normal config option" do
+    before do
+      @doc = parse_member(<<-EOS)
+        /**
+         * @cfg foo Something
+         */
+      EOS
+    end
+    it "is not required by default" do
+      @doc[:required].should_not == true
+    end
+  end
+
+  describe "a config option labeled as required" do
+    before do
+      @doc = parse_member(<<-EOS)
+        /**
+         * @cfg foo (required) Something
+         */
+      EOS
+    end
+    it "has required flag set to true" do
+      @doc[:required].should == true
+    end
+  end
+
+  describe "a class with @cfg (required)" do
+    before do
+      @doc = parse(<<-EOS)["MyClass"]
+        /**
+         * @class MyClass
+         * @cfg foo (required)
+         */
+      EOS
+    end
+    it "doesn't become a required class" do
+      @doc[:required].should_not == true
+    end
+    it "contains required config" do
+      @doc[:members][0][:required].should == true
+    end
+  end
+
+end
-- 
GitLab