@openneuro/server 3.36.6-alpha.0 → 3.36.6-alpha.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@openneuro/server",
-  "version": "3.36.6-alpha.0",
+  "version": "3.36.6-alpha.4",
   "description": "Core service for the OpenNeuro platform.",
   "license": "MIT",
   "main": "src/server.js",
@@ -61,6 +61,7 @@
     "react-dom": "^17.0.1",
     "redlock": "^4.0.0",
     "request": "^2.83.0",
+    "semver": "^5.5.0",
     "sitemap": "^2.1.0",
     "subscriptions-transport-ws": "0.9.18",
     "superagent": "^3.8.2",
@@ -81,6 +82,7 @@
     "@types/ioredis": "^4.17.1",
     "@types/jest": "^26.0.23",
     "@types/nodemailer": "^6.4.1",
+    "@types/semver": "^5",
     "apollo-link-schema": "^1.2.5",
     "babel-eslint": "^10.1.0",
     "core-js": "^3.10.1",
@@ -102,5 +104,5 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "bf1e240880aec72270e1bb31648d07045dca3d56"
+  "gitHead": "3bb8cca2661acc0375c29c2f3cf32eedc53ed8ed"
 }

package/src/graphql/resolvers/__tests__/dataset.spec.js

@@ -27,16 +27,20 @@ describe('dataset resolvers', () => {
     })
   })
   describe('snapshotCreationComparison()', () => {
-    it('sorts array of objects by the "created" property', () => {
+    it('sorts array of objects by the "created" and "tag" properties', () => {
       const testArray = [
-        { id: 2, created: new Date('2018-11-20T00:05:43.473Z') },
-        { id: 1, created: new Date('2018-11-19T00:05:43.473Z') },
-        { id: 3, created: new Date('2018-11-23T00:05:43.473Z') },
+        { id: 2, created: new Date('2018-11-20T00:05:43.473Z'), tag: '1.0.0' },
+        { id: 1, created: new Date('2018-11-19T00:05:43.473Z'), tag: '1.0.1' },
+        { id: 3, created: new Date('2018-11-23T00:05:43.473Z'), tag: '1.0.2' },
+        { id: 5, created: new Date('2018-11-23T00:05:43.473Z'), tag: '1.0.10' },
+        { id: 4, created: new Date('2018-11-23T00:05:43.473Z'), tag: '1.0.3' },
       ]
       const sorted = testArray.sort(ds.snapshotCreationComparison)
       expect(sorted[0].id).toBe(1)
       expect(sorted[1].id).toBe(2)
       expect(sorted[2].id).toBe(3)
+      expect(sorted[3].id).toBe(4)
+      expect(sorted[4].id).toBe(5)
     })
     it('sorts array of objects by the "created" property as strings', () => {
       const testArray = [
@@ -60,18 +64,16 @@ describe('dataset resolvers', () => {
       mockingoose.Dataset.toReturn(true, 'count')
       // capture and check datalad delete request
       request.del = url => ({
-          set: (header1, headerValue1) => ({
+        set: (header1, headerValue1) => ({
           set: () => ({
-              send: ({ filenames}) => {
-              expect(url).toEqual(
-                'http://datalad-0/datasets/ds999999/files',
-              )
+            send: ({ filenames }) => {
+              expect(url).toEqual('http://datalad-0/datasets/ds999999/files')
               expect(filenames).toEqual([':sub-99'])
               expect(header1).toEqual('Cookie')
               expect(headerValue1).toMatch(/^accessToken=/)
-            }
-            }),
+            },
           }),
+        }),
       })
 
       ds.deleteFiles(

package/src/graphql/resolvers/dataset.js

@@ -23,6 +23,7 @@ import { UpdatedFile } from '../utils/file.js'
 import { getDatasetWorker } from '../../libs/datalad-service.js'
 import { getDraftHead } from '../../datalad/dataset.js'
 import { getFileName } from '../../datalad/files.js'
+import semver from 'semver'
 
 export const dataset = async (obj, { id }, { user, userInfo }) => {
   await checkDatasetRead(id, user, userInfo)
@@ -41,8 +42,14 @@ export const datasets = (parent, args, { user, userInfo }) => {
   }
 }
 
-export const snapshotCreationComparison = ({ created: a }, { created: b }) => {
-  return new Date(a).getTime() - new Date(b).getTime()
+export const snapshotCreationComparison = (
+  { created: a, tag: a_tag },
+  { created: b, tag: b_tag },
+) => {
+  return (
+    new Date(a).getTime() - new Date(b).getTime() ||
+    semver.compare(a_tag, b_tag)
+  )
 }
 
 /**