Add source location in definitions for sourcearchive packages

clearlydefined / service

The service side of clearlydefined.io

MIT License

45 stars 40 forks source link

Add source location in definitions for sourcearchive packages #1108

Closed qtomlinson closed 5 months ago

qtomlinson commented 5 months ago

definitionServiceTest.js is getting big. Separate the aggregator related tests to aggregatorTests.js.
For a package containing source, there is existing logic to build source location in its definition. Add the source location for sourcearchive packages as well.
Add unit tests for all cases of building source location

Task: https://github.com/clearlydefined/service/issues/1103

qtomlinson commented 5 months ago

@lumaxis @elrayle This fixes future definition results. How about the previously computed definitions that are currently in storage? We can discuss whether we want to fix those at our dev sync meeting this week.