导致JUnit测试失败的Java assertEquals

导致JUnit测试失败的Java assertEquals,java,junit,hashcode,jdbctemplate,netbeans-8,Java,Junit,Hashcode,Jdbctemplate,Netbeans 8,我正在Netbeans中处理一个javajdbctemplate项目,我很难找出我的equals和hashcode方法覆盖Dao的assertEquals测试的错误。我被告知需要对对象进行深入比较,但从我所看到的情况来看,我的代码已经在这样做了。下面是我涉及这个问题的不同课程 这是我的组织。班级: 以下是我的组织Dao实现: My Organization.class使用Location对象输入其他位置信息。因此,在下面,我必须先添加一个位置,然后再尝试添加一个组织。目前,添加位置的测试已成功

我正在Netbeans中处理一个javajdbctemplate项目,我很难找出我的equals和hashcode方法覆盖Dao的assertEquals测试的错误。我被告知需要对对象进行深入比较,但从我所看到的情况来看,我的代码已经在这样做了。下面是我涉及这个问题的不同课程

这是我的组织。班级:

以下是我的组织Dao实现:

My Organization.class使用Location对象输入其他位置信息。因此,在下面,我必须先添加一个位置,然后再尝试添加一个组织。目前,添加位置的测试已成功

public class SuperSighting_DaoTests {
    
    LocationDao ldao;
    OrgDao odao;
    PowerDao pdao;
    SightingDao sidao;
    SupeDao sudao;
    
    public SuperSighting_DaoTests() {
    }
    
    @BeforeClass
    public static void setUpClass() {
    }
    
    @AfterClass
    public static void tearDownClass() {
    }
    
    @Before
    public void setUp() {
        ApplicationContext ctx
        = new ClassPathXmlApplicationContext("test-applicationContext.xml");
            
            ldao = ctx.getBean("LocationDao", LocationDao.class);
            odao = ctx.getBean("OrgDao", OrgDao.class);
            pdao = ctx.getBean("PowerDao", PowerDao.class);
            sidao = ctx.getBean("SightingDao", SightingDao.class);
            sudao = ctx.getBean("SupeDao", SupeDao.class);
            
            // delete all supes
            List<Supe> supes = sudao.getAllSupes(); for (Supe currentSupe : supes) {
            sudao.deleteSupe(currentSupe.getSupeId()); 
            }
            // delete all powers
            List<Power> powers = pdao.getAllPowers(); for (Power currentPower : powers) {
            pdao.deletePower(currentPower.getPowerId()); 
            }
            //delete all organizations
            List<Organization> orgs = odao.getAllOrganizations(); for (Organization currentOrg : orgs) {
            odao.deleteOrganization(currentOrg.getOrgId()); 
            }
            // delete all locations
            List<Location> locations = ldao.getAllLocations(); for (Location currentLocation : locations) {
            ldao.deleteLocation(currentLocation.getLocationId()); 
            }
            // delete all sightings
            List<Sighting> sightings = sidao.getAllSightings(); for (Sighting currentSighting : sightings) {
            sidao.deleteSighting(currentSighting.getSightingId()); 
            }
    }
    @Test
    public void testAddGetOrganization() {
        
        Location loc = new Location();
        loc.setLocName("Legion of Doom");
        loc.setLocStreetAddress("127 Taco St.");
        loc.setLocCity("Smalltown");
        loc.setLocState("MA");
        loc.setLocZipCode("19698");
        loc.setLocLat("39.16567925978815");
        loc.setLocLong("-75.59452746539126");
        
        ldao.addLocation(loc);
    
        Organization org = new Organization();
        org.setOrgName("Legion of Doom");
        org.setOrgDescription("evil organization");
        org.setOrgPhone("333-444-5678");
        org.setOrgEmail("lod@evil.org");
        org.setOrgLocation(loc);
        
        odao.addOrganization(org);

 
        
        Organization fromDao = odao.getOrganizationById(org.getOrgId());
        fromDao.toString();

        assertEquals(fromDao, org);
    
    }
测试完成后,我可以看到比较的项目完全相同,但测试失败,并给出以下错误:

测试运行:8,失败:1,错误:0,跳过:0,经过的时间:2.931
sec删除:完全错误-应该更加小心地检查屏幕截图

我想删除这个问题,但我不能-谢谢你的快速回复

我不知怎么解决了这个问题。。。事实证明,我的手机和电子邮件系统正在开启检索功能——你可以在截图中看到这一点……在长时间盯着看之后,很容易错过

我检查了我所有的代码并切换了顺序以匹配数据库,现在我的测试是绿色的


再次感谢

使用调试器在断言处设置断点,并检查这两个对象是否真的相同。我看到了问题:准备好的语句(如SQL\u INSERT\u组织和SQL\u UPDATE\u组织)的定义在DAO类的开头,使用时间是几十行之后,因此,很容易忽略准备好的语句期望在org_电话之前发送org_电子邮件,但您在organization.getOrgPhone、organization.getOrgEmail中设置了相反顺序的参数,。这就是为什么我要么内联这些常量,要么在使用它们的方法上方定义它们的原因之一……是的,我后来注意到,在检索对象时,这两个项正在切换。
public class OrgDaoDBImpl implements OrgDao{
    
    private JdbcTemplate jdbcTemplate;

    public void setJdbcTemplate(JdbcTemplate jdbcTemplate) {
        this.jdbcTemplate = jdbcTemplate;
    }

    
    //ORGANIZATIONS - PREPARED STATEMENTS
    
    private static final String SQL_INSERT_ORGANIZATION
            = "insert into `organization` (org_name, org_description, org_email, "
            + "org_phone, location_id) "
            + "values (?, ?, ?, ?, ?)";

    private static final String SQL_DELETE_ORGANIZATION
            = "delete from `organization` where org_id = ?";

    private static final String SQL_UPDATE_ORGANIZATION
            = "update `organization` set org_name = ?, org_description = ?, org_email = ?, "
            + "org_phone = ?, location_id = ? "
            + "where org_id =  ?";

    private static final String SQL_SELECT_ORGANIZATION
            = "select * from `organization` where org_id = ?";

    private static final String SQL_SELECT_ALL_ORGANIZATIONS
            = "select * from `organization`";
    

    
    private static final String SQL_SELECT_LOCATION_BY_ORG_ID
            = "select l.location_id, l.loc_name, l.loc_street_address, "
            + "l.loc_city, l.loc_state, l.loc_zip_code, l.loc_lat, l.loc_long " 
            + "from location l "
            + "join organization o on l.location_id = o.location_id  " 
            + "where o.org_id = ?";
 
    @Override
    @Transactional(propagation = Propagation.REQUIRED, readOnly = false)
    public void addOrganization(Organization organization) {
        jdbcTemplate.update(SQL_INSERT_ORGANIZATION,
            organization.getOrgName(),
            organization.getOrgDescription(),
            organization.getOrgPhone(),
            organization.getOrgEmail(),
            organization.getOrgLocation().getLocationId());
            organization.setOrgId(
                    jdbcTemplate.queryForObject("select LAST_INSERT_ID()", Integer.class));
            
    }

    @Override
    public void deleteOrganization(int organizationId) {
        jdbcTemplate.update(SQL_DELETE_ORGANIZATION, organizationId);
    }

    @Override
    public void updateOrganization(Organization organization) {
        jdbcTemplate.update(SQL_UPDATE_ORGANIZATION,
            organization.getOrgName(),
            organization.getOrgDescription(),
            organization.getOrgPhone(),
            organization.getOrgEmail(),
            organization.getOrgLocation().getLocationId());
    }

    @Override
    public Organization getOrganizationById(int id) {
        try {
            Organization org = jdbcTemplate.queryForObject(SQL_SELECT_ORGANIZATION,
                new OrgMapper(),
                id);
            org.setOrgLocation(findLocationForOrganization(org));
            return org;
        } catch (EmptyResultDataAccessException ex) {
            return null; 
            }
    }

    @Override
    public List<Organization> getAllOrganizations() {
        return jdbcTemplate.query(SQL_SELECT_ALL_ORGANIZATIONS, new OrgMapper());
    }

    @Override
    public List<Organization> getAllOrgsBySupeId(int supeId) {
        throw new UnsupportedOperationException("Not supported yet."); //To change body of generated methods, choose Tools | Templates.
    }
    
    
    /////////////////////
    //*HELPER METHODS*//
    //***************//   
    
    //FIND Location associated with an organization
    private Location findLocationForOrganization(Organization org) {
        return jdbcTemplate.queryForObject(SQL_SELECT_LOCATION_BY_ORG_ID,
                                            new LocationMapper(), 
                                            org.getOrgId());
    }

    
    //ASSOCIATE the location with the org entry
    private List<Organization>associateLocationWithOrg(List<Organization> orgList) {
        // set the complete list of author ids for each book
        for (Organization currentOrg : orgList) {
            // add the Location to current Org
            currentOrg.setOrgLocation(findLocationForOrganization(currentOrg)); }
        return orgList; 
    }
    
    
    /////////////
    //*MAPPERS*/
    //*******//
    
    private static final class OrgMapper implements RowMapper<Organization> {
        
        @Override
        public Organization mapRow(ResultSet rs, int i) throws SQLException {
            Organization org = new Organization();
            org.setOrgId(rs.getInt("org_id"));
            org.setOrgName(rs.getString("org_name"));
            org.setOrgDescription(rs.getString("org_description"));
            org.setOrgPhone(rs.getString("org_phone"));
            org.setOrgEmail(rs.getString("org_email"));
            
              
            
            return org;
        
        }    
    }
    
    private static class LocationMapper implements RowMapper<Location>{
        
        @Override
        public Location mapRow(ResultSet rs, int i) throws SQLException {
            Location loc = new Location();
            loc.setLocationId(rs.getInt("location_id"));
            loc.setLocName(rs.getString("loc_name"));
            loc.setLocStreetAddress(rs.getString("loc_street_address"));
            loc.setLocCity(rs.getString("loc_city"));
            loc.setLocState(rs.getString("loc_state"));
            loc.setLocZipCode(rs.getString("loc_zip_code"));
            loc.setLocLat(rs.getString("loc_lat"));
            loc.setLocLong(rs.getString("loc_long"));

            
            return loc;
        }
    }
}

public class SuperSighting_DaoTests {
    
    LocationDao ldao;
    OrgDao odao;
    PowerDao pdao;
    SightingDao sidao;
    SupeDao sudao;
    
    public SuperSighting_DaoTests() {
    }
    
    @BeforeClass
    public static void setUpClass() {
    }
    
    @AfterClass
    public static void tearDownClass() {
    }
    
    @Before
    public void setUp() {
        ApplicationContext ctx
        = new ClassPathXmlApplicationContext("test-applicationContext.xml");
            
            ldao = ctx.getBean("LocationDao", LocationDao.class);
            odao = ctx.getBean("OrgDao", OrgDao.class);
            pdao = ctx.getBean("PowerDao", PowerDao.class);
            sidao = ctx.getBean("SightingDao", SightingDao.class);
            sudao = ctx.getBean("SupeDao", SupeDao.class);
            
            // delete all supes
            List<Supe> supes = sudao.getAllSupes(); for (Supe currentSupe : supes) {
            sudao.deleteSupe(currentSupe.getSupeId()); 
            }
            // delete all powers
            List<Power> powers = pdao.getAllPowers(); for (Power currentPower : powers) {
            pdao.deletePower(currentPower.getPowerId()); 
            }
            //delete all organizations
            List<Organization> orgs = odao.getAllOrganizations(); for (Organization currentOrg : orgs) {
            odao.deleteOrganization(currentOrg.getOrgId()); 
            }
            // delete all locations
            List<Location> locations = ldao.getAllLocations(); for (Location currentLocation : locations) {
            ldao.deleteLocation(currentLocation.getLocationId()); 
            }
            // delete all sightings
            List<Sighting> sightings = sidao.getAllSightings(); for (Sighting currentSighting : sightings) {
            sidao.deleteSighting(currentSighting.getSightingId()); 
            }
    }
    @Test
    public void testAddGetOrganization() {
        
        Location loc = new Location();
        loc.setLocName("Legion of Doom");
        loc.setLocStreetAddress("127 Taco St.");
        loc.setLocCity("Smalltown");
        loc.setLocState("MA");
        loc.setLocZipCode("19698");
        loc.setLocLat("39.16567925978815");
        loc.setLocLong("-75.59452746539126");
        
        ldao.addLocation(loc);
    
        Organization org = new Organization();
        org.setOrgName("Legion of Doom");
        org.setOrgDescription("evil organization");
        org.setOrgPhone("333-444-5678");
        org.setOrgEmail("lod@evil.org");
        org.setOrgLocation(loc);
        
        odao.addOrganization(org);

 
        
        Organization fromDao = odao.getOrganizationById(org.getOrgId());
        fromDao.toString();

        assertEquals(fromDao, org);
    
    }