Skip to content

python api tests & code cleanup #603

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.

Sign up for GitHub

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

Jump to bottom
Merged
zkaoudi merged 52 commits into from
Nov 17, 2025
Merged

python api tests & code cleanup #603

zkaoudi merged 52 commits into from
Nov 17, 2025

Conversation

@mspruc
Copy link
Contributor

@mspruc mspruc commented Sep 4, 2025

No description provided.

zkaoudi
zkaoudi requested changes Sep 4, 2025
View reviewed changes
Copy link
Contributor

@zkaoudi zkaoudi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

build is failing

@mspruc mspruc marked this pull request as draft November 5, 2025 08:49
@mspruc mspruc marked this pull request as ready for review November 5, 2025 19:38
@mspruc mspruc marked this pull request as draft November 5, 2025 19:38
mspruc added 26 commits November 10, 2025 09:11
@mspruc
add resources folder & contents
31991c0
@mspruc
licenses
fd8382a
@mspruc
license in __init__
5786d9f
@mspruc
change file type of sample_data
9f1c811
@mspruc
fix sample_data in test_filter
5760984
@mspruc
fix test
6bc722f
@mspruc
test data for test_json.py
ccf9c7d
@mspruc
move python tests after maven
35efdbd
@mspruc
move python dependencies before maven tests so tests can use them
e760a94
@mspruc
add missing arg to pytest
9218f3a
@mspruc
add resources for json test
5e30917
@mspruc
move over to mvn runner instead of bin
37b87fa
@mspruc
enable custom properties file
f81d012
@mspruc
add config to python test runner
860816f
@mspruc
add meta func for config files with pytest
365374f
@mspruc
add test directory to pytest
d12149b
@mspruc
remove unused quotations
27b036f
@mspruc
add omitted comma
2487a46
@mspruc
disable test requiring missing implementations
76ba1b6
@mspruc
explicitly write pom to exec from, pytest should now be executable fr…
e1ea95c 
…om root
@mspruc
add better process waiting
9b9c797
@mspruc
maven process for test_json
670215d
@mspruc
make json api respect custom configs
4471bb0
@mspruc
rm test_join as joins are already tested in filter and it was not set…
c34ed59 
…up to be run singularly
@mspruc
skip test missing impl
85503b6
@mspruc
update readme with pytest specifics
26f939c
@mspruc mspruc marked this pull request as ready for review November 17, 2025 08:46
@zkaoudi zkaoudi merged commit 1e0f9e8 into apache:main Nov 17, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@zkaoudi zkaoudi zkaoudi approved these changes

@juripetersen juripetersen juripetersen approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mspruc @zkaoudi @juripetersen