Skip to content

Conversation

@mcarans
Copy link
Contributor

@mcarans mcarans commented Jan 10, 2025

No description provided.

@github-actions
Copy link

github-actions bot commented Jan 10, 2025

Test Results

139 tests  +1   139 ✅ +1   1m 51s ⏱️ +28s
  1 suites ±0     0 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit 245c347. ± Comparison against base commit c13ff6c.

This pull request removes 3 and adds 4 tests. Note that renamed tests count towards both.
tests.hdx.data.test_dataset_title_helper.TestDatasetTitleHelper ‑ test_fuzzy_match_dates_in_title
tests.hdx.data.test_dataset_title_helper.TestDatasetTitleHelper ‑ test_get_date_from_title
tests.hdx.data.test_filestore_helper.TestFilestoreHelper ‑ test_dataset_update_filestore_resource
tests.hdx.utilities.test_dataset_title_helper.TestDatasetTitleHelper ‑ test_fuzzy_match_dates_in_title
tests.hdx.utilities.test_dataset_title_helper.TestDatasetTitleHelper ‑ test_get_date_from_title
tests.hdx.utilities.test_filestore_helper.TestFilestoreHelper ‑ test_dataset_update_filestore_resource
tests.hdx.utilities.test_hdx_error_handler.TestHDXErrorHandler ‑ test_hdx_error_handler

♻️ This comment has been updated with latest results.

@mcarans mcarans requested a review from b-j-mills January 10, 2025 03:19
@coveralls
Copy link

coveralls commented Jan 10, 2025

Coverage Status

coverage: 97.033% (-0.4%) from 97.474%
when pulling 245c347 on hdx_error_handler
into c13ff6c on main.

Copy link
Contributor

@b-j-mills b-j-mills left a comment

Choose a reason for hiding this comment

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

Looks great!

@mcarans mcarans merged commit 76d76cb into main Jan 12, 2025
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants