Merge pull request #38 from julioalucero/print_schema_stats

Updates test and dummy app with models and schema.rb

Author: hmdros

lib/rails_stats/json_formatter.rb

@@ -79,4 +79,4 @@ def schema_info
         end
       end
   end
-end
+end

test/dummy/app/models/comment.rb

@@ -0,0 +1,3 @@
+class Comments < ApplicationRecord
+  belongs_to :commentable, polymorphic: true
+end

test/dummy/app/models/pet.rb

@@ -0,0 +1,2 @@
+class Pets < User
+end

test/dummy/app/models/user.rb

@@ -0,0 +1,2 @@
+class Users < ApplicationRecord
+end

test/dummy/db/schema.rb

@@ -0,0 +1,10 @@
+
+ActiveRecord::Schema[7.0].define(version: 2023_04_25_154701) do
+  create_table "users", force: :cascade do |t|
+    t.string "email"
+  end
+  create_table "comments", force: :cascade do |t|
+    t.bigint  :commentable_id
+    t.string  :commentable_type
+  end
+end

test/fixtures/console-output.txt

@@ -1,44 +1,48 @@
 +-----------------------|------------|----------------+
 |                  Name | Total Deps | 1st Level Deps |
 +-----------------------|------------|----------------+
-|     simplecov-console | 7          | 3              |
-|               codecov | 5          | 2              |
+|                 rails | 50         | 14             |
+|            sass-rails | 31         | 1              |
+|           web-console | 26         | 4              |
+|             webpacker | 25         | 3              |
+|              jbuilder | 17         | 2              |
+| spring-watcher-listen | 5          | 2              |
 |           rails_stats | 4          | 2              |
-|             simplecov | 3          | 3              |
-|       minitest-around | 1          | 1              |
-|               bundler | 0          | 0              |
+|                listen | 3          | 2              |
+|              bootsnap | 1          | 1              |
+|                  puma | 1          | 1              |
 |                byebug | 0          | 0              |
-|              minitest | 0          | 0              |
-| minitest-spec-context | 0          | 0              |
+|                spring | 0          | 0              |
+|           tzinfo-data | 0          | 0              |
 +-----------------------|------------|----------------+
 
-      Declared Gems   9
-         Total Gems   18
-  Unpinned Versions   8
+      Declared Gems   13
+         Total Gems   71
+  Unpinned Versions   7
         Github Refs   0
 
 +----------------------+---------+---------+---------+---------+---------+-----+-------+
 | Name                 | Files   | Lines   |     LOC | Classes | Methods | M/C | LOC/M |
 +----------------------+---------+---------+---------+---------+---------+-----+-------+
 | Channels             |       2 |       8 |       8 |       2 |       0 |   0 |     0 |
-| Configuration        |      19 |     417 |     111 |       1 |       0 |   0 |     0 |
+| Configuration        |      19 |     419 |     113 |       1 |       0 |   0 |     0 |
 | Controllers          |       1 |       7 |       6 |       1 |       1 |   1 |     4 |
 | Helpers              |       1 |       3 |       3 |       0 |       0 |   0 |     0 |
 | Javascripts          |       3 |      27 |       7 |       0 |       0 |   0 |     0 |
 | Jobs                 |       1 |       7 |       2 |       1 |       0 |   0 |     0 |
 | Libraries            |       1 |       1 |       1 |       0 |       0 |   0 |     0 |
 | Mailers              |       1 |       4 |       4 |       1 |       0 |   0 |     0 |
 | Model Tests          |       2 |       5 |       4 |       2 |       0 |   0 |     0 |
-| Models               |       1 |       3 |       3 |       1 |       0 |   0 |     0 |
+| Models               |       4 |      10 |      10 |       4 |       0 |   0 |     0 |
 | Spec Support         |       1 |       1 |       1 |       0 |       0 |   0 |     0 |
 | Test Support         |       1 |       1 |       1 |       0 |       0 |   0 |     0 |
 +----------------------+---------+---------+---------+---------+---------+-----+-------+
-| Code                 |      30 |     477 |     145 |       7 |       1 |   0 |   143 |
+| Code                 |      33 |     486 |     154 |      10 |       1 |   0 |   152 |
 | Tests                |       4 |       7 |       6 |       2 |       0 |   0 |     0 |
-| Total                |      34 |     484 |     151 |       9 |       1 |   0 |   149 |
+| Total                |      37 |     493 |     160 |      12 |       1 |   0 |   158 |
 +----------------------+---------+---------+---------+---------+---------+-----+-------+
-  Code LOC: 145     Test LOC: 6     Code to Test Ratio: 1:0.0  Files: 34
+  Code LOC: 154     Test LOC: 6     Code to Test Ratio: 1:0.0  Files: 37
 
-  STI models count: 0 STI classes
-  Polymorphic models count: 0 polymorphic associations
-  Schema Stats: No schema.rb or structure.sql file found
+  STI models count: 1 STI classes
+  Polymorphic models count: 1 polymorphic associations
+  Schema Stats: 2 `create_table` calls in schema.rb

test/lib/rails_stats/json_formatter_test.rb

@@ -34,10 +34,10 @@
       "loc_over_m": "0"
     }, {
       "name": "Models",
-      "files": "1",
-      "lines": "3",
-      "loc": "3",
-      "classes": "1",
+      "files": "4",
+      "lines": "10",
+      "loc": "10",
+      "classes": "4",
       "methods": "0",
       "m_over_c": "0",
       "loc_over_m": "0"
@@ -89,8 +89,8 @@
     }, {
       "name": "Configuration",
       "files": "19",
-      "lines": "417",
-      "loc": "111",
+      "lines": "419",
+      "loc": "113",
       "classes": "1",
       "methods": "0",
       "m_over_c": "0",
@@ -115,27 +115,27 @@
       "loc_over_m": "0"
     }, {
       "name": "Total",
-      "files": "34",
-      "lines": "484",
-      "loc": "151",
-      "classes": "9",
+      "files": "37",
+      "lines": "493",
+      "loc": "160",
+      "classes": "12",
       "methods": "1",
       "m_over_c": "0",
-      "loc_over_m": "149",
+      "loc_over_m": "158",
       "code_to_test_ratio": "0.0",
       "total": true
     }, {
       "schema_stats": {
-        "schema_path": "db/schema.rb",
-        "create_table_calls_count": 5
+        "schema_path": "#{File.absolute_path("./test/dummy/db/schema.rb")}",
+        "create_table calls count": 2
       }
     }, {
       "sti_stats": {
-        "sti_models_count": 3
+        "sti_models_count": 1
       }
     }, {
       "polymorphic_stats": {
-        "polymorphic_models_count": 2
+        "polymorphic_models_count": 1
       }
     }]
     EOS
@@ -146,12 +146,22 @@
       calculator = RailsStats::StatsCalculator.new(root_directory)
       formatter  = RailsStats::JSONFormatter.new(calculator)
 
-      sorted_expectation = JSON.parse(JSON_STRING).sort {|x, y| x["name"] <=> y["name"] }
-      sorted_result      = formatter.result.sort {|x, y| x["name"] <=> y["name"] }
+      parsed_expectation = JSON.parse(JSON_STRING)
+      parsed_result      = formatter.result
+
+      named_expectation    = parsed_expectation.select { |h| h.key?("name") }.sort_by { |h| h["name"] }
+      metadata_expectation = parsed_expectation.reject { |h| h.key?("name") }
+
+      named_result    = parsed_result.select { |h| h.key?("name") }.sort_by { |h| h["name"] }
+      metadata_result = parsed_result.reject { |h| h.key?("name") }
+
+      named_expectation.each_with_index do |x, i|
+        assert_equal x, named_result[i]
+      end
 
-      sorted_expectation.each_with_index do |x, i|
-        assert_equal x, sorted_result[i]
+      metadata_expectation.each_with_index do |x, i|
+        assert_equal x, metadata_result[i]
       end
     end
   end
-end
+end