Merge pull request #250 from jekyll/deprecate-documents

Deprecate documents endpoint completely

Author: mertkahyaoglu

lib/jekyll-admin/server/collection.rb

@@ -10,13 +10,6 @@ class Server < Sinatra::Base
         json collection.to_api(:include_documents => true)
       end
 
-      get "/:collection_id/documents" do
-        ensure_collection
-        msg = "The `documents` endpoint is deprecated and may be removed at any time."
-        Jekyll::Deprecator.deprecation_message msg
-        json collection.docs.map(&:to_api)
-      end
-
       get "/:collection_id/*" do
         ensure_document
         json document.to_api(:include_content => true)

spec/jekyll-admin/server/collection_spec.rb

@@ -57,40 +57,24 @@ def app
       expect(first_document).to_not have_key("raw_content")
       expect(first_document).to_not have_key("content")
     end
-  end
-
-  context "document index" do
-    it "returns collection documents" do
-      get "/collections/posts/documents"
-      expect(last_response).to be_ok
-      expect(first_response["title"]).to eq("Test")
-    end
 
     it "includes front matter defaults" do
-      get "/collections/posts/documents"
+      get "/collections/posts"
       expect(last_response).to be_ok
-      expect(first_response.key?("some_front_matter")).to eq(true)
+      expect(first_document.key?("some_front_matter")).to eq(true)
     end
 
     it "doesn't include the raw front matter" do
-      get "/collections/posts/documents"
-      expect(last_response).to be_ok
-      expect(first_response).to_not have_key("front_matter")
-    end
-
-    it "doesn't include content in the index" do
-      get "/collections/posts/documents"
+      get "/collections/posts"
       expect(last_response).to be_ok
-      expect(first_response).to_not have_key("content")
-      expect(first_response).to_not have_key("raw_content")
-      expect(first_response).to_not have_key("output")
+      expect(first_document).to_not have_key("front_matter")
     end
 
-    it "doesn't include next/previous in the index" do
-      get "/collections/posts/documents"
+    it "doesn't include next/previous" do
+      get "/collections/posts"
       expect(last_response).to be_ok
-      expect(first_response).to_not have_key("next")
-      expect(first_response).to_not have_key("previous")
+      expect(first_document).to_not have_key("next")
+      expect(first_document).to_not have_key("previous")
     end
   end