CV2-6022 update tests #486
Closed
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Develop into master See merge request meedan/alegre!10
* CHECK-176 rewrite as per discussion with Caio (#48) * CHECK-176 rewrite as per discussion with Caio * CHECK-176 alter re feedback * CHECK-176 alter test db name * CHECK-176 hacky check for existence of column to prevent test failures in travis. * CHECK-176 fix missing doc_id in function call * CHECK-176 forgot that context is a list, whoops * recreate original migration and slight test tweak * CHECK-176 entering a land of pure voodoo * CHECK-176 alter PR after review with Caio * Fix deploy typo (#50) * Fix typo in ecs deploy command referencing QA container for Live deployment. Co-authored-by: Martin Peck <martin@meedan.com> Co-authored-by: Devin Gaffney <itsme@devingaffney.com> Co-authored-by: Martin Peck <martin@meedan.com>
This reverts commit c9b2787.
…oyment. (#52) Co-authored-by: Martin Peck <martin@meedan.com>
This reverts commit 6fbddf4.
Co-authored-by: Martin Peck <martin@meedan.com>
* add better merger for contexts * add unit tests * add import * update tests * updates
* fix a couple more tests that could randomly fail * fix paste typo * refactor equal to true
…#383) * CV2-4437 stub out article work to prevent illegal instruction crashes * Stub out another test
Co-authored-by: Martin Peck <martin@meedan.com>
…te (#411) * CV2-4605 push up an initial idea to see how it plays against test suite * update tests and tweak shared model logic / likely deprecated controller to resolve test issues * More refactoring * maybe not load this model directly in the test... * fix bad return * more test fixing * Comment expected failing test * remove legacy endpoints * remove references to unused controller --------- Co-authored-by: computermacgyver <computermacgyver>
* Build and deploy for this branch. * Update setuptools. * Pin setuptools to 75.1.0. * Remove build and deploy for this branch in preparation for merge. --------- Co-authored-by: Martin Peck <martin@meedan.com>
* CV2-5890 remove any ties to shared model for audio model (#476) * remove audio model config * CV2-5892 gut video (#477) * remove more dependencies * remove another mention of config var * CV2-5891 remove threatexchange (#478) * remove ruby * CV2-5778-change-language-detection-agreement (#474) * Update langid.py to reflect `Both CLD3 and FastText must agree and BOTH of them is more than "Threshold" confident` lowering threshold to 0.7 * update langid.py to check if `fasttext_result['result']['language']` and `cld_result['result']['language']` are not None before calculating the minimum confidence * update langid.py write the if condition in one line * update langid.py to check if fasttext_result['result']['confidence'] and cld_result['result']['confidence'] not None --------- Co-authored-by: Devin Gaffney <itsme@devingaffney.com> Co-authored-by: ahmednasserswe <ahmed.nasser.swe@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
10 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Updates tests to move to sync endpoint in lieu of soon to be deprecated endpoint
Reference: CV2-6022
How has this been tested?
Ran through most of these test updates individually, one at a time. Should be good to go!
Have you considered secure coding practices when writing this code?
None