addressing comments in code review

.gitignore

@@ -1,4 +1,3 @@
-.vscode
 *.class
 .factorypath
 target/

src/test/java/api/loans/DeclareLostAPITests.java

@@ -459,16 +459,15 @@ public void cannotDeclareItemLostTwice() {
   }
 
   @Test
-  public void shouldNotChargeOverdueOnCheckinWhenItemLostAndRefundFeePeriodPassed() {
+  public void shouldNotChargeOverdueFeesDuringCheckInWhenItemDeclaredLostAndRefundFeePeriodHasPassed() {
     ItemResource item = itemsFixture.basedUponSmallAngryPlanet();
 
     IndividualResource overduePolicy = overdueFinePoliciesFixture.facultyStandard();
     IndividualResource lostItemPolicy = lostItemFeePoliciesFixture.ageToLostAfterOneWeek();
 
     policiesActivation.use(PoliciesToActivate.builder()
       .lostItemPolicy(lostItemPolicy)
-      .overduePolicy(overduePolicy)
-    );
+      .overduePolicy(overduePolicy));
 
     IndividualResource loan = checkOutFixture
       .checkOutByBarcode(item, usersFixture.jessica());

src/test/java/api/loans/OverrideRenewByBarcodeTests.java

@@ -713,7 +713,7 @@ public void renewalNoticeIsSentWhenPolicyDefinesRenewalNoticeConfiguration() {
   }
 
   @Test
-  public void shouldNotChargeOverdueOnRenewalWhenItemAgedToLostAndRefundFeePeriodPast() {
+  public void shouldNotChargeOverdueFeesDuringRenewalWhenItemHasAgedToLostAndRefundFeePeriodHasPassed() {
 
     IndividualResource overDueFinePolicy = overdueFinePoliciesFixture.facultyStandard();
     IndividualResource lostItemPolicy = lostItemFeePoliciesFixture.ageToLostAfterOneWeek();
@@ -733,7 +733,7 @@ public void shouldNotChargeOverdueOnRenewalWhenItemAgedToLostAndRefundFeePeriodP
     assertThat(renewedLoan, hasNoOverdueFine());
   }
 
-  public void shouldNotChargeOverdueOnRenewalWhenItemDeclaredLostAndRefundFeePeriodPassed() {
+  public void shouldNotChargeOverdueFeesDuringRenewalWhenItemIsDeclaredLostAndRefundFeePeriodHasPassed() {
     ItemResource item = itemsFixture.basedUponSmallAngryPlanet();
     UserResource user = usersFixture.jessica();
 
@@ -745,8 +745,7 @@ public void shouldNotChargeOverdueOnRenewalWhenItemDeclaredLostAndRefundFeePerio
       .overduePolicy(overduePolicy)
     );
 
-    IndividualResource loan = checkOutFixture
-      .checkOutByBarcode(item, user);
+    IndividualResource loan = checkOutFixture.checkOutByBarcode(item, user);
 
     // advance system time by five weeks to accrue fines before declared lost
     final DateTime declareLostDate = now(UTC).plusWeeks(5);

src/test/java/api/loans/agetolost/ScheduledAgeToLostFeeChargingApiTest.java

@@ -324,7 +324,7 @@ public void shouldCloseLoanWhenNoFeesToCharge() {
   }
 
   @Test
-  public void shouldNotChargeOverdueOnCheckinWhenAgedToLostAndRefundFeePeriodPassed() {
+  public void shouldNotChargeOverdueFeesDuringCheckInWhenItemHasAgedToLostAndRefundFeePeriodHasPassed() {
     // for a loan that charges both lost fees and overdue fines
     // where the policies are set such that:
     // 1. the item charges overdue fees on checkin
@@ -348,7 +348,6 @@ public void shouldNotChargeOverdueOnCheckinWhenAgedToLostAndRefundFeePeriodPasse
     final DateTime checkInDate = now(UTC).plusWeeks(9);
     mockClockManagerToReturnFixedDateTime(checkInDate);
     checkInFixture.checkInByBarcode(result.getItem(), checkInDate);
-
     assertThat(loansFixture.getLoanById(loanId), hasNoOverdueFine());
 
   }

src/test/java/api/support/fixtures/AgeToLostFixture.java

@@ -92,14 +92,12 @@ public AgeToLostResult createAgedToLostLoan(UnaryOperator<HoldingBuilder> holdin
     return ageToLostResult;
   }
 
-  public AgeToLostResult createLoanAgeToLostAndChargeFeesWithOverdues(IndividualResource lostPolicy, IndividualResource overduePolicy) {
-
+  public AgeToLostResult createLoanAgeToLostAndChargeFeesWithOverdues(IndividualResource lostPolicy, IndividualResource overduePolicy) {   
       return createLoanAgeToLostAndChargeFees(UnaryOperator.identity(), 
         PoliciesToActivate.builder()
           .lostItemPolicy(lostPolicy)
           .overduePolicy(overduePolicy)
       );
-
   }
 
   public AgeToLostResult createLoanAgeToLostAndChargeFees(LostItemFeePolicyBuilder builder) {