archangel 0.3.0 → 0.4.0

data/spec/features/frontend/liquid/variables/site_spec.rb

@@ -0,0 +1,46 @@
+# frozen_string_literal: true
+
+require "rails_helper"
+
+RSpec.describe "Liquid custom variables", type: :feature do
+  let(:site) { create(:site, name: "Site A", locale: "en") }
+
+  let(:resource_content) do
+    %(
+      Site Name: {{ site.name }}
+      Site Locale: {{ site.locale }}
+      Site Logo: {{ site.logo }}
+      Site Unknown: >{{ site.unknown_variable }}<
+    )
+  end
+
+  before do
+    create(:page, site: site, slug: "amazing", content: resource_content)
+  end
+
+  describe "for `site` variable object" do
+    it "knows the `site` name" do
+      visit "/amazing"
+
+      expect(page).to have_content("Site Name: Site A")
+    end
+
+    it "knows the `site` locale" do
+      visit "/amazing"
+
+      expect(page).to have_content("Site Locale: en")
+    end
+
+    it "knows the `site` logo" do
+      visit "/amazing"
+
+      expect(page).to have_content("Site Logo: #{site.logo}")
+    end
+
+    it "returns blank for unknown `site` properties" do
+      visit "/amazing"
+
+      expect(page).to have_content("Site Unknown: ><")
+    end
+  end
+end

data/spec/features/frontend/liquid/variables/unknown_spec.rb

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+require "rails_helper"
+
+RSpec.describe "Liquid custom variables", type: :feature do
+  let(:site) { create(:site) }
+
+  describe "for unknown variables" do
+    it "responds with blank value" do
+      create(:page, site: site,
+                    slug: "amazing",
+                    content: "Unknown Variable: ~{{ unknown_variable }}~")
+
+      visit "/amazing"
+
+      expect(page).to have_content("Unknown Variable: ~~")
+    end
+  end
+end

data/spec/features/frontend/metatags/default_spec.rb

@@ -0,0 +1,46 @@
+# frozen_string_literal: true
+
+require "rails_helper"
+
+RSpec.describe "Meta tag", type: :feature do
+  describe "default metatags" do
+    let(:site) { create(:site, name: "Site A") }
+
+    it "contains canonical meta tag" do
+      create(:page, site: site, slug: "amazing")
+
+      visit "/amazing"
+
+      canonical = "link[rel='canonical'][href='#{page.current_host}/amazing']"
+      expect(page).to have_css(canonical, visible: false)
+    end
+
+    it "contains default title format (e.g. Page Name | Site Name)" do
+      create(:page, site: site, slug: "amazing", title: "Page A")
+
+      visit "/amazing"
+
+      expect(page).to have_title("Page A | Site A")
+    end
+
+    it "always uses Page.title for title (doesn't allow metatag overwrite)" do
+      resource = create(:page, site: site, slug: "amazing", title: "Page A")
+      create(:metatag, metatagable: resource,
+                       name: "title", content: "Attempted Page Title")
+
+      visit "/amazing"
+
+      expect(page).to have_title("Page A | Site A")
+    end
+
+    it "does not use custom meta tag title (doesn't allow metatag overwrite)" do
+      resource = create(:page, site: site, slug: "amazing", title: "Page A")
+      create(:metatag, metatagable: resource,
+                       name: "title", content: "Attempted Page Title")
+
+      visit "/amazing"
+
+      expect(page).not_to have_title("Attempted Page Title | Site A")
+    end
+  end
+end

data/spec/features/frontend/metatags/page_spec.rb

@@ -0,0 +1,58 @@
+# frozen_string_literal: true
+
+require "rails_helper"
+
+RSpec.describe "Meta tag", type: :feature do
+  describe "Page metatags" do
+    let(:site) { create(:site, name: "Site A") }
+    let(:resource) do
+      create(:page, site: site, slug: "amazing", title: "Page A")
+    end
+
+    before do
+      create(:metatag, metatagable: site,
+                       name: "description",
+                       content: "Site description")
+      create(:metatag, metatagable: site,
+                       name: "author",
+                       content: "Archangel")
+
+      create(:metatag, metatagable: resource,
+                       name: "description",
+                       content: "Page description")
+      create(:metatag, metatagable: resource,
+                       name: "keywords",
+                       content: "useful,page,keywords")
+    end
+
+    it "contains Page description meta tag" do
+      visit "/amazing"
+
+      expect(page).to have_meta(:description, "Page description")
+    end
+
+    it "does not use Site description when Page description meta tag used" do
+      visit "/amazing"
+
+      expect(page).not_to have_meta(:description, "Site description")
+    end
+
+    it "contains Page keywords meta tag" do
+      visit "/amazing"
+
+      expect(page).to have_meta(:keywords, "useful,page,keywords")
+    end
+
+    it "contains Site author meta tag" do
+      visit "/amazing"
+
+      expect(page).to have_meta(:author, "Archangel")
+    end
+
+    it "contains correct Page title (e.g. Page Name | Site Name)" do
+      visit "/amazing"
+
+      expect(page).to have_title("Page A | Site A")
+    end
+  end
+end

data/spec/features/frontend/metatags/site_spec.rb

@@ -0,0 +1,42 @@
+# frozen_string_literal: true
+
+require "rails_helper"
+
+RSpec.describe "Meta tag", type: :feature do
+  describe "Site metatags" do
+    let(:site) { create(:site, name: "Site A") }
+
+    before do
+      create(:metatag, metatagable: site,
+                       name: "description",
+                       content: "Site description")
+      create(:metatag, metatagable: site,
+                       name: "author",
+                       content: "Archangel")
+    end
+
+    it "contains Site description meta tag" do
+      create(:page, site: site, slug: "amazing", title: "Page A")
+
+      visit "/amazing"
+
+      expect(page).to have_meta(:description, "Site description")
+    end
+
+    it "contains Site author meta tag" do
+      create(:page, site: site, slug: "amazing", title: "Page A")
+
+      visit "/amazing"
+
+      expect(page).to have_meta(:author, "Archangel")
+    end
+
+    it "contains the correct Page title (e.g. Page Name | Site Name)" do
+      create(:page, site: site, slug: "amazing", title: "Page A")
+
+      visit "/amazing"
+
+      expect(page).to have_title("Page A | Site A")
+    end
+  end
+end

data/spec/fixtures/schemas/backend/assets/wysiwyg_error.json

@@ -0,0 +1,11 @@
+{
+  "type" : "object",
+  "required" : [
+    "success",
+    "error"
+  ],
+  "properties" : {
+    "success" : { "type" : "boolean" },
+    "error" : { "type" : "string" }
+  }
+}

data/spec/fixtures/schemas/backend/assets/wysiwyg_url.json

@@ -0,0 +1,11 @@
+{
+  "type" : "object",
+  "required" : [
+    "success",
+    "url"
+  ],
+  "properties" : {
+    "success" : { "type" : "boolean" },
+    "url" : { "type" : "string" }
+  }
+}

data/spec/fixtures/schemas/frontend/errors/not_found.json

@@ -0,0 +1,11 @@
+{
+  "type" : "object",
+  "required" : [
+    "status",
+    "error"
+  ],
+  "properties" : {
+    "status" : { "type" : "integer" },
+    "error" : { "type" : "string" }
+  }
+}

data/spec/fixtures/schemas/frontend/pages/show.json

@@ -0,0 +1,25 @@
+{
+  "type": "object",
+  "required": ["page"],
+  "properties": {
+    "page" : {
+      "type" : "object",
+      "required" : [
+        "id",
+        "title",
+        "homepage",
+        "content",
+        "permalink",
+        "published_at"
+      ],
+      "properties" : {
+        "id" : { "type" : "integer" },
+        "title" : { "type" : "string" },
+        "homepage" : { "type" : "boolean" },
+        "content" : { "type" : "string" },
+        "permalink" : { "type" : "string" },
+        "published_at" : { "type" : "string", "format": "date-time" }
+      }
+    }
+  }
+}

data/spec/requests/backend/assets_spec.rb

@@ -0,0 +1,81 @@
+# frozen_string_literal: true
+
+require "rails_helper"
+
+RSpec.describe "Backend - Pages", type: :request do
+  before { stub_authorization!(create(:user)) }
+
+  describe "POST /backend/assets/wysiwyg(.json) (#wysiwyg)" do
+    describe "with valid attributes" do
+      it "responds with 200 status" do
+        post "/backend/assets/wysiwyg",
+             headers: { accept: "application/json" },
+             params: { file: fixture_file_upload(uploader_test_image) }
+
+        expect(response).to have_http_status(:ok)
+      end
+
+      it "responds with JSON schema" do
+        post "/backend/assets/wysiwyg",
+             headers: { accept: "application/json" },
+             params: { file: fixture_file_upload(uploader_test_image) }
+
+        expect(response).to match_response_schema("backend/assets/wysiwyg_url")
+      end
+
+      it "responds with success:true JSON node" do
+        post "/backend/assets/wysiwyg",
+             headers: { accept: "application/json" },
+             params: { file: fixture_file_upload(uploader_test_image) }
+
+        expect(json_response[:success]).to eq(true)
+      end
+
+      it "responds with url:string JSON node" do
+        post "/backend/assets/wysiwyg",
+             headers: { accept: "application/json" },
+             params: { file: fixture_file_upload(uploader_test_image) }
+
+        expect(json_response[:url])
+          .to start_with("/uploads/archangel/asset/file/")
+      end
+    end
+
+    describe "with invalid attributes" do
+      it "responds with 200 status" do
+        post "/backend/assets/wysiwyg",
+             headers: { accept: "application/json" },
+             params: { file: fixture_file_upload(uploader_test_stylesheet) }
+
+        expect(response).to have_http_status(:ok)
+      end
+
+      it "responds with error JSON schema" do
+        post "/backend/assets/wysiwyg",
+             headers: { accept: "application/json" },
+             params: { file: fixture_file_upload(uploader_test_stylesheet) }
+
+        expect(response)
+          .to match_response_schema("backend/assets/wysiwyg_error")
+      end
+
+      it "responds with success:false JSON node" do
+        post "/backend/assets/wysiwyg",
+             headers: { accept: "application/json" },
+             params: { file: fixture_file_upload(uploader_test_stylesheet) }
+
+        expect(json_response[:success]).to eq(false)
+      end
+
+      it "responds with error:string JSON node" do
+        post "/backend/assets/wysiwyg",
+             headers: { accept: "application/json" },
+             params: { file: fixture_file_upload(uploader_test_stylesheet) }
+
+        expected_error = "You are not allowed to upload \"css\" files"
+
+        expect(json_response[:error]).to include(expected_error)
+      end
+    end
+  end
+end

data/spec/requests/backend/designs_spec.rb

@@ -0,0 +1,115 @@
+# frozen_string_literal: true
+
+require "rails_helper"
+
+RSpec.describe "Backend - Designs", type: :request do
+  before { stub_authorization!(create(:user)) }
+
+  let(:valid_attributes) do
+    {
+      name: "New Design Name",
+      content: %(
+        <p>BEFORE DESIGN<p>
+        <p>{{ content_for_layout }}<p>
+        <p>AFTER DESIGN<p>
+      ),
+      partial: false
+    }
+  end
+
+  describe "GET /backend/designs/:id (#show)" do
+    let(:record) { create(:design) }
+
+    describe "with a valid Design id" do
+      it "returns a 200 status" do
+        get "/backend/designs/#{record.id}"
+
+        expect(response).to have_http_status(:ok)
+      end
+    end
+
+    describe "with an invalid Design id" do
+      it "returns 404 status" do
+        get "/backend/designs/0"
+
+        expect(response).to have_http_status(:not_found)
+      end
+
+      it "shows the error page" do
+        get "/backend/designs/0"
+
+        expect(response.body)
+          .to include("Page not found. Could not find what was requested")
+      end
+    end
+  end
+
+  describe "POST /backend/designs (#create)" do
+    describe "with valid attributes" do
+      it "returns 302 status" do
+        post "/backend/designs", params: { design: valid_attributes }
+
+        expect(response).to have_http_status(:found)
+      end
+
+      it "redirects to the listing" do
+        post "/backend/designs", params: { design: valid_attributes }
+
+        expect(response).to redirect_to("/backend/designs")
+      end
+    end
+  end
+
+  describe "GET /backend/designs/:id/edit (#edit)" do
+    let(:record) { create(:design) }
+
+    it "returns a 200 status" do
+      get "/backend/designs/#{record.id}/edit"
+
+      expect(response).to have_http_status(:ok)
+    end
+  end
+
+  describe "PATCH /backend/designs/:id (#update)" do
+    let(:record) { create(:design) }
+
+    describe "with valid attributes" do
+      it "redirects after updating resource" do
+        patch "/backend/designs/#{record.id}", params: {
+          design: valid_attributes
+        }
+
+        expect(response).to redirect_to("/backend/designs")
+      end
+    end
+
+    describe "with invalid attributes" do
+      it "fails without name" do
+        invalid_attributes = valid_attributes.merge(name: "")
+
+        patch "/backend/designs/#{record.id}", params: {
+          design: invalid_attributes
+        }
+
+        expect(response.body).to include("can&#39;t be blank")
+      end
+    end
+  end
+
+  describe "DELETE /backend/designs/:id (#destroy)" do
+    let(:record) { create(:design) }
+
+    it "redirects to the listing" do
+      delete "/backend/designs/#{record.id}"
+
+      expect(response).to redirect_to("/backend/designs")
+    end
+
+    it "ensures the Design no longer accessable" do
+      delete "/backend/designs/#{record.id}"
+      get "/backend/designs/#{record.id}"
+
+      expect(response).to have_http_status(:not_found)
+    end
+  end
+end