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 test properties to TC metadata #88

Closed
wants to merge 8 commits into from
Closed

Conversation

elv1s42
Copy link
Contributor

@elv1s42 elv1s42 commented Oct 25, 2023

This PR implements #87

@elv1s42
Copy link
Contributor Author

elv1s42 commented Oct 25, 2023

I created a local package from the source code and tested it, new metadata displayed properly and accessible via TeamCity API

@NikolayPianikov
Copy link
Member

@elv1s42 Thanks for the PR. Can you send a screenshot with an example of how it is displayed in TeamCity?

@elv1s42
Copy link
Contributor Author

elv1s42 commented Feb 8, 2024

Closing the PR for now as more code changes required. Currently, TC does not receive TestCase and TestFixture properties

@elv1s42 elv1s42 closed this Feb 8, 2024
@elv1s42
Copy link
Contributor Author

elv1s42 commented Feb 9, 2024

@elv1s42 Thanks for the PR. Can you send a screenshot with an example of how it is displayed in TeamCity?

@NikolayPianikov , sorry, I missed the question.
The metadata is diaplayed under the test node in TC:
image

It's quite useful to see the default NUnit attribute values (like Description) as well as some custom values that you can add. For example, test metadata like logs and screenshots is displayed nicely too (there is no need to locate the files in build artefacts tab).

The issue that I am dealing with right now is that in some cases the metadata is not attached to the test. It happens for the metadata that is passed together with [TestCase] and [TestFixture] attributes. I want to deal with this issue first and then creata a new PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants