ashokdey / todo-rest

app/experiments/mongo-delete.js

const {MongoClient} = require('mongodb');


MongoClient.connect('mongodb://localhost:27017/JokesApp', (err, db) => {

    if (err){

        return console.log('failed to connect to mongodb server. ', err);

console.log looks like debug code. Are you sure you do not want to remove it?

    }


    console.log('Connected to MongoDB Server');


    //deleteMany - delete all the occcurences

    db.collection('Jokes').deleteMany({joke : 'Sample joke 2'}).then((result) => {

        console.log(result.result);

console.log looks like debug code. Are you sure you do not want to remove it?

    }, (err) => {

        cosole.log('Error deleting many. E : ', err)

The variable cosole seems to be never declared. If this is a global, consider adding a /** global: cosole */ comment.

    });


    //deleteOne - deletes only first occurence of all the occurences

    db.collection('Jokes').deleteOne({joke : 'Test 1'}).then((result) => {

        console.log(result.result);

console.log looks like debug code. Are you sure you do not want to remove it?

    });


    db.collection('Jokes').findOneAndDelete({joke : 'Test 2'}).then((doc) => {

        console.log(doc);

console.log looks like debug code. Are you sure you do not want to remove it?

    }, (err) => {

        console.log('Error find one and delete. E: ', err);

console.log looks like debug code. Are you sure you do not want to remove it?

    });

There is no return statement in this branch, but you do return something in other branches. Did you maybe miss it? If you do not want to return anything, consider adding return undefined; explicitly.

    //close the DB

    // db.close();

});