archangel 0.3.0 → 0.4.0

data/spec/support/timecop.rb

@@ -0,0 +1,15 @@
+# frozen_string_literal: true
+
+require "timecop"
+
+RSpec.configure do |config|
+  config.include ActiveSupport::Testing::TimeHelpers
+
+  config.before do
+    Timecop.freeze
+  end
+
+  config.after do
+    Timecop.return
+  end
+end

data/spec/unit/helpers/archangel/application_helper_spec.rb

@@ -0,0 +1,65 @@
+# frozen_string_literal: true
+
+require "rails_helper"
+
+module Archangel
+  RSpec.describe ApplicationHelper, type: :helper do
+    before do
+      without_partial_double_verification do
+        allow(helper).to receive(:current_site).and_return(site)
+      end
+    end
+
+    let(:site) { create(:site) }
+
+    context "with #frontend_resource_path" do
+      it "returns the permalink with a string" do
+        expect(helper.frontend_resource_path("foo/bar")).to eq("/foo/bar")
+      end
+
+      it "returns the permalink with a Page resource" do
+        parent = create(:page, site: site, slug: "foo")
+        page = create(:page, site: site, parent: parent, slug: "bar")
+
+        expect(helper.frontend_resource_path(page)).to eq("/foo/bar")
+      end
+    end
+
+    context "with #locale" do
+      it "returns default application locale" do
+        expect(helper.locale).to eq("en")
+      end
+    end
+
+    context "with #text_direction" do
+      it "returns default `ltr` text direction" do
+        expect(helper.text_direction).to eq("ltr")
+      end
+
+      it "returns rtl text direction" do
+        without_partial_double_verification do
+          allow(helper).to receive(:text_direction).and_return("rtl")
+        end
+
+        expect(helper.text_direction).to eq("rtl")
+      end
+    end
+
+    context "with #active_backend_menu_for" do
+      before do
+        without_partial_double_verification do
+          allow(helper).to receive(:params)
+            .and_return(controller: "archangel/backend/pages")
+        end
+      end
+
+      it "returns `true` when in the current controller" do
+        expect(helper.active_backend_menu_for("pages")).to be_truthy
+      end
+
+      it "returns `false` when in not the current controller" do
+        expect(helper.active_backend_menu_for("users")).to be_falsey
+      end
+    end
+  end
+end

data/spec/unit/helpers/archangel/flash_helper_spec.rb

@@ -0,0 +1,41 @@
+# frozen_string_literal: true
+
+require "rails_helper"
+
+module Archangel
+  RSpec.describe FlashHelper, type: :helper do
+    context "with #flash_class_for(obj)" do
+      it "returns success class" do
+        expect(helper.flash_class_for("success")).to eq("success")
+      end
+
+      it "returns `error` class" do
+        expect(helper.flash_class_for("error")).to eq("danger")
+      end
+
+      it "returns `alert` class" do
+        expect(helper.flash_class_for("alert")).to eq("warning")
+      end
+
+      it "returns `notice` class" do
+        expect(helper.flash_class_for("notice")).to eq("info")
+      end
+
+      it "returns downcased for unknown class" do
+        expect(helper.flash_class_for("Unknown")).to eq("unknown")
+      end
+
+      it "returns with dashes in place of spaces for class" do
+        expect(helper.flash_class_for("foo  bar")).to eq("foo-bar")
+      end
+
+      it "returns with dashes in class" do
+        expect(helper.flash_class_for("foo--bar")).to eq("foo-bar")
+      end
+
+      it "returns with underscores in class" do
+        expect(helper.flash_class_for("foo_bar")).to eq("foo_bar")
+      end
+    end
+  end
+end

data/spec/{lib → unit/lib}/archangel/commands/extension_command_spec.rb

@@ -20,18 +20,23 @@ module Archangel
 
           after { after_generation(extension_path) }
 
-          it "should write common directories" do
-            %w[app bin config lib spec].each do |dir|
+          it "writes common directories" do
+            %w[
+              bin config lib spec
+            ].each do |dir|
               expect(glob_directories_in(extension_path)).to include(dir)
             end
           end
 
-          it "should write common files" do
-            %w[.editorconfig .gitignore .rspec .rubocop.yml Gemfile
-               MIT-LICENSE Rakefile README.md].each do |file|
+          it "writes common files" do
+            %w[
+              .gitignore .rspec Gemfile MIT-LICENSE Rakefile README.md
+            ].each do |file|
               expect(glob_files_in(extension_path)).to include(file)
             end
+          end
 
+          it "writes gemspec file" do
             expect(glob_files_in(extension_path))
               .to include("#{extension_full_name}.gemspec")
           end
@@ -46,7 +51,7 @@ module Archangel
 
           after { after_generation(extension_path) }
 
-          it "should write common files for `sample`" do
+          it "writes common files for `sample`" do
             expect(glob_files_in(extension_path))
               .to include("#{extension_full_name}.gemspec")
           end

data/spec/{lib → unit/lib}/archangel/commands/theme_command_spec.rb

@@ -20,18 +20,23 @@ module Archangel
 
           after { after_generation(theme_path) }
 
-          it "should write common directories" do
-            %w[app bin lib spec].each do |dir|
+          it "writes common directories" do
+            %w[
+              app bin lib spec
+            ].each do |dir|
               expect(glob_directories_in(theme_path)).to include(dir)
             end
           end
 
-          it "should write common files" do
-            %w[.editorconfig .gitignore .rspec .rubocop.yml Gemfile
-               MIT-LICENSE Rakefile README.md].each do |file|
+          it "writes common files" do
+            %w[
+              .gitignore .rspec Gemfile MIT-LICENSE Rakefile README.md
+            ].each do |file|
               expect(glob_files_in(theme_path)).to include(file)
             end
+          end
 
+          it "writes gemspec file" do
             expect(glob_files_in(theme_path))
               .to include("#{theme_full_name}.gemspec")
           end
@@ -46,7 +51,7 @@ module Archangel
 
           after { after_generation(theme_path) }
 
-          it "should write common files for `sample`" do
+          it "writes common files for `sample`" do
             expect(glob_files_in(theme_path))
               .to include("#{theme_full_name}.gemspec")
           end

data/spec/{models → unit/models}/archangel/asset_spec.rb

@@ -4,11 +4,7 @@ require "rails_helper"
 
 module Archangel
   RSpec.describe Asset, type: :model do
-    context "callbacks" do
-      it { is_expected.to callback(:save_asset_attributes).before(:save) }
-    end
-
-    context "validations" do
+    context "with validations" do
       it { is_expected.to validate_presence_of(:file) }
       it { is_expected.to validate_presence_of(:file_name) }
 
@@ -19,16 +15,12 @@ module Archangel
       it { is_expected.to allow_value("foo_bar.jpg").for(:file_name) }
       it { is_expected.to allow_value("18.jpg").for(:file_name) }
 
-      it { is_expected.to_not allow_value("error").for(:file_name) }
-      it { is_expected.to_not allow_value("error.j").for(:file_name) }
-      it { is_expected.to_not allow_value("with.dot.jpg").for(:file_name) }
-      it { is_expected.to_not allow_value("with space.jpg").for(:file_name) }
-      it { is_expected.to_not allow_value("with/slash.jpg").for(:file_name) }
-      it { is_expected.to_not allow_value("with/numbers.18").for(:file_name) }
-    end
-
-    context "associations" do
-      it { is_expected.to belong_to(:site) }
+      it { is_expected.not_to allow_value("error").for(:file_name) }
+      it { is_expected.not_to allow_value("error.j").for(:file_name) }
+      it { is_expected.not_to allow_value("with.dot.jpg").for(:file_name) }
+      it { is_expected.not_to allow_value("with space.jpg").for(:file_name) }
+      it { is_expected.not_to allow_value("with/slash.jpg").for(:file_name) }
+      it { is_expected.not_to allow_value("with/numbers.18").for(:file_name) }
     end
   end
 end

data/spec/{models → unit/models}/archangel/collection_spec.rb

@@ -4,13 +4,7 @@ require "rails_helper"
 
 module Archangel
   RSpec.describe Collection, type: :model do
-    context "callbacks" do
-      it { is_expected.to callback(:parameterize_slug).before(:validation) }
-
-      it { is_expected.to callback(:column_reset).after(:destroy) }
-    end
-
-    context "validations" do
+    context "with validations" do
       it { is_expected.to validate_presence_of(:name) }
       it { is_expected.to validate_presence_of(:slug) }
 
@@ -22,14 +16,7 @@ module Archangel
       end
     end
 
-    context "associations" do
-      it { is_expected.to belong_to(:site) }
-
-      it { is_expected.to have_many(:entries) }
-      it { is_expected.to have_many(:fields) }
-    end
-
-    context "#to_param" do
+    context "with #to_param" do
       it "uses `slug` as the identifier for routes" do
         resource = build(:collection, slug: "foo")
 
@@ -37,7 +24,7 @@ module Archangel
       end
     end
 
-    context "#column_reset" do
+    context "with #column_reset" do
       it "resets `slug` to `slug` + current time" do
         resource = create(:collection)
 

data/spec/unit/models/archangel/design_spec.rb

@@ -0,0 +1,22 @@
+# frozen_string_literal: true
+
+require "rails_helper"
+
+module Archangel
+  RSpec.describe Design, type: :model do
+    subject(:resource) { described_class.new }
+
+    context "with validations" do
+      it { is_expected.to validate_presence_of(:content) }
+      it { is_expected.to validate_presence_of(:name) }
+
+      it { is_expected.to allow_value(true).for(:partial) }
+      it { is_expected.to allow_value(false).for(:partial) }
+
+      it { is_expected.not_to allow_value(nil).for(:partial) }
+
+      it { is_expected.to allow_value("{{ foo }}").for(:content) }
+      it { is_expected.not_to allow_value("{{ foo }").for(:content) }
+    end
+  end
+end

data/spec/unit/models/archangel/entry_spec.rb

@@ -0,0 +1,104 @@
+# frozen_string_literal: true
+
+require "rails_helper"
+
+module Archangel
+  RSpec.describe Entry, type: :model do
+    context "with validations" do
+      it { is_expected.to validate_presence_of(:collection_id) }
+
+      it { is_expected.to allow_value(nil).for(:value) }
+      it { is_expected.to allow_value("").for(:value) }
+
+      it { is_expected.to allow_value(nil).for(:published_at) }
+      it { is_expected.to allow_value("").for(:published_at) }
+      it { is_expected.to allow_value(Time.current).for(:published_at) }
+
+      it { is_expected.not_to allow_value("invalid").for(:published_at) }
+    end
+
+    context "with scopes" do
+      context "with .available" do
+        it "returns all where published_at <= now" do
+          entry = create(:entry)
+
+          expect(described_class.available.first).to eq(entry)
+        end
+
+        it "returns all where published_at <= now in the future" do
+          entry = create(:entry, :future)
+
+          expect(described_class.available.first).not_to eq(entry)
+        end
+      end
+
+      context "with .available" do
+        it "returns all where published_at <= now" do
+          entry = create(:entry)
+
+          expect(described_class.available.first).to eq(entry)
+        end
+
+        it "returns all where published_at <= now not in the future" do
+          entry = create(:entry, :future)
+
+          expect(described_class.available.first).not_to eq(entry)
+        end
+      end
+
+      context "with .unpublished" do
+        it "returns all where available_at is nil" do
+          entry = create(:entry, :unpublished)
+
+          expect(described_class.unpublished.first).to eq(entry)
+        end
+
+        it "returns all where available_at is > now" do
+          entry = create(:entry, :future)
+
+          expect(described_class.unpublished.first).to eq(entry)
+        end
+      end
+    end
+
+    context "with .available?" do
+      it "is available" do
+        entry = build(:entry, published_at: 1.minute.ago)
+
+        expect(entry).to be_available
+      end
+
+      it "is not available in the future" do
+        entry = build(:entry, published_at: 1.week.from_now)
+
+        expect(entry).not_to be_available
+      end
+
+      it "is not available" do
+        entry = build(:entry, :unpublished)
+
+        expect(entry).not_to be_available
+      end
+    end
+
+    context "with .published?" do
+      it "is published" do
+        entry = build(:entry)
+
+        expect(entry).to be_published
+      end
+
+      it "is published in the future" do
+        entry = build(:entry, published_at: 1.week.from_now)
+
+        expect(entry).to be_published
+      end
+
+      it "is not published" do
+        entry = build(:entry, :unpublished)
+
+        expect(entry).not_to be_published
+      end
+    end
+  end
+end

data/spec/{models → unit/models}/archangel/field_spec.rb

@@ -4,7 +4,9 @@ require "rails_helper"
 
 module Archangel
   RSpec.describe Field, type: :model do
-    context "validations" do
+    subject(:resource) { described_class.new }
+
+    context "with validations" do
       it { is_expected.to validate_presence_of(:classification) }
       it { is_expected.to validate_presence_of(:collection_id).on(:update) }
       it { is_expected.to validate_presence_of(:label) }
@@ -15,10 +17,10 @@ module Archangel
       it { is_expected.to allow_value(false).for(:required) }
       it { is_expected.to allow_value(0).for(:required) }
 
-      it { is_expected.to_not allow_value(nil).for(:required) }
+      it { is_expected.not_to allow_value(nil).for(:required) }
 
       it "allows certain classifications" do
-        expect(subject).to(
+        expect(resource).to(
           validate_inclusion_of(:classification)
             .in_array(Archangel::Field::CLASSIFICATIONS)
         )
@@ -32,9 +34,5 @@ module Archangel
         )
       end
     end
-
-    context "associations" do
-      it { is_expected.to belong_to(:collection) }
-    end
   end
 end