Skip to content

🧹 Improve discovery filters parsing in the AWS provider.#6310

Merged
czunker merged 1 commit intomainfrom
vasil/improve-aws-opts-parsing
Dec 16, 2025
Merged

🧹 Improve discovery filters parsing in the AWS provider.#6310
czunker merged 1 commit intomainfrom
vasil/improve-aws-opts-parsing

Conversation

@VasilSirakov
Copy link
Copy Markdown
Contributor

No description provided.

Signed-off-by: Vasil Sirakov <sirakov97@gmail.com>
}
}
}
return defaultVal
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Do we need a defaultValue as a separate parameter?

I only see false which would be the default anyway.

Copy link
Copy Markdown
Contributor Author

@VasilSirakov VasilSirakov Dec 16, 2025

Choose a reason for hiding this comment

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

You are right, we only need false as a default but I exposed it just to make the fn more generic. I can remove it if you think it's not worth it.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I don't know much about the AWS part.
Let's keep it.

@github-actions
Copy link
Copy Markdown
Contributor

Test Results

4 697 tests  +13   4 693 ✅ +13   2m 34s ⏱️ +14s
  415 suites ± 0       4 💤 ± 0 
   31 files   ± 0       0 ❌ ± 0 

Results for commit b6d6ee8. ± Comparison against base commit 4a5a0aa.

This pull request removes 3 and adds 16 tests. Note that renamed tests count towards both.
go.mondoo.com/cnquery/v12/providers/aws/connection ‑ TestParseOptsToFilters
go.mondoo.com/cnquery/v12/providers/aws/connection ‑ TestParseOptsToFilters/all_opts_are_mapped_to_discovery_filters_correctly
go.mondoo.com/cnquery/v12/providers/aws/connection ‑ TestParseOptsToFilters/empty_opts_are_mapped_to_discovery_filters_correctly
go.mondoo.com/cnquery/v12/providers/aws/connection ‑ TestDiscoveryFiltersFromOpts
go.mondoo.com/cnquery/v12/providers/aws/connection ‑ TestDiscoveryFiltersFromOpts/all_opts_are_mapped_to_discovery_filters_correctly
go.mondoo.com/cnquery/v12/providers/aws/connection ‑ TestDiscoveryFiltersFromOpts/empty_opts_are_mapped_to_discovery_filters_correctly
go.mondoo.com/cnquery/v12/providers/aws/connection ‑ TestDiscoveryFiltersFromOpts/nil_opts_are_mapped_to_discovery_filter_correctly
go.mondoo.com/cnquery/v12/providers/aws/connection ‑ TestParseBoolOpt
go.mondoo.com/cnquery/v12/providers/aws/connection ‑ TestParseBoolOpt/parses_false_values_correctly
go.mondoo.com/cnquery/v12/providers/aws/connection ‑ TestParseBoolOpt/parses_true_values_correctly
go.mondoo.com/cnquery/v12/providers/aws/connection ‑ TestParseBoolOpt/returns_default_for_missing_key
go.mondoo.com/cnquery/v12/providers/aws/connection ‑ TestParseCsvSliceOpt
go.mondoo.com/cnquery/v12/providers/aws/connection ‑ TestParseCsvSliceOpt/parses_comma-separated_values_correctly
…

@czunker czunker merged commit 6c16716 into main Dec 16, 2025
20 checks passed
@czunker czunker deleted the vasil/improve-aws-opts-parsing branch December 16, 2025 09:39
@github-actions github-actions bot locked and limited conversation to collaborators Dec 16, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants