Skip to content

Commit 65a654e

Browse files
rjurneyrjurney
authored
Convert nose / unittest tests to pytest (#524)
* Converted tests to pytest. Build a Python package. Update requirements.txt and split out requirements-dev.txt. Version bumps. * Restore Python .gitignore * Extra newline removed * Added VERSION file set to 0.8.5 * isort; fiex edgesDF variable name. * Back out Dockerfile changes * Back out version change in build.sbt * Backout changes to config and run-tests * Back out pytest conversion * Back out version changes to make nose tests pass * Remove changes to requirements * Put nose back in requirements.txt * Remove version bump to version.sbt * Remove packages related to testing * Remove old setup.py / setup.cfg * New pyproject.toml and poetry.lock * Short README for Python package, poetry won't allow a ../README.md path * Remove requirements files in favor of pyproject.toml * Try to poetrize CI build * pyspark min 3.4 * Local python README in pyproject.toml * Trying to remove he working folder to debug scala issue * Set Python working directory again * Accidental newline * Install Python for test... * Run tests from python/ folder * Try running tests from python/ * poetry run the unit tests * poetry run the tests * Try just using 'python' instead of a path * poetry run the last line, graphframes.main * Remove test/ folder from style paths, it doesn't exist * Remove .vscode * VERSION back to 0.8.4 * Remove tutorials reference * VERSION is a Python thing, it belongs in python/ * Include the README.md and LICENSE in the Python package * Some classifiers for pyproject.toml * Trying poetry install action instead of manual install * Removing SPARK_HOME * Returned SPARK_HOME settings * Minimized the PR to just these files * Created tutorials dependency group to minimize main bloat * Make motif.py execute in whole again * Minor isort format and cleanup of download.py * Minor isort format and cleanup of utils.py * Removed case sensitivity from the script - that was confusing people who just pasted or tried to run the code without a new SparkSession. * motif.py now matches tutorial code, runs and handles case insensitivity. * Setup a 'graphframes stackexchange' comand. * Make graphframes.tutorials.motif use a checkpoint dir unique, and from SparkSession.sparkContext. Use click.echo instead of print * Use spark.sparkContext.setCheckpointDir directly instead of instantiating a SparkContext. print-->click.echo * Using 'from __future__ import annotations' intsead of List and Tuple * Now retry three times if we can't connect for any reason in 'graphframes stackexchange' command. * Convert nose/unittest tests to pytest * +pytest * black formatted unit tests * Make tests pass flake8 * Update tests.py to remove stale self.assertEquals * Responding to a couple comments * Removed unused SparkContext reference * Update version test code * Revert "Update version test code" This reverts commit d696462. * Revert "Removed unused SparkContext reference" This reverts commit f24ef7f. * Revert "Responding to a couple comments" This reverts commit b1fd254. * Responding to comment * +black format * Update tests.py to remove unused import
1 parent bc31faa commit 65a654e

4 files changed

Lines changed: 269 additions & 143 deletions

File tree

0 commit comments

Comments
 (0)