Skip to content

Commit 929d7ad

Browse files
rearrange graph tests
1 parent a9d3844 commit 929d7ad

File tree

5 files changed

+67
-86
lines changed

5 files changed

+67
-86
lines changed

Diff for: .drone.star

+3-22
Original file line numberDiff line numberDiff line change
@@ -94,13 +94,13 @@ config = {
9494
"apiDepthInfinity",
9595
"apiLocks",
9696
"apiActivities",
97-
"apiServiceAvailability",
9897
],
9998
"skip": False,
10099
},
101-
"settings": {
100+
"settingsAndNotification": {
102101
"suites": [
103102
"apiSettings",
103+
"apiNotification",
104104
],
105105
"skip": False,
106106
"withRemotePhp": [True],
@@ -120,6 +120,7 @@ config = {
120120
},
121121
"graphBasicAndGroup": {
122122
"suites": [
123+
"apiServiceAvailability",
123124
"apiGraph",
124125
"apiGraphGroup",
125126
],
@@ -195,26 +196,6 @@ config = {
195196
],
196197
"accounts_hash_difficulty": "default",
197198
},
198-
"notification": {
199-
"suites": [
200-
"apiNotification",
201-
],
202-
"skip": False,
203-
"withRemotePhp": [True],
204-
"emailNeeded": True,
205-
"extraEnvironment": {
206-
"EMAIL_HOST": EMAIL_SMTP_HOST,
207-
"EMAIL_PORT": EMAIL_PORT,
208-
},
209-
"extraServerEnvironment": {
210-
"OCIS_ADD_RUN_SERVICES": "notifications",
211-
"NOTIFICATIONS_SMTP_HOST": EMAIL_SMTP_HOST,
212-
"NOTIFICATIONS_SMTP_PORT": EMAIL_SMTP_PORT,
213-
"NOTIFICATIONS_SMTP_INSECURE": "true",
214-
"NOTIFICATIONS_SMTP_SENDER": EMAIL_SMTP_SENDER,
215-
"NOTIFICATIONS_DEBUG_ADDR": "0.0.0.0:9174",
216-
},
217-
},
218199
"antivirus": {
219200
"suites": [
220201
"apiAntivirus",

Diff for: docs/ocis/development/testing.md

+9-9
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ While running the tests, oCIS server is started with [ociswrapper](https://githu
6262

6363
```bash
6464
WITH_WRAPPER=false \
65-
BEHAT_FEATURE='tests/acceptance/features/apiGraphGroup/createUser.feature:26' \
65+
BEHAT_FEATURE='tests/acceptance/features/apiGraphUser/createUser.feature:26' \
6666
make -C tests/acceptance/docker test-ocis-feature-ocis-storage
6767
```
6868

@@ -119,7 +119,7 @@ The tests for a single feature (a feature file) can also be run against the diff
119119
For example;
120120

121121
```bash
122-
BEHAT_FEATURE='tests/acceptance/features/apiGraphGroup/createUser.feature' \
122+
BEHAT_FEATURE='tests/acceptance/features/apiGraphUser/createUser.feature' \
123123
make -C tests/acceptance/docker test-ocis-feature-ocis-storage
124124
```
125125

@@ -130,23 +130,23 @@ make -C tests/acceptance/docker test-ocis-feature-ocis-storage
130130
And to run a single scenario in a feature, you can do:
131131

132132
{{< hint info >}}
133-
A specific scenario from a feature can be run by adding `:<line-number>` at the end of the feature file path. For example, to run the scenario at line 26 of the feature file `apiGraphUserGroup/createUser.feature`, simply add the line number like this: `apiGraphUserGroup/createUser.feature:26`. Note that the line numbers mentioned in the examples might not always point to a scenario, so always check the line numbers before running the test.
133+
A specific scenario from a feature can be run by adding `:<line-number>` at the end of the feature file path. For example, to run the scenario at line 26 of the feature file `apiGraphUser/createUser.feature`, simply add the line number like this: `apiGraphUser/createUser.feature:26`. Note that the line numbers mentioned in the examples might not always point to a scenario, so always check the line numbers before running the test.
134134
{{< /hint >}}
135135

136136
```bash
137-
BEHAT_FEATURE='tests/acceptance/features/apiGraphGroup/createUser.feature:26' \
137+
BEHAT_FEATURE='tests/acceptance/features/apiGraphUser/createUser.feature:26' \
138138
make -C tests/acceptance/docker test-ocis-feature-ocis-storage
139139
```
140140

141141
Similarly, with S3 storage;
142142

143143
```bash
144144
# run a whole feature
145-
BEHAT_FEATURE='tests/acceptance/features/apiGraphGroup/createUser.feature' \
145+
BEHAT_FEATURE='tests/acceptance/features/apiGraphUser/createUser.feature' \
146146
make -C tests/acceptance/docker test-ocis-feature-s3ng-storage
147147

148148
# run a single scenario
149-
BEHAT_FEATURE='tests/acceptance/features/apiGraphGroup/createUser.feature:26' \
149+
BEHAT_FEATURE='tests/acceptance/features/apiGraphUser/createUser.feature:26' \
150150
make -C tests/acceptance/docker test-ocis-feature-s3ng-storage
151151
```
152152

@@ -227,16 +227,16 @@ Useful environment variables:
227227
`BEHAT_FEATURE`: to run a single feature
228228

229229
{{< hint info >}}
230-
A specific scenario from a feature can be run by adding `:<line-number>` at the end of the feature file path. For example, to run the scenario at line 26 of the feature file `apiGraphUserGroup/createUser.feature`, simply add the line number like this: `apiGraphUserGroup/createUser.feature:26`. Note that the line numbers mentioned in the examples might not always point to a scenario, so always check the line numbers before running the test.
230+
A specific scenario from a feature can be run by adding `:<line-number>` at the end of the feature file path. For example, to run the scenario at line 26 of the feature file `apiGraphUser/createUser.feature`, simply add the line number like this: `apiGraphUser/createUser.feature:26`. Note that the line numbers mentioned in the examples might not always point to a scenario, so always check the line numbers before running the test.
231231
{{< /hint >}}
232232

233233
> Example:
234234
>
235-
> BEHAT_FEATURE=tests/acceptance/features/apiGraphUserGroup/createUser.feature
235+
> BEHAT_FEATURE=tests/acceptance/features/apiGraphUser/createUser.feature
236236
>
237237
> Or
238238
>
239-
> BEHAT_FEATURE=tests/acceptance/features/apiGraphUserGroup/createUser.feature:13
239+
> BEHAT_FEATURE=tests/acceptance/features/apiGraphUser/createUser.feature:13
240240
241241
`BEHAT_SUITE`: to run a single suite
242242

Diff for: tests/acceptance/expected-failures-localAPI-on-OCIS-storage.md

+41-41
Original file line numberDiff line numberDiff line change
@@ -27,58 +27,58 @@ The expected failures in this file are from features in the owncloud/ocis repo.
2727

2828
#### [A User can get information of another user with Graph API](https://github.com/owncloud/ocis/issues/5125)
2929

30-
- [apiGraphUserGroup/getUser.feature:84](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/getUser.feature#L84)
31-
- [apiGraphUserGroup/getUser.feature:85](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/getUser.feature#L85)
32-
- [apiGraphUserGroup/getUser.feature:86](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/getUser.feature#L86)
33-
- [apiGraphUserGroup/getUser.feature:87](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/getUser.feature#L87)
34-
- [apiGraphUserGroup/getUser.feature:88](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/getUser.feature#L88)
35-
- [apiGraphUserGroup/getUser.feature:89](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/getUser.feature#L89)
36-
- [apiGraphUserGroup/getUser.feature:90](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/getUser.feature#L90)
37-
- [apiGraphUserGroup/getUser.feature:91](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/getUser.feature#L91)
38-
- [apiGraphUserGroup/getUser.feature:92](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/getUser.feature#L92)
39-
- [apiGraphUserGroup/getUser.feature:93](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/getUser.feature#L93)
40-
- [apiGraphUserGroup/getUser.feature:94](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/getUser.feature#L94)
41-
- [apiGraphUserGroup/getUser.feature:95](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/getUser.feature#L95)
42-
- [apiGraphUserGroup/getUser.feature:637](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/getUser.feature#L637)
43-
- [apiGraphUserGroup/getUser.feature:638](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/getUser.feature#L638)
44-
- [apiGraphUserGroup/getUser.feature:639](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/getUser.feature#L639)
45-
- [apiGraphUserGroup/getUser.feature:640](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/getUser.feature#L640)
46-
- [apiGraphUserGroup/getUser.feature:641](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/getUser.feature#L641)
47-
- [apiGraphUserGroup/getUser.feature:642](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/getUser.feature#L642)
48-
- [apiGraphUserGroup/getUser.feature:643](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/getUser.feature#L643)
49-
- [apiGraphUserGroup/getUser.feature:644](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/getUser.feature#L644)
50-
- [apiGraphUserGroup/getUser.feature:645](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/getUser.feature#L645)
51-
- [apiGraphUserGroup/getUser.feature:646](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/getUser.feature#L646)
52-
- [apiGraphUserGroup/getUser.feature:647](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/getUser.feature#L647)
53-
- [apiGraphUserGroup/getUser.feature:648](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/getUser.feature#L648)
54-
- [apiGraphUserGroup/getUser.feature:663](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/getUser.feature#L663)
55-
- [apiGraphUserGroup/getUser.feature:664](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/getUser.feature#L664)
56-
- [apiGraphUserGroup/getUser.feature:665](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/getUser.feature#L665)
30+
- [apiGraphUser/getUser.feature:84](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUser/getUser.feature#L84)
31+
- [apiGraphUser/getUser.feature:85](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUser/getUser.feature#L85)
32+
- [apiGraphUser/getUser.feature:86](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUser/getUser.feature#L86)
33+
- [apiGraphUser/getUser.feature:87](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUser/getUser.feature#L87)
34+
- [apiGraphUser/getUser.feature:88](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUser/getUser.feature#L88)
35+
- [apiGraphUser/getUser.feature:89](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUser/getUser.feature#L89)
36+
- [apiGraphUser/getUser.feature:90](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUser/getUser.feature#L90)
37+
- [apiGraphUser/getUser.feature:91](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUser/getUser.feature#L91)
38+
- [apiGraphUser/getUser.feature:92](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUser/getUser.feature#L92)
39+
- [apiGraphUser/getUser.feature:93](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUser/getUser.feature#L93)
40+
- [apiGraphUser/getUser.feature:94](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUser/getUser.feature#L94)
41+
- [apiGraphUser/getUser.feature:95](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUser/getUser.feature#L95)
42+
- [apiGraphUser/getUser.feature:637](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUser/getUser.feature#L637)
43+
- [apiGraphUser/getUser.feature:638](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUser/getUser.feature#L638)
44+
- [apiGraphUser/getUser.feature:639](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUser/getUser.feature#L639)
45+
- [apiGraphUser/getUser.feature:640](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUser/getUser.feature#L640)
46+
- [apiGraphUser/getUser.feature:641](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUser/getUser.feature#L641)
47+
- [apiGraphUser/getUser.feature:642](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUser/getUser.feature#L642)
48+
- [apiGraphUser/getUser.feature:643](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUser/getUser.feature#L643)
49+
- [apiGraphUser/getUser.feature:644](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUser/getUser.feature#L644)
50+
- [apiGraphUser/getUser.feature:645](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUser/getUser.feature#L645)
51+
- [apiGraphUser/getUser.feature:646](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUser/getUser.feature#L646)
52+
- [apiGraphUser/getUser.feature:647](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUser/getUser.feature#L647)
53+
- [apiGraphUser/getUser.feature:648](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUser/getUser.feature#L648)
54+
- [apiGraphUser/getUser.feature:663](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUser/getUser.feature#L663)
55+
- [apiGraphUser/getUser.feature:664](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUser/getUser.feature#L664)
56+
- [apiGraphUser/getUser.feature:665](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUser/getUser.feature#L665)
5757

5858
#### [Normal user can get expanded members information of a group](https://github.com/owncloud/ocis/issues/5604)
5959

60-
- [apiGraphUserGroup/getGroup.feature:399](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/getGroup.feature#L399)
61-
- [apiGraphUserGroup/getGroup.feature:400](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/getGroup.feature#L400)
62-
- [apiGraphUserGroup/getGroup.feature:401](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/getGroup.feature#L401)
63-
- [apiGraphUserGroup/getGroup.feature:460](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/getGroup.feature#L460)
64-
- [apiGraphUserGroup/getGroup.feature:461](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/getGroup.feature#L461)
65-
- [apiGraphUserGroup/getGroup.feature:462](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/getGroup.feature#L462)
66-
- [apiGraphUserGroup/getGroup.feature:508](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/getGroup.feature#L508)
67-
- [apiGraphUserGroup/getGroup.feature:509](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/getGroup.feature#L509)
68-
- [apiGraphUserGroup/getGroup.feature:510](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/getGroup.feature#L510)
60+
- [apiGraphGroup/getGroup.feature:399](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphGroup/getGroup.feature#L399)
61+
- [apiGraphGroup/getGroup.feature:400](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphGroup/getGroup.feature#L400)
62+
- [apiGraphGroup/getGroup.feature:401](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphGroup/getGroup.feature#L401)
63+
- [apiGraphGroup/getGroup.feature:460](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphGroup/getGroup.feature#L460)
64+
- [apiGraphGroup/getGroup.feature:461](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphGroup/getGroup.feature#L461)
65+
- [apiGraphGroup/getGroup.feature:462](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphGroup/getGroup.feature#L462)
66+
- [apiGraphGroup/getGroup.feature:508](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphGroup/getGroup.feature#L508)
67+
- [apiGraphGroup/getGroup.feature:509](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphGroup/getGroup.feature#L509)
68+
- [apiGraphGroup/getGroup.feature:510](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphGroup/getGroup.feature#L510)
6969

7070
#### [Same users can be added in a group multiple time](https://github.com/owncloud/ocis/issues/5702)
7171

72-
- [apiGraphUserGroup/addUserToGroup.feature:295](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/addUserToGroup.feature#L295)
72+
- [apiGraphGroup/addUserToGroup.feature:295](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphGroup/addUserToGroup.feature#L295)
7373

7474
#### [Users are added in a group with wrong host in host-part of user](https://github.com/owncloud/ocis/issues/5871)
7575

76-
- [apiGraphUserGroup/addUserToGroup.feature:379](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/addUserToGroup.feature#L379)
77-
- [apiGraphUserGroup/addUserToGroup.feature:393](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/addUserToGroup.feature#L393)
76+
- [apiGraphGroup/addUserToGroup.feature:379](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphGroup/addUserToGroup.feature#L379)
77+
- [apiGraphGroup/addUserToGroup.feature:393](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphGroup/addUserToGroup.feature#L393)
7878

7979
#### [Adding the same user as multiple members in a single request results in listing the same user twice in the group](https://github.com/owncloud/ocis/issues/5855)
8080

81-
- [apiGraphUserGroup/addUserToGroup.feature:430](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/addUserToGroup.feature#L430)
81+
- [apiGraphGroup/addUserToGroup.feature:430](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphGroup/addUserToGroup.feature#L430)
8282

8383
#### [Shared file locking is not possible using different path](https://github.com/owncloud/ocis/issues/7599)
8484

@@ -336,7 +336,7 @@ The expected failures in this file are from features in the owncloud/ocis repo.
336336
# - [apiOcm/share.feature:1178](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiOcm/share.feature#L1178)
337337

338338
#### [Delete user containing + in username returns 404 but user gets deleted](https://github.com/owncloud/ocis/issues/11027)
339-
- [apiGraphUserGroup/deleteUser.feature:28](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUserGroup/deleteUser.feature#L28)
339+
- [apiGraphUser/deleteUser.feature:28](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraphUser/deleteUser.feature#L28)
340340

341341
Note: always have an empty line at the end of this file.
342342
The bash script that processes this file requires that the last line has a newline on the end.

0 commit comments

Comments
 (0)