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

Initial start tests #42

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

Conversation

AnthonySinitsa
Copy link

I have updated the README slightly, I've put how to install requirements.txt for the dependencies and also added a testing section to know what command to run.

In LangTrans.py I have made 2 changes, it was erroring out on those lines due to incorrect arguments amount. However take those changes with a grain of salt.

I have also moved LangTrans.py into it's own folder for clarity, if you decide to not want it in it's own folder and want it in the root you will have to change the import locations inside of test_LangTrans.py
Must change from:
from LangTrans.LangTrans import 'functionName'
to
from LangTrans import 'functionName'

The testing file that I have added does not test all the functions that exist, this is just a starting point.

@B-R-P
Copy link
Collaborator

B-R-P commented Aug 17, 2023

Having LangTrans as a single file without a folder is good for now. I've added unit tests for the functions, using unittest instead of pytest in FunctionTest.py . You folks can help me in enhancing these tests; some of them need improvement, and certain functions haven't been tested yet like loading YAML. Porting and improving the your tests would be helpful, particularly the commit 7f55de7

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

Successfully merging this pull request may close these issues.

None yet

4 participants