Add unit tests and update some missing properties and data types

- Removed create prepayment - method performed by banktransactions endpoint
- Add attachment array to Manual Journal
- Add Edition ENUM to Organisation
Overpayment
- Change remainingCredit property changed from String to Double
- Add attachment array
Payment
- Change isReconciled property from String to boolean
- Add attachment array to Prepayment
- Add attachment array to Purchase Order
- Add attachments and validationerrors array to Receipt
- Add attachment array to Repeating Invoice
Unit tests added
- CreditNotes
- Employees
- ExpenseClaims
- Invoices
- Items
- Journals
- LinkedTransactions
- ManualJournals
- Organisation
- Overpayments
- PaymentServices
- Payments
- Prepayments
- PurchaseOrders
- Receipts
- RepeatingInvoices
- TrackingCategories
- Users

Author: SidneyAllen

src/main/java/com/xero/api/client/AccountingApi.java

@@ -1627,36 +1627,6 @@ public PaymentServices createPaymentService(PaymentServices paymentServices) thr
         }
     }
   /**
-    * Allows you to create prepayments
-    * <p><b>200</b> - Success - return response of type Prepayments array for newly created Prepayment
-    * <p><b>400</b> - A failed request due to validation error
-    * @param prepayments The prepayments parameter
-    * @return Prepayments
-    * @throws IOException if an error occurs while attempting to invoke the API
-    **/
-    public Prepayments createPrepayment(Prepayments prepayments) throws IOException {
-        try {
-            String strBody = null;
-            Map<String, String> params = null;
-            String correctPath = "/Prepayments";
-            UriBuilder uriBuilder = UriBuilder.fromUri(apiClient.getBasePath() + correctPath);
-            String url = uriBuilder.build().toString();
-
-            
-            strBody = apiClient.getObjectMapper().writeValueAsString(prepayments);
-
-            String response = this.DATA(url, strBody, params, "PUT");
-
-            TypeReference<Prepayments> typeRef = new TypeReference<Prepayments>() {};
-            return apiClient.getObjectMapper().readValue(response, typeRef);           
-
-        } catch (IOException e) {
-            throw xeroExceptionHandler.handleBadRequest(e.getMessage());
-        } catch (XeroApiException e) {
-            throw xeroExceptionHandler.handleBadRequest(e.getMessage(), e.getResponseCode(),JSONUtils.isJSONValid(e.getMessage()));
-        }
-    }
-  /**
     * Allows you to create an Allocation for prepayments
     * <p><b>200</b> - Success - return response of type Allocations array of Allocation for all Prepayment
     * <p><b>400</b> - A failed request due to validation error
@@ -1891,8 +1861,8 @@ public Attachments createReceiptAttachmentByFileName(UUID receiptID, String file
     }
   /**
     * Allows you to retrieve a history records of an Receipt
-    * <p><b>200</b> - Success - return response of type HistoryRecords array for newly created HistoryRecord for Receipt
-    * <p><b>400</b> - A failed request due to validation error
+    * <p><b>200</b> - Unsupported - return response incorrect exception, API is not able to create HistoryRecord for Receipts
+    * <p><b>400</b> - Unsupported - return response incorrect exception, API is not able to create HistoryRecord for Receipts
     * @param receiptID Unique identifier for a Receipt
     * @param historyRecords The historyRecords parameter
     * @return HistoryRecords

src/main/java/com/xero/models/accounting/ManualJournal.java

@@ -18,6 +18,7 @@
 import com.fasterxml.jackson.annotation.JsonProperty;
 import com.fasterxml.jackson.annotation.JsonCreator;
 import com.fasterxml.jackson.annotation.JsonValue;
+import com.xero.models.accounting.Attachment;
 import com.xero.models.accounting.LineAmountTypes;
 import com.xero.models.accounting.ManualJournalLine;
 import com.xero.models.accounting.ValidationError;
@@ -123,6 +124,10 @@ public static StatusEnum fromValue(String text) {
   @JsonProperty("ValidationErrors")
   private List<ValidationError> validationErrors = null;
 
+  
+  @JsonProperty("Attachments")
+  private List<Attachment> attachments = null;
+
   public ManualJournal narration(String narration) {
     this.narration = narration;
     return this;
@@ -342,6 +347,32 @@ public void setValidationErrors(List<ValidationError> validationErrors) {
     this.validationErrors = validationErrors;
   }
 
+  public ManualJournal attachments(List<Attachment> attachments) {
+    this.attachments = attachments;
+    return this;
+  }
+
+  public ManualJournal addAttachmentsItem(Attachment attachmentsItem) {
+    if (this.attachments == null) {
+      this.attachments = new ArrayList<Attachment>();
+    }
+    this.attachments.add(attachmentsItem);
+    return this;
+  }
+
+   /**
+   * Displays array of attachments from the API
+   * @return attachments
+  **/
+  @ApiModelProperty(value = "Displays array of attachments from the API")
+  public List<Attachment> getAttachments() {
+    return attachments;
+  }
+
+  public void setAttachments(List<Attachment> attachments) {
+    this.attachments = attachments;
+  }
+
 
   @Override
   public boolean equals(java.lang.Object o) {
@@ -363,12 +394,13 @@ public boolean equals(java.lang.Object o) {
         Objects.equals(this.updatedDateUTC, manualJournal.updatedDateUTC) &&
         Objects.equals(this.manualJournalID, manualJournal.manualJournalID) &&
         Objects.equals(this.warnings, manualJournal.warnings) &&
-        Objects.equals(this.validationErrors, manualJournal.validationErrors);
+        Objects.equals(this.validationErrors, manualJournal.validationErrors) &&
+        Objects.equals(this.attachments, manualJournal.attachments);
   }
 
   @Override
   public int hashCode() {
-    return Objects.hash(narration, journalLines, date, lineAmountTypes, status, url, showOnCashBasisReports, hasAttachments, updatedDateUTC, manualJournalID, warnings, validationErrors);
+    return Objects.hash(narration, journalLines, date, lineAmountTypes, status, url, showOnCashBasisReports, hasAttachments, updatedDateUTC, manualJournalID, warnings, validationErrors, attachments);
   }
 
 
@@ -389,6 +421,7 @@ public String toString() {
     sb.append("    manualJournalID: ").append(toIndentedString(manualJournalID)).append("\n");
     sb.append("    warnings: ").append(toIndentedString(warnings)).append("\n");
     sb.append("    validationErrors: ").append(toIndentedString(validationErrors)).append("\n");
+    sb.append("    attachments: ").append(toIndentedString(attachments)).append("\n");
     sb.append("}");
     return sb.toString();
   }

src/main/java/com/xero/models/accounting/Organisation.java

@@ -448,6 +448,45 @@ public static PropertyClassEnum fromValue(String text) {
   @JsonProperty("Class")
   private PropertyClassEnum propertyClass;
 
+  /**
+   * BUSINESS or PARTNER. Partner edition organisations are sold exclusively through accounting partners and have restricted functionality (e.g. no access to invoicing)
+   */
+  public enum EditionEnum {
+    BUSINESS("BUSINESS"),
+    
+    PARTNER("PARTNER");
+
+    private String value;
+
+    EditionEnum(String value) {
+      this.value = value;
+    }
+
+    @JsonValue
+    public String getValue() {
+      return value;
+    }
+
+    @Override
+    public String toString() {
+      return String.valueOf(value);
+    }
+
+    @JsonCreator
+    public static EditionEnum fromValue(String text) {
+      for (EditionEnum b : EditionEnum.values()) {
+        if (String.valueOf(b.value).equals(text)) {
+          return b;
+        }
+      }
+      throw new IllegalArgumentException("Unexpected value '" + text + "'");
+    }
+  }
+
+  
+  @JsonProperty("Edition")
+  private EditionEnum edition;
+
 
   @JsonProperty("LineOfBusiness")
   private String lineOfBusiness;
@@ -936,6 +975,24 @@ public void setPropertyClass(PropertyClassEnum propertyClass) {
     this.propertyClass = propertyClass;
   }
 
+  public Organisation edition(EditionEnum edition) {
+    this.edition = edition;
+    return this;
+  }
+
+   /**
+   * BUSINESS or PARTNER. Partner edition organisations are sold exclusively through accounting partners and have restricted functionality (e.g. no access to invoicing)
+   * @return edition
+  **/
+  @ApiModelProperty(value = "BUSINESS or PARTNER. Partner edition organisations are sold exclusively through accounting partners and have restricted functionality (e.g. no access to invoicing)")
+  public EditionEnum getEdition() {
+    return edition;
+  }
+
+  public void setEdition(EditionEnum edition) {
+    this.edition = edition;
+  }
+
   public Organisation lineOfBusiness(String lineOfBusiness) {
     this.lineOfBusiness = lineOfBusiness;
     return this;
@@ -1086,6 +1143,7 @@ public boolean equals(java.lang.Object o) {
         Objects.equals(this.organisationEntityType, organisation.organisationEntityType) &&
         Objects.equals(this.shortCode, organisation.shortCode) &&
         Objects.equals(this.propertyClass, organisation.propertyClass) &&
+        Objects.equals(this.edition, organisation.edition) &&
         Objects.equals(this.lineOfBusiness, organisation.lineOfBusiness) &&
         Objects.equals(this.addresses, organisation.addresses) &&
         Objects.equals(this.phones, organisation.phones) &&
@@ -1095,7 +1153,7 @@ public boolean equals(java.lang.Object o) {
 
   @Override
   public int hashCode() {
-    return Objects.hash(organisationID, apIKey, name, legalName, paysTax, version, organisationType, baseCurrency, countryCode, isDemoCompany, organisationStatus, registrationNumber, taxNumber, financialYearEndDay, financialYearEndMonth, salesTaxBasis, salesTaxPeriod, defaultSalesTax, defaultPurchasesTax, periodLockDate, endOfYearLockDate, createdDateUTC, timezone, organisationEntityType, shortCode, propertyClass, lineOfBusiness, addresses, phones, externalLinks, paymentTerms);
+    return Objects.hash(organisationID, apIKey, name, legalName, paysTax, version, organisationType, baseCurrency, countryCode, isDemoCompany, organisationStatus, registrationNumber, taxNumber, financialYearEndDay, financialYearEndMonth, salesTaxBasis, salesTaxPeriod, defaultSalesTax, defaultPurchasesTax, periodLockDate, endOfYearLockDate, createdDateUTC, timezone, organisationEntityType, shortCode, propertyClass, edition, lineOfBusiness, addresses, phones, externalLinks, paymentTerms);
   }
 
 
@@ -1130,6 +1188,7 @@ public String toString() {
     sb.append("    organisationEntityType: ").append(toIndentedString(organisationEntityType)).append("\n");
     sb.append("    shortCode: ").append(toIndentedString(shortCode)).append("\n");
     sb.append("    propertyClass: ").append(toIndentedString(propertyClass)).append("\n");
+    sb.append("    edition: ").append(toIndentedString(edition)).append("\n");
     sb.append("    lineOfBusiness: ").append(toIndentedString(lineOfBusiness)).append("\n");
     sb.append("    addresses: ").append(toIndentedString(addresses)).append("\n");
     sb.append("    phones: ").append(toIndentedString(phones)).append("\n");

src/main/java/com/xero/models/accounting/Overpayment.java

@@ -19,6 +19,7 @@
 import com.fasterxml.jackson.annotation.JsonCreator;
 import com.fasterxml.jackson.annotation.JsonValue;
 import com.xero.models.accounting.Allocation;
+import com.xero.models.accounting.Attachment;
 import com.xero.models.accounting.Contact;
 import com.xero.models.accounting.CurrencyCode;
 import com.xero.models.accounting.LineAmountTypes;
@@ -165,7 +166,7 @@ public static StatusEnum fromValue(String text) {
 
 
   @JsonProperty("RemainingCredit")
-  private String remainingCredit;
+  private Double remainingCredit;
 
 
   @JsonProperty("Allocations")
@@ -179,6 +180,10 @@ public static StatusEnum fromValue(String text) {
   @JsonProperty("HasAttachments")
   private Boolean hasAttachments;
 
+  
+  @JsonProperty("Attachments")
+  private List<Attachment> attachments = null;
+
   public Overpayment type(TypeEnum type) {
     this.type = type;
     return this;
@@ -421,7 +426,7 @@ public void setCurrencyRate(Double currencyRate) {
     this.currencyRate = currencyRate;
   }
 
-  public Overpayment remainingCredit(String remainingCredit) {
+  public Overpayment remainingCredit(Double remainingCredit) {
     this.remainingCredit = remainingCredit;
     return this;
   }
@@ -431,11 +436,11 @@ public Overpayment remainingCredit(String remainingCredit) {
    * @return remainingCredit
   **/
   @ApiModelProperty(value = "The remaining credit balance on the overpayment")
-  public String getRemainingCredit() {
+  public Double getRemainingCredit() {
     return remainingCredit;
   }
 
-  public void setRemainingCredit(String remainingCredit) {
+  public void setRemainingCredit(Double remainingCredit) {
     this.remainingCredit = remainingCredit;
   }
 
@@ -509,6 +514,32 @@ public void setHasAttachments(Boolean hasAttachments) {
     this.hasAttachments = hasAttachments;
   }
 
+  public Overpayment attachments(List<Attachment> attachments) {
+    this.attachments = attachments;
+    return this;
+  }
+
+  public Overpayment addAttachmentsItem(Attachment attachmentsItem) {
+    if (this.attachments == null) {
+      this.attachments = new ArrayList<Attachment>();
+    }
+    this.attachments.add(attachmentsItem);
+    return this;
+  }
+
+   /**
+   * See Attachments
+   * @return attachments
+  **/
+  @ApiModelProperty(value = "See Attachments")
+  public List<Attachment> getAttachments() {
+    return attachments;
+  }
+
+  public void setAttachments(List<Attachment> attachments) {
+    this.attachments = attachments;
+  }
+
 
   @Override
   public boolean equals(java.lang.Object o) {
@@ -535,12 +566,13 @@ public boolean equals(java.lang.Object o) {
         Objects.equals(this.remainingCredit, overpayment.remainingCredit) &&
         Objects.equals(this.allocations, overpayment.allocations) &&
         Objects.equals(this.payments, overpayment.payments) &&
-        Objects.equals(this.hasAttachments, overpayment.hasAttachments);
+        Objects.equals(this.hasAttachments, overpayment.hasAttachments) &&
+        Objects.equals(this.attachments, overpayment.attachments);
   }
 
   @Override
   public int hashCode() {
-    return Objects.hash(type, contact, date, status, lineAmountTypes, lineItems, subTotal, totalTax, total, updatedDateUTC, currencyCode, overpaymentID, currencyRate, remainingCredit, allocations, payments, hasAttachments);
+    return Objects.hash(type, contact, date, status, lineAmountTypes, lineItems, subTotal, totalTax, total, updatedDateUTC, currencyCode, overpaymentID, currencyRate, remainingCredit, allocations, payments, hasAttachments, attachments);
   }
 
 
@@ -566,6 +598,7 @@ public String toString() {
     sb.append("    allocations: ").append(toIndentedString(allocations)).append("\n");
     sb.append("    payments: ").append(toIndentedString(payments)).append("\n");
     sb.append("    hasAttachments: ").append(toIndentedString(hasAttachments)).append("\n");
+    sb.append("    attachments: ").append(toIndentedString(attachments)).append("\n");
     sb.append("}");
     return sb.toString();
   }

src/main/java/com/xero/models/accounting/Payment.java

@@ -89,7 +89,7 @@ public class Payment {
 
 
   @JsonProperty("IsReconciled")
-  private String isReconciled;
+  private Boolean isReconciled;
 
   /**
    * The status of the payment.
@@ -429,7 +429,7 @@ public void setReference(String reference) {
     this.reference = reference;
   }
 
-  public Payment isReconciled(String isReconciled) {
+  public Payment isReconciled(Boolean isReconciled) {
     this.isReconciled = isReconciled;
     return this;
   }
@@ -439,11 +439,11 @@ public Payment isReconciled(String isReconciled) {
    * @return isReconciled
   **/
   @ApiModelProperty(value = "An optional parameter for the payment. A boolean indicating whether you would like the payment to be created as reconciled when using PUT, or whether a payment has been reconciled when using GET")
-  public String getIsReconciled() {
+  public Boolean getIsReconciled() {
     return isReconciled;
   }
 
-  public void setIsReconciled(String isReconciled) {
+  public void setIsReconciled(Boolean isReconciled) {
     this.isReconciled = isReconciled;
   }