Skip to content

Feature/biotype sql

Marek Szuba requested to merge feature/biotype_sql into master

Created by: magaliruffier

Requirements

  • Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion;
  • Review the contributing guidelines for this repository; remember in particular:
    • do not modify code without testing for regression
    • provide simple unit tests to test the changes
    • the PR must not fail unit testing
    • if you're adding/updating documentation of an endpoint, make sure you add/update the necessary parameters to the (template) configuration files in the ensembl-rest_private repo

Description

Raw SQL queries have been replaced with API methods. Changes to core API will need to be integrated prior to this PR.

Benefits

The REST endpoint behaviour is protected from underlying schema changes as long as the methods return consistent outputs.

Possible Drawbacks

Some of the tests are still sensitive to order of results.

Testing

Have you added/modified unit tests to test the changes?

Behaviour of the endpoints was already tested, as only the method of fetching the data has changed. However, some order-dependent tests had to be updated.

If so, do the tests pass/fail?

Yes

Have you run the entire test suite and no regression was detected?

Yes.

Changelog

The output of the various endpoints remains unchanged

Merge request reports