Contributing

This page provides a guide for developers wishing to contribute to Sphinx-Needs.

Bugs, Features and PRs

For bug reports and well-described technical feature requests, please use our issue tracker:
https://github.com/useblocks/sphinx-needs/issues

For feature ideas and questions, please use our discussion board:
https://github.com/useblocks/sphinx-needs/discussions

If you have already created a PR, you can send it in. Our CI workflow will check (test and code styles) and a maintainer will perform a review, before we can merge it. Your PR should conform with the following rules:

  • A meaningful description or link, which describes the change

  • The changed code (for sure :) )

  • Test cases for the change (important!)

  • Updated documentation, if behavior gets changed or new options/directives are introduced.

  • Update of docs/changelog.rst.

  • If this is your first PR, feel free to add your name in the AUTHORS file.

Installing Dependencies

To develop Sphinx-Needs it can be installed, with development extras, into an existing Python environment using pip:

pip install sphinx-needs[test,benchmark,docs]

or using uv to install the dependencies into an isolated environment:

uv sync

To run the formatting and linting suite, pre-commit is used:

pre-commit install  # to auto-run on every commit
pre-commit run --all-files  # to run manually

To run testing and documentation building, tox is used:

tox -av  # to see all environments

Note, it is recommended to also install the tox-uv plugin, which will use uv to create isolated environments faster, and to use pyenv to manage multiple Python versions.

Build docs

To build the Sphinx-Needs documentation stored under /docs, run:

# Build HTML pages with the furo theme,
# and first remove all old build files
CLEAN=true tox -e docs-furo

or to build with a different builder:

# Check links in the documentation
CLEAN=true BUILDER=linkcheck tox -e docs-furo

Running Tests

You can either run the tests directly using pytest, in an existing environment:

pytest tests/

Or use tox (recommended):

tox -e py39

Note some tests use syrupy to perform snapshot testing. These snapshots can be updated by running:

pytest tests/ --snapshot-update

Hint

Please be sure to have the dependencies of the official documentation also installed:

pip install -r docs/requirements.txt

Running JS Testcases with PyTest

Setup Cypress Locally

  • Install Node JS on your computer and ensure it can be accessed through the CMD.

  • Install Cypress using the npm package manager by running npm install cypress. Visit this link for more information on how to install Cypress.

  • Verify if Cypress is installed correctly and is executable by running: npx cypress verify. Get out this page for more information about Cypress commandline.

  • If everything is successful then we can use Cypress.

Enable Cypress Test in Python Test Files

Under the js_test folder, you can save your Cypress JS test files (files should end with: *.cy.js). For each Cypress JS test file, you will need to write the Cypress JS test cases in the file. You can read more from the Cypress Docs. You can also check the tests/js_test/sn-collapse-button.cy.js file as reference.

In your Python test files, you must mark every JS related test case with the marker - jstest and you must include the spec_pattern key-value pair as part of the test_app fixture parameter. Also, you need to pass the test_server fixture to your test function for it to use the automated HTTP test server. For example, your test case could look like this:

# tests/test_sn_collapse_button.py

import pytest


@pytest.mark.jstest
@pytest.mark.parametrize(
    "test_app",
    [
        {
            "buildername": "html",
            "srcdir": "doc_test/variant_doc",
            "tags": ["tag_a"],
            "spec_pattern": "js_test/js-test-sn-collapse-button.cy.js"
        }
    ],
    indirect=True,
)
def test_collapse_button_in_docs(test_app, test_server):
    ...

Note

The spec_pattern key is required to ensure Cypress locates your test files or folder. Visit this link for more info on how to set the spec_pattern.

After you set the spec_pattern key-value pair as part of the test_app fixture parameter, you can call app.test_js() in your Python test case to run a JS test for the spec_pattern you provided. For example, you can use app.test_js() like below:

# tests/test_sn_collapse_button.py

import pytest


@pytest.mark.jstest
@pytest.mark.parametrize(
    "test_app",
    [
        {
            "buildername": "html",
            "srcdir": "doc_test/variant_doc",
            "tags": ["tag_a"],
            "spec_pattern": "js_test/js-test-sn-collapse-button.cy.js"
        }
    ],
    indirect=True,
)
def test_collapse_button_in_docs(test_app, test_server):
    """Check if the Sphinx-Needs collapse button works in the provided documentation source."""
    app = test_app
    app.build()

    # Call `app.test_js()` to run the JS test for a particular specPattern
    js_test_result = app.test_js()

    # Check the return code and stdout
    assert js_test_result["returncode"] == 0
    assert "All specs passed!" in js_test_result["stdout"].decode("utf-8")

Note

app.test_js() will return a dictionary object containing the returncode, stdout, and stderr. Example:

return {
    "returncode": 0,
    "stdout": "Test passed string",
    "stderr": "Errors encountered,
}

You can run the make test-js command to check all JS testcases.

Note

The http_server process invoked by the make test-js command may not terminate properly in some instances. Kindly check your system’s monitoring app to end the process if not terminated automatically.

Benchmarks

Sphinx-Needs own documentation is used for creating a benchmark for each PR. If the runtime takes 10% longer as the previous ones, the benchmark test will fail.

Benchmark test cases are available under tests/benchmarks.

The results for each PR/commit get added to a chart, which is available under http://useblocks.com/sphinx-needs/bench/index.html.

The benchmark data is stored on the benchmarks branch, which is also used by github-pages as source.

Publishing a new release

There is a release pipeline installed for the CI.

This gets triggered automatically, if a tag is created and pushed. The tag must follow the format: [0-9].[0-9]+.[0-9]. Otherwise the release jobs won’t trigger.

The release jobs will build the source and wheel distribution and try to upload them.

Structure of the extension’s logic

The following is an outline of the build events which this extension adds to the Sphinx build process:

  1. After configuration has been initialised (config-inited event):

    • Register additional directives, directive options and warnings (load_config)

    • Check configuration consistency (check_configuration)

  2. Before reading changed documents (env-before-read-docs event):

    • Initialise BuildEnvironment variables (prepare_env)

    • Register services (prepare_env)

    • Register functions (prepare_env)

    • Initialise default extra options (prepare_env)

    • Initialise extra link types (prepare_env)

    • Ensure default configurations are set (prepare_env)

    • Start process timing, if enabled (prepare_env)

    • Load external needs (load_external_needs)

  3. For all removed and changed documents (env-purge-doc event):

    • Remove all cached need items that originate from the document (purge_needs)

  4. For changed documents (doctree-read event, priority 880 of transforms)

    • Determine and add data on parent sections and needs(analyse_need_locations)

    • Remove Need nodes marked as hidden (analyse_need_locations)

  5. When building in parallel mode (env-merge-info event), merge BuildEnvironment data (merge_data)

  6. After all documents have been read and transformed (env-updated event) (NOTE these are skipped for needs builder)

    • Copy vendored JS libraries (with CSS) to build folder (install_lib_static_files)

    • Generate permalink file (install_permalink_file)

    • Copy vendored CSS files to build folder (install_styles_static_files)

  7. Note, the BuildEnvironment is cached at this point, only if any documents were updated.

  8. For all changed documents, or their dependants (doctree-resolved)

    • Replace all Needextract nodes with a list of the collected Need (process_creator)

    • Remove all Need nodes, if needs_include_needs is True (process_need_nodes)

    • Call dynamic functions, set as values on the need data items and replace them with their return values (process_need_nodes -> resolve_dynamic_values)

    • Replace needs data variant values (process_need_nodes -> resolve_variants_options)

    • Check for dead links (process_need_nodes -> check_links)

    • Generate back links (process_need_nodes -> create_back_links)

    • Process constraints, for each Need node (process_need_nodes -> process_constraints)

    • Perform all modifications on need data items, due to Needextend nodes (process_need_nodes -> extend_needs_data)

    • Format each Need node to give the desired visual output (process_need_nodes -> print_need_nodes)

    • Process all other need specific nodes, replacing them with the desired visual output (process_creator)

  9. At the end of the build (build-finished event)

    • Call all user defined need data checks, a.k.a needs_warnings (process_warnings)

    • Write the needs.json to the output folder, if needs_build_json = True (build_needs_json)

    • Write the needs.json per ID to the output folder, if needs_build_json_per_id = True (build_needs_id_json)

    • Write all UML files to the output folder, if needs_build_needumls = True (build_needumls_pumls)

    • Print process timing, if needs_debug_measurement = True (process_timing)

Maintainers

Daniel Woste <daniel@useblocks.com>

Chris Sewell <chrisj_sewell@hotmail.com>

Contributors

Marco Heinemann <marco@useblocks.com>

Trevor Lovett <trevor.lovett@gmail.com>

Magnus Lööf <magnus.loof@gmail.com>

Harri Kaimio

Anders Thuné

Daniel Eades <danieleades@hotmail.com>

Philip Partsch <philip.partsch@googlemail.com>

David Le Nir <david.lenir.e@thalesdigital.io>

Baran Barış Yıldızlı <arisbbyil@gmail.com>

Roberto Rötting <roberto.roetting@gmail.com>

Nirmal Sasidharan <nirmal.sasidharan@de.bosch.com>

Jacob Allen <jacob.allen@etas.com>

Jörg Kreuzberger <j.kreuzberger@procitec.de>

Duodu Randy <duodurandy19@gmail.com>

Christian Wappler <chri.wapp@gmail.com>

Simon Leiner <simon@leiner.me>