seiten 0.0.8 → 1.0.0

data/test/fixtures/breadcrumb.html

@@ -0,0 +1 @@
+<ul class="breadcrumb"><li class="breadcrumb__item"><a href="/our-team">About</a></li><li class="breadcrumb__item"><span class="breadcrumb__separator">&gt;</span><a href="/about/our-team">Our Team</a></li><li class="breadcrumb__item breadcrumb__item--current"><span class="breadcrumb__separator">&gt;</span><a href="/about/our-team/switzerland">Switzerland</a></li></ul>

data/test/fixtures/navigation.html

@@ -0,0 +1 @@
+<ul class="navigation"><li class="navigation__item navigation__item--active navigation__item--current test-class" data-behaviour="hello"><a href="/">Home</a></li><li class="navigation__item"><a href="/blog">Blog</a></li><li class="navigation__item navigation__item--parent"><a href="/products">Products</a><ul class="navigation__nodes"><li class="navigation__item"><a href="/products/logo-design">Logo Design</a></li><li class="navigation__item"><a href="/products/web-development">Web Development</a></li><li class="navigation__item"><a href="/contact">Hire us</a></li></ul></li><li class="navigation__item navigation__item--parent"><a href="/our-team">About</a><ul class="navigation__nodes"><li class="navigation__item navigation__item--parent"><a href="/about/our-team">Our Team</a></li><li class="navigation__item"><a href="/about/works">Works</a></li><li class="navigation__item navigation__item--parent"><a href="/about/partners">Partners</a></li></ul></li><li class="navigation__item"><a href="/contact">Contact</a></li><li class="navigation__item"><a href="/localization">I18n</a></li></ul>

data/test/integration/layout_test.rb

@@ -1,26 +1,25 @@
 require 'test_helper'
 
 class LayoutTest < ActionDispatch::IntegrationTest
-
   def test_returns_default_layout
-    visit "/about/works"
-    assert has_content?("Default Layout"), "not rendered default layout"
+    get '/about/works'
+    assert_select 'p', 'Default Layout'
   end
 
   def test_returns_home_layout
-    visit "/"
-    assert has_content?("Home Layout"), "not rendered home layout"
+    get '/'
+    assert_select 'p', 'Home Layout'
   end
 
   def test_returns_inherited_layout
-    visit "/products/logo-design"
-    assert has_content?("Products Layout"), "not rendered inherited layout"
+    get '/products/logo-design'
+    assert_select 'p', 'Products Layout'
   end
 
   def test_returns_no_inherited_layout
-    visit "/about/our-team/switzerland"
-    assert has_content?("Default Layout"), "not rendered default layout"
-    visit "/contact"
-    assert has_content?("Default Layout"), "not rendered default layout"
+    get '/about/our-team/switzerland'
+    assert_select 'p', 'Default Layout'
+    get '/contact'
+    assert_select 'p', 'Default Layout'
   end
 end

data/test/integration/navigation_test.rb

@@ -1,69 +1,69 @@
 require 'test_helper'
 
 class NavigationTest < ActionDispatch::IntegrationTest
-
-  def test_returns_2_level_deep_navigation
-    visit "/"
-    assert has_content?("Home")
-    assert has_content?("Products")
-    assert has_content?("Logo Design")
-    assert has_content?("Web Development")
-    assert has_content?("Hire us")
-    assert has_content?("About")
-    assert has_content?("Our Team")
-    assert has_content?("Works")
-    assert has_content?("Partners")
-    assert has_content?("Contact")
+  def test_should_ignore_active_storage_routes
+    assert_raise ActionController::RoutingError do
+      get '/rails/active_storage/test'
+    end
   end
 
-  def test_does_not_return_3_level_deep_pages
-    visit "/"
-    assert !has_content?("Daniel Puglisi")
-    assert !has_content?("Codegestalt")
-    assert !has_content?("Kreatify")
+  def test_should_raise_seiten_routing_error
+    assert_raise Seiten::Errors::RoutingError do
+      get '/non-existing-route'
+    end
   end
 
-  def test_returns_home_url
-    visit "/contact"
-    assert_equal "/contact", current_path
-    click_link "Home"
-    assert_equal "/", current_path
+  def test_returns_2_level_deep_navigation
+    get '/'
+    assert_select 'a', 'Home'
+    assert_select 'a', 'Products'
+    assert_select 'a', 'Logo Design'
+    assert_select 'a', 'Web Development'
+    assert_select 'a', 'Hire us'
+    assert_select 'a', 'About'
+    assert_select 'a', 'Our Team'
+    assert_select 'a', 'Works'
+    assert_select 'a', 'Partners'
+    assert_select 'a', 'Contact'
+    assert_select 'a', { count: 0, text: 'Daniel Puglisi' }
+    assert_select 'a', { count: 0, text: 'Codegestalt' }
+    assert_select 'a', { count: 0, text: 'Kreatify' }
+    assert_select '.navigation__item--active', count: 1
+    assert_select 'title', 'My awesome Web Agency'
   end
 
   def test_returns_sub_navigation
-    visit "/about/partners"
-    assert has_content?("Daniel Puglisi")
-    assert has_content?("Codegestalt")
-    assert has_content?("Kreatify")
+    get '/about/partners'
+    assert_select 'a', 'Daniel Puglisi'
+    assert_select 'a', 'Codegestalt'
+    assert_select 'a', 'Kreatify'
+    assert_select '.navigation__item--active', count: 2
+    assert_select 'title', 'Partners'
   end
 
   def test_returns_breadcrumb_navigation
-    visit "/about/our-team/switzerland"
-    assert has_content?("> About")
-    assert has_content?("> Our Team")
-    assert has_content?("> Switzerland")
-  end
-
-  def test_returns_clicked_breadcrumb_page
-    visit "/about/our-team/switzerland"
-    find(".breadcrumb a", text: "Our Team").click
-    assert_equal "/about/our-team", current_path
+    get '/about/our-team/switzerland'
+    assert_select '.breadcrumb a', 'About'
+    assert_select '.breadcrumb a', 'Our Team'
+    assert_select '.breadcrumb a', 'Switzerland'
+    assert_select '.navigation__item--active', count: 2
+    assert_select '.breadcrumb__item--current', count: 1
+    assert_select 'title', 'Switzerland'
   end
 
-  def test_returns_clicked_navigation_page
-    visit "/"
-    click_link "Web Development"
-    assert_equal "/products/web-development", current_path
-  end
-
-  def test_returns_external_links
-    visit "/about/partners"
-    assert has_xpath?("//a[@href='http://danielpuglisi.com']")
+  def test_returns_page_with_navigation_which_is_not_defined_in_navigation_config
+    get '/secret'
+    assert_response :success
+    assert_select 'p', 'pages#secret'
+    assert_select '.navigation__item--active', count: 0
+    assert_select 'title', nil
   end
 
-  def test_returns_page_with_navigation_which_is_not_defined_in_navigation_config
-    visit "/secret"
-    assert_equal 200, status_code
-    has_content?("This is a secret page")
+  def test_returns_current_page_of_custom_controller
+    get '/blog'
+    assert_select 'title', 'Blog'
+    assert_select 'p', 'posts#index'
+    assert_select '.navigation__item--active', count: 1
+    assert_select '.navigation__item--active a', text: 'Blog'
   end
 end

data/test/lib/seiten/helpers/frontend_test.rb

@@ -0,0 +1,37 @@
+require 'test_helper'
+
+class Seiten::Helpers::FrontendTest < ActionView::TestCase
+  include Seiten::Helpers::Frontend
+
+  def navigation
+    @navigation ||= Seiten::Navigation.find_by(id: 'application.en')
+  end
+
+  def current_navigation
+    @current_navigation ||= navigation
+  end
+
+  def current_page
+    @current_page
+  end
+
+  test '#link_to_seiten_page' do
+    page = current_navigation.pages.find_by(slug: 'about/our-team')
+
+    assert_equal '<a href="/about/our-team">Our Team</a>', link_to_seiten_page(page)
+  end
+
+  test '#seiten_navigation' do
+    @current_page = current_navigation.pages.find_by(slug: '')
+    html = File.read(File.join(FIXTURES_DIR, 'navigation.html')).strip
+
+    assert_equal html, seiten_navigation
+  end
+
+  test '#seiten_breadcrumb' do
+    @current_page = current_navigation.pages.find_by(slug: 'about/our-team/switzerland')
+    html = File.read(File.join(FIXTURES_DIR, 'breadcrumb.html')).strip
+
+    assert_equal html, seiten_breadcrumb
+  end
+end

data/test/lib/seiten/html/breadcrumb_test.rb

@@ -0,0 +1,29 @@
+require 'test_helper'
+
+class Seiten::HTML::BreadcrumbTest < ActiveSupport::TestCase
+  def context
+    controller = ApplicationController.new
+    controller.view_context.tap do |context|
+      context.instance_eval do
+        def url_options
+          { host: 'localhost:3000' }
+        end
+      end
+    end
+  end
+
+  def navigation
+    @navigation ||= Seiten::Navigation.find_by(id: 'application.en')
+  end
+
+  def current_page
+    @current_page ||= navigation.pages.find_by(slug: 'about/our-team/switzerland')
+  end
+
+  test '::new' do
+    nav = Seiten::HTML::Breadcrumb.new(context, page: current_page)
+    html = File.read(File.join(FIXTURES_DIR, 'breadcrumb.html')).strip
+
+    assert_equal html, nav.body
+  end
+end

data/test/lib/seiten/html/helpers_test.rb

@@ -0,0 +1,55 @@
+require 'test_helper'
+
+class Seiten::HTML::HelpersTest < ActiveSupport::TestCase
+
+  def navigation
+    @navigation ||= Seiten::Navigation.find_by(id: 'application.en')
+  end
+
+  def current_page
+    @current_page ||= navigation.pages.find(1)
+  end
+
+  def helpers
+    Seiten::HTML::Helpers
+  end
+
+  test '#build_page_modifiers' do
+    page_1 = navigation.pages.new(id: 99)
+    page_2 = navigation.pages.new(id: 200, parent_id: 99)
+    assert_equal [:parent], helpers.build_page_modifiers(page_1, nil)
+    assert_equal [], helpers.build_page_modifiers(page_2, nil)
+
+    assert_equal [:parent, :active, :current], helpers.build_page_modifiers(page_1, page_1)
+    assert_equal [], helpers.build_page_modifiers(page_2, page_1)
+
+    assert_equal [:parent, :active, :expanded], helpers.build_page_modifiers(page_1, page_2)
+    assert_equal [:active, :current], helpers.build_page_modifiers(page_2, page_2)
+  end
+
+  test '#build_classes' do
+    class_options = Seiten.config[:html][:navigation].dup
+    modifier_options = Seiten.config[:html][:modifier].dup
+
+    assert_equal 'navigation', helpers.build_classes(class_options: class_options, modifier_options: modifier_options)
+    assert_equal 'navigation__item', helpers.build_classes(:item, class_options: class_options, modifier_options: modifier_options)
+    assert_equal 'navigation__item navigation__item--active', helpers.build_classes(:item, modifiers: [:active], class_options: class_options, modifier_options: modifier_options)
+    assert_equal 'navigation__item navigation__item--active navigation__item--parent', helpers.build_classes(:item, modifiers: [:active, :parent], class_options: class_options, modifier_options: modifier_options)
+    assert_equal 'navigation__nodes navigation__nodes--active navigation__nodes--parent', helpers.build_classes(:nodes, modifiers: [:active, :parent], class_options: class_options, modifier_options: modifier_options)
+
+    # Works with custom html class config
+    class_options[:base] = 'navbar'
+    class_options[:item] = 'navbar-item'
+    class_options[:nodes] = 'navbar-dropdown'
+    modifier_options[:base] = 'is'
+    modifier_options[:separator] = '-'
+    assert_equal 'navbar', helpers.build_classes(class_options: class_options, modifier_options: modifier_options)
+    assert_equal 'navbar-item', helpers.build_classes(:item, class_options: class_options, modifier_options: modifier_options)
+    assert_equal 'navbar-item is-active', helpers.build_classes(:item, modifiers: [:active], class_options: class_options, modifier_options: modifier_options)
+    assert_equal 'navbar-item is-active is-parent', helpers.build_classes(:item, modifiers: [:active, :parent], class_options: class_options, modifier_options: modifier_options)
+    assert_equal 'navbar-dropdown is-active is-parent', helpers.build_classes(:nodes, modifiers: [:active, :parent], class_options: class_options, modifier_options: modifier_options)
+
+    # Merges extension-class
+    assert_equal 'navbar-item is-active is-parent extension-class', helpers.build_classes(:item, modifiers: [:active, :parent], merge: 'extension-class', class_options: class_options, modifier_options: modifier_options)
+  end
+end

data/test/lib/seiten/html/navigation_test.rb

@@ -0,0 +1,29 @@
+require 'test_helper'
+
+class Seiten::HTML::NavigationTest < ActiveSupport::TestCase
+  def context
+    controller = ApplicationController.new
+    controller.view_context.tap do |context|
+      context.instance_eval do
+        def url_options
+          { host: 'localhost:3000' }
+        end
+      end
+    end
+  end
+
+  def navigation
+    @navigation ||= Seiten::Navigation.find_by(id: 'application.en')
+  end
+
+  def current_page
+    @current_page ||= navigation.pages.find_by(slug: '')
+  end
+
+  test '::new' do
+    nav = Seiten::HTML::Navigation.new(context, navigation: navigation, current_page: current_page)
+    html = File.read(File.join(FIXTURES_DIR, 'navigation.html')).strip
+
+    assert_equal html, nav.body
+  end
+end

data/test/lib/seiten/navigation_test.rb

@@ -0,0 +1,45 @@
+require 'test_helper'
+
+class Seiten::NavigationTest < ActiveSupport::TestCase
+
+  def navigation
+    @navigation ||= Seiten::Navigation.new(
+      name: 'test',
+      locale: 'en'
+    )
+  end
+
+  test '::find_by' do
+    assert_equal 'application.en', Seiten::Navigation.find_by(name: 'application', locale: 'en').id
+    assert_equal 'application.de', Seiten::Navigation.find_by(name: 'application', locale: 'de').id
+    assert_equal 'application.en', Seiten::Navigation.find_by(locale: 'en').id
+    assert_equal 'application.de', Seiten::Navigation.find_by(locale: 'de').id
+  end
+
+  test '::where' do
+    assert_equal ['application.de', 'application.en'], Seiten::Navigation.where(name: 'application').map(&:id).sort
+  end
+
+
+  test '#id' do
+    nav = Seiten::Navigation.new(name: 'site', locale: 'en')
+    assert_equal 'site.en', nav.id
+  end
+
+  test '#pages' do
+    nav = Seiten::Navigation.new(name: 'test', locale: 'en')
+    pages = Seiten::PageCollection.new
+    nav.page_collection = pages
+
+    assert_equal pages, nav.pages
+  end
+
+  test '#pages=' do
+    page = Seiten::Page.new
+
+    navigation.pages = [page]
+
+    assert_equal page.navigation_id, navigation.id
+    assert_equal [page], navigation.pages.all
+  end
+end

data/test/lib/seiten/page_collection_builder_test.rb

@@ -0,0 +1,141 @@
+require 'test_helper'
+
+class Seiten::PageCollectionBuilderTest < ActiveSupport::TestCase
+
+  def page_collection
+    @page_collection ||= Seiten::PageCollection.new
+  end
+
+  def options
+    @options ||= Hash[:pages, []]
+  end
+
+  test 'should build root page' do
+    options[:pages] << { "title" => "Home", "root" => true }
+    Seiten::PageCollectionBuilder.call(page_collection, options)
+
+    assert_equal "Home", page_collection.find_by(slug: '').title
+  end
+
+  test 'should build page with a defined url' do
+    options[:pages] << { "title" => "About us", "url" => 'about' }
+    Seiten::PageCollectionBuilder.call(page_collection, options)
+
+    assert_equal "About us", page_collection.find_by(slug: 'about').title
+  end
+
+  test 'should build a child page with a defined url' do
+    options[:pages] << { "title" => "About us", "nodes" => [{ "title" => "Team", "url" => "/team" }] }
+    Seiten::PageCollectionBuilder.call(page_collection, options)
+
+    assert_equal "Team", page_collection.find_by(slug: 'team').title
+  end
+
+  test 'should build page without an url' do
+    options[:pages] << { "title" => "About us", "url" => false }
+    Seiten::PageCollectionBuilder.call(page_collection, options)
+
+    assert_equal "About us", page_collection.where(slug: nil).first.title
+  end
+
+  test 'should build child page' do
+    options[:pages] << { "title" => "About us", "nodes" => [{ "title" => "Team" }] }
+    Seiten::PageCollectionBuilder.call(page_collection, options)
+
+    assert_equal "About us", page_collection.find_by(slug: 'about-us').title
+    assert_equal "Team", page_collection.find_by(slug: 'about-us/team').title
+  end
+
+  test 'should build child page of child page' do
+    options[:pages] << { "title" => "About us", "nodes" => [{ "title" => "Team", "nodes" => [{ "title" => "Management" }] }] }
+    Seiten::PageCollectionBuilder.call(page_collection, options)
+
+    parent = page_collection.find_by(slug: 'about-us')
+    child_1 = page_collection.find_by(slug: 'about-us/team')
+    child_2 = page_collection.find_by(slug: 'about-us/team/management')
+
+    assert_equal "About us", parent.title
+    assert_equal "Team", child_1.title
+    assert_equal "Management", child_2.title
+
+    assert_nil parent.parent_id
+    assert_equal parent.id, child_1.parent_id
+    assert_equal child_1.id, child_2.parent_id
+  end
+
+  test 'should build child page without parent url' do
+    options[:pages] << { "title" => "About us", "url" => false, "nodes" => [{ "title" => "Team" }] }
+    Seiten::PageCollectionBuilder.call(page_collection, options)
+
+    assert_equal "About us", page_collection.where(slug: nil).first.title
+    assert_equal "Team", page_collection.find_by(slug: 'team').title
+  end
+
+  test 'should build a page that refers to its first child' do
+    options[:pages] << { "title" => "About us", "refer" => true, "nodes" => [{ "title" => "Team" }] }
+    Seiten::PageCollectionBuilder.call(page_collection, options)
+
+    assert_equal "/team", page_collection.where(slug: nil).first.refer
+  end
+
+  test 'should build a referer page with an absolute path' do
+    options[:pages] << { "title" => "About us", "refer" => "/team" }
+    Seiten::PageCollectionBuilder.call(page_collection, options)
+
+    assert_equal "/team", page_collection.where(slug: nil).first.refer
+  end
+
+  test 'should build a referer page with an external path' do
+    options[:pages] << { "title" => "About us", "refer" => "https://codegestalt.com" }
+    Seiten::PageCollectionBuilder.call(page_collection, options)
+
+    assert_equal "https://codegestalt.com", page_collection.where(slug: nil).first.refer
+  end
+
+  test 'should build a page with default layout' do
+    options[:pages] << { "title" => "About us" }
+    Seiten::PageCollectionBuilder.call(page_collection, options)
+
+    assert_equal "application", page_collection.find_by(slug: 'about-us').layout
+  end
+
+  test 'should build a page with a layout' do
+    options[:pages] << { "title" => "About us", "layout" => "team", "nodes" => [{ "title" => "Team" }] }
+    Seiten::PageCollectionBuilder.call(page_collection, options)
+
+    assert_equal "team", page_collection.find_by(slug: 'about-us').layout
+    assert_equal "team", page_collection.find_by(slug: 'about-us/team').layout
+  end
+
+  test 'should build a page with an inherited layout' do
+    options[:pages] << { "title" => "About us", "layout" => { "name" => "team", "inherit" => true }, "nodes" => [{ "title" => "Team" }] }
+    Seiten::PageCollectionBuilder.call(page_collection, options)
+
+    assert_equal "team", page_collection.find_by(slug: 'about-us').layout
+    assert_equal "team", page_collection.find_by(slug: 'about-us/team').layout
+  end
+
+  test 'should build a page without inheriting the layout' do
+    options[:pages] << { "title" => "About us", "layout" => { "name" => "team", "inherit" => false }, "nodes" => [{ "title" => "Team" }] }
+    Seiten::PageCollectionBuilder.call(page_collection, options)
+
+    assert_equal "team", page_collection.find_by(slug: 'about-us').layout
+    assert_equal "application", page_collection.find_by(slug: 'about-us/team').layout
+  end
+
+  test 'should raise page refer exception' do
+    options[:pages] << { "title" => 'Refer', "refer" => 'home' }
+    err = assert_raises Seiten::Errors::PageError do
+      Seiten::PageCollectionBuilder.call(page_collection, options)
+    end
+    assert_match "The `refer` option must be `true` or an absolute or external path", err.message
+  end
+
+  test 'should raise page url exception' do
+    options[:pages] << { "title" => 'External path', "url" => 'https://codegestalt.com' }
+    err = assert_raises Seiten::Errors::PageError do
+      Seiten::PageCollectionBuilder.call(page_collection, options)
+    end
+    assert_match "The `url` option can not be an external path. Use the `refer` option to link to external resources.", err.message
+  end
+end

data/test/lib/seiten/page_collection_test.rb

@@ -0,0 +1,59 @@
+require 'test_helper'
+
+class Seiten::PageCollectionTest < ActiveSupport::TestCase
+  def navigation
+    @navigation ||= Seiten::Navigation.new(name: :test, locale: :en)
+  end
+
+  def page_collection
+    @page_collection ||= Seiten::PageCollection.new(
+      navigation_id: 'test.en',
+      pages: [
+        Seiten::Page.new(navigation_id: 'test.en', id: 1),
+        Seiten::Page.new(navigation_id: 'test.en', id: 2, parent_id: 1),
+        Seiten::Page.new(navigation_id: 'test.en', id: 3, parent_id: 1),
+        Seiten::Page.new(navigation_id: 'test.en', id: 4),
+        Seiten::Page.new(navigation_id: 'test.en', id: 5)
+      ]
+    )
+  end
+
+  setup do
+    Seiten.navigations << navigation
+  end
+
+  teardown do
+    Seiten.navigations = Seiten.navigations - [navigation]
+  end
+
+  test '#navigation' do
+    assert_equal navigation, page_collection.navigation
+  end
+
+  test '#all' do
+    assert_equal [1,2,3,4,5], page_collection.all.map(&:id)
+  end
+
+  test '#find' do
+    assert_equal 3, page_collection.find(3).id
+  end
+
+  test '#find_by' do
+    assert_equal 2, page_collection.find_by(id: 2).id
+    assert_equal 3, page_collection.find_by(id: 3, parent_id: 1).id
+    assert_nil page_collection.find_by(id: 2, parent_id: 2)
+  end
+
+  test '#where' do
+    assert_equal [1], page_collection.where(id: 1).map(&:id)
+    assert_equal [2,3], page_collection.where(parent_id: 1).map(&:id)
+    assert_equal [], page_collection.where(parent_id: 2)
+  end
+
+  test '#new' do
+    page = page_collection.new(id: 6)
+    assert_equal 6, page.id
+    assert_equal 'test.en', page.navigation_id
+    assert_equal [1,2,3,4,5,6], page_collection.all.map(&:id)
+  end
+end