Skip to content

Allow extra properties & missing properties in testcases#421

Merged
MaximilianSoerenPollak merged 3 commits intoeclipse-score:mainfrom
MaximilianSoerenPollak:MSP_allow_incomplete_testlink_attributes
Feb 27, 2026
Merged

Allow extra properties & missing properties in testcases#421
MaximilianSoerenPollak merged 3 commits intoeclipse-score:mainfrom
MaximilianSoerenPollak:MSP_allow_incomplete_testlink_attributes

Conversation

@MaximilianSoerenPollak
Copy link
Contributor

@MaximilianSoerenPollak MaximilianSoerenPollak commented Feb 25, 2026

📌 Description

This now allows our testlinker to parse more scenarios.
Scenarios that now should no longer error:

  • Extra properties we do not have in Testlinks
  • Not all properties Provided

If not all properties are provided testlinker will not create a testlink but instead will create a log output, similar to when all properties or Fully/Partiallyverified are missing

Closes #413

🚨 Impact Analysis

  • This change does not violate any tool requirements and is covered by existing tool requirements
  • This change does not violate any design decisions
  • Otherwise I have created a ticket for new tool qualification

✅ Checklist

  • Added/updated documentation for new or changed features
  • Added/updated tests to cover the changes
  • Followed project coding standards and guidelines

@github-actions
Copy link

github-actions bot commented Feb 25, 2026

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run //src:license-check

Status: ⚠️ Needs Review

Click to expand output
[License Check Output]
Extracting Bazel installation...
Starting local Bazel server (8.3.0) and connecting to it...
INFO: Invocation ID: 34718175-bfc5-4dfc-a6bd-f870d9d05c26
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
    currently loading: src
Loading: 0 packages loaded
    currently loading: src
Analyzing: target //src:license-check (1 packages loaded)
Analyzing: target //src:license-check (1 packages loaded, 0 targets configured)
Analyzing: target //src:license-check (1 packages loaded, 0 targets configured)

Analyzing: target //src:license-check (66 packages loaded, 9 targets configured)

Analyzing: target //src:license-check (70 packages loaded, 9 targets configured)

Analyzing: target //src:license-check (89 packages loaded, 153 targets configured)

Analyzing: target //src:license-check (128 packages loaded, 1672 targets configured)

Analyzing: target //src:license-check (129 packages loaded, 2146 targets configured)

Analyzing: target //src:license-check (140 packages loaded, 2576 targets configured)

Analyzing: target //src:license-check (140 packages loaded, 2576 targets configured)

Analyzing: target //src:license-check (140 packages loaded, 2576 targets configured)

Analyzing: target //src:license-check (144 packages loaded, 4589 targets configured)

INFO: Analyzed target //src:license-check (145 packages loaded, 4715 targets configured).
[12 / 16] JavaToolchainCompileClasses external/rules_java+/toolchains/platformclasspath_classes; 0s disk-cache, processwrapper-sandbox ... (2 actions running)
[14 / 16] JavaToolchainCompileBootClasspath external/rules_java+/toolchains/platformclasspath.jar; 0s disk-cache, processwrapper-sandbox
[15 / 16] Building src/license.check.license_check.jar (); 0s disk-cache, multiplex-worker
INFO: Found 1 target...
Target //src:license.check.license_check up-to-date:
  bazel-bin/src/license.check.license_check
  bazel-bin/src/license.check.license_check.jar
INFO: Elapsed time: 23.798s, Critical Path: 2.60s
INFO: 16 processes: 12 internal, 3 processwrapper-sandbox, 1 worker.
INFO: Build completed successfully, 16 total actions
INFO: Running command line: bazel-bin/src/license.check.license_check src/formatted.txt <args omitted>
usage: org.eclipse.dash.licenses.cli.Main [-batch <int>] [-cd <url>]
       [-confidence <int>] [-ef <url>] [-excludeSources <sources>] [-help] [-lic
       <url>] [-project <shortname>] [-repo <url>] [-review] [-summary <file>]
       [-timeout <seconds>] [-token <token>]

@github-actions
Copy link

The created documentation from the pull request is available at: docu-html

The pre-commit already takes care of everything the formating workflow
has tested.
Therefore this can be deleted.
@MaximilianSoerenPollak MaximilianSoerenPollak force-pushed the MSP_allow_incomplete_testlink_attributes branch from e36abab to 25c49dd Compare February 27, 2026 08:49
@MaximilianSoerenPollak MaximilianSoerenPollak merged commit c120767 into eclipse-score:main Feb 27, 2026
13 checks passed
# and self.result is not None
# and self.TestType is not None
# and self.DerivationTechnique is not None
# ):
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove commented code


def build_test_needs_from_files(
app: Sphinx, env: BuildEnvironment, xml_paths: list[Path]
app: Sphinx, enw_: BuildEnvironment, xml_paths: list[Path]
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
app: Sphinx, enw_: BuildEnvironment, xml_paths: list[Path]
app: Sphinx, env_: BuildEnvironment, xml_paths: list[Path]

@MaximilianSoerenPollak
Copy link
Contributor Author

@a-zw I will do a cleanup PR and will add those things in there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

docs_as_code testlinker ignore extra properties

3 participants