Async UnhandledPromiseRejection Warning

Version imprimable

Bonjour,

Je suis entrain d'utiliser Promise.reject
Et j'ai cet erreur qui s'affiche:

Citation:

Unhandled promise rejection warning: version is not released

Comment je peux résoudre ce warning?
J'essaye d'utiliser try/catch mais je sais pas si je le fais correctement

Merci pour votre aide :)

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 
public async retrieveVersionFromJira(versionName: string): 
 Promise<ReleaseVersion> {
    try{
    const searchVersionsUri = config.jiraApiUri + 'versions';
    const jsonResp = await this.jiraClient.get(searchVersionsUri);
    const version: any = jsonResp.find(version => {
        if (version.name == versionName) {
            if (version.released == true) {            
                  return Promise.reject("version " + versionName + " is not released");
 
            }
        }
    });
    if (!version) {
        return Promise.reject("missing version " + versionName + " on jira");
    }
    return new ReleaseVersion(version.id, version.name, version.released);
}
catch{
   return Promise.reject("error test")
}
}

29/08/2019, 20h39
danielhagnoul

:salut:

Il me semble que jiraClient c'est du Java, rien à voir avec du JS, mauvais forum.

Ça ressemble à du Typescript.

Ton process est bizarre. Si version.released == true tu envoies le warning. Ça devrait être l'inverse:

Code:

1
2
3
4
5
6
 
        if (version.name == versionName) {
            if (!version.released) {
                throw "version " + versionName + " is not released";
            }
        }

Et pour être plus logique, tu peux changer la structure de ton code:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
public async retrieveVersionFromJira(versionName: string): Promise<ReleaseVersion> {
  try {
    const searchVersionsUri = config.jiraApiUri + 'versions';
    const jsonResp = await this.jiraClient.get(searchVersionsUri);
    const version: any = jsonResp.find(version => version.name === versionName);
 
    if (!version) {
      return Promise.reject("missing version " + versionName + " on jira");
    }
    if (!version.released) {
      return Promise.reject("version " + versionName + " is not released");
    }
    return new ReleaseVersion(version.id, version.name, version.released);
  }
  catch (err) {
    return Promise.reject("error test " + err.message)
  }
}

Parce que dans l'état actuel de ton code, le find ne fait rien. Ici, tu effectues d'abord un find tout simple pour trouver la version. Puis tu fais tes deux tests (si la version est trouvée, si la version est released). C'est plus facile de bien séparer les étapes!

30/08/2019, 12h58
Marco46

C'est du TypeScript, j'ai déplacé dans le bon forum.
01/09/2019, 16h58
Paleo

Citation:

Envoyé par marcilles

Comment je peux résoudre ce warning?

Remplace tous les return Promise.reject par des throw new Error.