Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

S2050 Refactor Submission.java #23

Merged
merged 27 commits into from
Dec 3, 2020
Merged

Conversation

sotimus
Copy link
Collaborator

@sotimus sotimus commented Nov 17, 2020

This project focuses on redesigning and rewriting one large method that parses one student submission file and builds a data structure for later evaluation.
Contributors:

  1. Jack Maccdonald (@jmmacdo4)
  2. Nick Garner (@nickrgarner)
  3. Sumitosh Pal (@sith2594 )
  4. Abhishek Gupta (@abhishekgupta930)

jmmacdo4 and others added 27 commits October 24, 2020 22:09
Merging Paul's MySQL test files and bug fixes
…noring extra sql statements before parsing first question
…SQL and Oracle

Folders with sample folders for MySQL as well as the previous Oracle sample files are provided, though now in separate subfolders.  Files folder now emptied, user must copy files from one of two sample folders for testing.  assignmentProperties sample files provided for both Oracle and MySQL (some SQL dialect differences illustrated).  Install and Usage docs updated accordingly.
1) Fix bug to make result sets scrollable (needed for testResultSetEqualContent); 2) Use newer driver class to avoid warnings, and 3) set up port variable to allow possibility of running MySQL server on other ports than 3306
Getting update to test files
Refactoring in readsubmission
Added some more tests to increase the coverage on readSubmission()
@wagnerpj42 wagnerpj42 merged commit cabba48 into wagnerpj42:master Dec 3, 2020
@wagnerpj42
Copy link
Owner

Refactoring of readSubmission() method in Submission class as part of NCSU class project S2050 and accompanying tests now merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants