Refactor sql in testplans #172

Open
wants to merge 3 commits into
from

Conversation

Projects
None yet
2 participants
@atodorov
Contributor

atodorov commented Mar 26, 2017

Replaced the hard-coded SQL with ORM queries and also added slight optimizations. I have also updated the existing tests a bit.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Mar 26, 2017

Coverage Status

Coverage decreased (-0.04%) to 45.364% when pulling 7e5ab09 on MrSenko:refactor_sql_in_testplans into 980b07b on Nitrate:develop.

Coverage Status

Coverage decreased (-0.04%) to 45.364% when pulling 7e5ab09 on MrSenko:refactor_sql_in_testplans into 980b07b on Nitrate:develop.

atodorov added some commits Mar 24, 2017

Refactor TP_PRINTABLE_CASE_TEXTS to ORM and improve tests
this commit improves the existing test by actually asserting on
a few more values. It also adds a test case when no plans were
selected.

Then it removes the SQL statement and replaces it with ORM query
making sure to filter out earlier versions of the test case text!
Refactor test plan export SQLs into ORM
at the same time group the queries together in only 2 queries
instead of the original 4 queries.
Improve testplan export tests
test plan references, tag names and component names
in the generated XML
@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Apr 14, 2017

Coverage Status

Coverage increased (+0.1%) to 45.482% when pulling 48d2ea5 on MrSenko:refactor_sql_in_testplans into 8f45beb on Nitrate:develop.

Coverage Status

Coverage increased (+0.1%) to 45.482% when pulling 48d2ea5 on MrSenko:refactor_sql_in_testplans into 8f45beb on Nitrate:develop.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment