GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Add @allure.manual decorator to simplify ALLURE_MANUAL=True label setting by betapl3b · Pull Request #688 · allure-framework/allure-python

Conversation

@betapl3b

@betapl3b betapl3b commented Sep 26, 2022 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

Context

For now, to mark your test case as a manual, you need to use decorator @allure.label("ALLURE_MANUAL", True). This thing is not intuitive and not even stated in documentation.

This PR is adding @allure.manual decorator which is replacing this usage.

Checklist

  • Sign Allure CLA
  • Provide unit tests

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
Сафин Альберт Адилевич added 2 commits September 26, 2022 12:36
Сафин Альберт Адилевич
Added a decorator to simplify ALLURE_MANUAL=True label usage. Added t…
bf89844 
…ests for "allure.manual" decorator
Сафин Альберт Адилевич
Fixed E302 blank lines
6d6000c
@CLAassistant

CLAassistant commented Sep 26, 2022 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@skhomuti
skhomuti self-requested a review September 27, 2022 15:23
@skhomuti

skhomuti commented Sep 27, 2022

Copy link
Copy Markdown
Contributor

@betapl3b thanks!
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
skhomuti
skhomuti suggested changes Sep 27, 2022
View reviewed changes
Comment thread allure-python-commons/src/_allure.py Resolved

Uh oh!

There was an error while loading. Please reload this page.
Сафин Альберт Адилевич
removed is_manual argument from manual decorator
4b075f8
skhomuti
skhomuti reviewed Sep 28, 2022
View reviewed changes
Comment thread allure-python-commons/src/_allure.py Resolved

Uh oh!

There was an error while loading. Please reload this page.
Сафин Альберт Адилевич and others added 3 commits September 28, 2022 12:40
Сафин Альберт Адилевич
allure.manual function replaced with decorator using Dynamic.label
f126e00
@skhomuti
Make manual non-callable
af8cdc6
@skhomuti
Add dynamic manual label
39bd4d3
skhomuti
skhomuti approved these changes Oct 6, 2022
View reviewed changes
@skhomuti
skhomuti merged commit cfb6395 into allure-framework:master Oct 6, 2022
beckerGil pushed a commit to beckerGil/allure-python that referenced this pull request Apr 10, 2023
@betapl3b @skhomuti @GilBecker-Anaplan
Add @allure.manual decorator to simplify ALLURE_MANUAL=True label set…
2ee800d 
…ting (allure-framework#688)

* Added a decorator to simplify ALLURE_MANUAL=True label usage. Added tests for "allure.manual" decorator
* Add dynamic `manual` label

Co-authored-by: skhomuti <[email protected]>
@betapl3b
betapl3b deleted the allure-manual-decorator branch February 16, 2024 12:54
IvanBuruyane pushed a commit to IvanBuruyane/allure-python that referenced this pull request Mar 19, 2024
@betapl3b @skhomuti
Add @allure.manual decorator to simplify ALLURE_MANUAL=True label set…
e5b028d 
…ting (allure-framework#688)

* Added a decorator to simplify ALLURE_MANUAL=True label usage. Added tests for "allure.manual" decorator
* Add dynamic `manual` label

Co-authored-by: skhomuti <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
@skhomuti skhomuti skhomuti approved these changes
Reviewers whose approvals may not affect merge requirements

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

3 participants

@betapl3b @CLAassistant @skhomuti
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.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.