Extract common ag inspection info into its own schema

[rhofvendahl]

A number of agricultural inspection schemas (FoodGradeInspection, AgricultureInspectionGeneric, several more in the works) share several common properties. The ideal solution would be to create a base AgricultureInspection schema which other inspections extend, but as far as I know that functionality isn't currently supported (see issue 162, issue 277).

An alternative to inheritance is to create a schema which includes properties common to multiple inspection schemas and include that in each. This PR implements that approach by creating an AgricultureInspectionCommonInfo schema and refactoring AgricultureInspectionGeneric (renamed from AgricultureInspectionReport for clarity) and FoodGradeInspection to make use of this common info schema.

[nissimsan]

I think this is a step in the wrong direction. "Ag" is not as clear as "Agriculture".

[OR13]

Agree

[BenjaminMoe]

https://github.com/w3c-ccg/traceability-vocab/actions/runs/3223964212/jobs/5274591511

        "estimatedCharges": "$135.00"
      } 
       [
        {
          keyword: 'enum',
          dataPath: '/commonInfo/type/0',
          schemaPath: '#/properties/type/oneOf/0/items/enum',
          params: { allowedValues: [Array] },
          message: 'should be equal to one of the allowed values'
        },
        {
          keyword: 'type',
          dataPath: '/commonInfo/type',
          schemaPath: '#/properties/type/oneOf/1/type',
          params: { type: 'string' },
          message: 'should be string'
        },
        {
          keyword: 'const',
          dataPath: '/commonInfo/type',
          schemaPath: '#/properties/type/oneOf/1/const',
          params: { allowedValue: [Array] },
          message: 'should be equal to constant'
        },
        {
          keyword: 'oneOf',
          dataPath: '/commonInfo/type',
          schemaPath: '#/properties/type/oneOf',
          params: { passingSchemas: null },
          message: 'should match exactly one schema in oneOf'
        }
      ]

      51 |         const isValid = ajv.validate(s.$id, input);
      52 |         if (!isValid) {
    > 53 |           console.error(s.$linkedData.term, '\n', s.example, '\n', ajv.errors);
         |                   ^
      54 |         }
      55 |
      56 |         return true;

      at __tests__/schemas-sanity.test.js:53:19
          at async Promise.all (index 44)
      at Object.<anonymous> (__tests__/schemas-sanity.test.js:37:18)

[BenjaminMoe]

__tests__/schemas-sanity.test.js

[BenjaminMoe]

https://github.com/w3c-ccg/traceability-vocab/blob/main/packages/traceability-schemas/__tests__/schemas-sanity.test.js#L36

[BenjaminMoe]

#586

[brownoxford]

OK to merge once Jest tests are all passing.

[BenjaminMoe]

Okay to merge once passing Jest tests

[OR13]

this will likely create other issues, but not a blocker.