geo_pattern 1.4.0 → 1.5.0

data/spec/pattern_validator_spec.rb

@@ -1,28 +1,30 @@
-require 'spec_helper'
+# frozen_string_literal: true
+
+require "spec_helper"
 
 RSpec.describe PatternValidator do
   subject(:validator) { PatternValidator.new(store) }
-  let(:store) { instance_double('GeoPattern::PatternStore') }
+  let(:store) { instance_double("GeoPattern::PatternStore") }
   let(:patterns) { [] }
 
   # Minimum valid object test
   it { expect(validator).not_to be_nil }
 
-  describe '#validate' do
-    context 'when valid pattern is validated' do
+  describe "#validate" do
+    context "when valid pattern is validated" do
       before :each do
-        allow(store).to receive(:known?).with('pattern1').and_return(true)
+        allow(store).to receive(:known?).with("pattern1").and_return(true)
       end
 
-      it { expect { validator.validate(%w(pattern1)) }.not_to raise_error }
+      it { expect { validator.validate(%w[pattern1]) }.not_to raise_error }
     end
 
-    context 'when invalid pattern is validated' do
+    context "when invalid pattern is validated" do
       before :each do
-        allow(store).to receive(:known?).with('pattern1').and_return(false)
+        allow(store).to receive(:known?).with("pattern1").and_return(false)
       end
 
-      it { expect { validator.validate(%w(pattern1)) }.to raise_error InvalidPatternError }
+      it { expect { validator.validate(%w[pattern1]) }.to raise_error InvalidPatternError }
     end
   end
 end

data/spec/seed_spec.rb

@@ -1,13 +1,15 @@
-require 'spec_helper'
+# frozen_string_literal: true
+
+require "spec_helper"
 
 RSpec.describe Seed do
   subject(:seed) { Seed.new(input) }
-  let(:input) { 'string' }
+  let(:input) { "string" }
 
   it { expect(seed).not_to be_nil }
 
-  describe '#[]' do
-    context 'when use an integer' do
+  describe "#[]" do
+    context "when use an integer" do
       it { expect(seed[1]).to be_kind_of String }
     end
   end

data/spec/spec_helper.rb

@@ -1,18 +1,19 @@
-# encoding: utf-8
-$LOAD_PATH << File.expand_path('../../lib', __FILE__)
+# frozen_string_literal: true
 
-require 'simplecov'
-SimpleCov.command_name 'rspec'
+$LOAD_PATH << File.expand_path("../lib", __dir__)
+
+require "simplecov"
+SimpleCov.command_name "rspec"
 SimpleCov.start
 
 # Pull in all of the gems including those in the `test` group
-require 'bundler'
+require "bundler"
 Bundler.require :default, :test, :development
 
-require 'geo_pattern'
+require "geo_pattern"
 
 # Loading support files
-GeoPattern::Helpers.require_files_matching_pattern ::File.expand_path('../support/**/*.rb', __FILE__)
+GeoPattern::Helpers.require_files_matching_pattern ::File.expand_path("support/**/*.rb", __dir__)
 
 # No need to add the namespace to every class tested
-include GeoPattern
+include GeoPattern # rubocop:disable Style/MixinUsage

data/spec/structure_generators/chevrons_generator_spec.rb

@@ -1,5 +1,7 @@
-require 'spec_helper'
+# frozen_string_literal: true
+
+require "spec_helper"
 
 RSpec.describe StructureGenerators::ChevronsGenerator do
-  it_behaves_like 'a structure generator', :chevrons
+  it_behaves_like "a structure generator", :chevrons
 end

data/spec/structure_generators/concentric_circles_generator_spec.rb

@@ -1,5 +1,7 @@
-require 'spec_helper'
+# frozen_string_literal: true
+
+require "spec_helper"
 
 RSpec.describe StructureGenerators::ConcentricCirclesGenerator do
-  it_behaves_like 'a structure generator', :concentric_circles
+  it_behaves_like "a structure generator", :concentric_circles
 end

data/spec/structure_generators/diamonds_generator_spec.rb

@@ -1,5 +1,7 @@
-require 'spec_helper'
+# frozen_string_literal: true
+
+require "spec_helper"
 
 RSpec.describe StructureGenerators::DiamondsGenerator do
-  it_behaves_like 'a structure generator', :diamonds
+  it_behaves_like "a structure generator", :diamonds
 end

data/spec/structure_generators/hexagons_generator_spec.rb

@@ -1,5 +1,7 @@
-require 'spec_helper'
+# frozen_string_literal: true
+
+require "spec_helper"
 
 RSpec.describe StructureGenerators::HexagonsGenerator do
-  it_behaves_like 'a structure generator', :hexagons
+  it_behaves_like "a structure generator", :hexagons
 end

data/spec/structure_generators/mosaic_squares_generator_spec.rb

@@ -1,5 +1,7 @@
-require 'spec_helper'
+# frozen_string_literal: true
+
+require "spec_helper"
 
 RSpec.describe StructureGenerators::MosaicSquaresGenerator do
-  it_behaves_like 'a structure generator', :mosaic_squares
+  it_behaves_like "a structure generator", :mosaic_squares
 end

data/spec/structure_generators/nested_squares_generator_spec.rb

@@ -1,5 +1,7 @@
-require 'spec_helper'
+# frozen_string_literal: true
+
+require "spec_helper"
 
 RSpec.describe StructureGenerators::NestedSquaresGenerator do
-  it_behaves_like 'a structure generator', :nested_squares
+  it_behaves_like "a structure generator", :nested_squares
 end

data/spec/structure_generators/octagons_generator_spec.rb

@@ -1,5 +1,7 @@
-require 'spec_helper'
+# frozen_string_literal: true
+
+require "spec_helper"
 
 RSpec.describe StructureGenerators::OctagonsGenerator do
-  it_behaves_like 'a structure generator', :octagons
+  it_behaves_like "a structure generator", :octagons
 end

data/spec/structure_generators/overlapping_circles_generator_spec.rb

@@ -1,5 +1,7 @@
-require 'spec_helper'
+# frozen_string_literal: true
+
+require "spec_helper"
 
 RSpec.describe StructureGenerators::OverlappingCirclesGenerator do
-  it_behaves_like 'a structure generator', :overlapping_circles
+  it_behaves_like "a structure generator", :overlapping_circles
 end

data/spec/structure_generators/overlapping_rings_generator_spec.rb

@@ -1,5 +1,7 @@
-require 'spec_helper'
+# frozen_string_literal: true
+
+require "spec_helper"
 
 RSpec.describe StructureGenerators::OverlappingRingsGenerator do
-  it_behaves_like 'a structure generator', :overlapping_rings
+  it_behaves_like "a structure generator", :overlapping_rings
 end

data/spec/structure_generators/plaid_generator_spec.rb

@@ -1,5 +1,7 @@
-require 'spec_helper'
+# frozen_string_literal: true
+
+require "spec_helper"
 
 RSpec.describe StructureGenerators::PlaidGenerator do
-  it_behaves_like 'a structure generator', :plaid
+  it_behaves_like "a structure generator", :plaid
 end

data/spec/structure_generators/plus_signs_generator_spec.rb

@@ -1,5 +1,7 @@
-require 'spec_helper'
+# frozen_string_literal: true
+
+require "spec_helper"
 
 RSpec.describe StructureGenerators::PlusSignsGenerator do
-  it_behaves_like 'a structure generator', :plus_signs
+  it_behaves_like "a structure generator", :plus_signs
 end

data/spec/structure_generators/sine_waves_generator_spec.rb

@@ -1,5 +1,7 @@
-require 'spec_helper'
+# frozen_string_literal: true
+
+require "spec_helper"
 
 RSpec.describe StructureGenerators::SineWavesGenerator do
-  it_behaves_like 'a structure generator', :sine_waves
+  it_behaves_like "a structure generator", :sine_waves
 end

data/spec/structure_generators/squares_generator_spec.rb

@@ -1,5 +1,7 @@
-require 'spec_helper'
+# frozen_string_literal: true
+
+require "spec_helper"
 
 RSpec.describe StructureGenerators::SquaresGenerator do
-  it_behaves_like 'a structure generator', :squares
+  it_behaves_like "a structure generator", :squares
 end

data/spec/structure_generators/tessellation_generator_spec.rb

@@ -1,5 +1,7 @@
-require 'spec_helper'
+# frozen_string_literal: true
+
+require "spec_helper"
 
 RSpec.describe StructureGenerators::TessellationGenerator do
-  it_behaves_like 'a structure generator', :tessellation
+  it_behaves_like "a structure generator", :tessellation
 end

data/spec/structure_generators/triangles_generator_spec.rb

@@ -1,5 +1,7 @@
-require 'spec_helper'
+# frozen_string_literal: true
+
+require "spec_helper"
 
 RSpec.describe StructureGenerators::TrianglesGenerator do
-  it_behaves_like 'a structure generator', :triangles
+  it_behaves_like "a structure generator", :triangles
 end

data/spec/structure_generators/xes_generator_spec.rb

@@ -1,5 +1,7 @@
-require 'spec_helper'
+# frozen_string_literal: true
+
+require "spec_helper"
 
 RSpec.describe StructureGenerators::XesGenerator do
-  it_behaves_like 'a structure generator', :xes
+  it_behaves_like "a structure generator", :xes
 end

data/spec/structure_spec.rb

@@ -1,17 +1,19 @@
-require 'spec_helper'
+# frozen_string_literal: true
+
+require "spec_helper"
 
 RSpec.describe Structure do
   subject(:metadata) { described_class.new(image: svg_image, preset: preset, generator: generator, name: name) }
 
-  let(:svg_image) { instance_double('GeoPattern::SvgImage') }
-  let(:svg_image_content) { fixtures_path('generated_patterns/sine_waves.svg').read.chomp }
-  let(:preset) { instance_double('GeoPattern::PatternPreset') }
-  let(:generator) { stub_const('GeoPattern::StructureGenerators::ChevronGenerator', Class.new) }
+  let(:svg_image) { instance_double("GeoPattern::SvgImage") }
+  let(:svg_image_content) { fixtures_path("generated_patterns/sine_waves.svg").read.chomp }
+  let(:preset) { instance_double("GeoPattern::PatternPreset") }
+  let(:generator) { stub_const("GeoPattern::StructureGenerators::ChevronGenerator", Class.new) }
 
   let(:name) { :chevron }
-  let(:fill_color_dark) { '#222' }
-  let(:fill_color_light) { '#ddd' }
-  let(:stroke_color) { '#000' }
+  let(:fill_color_dark) { "#222" }
+  let(:fill_color_light) { "#ddd" }
+  let(:stroke_color) { "#000" }
   let(:stroke_opacity) { 0.02 }
   let(:opacity_min) { 0.02 }
   let(:opacity_max) { 0.15 }
@@ -27,12 +29,12 @@ RSpec.describe Structure do
     allow(preset).to receive(:opacity_max).and_return(opacity_max)
   end
 
-  it_behaves_like 'a metadata argument', :name
-  it_behaves_like 'a metadata argument', :generator
-  it_behaves_like 'a forwarded metadata argument', :fill_color_dark
-  it_behaves_like 'a forwarded metadata argument', :fill_color_light
-  it_behaves_like 'a forwarded metadata argument', :stroke_color
-  it_behaves_like 'a forwarded metadata argument', :stroke_opacity
-  it_behaves_like 'a forwarded metadata argument', :opacity_min
-  it_behaves_like 'a forwarded metadata argument', :opacity_max
+  it_behaves_like "a metadata argument", :name
+  it_behaves_like "a metadata argument", :generator
+  it_behaves_like "a forwarded metadata argument", :fill_color_dark
+  it_behaves_like "a forwarded metadata argument", :fill_color_light
+  it_behaves_like "a forwarded metadata argument", :stroke_color
+  it_behaves_like "a forwarded metadata argument", :stroke_opacity
+  it_behaves_like "a forwarded metadata argument", :opacity_min
+  it_behaves_like "a forwarded metadata argument", :opacity_max
 end

data/spec/support/aruba.rb

@@ -1,6 +1,7 @@
-# encoding: utf-8
-require 'aruba/api'
-require 'aruba/reporting'
+# frozen_string_literal: true
+
+require "aruba/rspec"
+require "aruba/api"
 
 # Spec Helpers
 module SpecHelper
@@ -9,7 +10,7 @@ module SpecHelper
     include ::Aruba::Api
 
     def dirs
-      @dirs ||= %w(tmp rspec)
+      @dirs ||= %w[tmp rspec]
     end
   end
 end
@@ -18,7 +19,6 @@ RSpec.configure do |c|
   c.include SpecHelper::Aruba
 
   c.before :each do
-    clean_current_dir
-    restore_env
+    setup_aruba
   end
 end

data/spec/support/helpers/fixtures.rb

@@ -1,7 +1,9 @@
+# frozen_string_literal: true
+
 module SpecHelper
   module Fixtures
     def fixtures_path(name)
-      base_path = Pathname.new(File.expand_path('../../../../fixtures', __FILE__))
+      base_path = Pathname.new(File.expand_path("../../../fixtures", __dir__))
       base_path + Pathname.new(name)
     end
   end

data/spec/support/kernel.rb

@@ -1,3 +1,7 @@
+# frozen_string_literal: true
+
+# rubocop:disable Style/EvalWithLocation
+
 module Kernel
   # Captures the given stream and returns it:
   #
@@ -23,14 +27,14 @@ module Kernel
   def capture(stream)
     stream = stream.to_s
     captured_stream = Tempfile.new(stream)
-    stream_io = eval("$#{stream}")
+    stream_io = eval("$#{stream}") # rubocop:disable Security/Eval
     origin_stream = stream_io.dup
     stream_io.reopen(captured_stream)
 
     yield
 
     stream_io.rewind
-    return captured_stream.read
+    captured_stream.read
   ensure
     captured_stream.close
     captured_stream.unlink
@@ -38,3 +42,5 @@ module Kernel
   end
   alias_method :silence, :capture
 end
+
+# rubocop:enable Style/EvalWithLocation

data/spec/support/matchers/image.rb

@@ -1,7 +1,9 @@
-require 'rspec/expectations'
+# frozen_string_literal: true
+
+require "rspec/expectations"
 
 RSpec::Matchers.define :have_image_with_rgb_color do |*expected|
-  expected = format('rgb(%s, %s, %s)', *expected.flatten)
+  expected = format("rgb(%s, %s, %s)", *expected.flatten) # rubocop:disable Style/FormatStringToken
 
   match do |actual|
     actual.image.include? expected

data/spec/support/matchers/name.rb

@@ -1,4 +1,6 @@
-require 'rspec/expectations'
+# frozen_string_literal: true
+
+require "rspec/expectations"
 
 RSpec::Matchers.define :have_name do |expected|
   match do |actual|

data/spec/support/rspec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec.configure do |config|
   config.expect_with :rspec do |expectations|
     expectations.include_chain_clauses_in_custom_matcher_descriptions = true
@@ -14,10 +16,10 @@ RSpec.configure do |config|
   config.warnings = false
 
   if config.files_to_run.one?
-    config.default_formatter = 'doc'
+    config.default_formatter = "doc"
   end
 
-  config.profile_examples = 10 if ENV.key? 'RSPEC_PROFILE'
+  config.profile_examples = 10 if ENV.key? "RSPEC_PROFILE"
 
   config.order = :random
   Kernel.srand config.seed

data/spec/support/shared_examples/generator.rb

@@ -1,16 +1,18 @@
-RSpec.shared_examples 'a structure generator' do |name|
-  subject {  described_class.new(seed, preset, svg_image) }
+# frozen_string_literal: true
 
-  let(:seed) { instance_double('GeoPattern::Seed') }
-  let(:preset) { instance_double('GeoPattern::PatternPreset') }
+RSpec.shared_examples "a structure generator" do |name|
+  subject { described_class.new(seed, preset, svg_image) }
+
+  let(:seed) { instance_double("GeoPattern::Seed") }
+  let(:preset) { instance_double("GeoPattern::PatternPreset") }
   let(:svg_image) { SvgImage.new }
-  let(:pattern) { instance_double('GeoPattern::Pattern') }
+  let(:pattern) { instance_double("GeoPattern::Pattern") }
 
   let(:name) { name }
 
-  let(:fill_color_dark) { '#222' }
-  let(:fill_color_light) { '#ddd' }
-  let(:stroke_color) { '#000' }
+  let(:fill_color_dark) { "#222" }
+  let(:fill_color_light) { "#ddd" }
+  let(:stroke_color) { "#000" }
   let(:stroke_opacity) { 0.02 }
   let(:opacity_min) { 0.02 }
   let(:opacity_max) { 0.15 }
@@ -37,10 +39,10 @@ RSpec.shared_examples 'a structure generator' do |name|
     subject.generate(pattern)
   end
 
-  it_behaves_like 'a named generator', name
+  it_behaves_like "a named generator", name
 end
 
-RSpec.shared_examples 'a named generator' do |name|
+RSpec.shared_examples "a named generator" do |name|
   it { is_expected.to have_name name }
   it { is_expected.to have_name name.to_s }
 end

data/spec/support/shared_examples/pattern.rb

@@ -1,4 +1,6 @@
-RSpec.shared_examples 'a chosen pattern' do |name|
+# frozen_string_literal: true
+
+RSpec.shared_examples "a chosen pattern" do |name|
   subject(:pattern) { GeoPattern.generate(input, patterns: name) }
 
   let(:file_name) { "#{name}.svg" }
@@ -8,16 +10,16 @@ RSpec.shared_examples 'a chosen pattern' do |name|
   end
 
   it { expect(pattern.structure).to be_name name }
-  it { check_binary_file_content(file_name, fixtures_path("generated_patterns/#{name}.svg")) }
+  it { expect(file_name).to have_same_file_content_as("%/generated_patterns/#{name}.svg") }
 end
 
-RSpec.shared_examples 'an invalid pattern' do |chosen_pattern|
+RSpec.shared_examples "an invalid pattern" do |chosen_pattern|
   subject(:pattern) { GeoPattern.generate(input, patterns: chosen_pattern) }
 
   it { expect { subject }.to raise_error InvalidPatternError }
 end
 
-RSpec.shared_examples 'an old style pattern' do |chosen_pattern, name|
+RSpec.shared_examples "an old style pattern" do |chosen_pattern, name|
   subject(:pattern) { GeoPattern.generate(input, patterns: chosen_pattern) }
 
   let(:file_name) { "#{name}.svg" }
@@ -27,5 +29,5 @@ RSpec.shared_examples 'an old style pattern' do |chosen_pattern, name|
   end
 
   it { expect(pattern.structure).to be_name name }
-  it { check_binary_file_content(file_name, fixtures_path("generated_patterns/#{name}.svg")) }
+  it { expect(file_name).to have_same_file_content_as("%/generated_patterns/#{name}.svg") }
 end

data/spec/support/shared_examples/pattern_name.rb

@@ -1,7 +1,9 @@
-RSpec.shared_examples 'a known pattern name' do
+# frozen_string_literal: true
+
+RSpec.shared_examples "a known pattern name" do
   it { silence(:stderr) { expect(pattern_store[pattern_name]).not_to be_nil } }
 end
 
-RSpec.shared_examples 'an unknown pattern name' do
+RSpec.shared_examples "an unknown pattern name" do
   it { silence(:stderr) { expect(pattern_store[pattern_name]).to be_nil } }
 end

data/spec/support/shared_examples/structure.rb

@@ -1,4 +1,6 @@
-RSpec.shared_examples 'a metadata argument' do |argument|
+# frozen_string_literal: true
+
+RSpec.shared_examples "a metadata argument" do |argument|
   describe "##{argument}" do
     it { expect(metadata.public_send(argument)).to eq public_send(argument) }
   end
@@ -10,38 +12,38 @@ RSpec.shared_examples 'a metadata argument' do |argument|
     end
 
     context "when #{argument} is defined" do
-      context 'when argument is not given' do
+      context "when argument is not given" do
         it { is_expected.to public_send(:"be_#{argument}", nil) }
       end
 
-      context 'when argument is the same as the defined one' do
+      context "when argument is the same as the defined one" do
         it { is_expected.to public_send(:"be_#{argument}", public_send(argument)) }
       end
 
-      context 'when argument is different from the defined one' do
-        it { is_expected.not_to public_send(:"be_#{argument}", 'blub') }
+      context "when argument is different from the defined one" do
+        it { is_expected.not_to public_send(:"be_#{argument}", "blub") }
       end
     end
   end
 end
 
-RSpec.shared_examples 'a forwarded metadata argument' do |argument|
+RSpec.shared_examples "a forwarded metadata argument" do |argument|
   describe "##{argument}" do
     it { expect(metadata.public_send(argument)).to eq public_send(argument) }
   end
 
   describe "##{argument}?" do
     context "when #{argument} is defined" do
-      context 'when argument is not given' do
+      context "when argument is not given" do
         it { is_expected.to public_send(:"be_#{argument}", nil) }
       end
 
-      context 'when argument is the same as the defined one' do
+      context "when argument is the same as the defined one" do
         it { is_expected.to public_send(:"be_#{argument}", public_send(argument)) }
       end
 
-      context 'when argument is different from the defined one' do
-        it { is_expected.not_to public_send(:"be_#{argument}", 'blub') }
+      context "when argument is different from the defined one" do
+        it { is_expected.not_to public_send(:"be_#{argument}", "blub") }
       end
     end
   end

data/spec/support/string.rb

@@ -1,2 +1,3 @@
-# encoding: utf-8
-require 'active_support/core_ext/string/strip'
+# frozen_string_literal: true
+
+require "active_support/core_ext/string/strip"

data/spec/svg_spec.rb

@@ -1,8 +1,10 @@
-require 'spec_helper'
+# frozen_string_literal: true
+
+require "spec_helper"
 
 RSpec.describe SvgImage do
-  context '#<=>' do
-    it 'is comparable' do
+  context "#<=>" do
+    it "is comparable" do
       svg1 = SvgImage.new
       svg2 = SvgImage.new