Avertissement Intellij Idea - "La promesse retournée est ignorée" avec aysnc/await

Question

J'utilise Express.js dans mon code avec Node.js v7.3. Dans ce cas, j'ai créé un User Router qui transmet les demandes à mon User Controller .

J'utilise async/await à l'intérieur des User Controller pour effectuer des appels asynchrones. Le problème est qu'IntelliJ me donne un avertissement disant que

La promesse retournée par login() est ignorée.

Le truc, c'est que je ne renvoie rien du tout du login() méthode.

Voici le code -

UserRouter.js

router.post('/login', function (req, res, next) {
    userController.login(req, res); // I get the warning here
});

UserController.js

exports.login = async function (req, res) {
    try {
        const verifiedUser = await someFunction(req.body.access_code);
        let user = await User.findOrCreateUser(verifiedUser);
        res.status(200).send(user);
    }
    catch (err) {
        res.status(400).send({success: false, error: err});
    }
};

Si j'écris la même méthode de connexion en utilisant uniquement des promesses natives, je n'obtiens pas cet avertissement. Ai-je mal compris quelque chose ou est-ce la faute d'IntelliJ ?

EDIT -

Merci à @Stephen, je comprends qu'une fonction asynchrone renvoie une promesse mais ne serait-il pas mieux qu'Intellij identifie que rien n'est renvoyé par la fonction asynchrone et n'affiche pas cet avertissement car lorsque j'enchaîne un .then() après le login() il fournit une fonction undefined dans le résultat final. Cela signifie que si nous ne retournons pas explicitement quelque chose de la fonction asynchrone, alors undefined est retourné ?

Answer 1

router.post('/login', function (req, res, next) {
    void userController.login(req, res); // I get the warning here
});

Vous devez utiliser l'opérateur "void".

Answer 2

El userController.login() renvoie une promesse, mais vous ne faites rien avec le résultat de la promesse en utilisant sa fonction then() fonction.

Par exemple :

userController.login(req, res).then(() => {
    // Do something after login is successful.
});

ou dans la syntaxe ES2017 :

await userController.login(req, res);

Si vous n'avez pas l'intention d'y faire quoi que ce soit, vous pouvez ignorer l'avertissement. L'avertissement est surtout présent parce que le fait de ne pas utiliser l'option then() sur une promesse est généralement une odeur de code.

Answer 3

Le problème est que je ne renvoie rien de la méthode login().

Une fonction déclarée "async" renvoie une Promise par définition. Voir par exemple https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/async_function

Cependant, l'avertissement IDEA n'est qu'une inspection. Vous pouvez appuyer sur "alt-enter, droite" sur l'avertissement et changer le niveau d'inspection pour faire disparaître l'avertissement. L'inspection se trouve dans la catégorie "JavaScript -> Probable bugs" et s'appelle "Result of method call returning a promise is ignored".

Answer 4

Si vous êtes vraiment maniaque comme moi et le then() n'est pas nécessaire mais vous avez besoin que l'avertissement disparaisse, une solution possible est :

functionWithAsync.error(console.error);

Answer 5

Une autre façon de se débarrasser de l'avertissement est de définir un fichier vide. then() :

userController.login(req, res); // <- Get the warning here

userController.login(req, res).then(); // <- No warning