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

Add expected error/success messages to view tests #2475

Open
Tracked by #765
timobrembeck opened this issue Oct 23, 2023 · 0 comments
Open
Tracked by #765

Add expected error/success messages to view tests #2475

timobrembeck opened this issue Oct 23, 2023 · 0 comments
Labels
🔨 enhancement This improves an existing feature ⁉️ prio: low Not urgent, can be resolved in the distant future. 😅 effort: medium Should be doable in <12h
Milestone

Comments

@timobrembeck
Copy link
Member

Motivation

After #2454, it is now possible to view the user messages in the server logs in debug mode.

Proposed Solution

We could use this to further improve the view tests by adding a new "expected messages" column to the each entry in the view config.
E.g. for normal get-views, we could make sure that no error messages are present, or for form-submit-views we require the correct success message to exist.

Creating the initial baseline version for these kind of tests is probably very time consuming, maybe it's possible to somehow script this by inserting empty lists first and then extracting the actual messages from the failed tests?

@timobrembeck timobrembeck added ⁉️ prio: low Not urgent, can be resolved in the distant future. 🔨 enhancement This improves an existing feature 😅 effort: medium Should be doable in <12h labels Oct 23, 2023
@timobrembeck timobrembeck added this to the Backlog milestone Oct 23, 2023
@timobrembeck timobrembeck mentioned this issue Oct 23, 2023
33 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔨 enhancement This improves an existing feature ⁉️ prio: low Not urgent, can be resolved in the distant future. 😅 effort: medium Should be doable in <12h
Projects
None yet
Development

No branches or pull requests

1 participant