2

AbstractTransactionalJUnit4SpringContextTests で JUnit4 を使用しており、以下に示すように、単純な CRUD 単体テストを作成したいと考えています。ただし、SQL 更新の呼び出しにより、トランザクション マネージャーがテスト インスタンスの新しい ID を生成しています。更新中に新しい ID が返されないため、テストしている行の主キーがなくなりました。これにより、残りのテストが失敗します。

update が呼び出されたときに、トランザクション マネージャーが書籍の新しい ID を生成しないようにする方法はありますか? (失敗した場合、CRUD をテストするためのより良い方法はありますか?)

@Test
public void testCRUDBook() {
    Book b1 = new Book(title, author);
    BookFactory factory = database.getBookFactory();
    int id = factory.createBook(b1);

    Book b2 = factory.readBook(id);        
    assertEquals(b1.getTitle(), b2.getTitle());
    assertEquals(b1.getAuthor(), b2.getAuthor());

    b2.setTitle("title 2");
    b2.setAuthor("author 2");
    assertTrue(factory.updateBook(b2));

    // The problem arises here as updating the book record causes a new Id 
    // to be generated so querying by Id is no longer possible.
    Book b3 = factory.readBook(b2.getId());        
    assertEquals(b3.getTitle(), "title 2");
    assertEquals(b3.getAuthor(), "author 2");    

    assertTrue(factory.deleteBook(b3));
}

本は次のようになります。

public class Book {
    private int id;
    private String title;
    private String author;

    public Book() {}

    // NEW
    public Book(String title, String author) {
        this.title = title;
        this.author = author;
    }

    // READ
    public Book(int id, String title, String author) {
        this.id = id;
        this.title = title;
        this.author = author;
    }

    // GENERIC ACCESSORS (left out for brevity)
}

完全を期すために - 工場:

public class BookFactory extends BaseDatabaseFactory {

        @Transactional
    public int createBook(final Book b) {
        final String insertSQL = "INSERT INTO book (author, title) VALUES (?, ?)";
        KeyHolder keyHolder = new GeneratedKeyHolder();
        try {
            int update = jdbcTemplate.update(
                    new PreparedStatementCreator() {
                        @Override
                        public PreparedStatement createPreparedStatement(Connection connection) throws SQLException {
                            PreparedStatement ps = connection.prepareStatement(insertSQL, PreparedStatement.RETURN_GENERATED_KEYS);
                            ps.setString(1, b.getTitle());
                            ps.setString(2, b.getAuthor());
                            return ps;
                        }
                    },
                    keyHolder);
            if (update != 1) {
                throw new IllegalStateException("Adding book record to database resulted in " + update + " records.");
            }
            return keyHolder.getKey().intValue();
        } catch (DataAccessException ex) {
            String msg = "Falied to create new book:" + b.toString() + "ex:" + ex.getMessage();
            throw new RuntimeException(msg);
        }
    }

    private static class BookRowMapper implements RowMapper {
        @Override
        public Object mapRow(ResultSet rs, int rowNum) throws SQLException {
            Book b = new Book();
            b.setId(rs.getInt("id"));
            b.setTitle(rs.getString("title"));
            b.setAuthor(rs.getString("author"));
            return b;
        }
    }

    @Transactional
    public Book readBook(int id) {
        Book b = null;
        try {
            String sql = "SELECT * FROM book WHERE id = " + id;
            b = (Book) jdbcTemplate.queryForObject(sql, new Object[]{}, new BookRowMapper());
        } catch (DataAccessException ex) {
            throw new RuntimeException("Falied to locate book.", ex);
        }
        return b;
    }    

    @Transactional
    public boolean updateBook(final Book b) {
        final String updateSQL = "UPDATE book SET author = ?, title = ? WHERE id = ?";
        try {
            jdbcTemplate.update(
                    new PreparedStatementCreator() {
                        @Override
                        public PreparedStatement createPreparedStatement(Connection connection) throws SQLException {
                            PreparedStatement ps = connection.prepareStatement(updateSQL);
                            ps.setString(1, b.getAuthor());
                            ps.setString(2, b.getTitle());
                            ps.setInt(3, b.getId());
                            return ps;
                        }
                    });
        } catch (DataAccessException ex) {
            log.error("Falied to update Book:" + b.toString(), ex);
            return false;
        }
        return true;
    }

    @Transactional
    public boolean deleteBook(final Book b) {
        try {
            jdbcTemplate.update("DELETE FROM book WHERE id = ?", b.getId());
        } catch (DataAccessException ex) {
            log.error("Falied to delete Book:" + b.toString(), ex);
            return false;
        }
        return true;
    }    
}
4

2 に答える 2

1

代わりにidを整数型にすることはできますか?b2を更新すると、ファクトリによって新しいインスタンスが作成されるように見えます。idはnull許容である必要があります。

于 2012-07-11T21:42:50.440 に答える
0

問題を文書化しようとしているときに問題を修正しました(きちんとしたファクトリコードを貼り付けることによって)。createBook(book)およびreadBook(int)ファクトリメソッドに@Transactionalアノテーションがありませんでした。トランザクションは呼び出し元のテストコンテナから継承されていると思っていましたが、そうではないようです。テストをトランザクション間で分割すると、行の複数のインスタンスが作成されている必要があります。

これが同じ問題に遭遇した他の誰かに役立つことを願っています。

于 2012-07-11T22:09:44.620 に答える