flexion / ef-cms

An Electronic Filing / Case Management System.
23 stars 10 forks source link

BUG: Court user is unable to serve documents #8952

Closed ttlenard closed 3 years ago

ttlenard commented 3 years ago

Describe the Bug A clear and concise description of what the bug is. Docket clerk indicated that they are unable to serve paper filing documents in this case: 15977-18 and also 25605-15W. They are getting an error message that states "Document cannot be served until the Petition is Served". In this particular case, the Petition was served back in 2018.

Acceptance Criteria

Note: UI validations/warnings originally implemented in #8763 are still valid/approved

Business Impact/Reason for Severity Critical - court users are unable to serve documents.

In which environment did you see this bug? PROD MIG

Who were you logged in as? Docket

What were you doing when you discovered this bug? (Using the application, demoing, smoke tests, testing other functionality, etc.) Using the application

To Reproduce Steps to reproduce the behavior:

  1. Go to case 15977-18 or 25605-15W
  2. Click on add paper filing
  3. Notice that you get the error message "Document cannot be served until the Petition is served"
  4. If you look at the docket record, there are multiple documents that have already been served on this case, including the petition.

Expected Behavior A clear and concise description of what you expected to happen.

Court should be able to serve documents if the Petition has already been served

Actual Behavior A clear and concise description of what actually happened. Court user is unable to serve documents in this case.

Screenshots If applicable, add screenshots to help explain your problem. image.png

Desktop (please complete the following information):

Smartphone (please complete the following information):

Cause of Bug, If Known

Process for Logging a Bug:

Severity Definition:

Definition of Done (Updated 4-14-21)

Product Owner

Engineering

ttlenard commented 3 years ago

I do notice that in this case, there is not a capital "P" Petition on the Docket. The Petitions in this case appear to have event codes "MISC" and "PTFR". While testing the story 8763 last week, we ran into problems with still being able to serve documents when the petition had not yet been served. It was explained in our DevOps call that the query was looking for ANY document that had been served already, so I am having a hard time with understanding why this is happening in this case this morning.

kkoskelin commented 3 years ago

For the sake of 8763, the story was specifically depending upon finding not just any document, but specifically a Petition which had been served by looking in the docket entries for an item with event code P.

Indeed, code in the repository relies upon the Petition being filed with P code, and not PTFR. Fixes needed:

ttlenard commented 3 years ago

FYI - We just got another report of Docket getting this error. Here is the docket #: 25605-15W

ttlenard commented 3 years ago

I copied these same test cases over from 8763. I also copied over a few more test cases that had failed with those 9 cases that existed on Prod. These came out of a comment on 8763 as well. It was the comment related to Service workflows.

1) Docket clerk adds docket entry when case status = NEW

2) Docket clerk adds docket entry when case status = NEW and accesses document from In Progress tab Scenario 1

3) Docket clerk views unserved court-issued document in Document View

4) Docket clerk views message with unserved court-issued document attached

5) Docket clerk adds paper filing when case status = NEW Login as docket clerk

6) Docket clerk adds paper filing when case status = NEW and accesses document from In Progress tab

7) Docket clerk views unserved paper filing document in Document View

8) Docket clerk views message with unserved paper-filing document attached

9) Practitioner updates their contact information and has case status = NEW

10) Admissions clerk updates practitioner contact info with case status = NEW

11) File an Entry of Appearance by a Private Practitioner

12) File an Answer by Respondent

13) Petitioner eFile's a document

14) Serve a document on a Case that doesn't have a Captital P Petition in it

mariahkannenberg commented 3 years ago

Blocked - Waiting for Court to finalize test cases. Once test cases are done, review w/engineers.

ttlenard commented 3 years ago

Here are the specific Docket entries that had issues. Will be good to test with these cases:

mariahkannenberg commented 3 years ago

Unblocked - test cases complete.

matthopson commented 3 years ago

@cholly75 being able to ensure expected behavior for external users such as IRS Super user, is dependent on exposing the case status to the PublicCase entity. It wouldn't necessarily be displayed in the UI, but it would be part of the network request.

matthopson commented 3 years ago

PR, pending ☝️ https://github.com/ustaxcourt/ef-cms/pull/1668

cholly75 commented 3 years ago

@matthopson Yes, this is fine.

kkoskelin commented 3 years ago

Please see: https://github.com/ustaxcourt/ef-cms/pull/1668/commits/98348f184785eae8c15dcd1a43f40cfa11bbdd77 I have updated this PR expections of the v1 and v2 endpoints. I am not altering the "contract", because frequently we do have values being returned for contactPrimary and status. Only now they will be returned more frequently. I believe this should not compromise any functionality of applications consuming the outputs of these endpoints. cc: @cholly75 @mariahkannenberg @mmarcotte

ttlenard commented 3 years ago

All tests passed! Thank you everyone for the hard work on this! I made sure to test with some of those specific cases listed above, and logic seems to be working for each scenario identified!

ttlenard commented 3 years ago

We are awaiting IRS environment testing