Upgrade test suite - #6
Merged
Merged
Conversation
… notification - Add allure-robotframework listener and generate results in CI - Publish Allure report (with history) to gh-pages on pushes to main - Upload Allure report as PR artifact and attach single-file report to email - Add contents: write permission needed for gh-pages deploy - Fix xunit.xml artifact path (results/xunit.xml) and .gitignore for allure dirs
- Verify admin cannot access user home unrestricted vs regular user redirect to login on invalid session - Verify regular user cannot access admin home page (known ServeRest issue) - Verify duplicated email/product name are rejected on creation - Verify double-clicking submit does not create duplicated users - Add supporting keywords/resources: products_api.resource, session invalidation, alert message assertion, double-click helper
…calls - simple-elf/allure-report-action's Docker build was failing (openjdk:8-jre-alpine removed from Docker Hub, action unmaintained) - Generate report and restore history directly via Allure CLI instead - Deploy allure-report/ (now includes history) to GitHub Pages
- store.robot: create products via API with unique, random names instead of relying on fixed items from the shared public ServeRest catalog, which can collide with clutter from other students/QA courses (observed a duplicated "Logitech MX Vertical" break a locator in strict mode); reload the page after creation so the new product's card renders before interacting with it - admin.robot & store.robot: fix teardowns that were silently skipping cleanup — a test's own [Teardown] replaces the suite's Test Teardown instead of running alongside it, so "Cleanup Login Test" has to be chained explicitly; and Run Keywords needs "AND" even with a single keyword, otherwise its arguments get misread as extra keyword names to run - uppercase all keyword arguments and local variables project-wide, for a consistent style
git fetch was failing with retries/delay on every run before the first Pages deploy, since gh-pages doesn't exist yet; now checks via git ls-remote first and skips the checkout step entirely instead of letting it fail
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
feat: add Allure reporting to CI and admin/login authorization test cases.