Fix non-development package installation

.github/workflows/QCSchema.yml

@@ -0,0 +1,86 @@
+---
+name: QCSchema
+
+# yamllint disable-line rule:truthy
+on: [push, pull_request]
+
+jobs:
+  test:
+    name: Test on Python ${{ matrix.python-version }} using ${{ matrix.env-type.env-type-name }}
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        python-version: [3.6, 3.7, 3.8, 3.9]
+        # yamllint disable rule:braces
+        env-type:
+          - { env-type-name: venv, shell: bash }
+          - { env-type-name: conda, shell: 'bash -l {0}' }
+          # yamllint enable rule:braces
+    defaults:
+      run:
+        shell: ${{ matrix.env-type.shell }}
+    steps:
+      - uses: actions/checkout@v2
+      - name: Set up Python ${{ matrix.python-version }}
+        uses: actions/setup-python@v2
+        with:
+          python-version: ${{ matrix.python-version }}
+        if: ${{ matrix.env-type.env-type-name == 'venv' }}
+      - name: Set up Python ${{ matrix.python-version }} with conda
+        uses: conda-incubator/setup-miniconda@v2
+        with:
+          activate-environment: test
+          auto-activate-base: false
+          auto-update-conda: true
+          miniforge-version: latest
+          python-version: ${{ matrix.python-version }}
+        if: ${{ matrix.env-type.env-type-name == 'conda' }}
+      - name: Print Python information
+        run: |
+          command -v python
+          $(command -v python) --version
+      - name: Update pip and setuptools
+        run: python -m pip install -U pip setuptools
+      - name: Print pip config
+        run: python -m pip config list
+      - name: Install test dependencies
+        run: python -m pip install pytest-cov
+      - name: Install package
+        run: python -m pip install .
+      - name: Print installed package contents
+        run: |
+          cd ~
+          PACKAGE_INSTALL_DIR=$(python -c 'import qcschema as _; print(_.__path__[0])')
+          echo "${PACKAGE_INSTALL_DIR}"
+          find $PACKAGE_INSTALL_DIR -type f | sort
+      - name: Print Python environment
+        run: python -m pip list
+      - name: Print conda environment
+        run: conda list
+        if: ${{ matrix.env-type.env-type-name == 'conda' }}
+      - name: Print conda info
+        run: conda info
+        if: ${{ matrix.env-type.env-type-name == 'conda' }}
+      - name: Print conda config
+        run: conda config --show
+        if: ${{ matrix.env-type.env-type-name == 'conda' }}
+      - name: Run tests
+        run: |
+          # Testing the installed package requires moving out of the source
+          # directory.  There are problems with the pytest cache when trying
+          # to run from a non-writable dir.
+          cd ~
+          PACKAGE_INSTALL_DIR=$(python -c 'import qcschema as _; print(_.__path__[0])')
+          echo "${PACKAGE_INSTALL_DIR}"
+          python -m pytest -v --cov=qcschema --cov-report=xml ${PACKAGE_INSTALL_DIR}
+      # - name: Install docs dependencies
+      #   run: |
+      #     python -m pip install sphinx==1.2.3 sphinxcontrib-napoleon sphinx_rtd_theme numpydoc
+      # - name: Make docs
+      #   run: make docs
+      # - name: CodeCov
+      #   users: codecov/codecov-action@v1
+      #   with:
+      #     file: ./coverage.xml
+      #     flags: unittests
+      #     name: codecov-py${{ matrix.python-version }}

MANIFEST.in

@@ -0,0 +1,2 @@
+recursive-include qcschema *.json
+recursive-include qcschema *.schema

tests/test_failures.py → qcschema/tests/test_failures.py

@@ -3,9 +3,8 @@
 """
 import jsonschema
 import pytest
-import os
 
-import test_helpers
+from qcschema.tests import test_helpers
 import qcschema
 
 ### Test input validation errors

tests/test_helpers.py → qcschema/tests/test_helpers.py

@@ -2,14 +2,9 @@
 Contains helper scripts to assist in testing the schema
 """
 
-import pytest
-import os
 import glob
-import copy
 import json
-import subprocess
-
-import qcschema
+import os
 
 
 def _read_json_file(*filename):
@@ -20,15 +15,28 @@ def _read_json_file(*filename):
 
 # Find a few required relative paths
 _test_path = os.path.dirname(os.path.abspath(__file__))
-_base_path = os.path.dirname(_test_path)
+
 
 def list_tests(folder, ext=".json", matcher=""):
     """
     Lists all tests in a given folder.
+
+    Since this function operates under the assumption that tests were expected
+    in the given folder, raise a RuntimeError if none were found.  This is a
+    workaround for tests being skipped when their parametrizations receive no
+    values.
     """
     files = glob.glob(os.path.join(_test_path, folder, "*" + matcher + "*" + ext))
     names = [os.path.basename(x).replace(ext, "") for x in files]
 
+    if not files:
+        raise RuntimeError(
+            "No files were found in the folder "
+            "{} with the extension '{}' and matcher '{}'".format(
+                os.path.abspath(folder), ext, matcher
+            )
+        )
+
     return files, names
 
 def get_test(name):

tests/test_schema.py → qcschema/tests/test_schema.py

@@ -3,9 +3,8 @@
 """
 import jsonschema
 import pytest
-import os
 
-import test_helpers
+from qcschema.tests import test_helpers
 import qcschema
 
 

setup.py

@@ -9,6 +9,7 @@
         url="https://github.com/MolSSI/QCSchema",
         license='',
         packages=setuptools.find_packages(),
+        include_package_data=True,
         install_requires=[
             'jsonschema',
             'pathlib2; python_version < "3.5"',  # redundant with jsonschema
@@ -28,5 +29,5 @@
         tests_require=[
             'pytest',
         ],
-        zip_safe=True,
+        zip_safe=False,
     )