zscaler / terraform-provider-zia

:cloud: Terraform Provider for Zscaler Internet Access :cloud:
MIT License
37 stars 5 forks source link

fix: Fixed zia_url_categories datasource search mechanism #370

Closed willguibr closed 1 month ago

willguibr commented 1 month ago

Provide a general summary of your changes in the title above. You should remove this overview, any sections and any section descriptions you don't need below before submitting. There isn't a strict requirement to use this template if you can structure your description and still cover these points.

Description

Describe your changes in detail through motivation and context. Why is this change required? What problem does it solve? If it fixes an open issue, link to the issue using GitHub's closing issues keywords1.

Has your change been tested?

Explain how the change has been tested and what you ran to confirm your change affects other parts of the code. Automated tests are generally expected and changes without tests should explain why they aren't required.

Screenshots (if appropriate):

Types of changes

What sort of change does your code introduce/modify?

Checklist:

codecov-commenter commented 1 month ago

Codecov Report

Attention: Patch coverage is 31.70732% with 28 lines in your changes missing coverage. Please review.

Project coverage is 68.29%. Comparing base (015e856) to head (47b1e62).

Files with missing lines Patch % Lines
zia/data_source_zia_url_categories.go 31.70% 28 Missing :warning:
Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/zscaler/terraform-provider-zia/pull/370/graphs/tree.svg?width=650&height=150&src=pr&token=A9J4AJS7F5&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=zscaler)](https://app.codecov.io/gh/zscaler/terraform-provider-zia/pull/370?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=zscaler) ```diff @@ Coverage Diff @@ ## master #370 +/- ## ========================================== - Coverage 68.50% 68.29% -0.22% ========================================== Files 82 82 Lines 13883 13910 +27 ========================================== - Hits 9511 9500 -11 - Misses 3780 3820 +40 + Partials 592 590 -2 ``` | [Files with missing lines](https://app.codecov.io/gh/zscaler/terraform-provider-zia/pull/370?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=zscaler) | Coverage Δ | | |---|---|---| | [zia/data\_source\_zia\_url\_categories.go](https://app.codecov.io/gh/zscaler/terraform-provider-zia/pull/370?src=pr&el=tree&filepath=zia%2Fdata_source_zia_url_categories.go&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=zscaler#diff-emlhL2RhdGFfc291cmNlX3ppYV91cmxfY2F0ZWdvcmllcy5nbw==) | `80.39% <31.70%> (-8.21%)` | :arrow_down: | ... and [4 files with indirect coverage changes](https://app.codecov.io/gh/zscaler/terraform-provider-zia/pull/370/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=zscaler)