Skip to content

Commit

Permalink
chore: version bump
Browse files Browse the repository at this point in the history
  • Loading branch information
willfarrell committed Oct 4, 2022
1 parent 76893fc commit d77a8cf
Show file tree
Hide file tree
Showing 9 changed files with 65 additions and 27 deletions.
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -63,6 +63,7 @@ typings/
# IDE
.idea
*.iml
.nova

# OS
.DS_Store
6 changes: 4 additions & 2 deletions lerna.json
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
{
"packages": ["schemas/*"],
"version": "2.6.10"
"packages": [
"schemas/*"
],
"version": "2.11.0"
}
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "@datastream/schema-monorepo",
"version": "2.10.3",
"version": "2.11.0",
"description": "DataStream Water Quality Exchange (WQX): JSON Schema and JSON Table Schema for DataStream",
"private": true,
"type": "module",
Expand Down
25 changes: 22 additions & 3 deletions schemas/data/bin/build-logic.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ import { subset, sort } from './build-lib.js'
const __dirname = dirname(fileURLToPath(import.meta.url))

const wqxRequiredIf = async (file) => {
console.log(file)
const [columnFrom, columnTo] = file.split('-')

const logicJSON = await readFile(
Expand All @@ -26,8 +25,28 @@ const wqxRequiredIf = async (file) => {
}

// TODO - remove once included in WQX look up
object.if.properties[columnFrom].enum.push('Nitrogen-15/Nitrogen-14 ratio')
object.if.properties[columnFrom].enum.push('Sulfur Delta 34')
if (file === 'CharacteristicName-MethodSpeciation') {
const stableIsotopes = await readFile(
join(
__dirname,
`/../src/logic/CharacteristicName-StableIsotope-MethodSpeciation.json`
)
)
.then((res) => JSON.parse(res))
.catch(() => ({}))

for (const characteristicName of stableIsotopes.if.properties
.CharacteristicName.enum) {
if (!object.if.properties[columnFrom].enum.includes(characteristicName)) {
console.log(
'CharacteristicName-MethodSpeciation missing (StableIsotope rule)',
characteristicName
)
object.if.properties[columnFrom].enum.push(characteristicName)
}
}
}
// END TODO

const list = [...new Set(sort(object.if.properties[columnFrom].enum))]

Expand Down
4 changes: 2 additions & 2 deletions schemas/data/package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion schemas/data/package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "@datastreamapp/data-schema",
"version": "2.10.3",
"version": "2.11.0",
"description": "DataStream Water Quality Data Schema",
"publishConfig": {
"cache": "~/.npm",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@
"Kjeldahl nitrogen",
"Nitrate",
"Nitrite",
"Nitrogen Delta 15",
"Nitrogen-15",
"Nitrogen-15/Nitrogen-14 ratio",
"Organic Nitrogen",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@
"Kjeldahl nitrogen",
"Nitrate",
"Nitrite",
"Nitrogen Delta 15",
"Nitrogen-15",
"Nitrogen-15/Nitrogen-14 ratio",
"Organic Nitrogen",
Expand Down
50 changes: 32 additions & 18 deletions schemas/data/test/logic.js
Original file line number Diff line number Diff line change
Expand Up @@ -97,10 +97,7 @@ test('Should rejects CharacteristicName AND NOT ResultSampleFraction', async (t)
CharacteristicName: 'Silver'
})
t.false(valid)
t.is(
checkProperty(validate.errors, 'required', 'ResultSampleFraction'),
true
)
t.is(checkProperty(validate.errors, 'required', 'ResultSampleFraction'), true)
})
test('Should accept CharacteristicName AND ResultSampleFraction', async (t) => {
const valid = validate({
Expand All @@ -115,6 +112,34 @@ test('Should accept CharacteristicName AND ResultSampleFraction', async (t) => {
)
})

// CharacteristicName-StableIsotope-ResultSampleFraction
test('Should reject StableIsotope CharacteristicName, MethodSpeciation required', async (t) => {
const valid = validate({
CharacteristicName: 'Nitrogen Delta 15'
})
t.false(valid)
t.is(checkProperty(validate.errors, 'required', 'CharacteristicName'), false)
t.is(checkProperty(validate.errors, 'required', 'MethodSpeciation'), true)
})
test('Should reject StableIsotope CharacteristicName', async (t) => {
const valid = validate({
CharacteristicName: 'Nitrogen Delta 15',
MethodSpeciation: 'as NH4'
})
t.false(valid)
t.is(checkProperty(validate.errors, 'required', 'CharacteristicName'), false)
t.is(checkProperty(validate.errors, 'enum', 'MethodSpeciation'), true)
})
test('Should accept StableIsotope CharacteristicName', async (t) => {
const valid = validate({
CharacteristicName: 'Nitrogen Delta 15',
MethodSpeciation: 'of NH4'
})
t.false(valid)
t.is(checkProperty(validate.errors, 'required', 'CharacteristicName'), false)
t.is(checkProperty(validate.errors, 'required', 'MethodSpeciation'), false)
})

// CharacteristicName-Nutrient-ResultSampleFraction
test('Should reject Nutrient CharacteristicName AND ResultSampleFraction', async (t) => {
const valid = validate({
Expand Down Expand Up @@ -579,10 +604,7 @@ test('Should accept columns without potential csv injection', async (t) => {
t.false(valid)
// console.log(valid, JSON.stringify(validate.errors, null, 2))
t.is(checkProperty(validate.errors, 'pattern', 'DatasetName'), false)
t.is(
checkProperty(validate.errors, 'pattern', 'MonitoringLocationID'),
false
)
t.is(checkProperty(validate.errors, 'pattern', 'MonitoringLocationID'), false)
t.is(
checkProperty(validate.errors, 'pattern', 'MonitoringLocationName'),
false
Expand All @@ -608,14 +630,7 @@ test('Should accept when Salinity and expected unit', async (t) => {
ResultUnit: 'PSU'
})
t.is(valid, false)
t.is(
checkProperty(
validate.errors,
'enum',
'ResultUnit'
),
false
)
t.is(checkProperty(validate.errors, 'enum', 'ResultUnit'), false)
})
test('Should reject when Salinity and `ppt`', async (t) => {
const valid = validate({
Expand All @@ -626,7 +641,6 @@ test('Should reject when Salinity and `ppt`', async (t) => {
t.is(checkProperty(validate.errors, 'enum', 'ResultUnit'), true)
})


// *** ResultDetectionQuantitationLimitUnit-Salinity *** //
test('Should accept when Salinity and expected ResultDetectionQuantitationLimitUnit', async (t) => {
const valid = validate({
Expand Down Expand Up @@ -659,4 +673,4 @@ test('Should reject when Salinity and ResultDetectionQuantitationLimitUnit=`ppt`
),
true
)
})
})

0 comments on commit d77a8cf

Please sign in to comment.