diff --git a/java/google/registry/export/sheet/SheetSynchronizer.java b/java/google/registry/export/sheet/SheetSynchronizer.java index 8d46f078e..b6702f7a8 100644 --- a/java/google/registry/export/sheet/SheetSynchronizer.java +++ b/java/google/registry/export/sheet/SheetSynchronizer.java @@ -71,7 +71,7 @@ class SheetSynchronizer { URL url = new URL(SPREADSHEET_URL_PREFIX + spreadsheetId); SpreadsheetEntry spreadsheet = spreadsheetService.getEntry(url, SpreadsheetEntry.class); WorksheetEntry worksheet = spreadsheet.getWorksheets().get(0); - worksheet.setRowCount(data.size()); + worksheet.setRowCount(data.size() + 1); // account for header row worksheet = worksheet.update(); ListFeed listFeed = spreadsheetService.getFeed(worksheet.getListFeedUrl(), ListFeed.class); List entries = listFeed.getEntries(); diff --git a/javatests/google/registry/export/sheet/SheetSynchronizerTest.java b/javatests/google/registry/export/sheet/SheetSynchronizerTest.java index 6c2e24bc6..58c91d370 100644 --- a/javatests/google/registry/export/sheet/SheetSynchronizerTest.java +++ b/javatests/google/registry/export/sheet/SheetSynchronizerTest.java @@ -78,7 +78,7 @@ public class SheetSynchronizerTest { public void testSynchronize_bothEmpty_doNothing() throws Exception { when(listFeed.getEntries()).thenReturn(ImmutableList.of()); sheetSynchronizer.synchronize("foobar", ImmutableList.>of()); - verify(worksheet).setRowCount(0); + verify(worksheet).setRowCount(1); verify(worksheet).update(); } @@ -88,7 +88,7 @@ public class SheetSynchronizerTest { when(listFeed.getEntries()).thenReturn(ImmutableList.of(entry)); sheetSynchronizer.synchronize("foobar", ImmutableList.of( ImmutableMap.of("key", "value"))); - verify(worksheet).setRowCount(1); + verify(worksheet).setRowCount(2); verify(worksheet).update(); verify(entry, atLeastOnce()).getCustomElements(); verifyNoMoreInteractions(entry); @@ -102,7 +102,7 @@ public class SheetSynchronizerTest { ImmutableMap.of("key", "new value"))); verify(entry.getCustomElements()).setValueLocal("key", "new value"); verify(entry).update(); - verify(worksheet).setRowCount(1); + verify(worksheet).setRowCount(2); verify(worksheet).update(); verify(entry, atLeastOnce()).getCustomElements(); verifyNoMoreInteractions(entry); @@ -117,7 +117,7 @@ public class SheetSynchronizerTest { ImmutableMap.of("key", "value"))); verify(entry.getCustomElements()).setValueLocal("key", "value"); verify(listFeed).insert(entry); - verify(worksheet).setRowCount(1); + verify(worksheet).setRowCount(2); verify(worksheet).update(); verify(listFeed).createEntry(); verify(entry, atLeastOnce()).getCustomElements(); @@ -129,7 +129,7 @@ public class SheetSynchronizerTest { ListEntry entry = makeListEntry(ImmutableMap.of("key", "value")); when(listFeed.getEntries()).thenReturn(ImmutableList.of(entry)); sheetSynchronizer.synchronize("foobar", ImmutableList.>of()); - verify(worksheet).setRowCount(0); + verify(worksheet).setRowCount(1); verify(worksheet).update(); verifyNoMoreInteractions(entry); }