From 171f1c01d11da64da757bf51db96842431985099 Mon Sep 17 00:00:00 2001 From: eisenhower444 Date: Tue, 12 Dec 2017 12:29:36 +0000 Subject: [PATCH 1/2] REFACTOR: Removed DebitAndCredit to the annual statement --- .../models/in/EpayeAnnualStatement.scala | 13 ++++-------- .../hmrc/epayeapi/models/in/EpayeReads.scala | 1 - .../models/in/EpayeTotalsResponse.scala | 4 ++-- .../models/out/AnnualStatementJson.scala | 12 +++++------ .../epayeapi/models/out/SummaryJson.scala | 4 ++-- .../connectors/EpayeConnectorSpec.scala | 12 +++++------ .../hmrc/epayeapi/models/JsonFixtures.scala | 8 ++++---- .../models/out/AnnualStatementJsonSpec.scala | 20 +++++++++---------- 8 files changed, 34 insertions(+), 40 deletions(-) diff --git a/app/uk/gov/hmrc/epayeapi/models/in/EpayeAnnualStatement.scala b/app/uk/gov/hmrc/epayeapi/models/in/EpayeAnnualStatement.scala index e1ff07e..8aace34 100644 --- a/app/uk/gov/hmrc/epayeapi/models/in/EpayeAnnualStatement.scala +++ b/app/uk/gov/hmrc/epayeapi/models/in/EpayeAnnualStatement.scala @@ -19,11 +19,6 @@ package uk.gov.hmrc.epayeapi.models.in import org.joda.time.LocalDate import uk.gov.hmrc.epayeapi.models.TaxYear -case class DebitAndCredit( - debit: BigDecimal = 0, - credit: BigDecimal = 0 -) - case class Cleared( payment: BigDecimal = 0, credit: BigDecimal = 0 @@ -32,9 +27,9 @@ case class Cleared( case class LineItem( taxYear: TaxYear, taxMonth: Option[EpayeTaxMonth], - charges: DebitAndCredit, + charges: BigDecimal, cleared: Cleared, - balance: DebitAndCredit, + balance: BigDecimal, dueDate: LocalDate, isSpecified: Boolean = false, codeText: Option[String] = None, @@ -42,9 +37,9 @@ case class LineItem( ) case class AnnualTotal( - charges: DebitAndCredit, + charges: BigDecimal, cleared: Cleared, - balance: DebitAndCredit + balance: BigDecimal ) case class AnnualStatementTable(lineItems: Seq[LineItem], totals: AnnualTotal) diff --git a/app/uk/gov/hmrc/epayeapi/models/in/EpayeReads.scala b/app/uk/gov/hmrc/epayeapi/models/in/EpayeReads.scala index 7d05f9f..a8cdbea 100644 --- a/app/uk/gov/hmrc/epayeapi/models/in/EpayeReads.scala +++ b/app/uk/gov/hmrc/epayeapi/models/in/EpayeReads.scala @@ -24,7 +24,6 @@ trait EpayeReads { implicit lazy val taxYearReads: Reads[TaxYear] = reads[TaxYear] implicit lazy val epayeTaxMonthReads: Reads[EpayeTaxMonth] = reads[EpayeTaxMonth] - implicit lazy val debitAndCreditReads: Reads[DebitAndCredit] = reads[DebitAndCredit] implicit lazy val clearedReads: Reads[Cleared] = reads[Cleared] implicit lazy val annualTotalReads: Reads[AnnualTotal] = reads[AnnualTotal] implicit lazy val lineItemReads: Reads[LineItem] = reads[LineItem] diff --git a/app/uk/gov/hmrc/epayeapi/models/in/EpayeTotalsResponse.scala b/app/uk/gov/hmrc/epayeapi/models/in/EpayeTotalsResponse.scala index 2a9da12..3c3e630 100644 --- a/app/uk/gov/hmrc/epayeapi/models/in/EpayeTotalsResponse.scala +++ b/app/uk/gov/hmrc/epayeapi/models/in/EpayeTotalsResponse.scala @@ -16,8 +16,8 @@ package uk.gov.hmrc.epayeapi.models.in -case class EpayeTotals(balance: DebitAndCredit) +case class EpayeTotals(balance: BigDecimal) case class EpayeTotalsItem(totals: EpayeTotals) case class EpayeTotalsResponse(rti: EpayeTotalsItem, nonRti: EpayeTotalsItem) { - def overall: BigDecimal = rti.totals.balance.debit + nonRti.totals.balance.debit + def overall: BigDecimal = rti.totals.balance + nonRti.totals.balance } \ No newline at end of file diff --git a/app/uk/gov/hmrc/epayeapi/models/out/AnnualStatementJson.scala b/app/uk/gov/hmrc/epayeapi/models/out/AnnualStatementJson.scala index 49c6a97..326b041 100644 --- a/app/uk/gov/hmrc/epayeapi/models/out/AnnualStatementJson.scala +++ b/app/uk/gov/hmrc/epayeapi/models/out/AnnualStatementJson.scala @@ -38,10 +38,10 @@ object NonRtiChargesJson { code <- lineItem.codeText } yield NonRtiChargesJson( code = code, - amount = lineItem.charges.debit, + amount = lineItem.charges, clearedByCredits = lineItem.cleared.credit, clearedByPayments = lineItem.cleared.payment, - balance = lineItem.balance.debit, + balance = lineItem.balance, dueDate = lineItem.dueDate ) } @@ -66,10 +66,10 @@ object EarlierYearUpdateJson { .find(_.itemType.contains("eyu")) .map { lineItem => EarlierYearUpdateJson( - lineItem.charges.debit, + lineItem.charges, lineItem.cleared.credit, lineItem.cleared.payment, - lineItem.balance.debit, + lineItem.balance, lineItem.dueDate ) } @@ -100,10 +100,10 @@ object MonthlyChargesJson { taxMonth = TaxMonth(taxYear, epayeTaxMonth.month) } yield MonthlyChargesJson( taxMonth = taxMonth, - amount = lineItem.charges.debit, + amount = lineItem.charges, clearedByCredits = lineItem.cleared.credit, clearedByPayments = lineItem.cleared.payment, - balance = lineItem.balance.debit, + balance = lineItem.balance, dueDate = lineItem.dueDate, isSpecified = lineItem.isSpecified, _links = SelfLink(Link.monthlyStatementLink(apiBaseUrl, empRef, taxYear, taxMonth)) diff --git a/app/uk/gov/hmrc/epayeapi/models/out/SummaryJson.scala b/app/uk/gov/hmrc/epayeapi/models/out/SummaryJson.scala index 250b37f..6575fc1 100644 --- a/app/uk/gov/hmrc/epayeapi/models/out/SummaryJson.scala +++ b/app/uk/gov/hmrc/epayeapi/models/out/SummaryJson.scala @@ -40,8 +40,8 @@ object SummaryJson { OutstandingCharges( total.overall, Breakdown( - rti = total.rti.totals.balance.debit, - nonRti = total.nonRti.totals.balance.debit + rti = total.rti.totals.balance, + nonRti = total.nonRti.totals.balance ) ), SummaryLinks(apiBaseUrl, empRef) diff --git a/test/uk/gov/hmrc/epayeapi/connectors/EpayeConnectorSpec.scala b/test/uk/gov/hmrc/epayeapi/connectors/EpayeConnectorSpec.scala index c5065ce..11b5f60 100644 --- a/test/uk/gov/hmrc/epayeapi/connectors/EpayeConnectorSpec.scala +++ b/test/uk/gov/hmrc/epayeapi/connectors/EpayeConnectorSpec.scala @@ -80,8 +80,8 @@ class EpayeConnectorSpec extends UnitSpec with MockitoSugar with ScalaFutures { Await.result(connector.getTotal(empRef, hc), 2.seconds) shouldBe EpayeSuccess( EpayeTotalsResponse( - EpayeTotalsItem(EpayeTotals(DebitAndCredit(100, 0))), - EpayeTotalsItem(EpayeTotals(DebitAndCredit(23, 0))) + EpayeTotalsItem(EpayeTotals(100)), + EpayeTotalsItem(EpayeTotals(23)) ) ) } @@ -99,12 +99,12 @@ class EpayeConnectorSpec extends UnitSpec with MockitoSugar with ScalaFutures { EpayeSuccess( EpayeAnnualStatement( rti = AnnualStatementTable( - List(LineItem(TaxYear(2017), Some(EpayeTaxMonth(1)), DebitAndCredit(100.2, 0), Cleared(0, 0), DebitAndCredit(100.2, 0), new LocalDate(2017, 5, 22), isSpecified = false, codeText = None)), - AnnualTotal(DebitAndCredit(100.2, 0), Cleared(0, 0), DebitAndCredit(100.2, 0)) + List(LineItem(TaxYear(2017), Some(EpayeTaxMonth(1)), 100.2, Cleared(0, 0), 100.2, new LocalDate(2017, 5, 22), isSpecified = false, codeText = None)), + AnnualTotal(100.2, Cleared(0, 0), 100.2) ), nonRti = AnnualStatementTable( - List(LineItem(TaxYear(2017), None, DebitAndCredit(20.0, 0), Cleared(0, 0), DebitAndCredit(20.0, 0), new LocalDate(2018, 2, 22), false, Some("P11D_CLASS_1A_CHARGE"))), - AnnualTotal(DebitAndCredit(20.0, 0), Cleared(0, 0), DebitAndCredit(20.0, 0)) + List(LineItem(TaxYear(2017), None, 20.0, Cleared(0, 0), 20.0, new LocalDate(2018, 2, 22), false, Some("P11D_CLASS_1A_CHARGE"))), + AnnualTotal(20.0, Cleared(0, 0), 20.0) ), unallocated = None ) diff --git a/test/uk/gov/hmrc/epayeapi/models/JsonFixtures.scala b/test/uk/gov/hmrc/epayeapi/models/JsonFixtures.scala index 1e219ef..428c267 100644 --- a/test/uk/gov/hmrc/epayeapi/models/JsonFixtures.scala +++ b/test/uk/gov/hmrc/epayeapi/models/JsonFixtures.scala @@ -34,17 +34,17 @@ object JsonFixtures { rti = AnnualStatementTable( lineItems = Seq(), totals = AnnualTotal( - charges = DebitAndCredit(0, 0), + charges = 0, cleared = Cleared(0, 0), - balance = DebitAndCredit(0, 0) + balance = 0 ) ), nonRti = AnnualStatementTable( lineItems = Seq(), totals = AnnualTotal( - charges = DebitAndCredit(0, 0), + charges = 0, cleared = Cleared(0), - balance = DebitAndCredit(0, 0) + balance = 0 ) ), unallocated = None diff --git a/test/uk/gov/hmrc/epayeapi/models/out/AnnualStatementJsonSpec.scala b/test/uk/gov/hmrc/epayeapi/models/out/AnnualStatementJsonSpec.scala index d712c98..c1846e3 100644 --- a/test/uk/gov/hmrc/epayeapi/models/out/AnnualStatementJsonSpec.scala +++ b/test/uk/gov/hmrc/epayeapi/models/out/AnnualStatementJsonSpec.scala @@ -46,9 +46,9 @@ class AnnualStatementJsonSpec extends WordSpec with Matchers { "AnnualStatementJson.apply._embedded.earlierYearUpdate" should { "contain the earlier year update if it is present" in { val emptyTotals = AnnualTotal( - charges = DebitAndCredit(), + charges = 0, cleared = Cleared(), - balance = DebitAndCredit() + balance = 0 ) val epayeAnnualStatement = @@ -60,9 +60,9 @@ class AnnualStatementJsonSpec extends WordSpec with Matchers { LineItem( taxYear, None, - DebitAndCredit(100), + 100, Cleared(10, 20), - DebitAndCredit(100 - 20 - 10), + 100 - 20 - 10, dueDate, codeText = None, itemType = Some("eyu") @@ -94,9 +94,9 @@ class AnnualStatementJsonSpec extends WordSpec with Matchers { LineItem( taxYear = taxYear, taxMonth = Some(EpayeTaxMonth(taxMonth.month)), - charges = DebitAndCredit(100, 0), + charges = 100, cleared = Cleared(payment = 10, credit = 20), - balance = DebitAndCredit(100 - 30, 0), + balance = 100 - 30, dueDate = dueDate, isSpecified = true, codeText = None @@ -120,9 +120,9 @@ class AnnualStatementJsonSpec extends WordSpec with Matchers { LineItem( taxYear = taxYear, taxMonth = None, - charges = DebitAndCredit(100, 0), + charges = 100, cleared = Cleared(payment = 10, credit = 20), - balance = DebitAndCredit(100 - 30, 0), + balance = 100 - 30, dueDate = dueDate, codeText = None ) @@ -139,9 +139,9 @@ class AnnualStatementJsonSpec extends WordSpec with Matchers { LineItem( taxYear = taxYear, taxMonth = None, - charges = DebitAndCredit(100, 0), + charges = 100, cleared = Cleared(payment = 10, credit = 20), - balance = DebitAndCredit(100 - 30, 0), + balance = 100 - 30, dueDate = dueDate, codeText = Some(code) ) From 585161857ea5d351848ae35667fdb93de5cacdb6 Mon Sep 17 00:00:00 2001 From: eisenhower444 Date: Tue, 12 Dec 2017 15:13:39 +0000 Subject: [PATCH 2/2] REFACTOR: Removed Cleared from the annual statement --- .../models/in/EpayeAnnualStatement.scala | 11 +- .../hmrc/epayeapi/models/in/EpayeReads.scala | 1 - .../models/out/AnnualStatementJson.scala | 12 +-- test/common/Fixtures.scala | 102 +++++------------- .../annual-statement/annual-statement.json | 64 +++-------- .../connectors/EpayeConnectorSpec.scala | 18 ++-- .../epayeapi/controllers/GetSummarySpec.scala | 10 +- .../hmrc/epayeapi/models/JsonFixtures.scala | 6 +- .../models/out/AnnualStatementJsonSpec.scala | 15 ++- 9 files changed, 72 insertions(+), 167 deletions(-) diff --git a/app/uk/gov/hmrc/epayeapi/models/in/EpayeAnnualStatement.scala b/app/uk/gov/hmrc/epayeapi/models/in/EpayeAnnualStatement.scala index 8aace34..7dcdafc 100644 --- a/app/uk/gov/hmrc/epayeapi/models/in/EpayeAnnualStatement.scala +++ b/app/uk/gov/hmrc/epayeapi/models/in/EpayeAnnualStatement.scala @@ -19,16 +19,12 @@ package uk.gov.hmrc.epayeapi.models.in import org.joda.time.LocalDate import uk.gov.hmrc.epayeapi.models.TaxYear -case class Cleared( - payment: BigDecimal = 0, - credit: BigDecimal = 0 -) - case class LineItem( taxYear: TaxYear, taxMonth: Option[EpayeTaxMonth], charges: BigDecimal, - cleared: Cleared, + payments: BigDecimal, + credits: BigDecimal, balance: BigDecimal, dueDate: LocalDate, isSpecified: Boolean = false, @@ -38,7 +34,8 @@ case class LineItem( case class AnnualTotal( charges: BigDecimal, - cleared: Cleared, + payments: BigDecimal, + credits: BigDecimal, balance: BigDecimal ) diff --git a/app/uk/gov/hmrc/epayeapi/models/in/EpayeReads.scala b/app/uk/gov/hmrc/epayeapi/models/in/EpayeReads.scala index a8cdbea..1565808 100644 --- a/app/uk/gov/hmrc/epayeapi/models/in/EpayeReads.scala +++ b/app/uk/gov/hmrc/epayeapi/models/in/EpayeReads.scala @@ -24,7 +24,6 @@ trait EpayeReads { implicit lazy val taxYearReads: Reads[TaxYear] = reads[TaxYear] implicit lazy val epayeTaxMonthReads: Reads[EpayeTaxMonth] = reads[EpayeTaxMonth] - implicit lazy val clearedReads: Reads[Cleared] = reads[Cleared] implicit lazy val annualTotalReads: Reads[AnnualTotal] = reads[AnnualTotal] implicit lazy val lineItemReads: Reads[LineItem] = reads[LineItem] diff --git a/app/uk/gov/hmrc/epayeapi/models/out/AnnualStatementJson.scala b/app/uk/gov/hmrc/epayeapi/models/out/AnnualStatementJson.scala index 326b041..25c0492 100644 --- a/app/uk/gov/hmrc/epayeapi/models/out/AnnualStatementJson.scala +++ b/app/uk/gov/hmrc/epayeapi/models/out/AnnualStatementJson.scala @@ -39,8 +39,8 @@ object NonRtiChargesJson { } yield NonRtiChargesJson( code = code, amount = lineItem.charges, - clearedByCredits = lineItem.cleared.credit, - clearedByPayments = lineItem.cleared.payment, + clearedByCredits = lineItem.credits, + clearedByPayments = lineItem.payments, balance = lineItem.balance, dueDate = lineItem.dueDate ) @@ -67,8 +67,8 @@ object EarlierYearUpdateJson { .map { lineItem => EarlierYearUpdateJson( lineItem.charges, - lineItem.cleared.credit, - lineItem.cleared.payment, + lineItem.credits, + lineItem.payments, lineItem.balance, lineItem.dueDate ) @@ -101,8 +101,8 @@ object MonthlyChargesJson { } yield MonthlyChargesJson( taxMonth = taxMonth, amount = lineItem.charges, - clearedByCredits = lineItem.cleared.credit, - clearedByPayments = lineItem.cleared.payment, + clearedByCredits = lineItem.credits, + clearedByPayments = lineItem.payments, balance = lineItem.balance, dueDate = lineItem.dueDate, isSpecified = lineItem.isSpecified, diff --git a/test/common/Fixtures.scala b/test/common/Fixtures.scala index 9fb6b4d..2ffc397 100644 --- a/test/common/Fixtures.scala +++ b/test/common/Fixtures.scala @@ -33,19 +33,10 @@ object Fixtures { | "taxMonth": { | "month": 7 | }, - | "charges": { - | "debit": 1200, - | "credit": 0 - | }, - | "cleared": { - | "cleared": 0, - | "payment": 0, - | "credit": 0 - | }, - | "balance": { - | "debit": 1200, - | "credit": 0 - | }, + | "charges":1200, + | "payments": 0, + | "credits": 0, + | "balance": 1200, | "dueDate": "2017-11-22", | "isSpecified": false, | "itemType": "month" @@ -54,19 +45,10 @@ object Fixtures { | "taxYear": { | "yearFrom": 2017 | }, - | "charges": { - | "debit": 700, - | "credit": 0 - | }, - | "cleared": { - | "cleared": 0, - | "payment": 300, - | "credit": 200 - | }, - | "balance": { - | "debit": 200, - | "credit": 0 - | }, + | "charges":700, + | "payments": 300, + | "credits": 200, + | "balance": 200, | "dueDate": "2017-04-22", | "isSpecified": false, | "itemType": "eyu" @@ -78,37 +60,19 @@ object Fixtures { | "taxMonth": { | "month": 3 | }, - | "charges": { - | "debit": 700, - | "credit": 0 - | }, - | "cleared": { - | "cleared": 0, - | "payment": 300, - | "credit": 200 - | }, - | "balance": { - | "debit": 200, - | "credit": 0 - | }, + | "charges":700, + | "payments": 300, + | "credits": 200, + | "balance": 200, | "dueDate": "2017-07-22", | "isSpecified": true | } | ], | "totals": { - | "charges": { - | "debit": 1900, - | "credit": 0 - | }, - | "cleared": { - | "cleared": 0, - | "payment": 300, - | "credit": 200 - | }, - | "balance": { - | "debit": 1400, - | "credit": 0 - | } + | "charges":1900, + | "payments": 300, + | "credits": 200, + | "balance": 1400 | } | }, | "nonRti": { @@ -117,19 +81,10 @@ object Fixtures { | "taxYear": { | "yearFrom": 2017 | }, - | "charges": { - | "debit": 300, - | "credit": 0 - | }, - | "cleared": { - | "cleared": 100, - | "payment": 30, - | "credit": 70 - | }, - | "balance": { - | "debit": 200, - | "credit": 0 - | }, + | "charges":300, + | "payments": 30, + | "credits": 70, + | "balance": 200, | "dueDate": "2017-07-22", | "isSpecified": false, | "itemType": "1481", @@ -137,19 +92,10 @@ object Fixtures { | } | ], | "totals": { - | "charges": { - | "debit": 0, - | "credit": 0 - | }, - | "cleared": { - | "cleared": 0, - | "payment": 0, - | "credit": 0 - | }, - | "balance": { - | "debit": 0, - | "credit": 0 - | } + | "charges":0, + | "payments": 0, + | "credits": 0, + | "balance": 0 | } | }, | "unallocated": 2000 diff --git a/test/resources/epaye/annual-statement/annual-statement.json b/test/resources/epaye/annual-statement/annual-statement.json index 05317b6..a3746fd 100644 --- a/test/resources/epaye/annual-statement/annual-statement.json +++ b/test/resources/epaye/annual-statement/annual-statement.json @@ -8,35 +8,19 @@ "taxMonth": { "month": 1 }, - "charges": { - "debit": 100.2, - "credit": 0 - }, - "cleared": { - "payment": 0, - "credit": 0 - }, - "balance": { - "debit": 100.2, - "credit": 0 - }, + "charges": 100.2, + "payments": 0, + "credits": 0, + "balance": 100.2, "dueDate": "2017-05-22", "isSpecified": false } ], "totals": { - "charges": { - "debit": 100.2, - "credit": 0 - }, - "cleared": { - "payment": 0, - "credit": 0 - }, - "balance": { - "debit": 100.2, - "credit": 0 - } + "charges": 100.2, + "payments": 0, + "credits": 0, + "balance": 100.2 } }, "nonRti": { @@ -45,36 +29,20 @@ "taxYear": { "yearFrom": 2017 }, - "charges": { - "debit": 20.0, - "credit": 0 - }, - "cleared": { - "payment": 0, - "credit": 0 - }, - "balance": { - "debit": 20.0, - "credit": 0 - }, + "charges": 20.0, + "payments": 0, + "credits": 0, + "balance": 20.0, "dueDate": "2018-02-22", "isSpecified": false, "codeText": "P11D_CLASS_1A_CHARGE" } ], "totals": { - "charges": { - "debit": 20.0, - "credit": 0 - }, - "cleared": { - "payment": 0, - "credit": 0 - }, - "balance": { - "debit": 20.0, - "credit": 0 - } + "charges": 20.0, + "payments": 0, + "credits": 0, + "balance": 20.0 } } } diff --git a/test/uk/gov/hmrc/epayeapi/connectors/EpayeConnectorSpec.scala b/test/uk/gov/hmrc/epayeapi/connectors/EpayeConnectorSpec.scala index 11b5f60..3895422 100644 --- a/test/uk/gov/hmrc/epayeapi/connectors/EpayeConnectorSpec.scala +++ b/test/uk/gov/hmrc/epayeapi/connectors/EpayeConnectorSpec.scala @@ -57,18 +57,12 @@ class EpayeConnectorSpec extends UnitSpec with MockitoSugar with ScalaFutures { |{ | "rti": { | "totals": { - | "balance": { - | "debit": 100, - | "credit": 0 - | } + | "balance": 100 | } | }, | "nonRti": { | "totals": { - | "balance": { - | "debit": 23, - | "credit": 0 - | } + | "balance": 23 | } | } |} @@ -99,12 +93,12 @@ class EpayeConnectorSpec extends UnitSpec with MockitoSugar with ScalaFutures { EpayeSuccess( EpayeAnnualStatement( rti = AnnualStatementTable( - List(LineItem(TaxYear(2017), Some(EpayeTaxMonth(1)), 100.2, Cleared(0, 0), 100.2, new LocalDate(2017, 5, 22), isSpecified = false, codeText = None)), - AnnualTotal(100.2, Cleared(0, 0), 100.2) + List(LineItem(TaxYear(2017), Some(EpayeTaxMonth(1)), 100.2, 0, 0, 100.2, new LocalDate(2017, 5, 22), isSpecified = false, codeText = None)), + AnnualTotal(100.2, 0, 0, 100.2) ), nonRti = AnnualStatementTable( - List(LineItem(TaxYear(2017), None, 20.0, Cleared(0, 0), 20.0, new LocalDate(2018, 2, 22), false, Some("P11D_CLASS_1A_CHARGE"))), - AnnualTotal(20.0, Cleared(0, 0), 20.0) + List(LineItem(TaxYear(2017), None, 20.0, 0, 0, 20.0, new LocalDate(2018, 2, 22), false, Some("P11D_CLASS_1A_CHARGE"))), + AnnualTotal(20.0, 0, 0, 20.0) ), unallocated = None ) diff --git a/test/uk/gov/hmrc/epayeapi/controllers/GetSummarySpec.scala b/test/uk/gov/hmrc/epayeapi/controllers/GetSummarySpec.scala index 30844f7..c62a3d2 100644 --- a/test/uk/gov/hmrc/epayeapi/controllers/GetSummarySpec.scala +++ b/test/uk/gov/hmrc/epayeapi/controllers/GetSummarySpec.scala @@ -82,18 +82,12 @@ class GetSummarySpec extends AppSpec with BeforeAndAfterEach { |{ | "rti": { | "totals": { - | "balance": { - | "debit": 100, - | "credit": 0 - | } + | "balance": 100 | } | }, | "nonRti": { | "totals": { - | "balance": { - | "debit": 23, - | "credit": 0 - | } + | "balance": 23 | } | } |} diff --git a/test/uk/gov/hmrc/epayeapi/models/JsonFixtures.scala b/test/uk/gov/hmrc/epayeapi/models/JsonFixtures.scala index 428c267..97efbd6 100644 --- a/test/uk/gov/hmrc/epayeapi/models/JsonFixtures.scala +++ b/test/uk/gov/hmrc/epayeapi/models/JsonFixtures.scala @@ -35,7 +35,8 @@ object JsonFixtures { lineItems = Seq(), totals = AnnualTotal( charges = 0, - cleared = Cleared(0, 0), + payments = 0, + credits = 0, balance = 0 ) ), @@ -43,7 +44,8 @@ object JsonFixtures { lineItems = Seq(), totals = AnnualTotal( charges = 0, - cleared = Cleared(0), + payments = 0, + credits = 0, balance = 0 ) ), diff --git a/test/uk/gov/hmrc/epayeapi/models/out/AnnualStatementJsonSpec.scala b/test/uk/gov/hmrc/epayeapi/models/out/AnnualStatementJsonSpec.scala index c1846e3..5265371 100644 --- a/test/uk/gov/hmrc/epayeapi/models/out/AnnualStatementJsonSpec.scala +++ b/test/uk/gov/hmrc/epayeapi/models/out/AnnualStatementJsonSpec.scala @@ -47,7 +47,8 @@ class AnnualStatementJsonSpec extends WordSpec with Matchers { "contain the earlier year update if it is present" in { val emptyTotals = AnnualTotal( charges = 0, - cleared = Cleared(), + payments = 0, + credits = 0, balance = 0 ) @@ -61,7 +62,8 @@ class AnnualStatementJsonSpec extends WordSpec with Matchers { taxYear, None, 100, - Cleared(10, 20), + 10, + 20, 100 - 20 - 10, dueDate, codeText = None, @@ -95,7 +97,8 @@ class AnnualStatementJsonSpec extends WordSpec with Matchers { taxYear = taxYear, taxMonth = Some(EpayeTaxMonth(taxMonth.month)), charges = 100, - cleared = Cleared(payment = 10, credit = 20), + payments = 10, + credits = 20, balance = 100 - 30, dueDate = dueDate, isSpecified = true, @@ -121,7 +124,8 @@ class AnnualStatementJsonSpec extends WordSpec with Matchers { taxYear = taxYear, taxMonth = None, charges = 100, - cleared = Cleared(payment = 10, credit = 20), + payments = 10, + credits = 20, balance = 100 - 30, dueDate = dueDate, codeText = None @@ -140,7 +144,8 @@ class AnnualStatementJsonSpec extends WordSpec with Matchers { taxYear = taxYear, taxMonth = None, charges = 100, - cleared = Cleared(payment = 10, credit = 20), + payments = 10, + credits = 20, balance = 100 - 30, dueDate = dueDate, codeText = Some(code)