Skip to content

Commit

Permalink
dev-support/APPEALS-24154 fix distribution spec tests (#18922)
Browse files Browse the repository at this point in the history
* skipped duplicates in distribution_spec_deprecated

* remove skipped test referenced by APPEALS-24154

* added tests to distribution spec and skipped them in distribution deprecated

* final skip for deprecated spec, comments in distribution spec

* comments in by_docket_date spec, added distribution_concern_spec

* deleted distribution_spec_deprecated.rb

* refactoring push priority job spec

* more changes to push priority job spec

* another change to before/after blocks

* update comments

* attempt fix for appeal series issue spec

* modify distribution concern spec

* revert change to appeal series issue spec

* remove distribution concern spec as it isn't working in GHA

* Craig/distribution spec changes (#18905)

* skipped duplicates in distribution_spec_deprecated

* remove skipped test referenced by APPEALS-24154

* added tests to distribution spec and skipped them in distribution deprecated

* final skip for deprecated spec, comments in distribution spec

* comments in by_docket_date spec, added distribution_concern_spec

* deleted distribution_spec_deprecated.rb

* refactoring push priority job spec

* more changes to push priority job spec

* another change to before/after blocks

* update comments

* attempt fix for appeal series issue spec

* modify distribution concern spec

* revert change to appeal series issue spec

* remove distribution concern spec as it isn't working in GHA

* fix bug with legacy efolder sync notification job that was causing errors

* add guard to poll docketed legacy appeals job spec in case seed data exists already

* fix database cleaner bug in ama efolder sync job spec
  • Loading branch information
craigrva authored Jul 5, 2023
1 parent 5056e71 commit c5feae7
Show file tree
Hide file tree
Showing 7 changed files with 378 additions and 1,850 deletions.
2 changes: 1 addition & 1 deletion spec/jobs/ama_notification_efolder_sync_job_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -180,7 +180,7 @@ def find_appeal_ids_from_first_document_sync
end

def clean_up_after_threads
DatabaseCleaner.clean_with(:truncation, except: %w[notification_events])
DatabaseCleaner.clean_with(:truncation, except: %w[notification_events vftypes issref])
end
end
end
2 changes: 1 addition & 1 deletion spec/jobs/legacy_notification_efolder_sync_job_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -196,7 +196,7 @@ def find_appeal_ids_from_first_document_sync
end

def clean_up_after_threads
DatabaseCleaner.clean_with(:truncation, except: %w[notification_events])
DatabaseCleaner.clean_with(:truncation, except: %w[notification_events vftypes issref])
end

def ensure_notification_events_exist
Expand Down
2 changes: 1 addition & 1 deletion spec/jobs/poll_docketed_legacy_appeals_job_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@

describe "polling for docketed appeals" do
before do
Seeds::NotificationEvents.new.seed!
Seeds::NotificationEvents.new.seed! unless NotificationEvent.count > 0
end

let!(:today) { Time.now.utc.iso8601 }
Expand Down
Loading

0 comments on commit c5feae7

Please sign in to comment.