Skip to content

Release v7 #444

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 90 commits into
base: main
Choose a base branch
from
Draft

Release v7 #444

wants to merge 90 commits into from

Conversation

jkoenig134
Copy link
Member

@jkoenig134 jkoenig134 commented Mar 5, 2025

Readiness checklist

  • I added/updated tests.
  • I ensured that the PR title is good enough for the changelog.
  • I labeled the PR.
  • I self-reviewed the PR.

jkoenig134 and others added 2 commits March 4, 2025 09:15
* refactor: make title optional in FileDTO

* refactor: update expander

* refactor: update FileMapper

* fix: linter

---------

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
* refactor: remove template from relationship in transport

* refactor: remove template from relationship in runtime logic

* fix: update more tests

* fix: revocation test

* fix: query relationships

* fix: remove strange check

* fix: template usage

* fix: remove ignore

---------

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
@jkoenig134 jkoenig134 added the breaking-change A breaking change label Mar 5, 2025
Copy link

codecov bot commented Mar 5, 2025

Codecov Report

Attention: Patch coverage is 81.12676% with 67 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...teAttribute/CreateAttributeRequestItemProcessor.ts 44.44% 20 Missing ⚠️
packages/runtime/src/dataViews/DataViewExpander.ts 26.92% 19 Missing ⚠️
...rship/TransferFileOwnershipRequestItemProcessor.ts 50.00% 13 Missing ⚠️
...ion/src/modules/attributes/AttributesController.ts 94.00% 6 Missing ⚠️
packages/runtime/src/modules/RequestModule.ts 76.92% 3 Missing ⚠️
...time/src/useCases/transport/files/UploadOwnFile.ts 57.14% 3 Missing ⚠️
.../modules/relationships/local/CachedRelationship.ts 75.00% 3 Missing ⚠️
Files with missing lines Coverage Δ
packages/app-runtime/src/AppStringProcessor.ts 78.57% <100.00%> (-0.80%) ⬇️
...nsumption/src/consumption/ConsumptionController.ts 96.77% <ø> (-0.06%) ⬇️
...nsumption/src/consumption/ConsumptionCoreErrors.ts 80.50% <100.00%> (+0.19%) ⬆️
...on/src/modules/attributes/local/QueryTranslator.ts 86.13% <ø> (+19.66%) ⬆️
packages/consumption/src/modules/requests/index.ts 100.00% <ø> (ø)
...eAttribute/ProposeAttributeRequestItemProcessor.ts 91.64% <100.00%> (+0.35%) ⬆️
...readAttribute/ReadAttributeRequestItemProcessor.ts 92.20% <100.00%> (+0.17%) ⬆️
...areAttribute/ShareAttributeRequestItemProcessor.ts 94.16% <100.00%> (+0.36%) ⬆️
...ssors/utility/validateAttributeMatchesWithQuery.ts 63.15% <ø> (-0.92%) ⬇️
...ckages/content/src/attributes/AbstractAttribute.ts 100.00% <100.00%> (ø)
... and 50 more

... and 3 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

britsta and others added 25 commits March 5, 2025 10:53
* feat: make title of AuthenticationRequestItem mandatory

* feat: make title mandatory in DVOs as well

* test: set title of AuthenticationRequestItems in tests

* feat: make AuthenticationRequestItem really mandatory

* test: set title of AuthenticationRequestItem in tests

* fix: failing test due to mandatory title of AuthenticationRequestItem

* refactor: use standardized value formatting within messages test

---------

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
* feat: update content

* feat: update consumption

* feat: update runtime

* chore: schemas

* chore: remove time frame tests

* fix: tests

* chore: remove *Current methods

* chore: update decider module

---------

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
* feat: add valiadation of tags on attribute creation

* chore: fix tests with invalid tags

* feat: check for tags when accepting attribute requests

* chore: remove duplicate test

* chore: styling

* chore: fix tags in tests

* chore: fix tags in tests

* chore: PR comments

* chore: fix eslint

* chore: pr comments

* chore: fix tests

* chore: pr comments

* chore: improve tag validation function

* chore: improve tag validation function

* chore: make tag seperator a constant on the attributes controller

* refactor: massively simplify validateTags method

* refactor: use string instead of RegEx for expected error messages

* feat: incorporate review comments

* refactor: remove variable declaration for values used once

* feat: add customTagPrefix to IdentityAttributeQuery and IQLQuery tests

* feat: add function for validating tags of AttributeQueries as well

* feat: apply tag validation to outgoing RequestItems and queries as well

* refactor: more general error message fitting invalid tags of AttributeQueries as well

* test: tag validation of outgoing CreateAttributeRequestItem

* test: tag validation of outgoing ReadAttributeRequestItem

* test: tag validation of outgoing ProposeAttributeRequestItem

* refactor: use more descriptive test values for invalid tags

* feat: apply tag validation to outgoing ShareAttributeRequestItems

* test: tag validation of outgoing ShareAttributeRequestItem

* fix: failing tests due to unchanged test values

* refactor: use singular in test names when testing a single invalid tag

* refactor: be more precise within test names

* feat: incorporate review comments

---------

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Co-authored-by: Julian König <[email protected]>
Co-authored-by: Britta Stallknecht <[email protected]>
Co-authored-by: Britta Stallknecht <[email protected]>
* fix: failing test due to added tag validation

* fix: failing test due to removal of validTo

* fix: failing test due to missing custom tag prefixes
* fix: remove onlyValid from AttributesController

* fix: remove params

* chore: rm function
* refactor: use standardized value formatting within messages test

* fix: failing messages test due to refactoring

* refactor: use standardized test value formatting for subject and body of Mail

* refactor: remove Petra Pan, Tina Turner and Martina Mustermann as test values

* refactor: use standardized value formatting within messages test

* fix: failing messages test due to refactoring

* refactor: use standardized test value formatting for subject and body of Mail

* refactor: remove Petra Pan, Tina Turner and Martina Mustermann as test values

* feat: add custom tag prefixes

* refactor: remove Herbert Müller from test values

* refactor: remove Hugo Becker from test values

* fix: wrong houseNo used

* fix: failing MessageContent test

* fix: failing MessageDVO test

---------

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
* fix: add x+%+ to tags

* fix: add x+%+ to tags

* fix: add x+%+ to tags
Milena-Czierlinski and others added 30 commits May 6, 2025 15:26
* chore: add todos

* feat: validate in uploadOwnFile; wip

* refactor: rename validateTagsOfAttribute and add validateTagsForType

* feat: validateTagsForType in uploadOwnFile

* refactor: use validateTagsForType in CanCreateRepositoryAttribute

* feat: improve code in uploadOwnFile use case

* feat: add validation in TransferFileOwnershipRequestItemProcessor

* test: use custom tags

---------

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
* refactor: remove truncatedReference from DTOs

* refactor: update mappers

* test: update tests

* chore: an
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
* chore: remove deprecated `@type`

* chore: remove test

* chore: remove deprecated event

* chore: remove deprecated facade methods

* chore: remove test
* fix: make tag validation less breaking

* tests: change tag prefix

* chore: update tests

* ci: trigger pipelines

* chore: use length

* chore: wording

* chore: add test
* refactor: do not process nmshd links anymore

* test: consolidate URL scheme tests and simplify valid URL handling

* fix: remove url scheme at another place

* ci: trigger pipeline

* chore: wording

* chore: wording
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking-change A breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants