pass msg_level and remove duplicated checks
properly catch 404 errors
study id from object
Merge pull request #606 from squirrelo/fix-study-desc-render

Pass msg_level and remove duplicated checks
BUG: Remove unnecessary fixture from #522

This block of code was making sure that we were no longer submitting
invalid investigation_types to EBI, but since #522 was merged, this is
no longer needed.

The problem was that the investigation type was being overwritten to
'Amplicon Sequencing' regardless of whatever the value was, so EBI was
rejecting the submitted XML files with the following message:

```
<RECEIPT receiptDate="2014-11-14T16:53:02.221Z" submissionFile="submission.xml" success="false">
    <SUBMISSION alias="qiime_submission_3"/>
        <MESSAGES>
            <ERROR>
                Study XML parse error: string value 'Amplicon Sequencing' is not a valid enumeration value for type of existing_study_type attribute in type of STUDY_TYPE element in type of DESCRIPTOR element in StudyType
            </ERROR>
            <ERROR>
                An exception occurred : string value 'Amplicon Sequencing' is not a valid enumeration value for type of existing_study_type attribute in type of STUDY_TYPE element in type of DESCRIPTOR element in StudyType
            </ERROR>
            <INFO>
                ADD action for the following XML: study.xml sample.xml experiment.xml run.xml
            </INFO>
            </MESSAGES>
            <ACTIONS>ADD</ACTIONS>
            <ACTIONS>ADD</ACTIONS>
            <ACTIONS>ADD</ACTIONS>
            <ACTIONS>ADD</ACTIONS>
            <ACTIONS>HOLD</ACTIONS>
</RECEIPT>
```
TST: Add more tests for EBI
TST: Use ENA_test instead of ENA
ENH: Appropriately use pop
Merge pull request #611 from ElDeveloper/ebi-bug-fix

BUG: Remove unnecessary fixture from #522
Prevent loading ontologies with test environment
Do not check DB name when checking test env

The database name should not factor into the decision of whether or not
the environment is a test environment. E.g., I have many test databases setup,
and they are named qiita_test2, my_qiita_test, etc.
Remove unused ENVIRONMENTS global
Remove ENA_test ontology

Taking a different approach instead: do not allow ontologies to be laoded
if it is a test environment
Typo: forgot to close string
Fix minor typo; thanks @ElDeveloper!
Revert "Remove unused ENVIRONMENTS global"

This reverts commit e01d3bd43581321541a12e0400bc3c4d337c964b.
Remove two unused variables
Fix travis to check scripts explicitly

just doing `flake8 scripts/` is not enough, since flake8 only checks py files
recursively, so it must be told to check the scripts directly
Fix failing tests

Forgot to change the tests for ontology.py
Merge pull request #615 from adamrp/flake8_checking

Flake8 checking