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

fix return status in a few cases and add integration tests against content modification through Applications #1421

Merged
merged 8 commits into from
Oct 16, 2024

Conversation

blizzz
Copy link
Member

@blizzz blizzz commented Oct 15, 2024

The fixes include a few cases were 500 was returned, where 403 was expected, and also where it should be 404.

These were all found when writing the missing tests – the fifth batch – for #1006.

Closes #1006

@blizzz blizzz added bug Something isn't working 3. to review Waiting for reviews tests labels Oct 15, 2024
@blizzz
Copy link
Member Author

blizzz commented Oct 15, 2024

/backport to stable0.8

@backportbot backportbot bot added the backport-request Pending backport by the backport-bot label Oct 15, 2024
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
…cess

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
@blizzz blizzz force-pushed the tests/1006/context-content-mod-tests branch 3 times, most recently from 0af6580 to 414fa2e Compare October 15, 2024 19:07
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
@blizzz blizzz force-pushed the tests/1006/context-content-mod-tests branch from 414fa2e to 0ee59a2 Compare October 15, 2024 19:38
@blizzz blizzz merged commit 6c1750b into main Oct 16, 2024
61 checks passed
@blizzz blizzz deleted the tests/1006/context-content-mod-tests branch October 16, 2024 10:08
@backportbot backportbot bot removed the backport-request Pending backport by the backport-bot label Oct 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews bug Something isn't working tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Extend integration tests around Contexts
2 participants