Skip to content

CV2-5077-Add-test-coverage-for-OpenAI-module-in-Alegre #491

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.

Sign up for GitHub

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

Merged
merged 15 commits into from
Feb 28, 2025
Merged

CV2-5077-Add-test-coverage-for-OpenAI-module-in-Alegre #491

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
c79e5fb
create test_openai.py and add `test_openai_get_document_body`
ahmednasserswe Feb 13, 2025
4d287a2
Update test_openai.py with `test_retrieve_openai_embeddings_handles_a…
ahmednasserswe Feb 13, 2025
d864f29
Update test_openai.py to mock redis cache
ahmednasserswe Feb 13, 2025
bce50e6
Update test_openai.py to include tests that call the api
ahmednasserswe Feb 13, 2025
22423dc
commented `test_openai_get_document_body` and `test_retrieve_openai_e…
ahmednasserswe Feb 14, 2025
653e9aa
uncommenting `test_openai_get_document_body` and `test_retrieve_opena…
ahmednasserswe Feb 15, 2025
b677ccc
mocking cache in `test_openai_get_document_body` and `test_retrieve_o…
ahmednasserswe Feb 17, 2025
bf1de40
asserting openai key is set during CI
ahmednasserswe Feb 17, 2025
5862523
adding a unit test to find if openai_api_key is in env
ahmednasserswe Feb 18, 2025
0bb458a
making TestRetrieveOpenAIEmbeddings implement BaseTestCase
ahmednasserswe Feb 18, 2025
5f081fc
[CV2-6182] add bash script to copy ssm params for CI to env and modif…
Feb 27, 2025
1e0480f
add execute permissions for script
Feb 27, 2025
7391af1
better comments and correct .env_file name
Feb 27, 2025
5d66ed8
added test OPENAI key for CI to encoded .env file https://meedan.atla…
Feb 28, 2025
4e24bbb
removing ssm extraction script and build step because credential now …
Feb 28, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Binary file modified .env_file.enc
View file
Open in desktop
Binary file not shown.
4 changes: 2 additions & 2 deletions .github/workflows/ci-tests.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -50,12 +50,12 @@ jobs:
run: |
openssl enc -aes-256-cbc -d -in .env_file.enc -out .env_file -k $DECRYPTION_PASSWORD

- name: Decrypt Alegre credentials
- name: Decrypt Alegre google credentials
env:
DECRYPTION_PASSWORD: ${{ secrets.DECRYPTION_PASSWORD }}
run: |
openssl aes-256-cbc -d -in google_credentials.json.enc -out google_credentials.json -k $DECRYPTION_PASSWORD

- name: Install redis tools
run: |
sudo apt-get -y install redis-tools
Expand Down
Loading