Top ten endpoint honors users with the same name (#379)

* Make the /top/10 endpoint honor users with the same name

tests/helpers.py

@@ -2,6 +2,7 @@ from CTFd import create_app
 from CTFd.models import *
 from sqlalchemy_utils import database_exists, create_database, drop_database
 from sqlalchemy.engine.url import make_url
+import datetime
 
 
 def create_ctfd(ctf_name="CTFd", name="admin", email="admin@ctfd.io", password="password", setup=True):
@@ -110,6 +111,7 @@ def gen_hint(db, chal, hint="This is a hint", cost=0, type=0):
 
 def gen_solve(db, teamid, chalid, ip='127.0.0.1', flag='rightkey'):
     solve = Solves(teamid, chalid, ip, flag)
+    solve.date = datetime.datetime.utcnow()
     db.session.add(solve)
     db.session.commit()
     return solve
@@ -117,6 +119,7 @@ def gen_solve(db, teamid, chalid, ip='127.0.0.1', flag='rightkey'):
 
 def gen_wrongkey(db, teamid, chalid, ip='127.0.0.1', flag='wrongkey'):
     wrongkey = WrongKeys(teamid, chalid, ip, flag)
+    wrongkey.date = datetime.datetime.utcnow()
     db.session.add(wrongkey)
     db.session.commit()
     return wrongkey

tests/user/test_scoreboard.py

@@ -0,0 +1,80 @@
+#!/usr/bin/env python
+# -*- coding: utf-8 -*-
+
+from CTFd.models import Teams, Solves, WrongKeys
+from CTFd.utils import get_config, set_config
+from CTFd import utils
+from tests.helpers import *
+from freezegun import freeze_time
+from mock import patch
+import json
+
+
+def test_top_10():
+    '''Make sure top10 returns correct information'''
+    app = create_ctfd()
+    with app.app_context():
+        register_user(app, name="user1", email="user1@ctfd.io")
+        register_user(app, name="user2", email="user2@ctfd.io")
+
+        chal1 = gen_challenge(app.db)
+        flag1 = gen_flag(app.db, chal=chal1.id, flag='flag')
+        chal1_id = chal1.id
+
+        chal2 = gen_challenge(app.db)
+        flag2 = gen_flag(app.db, chal=chal2.id, flag='flag')
+        chal2_id = chal2.id
+
+        # Generates solve for user1
+        with freeze_time("2017-10-3 03:21:34"):
+            gen_solve(app.db, teamid=2, chalid=chal1_id)
+
+        with freeze_time("2017-10-4 03:25:45"):
+            gen_solve(app.db, teamid=2, chalid=chal2_id)
+
+        # Generate solve for user2
+        with freeze_time("2017-10-3 03:21:34"):
+            gen_solve(app.db, teamid=3, chalid=chal1_id)
+
+        client = login_as_user(app)
+        r = client.get('/top/10')
+        response = r.get_data(as_text=True)
+
+        saved = '''{
+          "places": {
+            "1": {
+              "id": 2,
+              "name": "user1",
+              "solves": [
+                {
+                  "chal": 1,
+                  "team": 2,
+                  "time": 1507000894,
+                  "value": 100
+                },
+                {
+                  "chal": 2,
+                  "team": 2,
+                  "time": 1507087545,
+                  "value": 100
+                }
+              ]
+            },
+            "2": {
+              "id": 3,
+              "name": "user2",
+              "solves": [
+                {
+                  "chal": 1,
+                  "team": 3,
+                  "time": 1507000894,
+                  "value": 100
+                }
+              ]
+            }
+          }
+        }'''
+        saved = json.loads(saved)
+        received = json.loads(response)
+        assert saved == received
+    destroy_ctfd(app)