fix: add timestamp to initial username history

src/user/create.js

@@ -55,7 +55,7 @@ module.exports = function (User) {
 
 		const bulkAdd = [
 			['username:uid', userData.uid, userData.username],
-			['user:' + userData.uid + ':usernames', timestamp, userData.username],
+			['user:' + userData.uid + ':usernames', timestamp, userData.username + ':' + timestamp],
 			['username:sorted', 0, userData.username.toLowerCase() + ':' + userData.uid],
 			['userslug:uid', userData.uid, userData.userslug],
 			['users:joindate', timestamp, userData.uid],

test/user.js

@@ -1137,17 +1137,12 @@ describe('User', function () {
 		});
 
 
-		it('should get history from set', function (done) {
+		it('should get history from set', async function () {
-			var now = Date.now();
+			const now = Date.now();
-			db.sortedSetAdd('user:' + testUid + ':usernames', now, 'derp:' + now, function (err) {
+			await db.sortedSetAdd('user:' + testUid + ':usernames', now, 'derp:' + now);
-				assert.ifError(err);
+			const data = await User.getHistory('user:' + testUid + ':usernames');
-				User.getHistory('user:' + testUid + ':usernames', function (err, data) {
+			assert.equal(data[0].value, 'derp');
-					assert.ifError(err);
+			assert.equal(data[0].timestamp, now);
-					assert.equal(data[0].value, 'derp');
-					assert.equal(data[0].timestamp, now);
-					done();
-				});
-			});
 		});
 
 		it('should return the correct ban reason', function (done) {