如何使用Mocha测试Meteor.users

Question

I have a function as follows:

if(Meteor.isServer) {
Meteor.methods({
    addUser: function (newUser) {
        check(newUser, { email: String, password: String });
        userId = Accounts.createUser(newUser);
        return userId;
    },
    getUser: function (userID) {
        check(userID, String);
        return Meteor.users.find({_id: userID}).fetch();
    }
});

And I am trying to test this function using Mocha:

if (Meteor.isServer) {
let testUser;
describe('Users', () => {
    it("Add User", (done) => {

        testUser = {email: 'test@test.test', password: 'test'};

        try {
            testUser._id = Meteor.call('addUser', testUser);
            console.log(Accounts.users.find({_id: testUser._id}).fetch());
            done();
        } catch (err) {
            assert.fail();
        }
    });

    it("Get user", (done) => {
        try {
            Meteor.call('getUser', testUser._id);
            done();
        } catch (err) {
            assert.fail();
        }
    });
});

And I know that the meteor call with 'addUser' works, because the console.log after that returns the user that I just made and the first test passes when I run it with "meteor test --driver-package practicalmeteor:mocha"

But then I come to the second testing part, where I try to get the user with the meteor call 'getUser', but then I get stuck:

'Cannot call method 'find' of undefined'

Now I know that the difference is that I use 'Meteor.users' instead of 'Account.users' to find the user, but I am totally in the dark what the difference is between these two. Should I replace all the Meteor.users method calls with Accounts.user method calls or not? How would you test this?

Answer 1

I just stumbled on this post, since I have dealt with the same issue some hours ago.

As I can see in your code, your testUser is defined in your first unit ( it("Add User"...){} ). I advise you not to use the value from the first unit in the second unit.

You may rather use beforeEach and afterEach to have a clean setup for each unit and then create a new user in the second test unit. You should also clean up your db after each unit:

describe('Users', () => {

    // use this for each test
    let testUser;


    beforeEach(() => {
        // let's always create a new clean testUser object
        testUser = {email: 'test@test.test', password: 'test'};
    });

    afterEach(() => {
        // Remove the user to keep our db clean
        if (testUser._id)
            Meteor.users.remove(testUser._id);
    });

    it("Add User", (done) => {
        testUser._id = Meteor.call('addUser', testUser);
        const users = Meteor.users.find({_id: testUser._id}).fetch();
        const user = users[0];
        assert.isNotNull(user);
        assert.isDefined(user);
        assert.equal(user._id, testUser._id);
        done();

    });

    it("Get user", (done) => {
        // get a new id from our previously tested
        // (and assume to be working) function
        testUser._id = Meteor.call('addUser', testUser);

        const user = Meteor.call('getUser', testUser._id);
        assert.isNotNull(user);
        assert.isDefined(user);
        assert.equal(user._id, testUser._id);
        done();
    });
});

I also found, that your 'getUser' method returns an array, so I changed it to:

getUser: function (userID) {
    check(userID, String);
    const users = Meteor.users.find({_id: userID}).fetch();
    return users[0];
}

All tested and running.