archangel 0.0.5 → 0.0.6

data/spec/models/archangel/site_spec.rb

@@ -4,8 +4,51 @@ require "rails_helper"
 
 module Archangel
   RSpec.describe Site, type: :model do
-    describe "ActiveModel validations" do
-      it { expect(subject).to validate_presence_of(:name) }
+    context "validations" do
+      it { is_expected.to validate_presence_of(:locale) }
+      it { is_expected.to validate_presence_of(:name) }
+
+      it { is_expected.to allow_value("").for(:theme) }
+
+      it "allows certain languages" do
+        expect(subject)
+          .to validate_inclusion_of(:locale).in_array(Archangel::LANGUAGES)
+      end
+
+      it "allows certain languages" do
+        expect(subject)
+          .to validate_inclusion_of(:theme).in_array(Archangel.themes)
+      end
+    end
+
+    it { is_expected.to have_many(:assets) }
+    it { is_expected.to have_many(:collections) }
+    it { is_expected.to have_many(:pages) }
+    it { is_expected.to have_many(:templates) }
+    it { is_expected.to have_many(:users) }
+    it { is_expected.to have_many(:widgets) }
+    it { is_expected.to have_many(:entries).through(:collections) }
+    it { is_expected.to have_many(:fields).through(:collections) }
+
+    context ".current" do
+      it "returns an existing object" do
+        resource = create(:site, name: "My Awesome New Site")
+
+        expect(described_class.current).to eq(resource)
+        expect(described_class.current.name).to eq("My Awesome New Site")
+      end
+
+      it "returns a new object" do
+        expect(described_class.current.name).to eq("Archangel")
+      end
+    end
+
+    context "#to_liquid" do
+      it "returns a Liquid object" do
+        resource = build(:site)
+
+        expect(resource.to_liquid).to be_a(Archangel::Liquid::Drops::SiteDrop)
+      end
     end
   end
 end

data/spec/models/archangel/template_spec.rb

@@ -5,20 +5,27 @@ require "rails_helper"
 module Archangel
   RSpec.describe Template, type: :model do
     context "validations" do
-      it { expect(subject).to validate_presence_of(:content) }
-      it { expect(subject).to validate_presence_of(:name) }
+      it { is_expected.to validate_presence_of(:content) }
+      it { is_expected.to validate_presence_of(:name) }
 
-      it { expect(subject).to allow_value(true).for(:partial) }
-      it { expect(subject).to allow_value(false).for(:partial) }
-      it { expect(subject).not_to allow_value(nil).for(:partial) }
+      it { is_expected.to allow_value(true).for(:partial) }
+      it { is_expected.to allow_value(false).for(:partial) }
 
-      it { expect(subject).to allow_value("{{ foo }}").for(:content) }
-      it { expect(subject).not_to allow_value("{{ foo }").for(:content) }
+      it { is_expected.to_not allow_value(nil).for(:partial) }
+
+      it { is_expected.to allow_value("{{ foo }}").for(:content) }
+      it { is_expected.to_not allow_value("{{ foo }").for(:content) }
     end
 
     context "associations" do
-      it { expect(subject).to belong_to(:parent) }
-      it { expect(subject).to belong_to(:site) }
+      it { is_expected.to belong_to(:site) }
+
+      it "belongs to Template" do
+        expect(subject).to(
+          belong_to(:parent).conditions(partial: false)
+                            .class_name("Archangel::Template")
+        )
+      end
     end
   end
 end

data/spec/models/archangel/user_spec.rb

@@ -7,27 +7,33 @@ module Archangel
     context "callbacks" do
       let(:resource) { create(:user) }
 
-      it do
-        expect(resource).to callback(:parameterize_username).before(:validation)
-      end
+      it { is_expected.to callback(:parameterize_username).before(:validation) }
+
+      it { is_expected.to callback(:column_default).after(:initialize) }
 
-      it { expect(resource).to callback(:column_default).after(:initialize) }
-      it { expect(resource).to callback(:column_reset).after(:destroy) }
+      it { is_expected.to callback(:column_reset).after(:destroy) }
     end
 
     context "validations" do
-      it { expect(subject).to validate_presence_of(:email) }
-      it { expect(subject).to validate_presence_of(:name) }
-      it { expect(subject).to validate_presence_of(:password).on(:create) }
-      it { expect(subject).to validate_presence_of(:role) }
-      it { expect(subject).to validate_presence_of(:username) }
+      it { is_expected.to validate_presence_of(:email) }
+      it { is_expected.to validate_presence_of(:name) }
+      it { is_expected.to validate_presence_of(:password).on(:create) }
+      it { is_expected.to validate_presence_of(:role) }
+      it { is_expected.to validate_presence_of(:username) }
+
+      it { is_expected.to validate_length_of(:password).is_at_least(8) }
+
+      it { is_expected.to validate_presence_of(:password).on(:update) }
 
-      it { expect(subject).to validate_length_of(:password).is_at_least(8) }
+      it "has a unique username scoped to Site" do
+        resource = build(:user)
 
-      it { expect(subject).to have_db_index(:email).unique(true) }
-      it { expect(subject).to have_db_index(:username).unique(true) }
+        expect(resource).to(
+          validate_uniqueness_of(:username).scoped_to(:site_id).case_insensitive
+        )
+      end
 
-      it do
+      it "allows certain roles" do
         expect(subject)
           .to validate_inclusion_of(:role).in_array(Archangel::ROLES)
       end
@@ -60,7 +66,30 @@ module Archangel
     end
 
     context "associations" do
-      it { expect(subject).to belong_to(:site) }
+      it { is_expected.to belong_to(:site) }
+    end
+
+    context "#to_param" do
+      it "uses `slug` as the identifier for routes" do
+        resource = build(:user, username: "foo")
+
+        expect(resource.to_param).to eq("foo")
+      end
+    end
+
+    context "#column_reset" do
+      before { ::Timecop.freeze }
+      after { ::Timecop.return }
+
+      it "resets `slug` to `slug` + current time" do
+        resource = create(:user)
+
+        username = resource.username
+
+        resource.destroy!
+
+        expect(resource.username).to eq "#{Time.current.to_i}_#{username}"
+      end
     end
   end
 end

data/spec/models/archangel/widget_spec.rb

@@ -5,25 +5,59 @@ require "rails_helper"
 module Archangel
   RSpec.describe Widget, type: :model do
     context "callbacks" do
-      it { expect(subject).to callback(:parameterize_slug).before(:validation) }
+      it { is_expected.to callback(:parameterize_slug).before(:validation) }
 
-      it { expect(subject).to callback(:column_reset).after(:destroy) }
+      it { is_expected.to callback(:column_reset).after(:destroy) }
     end
 
     context "validations" do
-      it { expect(subject).to validate_presence_of(:content) }
-      it { expect(subject).to validate_presence_of(:name) }
-      it { expect(subject).to validate_presence_of(:slug) }
+      it { is_expected.to validate_presence_of(:content) }
+      it { is_expected.to validate_presence_of(:name) }
+      it { is_expected.to validate_presence_of(:slug) }
 
-      it { expect(subject).to have_db_index(:slug).unique(true) }
+      it { is_expected.to allow_value("{{ foo }}").for(:content) }
+      it { is_expected.to_not allow_value("{{ foo }").for(:content) }
 
-      it { expect(subject).to allow_value("{{ foo }}").for(:content) }
-      it { expect(subject).not_to allow_value("{{ foo }").for(:content) }
+      it "has a unique slug scoped to Site" do
+        resource = build(:widget)
+
+        expect(resource)
+          .to validate_uniqueness_of(:slug).scoped_to(:site_id).case_insensitive
+      end
     end
 
     context "associations" do
-      it { expect(subject).to belong_to(:site) }
-      it { expect(subject).to belong_to(:template) }
+      it { is_expected.to belong_to(:site) }
+
+      it "belongs to Template" do
+        expect(subject).to(
+          belong_to(:template).conditions(partial: true)
+                              .class_name("Archangel::Template")
+        )
+      end
+    end
+
+    context "#to_param" do
+      it "uses `slug` as the identifier for routes" do
+        resource = build(:widget, slug: "foo")
+
+        expect(resource.to_param).to eq("foo")
+      end
+    end
+
+    context "#column_reset" do
+      before { ::Timecop.freeze }
+      after { ::Timecop.return }
+
+      it "resets `slug` to `slug` + current time" do
+        resource = create(:widget)
+
+        slug = resource.slug
+
+        resource.destroy!
+
+        expect(resource.slug).to eq "#{Time.current.to_i}_#{slug}"
+      end
     end
   end
 end

data/spec/policies/archangel/asset_policy_spec.rb

@@ -6,17 +6,34 @@ module Archangel
   RSpec.describe AssetPolicy, type: :policy do
     subject { described_class.new(user, record) }
 
-    let(:user) { create(:user) }
     let(:record) { create(:asset) }
 
-    it { should permit(:index) }
-    it { should permit(:show) }
-    it { should permit(:create) }
-    it { should permit(:new) }
-    it { should permit(:update) }
-    it { should permit(:edit) }
-    it { should permit(:destroy) }
+    context "with `admin` role" do
+      let(:user) { create(:user, :admin) }
 
-    it { should permit(:wysiwyg) }
+      it { is_expected.to permit(:index) }
+      it { is_expected.to permit(:show) }
+      it { is_expected.to permit(:new) }
+      it { is_expected.to permit(:create) }
+      it { is_expected.to permit(:edit) }
+      it { is_expected.to permit(:update) }
+      it { is_expected.to permit(:destroy) }
+
+      it { is_expected.to permit(:wysiwyg) }
+    end
+
+    context "with `editor` role" do
+      let(:user) { create(:user, :editor) }
+
+      it { is_expected.to permit(:index) }
+      it { is_expected.to permit(:show) }
+      it { is_expected.to permit(:new) }
+      it { is_expected.to permit(:create) }
+      it { is_expected.to permit(:edit) }
+      it { is_expected.to permit(:update) }
+      it { is_expected.to permit(:destroy) }
+
+      it { is_expected.to permit(:wysiwyg) }
+    end
   end
 end

data/spec/policies/archangel/collection_policy_spec.rb

@@ -6,15 +6,30 @@ module Archangel
   RSpec.describe CollectionPolicy, type: :policy do
     subject { described_class.new(user, record) }
 
-    let(:user) { create(:user) }
     let(:record) { create(:collection) }
 
-    it { should permit(:index) }
-    it { should permit(:show) }
-    it { should permit(:create) }
-    it { should permit(:new) }
-    it { should permit(:update) }
-    it { should permit(:edit) }
-    it { should permit(:destroy) }
+    context "with `admin` role" do
+      let(:user) { create(:user, :admin) }
+
+      it { is_expected.to permit(:index) }
+      it { is_expected.to permit(:show) }
+      it { is_expected.to permit(:new) }
+      it { is_expected.to permit(:create) }
+      it { is_expected.to permit(:edit) }
+      it { is_expected.to permit(:update) }
+      it { is_expected.to permit(:destroy) }
+    end
+
+    context "with `editor` role" do
+      let(:user) { create(:user, :editor) }
+
+      it { is_expected.to permit(:index) }
+      it { is_expected.to permit(:show) }
+      it { is_expected.to permit(:new) }
+      it { is_expected.to permit(:create) }
+      it { is_expected.to permit(:edit) }
+      it { is_expected.to permit(:update) }
+      it { is_expected.to permit(:destroy) }
+    end
   end
 end

data/spec/policies/archangel/entries_policy_spec.rb

@@ -6,17 +6,34 @@ module Archangel
   RSpec.describe EntryPolicy, type: :policy do
     subject { described_class.new(user, record) }
 
-    let(:user) { create(:user) }
     let(:record) { create(:entry) }
 
-    it { should permit(:index) }
-    it { should permit(:show) }
-    it { should permit(:create) }
-    it { should permit(:new) }
-    it { should permit(:update) }
-    it { should permit(:edit) }
-    it { should permit(:destroy) }
+    context "with `admin` role" do
+      let(:user) { create(:user, :admin) }
 
-    it { should permit(:sort) }
+      it { is_expected.to permit(:index) }
+      it { is_expected.to permit(:show) }
+      it { is_expected.to permit(:new) }
+      it { is_expected.to permit(:create) }
+      it { is_expected.to permit(:edit) }
+      it { is_expected.to permit(:update) }
+      it { is_expected.to permit(:destroy) }
+
+      it { is_expected.to permit(:sort) }
+    end
+
+    context "with `editor` role" do
+      let(:user) { create(:user, :editor) }
+
+      it { is_expected.to permit(:index) }
+      it { is_expected.to permit(:show) }
+      it { is_expected.to permit(:new) }
+      it { is_expected.to permit(:create) }
+      it { is_expected.to permit(:edit) }
+      it { is_expected.to permit(:update) }
+      it { is_expected.to permit(:destroy) }
+
+      it { is_expected.to permit(:sort) }
+    end
   end
 end

data/spec/policies/archangel/page_policy_spec.rb

@@ -6,15 +6,30 @@ module Archangel
   RSpec.describe PagePolicy, type: :policy do
     subject { described_class.new(user, record) }
 
-    let(:user) { create(:user) }
     let(:record) { create(:page) }
 
-    it { should permit(:index) }
-    it { should permit(:show) }
-    it { should permit(:create) }
-    it { should permit(:new) }
-    it { should permit(:update) }
-    it { should permit(:edit) }
-    it { should permit(:destroy) }
+    context "with `admin` role" do
+      let(:user) { create(:user, :admin) }
+
+      it { is_expected.to permit(:index) }
+      it { is_expected.to permit(:show) }
+      it { is_expected.to permit(:new) }
+      it { is_expected.to permit(:create) }
+      it { is_expected.to permit(:edit) }
+      it { is_expected.to permit(:update) }
+      it { is_expected.to permit(:destroy) }
+    end
+
+    context "with `editor` role" do
+      let(:user) { create(:user, :editor) }
+
+      it { is_expected.to permit(:index) }
+      it { is_expected.to permit(:show) }
+      it { is_expected.to permit(:new) }
+      it { is_expected.to permit(:create) }
+      it { is_expected.to permit(:edit) }
+      it { is_expected.to permit(:update) }
+      it { is_expected.to permit(:destroy) }
+    end
   end
 end

data/spec/policies/archangel/site_policy_spec.rb

@@ -6,25 +6,27 @@ module Archangel
   RSpec.describe SitePolicy, type: :policy do
     subject { described_class.new(user, record) }
 
-    let(:user) { create(:user) }
     let(:record) { create(:site) }
 
-    it { should permit(:show) }
-
     context "with `admin` role" do
       let(:user) { create(:user, :admin) }
 
-      it { should permit(:update) }
-      it { should permit(:edit) }
-      it { should permit(:sample) }
+      it { is_expected.to permit(:show) }
+      it { is_expected.to permit(:update) }
+      it { is_expected.to permit(:edit) }
+
+      it { is_expected.to permit(:sample) }
     end
 
     context "with `editor` role" do
       let(:user) { create(:user, :editor) }
 
-      it { should_not permit(:update) }
-      it { should_not permit(:edit) }
-      it { should permit(:sample) }
+      it { is_expected.to permit(:show) }
+
+      it { is_expected.to permit(:sample) }
+
+      it { is_expected.to_not permit(:update) }
+      it { is_expected.to_not permit(:edit) }
     end
   end
 end

data/spec/policies/archangel/template_policy_spec.rb

@@ -6,15 +6,30 @@ module Archangel
   RSpec.describe TemplatePolicy, type: :policy do
     subject { described_class.new(user, record) }
 
-    let(:user) { create(:user) }
     let(:record) { create(:template) }
 
-    it { should permit(:index) }
-    it { should permit(:show) }
-    it { should permit(:create) }
-    it { should permit(:new) }
-    it { should permit(:update) }
-    it { should permit(:edit) }
-    it { should permit(:destroy) }
+    context "with `admin` role" do
+      let(:user) { create(:user, :admin) }
+
+      it { is_expected.to permit(:index) }
+      it { is_expected.to permit(:show) }
+      it { is_expected.to permit(:new) }
+      it { is_expected.to permit(:create) }
+      it { is_expected.to permit(:edit) }
+      it { is_expected.to permit(:update) }
+      it { is_expected.to permit(:destroy) }
+    end
+
+    context "with `editor` role" do
+      let(:user) { create(:user, :editor) }
+
+      it { is_expected.to permit(:index) }
+      it { is_expected.to permit(:show) }
+      it { is_expected.to permit(:new) }
+      it { is_expected.to permit(:create) }
+      it { is_expected.to permit(:edit) }
+      it { is_expected.to permit(:update) }
+      it { is_expected.to permit(:destroy) }
+    end
   end
 end