Skip to content
Snippets Groups Projects

Uc20

Merged Uc20
2 unresolved threads
Merged sbobo3 requested to merge UC20 into master
2 unresolved threads

This merge request contains all of the deliverables for UC20, including:

  • UI changes (search parameters, proper error messages, and search results table)
  • DAO class to initiate queries to database, and bean class to store results from query
  • Unit tests
  • Selenium tests

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • sbobo3 changed milestone to %Iteration 2

    changed milestone to %Iteration 2

    • I think unit tests still have one error, I found an error while running mvn test -Dtest=CauseOfDeathTrendsReportTest test

      This is part of it:

      Failed tests: junit.framework.TestSuite$1.warning(junit.framework.TestSuite$1) Run 1: TestSuite$1.warning Exception in constructor: testTopTwoDeathsForAllValid (java.lang.Error: Unresolved compilation problem: The declared package "edu.ncsu.csc.itrust.unit.dao.CauseOfDeathTrendsReportDAO" does not match the expected package "edu.ncsu.csc.itrust.unit.dao.deaths"

      Edited by htmoss2
    • I am getting the same error. Looks like it is a compile error in the CauseOfDeathTrendsReportDAO. I will add a comment on the line.

    • Please register or sign in to reply
1 package edu.ncsu.csc.itrust.unit.dao.CauseOfDeathTrendsReportDAO;
  • Selenium tests are passing. Code looks good to me.

  • sbobo3 added 1 commit

    added 1 commit

    Compare with previous version

  • Thanks for the fix. All testing passing.

  • adityab3 approved this merge request

    approved this merge request

  • sbobo3 added 1 commit

    added 1 commit

    • a5cf08d1 - Updated to include iTrust.iml

    Compare with previous version

  • sbobo3 added 1 commit

    added 1 commit

    Compare with previous version

  • merged

  • htmoss2 mentioned in commit 9c652e5e

    mentioned in commit 9c652e5e

  • Please register or sign in to reply
    Loading