Repository navigation
Convert nose / unittest tests to pytest #524
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 1 commit
Commits
Show all changes
77 commits
Select commit
Hold shift + click to select a range
f4e9cdb
Converted tests to pytest. Build a Python package. Update requirement…
rjurney c256244
Restore Python .gitignore
rjurney 6c3df0b
Extra newline removed
rjurney b2838d2
Merge branch 'master' of github.com:graphframes/graphframes into rjur…
rjurney caf5091
Added VERSION file set to 0.8.5
rjurney 7cfa2d1
isort; fiex edgesDF variable name.
rjurney 2ca9a15
Merge branch 'master' of github.com:graphframes/graphframes into rjur…
rjurney a8bf0be
Back out Dockerfile changes
rjurney 54a942d
Back out version change in build.sbt
rjurney 8b0e346
Backout changes to config and run-tests
rjurney 46c2b93
Back out pytest conversion
rjurney 18b5da0
Back out version changes to make nose tests pass
rjurney 8eca097
Remove changes to requirements
rjurney 277c06f
Put nose back in requirements.txt
rjurney b55ee48
Remove version bump to version.sbt
rjurney f8a8fd9
Remove packages related to testing
rjurney bc2cb36
Remove old setup.py / setup.cfg
rjurney 728be33
New pyproject.toml and poetry.lock
rjurney 3cea1a8
Short README for Python package, poetry won't allow a ../README.md path
rjurney 87cc975
Remove requirements files in favor of pyproject.toml
rjurney 6f84a5a
Try to poetrize CI build
rjurney 9a8eef0
pyspark min 3.4
rjurney 75ecd99
Local python README in pyproject.toml
rjurney 80231d0
Trying to remove he working folder to debug scala issue
rjurney 2a9170b
Set Python working directory again
rjurney 3de2263
Accidental newline
rjurney 4662717
Install Python for test...
rjurney 1b7b9f8
Run tests from python/ folder
rjurney 58da493
Try running tests from python/
rjurney 9f4aa24
poetry run the unit tests
rjurney 11b2782
poetry run the tests
rjurney 9772344
Try just using 'python' instead of a path
rjurney d55dbfe
poetry run the last line, graphframes.main
rjurney 2fc4d08
Remove test/ folder from style paths, it doesn't exist
rjurney 8297a13
Remove .vscode
rjurney 2035d98
VERSION back to 0.8.4
rjurney f9f4bd7
Remove tutorials reference
rjurney 9ddd6b2
VERSION is a Python thing, it belongs in python/
rjurney 7065647
Include the README.md and LICENSE in the Python package
rjurney a6c7e91
Some classifiers for pyproject.toml
rjurney 51e3e6d
Trying poetry install action instead of manual install
rjurney 272be06
Removing SPARK_HOME
rjurney 4587999
Returned SPARK_HOME settings
rjurney 2422b22
Minimized the PR to just these files
rjurney 073dced
Merge in rjurney/build-upgrades and in turn master
rjurney 0a1faba
Created tutorials dependency group to minimize main bloat
rjurney c0d6d7b
Make motif.py execute in whole again
rjurney 5bb4c26
Minor isort format and cleanup of download.py
rjurney 99e6a4d
Minor isort format and cleanup of utils.py
rjurney 662e197
Removed case sensitivity from the script - that was confusing people …
rjurney beaa35d
motif.py now matches tutorial code, runs and handles case insensitivity.
rjurney 1bf4a9e
Regenerate poetry.lock
rjurney ef19784
Setup a 'graphframes stackexchange' comand.
rjurney 4400cb4
Make graphframes.tutorials.motif use a checkpoint dir unique, and fro…
rjurney d549c56
Use spark.sparkContext.setCheckpointDir directly instead of instantia…
rjurney b970636
Using 'from __future__ import annotations' intsead of List and Tuple
rjurney 3788941
Now retry three times if we can't connect for any reason in 'graphfra…
rjurney e95bbbe
Merge master
rjurney 2372231
Merge branch 'master' of github.com:graphframes/graphframes into rjur…
rjurney 223bb8a
Convert nose/unittest tests to pytest
rjurney 9c9c2fb
+pytest
rjurney 1b786fb
Merge branch 'master' of github.com:graphframes/graphframes into rjur…
rjurney 265fea0
Merge main
rjurney 9023261
black formatted unit tests
rjurney bb72c26
Make tests pass flake8
rjurney f3983b5
Update tests.py to remove stale self.assertEquals
rjurney 86a7c49
Merge branch 'master' of github.com:graphframes/graphframes into rjur…
rjurney d06d0db
Merge branch 'rjurney/pytest' of github.com:graphframes/graphframes i…
rjurney b1fd254
Responding to a couple comments
rjurney f24ef7f
Removed unused SparkContext reference
rjurney d696462
Update version test code
rjurney 24ea47e
Revert "Update version test code"
rjurney aa6ad40
Revert "Removed unused SparkContext reference"
rjurney 9621ae9
Revert "Responding to a couple comments"
rjurney b74bcdc
Responding to comment
rjurney 200152e
+black format
rjurney 45d58bb
Update tests.py to remove unused import
rjurney File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
black formatted unit tests
- Loading branch information
commit 902326148670daf698fd4f5b8ad4aec99cf0b89a
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's move it to python/tests/test.py?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should really split it up into parts, but I want to wait and just keep it
tests.pyfor now. I'll do another PR soon to break it up into liketests/connected_components.pyand stuff.