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

Change how system works with config and Tester classes #44

Closed
9 of 10 tasks
dservich opened this issue Oct 12, 2015 · 7 comments
Closed
9 of 10 tasks

Change how system works with config and Tester classes #44

dservich opened this issue Oct 12, 2015 · 7 comments
Assignees

Comments

@dservich
Copy link
Collaborator

Have a student.txt file with the handle and identifier number and be able to select the student to run the single test on.

  • Remove config select
  • Remove radio buttons for batch select
  • Add a panel to view the students from a selected course
  • Save the selected students to an array list
  • Read arraylist from settings in BatchTester
  • Run BatchTester from array list
  • adjust how the configSingle and configBatch files are used
  • Determine the use of both the singleTest and the BatchTest, or just use batchTest?
  • Eliminate the batch tester config and just read the information directly from the folder(will lose student handle and class name info associated with the student, so we might want to think of a better way to do this for next Sprint)

Move to another issue/Sprint

  • Allow the user to select which file contains the students (for multiple courses)
@ryosua
Copy link
Owner

ryosua commented Oct 21, 2015

Maybe get rid of config files altogether and consolidate SingleTester and BatchTester.

@ryosua
Copy link
Owner

ryosua commented Oct 21, 2015

I'll help on this.

@ryosua ryosua changed the title Allow running of single test for different users Change how system works with config and Tester classes Oct 21, 2015
@ryosua
Copy link
Owner

ryosua commented Oct 28, 2015

img_0851

ryosua pushed a commit that referenced this issue Nov 3, 2015
@ryosua
Copy link
Owner

ryosua commented Nov 3, 2015

@dservich I think we need to talk to Dave to see if he really needs the other info associated with the students. If he doesn't than it simplifies the system by getting rid of the config file, but if he does need the info than we should figure out a way to enter it through the GUI I think.

I am going to merge what we have so far, because I think that it is close enough for the sprint. The code could use some refactoring and tests though.

@ryosua
Copy link
Owner

ryosua commented Nov 4, 2015

Actually, we might just want to get rid of the config to fix #55. I'll open a new branch, and then if we merge all that change in, it will be easy to revert.

@ryosua
Copy link
Owner

ryosua commented Nov 4, 2015

@ryosua
Copy link
Owner

ryosua commented Nov 4, 2015

Can close if #56 is merged.

@ryosua ryosua closed this as completed Nov 4, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants