archangel 0.0.8 → 0.3.0

data/lib/archangel/liquid/tags/csrf_meta_tags_tag.rb

@@ -16,6 +16,7 @@ module Archangel
         # @param context [Object] the Liquid context
         # @return [String] the CSRF meta tags
         #
+        # :reek:UtilityFunction
         def render(context)
           context.registers[:view].csrf_meta_tags
         end

data/lib/archangel/liquid/tags/locale_tag.rb

@@ -16,6 +16,7 @@ module Archangel
         # @param context [Object] the Liquid context
         # @return [String] the locale
         #
+        # :reek:UtilityFunction
         def render(context)
           context.registers[:view].locale
         end

data/lib/archangel/liquid/tags/meta_tags_tag.rb

@@ -18,6 +18,7 @@ module Archangel
         # @param context [Object] the Liquid context
         # @return [String] the meta tags
         #
+        # :reek:UtilityFunction
         def render(context)
           context.registers[:view].display_meta_tags
         end

data/lib/archangel/liquid/tags/noembed_tag.rb

@@ -0,0 +1,140 @@
+# frozen_string_literal: true
+
+module Archangel
+  module Liquid
+    module Tags
+      ##
+      # Noembed custom tag for Liquid
+      #
+      # For more information about Noembed, see https://noembed.com/
+      #
+      # Example
+      #   {% noembed "https://www.youtube.com/watch?v=-X2atEH7nCg" %}
+      #   {% noembed "https://vimeo.com/70707344" %}
+      #   {% noembed "https://vimeo.com/70707344" remote: true %}
+      #   {% noembed "https://vimeo.com/70707344" maxwidth:600 %}
+      #   {% noembed "https://vimeo.com/70707344" maxheight:400 %}
+      #
+      class NoembedTag < ApplicationTag
+        require "digest/md5"
+        require "net/http"
+
+        ##
+        # Noembed for Liquid
+        #
+        # @param tag_name [String] the Liquid tag name
+        # @param markup [String] the passed options
+        # @param options [Object] options
+        #
+        def initialize(tag_name, markup, options)
+          super
+
+          match = URL_ATTRIBUTES_SYNTAX.match(markup)
+
+          if match.blank?
+            raise ::Liquid::SyntaxError, Archangel.t("errors.syntax.noembed")
+          end
+
+          @url = ::Liquid::Variable.new(match[:url], options).name
+          @attributes = { remote: false }
+
+          match[:attributes].scan(KEY_VALUE_ATTRIBUTES_SYNTAX) do |key, value|
+            @attributes[key.to_sym] = ::Liquid::Expression.parse(value)
+          end
+        end
+
+        ##
+        # Render the Widget
+        #
+        # @param _context [Object] the Liquid context
+        # @return [String] the rendered Widget
+        #
+        def render(_context)
+          return if url.blank?
+
+          noembed_id = "noembed-#{Digest::MD5.hexdigest(url)}"
+
+          content = if %w[1 true yes y].include?(attributes[:remote].to_s)
+                      build_js_request(noembed_id)
+                    else
+                      build_ruby_request
+                    end
+
+          content_tag(:div, raw(content), id: noembed_id, class: "noembed")
+        end
+
+        protected
+
+        attr_reader :attributes, :url
+
+        def build_ruby_request
+          uri = URI(noembed_url)
+          uri.query = URI.encode_www_form(query_params)
+
+          res = Net::HTTP.get_response(uri)
+
+          unless res.is_a?(Net::HTTPSuccess)
+            return "<span class='status error'>Error connecting to noembed " \
+                   "server.</span>"
+          end
+
+          resp = JSON.parse(res.body)
+
+          if resp.fetch("html", nil).blank?
+            message = resp["error"] || "Unknown error."
+            "<span class='status error'>#{message}</span>"
+          else
+            resp["html"]
+          end
+        end
+
+        def build_js_request(noembed_id)
+          noembed_request_url = [
+            noembed_url,
+            query_params.to_query
+          ].join("?")
+
+          <<-NOEMBED
+  <script type="text/javascript">
+    document.addEventListener("DOMContentLoaded", function() {
+      var noembedElement = document.querySelector('##{noembed_id}'),
+          noembedRequest = new XMLHttpRequest();
+      noembedElement.innerHTML = '<span class="status">Embedding&hellip;</span>';
+      noembedRequest.open('GET', '#{noembed_request_url}', true);
+      noembedRequest.setRequestHeader('Content-type', 'application/json');
+      noembedRequest.onload = function() {
+        var response = JSON.parse(noembedRequest.responseText)
+        if (response.html) {
+          noembedElement.innerHTML = response.html;
+        }
+        else {
+          noembedElement.innerHTML = '<span class="status error">' + (response.error || 'Unknown error.') + '</span>';
+        }
+      };
+      noembedRequest.onerror = function () {
+        noembedElement.innerHTML = '<span class="status error">Error connecting to noembed server.</span>';
+      };
+      noembedRequest.send(null);
+    });
+  </script>
+          NOEMBED
+        end
+
+        def noembed_url
+          "https://noembed.com/embed"
+        end
+
+        def query_params
+          {
+            url: url,
+            maxwidth: attributes.fetch(:maxwidth, nil),
+            maxheight: attributes.fetch(:maxheight, nil),
+            nowrap: attributes.fetch(:nowrap, "on")
+          }.compact.reject { |_, val| val.blank? }
+        end
+      end
+    end
+  end
+end
+
+::Liquid::Template.register_tag("noembed", Archangel::Liquid::Tags::NoembedTag)

data/lib/archangel/liquid/tags/text_direction_tag.rb

@@ -16,6 +16,7 @@ module Archangel
         # @param context [Object] the Liquid context
         # @return [String] the text direction
         #
+        # :reek:UtilityFunction
         def render(context)
           context.registers[:view].text_direction
         end

data/lib/archangel/liquid/tags/theme_javascript_tag.rb

@@ -16,6 +16,7 @@ module Archangel
         # @param context [Object] the Liquid context
         # @return [String] the Javascript for the theme
         #
+        # :reek:UtilityFunction
         def render(context)
           view = context.registers[:view]
 

data/lib/archangel/liquid/tags/theme_stylesheet_tag.rb

@@ -16,6 +16,7 @@ module Archangel
         # @param context [Object] the Liquid context
         # @return [String] the stylesheet for the theme
         #
+        # :reek:UtilityFunction
         def render(context)
           view = context.registers[:view]
 

data/lib/archangel/testing_support/factories/archangel_assets.rb

@@ -5,8 +5,8 @@ FactoryBot.define do
     site
     sequence(:file_name) { |n| "image-#{n}.jpg" }
     file { fixture_file_upload(uploader_test_image) }
-    content_type "image/jpg"
-    file_size 123
+    content_type { "image/jpg" }
+    file_size { 123 }
 
     trait :deleted do
       deleted_at { Time.current }

data/lib/archangel/testing_support/factories/archangel_collections.rb

@@ -12,7 +12,7 @@ FactoryBot.define do
 
     trait :with_fields do
       transient do
-        fields_count 3
+        fields_count { 3 }
       end
 
       after(:create) do |collection, evaluator|

data/lib/archangel/testing_support/factories/archangel_entries.rb

@@ -11,7 +11,7 @@ FactoryBot.define do
     end
 
     trait :unavailable do
-      available_at nil
+      available_at { nil }
     end
 
     trait :future do

data/lib/archangel/testing_support/factories/archangel_fields.rb

@@ -5,12 +5,12 @@ FactoryBot.define do
     collection
     sequence(:label) { |n| "Field #{n} Label" }
     sequence(:slug) { |n| "field-#{n}" }
-    value "Field default value"
-    classification "string"
-    required false
+    value { "Field default value" }
+    classification { "string" }
+    required { false }
 
     trait :required do
-      required true
+      required { true }
     end
 
     trait :deleted do

data/lib/archangel/testing_support/factories/archangel_metatags.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+FactoryBot.define do
+  factory :metatag, class: "Archangel::Metatag" do
+    association(:metatagable, factory: :page)
+    name { "description" }
+    content { "This is the description of this Page" }
+
+    trait :for_page do
+      association(:metatagable, factory: :page)
+      name { "description" }
+      content { "This is the description of the Page" }
+    end
+
+    trait :for_site do
+      association(:metatagable, factory: :site)
+      name { "description" }
+      content { "This is the description of the Site" }
+    end
+  end
+end

data/lib/archangel/testing_support/factories/archangel_pages.rb

@@ -3,13 +3,11 @@
 FactoryBot.define do
   factory :page, class: "Archangel::Page" do
     site
-    template nil
+    template { nil }
     sequence(:title) { |n| "Page #{n} Title" }
     sequence(:slug) { |n| "page-#{n}" }
-    content "<p>Content of the page</p>"
-    meta_keywords "default,keywords,of,my,page"
-    meta_description "Default description of my page"
-    homepage false
+    content { "<p>Content of the page</p>" }
+    homepage { false }
     published_at { Time.current }
 
     trait :with_parent do
@@ -21,11 +19,11 @@ FactoryBot.define do
     end
 
     trait :homepage do
-      homepage true
+      homepage { true }
     end
 
     trait :unpublished do
-      published_at nil
+      published_at { nil }
     end
 
     trait :future do

data/lib/archangel/testing_support/factories/archangel_sites.rb

@@ -5,9 +5,7 @@ FactoryBot.define do
     initialize_with { Archangel::Site.first_or_create }
 
     sequence(:name) { |n| "Site #{n} Name" }
-    locale "en"
-    meta_keywords "default,keywords,of,my,site"
-    meta_description "Default description of my site"
+    locale { "en" }
 
     trait :logo do
       logo { fixture_file_upload(uploader_test_image) }

data/lib/archangel/testing_support/factories/archangel_templates.rb

@@ -4,15 +4,15 @@ FactoryBot.define do
   factory :template, class: "Archangel::Template" do
     site
     sequence(:name) { |n| "Template #{n} Name" }
-    content <<-CONTENT
-      <p>BEFORE TEMPLATE<p>
-      <p>{{ content_for_layout }}<p>
-      <p>AFTER TEMPLATE<p>
-    CONTENT
-    partial false
+    content do
+      "<p>BEFORE TEMPLATE<p>
+       <p>{{ content_for_layout }}<p>
+       <p>AFTER TEMPLATE<p>"
+    end
+    partial { false }
 
     trait :partial do
-      partial true
+      partial { true }
     end
 
     trait :deleted do

data/lib/archangel/testing_support/factories/archangel_users.rb

@@ -6,7 +6,7 @@ FactoryBot.define do
     sequence(:name) { |n| "User #{n}" }
     sequence(:username) { |n| "username#{n}" }
     sequence(:email) { |n| "user#{n}@example.com" }
-    password "password"
+    password { "password" }
     sequence(:confirmation_token) { |n| "token-#{n}" }
     confirmed_at { Time.current }
     confirmation_sent_at { Time.current }
@@ -16,34 +16,34 @@ FactoryBot.define do
     end
 
     trait :admin do
-      role "admin"
+      role { "admin" }
     end
 
     trait :editor do
-      role "editor"
+      role { "editor" }
     end
 
     trait :unconfirmed do
-      confirmed_at nil
+      confirmed_at { nil }
     end
 
     trait :reset do
-      reset_password_token "abc123"
+      reset_password_token { "abc123" }
       reset_password_sent_at { Time.current }
     end
 
     trait :locked do
-      failed_attempts 10
-      unlock_token "123abc"
+      failed_attempts { 10 }
+      unlock_token { "123abc" }
       locked_at { Time.current }
     end
 
     trait :tracks do
-      sign_in_count 2
+      sign_in_count { 2 }
       current_sign_in_at { Time.current }
       last_sign_in_at { 1.day.ago }
-      current_sign_in_ip "127.0.0.1"
-      last_sign_in_ip "127.0.0.1"
+      current_sign_in_ip { "127.0.0.1" }
+      last_sign_in_ip { "127.0.0.1" }
     end
 
     trait :deleted do

data/lib/archangel/testing_support/factories/archangel_widgets.rb

@@ -3,10 +3,10 @@
 FactoryBot.define do
   factory :widget, class: "Archangel::Widget" do
     site
-    template nil
+    template { nil }
     sequence(:name) { |n| "Widget #{n}" }
     sequence(:slug) { |n| "widget-#{n}" }
-    content "<p>Content of the widget</p>"
+    content { "<p>Content of the widget</p>" }
 
     trait :with_template do
       association :template, factory: :template, partial: true

data/lib/archangel/testing_support/matchers/have_meta.rb

@@ -0,0 +1,18 @@
+# frozen_string_literal: true
+
+RSpec::Matchers.define :have_meta do |name, expected|
+  match do |_actual|
+    has_css?("meta[name='#{name}'][content='#{expected}']", visible: false)
+  end
+
+  failure_message do |_actual|
+    actual = first("meta[name='#{name}']")
+
+    if actual
+      "expected that meta #{name} would have content='#{expected}' " \
+      "but was '#{actual[:content]}'"
+    else
+      "expected that meta #{name} would exist with content='#{expected}'"
+    end
+  end
+end

data/lib/archangel/testing_support/matchers/have_title.rb

@@ -0,0 +1,18 @@
+# frozen_string_literal: true
+
+RSpec::Matchers.define :have_title do |expected|
+  match do |_actual|
+    has_css?("title", text: expected)
+  end
+
+  failure_message_for_should do |_actual|
+    actual = first("title")
+
+    if actual
+      "expected that title would have been '#{expected}' " \
+      "but was '#{actual.text}'"
+    else
+      "expected that title would exist with '#{expected}'"
+    end
+  end
+end

data/lib/archangel/testing_support/rake/dummy_rake.rb

@@ -25,7 +25,6 @@ namespace :dummy do
       "--auto-accept",
       "--migrate=false",
       "--seed=false",
-      "--sample=false",
       "--quiet"
     ]
 
@@ -48,7 +47,6 @@ namespace :dummy do
               "--auto-run-migrations",
               "--auto-accept",
               "--migrate",
-              "--skip-sample",
               "--skip-seed",
               "--skip-turbolinks",
               "--route-path=#{lib}",