| Age | Commit message (Collapse) | Author |
|---|
|
ATM, editing case-attributes is very buggy.
* wqflask/tests/integration/wqflask/test_metadata_edits.py: Delete.
* wqflask/wqflask/decorators.py: Remove:
gn3.authentication.get_groups_by_user_uid import.
(case_attributes_edit_access): Delete.
* wqflask/tests/integration/wqflask/test_metadata_edits.py: Remove
wqflask.decorators.case_attributes_edit_access,
gn3.db.case_attributes.get_case_attributes,
gn3.db.case_attributes.get_unreviewed_diffs,
gn3.db.case_attributes.insert_case_attribute_audit,
gn3.db.case_attributes.reject_case_attribute and
gn3.db.case_attributes.approve_case_attribute.
(display_phenotype_metadata): Remove headers arg when rendering
template.
(update_phenotype): Remove case attribute headers list when updating a
phenotype.
(show_case_attribute_columns, update_case_attributes): Delete.
(reject_case_attribute_data, approve_case_attribute_data): Ditto.
* wqflask/wqflask/templates/edit_phenotype.html: Remove link that
shows all the listed case-attributes.
Signed-off-by: Munyoki Kilyungi <me@bonfacemunyoki.com>
|
|
ATM, it's hard to by-pass the authentication
decorators, and as such, skip this test for. This
may be deleted later.
|
|
* wqflask/tests/integration/wqflask/test_metadata_edits.py
(test_show_case_attributes): Mock
"database_connection". Fixes this failing test:
<https://ci.genenetwork.org/jobs/genenetwork2/223>
|
|
* wqflask/wqflask/metadata_edits.py: Import
"gn3.db.sample_data.get_case_attributes".
(show_case_attribute_columns): New function/ end-point to show
case-attributes.
* wqflask/tests/integration/wqflask/test_metadata_edits.py: New
integration tests for the above.
* wqflask/tests/integration/wqflask/__init__.py: New file.
* wqflask/wqflask/templates/case_attributes.html: New template file to
display the above.
|
|
|
|
* wqflask/tests/integration/test_glossary.py: Delete it. Earlier
renamed to test_markdown_routes.
|
|
|
|
|
