proxmox-mirrors/mirror_frr
1
0
Fork 0
mirror of https://git.proxmox.com/git/mirror_frr synced 2025-06-04 02:53:32 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

doc: Require unified config for all new topotests

Browse Source 
Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
This commit is contained in:
Donatas Abraitis 2024-10-21 09:10:13 +03:00
parent 5d5da525b9
commit d155d62785
1 changed files with 8 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
doc/developer/topotests.rst
View File

@ -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: