decidim 0.0.2 → 0.0.3

data/decidim-results/lib/decidim/results/feature.rb

@@ -0,0 +1,52 @@
+# frozen_string_literal: true
+
+require_dependency "decidim/features/namer"
+
+Decidim.register_feature(:results) do |feature|
+  feature.engine = Decidim::Results::ListEngine
+  feature.admin_engine = Decidim::Results::AdminEngine
+  feature.icon = "decidim/results/icon.svg"
+
+  feature.on(:before_destroy) do |instance|
+    raise StandardError, "Can't remove this feature" if Decidim::Results::Result.where(feature: instance).any?
+  end
+
+  feature.register_resource do |resource|
+    resource.model_class_name = "Decidim::Results::Result"
+    resource.template = "decidim/results/results/linked_results"
+  end
+
+  feature.settings(:global) do |settings|
+    settings.attribute :comments_always_enabled, type: :boolean, default: true
+  end
+
+  feature.settings(:step) do |settings|
+    settings.attribute :comments_enabled, type: :boolean, default: true
+  end
+
+  feature.seeds do
+    Decidim::ParticipatoryProcess.all.each do |process|
+      next unless process.steps.any?
+
+      feature = Decidim::Feature.create!(
+        name: Decidim::Features::Namer.new(process.organization.available_locales, :results).i18n_name,
+        manifest_name: :results,
+        participatory_process: process
+      )
+
+      3.times do
+        result = Decidim::Results::Result.create!(
+          feature: feature,
+          scope: process.organization.scopes.sample,
+          category: process.categories.sample,
+          title: Decidim::Faker::Localized.sentence(2),
+          description: Decidim::Faker::Localized.wrapped("<p>", "</p>") do
+            Decidim::Faker::Localized.paragraph(3)
+          end
+        )
+
+        Decidim::Comments::Seed.comments_for(result)
+      end
+    end
+  end
+end

data/decidim-results/lib/decidim/results/list_engine.rb

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+require "searchlight"
+require "kaminari"
+
+module Decidim
+  module Results
+    # This is the engine that runs on the public interface of `decidim-results`.
+    # It mostly handles rendering the created results associated to a participatory
+    # process.
+    class ListEngine < ::Rails::Engine
+      isolate_namespace Decidim::Results
+
+      routes do
+        resources :results, only: [:index, :show]
+        root to: "results#index"
+      end
+    end
+  end
+end

data/decidim-results/lib/decidim/results/test/factories.rb

@@ -0,0 +1,11 @@
+# frozen_string_literal: true
+require "decidim/faker/localized"
+require "decidim/dev"
+
+FactoryGirl.define do
+  factory :result, class: Decidim::Results::Result do
+    title { Decidim::Faker::Localized.sentence(3) }
+    description { Decidim::Faker::Localized.wrapped("<p>", "</p>") { Decidim::Faker::Localized.sentence(4) } }
+    feature { build(:feature, manifest_name: "results") }
+  end
+end

data/decidim-results/spec/commands/create_result_spec.rb

@@ -0,0 +1,90 @@
+require "spec_helper"
+
+describe Decidim::Results::Admin::CreateResult do
+  let(:organization) { create :organization, available_locales: [:en] }
+  let(:participatory_process) { create :participatory_process, organization: organization }
+  let(:current_feature) { create :feature, manifest_name: "results", participatory_process: participatory_process }
+  let(:scope) { create :scope, organization: organization }
+  let(:category) { create :category, participatory_process: participatory_process }
+  let(:meeting_feature) do
+    create(:feature, manifest_name: "meetings", participatory_process: participatory_process)
+  end
+  let(:meeting) do
+    create(
+      :meeting,
+      feature: meeting_feature
+    )
+  end
+  let(:proposal_feature) do
+    create(:feature, manifest_name: "proposals", participatory_process: participatory_process)
+  end
+  let(:proposals) do
+    create_list(
+      :proposal,
+      3,
+      feature: proposal_feature
+    )
+  end
+  let(:form) do
+    double(
+      :invalid? => invalid,
+      title: {en: "title"},
+      description: {en: "description"},
+      proposal_ids: proposals.map(&:id),
+      scope: scope,
+      category: category,
+      current_feature: current_feature
+    )
+  end
+  let(:invalid) { false }
+
+  subject { described_class.new(form) }
+
+  context "when the form is not valid" do
+    let(:invalid) { true }
+
+    it "is not valid" do
+      expect { subject.call }.to broadcast(:invalid)
+    end
+  end
+
+  context "when everything is ok" do
+    let(:result) { Decidim::Results::Result.last }
+
+    it "creates the result" do
+      expect { subject.call }.to change { Decidim::Results::Result.count }.by(1)
+    end
+
+    it "sets the scope" do
+      subject.call
+      expect(result.scope).to eq scope
+    end
+
+    it "sets the category" do
+      subject.call
+      expect(result.category).to eq category
+    end
+
+    it "sets the feature" do
+      subject.call
+      expect(result.feature).to eq current_feature
+    end
+
+    it "links proposals" do
+      subject.call
+      linked_proposals = result.linked_resources(:proposals, "included_proposals")
+      expect(linked_proposals).to match_array(proposals)
+    end
+
+    it "links meetings" do
+      proposals.each do |proposal|
+        proposal.link_resources([meeting], "proposals_from_meeting")
+      end
+
+      subject.call
+      linked_meetings = result.linked_resources(:meetings, "meetings_through_proposals")
+
+      expect(linked_meetings).to eq [meeting]
+    end
+  end
+end

data/decidim-results/spec/commands/update_result_spec.rb

@@ -0,0 +1,83 @@
+require "spec_helper"
+
+describe Decidim::Results::Admin::UpdateResult do
+  let(:result) { create :result}
+  let(:organization) { result.feature.organization }
+  let(:scope) { create :scope, organization: organization }
+  let(:category) { create :category, participatory_process: result.feature.participatory_process }
+  let(:participatory_process) { result.feature.participatory_process }
+  let(:meeting_feature) do
+    create(:feature, manifest_name: :meetings, participatory_process: participatory_process)
+  end
+  let(:meeting) do
+    create(
+      :meeting,
+      feature: meeting_feature
+    )
+  end
+  let(:proposal_feature) do
+    create(:feature, manifest_name: :proposals, participatory_process: participatory_process)
+  end
+  let(:proposals) do
+    create_list(
+      :proposal,
+      3,
+      feature: proposal_feature
+    )
+  end
+  let(:form) do
+    double(
+      :invalid? => invalid,
+      title: {en: "title"},
+      description: {en: "description"},
+      proposal_ids: proposals.map(&:id),
+      scope: scope,
+      category: category
+    )
+  end
+  let(:invalid) { false }
+
+  subject { described_class.new(form, result) }
+
+  context "when the form is not valid" do
+    let(:invalid) { true }
+
+    it "is not valid" do
+      expect { subject.call }.to broadcast(:invalid)
+    end
+  end
+
+  context "when everything is ok" do
+    it "updates the result" do
+      subject.call
+      expect(translated(result.title)).to eq "title"
+    end
+
+    it "sets the scope" do
+      subject.call
+      expect(result.scope).to eq scope
+    end
+
+    it "sets the category" do
+      subject.call
+      expect(result.category).to eq category
+    end
+
+    it "links proposals" do
+      subject.call
+      linked_proposals = result.linked_resources(:proposals, "included_proposals")
+      expect(linked_proposals).to match_array(proposals)
+    end
+
+    it "links meetings" do
+      proposals.each do |proposal|
+        proposal.link_resources([meeting], "proposals_from_meeting")
+      end
+
+      subject.call
+      linked_meetings = result.linked_resources(:meetings, "meetings_through_proposals")
+
+      expect(linked_meetings).to eq [meeting]
+    end
+  end
+end

data/decidim-results/spec/factories.rb

@@ -0,0 +1,6 @@
+require "decidim/core/test/factories"
+require "decidim/admin/test/factories"
+require "decidim/comments/test/factories"
+require "decidim/results/test/factories"
+require "decidim/proposals/test/factories"
+require "decidim/meetings/test/factories"

data/decidim-results/spec/features/admin_manages_results_spec.rb

@@ -0,0 +1,16 @@
+# frozen_string_literal: true
+
+require "spec_helper"
+require_relative "../shared/admin_shared_context"
+require_relative "../shared/manage_results_examples"
+
+describe "Admin manages results", type: :feature do
+  include_context "admin"
+  it_behaves_like "manage results"
+
+  before do
+    switch_to_host(organization.host)
+    login_as user, scope: :user
+    visit decidim_admin.manage_feature_path(participatory_process_id: participatory_process, feature_id: current_feature)
+  end
+end

data/decidim-results/spec/features/explore_results_spec.rb

@@ -0,0 +1,186 @@
+require "spec_helper"
+
+describe "Explore results", type: :feature do
+  include_context "feature"
+  let(:manifest_name) { "results" }
+
+  let(:results_count) { 5 }
+  let!(:scope) { create :scope, organization: organization }
+  let!(:results) do
+    create_list(
+      :result,
+      results_count,
+      feature: feature
+    )
+  end
+
+  before do
+    visit path
+  end
+
+  context "index" do
+    let(:path) { decidim_results.results_path(participatory_process_id: participatory_process.id, feature_id: feature.id) }
+
+    it "shows all results for the given process" do
+      expect(page).to have_selector("article.card", count: results_count)
+
+      results.each do |result|
+        expect(page).to have_content(translated result.title)
+      end
+    end
+  end
+
+  context "show" do
+    let(:path) { decidim_results.result_path(id: result.id, participatory_process_id: participatory_process.id, feature_id: feature.id) }
+    let(:results_count) { 1 }
+    let(:result) { results.first }
+
+    it "shows all result info" do
+      expect(page).to have_i18n_content(result.title)
+      expect(page).to have_i18n_content(result.description)
+
+      within ".section.view-side" do
+        expect(page).to have_content(/Proposals/i)
+        expect(page).to have_content(/Meetings/i)
+        expect(page).to have_content(/Comments/i)
+        expect(page).to have_content(/Attendees/i)
+        expect(page).to have_content(/Supports/i)
+        expect(page).to have_content(/Contributions/i)
+      end
+    end
+
+    context "without category or scope" do
+      it "does not show any tag" do
+        expect(page).not_to have_selector("ul.tags.tags--result")
+      end
+    end
+
+    context "with a category" do
+      let(:result) do
+        result = results.first
+        result.category = create :category, participatory_process: participatory_process
+        result.save
+        result
+      end
+
+      it "shows tags for category" do
+        expect(page).to have_selector("ul.tags.tags--result")
+        within "ul.tags.tags--result" do
+          expect(page).to have_content(translated(result.category.name))
+        end
+      end
+
+      it "links to the filter for this category" do
+        within "ul.tags.tags--result" do
+          click_link translated(result.category.name)
+        end
+        expect(page).to have_select("filter_category_id", selected: translated(result.category.name))
+      end
+    end
+
+    context "with a scope" do
+      let(:result) do
+        result = results.first
+        result.scope = create :scope, organization: organization
+        result.save
+        result
+      end
+
+      it "shows tags for scope" do
+        expect(page).to have_selector("ul.tags.tags--result")
+        within "ul.tags.tags--result" do
+          expect(page).to have_content(result.scope.name)
+        end
+      end
+
+      it "links to the filter for this scope" do
+        within "ul.tags.tags--result" do
+          click_link result.scope.name
+        end
+        expect(page).to have_checked_field(result.scope.name)
+      end
+    end
+
+    context "when a proposal has comments" do
+      let(:result) { results.first }
+      let(:author) { create(:user, :confirmed, organization: feature.organization)}
+      let!(:comments) { create_list(:comment, 3, commentable: result) }
+
+      before do
+        visit current_path
+      end
+
+      it "shows the comments" do
+        comments.each do |comment|
+          expect(page).to have_content(comment.body)
+        end
+      end
+    end
+
+    context "with linked proposals" do
+      let(:proposal_feature) do
+        create(:feature, manifest_name: :proposals, participatory_process: result.feature.participatory_process)
+      end
+      let(:proposals) { create_list(:proposal, 3, feature: proposal_feature) }
+
+      before do
+        result.link_resources(proposals, "included_proposals")
+        visit current_path
+      end
+
+      it "shows related proposals" do
+        proposals.each do |proposal|
+          expect(page).to have_content(proposal.title)
+          expect(page).to have_content(proposal.author_name)
+          expect(page).to have_content(proposal.votes.size)
+        end
+      end
+    end
+
+    context "with linked proposals" do
+      let(:meeting_feature) do
+        create(:feature, manifest_name: :meetings, participatory_process: result.feature.participatory_process)
+      end
+      let(:meetings) { create_list(:meeting, 3, feature: meeting_feature) }
+
+      before do
+        result.link_resources(meetings, "meetings_through_proposals")
+        visit current_path
+      end
+
+      it "shows related meetings" do
+        meetings.each do |meeting|
+          expect(page).to have_i18n_content(meeting.title)
+          expect(page).to have_i18n_content(meeting.description)
+        end
+      end
+    end
+
+    context "when filtering" do
+      before do
+        create(:result, feature: feature, scope: scope)
+        visit_feature
+      end
+
+      context "by origin 'official'" do
+        it "lists the filtered results" do
+          within ".filters" do
+            check scope.name
+          end
+
+          expect(page).to have_css(".card--result", count: 1)
+        end
+      end
+
+      context "by origin 'citizenship'" do
+        it "lists the filtered results" do
+          within ".filters" do
+            check scope.name
+          end
+
+          expect(page).to have_css(".card--result", count: results.size)
+        end
+      end
+    end
+  end
+end