digitalbazaar / mocha-w3c-interop-reporter

Mocha W3C Interoperability Test Suite reporter
BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

Allow linking test titles to specs. #37

Closed BigBlueHat closed 2 months ago

BigBlueHat commented 3 months ago

Rather than just having this at the comment level, we can move these into the test data itself.

The links to add would need to be added to the test data in a similar way as test.cell stuff is added here: https://github.com/w3c/vc-data-model-2.0-test-suite/blob/main/tests/10-vcdm2.js#L40-L43

Not sure that's ideal...but it does work.

I can provide readme updates, if y'all think this approach is worth doing.

The easy way to add the data to a test is just...

it('test', async function() {
  this.test.link = 'https://example.com/';
});

It's a little distracting...but works.

BigBlueHat commented 3 months ago

p.s. also test it with a suite with no links to ensure it works with reports with and with out links, but I think the modified test data already covers that.

Yeah, I've been testing it with VCDM tests, but I think this still needs work.

BigBlueHat commented 2 months ago

Did more local testing and this is good to go--and I plan to use it soon on the VCDMv2 test suite, so merging.