Async cache won't store exceptions

[akmalviya03]

Earlier async cache was caching exceptions.
With new changes in fetch method of AsyncCache class will no longer store exception.
By default AsyncCache will store exception, by setting _canCacheException to false. It won't store exception.

Please refer dart-lang/core#366 for video demo of both issue and solution along with minimal reproducible code.

---

• I’ve reviewed the contributor guide and applied the relevant portions to this PR.

[google-cla]

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

[MDSADABWASIM]

Any update on this @akmalviya03 @lrhn

[lrhn]

SGTM.
@natebosch WDYT?

[natebosch]

@natebosch WDYT?

This one doesn't have the same issue with a Future field that loses it's expected association like #260

I don't think I have any concerns about landing this one.

They are interesting to consider together though - adding this capability might give stronger motivation for accepting #260

[akmalviya03]

@natebosch @lrhn Any update on this ?

[mosuem]

Closing as the dart-lang/async repository is merged into the dart-lang/core monorepo. Please re-open this PR there!