Skip to content

Commit

Permalink
Merge pull request #17172 from opensourcerouting/fix/doc_unified_config
Browse files Browse the repository at this point in the history
doc: Require unified config for all new topotests
  • Loading branch information
Jafaral authored Oct 21, 2024
2 parents 2f84a26 + d155d62 commit cf1da3e
Showing 1 changed file with 8 additions and 7 deletions.
15 changes: 8 additions & 7 deletions doc/developer/topotests.rst
Original file line number Diff line number Diff line change
Expand Up @@ -1393,21 +1393,22 @@ or using unified config (specifying which daemons to run is optional):
Requirements:

- Directory name for a new topotest must not contain hyphen (``-``) characters.
To separate words, use underscores (``_``). For example, ``tests/topotests/bgp_new_example``.
To separate words, use underscores (``_``). For example, ``tests/topotests/bgp_new_example``;
- Test code should always be declared inside functions that begin with the
``test_`` prefix. Functions beginning with different prefixes will not be run
by pytest.
by pytest;
- Configuration files and long output commands should go into separated files
inside folders named after the equipment.
inside folders named after the equipment;
- Tests must be able to run without any interaction. To make sure your test
conforms with this, run it without the :option:`-s` parameter.
conforms with this, run it without the :option:`-s` parameter;
- Use `black <https://github.com/psf/black>`_ code formatter before creating
a pull request. This ensures we have a unified code style.
a pull request. This ensures we have a unified code style;
- Mark test modules with pytest markers depending on the daemons used during the
tests (see :ref:`topotests-markers`)
tests (see :ref:`topotests-markers`);
- Always use IPv4 :rfc:`5737` (``192.0.2.0/24``, ``198.51.100.0/24``,
``203.0.113.0/24``) and IPv6 :rfc:`3849` (``2001:db8::/32``) ranges reserved
for documentation.
for documentation;
- Use unified config (``frr.conf``) for all new [tests](#writing-tests).

Tips:

Expand Down

0 comments on commit cf1da3e

Please sign in to comment.