Refactor specs

- Create `spec_helper`
- Disable random spec order to allow spec to pass
- Update RuboCop
- Use `described_class` (autofix `RSpec/DescribedClass` offenses)
- Remove unused variables to fix the following warnings:
    ```
    spec/cached_asset_file_spec.rb:51: warning: assigned but unused variable - known_document_0
    spec/transformation_pipeline/transformations/transformation_spec.rb:19: warning: assigned but unused variable - svg
    ```
- Add SimpleCov for code coverage

Author: tagliala

.gitignore

@@ -2,17 +2,18 @@
 *.rbc
 .bundle
 .config
+.ruby-*
 .yardoc
-Gemfile.lock
-InstalledFiles
-_yardoc
 coverage
 doc/
+Gemfile.lock
+InstalledFiles
 lib/bundler/man
 pkg
 rdoc
+spec/examples.txt
 spec/reports
 test/tmp
 test/version_tmp
 tmp
-.ruby-*
+_yardoc

.rspec

@@ -0,0 +1 @@
+--require spec_helper

.rubocop_todo.yml

@@ -10,8 +10,10 @@ gem "pry"
 gem "rake"
 gem "rspec"
 
-gem "rubocop", "1.66.1"
-gem "rubocop-packaging", "0.5.2"
-gem "rubocop-performance", "1.22.1"
-gem "rubocop-rake", "0.6.0"
-gem "rubocop-rspec", "3.0.5"
+gem "rubocop", "1.69.0", require: false
+gem "rubocop-packaging", "0.5.2", require: false
+gem "rubocop-performance", "1.23.0", require: false
+gem "rubocop-rake", "0.6.0", require: false
+gem "rubocop-rspec", "3.2.0", require: false
+
+gem "simplecov", require: false

Gemfile

@@ -1,29 +1,28 @@
 # frozen_string_literal: true
 
-require "inline_svg/finds_asset_paths"
-require "inline_svg/asset_file"
+require 'spec_helper'
 
-describe InlineSvg::AssetFile do
+RSpec.describe InlineSvg::AssetFile do
   it "reads data from a file, after qualifying a full path" do
     example_svg_path = File.expand_path(__FILE__, 'files/example.svg')
     expect(InlineSvg::FindsAssetPaths).to receive(:by_filename).with('some filename').and_return example_svg_path
 
-    expect(InlineSvg::AssetFile.named('some filename')).to include('This is a test')
+    expect(described_class.named('some filename')).to include('This is a test')
   end
 
   it "complains when the file cannot be read" do
     allow(InlineSvg::FindsAssetPaths).to receive(:by_filename).and_return('/this/path/does/not/exist')
 
     expect do
-      InlineSvg::AssetFile.named('some missing file')
+      described_class.named('some missing file')
     end.to raise_error InlineSvg::AssetFile::FileNotFound
   end
 
   it "complains when the file path was not found" do
     allow(InlineSvg::FindsAssetPaths).to receive(:by_filename).and_return(nil)
 
     expect do
-      InlineSvg::AssetFile.named('some missing file')
+      described_class.named('some missing file')
     end.to raise_error InlineSvg::AssetFile::FileNotFound
   end
 end

spec/asset_file_spec.rb

@@ -1,21 +1,20 @@
 # frozen_string_literal: true
 
-require 'pathname'
-require "inline_svg"
+require 'spec_helper'
 
-describe InlineSvg::CachedAssetFile do
+RSpec.describe InlineSvg::CachedAssetFile do
   let(:fixture_path) { Pathname.new(File.expand_path('files/static_assets', __dir__)) }
 
   it "loads assets under configured paths" do
     known_document = File.read(fixture_path.join("assets0", "known-document.svg"))
 
-    asset_loader = InlineSvg::CachedAssetFile.new(paths: fixture_path.join("assets0"))
+    asset_loader = described_class.new(paths: fixture_path.join("assets0"))
 
     expect(asset_loader.named("known-document.svg")).to eq(known_document)
   end
 
   it "does not include assets outside of configured paths" do
-    asset_loader = InlineSvg::CachedAssetFile.new(paths: fixture_path.join("assets0"))
+    asset_loader = described_class.new(paths: fixture_path.join("assets0"))
 
     expect(fixture_path.join("assets1", "other-document.svg")).to be_file
     expect do
@@ -29,7 +28,7 @@
 
     expect(known_document_0).not_to eq(known_document_1)
 
-    asset_loader = InlineSvg::CachedAssetFile.new(paths: fixture_path)
+    asset_loader = described_class.new(paths: fixture_path)
 
     expect(known_document_0).to eq(asset_loader.named("assets0/known-document.svg"))
     expect(known_document_1).to eq(asset_loader.named("assets1/known-document.svg"))
@@ -41,17 +40,16 @@
 
     expect(known_document).not_to eq(known_document_2)
 
-    asset_loader = InlineSvg::CachedAssetFile.new(paths: fixture_path.join("assets0"), filters: /\.svg/)
+    asset_loader = described_class.new(paths: fixture_path.join("assets0"), filters: /\.svg/)
 
     expect(asset_loader.named("known-document")).to eq(known_document)
     expect(asset_loader.named("known-document-two")).to eq(known_document_2)
   end
 
   it "filters wanted files by simple string matching" do
-    known_document_0 = File.read(fixture_path.join("assets0", "known-document.svg"))
     known_document_1 = File.read(fixture_path.join("assets1", "known-document.svg"))
 
-    asset_loader = InlineSvg::CachedAssetFile.new(paths: fixture_path, filters: "assets1")
+    asset_loader = described_class.new(paths: fixture_path, filters: "assets1")
 
     expect do
       asset_loader.named("assets0/known-document.svg")
@@ -63,7 +61,7 @@
   it "filters wanted files by regex matching" do
     known_document_1 = File.read(fixture_path.join("assets1", "known-document.svg"))
 
-    asset_loader = InlineSvg::CachedAssetFile.new(paths: fixture_path, filters: ["assets1", /\.svg/])
+    asset_loader = described_class.new(paths: fixture_path, filters: ["assets1", /\.svg/])
 
     expect do
       asset_loader.named("assets1/some-file.txt")

spec/cached_asset_file_spec.rb

@@ -1,9 +1,8 @@
 # frozen_string_literal: true
 
-require 'pathname'
-require "inline_svg"
+require 'spec_helper'
 
-describe InlineSvg::FindsAssetPaths do
+RSpec.describe InlineSvg::FindsAssetPaths do
   context "when sprockets finder returns an object which supports only the pathname method" do
     it "returns fully qualified file paths from Sprockets" do
       sprockets = double('SprocketsDouble')
@@ -15,7 +14,7 @@
         config.asset_finder = sprockets
       end
 
-      expect(InlineSvg::FindsAssetPaths.by_filename('some-file')).to eq Pathname('/full/path/to/some-file')
+      expect(described_class.by_filename('some-file')).to eq Pathname('/full/path/to/some-file')
     end
   end
 
@@ -30,7 +29,7 @@
         config.asset_finder = sprockets
       end
 
-      expect(InlineSvg::FindsAssetPaths.by_filename('some-file')).to eq Pathname('/full/path/to/some-file')
+      expect(described_class.by_filename('some-file')).to eq Pathname('/full/path/to/some-file')
     end
   end
 
@@ -44,7 +43,7 @@
         config.asset_finder = sprockets
       end
 
-      expect(InlineSvg::FindsAssetPaths.by_filename('some-file')).to be_nil
+      expect(described_class.by_filename('some-file')).to be_nil
     end
   end
 
@@ -59,7 +58,7 @@
         config.asset_finder = propshaft
       end
 
-      expect(InlineSvg::FindsAssetPaths.by_filename('some-file')).to eq Pathname('/full/path/to/some-file')
+      expect(described_class.by_filename('some-file')).to eq Pathname('/full/path/to/some-file')
     end
   end
 
@@ -74,7 +73,7 @@
         config.asset_finder = shakapacker
       end
 
-      expect(InlineSvg::FindsAssetPaths.by_filename('some-file')).to eq Pathname('/full/path/to/some-file')
+      expect(described_class.by_filename('some-file')).to eq Pathname('/full/path/to/some-file')
     end
   end
 
@@ -89,7 +88,7 @@
         config.asset_finder = shakapacker
       end
 
-      expect(InlineSvg::FindsAssetPaths.by_filename('some-file')).to eq Pathname('https://my-fancy-domain.test/full/path/to/some-file')
+      expect(described_class.by_filename('some-file')).to eq Pathname('https://my-fancy-domain.test/full/path/to/some-file')
     end
   end
 end

spec/finds_asset_paths_spec.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'inline_svg'
+require 'spec_helper'
 
 class WorkingCustomTransform < InlineSvg::CustomTransformation
   def transform(doc)
@@ -11,7 +11,7 @@ def transform(doc)
   end
 end
 
-describe InlineSvg::ActionView::Helpers do
+RSpec.describe InlineSvg::ActionView::Helpers do
   let(:helper) { (Class.new { include InlineSvg::ActionView::Helpers }).new }
 
   shared_examples "inline_svg helper" do |helper_method:|

spec/helpers/inline_svg_spec.rb

@@ -1,12 +1,12 @@
 # frozen_string_literal: true
 
-require "inline_svg/id_generator"
+require 'spec_helper'
 
-describe InlineSvg::IdGenerator do
+RSpec.describe InlineSvg::IdGenerator do
   it "generates a hexencoded ID based on a salt and a random value" do
     randomizer = -> { "some-random-value" }
 
-    expect(InlineSvg::IdGenerator.generate("some-base", "some-salt", randomness: randomizer))
+    expect(described_class.generate("some-base", "some-salt", randomness: randomizer))
       .to eq("at2c17mkqnvopy36iccxspura7wnreqf")
   end
 end