Skip to content

Commit

Permalink
don't leave summary file behind (#47)
Browse files Browse the repository at this point in the history
* donot leave summary file behind

* server.crt -> ci_rootca.crt
  • Loading branch information
antgonza authored Aug 4, 2023
1 parent 33393e0 commit 508d124
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 3 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/qiita-plugin-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@ jobs:
conda create -q --yes -n qtp-sequencing python=3.9 pip pigz quast fqtools
conda activate qtp-sequencing
export QIITA_SERVER_CERT=`pwd`/qiita-dev/qiita_core/support_files/server.crt
export QIITA_SERVER_CERT=`pwd`/qiita-dev/qiita_core/support_files/ci_rootca.crt
export QIITA_CONFIG_FP=`pwd`/qiita-dev/qiita_core/support_files/config_test_local.cfg
pip install -U pip
pip install .
Expand All @@ -90,7 +90,7 @@ jobs:
shell: bash -l {0}
run: |
conda activate qiita
export QIITA_SERVER_CERT=`pwd`/qiita-dev/qiita_core/support_files/server.crt
export QIITA_SERVER_CERT=`pwd`/qiita-dev/qiita_core/support_files/ci_rootca.crt
export QIITA_CONFIG_FP=`pwd`/qiita-dev/qiita_core/support_files/config_test_local.cfg
sed "s#/home/runner/work/qiita/qiita#${PWD}/qiita-dev/#g" `pwd`/qiita-dev/qiita_core/support_files/config_test.cfg > ${QIITA_CONFIG_FP}
Expand Down Expand Up @@ -123,7 +123,7 @@ jobs:
COVER_PACKAGE: ${{ matrix.cover_package }}
run: |
conda activate qtp-sequencing
export QIITA_SERVER_CERT=`pwd`/qiita-dev/qiita_core/support_files/server.crt
export QIITA_SERVER_CERT=`pwd`/qiita-dev/qiita_core/support_files/ci_rootca.crt
export QIITA_CONFIG_FP=`pwd`/qiita-dev/qiita_core/support_files/config_test_local.cfg
nosetests --with-doctest --with-coverage -v --cover-package=qtp_sequencing
Expand Down
2 changes: 2 additions & 0 deletions qtp_sequencing/summary.py
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@
# -----------------------------------------------------------------------------

from hashlib import md5
from os import remove
from os.path import basename, join, dirname, exists
from base64 import b64encode
from io import BytesIO
Expand Down Expand Up @@ -163,6 +164,7 @@ def _summary_not_demultiplexed(artifact_type, filepaths):
fdata = f'{dirname(fp)}/qtp-sequencing-validate-data.csv'
if exists(fdata):
df = pd.read_csv(fdata, index_col=None)
remove(fdata)

if df is None:
cmd = f'fqtools count {fp}'
Expand Down

0 comments on commit 508d124

Please sign in to comment.