Support namespaced models

[jbigler]

When using a namespaced model such as:

module Accounts
  class User < ApplicationRecord
    self.ignored_columns += "old_name"
 end
end

rubocop-migration continues to give me an error if I try to run remove_column :accounts_users, :old_name

Note: the module has a table name prefix defined:

module Accounts
  def self.table_name_prefix
    "accounts_"
  end
end

P.S. I have no errors when using remove in change_table blocks, but I think they are not being detected at all, because they never seem to produce errors.

[r7kamura]

Thanks for nice feedback 👍
Surely, as you say, if the model name is not as per the convention, it will fail.

If there is a nice way to implement it, I would like to adopt it. Do you have any ideas?
If we can't seem to come up with anything specific, I'll just give up and say that we will only support models as per convention.

[jbigler]

Yes, I see your point.
While it would be difficult to find a model that was not named per conventions, namespaces generally follow a convention as well. As long as the namespaces match the folder names, I imagine it could be handled by something such as a recursive check for directories matching them.
I'll try to come up with something tested later as a PR if you're interested, but a quick ChatGPT conversation came up with something such as:

def find_model_file(table_name, base_dir = "app/models")
  # Split the table name into parts based on underscores
  parts = table_name.split("_")
  
  # Recursive helper to build paths
  def recursive_find(parts, current_path)
    # If no more parts to process, check if the file exists
    if parts.empty?
      file_path = "#{current_path}.rb"
      return file_path if File.exist?(file_path)
      return nil
    end

    # Process the next part as a potential directory or file name
    part = parts.shift
    directory_path = File.join(current_path, part)
    file_path = File.join(current_path, "#{part}.rb")

    # Check the directory path recursively
    if File.directory?(directory_path)
      found = recursive_find(parts.dup, directory_path)
      return found if found
    end

    # Check if the file path exists
    if File.exist?(file_path)
      return file_path
    end

    nil # Nothing found
  end

  # Start the recursive search
  recursive_find(parts, base_dir)
end