Merge pull request #5 from OpenSourcePolitics/fix/unexpected_double_s…

…copes_displaying

Fix/unexpected double scopes displaying

app/helpers/concerns/decidim/simple_proposal/scopes_helper_override.rb

@@ -29,7 +29,7 @@ def simple_scope_options(root: false, options: {})
         end
 
         def ancestors
-          @ancestors ||= Decidim::Scope.where(parent_id: nil)
+          @ancestors ||= current_organization.scopes.where(parent_id: nil)
         end
 
         def children_after_parent(ancestor, array, prefix)

spec/helpers/concerns/decidim/simple_proposal/scopes_helper_override_spec.rb

@@ -0,0 +1,26 @@
+# frozen_string_literal: true
+
+require "spec_helper"
+
+module Decidim
+  describe ScopesHelper, type: :helper do
+    describe "#ancestors" do
+      let(:first_organization) { create(:organization) }
+      let(:second_organization) { create(:organization) }
+
+      let!(:first_scope) { create(:scope, organization: first_organization) }
+      let!(:second_scope) { create(:scope, organization: first_organization) }
+      let!(:third_scope) { create(:scope, organization: second_organization) }
+      let!(:fourth_scope) { create(:scope, organization: second_organization) }
+
+      it "returns only the scopes for the current organization" do
+        allow(helper).to receive(:current_organization).and_return(first_organization)
+
+        result = helper.send(:ancestors)
+
+        expect(result).to match_array([first_scope, second_scope])
+        expect(result).not_to include(third_scope, fourth_scope)
+      end
+    end
+  end
+end