diff options
author | prathamesh | 2013-07-02 13:15:24 +0530 |
---|---|---|
committer | prathamesh | 2013-07-02 13:15:24 +0530 |
commit | 37625e5c91f2bf957c6739981f7698e0d880b6ed (patch) | |
tree | 387e52fa104f9f3562dad3f7690bec7493730310 | |
parent | fe868c043487af8ea390165325309ad29daacc8e (diff) | |
parent | 610709c2ff35263741ca6e41c3648439806b564a (diff) | |
download | online_test-37625e5c91f2bf957c6739981f7698e0d880b6ed.tar.gz online_test-37625e5c91f2bf957c6739981f7698e0d880b6ed.tar.bz2 online_test-37625e5c91f2bf957c6739981f7698e0d880b6ed.zip |
Merge branch 'C-Cpp' of https://github.com/prathamesh920/online_test into fresh_C_Cpp
-rwxr-xr-x | testapp/c_cpp_files/main.cpp | 30 | ||||
-rwxr-xr-x | testapp/c_cpp_files/main2.c | 28 | ||||
-rwxr-xr-x | testapp/c_cpp_files/main_array_sum.cpp | 32 | ||||
-rwxr-xr-x | testapp/c_cpp_files/main_fact.cpp | 30 | ||||
-rwxr-xr-x | testapp/c_cpp_files/main_greatest.cpp | 32 | ||||
-rwxr-xr-x | testapp/c_cpp_files/main_hello_name.c | 29 | ||||
-rwxr-xr-x | testapp/c_cpp_files/main_palindrome.cpp | 30 | ||||
-rwxr-xr-x | testapp/c_cpp_files/sample.c | 20 | ||||
-rwxr-xr-x | testapp/c_cpp_files/sample.cpp | 7 | ||||
-rwxr-xr-x | testapp/code_server.py | 372 | ||||
-rw-r--r-- | testapp/exam/forms.py | 33 | ||||
-rw-r--r-- | testapp/exam/models.py | 5 | ||||
-rw-r--r-- | testapp/exam/xmlrpc_clients.py | 20 | ||||
-rw-r--r-- | testapp/templates/exam/question.html | 9 | ||||
-rw-r--r-- | testapp/test_server.py | 115 |
15 files changed, 679 insertions, 113 deletions
diff --git a/testapp/c_cpp_files/main.cpp b/testapp/c_cpp_files/main.cpp new file mode 100755 index 0000000..ffd110e --- /dev/null +++ b/testapp/c_cpp_files/main.cpp @@ -0,0 +1,30 @@ +#include <stdio.h> +#include <stdlib.h> + +extern int add(int, int); + +template <class T> + +void check(T expect, T result) +{ + if (expect == result) + { + //printf("Correct:\n Expected %d got %d \n",expect,result); + } + else + { + printf("\nIncorrect:\n Expected %d got %d \n",expect,result); + exit (1); + } +} + +int main(void) +{ + int result; + result = add(0,0); + check(0, result); + result = add(2,3); + check(5,result); + printf("All Correct\n"); + return 0; +} diff --git a/testapp/c_cpp_files/main2.c b/testapp/c_cpp_files/main2.c new file mode 100755 index 0000000..3d28ff6 --- /dev/null +++ b/testapp/c_cpp_files/main2.c @@ -0,0 +1,28 @@ +#include <stdio.h> +#include <stdlib.h> + +extern int add(int, int, int); + +template <class T> +void check(T expect,T result) +{ + if (expect == result) + { + printf("Correct:\n Expected %d got %d \n",expect,result); + } + else + { + printf("Incorrect:\n Expected %d got %d \n",expect,result); + exit (0); + } +} + +int main(void) +{ + int result; + result = add(0,0,0); + check(0, result); + result = add(2,3,3); + check(8,result); + printf("All Correct\n"); +} diff --git a/testapp/c_cpp_files/main_array_sum.cpp b/testapp/c_cpp_files/main_array_sum.cpp new file mode 100755 index 0000000..72eee46 --- /dev/null +++ b/testapp/c_cpp_files/main_array_sum.cpp @@ -0,0 +1,32 @@ +#include <stdio.h> +#include <stdlib.h> + +extern int array_sum(int []); + +template <class T> + +void check(T expect,T result) +{ + if (expect == result) + { + //printf("Correct:\n Expected %d got %d \n",expect,result); + } + else + { + printf("\nIncorrect:\n Expected %d got %d \n",expect,result); + exit (1); + } +} + +int main(void) +{ + int result; + int a[55555] = {1,2,3,0,0}; + result = array_sum(a); + check(6, result); + int b[] = {1,2,3,4,5}; + result = array_sum(b); + check(15,result); + printf("All Correct\n"); + return 0; +} diff --git a/testapp/c_cpp_files/main_fact.cpp b/testapp/c_cpp_files/main_fact.cpp new file mode 100755 index 0000000..ff65456 --- /dev/null +++ b/testapp/c_cpp_files/main_fact.cpp @@ -0,0 +1,30 @@ +#include <stdio.h> +#include <stdlib.h> + +extern int factorial(int); + +template <class T> + +void check(T expect, T result) +{ + if (expect == result) + { + //printf("Correct:\n Expected %d got %d \n",expect,result); + } + else + { + printf("\nIncorrect:\n Expected %d got %d \n",expect,result); + exit (1); + } +} + +int main(void) +{ + int result; + result = factorial(0); + check(1, result); + result = factorial(3); + check(6, result); + printf("All Correct\n"); + return 0; +} diff --git a/testapp/c_cpp_files/main_greatest.cpp b/testapp/c_cpp_files/main_greatest.cpp new file mode 100755 index 0000000..fe9ff5b --- /dev/null +++ b/testapp/c_cpp_files/main_greatest.cpp @@ -0,0 +1,32 @@ +#include <stdio.h> +#include <stdlib.h> + +extern int greatest(int, int, int); + +template <class T> + +void check(T expect, T result) +{ + if (expect == result) + { + //printf("Correct:\n Expected %d got %d \n",expect,result); + } + else + { + printf("\nIncorrect:\n Expected %d got %d \n",expect,result); + exit (1); + } +} + +int main(void) +{ + int result; + result = greatest(1, 2, 3); + check(3, result); + result = greatest(5, 9, 2); + check(9, result); + result = greatest(7, 2, 4); + check(7, result); + printf("All Correct\n"); + return 0; +} diff --git a/testapp/c_cpp_files/main_hello_name.c b/testapp/c_cpp_files/main_hello_name.c new file mode 100755 index 0000000..71b83a2 --- /dev/null +++ b/testapp/c_cpp_files/main_hello_name.c @@ -0,0 +1,29 @@ +#include <stdio.h> +#include <stdlib.h> + + +void check(char expect[], char result[]) +{ + if (expect == result) + { + printf("Correct:expected %s got %s \n",expect,result); + } + else + { + printf("ERROR:expected %s got %s \n",expect,result); + exit (0); + } +} + +int main(void) +{ + char result[20]; + char A[20]=" pratham"; + char B[20]=" sir"; + result[20] = message(A); + printf("%s",result); + check("hello pratham", result); + result[20] = message(B); + check("hello sir",result); + printf("All Correct\n"); +} diff --git a/testapp/c_cpp_files/main_palindrome.cpp b/testapp/c_cpp_files/main_palindrome.cpp new file mode 100755 index 0000000..a5d67b5 --- /dev/null +++ b/testapp/c_cpp_files/main_palindrome.cpp @@ -0,0 +1,30 @@ +#include <stdio.h> +#include <stdlib.h> + +extern bool palindrome(int); + +template <class T> + +void check(T expect, T result) +{ + if (expect == result) + { + //printf("Correct:\n Expected %d got %d \n",expect,result); + } + else + { + printf("\nIncorrect:\n Expected %d got %d \n",expect,result); + exit (1); + } +} + +int main(void) +{ + bool result; + result = palindrome(123); + check(false, result); + result = palindrome(121); + check(true, result); + printf("All Correct\n"); + return 0; +} diff --git a/testapp/c_cpp_files/sample.c b/testapp/c_cpp_files/sample.c new file mode 100755 index 0000000..660f862 --- /dev/null +++ b/testapp/c_cpp_files/sample.c @@ -0,0 +1,20 @@ +#include<stdio.h> + +void main(int argc , char * argv[]) +{ + int i,sum=0; + +/* if(argc!=3) + { + printf("you have forgot to type numbers."); + exit(1); + } +*/ + printf("The sum is : "); + + for(i=1;i<argc;i++) + sum = sum + atoi(argv[i]); + + printf("%d",sum); + +} diff --git a/testapp/c_cpp_files/sample.cpp b/testapp/c_cpp_files/sample.cpp new file mode 100755 index 0000000..ab77b85 --- /dev/null +++ b/testapp/c_cpp_files/sample.cpp @@ -0,0 +1,7 @@ +#include <iostream> +#include<cstdlib> +int main(int argc, char* argv[]) +{ + std::cout << atoi(argv[1])+atoi(argv[2]) << std::endl; + return 0; +} diff --git a/testapp/code_server.py b/testapp/code_server.py index 1276c76..d6ae75d 100755 --- a/testapp/code_server.py +++ b/testapp/code_server.py @@ -1,13 +1,13 @@ #!/usr/bin/env python """This server runs an XMLRPC server that can be submitted code and tests -and returns the output. It *should* be run as root and will run as the user -'nobody' so as to minimize any damange by errant code. This can be configured +and returns the output. It *should* be run as root and will run as the user +'nobody' so as to minimize any damange by errant code. This can be configured by editing settings.py to run as many servers as desired. One can also specify the ports on the command line. Here are examples:: $ sudo ./code_server.py # Runs servers based on settings.py:SERVER_PORTS one server per port given. - + or:: $ sudo ./code_server.py 8001 8002 8003 8004 8005 @@ -34,6 +34,7 @@ from settings import SERVER_PORTS, SERVER_TIMEOUT, SERVER_POOL_PORT MY_DIR = abspath(dirname(__file__)) + def run_as_nobody(): """Runs the current process as nobody.""" # Set the effective uid and to that of nobody. @@ -46,15 +47,16 @@ def run_as_nobody(): # c.f. http://pguides.net/python/timeout-a-function class TimeoutException(Exception): pass - + + def timeout_handler(signum, frame): """A handler for the ALARM signal.""" raise TimeoutException('Code took too long to run.') -################################################################################ +############################################################################### # `CodeServer` class. -################################################################################ +############################################################################### class CodeServer(object): """A code server that executes user submitted test code, tests it and reports if the code was correct or not. @@ -63,29 +65,29 @@ class CodeServer(object): self.port = port self.queue = queue msg = 'Code took more than %s seconds to run. You probably '\ - 'have an infinite loop in your code.'%SERVER_TIMEOUT + 'have an infinite loop in your code.' % SERVER_TIMEOUT self.timeout_msg = msg def run_python_code(self, answer, test_code, in_dir=None): - """Tests given Python function (`answer`) with the `test_code` supplied. - If the optional `in_dir` keyword argument is supplied it changes the - directory to that directory (it does not change it back to the original when - done). This function also timesout when the function takes more than - SERVER_TIMEOUT seconds to run to prevent runaway code. - + """Tests given Python function (`answer`) with the `test_code` + supplied. If the optional `in_dir` keyword argument is supplied + it changes the directory to that directory (it does not change + it back to the original when done). This function also timesout + when the function takes more than SERVER_TIMEOUT seconds to run + to prevent runaway code. Returns ------- - + A tuple: (success, error message). - + """ if in_dir is not None and isdir(in_dir): os.chdir(in_dir) - + # Add a new signal handler for the execution of this code. old_handler = signal.signal(signal.SIGALRM, timeout_handler) signal.alarm(SERVER_TIMEOUT) - + success = False tb = None try: @@ -111,8 +113,8 @@ class CodeServer(object): finally: del tb # Set back any original signal handler. - signal.signal(signal.SIGALRM, old_handler) - + signal.signal(signal.SIGALRM, old_handler) + # Cancel the signal if any, see signal.alarm documentation. signal.alarm(0) @@ -133,25 +135,26 @@ class CodeServer(object): If not, we assume they are relative paths w.r.t. the location of this code_server script. - If the optional `in_dir` keyword argument is supplied it changes the - directory to that directory (it does not change it back to the original when - done). + If the optional `in_dir` keyword argument is supplied it changes the + directory to that directory (it does not change it back to the original + when done). Returns ------- - + A tuple: (success, error message). - + """ if in_dir is not None and isdir(in_dir): os.chdir(in_dir) def _set_exec(fname): - os.chmod(fname, stat.S_IRUSR|stat.S_IWUSR|stat.S_IXUSR - |stat.S_IRGRP|stat.S_IWGRP|stat.S_IXGRP - |stat.S_IROTH|stat.S_IWOTH|stat.S_IXOTH) + os.chmod(fname, stat.S_IRUSR | stat.S_IWUSR | stat.S_IXUSR + | stat.S_IRGRP | stat.S_IWGRP | stat.S_IXGRP + | stat.S_IROTH | stat.S_IWOTH | stat.S_IXOTH) submit_f = open('submit.sh', 'w') - submit_f.write(answer.lstrip()); submit_f.close() + submit_f.write(answer.lstrip()) + submit_f.close() submit_path = abspath(submit_f.name) _set_exec(submit_path) @@ -168,7 +171,8 @@ class CodeServer(object): # Do whatever testing needed. success = False try: - success, err = self.check_bash_script(ref_path, submit_path, test_case_path) + success, err = self.check_bash_script(ref_path, submit_path, + test_case_path) except TimeoutException: err = self.timeout_msg except: @@ -176,7 +180,7 @@ class CodeServer(object): err = "Error: {0}".format(repr(value)) finally: # Set back any original signal handler. - signal.signal(signal.SIGALRM, old_handler) + signal.signal(signal.SIGALRM, old_handler) # Delete the created file. os.remove(submit_path) @@ -201,10 +205,12 @@ class CodeServer(object): # Runaway code, so kill it. proc.kill() # Re-raise exception. - raise + #raise + stderr = self.timeout_msg + stdout = '' return proc, stdout, stderr - def check_bash_script(self, ref_script_path, submit_script_path, + def check_bash_script(self, ref_script_path, submit_script_path, test_case_path=None): """ Function validates student script using instructor script as reference. Test cases can optionally be provided. The first argument @@ -214,72 +220,297 @@ class CodeServer(object): permission. The Third optional argument is the path to test the scripts. Each line in this file is a test case and each test case is passed to the script as standard arguments. - + Returns -------- - returns (True, "Correct answer") : If the student script passes all test - cases/have same output, when compared to the instructor script - - returns (False, error_msg): If - the student script fails a single test/have dissimilar output, when - compared to the instructor script. - - Returns (False, error_msg): If mandatory arguments are not files or if the - required permissions are not given to the file(s). - + returns (True, "Correct answer") : If the student script passes all + test cases/have same output, when compared to the instructor script + + returns (False, error_msg): If the student script fails a single + test/have dissimilar output, when compared to the instructor script. + + Returns (False, error_msg): If mandatory arguments are not files or if + the required permissions are not given to the file(s). + """ if not isfile(ref_script_path): - return False, "No file at %s"%ref_script_path + return False, "No file at %s" % ref_script_path if not isfile(submit_script_path): - return False, 'No file at %s'%submit_script_path + return False, 'No file at %s' % submit_script_path if not os.access(ref_script_path, os.X_OK): - return False, 'Script %s is not executable'%ref_script_path + return False, 'Script %s is not executable' % ref_script_path if not os.access(submit_script_path, os.X_OK): - return False, 'Script %s is not executable'%submit_script_path - + return False, 'Script %s is not executable' % submit_script_path + if test_case_path is None: ret = self._run_command(ref_script_path, stdin=None, - stdout=subprocess.PIPE, stderr=subprocess.PIPE) + stdout=subprocess.PIPE, + stderr=subprocess.PIPE) proc, inst_stdout, inst_stderr = ret - ret = self._run_command(submit_script_path, stdin=None, - stdout=subprocess.PIPE, stderr=subprocess.PIPE) + ret = self._run_command(submit_script_path, stdin=None, + stdout=subprocess.PIPE, + stderr=subprocess.PIPE) proc, stdnt_stdout, stdnt_stderr = ret if inst_stdout == stdnt_stdout: return True, 'Correct answer' else: - err = "Error: expected %s, got %s"%(inst_stderr, stdnt_stderr) + err = "Error: expected %s, got %s" % (inst_stderr, + stdnt_stderr) return False, err else: if not isfile(test_case_path): - return False, "No test case at %s"%test_case_path + return False, "No test case at %s" % test_case_path if not os.access(ref_script_path, os.R_OK): - return False, "Test script %s, not readable"%test_case_path - valid_answer = True # We initially make it one, so that we can stop - # once a test case fails - loop_count = 0 # Loop count has to be greater than or equal to one. - # Useful for caching things like empty test files,etc. + return False, "Test script %s, not readable" % test_case_path + valid_answer = True # We initially make it one, so that we can + # stop once a test case fails + loop_count = 0 # Loop count has to be greater than or + # equal to one. + # Useful for caching things like empty + # test files,etc. test_cases = open(test_case_path).readlines() num_lines = len(test_cases) for test_case in test_cases: loop_count += 1 if valid_answer: - args = [ ref_script_path ] + [x for x in test_case.split()] - ret = self._run_command(args, stdin=None, - stdout=subprocess.PIPE, stderr=subprocess.PIPE) + args = [ref_script_path] + [x for x in test_case.split()] + ret = self._run_command(args, stdin=None, + stdout=subprocess.PIPE, + stderr=subprocess.PIPE) proc, inst_stdout, inst_stderr = ret - args = [ submit_script_path ] + [x for x in test_case.split()] - ret = self._run_command(args, stdin=None, - stdout=subprocess.PIPE, stderr=subprocess.PIPE) + args = [submit_script_path]+[x for x in test_case.split()] + ret = self._run_command(args, stdin=None, + stdout=subprocess.PIPE, + stderr=subprocess.PIPE) proc, stdnt_stdout, stdnt_stderr = ret valid_answer = inst_stdout == stdnt_stdout if valid_answer and (num_lines == loop_count): return True, "Correct answer" else: - err = "Error: expected %s, got %s"%(inst_stdout+inst_stderr, - stdnt_stdout+stdnt_stderr) + err = "Error:expected %s, got %s" % (inst_stdout+inst_stderr, + stdnt_stdout+stdnt_stderr) return False, err + def run_c_code(self, answer, test_code, in_dir=None): + """Tests given C code (`answer`) with the `test_code` supplied. + + The testcode is a path to the reference code. + The reference code will call the function submitted by the student. + The reference code will check for the expected output. + + If the path's start with a "/" then we assume they are absolute paths. + If not, we assume they are relative paths w.r.t. the location of this + code_server script. + + If the optional `in_dir` keyword argument is supplied it changes the + directory to that directory (it does not change it back to the original + when done). + + Returns + ------- + + A tuple: (success, error message). + + """ + if in_dir is not None and isdir(in_dir): + os.chdir(in_dir) + + # File extension must be .c + submit_f = open('submit.c', 'w') + submit_f.write(answer.lstrip()) + submit_f.close() + submit_path = abspath(submit_f.name) + + ref_path = test_code.strip() + if not ref_path.startswith('/'): + ref_path = join(MY_DIR, ref_path) + + # Add a new signal handler for the execution of this code. + old_handler = signal.signal(signal.SIGALRM, timeout_handler) + signal.alarm(SERVER_TIMEOUT) + + # Do whatever testing needed. + success = False + try: + success, err = self._check_c_cpp_code(ref_path, submit_path) + except TimeoutException: + err = self.timeout_msg + except: + type, value = sys.exc_info()[:2] + err = "Error: {0}".format(repr(value)) + finally: + # Set back any original signal handler. + signal.signal(signal.SIGALRM, old_handler) + + # Delete the created file. + os.remove(submit_path) + + # Cancel the signal if any, see signal.alarm documentation. + signal.alarm(0) + + # Put us back into the server pool queue since we are free now. + self.queue.put(self.port) + + return success, err + + def _compile_command(self, cmd, *args, **kw): + """Compiles C/C++ code and returns errors if any. + Run a command in a subprocess while blocking, the process is killed + if it takes more than 2 seconds to run. Return the Popen object, the + stderr. + """ + try: + proc_compile = subprocess.Popen(cmd, shell=True, stdin=None, + stdout=subprocess.PIPE, + stderr=subprocess.PIPE) + out, err = proc_compile.communicate() + except TimeoutException: + # Runaway code, so kill it. + proc_compile.kill() + # Re-raise exception. + raise + return proc_compile, err + + def _check_c_cpp_code(self, ref_code_path, submit_code_path): + """ Function validates student code using instructor code as + reference.The first argument ref_code_path, is the path to + instructor code, it is assumed to have executable permission. + The second argument submit_code_path, is the path to the student + code, it is assumed to have executable permission. + + Returns + -------- + + returns (True, "Correct answer") : If the student function returns + expected output when called by reference code. + + returns (False, error_msg): If the student function fails to return + expected output when called by reference code. + + Returns (False, error_msg): If mandatory arguments are not files or + if the required permissions are not given to the file(s). + + """ + if not isfile(ref_code_path): + return False, "No file at %s" % ref_code_path + if not isfile(submit_code_path): + return False, 'No file at %s' % submit_code_path + + success = False + output_path = os.getcwd() + '/output' + compile_command = "g++ %s -c -o %s" % (submit_code_path, output_path) + ret = self._compile_command(compile_command, stdin=None, + stdout=subprocess.PIPE, + stderr=subprocess.PIPE) + proc, inst_stderr = ret + + # Only if compilation is successful, the program is executed + # And tested with testcases + if inst_stderr == '': + executable = os.getcwd() + '/executable' + compile_main = "g++ %s %s -o %s" % (ref_code_path, output_path, + executable) + ret = self._compile_command(compile_main, stdin=None, + stdout=subprocess.PIPE, + stderr=subprocess.PIPE) + proc, main_err = ret + if main_err == '': + args = [executable] + ret = self._run_command(args, stdin=None, + stdout=subprocess.PIPE, + stderr=subprocess.PIPE) + proc, stdout, stderr = ret + if proc.returncode == 0: + success, err = True, "Correct answer" + else: + err = stdout + "\n" + stderr + success = False + os.remove(executable) + else: + err = "Error:" + try: + error_lines = main_err.splitlines() + for e in error_lines: + err = err + "\n" + e.split(":", 1)[1] + except: + err = err + "\n" + main_err + success = False + os.remove(output_path) + else: + err = "Compilation Error:" + try: + error_lines = inst_stderr.splitlines() + for e in error_lines: + if ':' in e: + err = err + "\n" + e.split(":", 1)[1] + else: + err = err + "\n" + e + except: + err = err + "\n" + inst_stderr + return success, err + + def run_cplus_code(self, answer, test_code, in_dir=None): + """Tests given C++ code (`answer`) with the `test_code` supplied. + + The testcode is a path to the reference code. + The reference code will call the function submitted by the student. + The reference code will check for the expected output. + + If the path's start with a "/" then we assume they are absolute paths. + If not, we assume they are relative paths w.r.t. the location of this + code_server script. + + If the optional `in_dir` keyword argument is supplied it changes the + directory to that directory (it does not change it back to the original + when done). + + Returns + ------- + + A tuple: (success, error message). + + """ + if in_dir is not None and isdir(in_dir): + os.chdir(in_dir) + + # The file extension must be .cpp + submit_f = open('submitstd.cpp', 'w') + submit_f.write(answer.lstrip()) + submit_f.close() + submit_path = abspath(submit_f.name) + + ref_path = test_code.strip() + if not ref_path.startswith('/'): + ref_path = join(MY_DIR, ref_path) + + # Add a new signal handler for the execution of this code. + old_handler = signal.signal(signal.SIGALRM, timeout_handler) + signal.alarm(SERVER_TIMEOUT) + + # Do whatever testing needed. + success = False + try: + success, err = self._check_c_cpp_code(ref_path, submit_path) + except TimeoutException: + err = self.timeout_msg + except: + type, value = sys.exc_info()[:2] + err = "Error: {0}".format(repr(value)) + finally: + # Set back any original signal handler. + signal.signal(signal.SIGALRM, old_handler) + + # Delete the created file. + os.remove(submit_path) + + # Cancel the signal if any, see signal.alarm documentation. + signal.alarm(0) + + # Put us back into the server pool queue since we are free now. + self.queue.put(self.port) + + return success, err def run(self): """Run XMLRPC server, serving our methods. @@ -291,9 +522,9 @@ class CodeServer(object): server.serve_forever() -################################################################################ +############################################################################### # `ServerPool` class. -################################################################################ +############################################################################### class ServerPool(object): """Manages a pool of CodeServer objects.""" def __init__(self, ports, pool_port=50000): @@ -330,7 +561,8 @@ class ServerPool(object): port = q.get() if was_waiting: print '*'*80 - print "No available servers, was waiting but got server later at %d."%port + print "No available servers, was waiting but got server \ + later at %d." % port print '*'*80 sys.stdout.flush() return port @@ -345,7 +577,7 @@ class ServerPool(object): server.serve_forever() -################################################################################ +############################################################################### def main(): run_as_nobody() if len(sys.argv) == 1: diff --git a/testapp/exam/forms.py b/testapp/exam/forms.py index 7bbf601..1b60a71 100644 --- a/testapp/exam/forms.py +++ b/testapp/exam/forms.py @@ -1,5 +1,5 @@ from django import forms -from exam.models import Profile,Quiz,Question +from exam.models import Profile, Quiz, Question from django.contrib.auth import authenticate from django.contrib.auth.models import User @@ -13,14 +13,17 @@ from string import letters, punctuation, digits import datetime QUESTION_TYPE_CHOICES = ( - ("python", "Python"), - ("bash", "Bash"), - ("mcq", "MCQ"), - ) + ("python", "Python"), + ("bash", "Bash"), + ("mcq", "MCQ"), + ("C", "C Language"), + ("C++", "C++ Language"), + ) UNAME_CHARS = letters + "._" + digits PWD_CHARS = letters + punctuation + digits + class UserRegisterForm(forms.Form): """A Class to create new form for User's Registration. It has the various fields and functions required to register @@ -28,7 +31,7 @@ class UserRegisterForm(forms.Form): username = forms.CharField\ (max_length=30, help_text='Letters, digits,\ - period and underscores only.') + period and underscores only.') email = forms.EmailField() password = forms.CharField(max_length=30, widget=forms.PasswordInput()) confirm_password = forms.CharField\ @@ -114,20 +117,20 @@ class UserLoginForm(forms.Form): class QuizForm(forms.Form): """Creates a form to add or edit a Quiz. It has the related fields and functions required.""" - + start_date = forms.DateField(initial=datetime.date.today) duration = forms.IntegerField() - active = forms.BooleanField(required = False) + active = forms.BooleanField(required=False) tags = TagField(widget=TagAutocomplete()) description = forms.CharField(max_length=256, widget=forms.Textarea\ - (attrs={'cols':20,'rows':1})) + (attrs={'cols': 20, 'rows': 1})) def save(self): start_date = self.cleaned_data["start_date"] duration = self.cleaned_data["duration"] active = self.cleaned_data['active'] description = self.cleaned_data["description"] - + new_quiz = Quiz() new_quiz.start_date = start_date new_quiz.duration = duration @@ -136,7 +139,7 @@ class QuizForm(forms.Form): new_quiz.save() class QuestionForm(forms.Form): - """Creates a form to add or edit a Question. + """Creates a form to add or edit a Question. It has the related fields and functions required.""" summary = forms.CharField(widget=forms.Textarea\ @@ -147,13 +150,13 @@ class QuestionForm(forms.Form): test = forms.CharField(widget=forms.Textarea\ (attrs={'cols': 40, 'rows': 1})) options = forms.CharField(widget=forms.Textarea\ - (attrs={'cols': 40, 'rows': 1}),required=False) + (attrs={'cols': 40, 'rows': 1}), required=False) type = forms.CharField(max_length=8, widget=forms.Select\ - (choices=QUESTION_TYPE_CHOICES)) + (choices=QUESTION_TYPE_CHOICES)) active = forms.BooleanField(required=False) - tags = TagField(widget=TagAutocomplete(),required=False) + tags = TagField(widget=TagAutocomplete(), required=False) snippet = forms.CharField(widget=forms.Textarea\ - (attrs={'cols': 40, 'rows': 1}),required=False) + (attrs={'cols': 40, 'rows': 1}), required=False) def save(self): summary = self.cleaned_data["summary"] diff --git a/testapp/exam/models.py b/testapp/exam/models.py index 00d32e4..fac01ec 100644 --- a/testapp/exam/models.py +++ b/testapp/exam/models.py @@ -18,7 +18,9 @@ QUESTION_TYPE_CHOICES = ( ("python", "Python"), ("bash", "Bash"), ("mcq", "MultipleChoice"), - ) + ("C", "C Language"), + ("C++", "C++ Language"), + ) ################################################################################ class Question(models.Model): @@ -233,4 +235,3 @@ the next.""" def __unicode__(self): u = self.user return u'Question paper for {0} {1}'.format(u.first_name, u.last_name) - diff --git a/testapp/exam/xmlrpc_clients.py b/testapp/exam/xmlrpc_clients.py index 817e37d..b846212 100644 --- a/testapp/exam/xmlrpc_clients.py +++ b/testapp/exam/xmlrpc_clients.py @@ -9,19 +9,24 @@ from settings import SERVER_PORTS, SERVER_POOL_PORT class ConnectionError(Exception): pass -################################################################################ +############################################################################### # `CodeServerProxy` class. -################################################################################ +############################################################################### + + class CodeServerProxy(object): """A class that manages accesing the farm of Python servers and making calls to them such that no one XMLRPC server is overloaded. """ def __init__(self): - pool_url = 'http://localhost:%d'%(SERVER_POOL_PORT) + pool_url = 'http://localhost:%d' % (SERVER_POOL_PORT) self.pool_server = ServerProxy(pool_url) self.methods = {"python": 'run_python_code', - "bash": 'run_bash_code'} - + "bash": 'run_bash_code', + "C": "run_c_code", + "C++": "run_cplus_code", + } + def run_code(self, answer, test_code, user_dir, language): """Tests given code (`answer`) with the `test_code` supplied. If the optional `in_dir` keyword argument is supplied it changes the directory @@ -68,11 +73,10 @@ class CodeServerProxy(object): else: done = True if not done: - raise ConnectionError("Couldn't connect to a server!") - proxy = ServerProxy('http://localhost:%d'%port) + raise ConnectionError("Couldn't connect to a server!") + proxy = ServerProxy('http://localhost:%d' % port) return proxy # views.py calls this Python server which forwards the request to one # of the running servers. code_server = CodeServerProxy() - diff --git a/testapp/templates/exam/question.html b/testapp/templates/exam/question.html index 1d801b0..11ea1aa 100644 --- a/testapp/templates/exam/question.html +++ b/testapp/templates/exam/question.html @@ -76,8 +76,13 @@ function update_time() <h4><u> {{ question.summary }} </u><font class=pull-right>(Marks : {{ question.points }}) </font></h4><br> <font size=3 face=arial> {{ question.description|safe }} </font> - - {% if error_message %}<h5>ERROR:</h5><div class="alert alert-error">{{ error_message }}</div>{% endif %} + {% if error_message %} + <div class="alert alert-error"> + {% for e in error_message.splitlines %} + {{ e|join:"" }} + <br/> + {% endfor%} + </div>{% endif %} <p id="status"></p> <form id="code" action="{{URL_ROOT}}/exam/{{ question.id }}/check/{{ paper.question_paper.id }}/" method="post"> {% csrf_token %} diff --git a/testapp/test_server.py b/testapp/test_server.py index 0790816..f2ea84c 100644 --- a/testapp/test_server.py +++ b/testapp/test_server.py @@ -6,42 +6,124 @@ up the code server as:: """ from exam.xmlrpc_clients import code_server + def check_result(result, check='correct answer'): - if check != 'correct answer': - assert result[0] == False - else: - assert result[0] == True - if "unable to connect" in result[1].lower(): + if check != 'correct answer': + assert result[0] == False + else: + assert result[0] == True + if "unable to connect" in result[1].lower(): assert result[0], result[1] - assert check in result[1].lower(), result[1] + assert check in result[1].lower(), result[1] def test_python(): """Test if server runs Python code as expected.""" src = 'while True: pass' result = code_server.run_code(src, '', '/tmp', language="python") check_result(result, 'more than ') - src = 'x = 1' - result = code_server.run_code(src, 'assert x == 1', '/tmp', + result = code_server.run_code(src, 'assert x == 1', '/tmp', language="python") check_result(result, 'correct answer') - result = code_server.run_code(src, 'assert x == 0', '/tmp', + result = code_server.run_code(src, 'assert x == 0', '/tmp', language="python") check_result(result, 'assertionerror') src = 'abracadabra' - result = code_server.run_code(src, 'assert x == 0', '/tmp', + result = code_server.run_code(src, 'assert x == 0', '/tmp', language="python") check_result(result, 'nameerror') + +def test_c(): + """Test if server runs c code as expected.""" + src = """ + #include<stdiol.h> + int ad(int a, int b) + {return a+b;} + """ + result = code_server.run_code(src, 'c_cpp_files/main.cpp', '/tmp', language="C") + check_result(result, 'error') + + src = """ + int add(int a, int b) + {return a+b} + """ + result = code_server.run_code(src, 'c_cpp_files/main.cpp', '/tmp', language="C") + check_result(result, 'compilation error') + + src = """ + int add(int a, int b) + {while(1>0){} + return a+b;} + """ + result = code_server.run_code(src, 'c_cpp_files/main.cpp', '/tmp', language="C") + check_result(result, 'more than') + + src = """ + int add(int a, int b) + {return a+b;} + """ + result = code_server.run_code(src, 'c_cpp_files/main.cpp', '/tmp', language="C") + check_result(result, 'correct answer') + + src = """ + #include<stdio.h> + int add(int a, int b) + {printf("All Correct");} + """ + result = code_server.run_code(src, 'c_cpp_files/main.cpp', '/tmp', language="C") + check_result(result, 'incorrect') + + +def test_cpp(): + """Test if server runs c code as expected.""" + src = """ + int add(int a, int b) + { + return a+b + } + """ + result = code_server.run_code(src, 'c_cpp_files/main.cpp', '/tmp', language="C++") + check_result(result, 'compilation error') + + src = """ + int add(int a, int b) + { + return a+b; + } + """ + result = code_server.run_code(src, 'c_cpp_files/main.cpp', '/tmp', language="C++") + check_result(result, 'correct answer') + + src = """ + int dd(int a, int b) + { + return a+b; + } + """ + result = code_server.run_code(src, 'c_cpp_files/main.cpp', '/tmp', language="C++") + check_result(result, 'error') + + src = """ + int add(int a, int b) + { + while(0==0) + {} + return a+b; + } + """ + result = code_server.run_code(src, 'c_cpp_files/main.cpp', '/tmp', language="C++") + check_result(result, 'more than') + def test_bash(): """Test if server runs Bash code as expected.""" src = """ #!/bin/bash [[ $# -eq 2 ]] && echo $(( $1 + $2 )) && exit $(( $1 + $2 )) """ - result = code_server.run_code(src, + result = code_server.run_code(src, 'docs/sample.sh\ndocs/sample.args', '/tmp', language="bash") check_result(result) @@ -49,7 +131,7 @@ def test_bash(): #!/bin/bash [[ $# -eq 2 ]] && echo $(( $1 - $2 )) && exit $(( $1 - $2 )) """ - result = code_server.run_code(src, + result = code_server.run_code(src, 'docs/sample.sh\ndocs/sample.args', '/tmp', language="bash") check_result(result, 'error') @@ -57,7 +139,7 @@ def test_bash(): #!/bin/bash while [ 1 ] ; do echo "" > /dev/null ; done """ - result = code_server.run_code(src, + result = code_server.run_code(src, 'docs/sample.sh\ndocs/sample.args', '/tmp', language="bash") check_result(result, 'more than ') @@ -65,15 +147,15 @@ def test_bash(): #!/bin/bash while [ 1 ] ; do echo "" > /dev/null ''' - result = code_server.run_code(src, - 'docs/sample.sh\ndocs/sample.args', '/tmp', language="bash") + result = code_server.run_code(src, + 'docs/sample.sh\ndocs/sample.args', '/tmp', language="bash") check_result(result, 'error') src = '''# Enter your code here. #!/bin/bash while [ 1 ] ; do echo "" > /dev/null ''' - result = code_server.run_code(src, + result = code_server.run_code(src, 'docs/sample.sh\ndocs/sample.args', '/tmp', language="bash") check_result(result, 'oserror') @@ -81,3 +163,4 @@ if __name__ == '__main__': test_python() test_bash() test_c() + test_cpp() |