Merge pull request #28 from smorimoto/fix-job-type

Fix possible exceptions by matching the type of Snapshot job to the spec

Author: Federico Builes

package-lock.json

@@ -32,7 +32,7 @@ describe('Snapshot', () => {
         version: '0.0.1'
       },
       context,
-      { id: 42, correlator: 'test' },
+      { id: '42', correlator: 'test' },
       new Date('2022-06-04T05:07:06.457Z')
     )
     snapshot.addManifest(manifest)
@@ -44,7 +44,7 @@ describe('Snapshot', () => {
       },
       version: 0,
       job: {
-        id: 42,
+        id: '42',
         correlator: 'test'
       },
       ref: 'foo/bar/baz',

src/snapshot.test.ts

@@ -16,7 +16,7 @@ Core functionality for creating a snapshot of a project's dependencies.
  */
 export type Job = {
   correlator: string
-  id: string | number
+  id: string
   html_url?: string // eslint-disable-line camelcase
 }