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.

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

feat: Add e2e tests for Reports, extend reports validation #541

Merged
merged 4 commits into from
Sep 13, 2024

Conversation

natalialanga
Copy link
Contributor

@natalialanga natalialanga commented Sep 12, 2024

Summary

  • PC-13726 Add e2e tests for Reports
  • PC-13759 Snapshot Rrule frequency lower than DAILY forbidden
  • PC-13823 Snapshot date time in the future forbidden
  • PC-14191 Thresholds lower than 0 allowed

@natalialanga natalialanga merged commit a8e49f9 into main Sep 13, 2024
5 checks passed
@natalialanga natalialanga deleted the PC-13726-reports-tests branch September 13, 2024 13:10
renovate bot referenced this pull request in nobl9/sloctl Sep 14, 2024
This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
|
[github.com/nobl9/nobl9-go](https://redirect.github.com/nobl9/nobl9-go)
| `v0.84.1-rc4` -> `v0.84.1-rc6` |
[![age](https://developer.mend.io/api/mc/badges/age/go/github.com%2fnobl9%2fnobl9-go/v0.84.1-rc6?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/go/github.com%2fnobl9%2fnobl9-go/v0.84.1-rc6?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/go/github.com%2fnobl9%2fnobl9-go/v0.84.1-rc4/v0.84.1-rc6?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/go/github.com%2fnobl9%2fnobl9-go/v0.84.1-rc4/v0.84.1-rc6?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>nobl9/nobl9-go (github.com/nobl9/nobl9-go)</summary>

###
[`v0.84.1-rc6`](https://redirect.github.com/nobl9/nobl9-go/releases/tag/v0.84.1-rc6)

[Compare
Source](https://redirect.github.com/nobl9/nobl9-go/compare/v0.84.1-rc5...v0.84.1-rc6)

#### What's Changed

- feat: Add e2e tests for Reports, extend reports validation by
[@&#8203;natalialanga](https://redirect.github.com/natalialanga) in
[https://github.com/nobl9/nobl9-go/pull/541](https://redirect.github.com/nobl9/nobl9-go/pull/541)

**Full Changelog**:
nobl9/nobl9-go@v0.84.1-rc5...v0.84.1-rc6

###
[`v0.84.1-rc5`](https://redirect.github.com/nobl9/nobl9-go/releases/tag/v0.84.1-rc5)

[Compare
Source](https://redirect.github.com/nobl9/nobl9-go/compare/v0.84.1-rc4...v0.84.1-rc5)

### What's Changed

#### 🚀 Features

- feat: PC-13758 Add limits for recurring events
([#&#8203;533](https://redirect.github.com/nobl9/nobl9-go/issues/533))
[@&#8203;kubaceg](https://redirect.github.com/kubaceg)
    > Added frequency validation in `BudgetAdjustments` rrule field.
- feat: PC-13724 Add custom thresholds for SHR
([#&#8203;530](https://redirect.github.com/nobl9/nobl9-go/issues/530))
[@&#8203;lukasz-dobek](https://redirect.github.com/lukasz-dobek)
    > `Thresholds` object with three fields (`GreenGreaterThan`,
    > `RedLessThanOrEqualTo` and `ShowNoData`) was added to
> `SystemHealthReviewConfig`. It allows users to define custom values
for
    > `Healthy` (`Green`), `AtRisk` (`Yellow`) and `Exhausted` (`Red`)
> thresholds and to decide whether to display or hide `NoData` SLOs on
> System Health Review report. It will be available once Nobl9 platform
    > version 1.97.2 is released.
- feat: Switch to govy for validation
([#&#8203;527](https://redirect.github.com/nobl9/nobl9-go/issues/527))
[@&#8203;nieomylnieja](https://redirect.github.com/nieomylnieja)
    > `nobl9-go` is switching to a brand new validation library,
    > [govy](https://redirect.github.com/nobl9/govy).
    > The library was previously part of nobl9-go, but is now being open
    > sourced! Go check it out ;)

#### 🧰 Maintenance

<details>
<summary>6 changes</summary>

- chore: Add explicit generic arguments as workaround for GoLand IDE bug
([#&#8203;539](https://redirect.github.com/nobl9/nobl9-go/issues/539))
[@&#8203;ditrytus](https://redirect.github.com/ditrytus)
- chore: export metric spec validation rules
([#&#8203;536](https://redirect.github.com/nobl9/nobl9-go/issues/536))
[@&#8203;kuklyy](https://redirect.github.com/kuklyy)
- chore: Update module golang.org/x/tools/cmd/goimports to v0.25.0
([#&#8203;540](https://redirect.github.com/nobl9/nobl9-go/issues/540))
[@&#8203;renovate](https://redirect.github.com/renovate)
- chore: Update module golang.org/x/text to v0.18.0
([#&#8203;535](https://redirect.github.com/nobl9/nobl9-go/issues/535))
[@&#8203;renovate](https://redirect.github.com/renovate)
- chore: Update dependency yaml to v2.5.1
([#&#8203;532](https://redirect.github.com/nobl9/nobl9-go/issues/532))
[@&#8203;renovate](https://redirect.github.com/renovate)
- chore: Update minor and patch Golang dependencies
([#&#8203;491](https://redirect.github.com/nobl9/nobl9-go/issues/491))
[@&#8203;renovate](https://redirect.github.com/renovate)

</details>

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "after 10pm every weekday,before 5am
every weekday,every weekend" (UTC), Automerge - At any time (no schedule
defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR was generated by [Mend Renovate](https://mend.io/renovate/).
View the [repository job
log](https://developer.mend.io/github/nobl9/sloctl).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOC43NC4xIiwidXBkYXRlZEluVmVyIjoiMzguNzQuMSIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOlsiZGVwZW5kZW5jaWVzIiwiZ29sYW5nIiwicmVub3ZhdGUiXX0=-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request go minor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants