Skip to content

Add nonResponsibilities to BoundedContext (#372) - #383

Draft
stefan-ka wants to merge 2 commits into
masterfrom
feature/372-not-responsible-for
Draft

stefan-ka wants to merge 2 commits into
masterfrom
feature/372-not-responsible-for

Conversation

@stefan-ka

@stefan-ka stefan-ka commented Oct 19, 2026 •

Copy link
Copy Markdown
Member

Closes #372

Changes

  • Grammar: new optional nonResponsibilities attribute on BoundedContext (comma-separated list of strings, = optional), symmetric to responsibilities:
    BoundedContext TicketSales implements FareSales {
      responsibilities = "Selling fare products", "Refunds"
      nonResponsibilities = "Gate validation", "Support cases"
    }
    
  • Formatter: nonResponsibilities starts on a new line, like responsibilities.
  • Merge Bounded Contexts refactoring: merges both lists (without duplicates) and drops entries that are a responsibility of the merged context.
  • Tests for parsing and merging.

The open question from the issue (optionally pointing to the context that is responsible) is not addressed; plain strings can be extended later without breaking models.

Build fix (separate commit)

jacocoTestReport failed on JDK 17 with Gradle 7.4.2 (Could not inspect the Groovy system, java.util.prefs not opened). Fixed via org.gradle.jvmargs in gradle.properties. The other feature branches are based on this branch, so they include the fix.

Not included

@codecov

codecov Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.8%. Comparing base (17d16de) to head (3d31b17).

Additional details and impacted files
@@           Coverage Diff            @@
##             master    #383   +/-   ##
========================================
  Coverage      91.8%   91.8%           
- Complexity     2230    2234    +4     
========================================
  Files           211     211           
  Lines          5942    5949    +7     
  Branches        807     808    +1     
========================================
+ Hits           5458    5466    +8     
  Misses          230     230           
+ Partials        254     253    -1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Allows to state explicitly what a Bounded Context is not responsible for,
symmetric to the existing responsibilities attribute. The Merge Bounded
Contexts refactoring merges the lists and drops entries that are a
responsibility of the merged context.
Uses a noun like the existing responsibilities attribute and the other
Bounded Context attributes, so both lists read consistently.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Add notResponsibleFor to BoundedContext

1 participant