Skip to content

Commit c75ad16

Browse files
jachanjnf
andauthored
changed NJ intake to has_many analytics (#5554)
* changed NJ intake to has_many analytics * tweaks how StateFileNjAnalytics object is created and associated to the source intake in the analytic-making background job * Added test for multiple submissions on one intake --------- Co-authored-by: jey <[email protected]>
1 parent eff711d commit c75ad16

File tree

3 files changed

+13
-2
lines changed

3 files changed

+13
-2
lines changed

app/jobs/state_file/create_nj_analytics_record_job.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ module StateFile
22
class CreateNjAnalyticsRecordJob < ApplicationJob
33
def perform(submission_id)
44
submission = EfileSubmission.find(submission_id)
5-
nj_analytics = submission.data_source.create_state_file_nj_analytics!
5+
nj_analytics = StateFileNjAnalytics.create(state_file_nj_intake: submission.data_source)
66
nj_analytics&.update(nj_analytics.calculated_fields)
77
end
88

app/models/state_file_nj_intake.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -114,7 +114,7 @@
114114
class StateFileNjIntake < StateFileBaseIntake
115115
self.ignored_columns += ["primary_signature_pin", "spouse_signature_pin"]
116116
encrypts :account_number, :routing_number, :raw_direct_file_data, :raw_direct_file_intake_data
117-
has_one :state_file_nj_analytics
117+
has_many :state_file_nj_analytics
118118

119119
enum household_rent_own: { unfilled: 0, rent: 1, own: 2, neither: 3, both: 4 }, _prefix: :household_rent_own
120120

spec/jobs/state_file/create_nj_analytics_record_job_spec.rb

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,5 +11,16 @@
1111
submitted_intake.reload
1212
expect(submitted_intake.state_file_nj_analytics).to be_present
1313
end
14+
15+
context "when multiple submissions exist" do
16+
let!(:submission_2) { create :efile_submission, :for_state, data_source: submitted_intake }
17+
it "allows multiple NJ analytics records to be attached to one intake" do
18+
expect(submitted_intake.state_file_nj_analytics).not_to be_present
19+
described_class.perform_now(submission.id)
20+
submitted_intake.reload
21+
described_class.perform_now(submission_2.id)
22+
expect(submitted_intake.state_file_nj_analytics.size).to eq 2
23+
end
24+
end
1425
end
1526
end

0 commit comments

Comments
 (0)