LOBSTER Report - no schema#524
Open
ThomasDebobenVispiron wants to merge 44 commits into
Open
Conversation
For future use we want to support a yaml file as config file fore the lobster_report tool. During implementation and testing phase the known .conf file still can be used.
For future use we want to support a yaml file as config file fore the lobster_report tool. During implementation and testing phase the known .conf file still can be used.
For future use we want to support a yaml file as config file fore the lobster_report tool. During implementation and testing phase the known .conf file still can be used.
…oftware-engineering/lobster into vtt/lobster_report_no_kind
For future use we want to support a more flexible configuration. Therefor we remove the schema (requrement, implementation, code) from the lobster files. Version without schema is 5. All items are now from base Item class. Remove schema from .lobster files with version 5. Only write Item's to get remove Requirement, Implementation and Acticity in the future. For transition period the old versions are still supported.
Remove schema from .lobster files with version 5. Only write Item's to get remove Requirement, Implementation and Acticity in the future. For transition period the old versions are still supported.
fix pylint
Remove schema from .lobster files with version 5. Only write Item's to get remove Requirement, Implementation and Acticity in the future. For transition period the old versions are still supported.
fix pylint
…re-engineering/lobster into vtt/lobster_report_v2
fix pylint fix bazel integration
ThomasDebobenVispiron
force-pushed
the
vtt/lobster_report_v2
branch
from
c624f5f to
2a74d83
Compare
add missing files
exclude yamale files for integration test diff
fix pylint fix bazel integration exclude yamale files for integration test diff
integration tests: adjust expected output
…re-engineering/lobster into vtt/lobster_report_v2
integration tests: adjust expected output
# Conflicts: # tests_system/lobster_codebeamer/lobster_codebeamer_test_runner.py
mugdhadhole1
requested changes
Feb 19, 2026
mugdhadhole1
left a comment
mugdhadhole1
left a comment
Contributor
There was a problem hiding this comment.
Since both old and new code currently run and the old one will be removed later, it will be very helpful if a comment exists marking where the old code exists.
mugdhadhole1
requested changes
Feb 19, 2026
Contributor
|
I'm attaching an example where I attempted to generate the HTML report. |
kedarnn
reviewed
Mar 11, 2026
adjust PR findings
adjust PR findings
adjust to new port handling of LobsterCodebeamerAsserter
adjust PR findings
adjust PR findings
…re-engineering/lobster into vtt/lobster_report_v2
mugdhadhole1
requested changes
Mar 20, 2026
kedarnn
requested changes
Mar 27, 2026
Contributor
There was a problem hiding this comment.
Why is this folder added at root level?
Comment on lines
+81
to
+82
| The tool supports three configuration attributes: | ||
| `output_file`, `codebeamer_url` and `files`. |
Contributor
There was a problem hiding this comment.
For the deprecated functionality the user still need to give kind. So I think over here we need to have documentations for both the usages.
SurajBDeore
reviewed
Mar 30, 2026
SurajBDeore
reviewed
Mar 30, 2026
SurajBDeore
reviewed
Mar 30, 2026
| @@ -0,0 +1,15 @@ | |||
| Requirements: !LevelDefinition | |||
Contributor
There was a problem hiding this comment.
I could not find any references to this file in tests, docs, or build targets.
SurajBDeore
reviewed
Mar 30, 2026
SurajBDeore
reviewed
Mar 30, 2026
Resolve merge conflicts.
Fixed Pylint issues that were introduced by the previous merge commit. Also improved one docstring.
phiwuu
requested changes
May 7, 2026
| def test_lobster_read_missing_versio_key(self, mock_file_open, mock_isfile): | ||
| with self.assertRaises(LOBSTER_Error): | ||
| lobster_read(self.mh, self.filename, self.level, self.items, self.source_info) | ||
| self.mh.error.assert_called_with(ANY, "required top-level key version not present") |
Member
There was a problem hiding this comment.
- Method
errorhas noassert_called_withmember (no-member) - Undefined variable
ANY(undefined-variable)
(detected by pylint)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Remove schema from .lobster files with version 5.
Only write Item's to get remove Requirement, Implementation and Acticity in the future.
For transition period the old versions are still supported.
Introduce new yaml config file for LOBSTER report.