mirror of
https://github.com/paper-trail-gem/paper_trail.git
synced 2022-11-09 11:33:19 -05:00
ad3806fcbb
Problem ------- `has_paper_trail` adds too many methods to the ActiveRecord model. > If I'm counting correctly, installing the paper_trail gem adds 36 instance > methods and 10 class methods to all of your active record models. Of those > 46, 13 have a prefix, either "pt_" or "paper_trail_". I don't know what the > best way to deal with this is. Ideally, we'd add far fewer methods to > people's models. If we were able to add fewer methods to models, then I > wouldn't mind prefixing all of them. > https://github.com/airblade/paper_trail/issues/703 Solution -------- Add only two methods to the AR model. 1. An instance method `#paper_trail` 2. A class method `.paper_trail` The instance method would return a `RecordTrail` and the class method would return a `ClassTrail`. Those names are totally up for debate. Advantages ---------- - Plain ruby, easy to understand - Adding new methods to e.g. the `RecordTrail` does not add any methods to the AR model. - Methods privacy is more strongly enforced. - Enables isolated testing of e.g. `RecordTrail`; it can be constructed with a mock AR instance. Disadvantages ------------- - Two new classes, though they are simple.
17 lines
365 B
Ruby
17 lines
365 B
Ruby
require "rails_helper"
|
|
|
|
describe Fluxor, type: :model do
|
|
describe "`be_versioned` matcher" do
|
|
it { is_expected.to_not be_versioned }
|
|
end
|
|
|
|
describe "Methods" do
|
|
describe "Class" do
|
|
describe ".paper_trail.enabled?" do
|
|
it "returns false" do
|
|
expect(Fluxor.paper_trail.enabled?).to eq(false)
|
|
end
|
|
end
|
|
end
|
|
end
|
|
end
|