Skip to content
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

Support CKAN 2.9 with Python 3 #98

Merged
merged 20 commits into from
Jul 9, 2021
Merged

Support CKAN 2.9 with Python 3 #98

merged 20 commits into from
Jul 9, 2021

Conversation

adborden
Copy link

FuhuXia and others added 14 commits May 11, 2021 17:02
@FuhuXia
remove CKAN 2.3 tests and functionality
05d17ec
Add CI tests for CKAN 2.8 and 2.9
f186d87
Update README
ec0405d
Update docker environment for matrix builds
d064eb1 
Support development and testing using CKAN_VERSION make variable. Move existing
legacy nose tests to docker-compose.legacy.yml and update make targets to
support it.
@FuhuXia
remove CKAN 2.3 tests and functionality
d3720b1
@FuhuXia
Merge branch 'feature/py3' into feature/py3-ci
d4e65f9
@FuhuXia
Merge branch 'feature/py3' into feature/docker-legacy
8761861
@FuhuXia
Merge pull request #95 from GSA/feature/py3-ci
9eb9451 
New CI tests for CKAN 2.8 and 2.9
@FuhuXia
Merge pull request #96 from GSA/feature/docker-legacy
dd21c4b 
Update docker environment
@FuhuXia
copy legacy tests to /nose
60ee3a8 
also combined test-inventory/test_export.py in to /tests
@FuhuXia
rewrite nose tests to pytest
534b4cd
@FuhuXia
futurize to python 3
b17da12 
$ pip install futurize
$ futurize --both-stages --write ckanext
@FuhuXia
replace paste\|pylons.* import with toolkit
280c5ad
@jbrown-xentity
Update test_collections_ui tests to pass
d430966
@github-actions
Copy link

github-actions bot commented Jun 2, 2021 •
edited
Loading

✅ Result of Pytest Coverage

---------- coverage: platform linux2, python 2.7.18-final-0 ----------

Name Stmts Miss Cover
ckanext/datajson/datajson_ckan_28.py 417 118 72%
ckanext/datajson/exceptions.py 2 0 100%
ckanext/datajson/harvester_base.py 4 0 100%
ckanext/datajson/harvester_datajson.py 46 13 72%
ckanext/datajson/helpers.py 117 44 62%
ckanext/datajson/package2pod.py 305 180 41%
ckanext/datajson/parse_datajson.py 94 21 78%
ckanext/datajson/plugin.py 238 97 59%
ckanext/datajson/tests/init.py 0 0 100%
ckanext/datajson/tests/factories.py 68 19 72%
ckanext/datajson/tests/mock_datajson_source.py 75 2 97%
ckanext/datajson/tests/test_collections_ui.py 101 8 92%
ckanext/datajson/tests/test_datajson_ckan_28_harvester.py 330 23 93%
ckanext/datajson/tests/test_datajson_ckan_all_harvester.py 171 8 95%
ckanext/datajson/tests/test_export.py 63 2 97%
TOTAL 2031 535 74%
Required test coverage of
================== 24 passed, 1449

jbrown-xentity and others added 6 commits June 2, 2021 16:24
@jbrown-xentity
Test cleanup
82d0498 
Clean up various test and configuration options
@jbrown-xentity
fix lint
019e99c
@jbrown-xentity
ckan2.3 cleanup and linting
5c31139
@FuhuXia
specify redis to be used for message queue
4756624
@FuhuXia
use postgres as db user for pg_isready
94507da 
otherwise it used root and gives error
@FuhuXia
add ckanext.datajson.inventory_links_enabled into test.ini
5781877 
somehow this is not working for us
config['ckanext.datajson.inventory_links_enabled'] = "True"
@mogul mogul merged commit 5781877 into main Jul 9, 2021
@mogul mogul deleted the feature/py3 branch July 9, 2021 23:49
@nickumia-reisys nickumia-reisys mentioned this pull request Jul 7, 2022
Closed
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@adborden @mogul @FuhuXia @jbrown-xentity