Skip to content

Commit 28d7afa

Browse files
committed
Removing duplicated tests
Signed-off-by: Friedrich Gonzalez <1517449+friedrichg@users.noreply.github.com>
1 parent 2f102ef commit 28d7afa

2 files changed

Lines changed: 19 additions & 230 deletions

File tree

integration/README.md

Lines changed: 19 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -22,25 +22,22 @@ docker stop cortex-test && docker rm cortex-test
2222

2323
## Coverage
2424

25-
| Command | Needs Cortex? | Tested? |
26-
|---------|:---:|:---:|
27-
| **rules load/list/print/get** | Yes | Yes |
28-
| **rules delete (group)** | Yes | Yes |
29-
| **rules delete-namespace** | Yes | Yes |
30-
| **rules lint** | No | Yes |
31-
| **rules check** | No | Yes |
32-
| **rules prepare** | No | Yes |
33-
| **rules parse/validate** | No | Yes |
34-
| **alertmanager load** | Yes | Yes |
35-
| **alertmanager load (with templates)** | Yes | Yes |
36-
| **alertmanager get** | Yes | Yes |
37-
| **alertmanager delete** | Yes | Yes |
38-
| **remote-read dump/stats** | Yes | Yes |
39-
| **remote-read export** | Yes | Yes |
40-
| **loadgen (write workload)** | Yes | Yes |
41-
| **analyse** (grafana/ruler/dashboard/rule-file) | No | No |
42-
| **acl** | No | No |
43-
| **bucket-validation** | Needs object store | No |
44-
| **config** (use-context) | No | No |
45-
| **alert verify** | Needs Cortex + alerting | No |
46-
| **push-gateway** | Needs push gateway | No |
25+
These integration tests require a running Cortex instance. Offline commands (lint, check, prepare, parse) are covered by unit tests in `pkg/rules/`.
26+
27+
| Command | Tested? |
28+
|---------|:---:|
29+
| **rules load/list/print/get** | Yes |
30+
| **rules delete (group)** | Yes |
31+
| **rules delete-namespace** | Yes |
32+
| **alertmanager load** | Yes |
33+
| **alertmanager load (with templates)** | Yes |
34+
| **alertmanager get** | Yes |
35+
| **alertmanager delete** | Yes |
36+
| **remote-read dump/stats** | Yes |
37+
| **remote-read export** | Yes |
38+
| **loadgen (write workload)** | Yes |
39+
| **analyse** (grafana/ruler/dashboard/rule-file) | No |
40+
| **bucket-validation** | No |
41+
| **config** (use-context) | No |
42+
| **alert verify** | No |
43+
| **push-gateway** | No |

integration/rules_offline_test.go

Lines changed: 0 additions & 208 deletions
This file was deleted.

0 commit comments

Comments
 (0)