1723
84%

Ran 28 Feb 2020 08:36PM UTC

Jobs 1

Files 45

Run time 7s

Badge

Embed ▾

Committed 28 Feb 2020 08:24PM UTC coverage: 83.084% (+0.06%) from 83.022%

Build # 1723

Build Type

push

travis-ci

Committed by

web-flow

Commit Message

Eliminate bad use of const in Optional.transform (#587)

It's invalid to create a const which references a type variable. The
language team decided that in order to avoid painful migration of common
Dart 1 usages such as `const []`, rather than generate an error, they
would replace the type variable with `Null`, since usually, for properly
covariant types, the result 'works'.

In cases where `transform` is chained with e.g., `or`, you can end up
with `Optional<T>.absent` replacing `T` with `Null`, but then do an
implicit covariance check on the incoming type in `or`, which should be
the `S` in `transform` but is found to be `Null`.

As a result, we revert the move to const construction in `transform` and
`transformNullable`.

This reverts this bit of constification that was added in e769045.

Run Details

1778 of 2140 relevant lines covered (83.08%)

1.42 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
6	1723.6 (SHARD=coveralls)	28 Feb 2020 08:36PM UTC	0	83.08	Travis Job 1723.6

Source Files on build 1723

Detailed source file information is not available for this build.

google / quiver-dart / 1723
84%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 1723

google / quiver-dart / 1723 84%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 1723

google / quiver-dart / 1723
84%

README BADGES
x