Skip to content

Commit

Permalink
Replace builders with only node argument with constructors
Browse files Browse the repository at this point in the history 
RISDEV-0000
  • Loading branch information
@malte-laukoetter
malte-laukoetter committed Jan 10, 2025
1 parent de6be8a commit 0149829
Show file tree
Hide file tree
Showing 21 changed files with 2,090 additions and 2,598 deletions.
74 changes: 34 additions & 40 deletions .../digitalservice/ris/norms/adapter/input/restapi/controller/ProprietaryControllerTest.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -192,29 +192,26 @@ void updatesProprietarySuccess() throws Exception {
final String eli = "eli/bund/bgbl-1/1990/s2954/2022-12-19/1/deu/regelungstext-1";
final LocalDate date = LocalDate.parse("1990-01-01");

final Proprietary proprietary = Proprietary
.builder()
.node(
XmlMapper.toNode(
"""
<akn:proprietary xmlns:akn="http://Inhaltsdaten.LegalDocML.de/1.7.1/" eId="meta-1_proprietary-1" GUID="952262d3-de92-4c1d-a06d-95aa94f5f21c" source="attributsemantik-noch-undefiniert">
<ris:legalDocML.de_metadaten xmlns:ris="http://MetadatenRIS.LegalDocML.de/1.7.1/">
<ris:fna start="1990-01-01" end="1994-12-31">new-fna</ris:fna>
<ris:art start="1990-01-01" end="1994-12-31">new-art</ris:art>
<ris:typ start="1990-01-01" end="1994-12-31">new-typ</ris:typ>
<ris:subtyp start="1990-01-01" end="1994-12-31">new-subtyp</ris:subtyp>
<ris:bezeichnungInVorlage start="1990-01-01" end="1994-12-31">new-bezeichnungInVorlage</ris:bezeichnungInVorlage>
<ris:artDerNorm start="1990-01-01" end="1994-12-31">SN,ÄN,ÜN</ris:artDerNorm>
<ris:normgeber start="1990-01-01" end="1994-12-31">DEU</ris:normgeber>
<ris:beschliessendesOrgan start="1990-01-01" end="1994-12-31" qualifizierteMehrheit="true">Bundestag</ris:beschliessendesOrgan>
<ris:organisationsEinheit start="1990-01-01" end="1994-12-31">Andere Organisationseinheit</ris:organisationsEinheit>
</ris:legalDocML.de_metadaten>
</akn:proprietary>
"""
)
final Proprietary proprietary = new Proprietary(
XmlMapper.toNode(
"""
<akn:proprietary xmlns:akn="http://Inhaltsdaten.LegalDocML.de/1.7.1/" eId="meta-1_proprietary-1" GUID="952262d3-de92-4c1d-a06d-95aa94f5f21c" source="attributsemantik-noch-undefiniert">
<ris:legalDocML.de_metadaten xmlns:ris="http://MetadatenRIS.LegalDocML.de/1.7.1/">
<ris:fna start="1990-01-01" end="1994-12-31">new-fna</ris:fna>
<ris:art start="1990-01-01" end="1994-12-31">new-art</ris:art>
<ris:typ start="1990-01-01" end="1994-12-31">new-typ</ris:typ>
<ris:subtyp start="1990-01-01" end="1994-12-31">new-subtyp</ris:subtyp>
<ris:bezeichnungInVorlage start="1990-01-01" end="1994-12-31">new-bezeichnungInVorlage</ris:bezeichnungInVorlage>
<ris:artDerNorm start="1990-01-01" end="1994-12-31">SN,ÄN,ÜN</ris:artDerNorm>
<ris:normgeber start="1990-01-01" end="1994-12-31">DEU</ris:normgeber>
<ris:beschliessendesOrgan start="1990-01-01" end="1994-12-31" qualifizierteMehrheit="true">Bundestag</ris:beschliessendesOrgan>
<ris:organisationsEinheit start="1990-01-01" end="1994-12-31">Andere Organisationseinheit</ris:organisationsEinheit>
</ris:legalDocML.de_metadaten>
</akn:proprietary>
"""
)
.build();
);

when(updateProprietaryFrameFromNormUseCase.updateProprietaryFrameFromNorm(any()))
.thenReturn(proprietary);
Expand Down Expand Up @@ -425,25 +422,22 @@ void updatesProprietarySuccess() throws Exception {
var eid = "hauptteil-1_abschnitt-0_art-1";
final LocalDate date = LocalDate.parse("1990-01-01");

final Proprietary proprietary = Proprietary
.builder()
.node(
XmlMapper.toNode(
"""
<akn:proprietary xmlns:akn="http://Inhaltsdaten.LegalDocML.de/1.7.1/" eId="meta-1_proprietary-1" GUID="952262d3-de92-4c1d-a06d-95aa94f5f21c" source="attributsemantik-noch-undefiniert">
<ris:legalDocML.de_metadaten xmlns:ris="http://MetadatenRIS.LegalDocML.de/1.7.1/">
<ris:artDerNorm start="1990-01-01" end="1994-12-31">SN,ÄN,ÜN</ris:artDerNorm>
<ris:einzelelement href="#hauptteil-1_abschnitt-0_art-1">
<ris:artDerNorm start="1990-01-01" end="1994-12-31">SN</ris:artDerNorm>
<ris:artDerNorm start="1995-01-01" end="2000-12-31">ÄN</ris:artDerNorm>
<ris:artDerNorm start="2001-01-01">ÜN</ris:artDerNorm>
</ris:einzelelement>
</ris:legalDocML.de_metadaten>
</akn:proprietary>
"""
)
final Proprietary proprietary = new Proprietary(
XmlMapper.toNode(
"""
<akn:proprietary xmlns:akn="http://Inhaltsdaten.LegalDocML.de/1.7.1/" eId="meta-1_proprietary-1" GUID="952262d3-de92-4c1d-a06d-95aa94f5f21c" source="attributsemantik-noch-undefiniert">
<ris:legalDocML.de_metadaten xmlns:ris="http://MetadatenRIS.LegalDocML.de/1.7.1/">
<ris:artDerNorm start="1990-01-01" end="1994-12-31">SN,ÄN,ÜN</ris:artDerNorm>
<ris:einzelelement href="#hauptteil-1_abschnitt-0_art-1">
<ris:artDerNorm start="1990-01-01" end="1994-12-31">SN</ris:artDerNorm>
<ris:artDerNorm start="1995-01-01" end="2000-12-31">ÄN</ris:artDerNorm>
<ris:artDerNorm start="2001-01-01">ÜN</ris:artDerNorm>
</ris:einzelelement>
</ris:legalDocML.de_metadaten>
</akn:proprietary>
"""
)
.build();
);

when(
updateProprietarySingleElementFromNormUseCase.updateProprietarySingleElementFromNorm(any())
Expand Down
69 changes: 33 additions & 36 deletions ...bund/digitalservice/ris/norms/adapter/input/restapi/mapper/ArticleResponseMapperTest.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -12,43 +12,40 @@ class ArticleResponseMapperTest {
@Test
void itMapsNormArticleCorrectly() {
// Given
var article1 = Article
.builder()
.node(
XmlMapper.toNode(
"""
<akn:article xmlns:akn="http://Inhaltsdaten.LegalDocML.de/1.7.1/" eId="hauptteil-1_art-1" GUID="cdbfc728-a070-42d9-ba2f-357945afef06" period="#meta-1_geltzeiten-1_geltungszeitgr-1" refersTo="hauptaenderung">
<akn:num eId="hauptteil-1_art-1_bezeichnung-1" GUID="25a9acae-7463-4490-bc3f-8258b629d7e9">Artikel 1</akn:num>
<akn:heading eId="hauptteil-1_art-1_überschrift-1" GUID="92827aa8-8118-4207-9f93-589345f0bab6">Änderung des Vereinsgesetzes</akn:heading>
<!-- Absatz (1) -->
<akn:paragraph eId="hauptteil-1_art-1_abs-1" GUID="48ead358-f901-41d3-a135-e372d5ef97b1">
<akn:num eId="hauptteil-1_art-1_abs-1_bezeichnung-1" GUID="ef3a32d2-df20-4978-914b-cd6288872208">
</akn:num>
<akn:list eId="hauptteil-1_art-1_abs-1_untergl-1" GUID="41675622-ed62-46e3-869f-94d99908b010">
<akn:intro eId="hauptteil-1_art-1_abs-1_untergl-1_intro-1" GUID="5d6fc824-7926-43b4-b243-b0096da183f9">
<akn:p eId="hauptteil-1_art-1_abs-1_untergl-1_intro-1_text-1" GUID="04879ca1-994b-4eb2-b59b-032e528d9ce5"> Das <akn:affectedDocument eId="hauptteil-1_art-1_abs-1_untergl-1_intro-1_text-1_bezugsdoc-1" GUID="88b3b9f3-e4a8-49c6-9320-b5b42150bcc5"
href="eli/bund/bgbl-1/1964/s593/1964-08-05/1/deu/regelungstext-1">Vereinsgesetz vom 5. August 1964 (BGBl. I S. 593), das zuletzt durch … geändert worden ist</akn:affectedDocument>, wird wie folgt geändert:</akn:p>
</akn:intro>
<!-- Nummer 1 wurde entfernt -->
<!-- Nummer 2 -->
<akn:point eId="hauptteil-1_art-1_abs-1_untergl-1_listenelem-2" GUID="b5fa1383-f26a-4904-a638-f48711fbcf2d">
<akn:num eId="hauptteil-1_art-1_abs-1_untergl-1_listenelem-2_bezeichnung-1" GUID="6f0f92b3-1a51-440c-9137-b44ab9d990ac">2.</akn:num>
<akn:content eId="hauptteil-1_art-1_abs-1_untergl-1_listenelem-2_inhalt-1" GUID="6cb14ab5-3a7f-45f4-9e85-00ac2fb0fe5e">
<akn:p eId="hauptteil-1_art-1_abs-1_untergl-1_listenelem-2_inhalt-1_text-1" GUID="db3fbe0f-b758-4cc4-b528-a723cacad94a">
<akn:mod eId="hauptteil-1_art-1_abs-1_untergl-1_listenelem-2_inhalt-1_text-1_ändbefehl-1" GUID="148c2f06-6e33-4af8-9f4a-3da67c888510" refersTo="aenderungsbefehl-ersetzen">In <akn:ref eId="hauptteil-1_art-1_abs-1_untergl-1_listenelem-2_inhalt-1_text-1_ändbefehl-1_ref-1"
GUID="61d3036a-d7d9-4fa5-b181-c3345caa3206" href="eli/bund/bgbl-1/1964/s593/1964-08-05/1/deu/regelungstext-1/hauptteil-1_art-20_abs-1_untergl-1_listenelem-2_inhalt-1_text-1/0-0.xml">§ 20 Absatz 1 Satz 2</akn:ref> wird die Angabe <akn:quotedText
eId="hauptteil-1_art-1_abs-1_untergl-1_listenelem-2_inhalt-1_text-1_ändbefehl-1_quottext-1" GUID="694459c4-ef66-4f87-bb78-a332054a2216" startQuote="„" endQuote="“">§ 9 Abs. 1 Satz 2, Abs. 2</akn:quotedText> durch die Wörter <akn:quotedText
eId="hauptteil-1_art-1_abs-1_untergl-1_listenelem-2_inhalt-1_text-1_ändbefehl-1_quottext-2" GUID="dd25bdb6-4ef4-4ef5-808c-27579b6ae196" startQuote="„" endQuote="“">§ 9 Absatz 1 Satz 2, Absatz 2 oder 3</akn:quotedText> ersetzt.</akn:mod>
</akn:p>
</akn:content>
</akn:point>
</akn:list>
</akn:paragraph>
</akn:article>
"""
)
var article1 = new Article(
XmlMapper.toNode(
"""
<akn:article xmlns:akn="http://Inhaltsdaten.LegalDocML.de/1.7.1/" eId="hauptteil-1_art-1" GUID="cdbfc728-a070-42d9-ba2f-357945afef06" period="#meta-1_geltzeiten-1_geltungszeitgr-1" refersTo="hauptaenderung">
<akn:num eId="hauptteil-1_art-1_bezeichnung-1" GUID="25a9acae-7463-4490-bc3f-8258b629d7e9">Artikel 1</akn:num>
<akn:heading eId="hauptteil-1_art-1_überschrift-1" GUID="92827aa8-8118-4207-9f93-589345f0bab6">Änderung des Vereinsgesetzes</akn:heading>
<!-- Absatz (1) -->
<akn:paragraph eId="hauptteil-1_art-1_abs-1" GUID="48ead358-f901-41d3-a135-e372d5ef97b1">
<akn:num eId="hauptteil-1_art-1_abs-1_bezeichnung-1" GUID="ef3a32d2-df20-4978-914b-cd6288872208">
</akn:num>
<akn:list eId="hauptteil-1_art-1_abs-1_untergl-1" GUID="41675622-ed62-46e3-869f-94d99908b010">
<akn:intro eId="hauptteil-1_art-1_abs-1_untergl-1_intro-1" GUID="5d6fc824-7926-43b4-b243-b0096da183f9">
<akn:p eId="hauptteil-1_art-1_abs-1_untergl-1_intro-1_text-1" GUID="04879ca1-994b-4eb2-b59b-032e528d9ce5"> Das <akn:affectedDocument eId="hauptteil-1_art-1_abs-1_untergl-1_intro-1_text-1_bezugsdoc-1" GUID="88b3b9f3-e4a8-49c6-9320-b5b42150bcc5"
href="eli/bund/bgbl-1/1964/s593/1964-08-05/1/deu/regelungstext-1">Vereinsgesetz vom 5. August 1964 (BGBl. I S. 593), das zuletzt durch … geändert worden ist</akn:affectedDocument>, wird wie folgt geändert:</akn:p>
</akn:intro>
<!-- Nummer 1 wurde entfernt -->
<!-- Nummer 2 -->
<akn:point eId="hauptteil-1_art-1_abs-1_untergl-1_listenelem-2" GUID="b5fa1383-f26a-4904-a638-f48711fbcf2d">
<akn:num eId="hauptteil-1_art-1_abs-1_untergl-1_listenelem-2_bezeichnung-1" GUID="6f0f92b3-1a51-440c-9137-b44ab9d990ac">2.</akn:num>
<akn:content eId="hauptteil-1_art-1_abs-1_untergl-1_listenelem-2_inhalt-1" GUID="6cb14ab5-3a7f-45f4-9e85-00ac2fb0fe5e">
<akn:p eId="hauptteil-1_art-1_abs-1_untergl-1_listenelem-2_inhalt-1_text-1" GUID="db3fbe0f-b758-4cc4-b528-a723cacad94a">
<akn:mod eId="hauptteil-1_art-1_abs-1_untergl-1_listenelem-2_inhalt-1_text-1_ändbefehl-1" GUID="148c2f06-6e33-4af8-9f4a-3da67c888510" refersTo="aenderungsbefehl-ersetzen">In <akn:ref eId="hauptteil-1_art-1_abs-1_untergl-1_listenelem-2_inhalt-1_text-1_ändbefehl-1_ref-1"
GUID="61d3036a-d7d9-4fa5-b181-c3345caa3206" href="eli/bund/bgbl-1/1964/s593/1964-08-05/1/deu/regelungstext-1/hauptteil-1_art-20_abs-1_untergl-1_listenelem-2_inhalt-1_text-1/0-0.xml">§ 20 Absatz 1 Satz 2</akn:ref> wird die Angabe <akn:quotedText
eId="hauptteil-1_art-1_abs-1_untergl-1_listenelem-2_inhalt-1_text-1_ändbefehl-1_quottext-1" GUID="694459c4-ef66-4f87-bb78-a332054a2216" startQuote="„" endQuote="“">§ 9 Abs. 1 Satz 2, Abs. 2</akn:quotedText> durch die Wörter <akn:quotedText
eId="hauptteil-1_art-1_abs-1_untergl-1_listenelem-2_inhalt-1_text-1_ändbefehl-1_quottext-2" GUID="dd25bdb6-4ef4-4ef5-808c-27579b6ae196" startQuote="„" endQuote="“">§ 9 Absatz 1 Satz 2, Absatz 2 oder 3</akn:quotedText> ersetzt.</akn:mod>
</akn:p>
</akn:content>
</akn:point>
</akn:list>
</akn:paragraph>
</akn:article>
"""
)
.build();
);

// When
final ArticleResponseSchema resultArticle = ArticleResponseMapper.fromNormArticle(article1);
Expand Down
68 changes: 31 additions & 37 deletions .../digitalservice/ris/norms/adapter/input/restapi/mapper/ProprietaryResponseMapperTest.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -11,27 +11,24 @@ class ProprietaryResponseMapperTest {

@Test
void convertsProprietaryToResponseSchema() {
final Proprietary proprietary = Proprietary
.builder()
.node(
XmlMapper.toNode(
"""
<akn:proprietary xmlns:akn="http://Inhaltsdaten.LegalDocML.de/1.7.1/" eId="meta-1_proprietary-1" GUID="952262d3-de92-4c1d-a06d-95aa94f5f21c" source="attributsemantik-noch-undefiniert">
<meta:legalDocML.de_metadaten xmlns:meta="http://Metadaten.LegalDocML.de/1.7.1/">
<meta:typ>gesetz</meta:typ>
<meta:fna>000-00-0</meta:fna>
<meta:fassung>verkuendungsfassung</meta:fassung>
</meta:legalDocML.de_metadaten>
<ris:legalDocML.de_metadaten xmlns:ris="http://MetadatenRIS.LegalDocML.de/1.7.1/">
<ris:fna start="1990-01-01" end="1994-12-31">111-11-1</ris:fna>
<ris:fna start="1995-01-01" end="2000-12-31">222-22-2</ris:fna>
<ris:fna start="2001-01-01">333-33-3</ris:fna>
</ris:legalDocML.de_metadaten>
</akn:proprietary>
"""
)
final Proprietary proprietary = new Proprietary(
XmlMapper.toNode(
"""
<akn:proprietary xmlns:akn="http://Inhaltsdaten.LegalDocML.de/1.7.1/" eId="meta-1_proprietary-1" GUID="952262d3-de92-4c1d-a06d-95aa94f5f21c" source="attributsemantik-noch-undefiniert">
<meta:legalDocML.de_metadaten xmlns:meta="http://Metadaten.LegalDocML.de/1.7.1/">
<meta:typ>gesetz</meta:typ>
<meta:fna>000-00-0</meta:fna>
<meta:fassung>verkuendungsfassung</meta:fassung>
</meta:legalDocML.de_metadaten>
<ris:legalDocML.de_metadaten xmlns:ris="http://MetadatenRIS.LegalDocML.de/1.7.1/">
<ris:fna start="1990-01-01" end="1994-12-31">111-11-1</ris:fna>
<ris:fna start="1995-01-01" end="2000-12-31">222-22-2</ris:fna>
<ris:fna start="2001-01-01">333-33-3</ris:fna>
</ris:legalDocML.de_metadaten>
</akn:proprietary>
"""
)
.build();
);

// When
var responseSchema = ProprietaryResponseMapper.fromProprietary(
Expand All @@ -45,24 +42,21 @@ void convertsProprietaryToResponseSchema() {

@Test
void convertsProprietaryEinzelelementToResponseSchema() {
final Proprietary proprietary = Proprietary
.builder()
.node(
XmlMapper.toNode(
"""
<akn:proprietary xmlns:akn="http://Inhaltsdaten.LegalDocML.de/1.7.1/" eId="meta-1_proprietary-1" GUID="952262d3-de92-4c1d-a06d-95aa94f5f21c" source="attributsemantik-noch-undefiniert">
<ris:legalDocML.de_metadaten xmlns:ris="http://MetadatenRIS.LegalDocML.de/1.7.1/">
<ris:einzelelement href="#hauptteil-1_abschnitt-0_art-1">
<ris:artDerNorm start="1990-01-01" end="1994-12-31">SN</ris:artDerNorm>
<ris:artDerNorm start="1995-01-01" end="2000-12-31">ÄN</ris:artDerNorm>
<ris:artDerNorm start="2001-01-01">ÜN</ris:artDerNorm>
</ris:einzelelement>
</ris:legalDocML.de_metadaten>
</akn:proprietary>
"""
)
final Proprietary proprietary = new Proprietary(
XmlMapper.toNode(
"""
<akn:proprietary xmlns:akn="http://Inhaltsdaten.LegalDocML.de/1.7.1/" eId="meta-1_proprietary-1" GUID="952262d3-de92-4c1d-a06d-95aa94f5f21c" source="attributsemantik-noch-undefiniert">
<ris:legalDocML.de_metadaten xmlns:ris="http://MetadatenRIS.LegalDocML.de/1.7.1/">
<ris:einzelelement href="#hauptteil-1_abschnitt-0_art-1">
<ris:artDerNorm start="1990-01-01" end="1994-12-31">SN</ris:artDerNorm>
<ris:artDerNorm start="1995-01-01" end="2000-12-31">ÄN</ris:artDerNorm>
<ris:artDerNorm start="2001-01-01">ÜN</ris:artDerNorm>
</ris:einzelelement>
</ris:legalDocML.de_metadaten>
</akn:proprietary>
"""
)
.build();
);

// When
var responseSchema = ProprietaryResponseMapper.fromProprietarySingleElement(
Expand Down
Loading

0 comments on commit 0149829

Please sign in to comment.