client.sadd not working inside sendDealIfNotSent function

If you execute the deal-metrics.js file in Chapter 2, you should get user:1,user:3 received all of the deals: deal:1,deal:2 instead of user:1 received all of the deals: deal:1,deal:2 because user:3 is added to deal:1 before you execute showUsersThatReceivedAllDeals(["deal:1", "deal:2"]);. Do you agree with me? If so, do you know how to fix it? If you don't agree, could you tell me why it omits user:3?