NaturalHistoryMuseum / ckanext-versioned-datastore

72%

main: 72%

LAST BUILD ON BRANCH josh/fix-multi-count-logic-error

branch: josh/fix-multi-count-logic-error

CHANGE BRANCH

x

Reset

• josh/fix-multi-count-logic-error
• HEAD
• ckan-2.9.5
• ckan-2.9.6
• dev
• ginger/autocomplete
• ginger/citation-version
• ginger/ckantools
• ginger/ckantools-0.0.4
• ginger/community-health
• ginger/deprecate-ckanpackager
• ginger/docker-compose-cmd
• ginger/download-init-hook
• ginger/downloads-v2
• ginger/eml-fixes
• ginger/eml-title
• ginger/faster-resource-check
• ginger/gabby-fixes
• ginger/ignore-missing
• ginger/ignore-missing-resources
• ginger/init-nav-slugs
• ginger/multi-resource-slug
• ginger/notifier-failing
• ginger/query-doi-versions
• ginger/remove-excel
• ginger/resource-exists
• ginger/rtd-config-update
• ginger/scrolling-query
• ginger/single-email-popup
• ginger/single-item-or
• ginger/skip-empty-rows
• ginger/slugerator
• ginger/sort-version
• ginger/standardisation
• ginger/standardise
• ginger/status
• ginger/status-cache
• ginger/status-patches
• ginger/sync-branch-workflow
• ginger/update-api-docs
• ginger/update-logos
• ginger/vernacular-dwc
• ginger/version-helpers
• ginger/zero-hit-resources
• josh/add_datastore_multisearch_counts_action
• josh/add_some_more_req_tests
• josh/catch-xlsx-null
• josh/ckan-2.10
• josh/datastore_get_resource_versions_perf
• josh/deduplicate_resource_ids
• josh/download_filter_out_none_counts
• josh/fix-docker-postgres-start-error
• josh/fix-long-http-line-in-es-error
• josh/fix-query-dois-error
• josh/fix-typo-on-datastore-page
• josh/fix_custom_download_filename_missing
• josh/guard_xlsx_save
• josh/preps
• josh/records_validation
• josh/update-splitgill
• josh/use_ckantest_image
• josh/vds_multi_direct
• main
• patch

Build # #661

Build Type

Pull #209

coveralls-python

Committed by jrdh

Commit Message

fix: fix summing logic to account for arc indices

If records in a resource are spread over multiple indices, the agg on _index will produce multiple results for a single resource which under the previous logic was not accounted for. This resulted in a counts dict which only contained the count for a resource from the last index iterated over containing data from that resource. This fixes this issue.

Closes: #208

Pull Request Pull Request #209: fix: fix summing logic to account for arc indices

Run Details

0 of 2 new or added lines in 1 file covered. (0.0%)

1 existing line in 1 file now uncovered.

2910 of 4036 relevant lines covered (72.1%)

0.72 hits per line