Glorious merge of code coverage and testem configuration

[Steve Lee]

Impressive work - congrats!
Steve Lee
OpenDirective http://opendirective.com


On 17 January 2018 at 19:13, Justin Obara <obara.justin at gmail.com> wrote:
> Hi Tony,
>
> I was thinking about this a bit more and maybe it doesn’t make sense. The
> test would only be for cases where the fluid object is missing, which really
> shouldn’t happen for these files. If we did want to test it, I suppose we’d
> have to have a separate test file where the fluid object isn’t included. I’m
> leaning towards just ignoring those branches, but am open to some other
> thoughts and suggestions.
>
> Also, we do have a manual test for running multiple versions of Infusion.
> https://build.fluidproject.org/infusion/tests/manual-tests/framework/core/multipleVersions/
>
> Thanks
> Justin
>
>
> On January 17, 2018 at 1:03:02 PM, Tony Atkins (tony at raisingthefloor.org)
> wrote:
>
> Hi, Justin.
>
> Could you elaborate on "add a test util for this that we plug into every
> test"?
>
> Thanks,
>
>
> Tony
>
> On 17 January 2018 at 17:31, Justin Obara <obara.justin at gmail.com> wrote:
>>
>> Thanks so much Tony et al for adding this. It’s really helpful to be able
>> to view the reports and see what are tests are actually covering.
>>
>> I’ve started using this for my most recent work and noticed that I was
>> getting a lower branch coverage than expected. I spoke with Tony about this
>> in the channel today. It seems this is because the
>>
>> var fluid_3_0_0 = fluid_3_0_0 || {};
>>
>> at the top of most of our files is treated as a branch because of the
>> conditional for the default value.
>>
>> I think we should either add a test util for this that we plug into every
>> test, or just instruct the code coverage to ignore it. If we go the ignore
>> route, we could follow this method
>>
>> var fluid_3_0_0 = fluid_3_0_0 || /* istanbul ignore next: the presence of
>> the fluid object can be inferred by the test running */ {};
>>
>>
>> https://github.com/gotwarlost/istanbul/blob/master/ignoring-code-for-coverage.md#ignore-default-assignments
>>
>> Thanks
>> Justin
>>
>>
>> On January 16, 2018 at 11:19:48 AM, Antranig Basman
>> (antranig.basman at colorado.edu) wrote:
>>
>> I'm happy to report the merge of a major and significantly useful pull
>> request
>> https://github.com/fluid-project/infusion/pull/822 from Tony Atkins which
>> adds long-awaited and much needed
>> code coverage and testem drivers to Infusion. As well as providing
>> amalgamated coverage reports derived from
>> both our browser and node-based tests, it provides an all-in-one testem
>> driver that will execute all such
>> tests in all available browsers and node.js.
>>
>> This will be invaluable to us in CI, and will shortly be a part of the
>> automated builds run on each pull
>> request. It will be important in future reviews to ensure that new work
>> has good coverage, coverage of
>> existing work does not degrade, and that we should work steadily to bring
>> areas of old implementation up to
>> reasonable standards of coverage (e.g. branch coverage of about 90%
>> represents a reasonable baseline. For
>> example we have interestingly poor coverage of jquery.keyboard-a11y.js via
>> automated tests.
>>
>> The coverage reports are generated automatically by running "npm test" and
>> can be accessed in the newly
>> generated "reports" directory. Machine-readable summaries of the coverage
>> are produced in "coverage".
>>
>> Let us all join in congratulating ADTKINS on almost a year of steering
>> this pull request to a successful
>> conclusion. Similar work is in progress for GPII projects but looks like
>> it will be blocked by moving those
>> projects over to a "monorepo" structure.
>>
>> Cheers,
>>
>> Antranig
>> _______________________________________________________
>> fluid-work mailing list - fluid-work at lists.idrc.ocad.ca
>> To unsubscribe, change settings or access archives,
>> see https://lists.idrc.ocad.ca/mailman/listinfo/fluid-work
>>
>>
>> _______________________________________________________
>> fluid-work mailing list - fluid-work at lists.idrc.ocad.ca
>> To unsubscribe, change settings or access archives,
>> see https://lists.idrc.ocad.ca/mailman/listinfo/fluid-work
>
>
>
> _______________________________________________________
> fluid-work mailing list - fluid-work at lists.idrc.ocad.ca
> To unsubscribe, change settings or access archives,
> see https://lists.idrc.ocad.ca/mailman/listinfo/fluid-work