Skip to content

Commit 0e0b8ff

Browse files
authored
Remove unused Decision Reviews base controllers (#20942)
* AppealsSubmission factory without DecisionReviewV1::Service * Remove unused base controller
1 parent eb57587 commit 0e0b8ff

7 files changed

+8
-58
lines changed

.github/CODEOWNERS

-1
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@ README.md @department-of-veterans-affairs/backend-review-group
2424
.devcontainer @department-of-veterans-affairs/backend-review-group @department-of-veterans-affairs/cto-engineers
2525
import-va-certs.sh @department-of-veterans-affairs/backend-review-group
2626
app/controllers/appeals_base_controller.rb @department-of-veterans-affairs/backend-review-group
27-
app/controllers/appeals_base_controller_v1.rb @department-of-veterans-affairs/backend-review-group
2827
app/controllers/application_controller.rb @department-of-veterans-affairs/va-api-engineers @department-of-veterans-affairs/backend-review-group @department-of-veterans-affairs/octo-identity
2928
app/controllers/bb_controller.rb @department-of-veterans-affairs/vfs-vaos @department-of-veterans-affairs/va-api-engineers @department-of-veterans-affairs/backend-review-group
3029
app/controllers/claims_base_controller.rb @department-of-veterans-affairs/va-api-engineers @department-of-veterans-affairs/backend-review-group

app/controllers/appeals_base_controller_v1.rb

-52
This file was deleted.

spec/factories/appeal_submissions.rb

+8-1
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,14 @@
1111
board_review_option { 'evidence_submission' }
1212
upload_metadata do
1313
user = User.find(user_uuid)
14-
DecisionReviewV1::Service.file_upload_metadata(user)
14+
{
15+
'veteranFirstName' => user.first_name,
16+
'veteranLastName' => user.last_name,
17+
'zipCode' => user.postal_code.to_s,
18+
'fileNumber' => user.ssn.to_s.strip,
19+
'source' => 'va.gov',
20+
'businessLine' => 'BVA'
21+
}.to_json
1522
end
1623
end
1724

spec/models/saved_claim/higher_level_review_spec.rb

-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
# frozen_string_literal: true
22

33
require 'rails_helper'
4-
require 'decision_review_v1/service'
54

65
RSpec.describe SavedClaim::HigherLevelReview, type: :model do
76
subject { described_class.new }

spec/models/saved_claim/notice_of_disagreement_spec.rb

-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
# frozen_string_literal: true
22

33
require 'rails_helper'
4-
require 'decision_review_v1/service'
54

65
RSpec.describe SavedClaim::NoticeOfDisagreement, type: :model do
76
subject { described_class.new }

spec/models/saved_claim/supplemental_claim_spec.rb

-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
# frozen_string_literal: true
22

33
require 'rails_helper'
4-
require 'decision_review_v1/service'
54

65
RSpec.describe SavedClaim::SupplementalClaim, type: :model do
76
subject { described_class.new }

spec/models/secondary_appeal_form_spec.rb

-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
# frozen_string_literal: true
22

33
require 'rails_helper'
4-
require 'decision_review_v1/service'
54

65
RSpec.describe SecondaryAppealForm, type: :model do
76
subject { build(:secondary_appeal_form4142) }

0 commit comments

Comments
 (0)