-
-
Notifications
You must be signed in to change notification settings - Fork 148
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
Added tests for CatalogTable.as_list_of_dicts (#1319) #1331
Conversation
Let me know if I over did it. I have change input structure and made the needed changes so that nothing gets break. Looking forward to your feedback. |
For some reason, when I run the test cases on my local setup these two test cases gets fail.I'm sure why I haven't changed anything in the code which effect this part of the code. Can you see why is this?
|
Welldone @daud99 , try installing the |
@daud99 try installing ersilia with the test extra again. This has been updated as a requirement in the pyproject.toml. |
@daud99 I don't understand why we rename |
* Implemented test case with empty and standard input.
@adebisi4145 Thankyou @DhanshreeA you're right. I just make it as suggested. Can you check it now! And I also made sure there is no conflict. Regards |
Thank you for taking your time to contribute to Ersilia, just a few checks before we proceed
Related to #1319