Limit isn't applied when a string is informed

[wesleimarinho]

Do you want to request a feature or report a bug?
doubt

What is the current behavior?
If I use {limit:'2'} or limit('2') the option isn't apllied (or a variable with string value instead of Number).

Is this the exptected behavior?

There is no information about this on https://mongoosejs.com/docs/api/query.html#query_Query-limit, despite mentioning it is expected a number.

[IslandRhythms]

Works fine for me

const mongoose = require('mongoose');
const {Schema} = mongoose;

const testSchema = new Schema({
    file: Number
});

const Test = mongoose.model('Test', testSchema);

async function run() {
    await mongoose.connect('mongodb://localhost:27017/', {useNewUrlParser: true,
    useUnifiedTopology: true,});
    await mongoose.connection.dropDatabase();

    for (let i = 0; i < 20; i++) {
        await Test.create({file: i});
    }

    console.log(await Test.find().limit(5));
    console.log(await Test.find({},null,{limit: 5}));
}

run();

[wesleimarinho]

Proper test case, according to the issue description is:

const mongoose = require('mongoose');
const {Schema} = mongoose;

const testSchema = new Schema({
    file: Number
});

const Test = mongoose.model('Test', testSchema);

async function run() {
    await mongoose.connect('mongodb://localhost:27017/', {useNewUrlParser: true,
    useUnifiedTopology: true,});
    await mongoose.connection.dropDatabase();

    for (let i = 0; i < 20; i++) {
        await Test.create({file: i});
    }

    console.log(await Test.find().limit(String.valueOf(5)));
    console.log(await Test.find({},null,{limit: '5'}));
}

run();

[IslandRhythms]

It expects a number, so putting a string is an improper use.