Add tax code/purchase order/invoice email

2 files changed, 156 insertions, 17 deletions

diff --git a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/persistence/TenantSerializerTest.java b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/persistence/TenantSerializerTest.java
index 9de856971c9..31794aad350 100644
--- a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/persistence/TenantSerializerTest.java
+++ b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/persistence/TenantSerializerTest.java
@@ -234,7 +234,11 @@ public class TenantSerializerTest {
                                 .withCity("Suddery")
                                 .withCountry("Sodor")
                                 .withAddress("Central Station")
-                                .withRegion("Irish Sea")));
+                                .withRegion("Irish Sea"))
+                        .withPurchaseOrder("PO42")
+                        .withTaxCode("1234L")
+                        .withInvoiceEmail("billing@mycomp.any")
+                );
 
         Slime slime = new Slime();
         Cursor parentCursor = slime.setObject();
diff --git a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/restapi/application/ApplicationApiCloudTest.java b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/restapi/application/ApplicationApiCloudTest.java
index 2b01d87c903..1b6425a324a 100644
--- a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/restapi/application/ApplicationApiCloudTest.java
+++ b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/restapi/application/ApplicationApiCloudTest.java
@@ -98,19 +98,68 @@ public class ApplicationApiCloudTest extends ControllerContainerCloudTest {
 
     @Test
     void tenant_info_billing() {
+        var expectedResponse = """
+                {
+                    "contact": {
+                        "name":"",
+                        "email":"",
+                        "emailVerified":false,
+                        "phone":""
+                    },
+                    "taxCode":"",
+                    "purchaseOrder":"",
+                    "invoiceEmail":""
+                }
+                """;
         var request = request("/application/v4/tenant/scoober/info/billing", GET)
                 .roles(Set.of(Role.reader(tenantName)));
-        tester.assertResponse(request, "{\"contact\":{\"name\":\"\",\"email\":\"\",\"emailVerified\":false,\"phone\":\"\"}}", 200);
-
-        var fullAddress = "{\"addressLines\":\"addressLines\",\"postalCodeOrZip\":\"postalCodeOrZip\",\"city\":\"city\",\"stateRegionProvince\":\"stateRegionProvince\",\"country\":\"country\"}";
-        var fullBillingContact = "{\"contact\":{\"name\":\"name\",\"email\":\"foo@example\",\"phone\":\"phone\"},\"address\":" + fullAddress + "}";
-
+        tester.assertJsonResponse(request, expectedResponse, 200);
+
+        var fullBillingContact = """
+                {
+                    "contact": {
+                        "name":"name",
+                        "email":"foo@example",
+                        "phone":"phone"
+                    },
+                    "taxCode":"1234L",
+                    "purchaseOrder":"PO9001",
+                    "invoiceEmail":"billing@mycomp.any",
+                    "address": {
+                        "addressLines":"addressLines",
+                        "postalCodeOrZip":"postalCodeOrZip",
+                        "city":"city",
+                        "stateRegionProvince":"stateRegionProvince",
+                        "country":"country"
+                    }
+                }
+                """;
         var updateRequest = request("/application/v4/tenant/scoober/info/billing", PUT)
                 .data(fullBillingContact)
                 .roles(Set.of(Role.administrator(tenantName)));
         tester.assertResponse(updateRequest, "{\"message\":\"Tenant info updated\"}", 200);
 
-        tester.assertResponse(request, "{\"contact\":{\"name\":\"name\",\"email\":\"foo@example\",\"emailVerified\":false,\"phone\":\"phone\"},\"address\":{\"addressLines\":\"addressLines\",\"postalCodeOrZip\":\"postalCodeOrZip\",\"city\":\"city\",\"stateRegionProvince\":\"stateRegionProvince\",\"country\":\"country\"}}", 200);
+        expectedResponse = """
+                {
+                    "contact": {
+                        "name":"name",
+                        "email":"foo@example",
+                        "emailVerified": false,
+                        "phone":"phone"
+                    },
+                    "taxCode":"1234L",
+                    "purchaseOrder":"PO9001",
+                    "invoiceEmail":"billing@mycomp.any",
+                    "address": {
+                        "addressLines":"addressLines",
+                        "postalCodeOrZip":"postalCodeOrZip",
+                        "city":"city",
+                        "stateRegionProvince":"stateRegionProvince",
+                        "country":"country"
+                    }
+                }
+                """;
+        tester.assertJsonResponse(request, expectedResponse, 200);
     }
 
     @Test
@@ -120,12 +169,32 @@ public class ApplicationApiCloudTest extends ControllerContainerCloudTest {
         tester.assertResponse(request, "{\"contacts\":[{\"audiences\":[\"tenant\",\"notifications\"],\"email\":\"developer@scoober\",\"emailVerified\":true}]}", 200);
 
 
-        var fullContacts = "{\"contacts\":[{\"audiences\":[\"tenant\"],\"email\":\"contact1@example.com\",\"emailVerified\":false},{\"audiences\":[\"notifications\"],\"email\":\"contact2@example.com\",\"emailVerified\":false},{\"audiences\":[\"tenant\",\"notifications\"],\"email\":\"contact3@example.com\",\"emailVerified\":false}]}";
+        var fullContacts = """
+                {
+                    "contacts":[
+                        {
+                            "audiences":["tenant"]
+                            ,"email":"contact1@example.com",
+                            "emailVerified":false
+                        },
+                        {
+                            "audiences":["notifications"],
+                            "email":"contact2@example.com",
+                            "emailVerified":false
+                        },
+                        {
+                            "audiences":["tenant","notifications"],
+                            "email":"contact3@example.com",
+                            "emailVerified":false
+                        }
+                    ]
+                }
+                """;
         var updateRequest = request("/application/v4/tenant/scoober/info/contacts", PUT)
                 .data(fullContacts)
                 .roles(Set.of(Role.administrator(tenantName)));
         tester.assertResponse(updateRequest, "{\"message\":\"Tenant info updated\"}", 200);
-        tester.assertResponse(request, fullContacts, 200);
+        tester.assertJsonResponse(request, fullContacts, 200);
     }
 
     @Test
@@ -150,13 +219,79 @@ public class ApplicationApiCloudTest extends ControllerContainerCloudTest {
         tester.assertResponse(postPartialContacts, "{\"message\":\"Tenant info updated\"}", 200);
 
         // Read back the updated info
-        tester.assertResponse(infoRequest, "{\"name\":\"\",\"email\":\"\",\"website\":\"\",\"contactName\":\"newName\",\"contactEmail\":\"foo@example.com\",\"contactEmailVerified\":false,\"billingContact\":{\"name\":\"billingName\",\"email\":\"\",\"emailVerified\":false,\"phone\":\"\"},\"contacts\":[{\"audiences\":[\"tenant\"],\"email\":\"contact1@example.com\",\"emailVerified\":false}]}", 200);
-
-        String fullAddress = "{\"addressLines\":\"addressLines\",\"postalCodeOrZip\":\"postalCodeOrZip\",\"city\":\"city\",\"stateRegionProvince\":\"stateRegionProvince\",\"country\":\"country\"}";
-        String fullBillingContact = "{\"name\":\"name\",\"email\":\"foo@example\",\"emailVerified\":false,\"phone\":\"phone\",\"address\":" + fullAddress + "}";
-        String fullContacts = "[{\"audiences\":[\"tenant\"],\"email\":\"contact1@example.com\",\"emailVerified\":false},{\"audiences\":[\"notifications\"],\"email\":\"contact2@example.com\",\"emailVerified\":false},{\"audiences\":[\"tenant\",\"notifications\"],\"email\":\"contact3@example.com\",\"emailVerified\":false}]";
-        String fullInfo = "{\"name\":\"name\",\"email\":\"foo@example\",\"website\":\"https://yahoo.com\",\"contactName\":\"contactName\",\"contactEmail\":\"contact@example.com\",\"contactEmailVerified\":false,\"address\":" + fullAddress + ",\"billingContact\":" + fullBillingContact + ",\"contacts\":" + fullContacts + "}";
-
+        var expectedResponse = """
+                {
+                    "name":"",
+                    "email":"",
+                    "website":"",
+                    "contactName":"newName",
+                    "contactEmail":"foo@example.com",
+                    "contactEmailVerified":false,
+                    "billingContact": {
+                        "name":"billingName",
+                        "email":"","emailVerified":false,
+                        "phone":"",
+                        "taxCode":"",
+                        "purchaseOrder":"",
+                        "invoiceEmail":""
+                    },
+                    "contacts": [
+                        {"audiences":["tenant"],"email":"contact1@example.com","emailVerified":false}
+                    ]
+                }
+                """;
+        tester.assertJsonResponse(infoRequest, expectedResponse, 200);
+
+        var fullInfo = """
+                {
+                    "name":"name",
+                    "email":"foo@example",
+                    "website":"https://yahoo.com",
+                    "contactName":"contactName",
+                    "contactEmail":"contact@example.com",
+                    "contactEmailVerified":false,
+                    "address": {
+                        "addressLines":"addressLines",
+                        "postalCodeOrZip":"postalCodeOrZip",
+                        "city":"city",
+                        "stateRegionProvince":"stateRegionProvince",
+                        "country":"country"
+                    },
+                    "billingContact": {
+                        "name":"name",
+                        "email":"foo@example",
+                        "emailVerified":false,
+                        "phone":"phone",
+                        "taxCode":"",
+                        "purchaseOrder":"",
+                        "invoiceEmail":"",
+                        "address": {
+                            "addressLines":"addressLines",
+                            "postalCodeOrZip":"postalCodeOrZip",
+                            "city":"city",
+                            "stateRegionProvince":"stateRegionProvince",
+                            "country":"country"
+                        }
+                    },
+                    "contacts": [
+                        {
+                            "audiences":["tenant"],
+                            "email":"contact1@example.com",
+                            "emailVerified":false
+                        },
+                        {
+                            "audiences":["notifications"],
+                            "email":"contact2@example.com",
+                            "emailVerified":false
+                        },
+                        {
+                            "audiences":["tenant","notifications"]
+                            ,"email":"contact3@example.com",
+                            "emailVerified":false
+                        }
+                    ]
+                }
+                """;
         // Now set all fields
         var postFull =
                 request("/application/v4/tenant/scoober/info", PUT)
@@ -165,7 +300,7 @@ public class ApplicationApiCloudTest extends ControllerContainerCloudTest {
         tester.assertResponse(postFull, "{\"message\":\"Tenant info updated\"}", 200);
 
         // Now compare the updated info with the full info we sent
-        tester.assertResponse(infoRequest, fullInfo, 200);
+        tester.assertJsonResponse(infoRequest, fullInfo, 200);
 
         var invalidBody = "{\"mail\":\"contact1@example.com\", \"mailType\":\"blurb\"}";
         var resendMailRequest =