Lint files with Rubocop

app/models/decidim/navigation_maps/blueprint.rb

@@ -17,14 +17,13 @@ class Blueprint < ApplicationRecord
                class_name: "Decidim::NavigationMaps::BlueprintArea",
                dependent: :destroy
 
-      validates :organization, presence: true
       validates :height, numericality: { greater_than: 0 }
 
       has_one_attached :image
       validates_upload :image, uploader: Decidim::NavigationMaps::BlueprintUploader
 
       def blueprint
-        areas.map { |area| [area.area_id.to_s, area.to_geoson] }.to_h
+        areas.to_h { |area| [area.area_id.to_s, area.to_geoson] }
       end
     end
   end

app/models/decidim/navigation_maps/blueprint_area.rb

@@ -12,8 +12,6 @@ class BlueprintArea < ApplicationRecord
 
       attribute :link_type, :string, default: "link"
 
-      validates :blueprint, presence: true
-
       def to_geoson
         {
           type: area_type,

db/seeds.rb

@@ -2,7 +2,7 @@
 
 print "Skipping seeds for decidim_navigation maps as required by SKIP_MODULE_SEEDS\n" if ENV["SKIP_MODULE_SEEDS"]
 
-if !ENV["SKIP_MODULE_SEEDS"] && (!Rails.env.production? || ENV["SEED"])
+if !ENV["SKIP_MODULE_SEEDS"] && (!Rails.env.production? || ENV.fetch("SEED", nil))
 
   print "Creating seeds for decidim_navigation maps...\n" unless Rails.env.test?
 

lib/tasks/navigation_maps_active_storage_migration_tasks.rake

@@ -15,9 +15,7 @@ namespace :navigation_maps do
         path = Rails.root.join("tmp/navigation_maps_editor_images_mappings.csv")
         dirname = File.dirname(path)
         FileUtils.mkdir_p(dirname) unless File.directory?(dirname)
-        File.open(path, "wb") do |file|
-          file.write(Decidim::Exporters::CSV.new(routes_mappings).export.read)
-        end
+        File.binwrite(path, Decidim::Exporters::CSV.new(routes_mappings).export.read)
       end
     end
 

spec/commands/save_area_spec.rb

@@ -55,7 +55,7 @@ module Decidim::NavigationMaps
       end
 
       it "updates the area" do
-        expect { subject.call }.to change(BlueprintArea, :count).by(0)
+        expect { subject.call }.not_to change(BlueprintArea, :count)
       end
     end
 

spec/forms/blueprint_form_spec.rb

@@ -43,7 +43,7 @@ module Decidim::NavigationMaps
         end
 
         it "image? responds true" do
-          expect(subject.image?).to eq(true)
+          expect(subject.image?).to be(true)
         end
       end
 
@@ -84,7 +84,7 @@ module Decidim::NavigationMaps
       let(:image) { nil }
 
       it "image? responds false" do
-        expect(subject.image?).not_to eq(true)
+        expect(subject.image?).not_to be(true)
       end
     end
   end

spec/lib/navigation_map_cell_helpers_spec.rb

@@ -33,7 +33,7 @@ def initialize(organization, content_block)
 
     describe "#valid_blueprints?" do
       it "returns true" do
-        expect(instance.valid_blueprints?).to eq(true)
+        expect(instance.valid_blueprints?).to be(true)
       end
     end