Allure step in thread (fixes #563)

[nigredon1991]

Test for example with thread:

from concurrent.futures import ThreadPoolExecutor

import allure
import pytest
@allure.step("lala")
def lala(x=1):
    print(f"run {x}")
    return 1


@allure.step("test")
def test_lala():
    with allure.step("Start in thread"):
        with ThreadPoolExecutor(max_workers=2) as executor:
            f_result = executor.map(lala, [1, 2])
        print(list(f_result))

Without this request allure report looks like this:

And with this request report looks like this:

In the request, the generation of the report has been corrected if threads are created during the launch process. Each step now remembers its parent, even if it was launched in a thread.

Context

Fix for #563

Checklist

• Sign Allure CLA
• Provide unit tests

[CLAassistant]

All committers have signed the CLA.

[rad96]

@nigredon1991
Hello
Why tests not executed for this Pull Request?
Also, can you write some tests for this functionality?
(I asked because maintainers will ask too i think)

[nigredon1991]

@nigredon1991
Hello
Why tests not executed for this Pull Request?
Also, can you write some tests for this functionality?
(I asked because maintainers will ask too i think)

If I understand correctly, then to launch I need an approval from maintainers.

[rad96]

@nigredon1991
Hello
Why tests not executed for this Pull Request?
Also, can you write some tests for this functionality?
(I asked because maintainers will ask too i think)

If I understand correctly, then to launch I need an approval from maintainers.

Check for example this pull request:
#560 (comment)
You can see section with test actions executed:

Do you have actions enabled for your repo?

[nigredon1991]

@nigredon1991
Hello
Why tests not executed for this Pull Request?
Also, can you write some tests for this functionality?
(I asked because maintainers will ask too i think)

If I understand correctly, then to launch I need an approval from maintainers.

Check for example this pull request:
#560 (comment)
You can see section with test actions executed:

Do you have actions enabled for your repo?

Thank you. Otherwise I didn’t think of it.

[rad96]

Looks like you still need write at least one test for this case
And remove draft status from PR

[nigredon1991]

Tests added.

[rad96]

So now we need to summon maintainer :)

@sseliverstov Can you please review this PR?

[sseliverstov]

Great job! Unfortunately, I can review it right now. I going to do it next week. Feel free to summon me again.

[rad96]

Great job! Unfortunately, I can review it right now. I going to do it next week. Feel free to summon me again.

@sseliverstov so... new week now
Can you review this PR for us? :)

[bugsnub]

Someone, please review and merge this. It is a long waiting PR/Change.

[nigredon1991]

@sseliverstov, can you review this PR? :)

[nigredon1991]

Maybe I need to do something else so that can merge?

[rad96]

@sseliverstov any feedback?

[nigredon1991]

Maybe I need to do something else so that can merge?