From 90510418a82f675292a0c794af2a3d7b7fe6d55e Mon Sep 17 00:00:00 2001 From: Max Schaefer Date: Wed, 23 Aug 2017 07:32:52 +0100 Subject: [PATCH] node_tests: Remove spurious assignments to `args`. In both cases, `args` is a local variable that goes out of scope immediately after the assignment. Since the variable isn't captured by a closure either, the assignment has no effect. --- frontend_tests/node_tests/dispatch.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/frontend_tests/node_tests/dispatch.js b/frontend_tests/node_tests/dispatch.js index 6742f612fd..7e0dc2f7ae 100644 --- a/frontend_tests/node_tests/dispatch.js +++ b/frontend_tests/node_tests/dispatch.js @@ -566,7 +566,7 @@ with_overrides(function (override) { var args = bot_stub.get_args('bot'); assert_same(args.bot, event.bot); - args = admin_stub.get_args('update_user_id', 'update_bot_data'); + admin_stub.get_args('update_user_id', 'update_bot_data'); }); }); @@ -579,7 +579,7 @@ with_overrides(function (override) { var args = bot_stub.get_args('email'); assert_same(args.email, event.bot.email); - args = admin_stub.get_args('update_user_id', 'update_bot_data'); + admin_stub.get_args('update_user_id', 'update_bot_data'); }); });