OmairK commented on pull request #9217:
URL: https://github.com/apache/airflow/pull/9217#issuecomment-644120451


   > @OmairK Okay. I looked at what the problem was. The problem is that 
`clear_db_import_errors` function doesn't reset the sequence. Each newly added 
element has a new identifier. However, if the object was created in another 
test, then the numbering continues in your test. If you run one test several 
times, only the first run will be correct, but all others will fail. I have 
prepared a patch that normalizes identifiers in responses.
   > https://github.com/apache/airflow/pull/9217/commits
   > This should help. What do you think about this?
   
   Ah it was the sequence all along, I get it thank you so much for the help. A 
couple of quick questions
   
   1. I was running these tests locally by `./breeze test-target 
tests/api_connexion/` these tests should have failed there too unless the 
sequence was automatically resetting after every test, which doesn't make 
sense. Is there some major difference between how breeze and github action CI 
handles the db state between tests ?
   2. If yes, how do you reenact your airflow github actions locally ? (This 
would help me a lot in the future) 
   
   P.S.: +1 for the commit message, it perfectly sums up the situation :)


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to