GraphQL: Add timestamps (#675)

* feat: add timestamps to graphql types

* chore: add in a default page size of 25 and update graphql tests to include querying for timestamps.

app/graphql/irida_schema.rb

@@ -10,6 +10,7 @@ class IridaSchema < GraphQL::Schema # rubocop:disable GraphQL/ObjectDescription
 
   max_depth 15
   max_complexity 550
+  default_page_size 25
   default_max_page_size 100
 
   # GraphQL-Ruby calls this when something goes wrong while running a query:

app/graphql/schema.graphql

@@ -63,9 +63,9 @@ type Attachment implements Node {
   byteSize: Int!
 
   """
-  Attachment creation date
+  Datetime of creation.
   """
-  createdAt: String!
+  createdAt: ISO8601DateTime!
 
   """
   Attachment file name
@@ -91,6 +91,11 @@ type Attachment implements Node {
   Persistent Unique Identifier of the attachment. For example, `INXT_ATT_AAAAAAAAAAAA`.
   """
   puid: ID!
+
+  """
+  Datetime of last update.
+  """
+  updatedAt: ISO8601DateTime!
 }
 
 """
@@ -257,6 +262,11 @@ type DirectUpload {
 A group
 """
 type Group implements Node {
+  """
+  Datetime of creation.
+  """
+  createdAt: ISO8601DateTime!
+
   """
   List of descendant groups of this group.
   """
@@ -357,6 +367,11 @@ type Group implements Node {
                                     `INXT_GRP_AAAAAAAAAA`.
   """
   puid: ID!
+
+  """
+  Datetime of last update.
+  """
+  updatedAt: ISO8601DateTime!
 }
 
 """
@@ -399,6 +414,11 @@ type GroupEdge {
   node: Group
 }
 
+"""
+An ISO 8601-encoded datetime
+"""
+scalar ISO8601DateTime @specifiedBy(url: "https://tools.ietf.org/html/rfc3339")
+
 """
 Represents untyped JSON
 """
@@ -453,6 +473,11 @@ type Mutation {
 A namespace
 """
 type Namespace implements Node {
+  """
+  Datetime of creation.
+  """
+  createdAt: ISO8601DateTime!
+
   """
   Description of the namespace.
   """
@@ -518,6 +543,11 @@ type Namespace implements Node {
                                     `INXT_GRP_AAAAAAAAAA`.
   """
   puid: ID!
+
+  """
+  Datetime of last update.
+  """
+  updatedAt: ISO8601DateTime!
 }
 
 """
@@ -559,6 +589,11 @@ type PageInfo {
 A project
 """
 type Project implements Node {
+  """
+  Datetime of creation.
+  """
+  createdAt: ISO8601DateTime!
+
   """
   Description of the project.
   """
@@ -623,6 +658,11 @@ type Project implements Node {
     """
     last: Int
   ): SampleConnection
+
+  """
+  Datetime of last update.
+  """
+  updatedAt: ISO8601DateTime!
 }
 
 """
@@ -890,6 +930,16 @@ type Sample implements Node {
     last: Int
   ): AttachmentConnection
 
+  """
+  Datetime when associated attachments were last updated.
+  """
+  attachmentsUpdatedAt: ISO8601DateTime
+
+  """
+  Datetime of creation.
+  """
+  createdAt: ISO8601DateTime!
+
   """
   Description of the sample.
   """
@@ -924,6 +974,11 @@ type Sample implements Node {
   Persistent Unique Identifier of the sample. For example, `INXT_SAM_AAAAAAAAAAAA`.
   """
   puid: ID!
+
+  """
+  Datetime of last update.
+  """
+  updatedAt: ISO8601DateTime!
 }
 
 """
@@ -1020,6 +1075,11 @@ type UpdateSampleMetadataPayload {
 A user
 """
 type User {
+  """
+  Datetime of creation.
+  """
+  createdAt: ISO8601DateTime!
+
   """
   User email.
   """
@@ -1029,4 +1089,9 @@ type User {
   ID of the user.
   """
   id: ID!
+
+  """
+  Datetime of last update.
+  """
+  updatedAt: ISO8601DateTime!
 }

app/graphql/types/attachment_type.rb

@@ -2,12 +2,11 @@
 
 module Types
   # Attachment Type
-  class AttachmentType < Types::BaseObject
+  class AttachmentType < Types::BaseType
     implements GraphQL::Types::Relay::Node
     description 'An attachment'
 
     field :byte_size, Integer, null: false, description: 'Attachment file size'
-    field :created_at, String, null: false, description: 'Attachment creation date'
     field :filename, String, null: false, description: 'Attachment file name'
     field :puid,
           ID,

app/graphql/types/base_type.rb

@@ -0,0 +1,10 @@
+# frozen_string_literal: true
+
+module Types
+  # Base Type
+  class BaseType < Types::BaseObject
+    # ISO8601DateTime field
+    field :created_at, GraphQL::Types::ISO8601DateTime, null: false, description: 'Datetime of creation.'
+    field :updated_at, GraphQL::Types::ISO8601DateTime, null: false, description: 'Datetime of last update.'
+  end
+end

app/graphql/types/namespace_type.rb

@@ -2,7 +2,7 @@
 
 module Types
   # Namespace Type
-  class NamespaceType < Types::BaseObject
+  class NamespaceType < Types::BaseType
     implements GraphQL::Types::Relay::Node
     description 'A namespace'
 

app/graphql/types/project_type.rb

@@ -2,7 +2,7 @@
 
 module Types
   # Project Type
-  class ProjectType < Types::BaseObject
+  class ProjectType < Types::BaseType
     implements GraphQL::Types::Relay::Node
     description 'A project'
 

app/graphql/types/sample_type.rb

@@ -2,7 +2,7 @@
 
 module Types
   # Sample Type
-  class SampleType < Types::BaseObject
+  class SampleType < Types::BaseType
     implements GraphQL::Types::Relay::Node
     description 'A sample'
 
@@ -25,6 +25,10 @@ class SampleType < Types::BaseObject
           complexity: 5,
           resolver: Resolvers::SampleAttachmentsResolver
 
+    field :attachments_updated_at, GraphQL::Types::ISO8601DateTime,
+          null: true,
+          description: 'Datetime when associated attachments were last updated.'
+
     def self.authorized?(object, context)
       super &&
         allowed_to?(

app/graphql/types/user_type.rb

@@ -2,7 +2,7 @@
 
 module Types
   # User Type
-  class UserType < Types::BaseObject
+  class UserType < Types::BaseType
     description 'A user'
 
     field :email, String, null: false, description: 'User email.'

test/graphql/attachments_query_test.rb

@@ -13,7 +13,9 @@ class AttachmentsQueryTest < ActiveSupport::TestCase
               id,
               metadata,
               filename,
-              byteSize
+              byteSize,
+              createdAt,
+              updatedAt
             }
           }
         }
@@ -30,7 +32,9 @@ class AttachmentsQueryTest < ActiveSupport::TestCase
             node {
               id,
               filename,
-              attachmentUrl
+              attachmentUrl,
+              createdAt,
+              updatedAt
             }
           }
         }

test/graphql/group_query_test.rb

@@ -12,6 +12,8 @@ class GroupQueryTest < ActiveSupport::TestCase
         id
         fullName
         fullPath
+        createdAt
+        updatedAt
         descendantGroups(includeParentDescendants: $includeParentDescendants) {
           nodes {
             name
@@ -32,6 +34,8 @@ class GroupQueryTest < ActiveSupport::TestCase
         id
         fullName
         fullPath
+        createdAt
+        updatedAt
         descendantGroups(includeParentDescendants: $includeParentDescendants) {
           nodes {
             name

test/graphql/groups_query_test.rb

@@ -11,6 +11,8 @@ class GroupsQueryTest < ActiveSupport::TestCase
           path
           description
           id
+          createdAt
+          updatedAt
         }
         totalCount
       }

test/graphql/namespace_query_test.rb

@@ -12,6 +12,8 @@ class NamespaceQueryTest < ActiveSupport::TestCase
         id
         fullName
         fullPath
+        createdAt
+        updatedAt
       }
     }
   GRAPHQL

test/graphql/project_query_test.rb

@@ -12,6 +12,8 @@ class ProjectQueryTest < ActiveSupport::TestCase
         id
         fullName
         fullPath
+        createdAt
+        updatedAt
       }
     }
   GRAPHQL
@@ -25,6 +27,8 @@ class ProjectQueryTest < ActiveSupport::TestCase
         id
         fullName
         fullPath
+        createdAt
+        updatedAt
       }
     }
   GRAPHQL

test/graphql/sample_query_test.rb

@@ -9,6 +9,9 @@ class SampleQueryTest < ActiveSupport::TestCase
         name
         description
         id
+        createdAt
+        updatedAt
+        attachmentsUpdatedAt
       }
     }
   GRAPHQL