camping 3.0.2 → 3.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6aa52530d539466610a0ae7b505a72176f38b2ba12975f776447dcfb4e6d6934
-  data.tar.gz: 31c521e39db07c10472bff1d6e56bec41b55052bf0e40b3a8165e1266873815b
+  metadata.gz: afe5fa876178ba1acb276392788f76f4df3451c8b212234892565b6cd93043b0
+  data.tar.gz: c521c1b9e5c149474a4590efbc17fe61bd681bc79e510441bf1bd572e6505351
 SHA512:
-  metadata.gz: 200a6af8d4e245e7bf1fac8bb5965ec9ec2a049fc2b3965b9129e5fcc296f251de7ec5fd83656bde13dac6e2869692f52dd5d707e5718e84a498da1f742f7d07
-  data.tar.gz: b80af113694c5303ab1c955876dd6e6e0eb78d0324fa2b4d16f9361b8c8114e0af1cfdffd6761d452a22a3ee5c8febc679078ed50978a2d8a1385806d6da208b
+  metadata.gz: 7c77c29289f3e917c182178a3c0593f4d5b0ae3352a08fde92a4787ec6d4b98330108fe37860cf73f45896b37d32235918fafe123f674ba93088aeb1760958a3
+  data.tar.gz: 8b95dff758b4fb8adac1704bce9de1980c50ee25a8c1bf8492e3cbc9d847cde87b131e24329fea495ab4e4ec62925ba578f3c30a9c17b9aae7514038d57f79b7

data/README.md

@@ -64,27 +64,38 @@ end
 ## Installation
 
 Interested yet?  Luckily it's quite easy to install Camping.  We'll be using
-a tool called RubyGems, so if you don't have that installed yet, go grab it!
-Once that's sorted out, open up a Terminal or Command Line and enter:
+a tool called RubyGems, and Bundler, so if you don't have that installed
+yet, go grab it! Once that's sorted out, open up a Terminal or Command
+Line and enter:
 
 ```
 gem install camping
 ```
 
+Also make certain to have Bundler installed:
+
+```
+gem install bundler
+```
+
 ~~Even better, install the Camping Omnibus, a full package of recommended libs:~~ Camping Omnibus will return for summer vacation.
 
+Now make a new directory filled with your camp essentials using the `camping new` command:
+
+```
+camping new Donuts # You can replace Donuts with whatever but CamelCased.
+```
+
+Move to your new directory, then use bundler to install all of your camp's dependencies:
+
 ```
-gem install camping-omnibus
+cd donuts; bundle install
 ```
 
-If not, you should be aware of that Camping itself only depends on
-[Rack](https://github.com/rack/rack), and if you're going to use the views you also
-need to install **[markaby](https://github.com/markaby/markaby)**, and if you're going to use the database you need
-**activerecord** as well.
+You can now run camping using the `camping` command. We recommend running camping in development mode locally. Make certain to prefix the camping command with `bundle exec` to run your app with the gems you've installed just for your camp:
 
 ```
-gem install markaby
-gem install activerecord
+bundle exec camping -e development
 ```
 
 ## Learning

data/Rakefile

@@ -70,6 +70,18 @@ Rake::TestTask.new(:test) do |t|
   t.test_files = FileList['test/app_*.rb', 'test/gear/gear_*.rb']
 end
 
+## Reloader Tests
+Rake::TestTask.new(:reloader) do |t|
+  t.libs << "test"
+  t.test_files = FileList['test/reload_*.rb']
+end
+
+## Reloader Tests
+Rake::TestTask.new(:configreloader) do |t|
+  t.libs << "test"
+  t.test_files = FileList['test/config_*.rb']
+end
+
 ## Diff
 desc "Compare camping and camping-unabridged"
 task :diff do
@@ -97,7 +109,7 @@ end
 error = false
 
 ## Check
-task :check => ["test", "check:valid", "check:equal", "check:size", "check:lines", "check:exit"]
+task :check => ["test", "reloader", "configreloader", "check:valid", "check:equal", "check:size", "check:lines", "check:exit"]
 namespace :check do
 
   desc "Check source code validity"

data/lib/camping/campguide.rb

@@ -0,0 +1,14 @@
+# Campguide is a small helper to map common errors in a Camping project to a quick resolution.
+module Campguide
+	Errors = {"wrong number of arguments (given 3, expected 0)" => "ArgumentError. This is sometimes caused when you try to send a request to a controller when a camping app hasn't made camp yet. make certain to call Camping.make_camp to set up your apps."}
+
+	class << self
+
+		# accepts string error messages and tries to match them with better explanations of the error.
+		def make_sense(error_message)
+			message = Errors[error_message]
+			puts message ? message : error_message
+		end
+
+	end
+end

data/lib/camping/commands.rb

@@ -237,9 +237,79 @@ task :test => 'test:all'
 namespace 'test' do
   Rake::TestTask.new('all') do |t|
     t.libs << 'test'
-    t.test_files = FileList['test/nuts_*.rb']
+    t.test_files = FileList['test/test_*.rb']
   end
 end
+TXT
+      end
+
+      # writes a test_helper
+      def make_test_helper
+        write 'test/test_helper.r', <<-TXT
+# Test Helper
+$:.unshift File.dirname(__FILE__) + '/../'
+
+begin
+  require 'rubygems'
+rescue LoadError
+end
+
+require 'minitest/autorun'
+require 'rack/test'
+require 'minitest/reporters'
+require 'minitest/hooks'
+Minitest::Reporters.use! [Minitest::Reporters::DefaultReporter.new(:color => true)]
+
+# Default TestCase with some helpers.
+# The same stuff we use for Camping itself.
+class TestCase < MiniTest::Test
+  include Rack::Test::Methods
+  include Minitest::Hooks
+
+  def self.inherited(mod)
+    mod.app = Object.const_get(mod.to_s[/\w+/])
+    super
+  end
+
+  class << self
+    attr_accessor :app
+  end
+
+  def setup
+    super
+    Camping.make_camp
+  end
+
+  def body() last_response.body end
+  def app()  self.class.app     end
+
+  def response_body() last_response.to_a end
+
+  def assert_reverse
+    begin
+      yield
+    rescue Exception
+    else
+      assert false, "Block didn't fail"
+    end
+  end
+
+  def assert_body(str, message="")
+    case str
+    when Regexp
+      assert_match(str, last_response.body.strip, message)
+    else
+      assert_equal(str.to_s, last_response.body.strip, message)
+    end
+  end
+
+  def assert_status(code, message="")
+    assert_equal(code, last_response.status, message)
+  end
+
+  def test_silly; end
+end
+
 TXT
       end
 
@@ -274,6 +344,7 @@ group :test do
   gem 'minitest', '~> 5.0'
   gem 'minitest-reporters'
   gem 'rack-test'
+  gem 'minitest-hooks'
 end
 
 GEM
@@ -287,6 +358,8 @@ Camping is really fun and I hope you enjoy it.
 
 Start camping by running: `camping` in the root directory.
 
+To start Camping in development mode run: `camping -e development
+
 READ
       end
 
@@ -360,47 +433,55 @@ RUBY
   end
 
   class Commands
+    class << self
 
-    # A helper method to spit out Routes for an application
-    def self.routes(theApp = Camping, silent = false)
-      routes = Camping::CommandsHelpers::RoutesParser.parse theApp
-      routes.display unless silent == true
-      return routes
-    end
+      # A helper method to spit out Routes for an application
+      def routes(theApp = Camping, silent = false)
+        routes = Camping::CommandsHelpers::RoutesParser.parse theApp
+        routes.display unless silent == true
+        return routes
+      end
 
-    def self.new_cmd(app_name=:Camp)
-
-      # Normalize the app_name
-      Camping::CommandsHelpers.app_name_from_input(app_name) => {app_name:, snake_name:, camel_name:}
-
-      # make a directory then move there.
-      # _original_dir = Dir.pwd
-      Dir.mkdir("#{snake_name}") unless Dir.exist?("#{snake_name}")
-      Dir.chdir("#{snake_name}")
-
-      # generate a new camping app in a directory named after it:
-      Generators::make_camp_file(camel_name)
-      Generators::make_gitignore()
-      Generators::make_rakefile()
-      Generators::make_ruby_version()
-      Generators::make_configkdl()
-      Generators::make_gemfile()
-      Generators::make_readme()
-      Generators::make_public_folder()
-      Generators::make_test_folder()
-
-      # optionally add omnibus support
-        # add src/ folder
-        # add lib/ folder
-        # add views/ folder
-
-      # optionally add a local database too, through guidebook
-        # add db/ folder
-        # add db/migrate folder
-        # add db/config.kdl
-        # append migrations stuff to Rakefile.
-        `ls`
-    end
+      def new_cmd(app_name=:Camp)
+
+        # Normalize the app_name
+        Camping::CommandsHelpers.app_name_from_input(app_name) => {app_name:, snake_name:, camel_name:}
+
+        # make a directory then move there.
+        # _original_dir = Dir.pwd
+        Dir.mkdir("#{snake_name}") unless Dir.exist?("#{snake_name}")
+        Dir.chdir("#{snake_name}")
+
+        # generate a new camping app in a directory named after it:
+        Generators::make_camp_file(camel_name)
+        Generators::make_gitignore()
+        Generators::make_rakefile()
+        Generators::make_ruby_version()
+        Generators::make_configkdl()
+        Generators::make_gemfile()
+        Generators::make_readme()
+        Generators::make_public_folder()
+        Generators::make_test_folder()
+        Generators::make_test_helper()
+
+        # optionally add omnibus support
+          # add src/ folder
+          # add lib/ folder
+          # add views/ folder
+
+        # optionally add a local database too, through guidebook
+          # add db/ folder
+          # add db/migrate folder
+          # add db/config.kdl
+          # append migrations stuff to Rakefile.
+          `ls`
+      end
+
+      # TODO: Create this generator
+      # generates the apps folder from apps found in camp.rb or config.ru
+      # def generate_apps_folder
+      # end
 
+    end
   end
 end

data/lib/camping/gear/filters.rb

@@ -1,48 +1,48 @@
 # This gear is originally from filtering_camping gem by judofyr, and techarc.
 module Gear
-	module Filters
-		module ClassMethods
-			def filters
-				@_filters ||= {:before => [], :after => []}
-			end
+  module Filters
+    module ClassMethods
+      def filters
+        @_filters ||= {:before => [], :after => []}
+      end
 
-			def before(actions, &blk)
-				actions = [actions] unless actions.is_a?(Array)
-				actions.each do |action|
-					filters[:before] << [action, blk]
-				end
-			end
+      def before(actions, &blk)
+        actions = [actions] unless actions.is_a?(Array)
+        actions.each do |action|
+          filters[:before] << [action, blk]
+        end
+      end
 
-			def after(actions, &blk)
-				actions = [actions] unless actions.is_a?(Array)
-				actions.each do |action|
-					filters[:after] << [action, blk]
-				end
-			end
-		end
+      def after(actions, &blk)
+        actions = [actions] unless actions.is_a?(Array)
+        actions.each do |action|
+          filters[:after] << [action, blk]
+        end
+      end
+    end
 
-		def self.included(mod)
-			mod.extend(ClassMethods)
-		end
+    def self.included(mod)
+      mod.extend(ClassMethods)
+    end
 
-		def self.setup(app, *a, &block) end
+    def self.setup(app, *a, &block) end
 
-		def run_filters(type)
-			o = self.class.to_s.split("::")
-			filters = Object.const_get(o.first).filters
-			filters[type].each do |filter|
-				if (filter[0].is_a?(Symbol) && (filter[0] == o.last.to_sym || filter[0] == :all)) ||
-				 	(filter[0].is_a?(String) && /^#{filter[0]}\/?$/ =~ @env.REQUEST_URI)
-				 	self.instance_eval(&filter[1])
-				end
-			end
-		end
+    def run_filters(type)
+      o = self.class.to_s.split("::")
+      filters = Object.const_get(o.first).filters
+      filters[type].each do |filter|
+        if (filter[0].is_a?(Symbol) && (filter[0] == o.last.to_sym || filter[0] == :all)) ||
+           (filter[0].is_a?(String) && /^#{filter[0]}\/?$/ =~ @env.REQUEST_URI)
+           self.instance_eval(&filter[1])
+        end
+      end
+    end
 
-		def service(*a)
-			run_filters(:before)
-			super(*a)
-			run_filters(:after)
-			self
-		end
-	end
+    def service(*a)
+      run_filters(:before)
+      super(*a)
+      run_filters(:after)
+      self
+    end
+  end
 end

data/lib/camping/gear/kuddly.rb

@@ -1,5 +1,4 @@
 # frozen_string_literal: true
-# lib/camping/config
 # load and parse settings.
 
 begin

data/lib/camping/{reloader.rb → loader.rb}

@@ -1,3 +1,6 @@
+require 'zeitwerk'
+require 'listen'
+
 module Camping
   # == The Camping Reloader
   #
@@ -31,18 +34,39 @@ module Camping
   # Blog and Wiki, but they will update themselves if yourapp.rb changes.
   #
   # You can also give Reloader more than one script.
-  class Reloader
+  class Loader
     attr_reader :file
+    Loaders = []
 
-    def initialize(file, &blk)
+    def initialize(file=nil, &blk)
       @file = file
       @mtime = Time.at(0)
       @requires = []
       @apps = {}
       @callback = blk
-
+      @root = Dir.pwd
+      @file = @root + '/camp.rb' if @file == nil
+      @zeit = Zeitwerk::Loader.new
+      Loaders << @zeit
+
+      # setup Zeit for this reloader
+      setup_zeit(@zeit)
+
+      # setup recursive listener on the apps and lib directories from the source script.
+      @listener = Listen.to("#{@root}/apps", "#{@root}/lib", "#{@root}") do |modified, added, removed|
+        @mtime = Time.now
+        reload!
+      end
+      start
     end
 
+    # pass through methods to the Listener.
+    # for testing purposes.
+    def processing_events?;@listener.processing? end
+    def stop;@listener.stop end
+    def pause;@listener.pause end
+    def start;@listener.start end
+
     def name
       @name ||= begin
         base = @file.dup
@@ -52,29 +76,29 @@ module Camping
       end
     end
 
-    # Loads the apps available in this script.  Use <tt>apps</tt> to get
-    # the loaded apps.
-    def load_apps(old_apps)
+    # remove_constants called inside this.
+    def load_everything()
       all_requires = $LOADED_FEATURES.dup
       all_apps = Camping::Apps.dup
 
       load_file
+      reload_directory("#{@root}/apps")
+      reload_directory("#{@root}/lib")
+      Camping.make_camp
     ensure
       @requires = []
-      dirs = []
       new_apps = Camping::Apps - all_apps
 
       @apps = new_apps.inject({}) do |hash, app|
         if file = app.options[:__FILE__]
           full = File.expand_path(file)
           @requires << [file, full]
-          dirs << full.sub(/\.[^.]+$/, '')
         end
 
         key = app.name.to_sym
         hash[key] = app
 
-        if !old_apps.include?(key)
+        apps.each do |app|
           @callback.call(app) if @callback
           app.create if app.respond_to?(:create)
         end
@@ -84,12 +108,13 @@ module Camping
 
       ($LOADED_FEATURES - all_requires).each do |req|
         full = full_path(req)
-        @requires << [req, full] if dirs.any? { |x| full.index(x) == 0 }
+        @requires << [req, full] # if dirs.any? { |x| full.index(x) == 0 }
       end
 
       @mtime = mtime
 
       self
+
     end
 
     # load_file
@@ -102,13 +127,15 @@ module Camping
       else
         load(@file)
       end
+      @requires << [@file, File.expand_path(@file)]
     end
 
-    # Removes all the apps defined in this script.
-    def remove_apps
-      @requires.each do |(path, full)|
-        $LOADED_FEATURES.delete(path)
-      end
+    # removes all constants recursively included using this script as a root.
+    # so everything in /apps, and /lib in relation from this script.
+    def remove_constants
+    	@requires.each do |(path, full)|
+     		$LOADED_FEATURES.delete(path)
+     	end
 
       @apps.each do |name, app|
         Camping::Apps.delete(app)
@@ -116,6 +143,7 @@ module Camping
       end.dup
     ensure
       @apps.clear
+      @requires.clear
     end
 
     # Reloads the file if needed.  No harm is done by calling this multiple
@@ -125,8 +153,10 @@ module Camping
       reload!
     end
 
+    # Force a reload.
     def reload!
-      load_apps(remove_apps)
+      remove_constants
+      load_everything
     end
 
     # Checks if both scripts watches the same file.
@@ -144,6 +174,46 @@ module Camping
 
     private
 
+    # sets up Zeit autoloading for the script locations.
+    def setup_zeit(loader)
+      loader.push_dir("#{@root}/apps") if can_add_directory "#{@root}/apps"
+      loader.push_dir("#{@root}/lib") if can_add_directory "#{@root}/lib"
+	    loader.enable_reloading if ENV['environment'] == 'development'
+	    loader.setup
+    end
+
+    # verifies that we can add a directory to the loader.
+    # used for testing to prevent multiple loaders from watching the same directory.
+    def can_add_directory(directory)
+      if Dir.exist?("#{@root}/apps")
+        Loaders.each do |loader|
+          return false if loader.dirs.include? directory
+        end
+        true
+      else
+        false
+      end
+    end
+
+    # Splits the descendent files and folders found in a given directory for eager loading and recursion.
+    def folders_and_files_in(directory)
+      directory = directory + "/*" # unless directory
+      [Dir.glob(directory).select {|f| !File.directory? f },
+      Dir.glob(directory).select {|f| File.directory? f }]
+    end
+
+    # Reloads a directory recursively. loading more shallow files before deeper files.
+    def reload_directory(directory)
+      files, folders = folders_and_files_in(directory)
+      files.each {|file|
+        @requires << [file, File.expand_path(file)]
+        load file
+      }
+      folders.each {|folder|
+        reload_directory folder
+      }
+    end
+
     def mtime
       @requires.map do |(path, full)|
         File.mtime(full)

data/lib/camping/loads.rb

@@ -8,8 +8,8 @@ require 'bundler/setup'
 
 # internal dependencies
 require 'camping/tools'
+require 'camping/campguide'
 require 'camping/gear/filters'
 require 'camping/gear/nancy'
 require 'camping/gear/inspection'
 require 'camping/gear/kuddly'
-

data/lib/camping/server.rb

@@ -3,7 +3,7 @@ require 'erb'
 require 'rack'
 require 'rackup'
 require 'camping/version'
-require 'camping/reloader'
+require 'camping/loader'
 require 'camping/commands'
 
 # == The Camping Server (for development)
@@ -128,9 +128,10 @@ module Camping
         exit
       end
 
+      @reloader.reload!
+      r = @reloader
+
       if options[:routes] == true
-        @reloader.reload!
-        r = @reloader
         eval("self", TOPLEVEL_BINDING).meta_def(:reload!) { r.reload!; nil }
         ARGV.clear
         Camping::Commands.routes
@@ -139,16 +140,11 @@ module Camping
 
       if options[:server] == "console"
         puts "** Starting console"
-        @reloader.reload!
-        r = @reloader
         eval("self", TOPLEVEL_BINDING).meta_def(:reload!) { r.reload!; nil }
         ARGV.clear
         IRB.start
         exit
       else
-        @reloader.reload!
-        r = @reloader
-        Camping.make_camp
         name = server.name[/\w+$/]
         puts "** Starting #{name} on #{options[:Host]}:#{options[:Port]}"
         super
@@ -215,7 +211,6 @@ module Camping
     def call(env)
       if ENV['environment'] == 'development'
         @reloader.reload
-        Camping.make_camp
       end
 
       # our switch statement iterates through possible app outcomes, no apps

data/lib/camping/session.rb

@@ -1,5 +1,5 @@
 require 'rack/session'
-class InsecureSecret < Exception #:nodoc: all
+class InsecureSecretError < Exception #:nodoc: all
 end
 module Camping
   # == Getting Started
@@ -31,7 +31,7 @@ module Camping
     def self.included(app)
       key    = "#{app}.state".downcase
       secret = app.options[:secret] || ['camping-secret',__FILE__, File.mtime('Rakefile')].join(":")*2
-      raise InsecureSecret, "You're Session Secret is too short. Minimum length is 64." if secret.length < 64
+      raise InsecureSecretError, "Your Session Secret is too short. Minimum length is 64." if secret.length < 64
       app.use Rack::Session::Cookie, :key => key, :secrets => secret
     end
   end

data/lib/camping/version.rb

@@ -1,5 +1,5 @@
 module Camping
-  VERSION = "3.0.2"
+  VERSION = "3.1.0"
   def self.version
     VERSION
   end

data/lib/camping-unabridged.rb

@@ -724,7 +724,6 @@ module Camping
     #
     # See: https://github.com/rack/rack/blob/main/SPEC.rdoc
     def call(e)
-      make_camp # TODO: Find a better, more consistent place for setting everything up.
       k,m,*a=X.D e["PATH_INFO"],e['REQUEST_METHOD'].downcase,e
       k.new(e,m,prx).service(*a).to_a
     rescue
@@ -752,8 +751,9 @@ module Camping
     def method_missing(m, c, *a)
       h = Hash === a[-1] ? a.pop : {}
       e = H[Rack::MockRequest.env_for('',h.delete(:env)||{})]
-      # puts "method missing failure for controller: #{c}, method: #{m} "
       k = X.const_get(c).new(e,m.to_s,prx)
+   #  rescue => error # : wrong number of arguments
+			# Campguide::make_sense error.message
       h.each { |i, v| k.send("#{i}=", v) }
       k.service(*a)
     end
@@ -914,6 +914,7 @@ module Camping
       # setup caller data
       sp = caller[0].split('`')[0].split(":")
       fl, ln, pr = sp[0], sp[1].to_i, nil
+      # ln = 0
 
       # Create the app
       Apps << a = eval(S.gsub(/Camping/,m.to_s), g, fl, ln)

data/lib/camping.rb

@@ -66,7 +66,7 @@ k.meta_def(:urls){[A.(k,"#{c.to_s.scan(/.[^A-Z]*/).map(&N.method(:[]))*'/'}",p)]
 .respond_to?(:urls) || mu==true)};end end;I=R()end;X=Controllers
 class<<self;def make_camp;X.M prx;Apps.map(&:make_camp) end;def routes;(Apps.map(&:routes)<<X.v).flatten end
 def prx;@_prx||=CampTools.normalize_slashes(O[:url_prefix])end
-def call e;make_camp;k,m,*a=X.D e["PATH_INFO"],e['REQUEST_METHOD'].
+def call e;k,m,*a=X.D e["PATH_INFO"],e['REQUEST_METHOD'].
 downcase,e;k.new(e,m,prx).service(*a).to_a;rescue;r500(:I,k,m,$!,:env=>e).to_a end
 def method_missing m,c,*a;h=Hash===a[-1]?a.pop : {};e=H[Rack::MockRequest.
 env_for('',h.delete(:env)||{})];k=X.const_get(c).new(e,m.to_s,prx);h.each{|i,v|

data/test/app_config.rb

@@ -5,16 +5,14 @@ module Config end
 
 class Config::Test < TestCase
 
-  def setup
+  def before_all
     write_config()
     Camping.goes :Config
     @options = Camping::Apps.select{|a| a.name == "Config" }.first.options
-    super
   end
 
-  def teardown
+  def after_all
     trash_config()
-    super
   end
 
   def test_config

data/test/app_generator.rb

@@ -40,4 +40,43 @@ class Generator::Test < TestCase
 
   end
 
+  # Generates expected directory Structure
+  #
+  # Camping has an expected directory structure:
+  #
+  #   .gitignore
+  #   .ruby-version
+  #   Gemfile
+  #   Rakefile
+  #   camp.rb
+  #   config.kdl
+  #   src/
+  #   lib/
+  #   public/
+  #   test/
+  #   apps/
+  #
+  # This test checks to make certain that the generator command creates this
+  # directory structure.
+  def test_app_generates_directory_structure
+    move_to_tmp
+    Camping::Commands.new_cmd
+
+    res, ignored = [Dir.glob('*').select {|f| !File.directory? f },
+    Dir.glob('*').select {|f| File.directory? f }], Dir.glob(".*")
+
+    assert res[0].include?('Gemfile'), "mising Gemfile"
+    assert res[0].include?('README.md'), "missing README.md"
+    assert res[0].include?('Rakefile'), "missing Rakefile"
+    assert res[0].include?('camp.rb'), "missing camp.rb"
+    assert res[0].include?('config.kdl'), "missing config.kdl"
+    assert res[1].include?('public'), "missing public folder."
+    assert res[1].include?('test'), "missing test folder."
+
+    assert ignored.include?('.gitignore'), ".gitignore is missing."
+    assert ignored.include?('.ruby-version'), ".ruby-version is missing."
+
+    leave_tmp
+  end
+
 end

data/test/app_inception.rb

@@ -2,6 +2,13 @@ require 'test_helper'
 require 'camping'
 require 'camping/commands'
 
+# The idea behind the inception stuff is that you can inherit settings
+# templates, etc... from a Camping app and just... extend it.
+# This is a tabled, but not abandoned idea. It became a little difficult
+# to inherit all of these things, along with views, Models, etc... that's
+# associated with a camping app. Getting this to work would mean a big
+# rewrite of Camping's core, which is a no no.
+
 Camping.goes :Inception
 # Inception.goes :Leonardo
 # Leonardo.goes :Donatello

data/test/app_loader.rb

@@ -0,0 +1,53 @@
+require 'test_helper'
+require 'fileutils'
+require 'camping/loader'
+
+module Donuts end
+module Loader end
+
+# for Reloading stuff
+module TestCaseLoaderToo
+  def loader
+    @loader ||= Camping::Loader.new(file)
+  end
+
+  def before_all
+    super
+    move_to_loader
+    loader.reload!
+  end
+
+  def after_all
+    leave_loader
+    super
+  end
+end
+
+class Donuts::Test < TestCase
+  include TestCaseLoaderToo
+  BASE = File.expand_path('../apps/loader/camp', __FILE__)
+  def file; BASE + '.rb' end
+
+  def test_that_our_apps_are_there
+    assert loader.apps.include?(:Donuts), "Donuts not found: #{loader.apps}"
+    assert loader.apps.include?(:Loader), "Loader not found: #{loader.apps}"
+  end
+
+  def test_output
+    # Checks that the view is overwritten successfully more than once.
+    get '/'
+    assert_body "chunky bacon", "Response is wrong in the loader."
+    assert_equal "text/html", last_response.headers['content-type']
+
+    # Checks that the view is not overwritten, because it's not reopened.
+    get '/post'
+    assert_body "_why", "Response is wrong in the loader."
+    assert_equal "text/html", last_response.headers['content-type']
+
+    # Checks that a downstream view is loaded properly.
+    get '/people'
+    assert_body "People are great am I right?", "Response is wrong in the loader."
+    assert_equal "text/html", last_response.headers['content-type']
+  end
+
+end

data/test/app_sessions.rb

@@ -44,8 +44,8 @@ class Sessions::Test < TestCase
     app.set :secret, "whateverloser"
     begin
       app.include Camping::Session
-    rescue InsecureSecret => e
-      message = "You're Session Secret is too short. Minimum length is 64."
+    rescue InsecureSecretError => e
+      message = "Your Session Secret is too short. Minimum length is 64."
       assert_equal(e.message, message, "You're session secret wasn't long enough.")
     end
 
@@ -54,11 +54,11 @@ class Sessions::Test < TestCase
     begin
       app.set :secret, "whateverloserwhateverloserwhateverloserwhateverloserwhateverloser"
       app.include Camping::Session
-    rescue InsecureSecret => e
+    rescue InsecureSecretError => e
       message = e
     end
 
-    assert_equal(e, message, "You're session secret wasn't long enough.")
+    assert_equal(e, message, "Your session secret wasn't long enough.")
   end
 end
 

data/test/apps/loader/apps/donuts/controllers/index.rb

@@ -0,0 +1,18 @@
+module Donuts
+  module Controllers
+    class Index
+      def get
+        render :index
+      end
+    end
+  end
+
+  # how to add a layout to every page
+  module Views
+
+    def index
+      "chunky bacon"
+    end
+
+  end
+end

data/test/apps/loader/apps/donuts.rb

@@ -0,0 +1,15 @@
+module Donuts
+  module Controllers
+    class Index
+      def get
+        "We've got lots of donuts!"
+      end
+    end
+
+    class People
+      def get
+        "People are great am I right?"
+      end
+    end
+  end
+end

data/test/apps/loader/camp.rb

@@ -0,0 +1,35 @@
+require 'camping'
+
+Camping.goes :Loader
+
+$LOAD_PATH << File.dirname(__FILE__)
+
+Camping.goes :Donuts
+
+module Donuts
+  module Controllers
+    class Index
+      def get
+        render :index
+      end
+    end
+    class Post
+      def get
+        render :post
+      end
+    end
+  end
+
+  # how to add a layout to every page
+  module Views
+
+    def index
+      "_why"
+    end
+
+    def post
+      "_why"
+    end
+
+  end
+end

data/test/apps/reloader/config.ru

@@ -1,5 +1,5 @@
 $LOAD_PATH << File.dirname(__FILE__) + '/../'
+$LOAD_PATH << File.dirname(__FILE__) + '/'
 
 require 'reloader'
 run Reloader
-

data/test/apps/reloader/reload_me.rb

@@ -1,2 +1 @@
 $counter += 1
-

data/test/apps/{reloader.rb → reloader/reloader.rb}

@@ -3,5 +3,4 @@ require 'camping'
 Camping.goes :Reloader
 
 $LOAD_PATH << File.dirname(__FILE__)
-require 'reloader/reload_me'
-
+require 'reload_me'

data/test/config_reloader.rb

@@ -0,0 +1,78 @@
+require_relative 'test_helper'
+require 'fileutils'
+require 'camping/loader'
+
+$counter = 0
+
+# for Reloading stuff
+module TestCaseLoader
+
+  def loader
+    @loader ||= Camping::Loader.new(file)
+  end
+
+  def before_all
+    super
+    move_to_reloader
+    loader.reload!
+    assert Object.const_defined?(:Reloader), "Reloader didn't load app"
+    # puts "before_all called"
+  end
+
+  def after_all
+    # puts "after_all called"
+    assert Object.const_defined?(:Reloader), "Test removed app"
+    loader.remove_constants
+    # breaks in CI for some reason.
+    # assert !Object.const_defined?(:Reloader), "Loader didn't remove app"
+    leave_reloader
+    super
+  end
+
+end
+
+class TestConfigRu < TestCase
+  include TestCaseLoader
+
+  BASE = File.expand_path('../apps/reloader', __FILE__)
+  def file; BASE + '/config.ru' end
+
+  def setup
+    super
+    $counter = 0
+    loader.reload!
+    # puts "setup called"
+  end
+
+  def test_counter
+    assert_equal 1, $counter
+  end
+
+  def test_forced_reload
+    loader.reload!
+    assert_equal 2, $counter
+  end
+
+  def test_that_touch_was_touched
+    FileUtils.touch(BASE + '/reloader.rb')
+    assert_equal 1, $counter
+  end
+
+  def test_mtime_reload
+    loader.reload
+    assert_equal 1, $counter
+
+    FileUtils.touch(BASE + '/reloader.rb')
+    loader.reload
+    assert_equal 2, $counter
+
+    FileUtils.touch(BASE + '/reload_me.rb')
+    loader.reload
+    assert_equal 3, $counter
+  end
+
+  def test_name
+    assert_equal Reloader, loader.apps[:reloader]
+  end
+
+end

data/test/reload_reloader.rb

@@ -0,0 +1,81 @@
+require_relative 'test_helper'
+require 'fileutils'
+require 'camping/loader'
+
+$counter = 0
+
+# for Reloading stuff
+module TestCaseLoader
+
+  def loader
+    @loader ||= Camping::Loader.new(file)
+  end
+
+  def before_all
+    super
+    move_to_reloader
+    loader.reload!
+    assert Object.const_defined?(:Reloader), "Reloader didn't load app"
+    # puts "before_all called"
+  end
+
+  def after_all
+    # puts "after_all called"
+    assert Object.const_defined?(:Reloader), "Test removed app"
+    loader.remove_constants
+    assert !Object.const_defined?(:Reloader), "Loader didn't remove app"
+    leave_reloader
+    super
+  end
+
+end
+
+class TestLoader < TestCase
+  include TestCaseLoader
+
+  BASE = File.expand_path('../apps/reloader', __FILE__)
+  def file; BASE + '/reloader.rb' end
+
+  def setup
+    super
+    $counter = 0
+    loader.reload!
+    # puts "setup called"
+  end
+
+  def test_counter
+    assert_equal 1, $counter
+  end
+
+  def test_forced_reload
+    loader.reload!
+    assert_equal 2, $counter
+  end
+
+  def test_that_touch_was_touched
+    FileUtils.touch(BASE + '/reloader.rb')
+    assert_equal 1, $counter
+  end
+
+  def test_mtime_reload
+    loader.reload
+    assert_equal 1, $counter
+
+    FileUtils.touch(BASE + '/reloader.rb')
+    loader.reload
+    assert_equal 2, $counter
+
+    FileUtils.touch(BASE + '/reload_me.rb')
+    loader.reload
+    assert_equal 3, $counter
+  end
+end
+
+# class TestConfigRu < TestLoader
+#   BASE = File.expand_path('../apps/reloader', __FILE__)
+#   def file; BASE + '/config.ru' end
+
+#   def test_name
+#     assert_equal Reloader, loader.apps[:reloader]
+#   end
+# end

data/test/test_helper.rb

@@ -14,7 +14,8 @@ end
 
 require 'minitest/autorun'
 require 'rack/test'
-require "minitest/reporters"
+require 'minitest/reporters'
+require 'minitest/hooks'
 Minitest::Reporters.use! [Minitest::Reporters::DefaultReporter.new(:color => true)]
 
 
@@ -32,6 +33,40 @@ module CommandLineCommands
     `rm -rf test/tmp` if File.exist?('test/tmp')
   end
 
+  # reloader helpers:
+  # move_to_apps
+  # moves to the apps directory in /test
+  def move_to_reloader
+    @original_dir = Dir.pwd
+    Dir.chdir "test"
+    Dir.chdir "apps"
+    Dir.chdir "reloader"
+   	Dir.mkdir("apps") unless Dir.exist?("apps")
+    Dir.mkdir("lib") unless Dir.exist?("lib")
+  end
+
+  # deletes the temporary directories found in the /apps directory for reloader testing.
+  def leave_reloader
+	  Dir.chdir @original_dir
+	  `rm -rf test/apps/reloader/apps` if File.exist?('test/apps/reloader/apps')
+	  `rm -rf test/apps/reloader/lib` if File.exist?('test/apps/reloader/lib')
+  end
+
+  # Moves to the loader directory
+  def move_to_loader
+    @original_dir = Dir.pwd
+    Dir.chdir "test"
+    Dir.chdir "apps"
+    Dir.chdir "loader"
+   	Dir.mkdir("apps") unless Dir.exist?("apps")
+    Dir.mkdir("lib") unless Dir.exist?("lib")
+  end
+
+  # deletes the temporary directories found in the /apps directory for reloader testing.
+  def leave_loader
+	  Dir.chdir @original_dir
+  end
+
   def write(file, content)
     raise "cannot write nil" unless file
     file = tmp_file(file)
@@ -70,6 +105,7 @@ end
 class TestCase < MiniTest::Test
   include Rack::Test::Methods
   include CommandLineCommands
+  include Minitest::Hooks
 
   def self.inherited(mod)
     mod.app = Object.const_get(mod.to_s[/\w+/])
@@ -80,6 +116,11 @@ class TestCase < MiniTest::Test
     attr_accessor :app
   end
 
+  def setup
+    super
+    Camping.make_camp
+  end
+
   def body() last_response.body end
   def app()  self.class.app     end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: camping
 version: !ruby/object:Gem::Version
-  version: 3.0.2
+  version: 3.1.0
 platform: ruby
 authors:
 - why the lucky stiff
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-04-18 00:00:00.000000000 Z
+date: 2023-06-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: mab
@@ -153,14 +153,15 @@ files:
 - lib/camping-unabridged.rb
 - lib/camping.rb
 - lib/camping/ar.rb
+- lib/camping/campguide.rb
 - lib/camping/commands.rb
 - lib/camping/gear/filters.rb
 - lib/camping/gear/inspection.rb
 - lib/camping/gear/kuddly.rb
 - lib/camping/gear/nancy.rb
+- lib/camping/loader.rb
 - lib/camping/loads.rb
 - lib/camping/mab.rb
-- lib/camping/reloader.rb
 - lib/camping/server.rb
 - lib/camping/session.rb
 - lib/camping/template.rb
@@ -177,25 +178,30 @@ files:
 - test/app_helpers.rb
 - test/app_inception.rb
 - test/app_inline_templates.rb
+- test/app_loader.rb
 - test/app_markup.rb
 - test/app_migrations.rb
 - test/app_partials.rb
 - test/app_prefixed.rb
-- test/app_reloader.rb
 - test/app_route_generating.rb
 - test/app_sessions.rb
 - test/app_simple.rb
 - test/apps/env_debug.rb
 - test/apps/forms.rb
 - test/apps/forward_to_other_controller.rb
+- test/apps/loader/apps/donuts.rb
+- test/apps/loader/apps/donuts/controllers/index.rb
+- test/apps/loader/camp.rb
 - test/apps/migrations.rb
 - test/apps/misc.rb
-- test/apps/reloader.rb
 - test/apps/reloader/config.ru
 - test/apps/reloader/reload_me.rb
+- test/apps/reloader/reloader.rb
 - test/apps/reloader_indirect.rb
 - test/apps/sessions.rb
+- test/config_reloader.rb
 - test/gear/gear_nancy.rb
+- test/reload_reloader.rb
 - test/test_helper.rb
 homepage: http://camping.rubyforge.org/
 licenses: []

data/test/app_reloader.rb

@@ -1,69 +0,0 @@
-require 'test_helper'
-require 'fileutils'
-require 'camping/reloader'
-
-$counter = 0
-
-module TestCaseReloader
-  def reloader
-    @reloader ||= Camping::Reloader.new(file)
-  end
-
-  def setup
-    super
-    reloader.reload!
-    assert Object.const_defined?(:Reloader), "Reloader didn't load app"
-  end
-
-  def teardown
-    super
-    assert Object.const_defined?(:Reloader), "Test removed app"
-    reloader.remove_apps
-    assert !Object.const_defined?(:Reloader), "Reloader didn't remove app"
-  end
-end
-
-class TestReloader < TestCase
-  include TestCaseReloader
-  BASE = File.expand_path('../apps/reloader', __FILE__)
-
-  def file; BASE + '.rb' end
-
-  def setup
-    $counter = 0
-    super
-  end
-
-  def test_counter
-    assert_equal 1, $counter
-  end
-
-  def test_forced_reload
-    reloader.reload!
-    assert_equal 2, $counter
-  end
-
-  def test_mtime_reload
-    reloader.reload
-    assert_equal 1, $counter
-
-    FileUtils.touch(BASE + '.rb')
-    sleep 1
-    reloader.reload
-    assert_equal 2, $counter
-
-    FileUtils.touch(BASE + '/reload_me.rb')
-    sleep 1
-    reloader.reload
-    assert_equal 3, $counter
-  end
-end
-
-class TestConfigRu < TestReloader
-  BASE = File.expand_path('../apps/reloader', __FILE__)
-  def file; BASE + '/config.ru' end
-
-  def test_name
-    assert_equal Reloader, reloader.apps[:reloader]
-  end
-end