fallback to overview.html when pages do not exist for compare option

rishijain · Oct 18, 2023 · 921ec24 · 921ec24
1 parent ebdeba5
commit 921ec24
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -5,6 +5,7 @@
 * [CHANGE] Disable VERBOSE warnings in test stubs (by [@fbuys][])
 * [CHANGE] Add rexml dependency for Ruby 3.0.0+ support (by [@fbuys][])
 * [BUGFIX] Raise error when the same branches are compared (by [@rishijain][])
+* [BUGFIX] Use overview.html as the fallback path when files does not exist during compare option (by [@rishijain][])
 
 # v4.8.1 / 2023-05-17 [(commits)](https://github.com/whitesmith/rubycritic/compare/v4.8.0...v4.8.1)
 

diff --git a/lib/rubycritic/generators/html/view_helpers.rb b/lib/rubycritic/generators/html/view_helpers.rb
@@ -24,7 +24,10 @@ def smell_location_path(location)
     end
 
     def code_index_path(root_directory, file_name)
-      file_path("#{File.expand_path(root_directory)}/#{file_name}")
+      root_directory_path = File.expand_path(root_directory)
+      index_path = "#{root_directory_path}/#{file_name}"
+      index_path = "#{root_directory_path}/overview.html" unless File.exist?(index_path)
+      file_path(index_path)
     end
 
     private