move clean logic to Application

lib/rscons.rb

@@ -57,25 +57,6 @@ module Rscons
       @application ||= Application.new
     end
 
-    # Remove all generated files.
-    #
-    # @return [void]
-    def clean
-      cache = Cache.instance
-      # remove all built files
-      cache.targets.each do |target|
-        FileUtils.rm_f(target)
-      end
-      # remove all created directories if they are empty
-      cache.directories.sort {|a, b| b.size <=> a.size}.each do |directory|
-        next unless File.directory?(directory)
-        if (Dir.entries(directory) - ['.', '..']).empty?
-          Dir.rmdir(directory) rescue nil
-        end
-      end
-      cache.clear
-    end
-
     # Return whether the given path is an absolute filesystem path.
     #
     # @param path [String] the path to examine.

lib/rscons/application.rb

@@ -29,11 +29,34 @@ module Rscons
     #   Process exit code (0 on success).
     def run(operation)
       # TODO
+      case operation
+      when "clean"
+        clean
+      end
       0
     end
 
     private
 
+    # Remove all generated files.
+    #
+    # @return [void]
+    def clean
+      cache = Cache.instance
+      # remove all built files
+      cache.targets.each do |target|
+        FileUtils.rm_f(target)
+      end
+      # remove all created directories if they are empty
+      cache.directories.sort {|a, b| b.size <=> a.size}.each do |directory|
+        next unless File.directory?(directory)
+        if (Dir.entries(directory) - ['.', '..']).empty?
+          Dir.rmdir(directory) rescue nil
+        end
+      end
+      cache.clear
+    end
+
     # Determine the number of threads to use by default.
     #
     # @return [Integer]

spec/rscons/application_spec.rb

@@ -1,6 +1,31 @@
 module Rscons
   describe Application do
 
+    describe ".clean" do
+      it "removes all build targets and created directories" do
+        cache = "cache"
+        expect(Rscons::Cache).to receive(:instance).and_return(cache)
+        expect(cache).to receive(:targets).and_return(["build/a.out", "build/main.o"])
+        expect(FileUtils).to receive(:rm_f).with("build/a.out")
+        expect(FileUtils).to receive(:rm_f).with("build/main.o")
+        expect(cache).to receive(:directories).and_return(["build/one", "build/one/two", "build", "other"])
+        expect(File).to receive(:directory?).with("build/one/two").and_return(true)
+        expect(Dir).to receive(:entries).with("build/one/two").and_return([".", ".."])
+        expect(Dir).to receive(:rmdir).with("build/one/two")
+        expect(File).to receive(:directory?).with("build/one").and_return(true)
+        expect(Dir).to receive(:entries).with("build/one").and_return([".", ".."])
+        expect(Dir).to receive(:rmdir).with("build/one")
+        expect(File).to receive(:directory?).with("build").and_return(true)
+        expect(Dir).to receive(:entries).with("build").and_return([".", ".."])
+        expect(Dir).to receive(:rmdir).with("build")
+        expect(File).to receive(:directory?).with("other").and_return(true)
+        expect(Dir).to receive(:entries).with("other").and_return([".", "..", "other.file"])
+        expect(cache).to receive(:clear)
+
+        Rscons.application.__send__(:clean)
+      end
+    end
+
     describe ".determine_n_threads" do
       context "when specified by environment variable" do
         before(:each) do

spec/rscons_spec.rb

@@ -1,29 +1,4 @@
 describe Rscons do
-  describe ".clean" do
-    it "removes all build targets and created directories" do
-      cache = "cache"
-      expect(Rscons::Cache).to receive(:instance).and_return(cache)
-      expect(cache).to receive(:targets).and_return(["build/a.out", "build/main.o"])
-      expect(FileUtils).to receive(:rm_f).with("build/a.out")
-      expect(FileUtils).to receive(:rm_f).with("build/main.o")
-      expect(cache).to receive(:directories).and_return(["build/one", "build/one/two", "build", "other"])
-      expect(File).to receive(:directory?).with("build/one/two").and_return(true)
-      expect(Dir).to receive(:entries).with("build/one/two").and_return([".", ".."])
-      expect(Dir).to receive(:rmdir).with("build/one/two")
-      expect(File).to receive(:directory?).with("build/one").and_return(true)
-      expect(Dir).to receive(:entries).with("build/one").and_return([".", ".."])
-      expect(Dir).to receive(:rmdir).with("build/one")
-      expect(File).to receive(:directory?).with("build").and_return(true)
-      expect(Dir).to receive(:entries).with("build").and_return([".", ".."])
-      expect(Dir).to receive(:rmdir).with("build")
-      expect(File).to receive(:directory?).with("other").and_return(true)
-      expect(Dir).to receive(:entries).with("other").and_return([".", "..", "other.file"])
-      expect(cache).to receive(:clear)
-
-      Rscons.clean
-    end
-  end
-
   describe ".absolute_path?" do
     context "on Windows" do
       it "returns whether a path is absolute" do