Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

IRS returns incorrect values in smoke tests #979

Closed
schbetsy opened this issue May 21, 2017 · 5 comments
Closed

IRS returns incorrect values in smoke tests #979

schbetsy opened this issue May 21, 2017 · 5 comments
Assignees
Labels
Milestone

Comments

@schbetsy
Copy link
Contributor

In this screenshot, the IRS shows 92 total LARs for this submission, but the actual submission only has 10 LARs. I've seen this frequently when uploading multiple files for an institution.

screen shot 2017-05-21 at 11 00 05 am

@schbetsy schbetsy added the bug label May 21, 2017
@jmarin jmarin added this to the Sprint 32 milestone May 21, 2017
@schbetsy
Copy link
Contributor Author

Here's the steps for one way I have generated this bug. Notes: I was using institution 6999998 for all submissions. HMDA_IS_DEMO was set to true.

  1. Clean slate in Postgres: in psql hmda, drop table lars cascade;, \q
  2. Start the API in the usual way: In sbt, clean, compile, project api, reStart (repeat reStop-reStart cycle as necessary until the demo data gets populated 😞 )
  3. Create submission. upload 90-line file. check irs... total lars = 90. ✅
  4. Create another submission. upload 400-line file. check irs... total lars = 490. ❌
  5. Just for fun, do it again: upload 10-line file. irs says total lines = 500. ❌

I've also seen it when running the whole docker setup.

@nickgrippin
Copy link

Confirmed, the issue seems to be an intermittent failure to delete the previous LAR submissions from the repository, affecting the IRS report and modified LAR endpoint.

@nickgrippin
Copy link

IRS reporting incorrect values will be addressed in #984

This ticket will stay open in order to address the bug with the modified LAR endpoint.

@schbetsy
Copy link
Contributor Author

schbetsy commented Jun 9, 2017

This seems to be resolved by #1000.

@nickgrippin
Copy link

I agree, that's what I'm seeing as well. Closing this ticket.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants