Change the way to calculate root directory

Print out list in alphabetical order

Change expectations based in the order of the output

Author: etagwerker

lib/rails_stats/code_statistics.rb

@@ -20,7 +20,8 @@ def initialize(root_directory)
 
     def to_s
       print_header
-      @statistics.each { |key, stats| print_line(key, stats) }
+      sorted_keys = @statistics.keys.sort
+      sorted_keys.each { |key| print_line(key, @statistics[key]) }
       print_splitter
 
       print_line("Code", @code_total)

lib/rails_stats/spec_statistics.rb

@@ -37,7 +37,7 @@ def calculate_statistics
       out = {}
       categorize_files.each do |key, list|
         out[key] = Util.calculate_file_statistics(list)
-      end 
+      end
       out
     end
 

lib/rails_stats/test_statistics.rb

@@ -35,7 +35,7 @@ def calculate_statistics
       out = {}
       categorize_files.each do |key, list|
         out[key] = Util.calculate_file_statistics(list)
-      end 
+      end
       out
     end
 

lib/rails_stats/util.rb

@@ -29,7 +29,7 @@ def calculate_shared_projects(shared, *args)
           projects[$1] = true
         end
       end
-      
+
       out = projects.keys
 
       # TODO: make sure none are children of other ones in there
@@ -56,14 +56,14 @@ def calculate_statistics(directories, type = :code, &block)
 
       directories.each do |dir_path|
         next unless File.directory?(dir_path)
-        
+
         key = nil
         if block_given?
           key = yield(dir_path)
         end
 
         key = path_to_name(dir_path, is_test) if !key || key.length == 0
-        
+
         out[key] ||= CodeStatisticsCalculator.new(is_test)
         out[key].add(calculate_directory_statistics(dir_path))
       end

test/lib/rails_stats/code_statistics_test.rb

@@ -8,14 +8,14 @@
 +----------------------+---------+---------+---------+---------+-----+-------+
 | Name                 | Lines   |     LOC | Classes | Methods | M/C | LOC/M |
 +----------------------+---------+---------+---------+---------+-----+-------+
-| Mailers              |       4 |       4 |       1 |       0 |   0 |     0 |
-| Models               |       3 |       3 |       1 |       0 |   0 |     0 |
-| Javascripts          |      27 |       7 |       0 |       0 |   0 |     0 |
-| Jobs                 |       7 |       2 |       1 |       0 |   0 |     0 |
-| Controllers          |       3 |       3 |       1 |       0 |   0 |     0 |
-| Helpers              |       3 |       3 |       0 |       0 |   0 |     0 |
 | Channels             |       8 |       8 |       2 |       0 |   0 |     0 |
 | Configuration        |     417 |     111 |       1 |       0 |   0 |     0 |
+| Controllers          |       3 |       3 |       1 |       0 |   0 |     0 |
+| Helpers              |       3 |       3 |       0 |       0 |   0 |     0 |
+| Javascripts          |      27 |       7 |       0 |       0 |   0 |     0 |
+| Jobs                 |       7 |       2 |       1 |       0 |   0 |     0 |
+| Mailers              |       4 |       4 |       1 |       0 |   0 |     0 |
+| Models               |       3 |       3 |       1 |       0 |   0 |     0 |
 +----------------------+---------+---------+---------+---------+-----+-------+
 | Code                 |     472 |     141 |       7 |       0 |   0 |     0 |
 | Tests                |       0 |       0 |       0 |       0 |   0 |     0 |
@@ -26,7 +26,8 @@
     EOS
 
     it "outputs useful stats for a Rails project" do
-      root_directory = File.absolute_path("./test/dummy")
+      root_directory = File.expand_path('../../../test/dummy', File.dirname(__FILE__))
+
       assert_output(TABLE) do
         RailsStats::CodeStatistics.new(root_directory).to_s
       end