update logic inside the tenant_klass_defined? method

[msasaki666]

I think options[:class_name] should also be a determining factor in tenant_klass_defined? method.

I think the following issue is related.
#105

[msasaki666]

@microsoft-github-policy-service agree

[codecov]

Codecov Report

Merging #202 (373b8b1) into master (fd42dbd) will increase coverage by 0.07%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #202      +/-   ##
==========================================
+ Coverage   82.61%   82.68%   +0.07%     
==========================================
  Files          14       14              
  Lines         719      722       +3     
==========================================
+ Hits          594      597       +3     
  Misses        125      125              

Impacted Files	Coverage Δ
lib/activerecord-multi-tenant/model_extensions.rb	96.51% <100.00%> (ø)
lib/activerecord-multi-tenant/multi_tenant.rb	88.15% <100.00%> (+0.48%)	⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[gurkanindibay]

Great solution @msasaki666 . The solution is like a charm.
Thank you to add the unit test to cover the cases.
In order to see if this solves #105, a test case is needed that covers this case.
Could you add a test to cover #105.

Thanks in advance for your great effort to make activerecord-multi-tenant better

[gurkanindibay]

Thanks

[msasaki666]

Great solution @msasaki666 . The solution is like a charm. Thank you to add the unit test to cover the cases. In order to see if this solves #105, a test case is needed that covers this case. Could you add a test to cover #105.

Thanks in advance for your great effort to make activerecord-multi-tenant better

Sure.
I'll add it later.

[msasaki666]

Great solution @msasaki666 . The solution is like a charm. Thank you to add the unit test to cover the cases. In order to see if this solves #105, a test case is needed that covers this case. Could you add a test to cover #105.

Thanks in advance for your great effort to make activerecord-multi-tenant better

@gurkanindibay
I have added it.

[gurkanindibay]

Thanks @msasaki666
Could you add a case as below as defined in #105

module MyModule
  class Tenant < ApplicationRecord; end

  class Foo < ApplicationRecord
    multi_tenant :tenant
  end
end

[msasaki666]

@gurkanindibay
Sure.
I have updated it.
I think all of tenant classes are subclasses of ActiveRecord::Base.
So, I also modify other tests.
I can't use ApplicationRecord class in tests, so I use ActiveRecord::Base class instead.

[gurkanindibay]

Thanks @msasaki666 now code is in master branch. Thank you for all of your efforts