Updating README file

- Removing getters and setters and adding missing constructor
- Improving the tests
- Adding an additional method to find byFirstName to improve the tests

Author: enriquezrene

README.md

@@ -82,37 +82,27 @@ Create a DynamoDB entity for this table:
 @DynamoDBTable(tableName = "User")
 public class User {
 
-  private String id;
-  private String firstName;
-  private String lastName;
-
-  @DynamoDBHashKey
-  @DynamoDBAutoGeneratedKey 
-  public String getId() {
-	return id;
-  }
-
-  @DynamoDBAttribute
-  public String getFirstName() {
-	return firstName;
-  }
-
-  @DynamoDBAttribute
-  public String getLastName() {
-	return lastName;
-  }
-       
-  public void setId(String id) {
-    this.id = id;
-  }
-  
-  public void setFirstName(String firstName) {
-    this.firstName = firstName;
-  }
+    @DynamoDBHashKey
+    @DynamoDBAutoGeneratedKey
+    private String id;
+
+    @DynamoDBAttribute
+    private String firstName;
+
+    @DynamoDBAttribute
+    private String lastName;
+
+    public User(String firstName, String lastName) {
+        this.firstName = firstName;
+        this.lastName = lastName;
+    }
+    
+    public User() {
+        super();
+    }
+    
+    // Getters and setters
 
-  public void setLastName(String lastName) {
-    this.lastName = lastName;
-  }
 }
 ```
 
@@ -123,7 +113,9 @@ package com.acme.repositories;
 
 @EnableScan
 public interface UserRepository extends CrudRepository<User, String> {
-  List<User> findByLastName(String lastName);
+    
+    List<User> findByLastName(String lastName);
+    List<User> findByFirstName(String firstName);
 }
 ```
 
@@ -133,7 +125,9 @@ or for paging and sorting...
 package com.acme.repositories;
 
 public interface UserRepository extends PagingAndSortingRepository<User, String> {
-  Page<User> findByLastName(String lastName,Pageable pageable);
+
+  Page<User> findByLastName(String lastName, Pageable pageable);
+  Page<User> findByFirstName(String firstName, Pageable pageable);
 
   @EnableScan 
   @EnableScanCount
@@ -145,45 +139,40 @@ And finally write a test client
 
 ```java
 @RunWith(SpringJUnit4ClassRunner.class)
-@SpringApplicationConfiguration(classes = { 
-    PropertyPlaceholderAutoConfiguration.class, DynamoDBConfig.class})
-    public class UserRepositoryIntegrationTest {
-       
+@SpringBootTest
+public class UserRepositoryIntegrationTest {
+
     @Autowired
-    UserRepository repository;
-        
+    private UserRepository repository;
+
     @Test
     public void sampleTestCase() {
         User dave = new User("Dave", "Matthews");
         repository.save(dave);
-    
+
         User carter = new User("Carter", "Beauford");
         repository.save(carter);
-    
-        List<User> result = repository.findByLastName("Matthews");
-        Assert.assertThat(result.size(), is(1));
-        Assert.assertThat(result, hasItem(dave));
+
+        List<User> matthewsFound = repository.findByLastName("Matthews");
+        Assert.assertFalse(matthewsFound.isEmpty());
+
+        List<User> cartersFound = repository.findByFirstName("Carter");
+        Assert.assertFalse(cartersFound.isEmpty());
+
     }
-    
+
     private static final long CAPACITY = 5L;
-    
+
     @Autowired
     private AmazonDynamoDB amazonDynamoDB;
-    
+
     @Before
     public void init() throws Exception {
-        // Delete User table in case it exists
-        amazonDynamoDB.listTables().getTableNames().stream().
-                filter(tableName -> tableName.equals(User.TABLE_NAME)).forEach(tableName -> {
-            amazonDynamoDB.deleteTable(tableName);
-        });
-	
-	//Create User table
-        amazonDynamoDB.createTable(new DynamoDBMapper(amazonDynamoDB)
-            .generateCreateTableRequest(User.class)
-	    .withProvisionedThroughput(new ProvisionedThroughput(CAPACITY, CAPACITY)));
+        TableUtils.createTableIfNotExists(amazonDynamoDB, new DynamoDBMapper(amazonDynamoDB)
+                .generateCreateTableRequest(User.class)
+                .withProvisionedThroughput(new ProvisionedThroughput(CAPACITY, CAPACITY)));
     }
-    
+
 }
 ```