Remove purdytest, update Minitest class name

Also change S3 endpoint config to not require a trailing slash.

.github/workflows/ruby.yml

@@ -28,6 +28,6 @@ jobs:
       with:
         redis-version: ${{ matrix.redis-version }}
     - name: Configure
-      run: cp config.yml.example.s3 config.yml
+      run: cp config.yml.erb.example config.yml
     - name: Run tests
       run: bundle exec rake test

.gitignore

@@ -1,4 +1,5 @@
 config.yml
+config.yml.erb
 cs_credentials.json
 pids
 .bundle

Gemfile

@@ -12,8 +12,8 @@ gem "mime-types"
 group :test do
   gem 'rake'
   gem 'rack-test'
-  gem 'purdytest', :require => false
   gem 'm'
+  gem 'minitest'
   gem 'minitest-stub_any_instance'
   gem 'webmock'
 end

Gemfile.lock

@@ -41,8 +41,6 @@ GEM
       ruby2_keywords (~> 0.0.1)
     netrc (0.11.0)
     public_suffix (5.0.3)
-    purdytest (2.0.0)
-      minitest (~> 5.5)
     rack (2.2.8)
     rack-protection (2.2.4)
       rack
@@ -97,8 +95,8 @@ DEPENDENCIES
   activesupport (~> 6.1.0)
   m
   mime-types
+  minitest
   minitest-stub_any_instance
-  purdytest
   rack-test
   rainbows
   rake

config.yml.erb.example

@@ -0,0 +1,17 @@
+development: &defaults
+  maintenance: false
+  redis:
+    host: <%= ENV["REDIS_HOST"] || "localhost" %>
+    port: <%= ENV["REDIS_PORT"] || "6379" %>
+  s3: &s3_defaults
+    endpoint: <%= ENV["S3_ENDPOINT"] || "http://127.0.0.1:9000" %>
+    region: <%= ENV["S3_REGION"] %>
+    access_key_id: <%= ENV["S3_ACCESS_KEY"] || "minioadmin" %>
+    secret_key_id: <%= ENV["S3_SECRET_KEY"] || "minioadmin" %>
+    bucket: <%= ENV["S3_BUCKET"] || "rs-development" %>
+test:
+  <<: *defaults
+staging:
+  <<: *defaults
+production:
+  <<: *defaults

config.yml.example.s3

@@ -1,26 +0,0 @@
-development: &defaults
-  maintenance: false
-  s3:
-    endpoint: "https://some-endpoint"
-    region: "region"
-    access_key_id: ""
-    secret_key_id: ""
-    bucket: "test-bucket"
-  redis:
-    host: localhost
-    port: 6379
-
-test:
-  <<: *defaults
-  s3:
-    endpoint: "https://some-endpoint"
-    region: "region"
-    access_key_id: ""
-    secret_key_id: ""
-    bucket: "test-bucket"
-
-staging:
-  <<: *defaults
-
-production:
-  <<: *defaults

lib/remote_storage/s3.rb

@@ -106,7 +106,7 @@ module RemoteStorage
     end
 
     def container_url_for(user)
-      "#{base_url}#{settings.s3["bucket"]}/#{user}"
+      "#{base_url}/#{settings.s3["bucket"]}/#{user}"
     end
   end
 

liquor-cabinet.rb

@@ -18,7 +18,7 @@ class LiquorCabinet < Sinatra::Base
 
     register Sinatra::ConfigFile
     set :environments, %w{development test production staging}
-    config_file 'config.yml'
+    config_file 'config.yml.erb'
   end
 
   configure :development do

spec/s3/app_spec.rb

@@ -2,7 +2,7 @@ require_relative "../spec_helper"
 
 describe "S3 provider" do
   def container_url_for(user)
-    "#{app.settings.s3["endpoint"]}#{app.settings.s3["bucket"]}/#{user}"
+    "#{app.settings.s3["endpoint"]}/#{app.settings.s3["bucket"]}/#{user}"
   end
 
   def storage_class

spec/spec_helper.rb

@@ -8,7 +8,6 @@ require_relative '../liquor-cabinet'
 require 'minitest/autorun'
 require "minitest/stub_any_instance"
 require 'rack/test'
-require 'purdytest'
 require "redis"
 require "rest_client"
 require "ostruct"
@@ -36,22 +35,22 @@ if app.settings.respond_to? :redis
   end
 end
 
-MiniTest::Spec.class_eval do
+Minitest::Spec.class_eval do
   def self.shared_examples
     @shared_examples ||= {}
   end
 end
 
-module MiniTest::Spec::SharedExamples
+module Minitest::Spec::SharedExamples
   def shared_examples_for(desc, &block)
-    MiniTest::Spec.shared_examples[desc] = block
+    Minitest::Spec.shared_examples[desc] = block
   end
 
   def it_behaves_like(desc)
-    self.instance_eval(&MiniTest::Spec.shared_examples[desc])
+    self.instance_eval(&Minitest::Spec.shared_examples[desc])
   end
 end
 
-Object.class_eval { include(MiniTest::Spec::SharedExamples) }
+Object.class_eval { include(Minitest::Spec::SharedExamples) }
 
 require_relative 'shared_examples'