Skip to content

Conversation

wurzelkuchen
Copy link

No description provided.

Copy link
Author

@wurzelkuchen wurzelkuchen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I checked the quick start guide and added some comments, please take care @RobertAvemarg


* as JUnit test via IDE
* `mvn clean test` for all tests
* `mvn clean test -Dtest=<NameOfTestClass>` for specific tests
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please add how to run a single test method

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done


### Generate Reports

* `mvn allure:serve` to generate and show the report
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe a line to run tests AND generate a report (just for copy and past purposes)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done


## Useful Annotations

### Test JUnit5
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe "Define Test"?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what do you mean?


`@NeodymiumTest` - on test methods

### Test JUnit4
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe "Define Test"?


Methods can have multiple `@DataSet()` annotations

* `@DataSet("<dataSetId>")` - limit method to data set with defined id
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That won't work, you have to use @DataSet(id="<dataSetId>")

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you are right. thanks!


{{< TODO >}}check if the info boxes are useful and move them to a good position{{< /TODO >}}

{{% note title="most common browser properties" %}}
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice look and feel, but still maybe a bit much for the sake of this quick start, also you have an additional chapter about configs below.

tests are annotated, so the Neodymium default browser is used. To execute all tests with every configured browser,
we annotate the AbstractTest class with `@Browser("<browserId>")` for each browser. Individual test methods within a
class can also be run with different browsers by annotating them with `@Browser("<browserId>")` or all browsers can be
suppressed using `@SuppressBrowsers`.
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove the SuppressBrowsers part

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

import com.xceptance.neodymium.util.Neodymium;
import org.junit.jupiter.api.BeforeEach;
@Browser
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove the default browser?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

configurations, `dev-*.properties` files can be used to override or add values. It is crucial to exclude these files
from version control to avoid unintended consequences for CI/CD and commiting secrets. Temporary property files can be
created during test execution using
`ConfigFactory.setProperty(Neodymium.TEMPORARY_CONFIG_FILE_PROPERTY_NAME, "file:" + fileLocation);` within a
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the temp config file name is mostly for unit tests isn't it?

recording is added to the Allure report. Otherwise, it's only added if appendAllRecordingsToAllureReport is set to
`true`.

## Accessibility
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Accessibility Testing

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

changed it

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.

3 participants