Fixes

Author: grphil

common/connectors/invokerconn/structs.go

@@ -1,7 +1,7 @@
 package invokerconn
 
 import (
-	models2 "testing_system/common/db/models"
+	"testing_system/common/db/models"
 )
 
 // This will be changed in later commits
@@ -16,6 +16,6 @@ type Job struct {
 	JobType  int  `json:"jobType" binding:"required"`
 	Test     int  `json:"test"`
 
-	Submit  *models2.Submission `json:"submit,omitempty"`
-	Problem *models2.Problem    `json:"problem,omitempty"`
+	Submit  *models.Submission `json:"submit,omitempty"`
+	Problem *models.Problem    `json:"problem,omitempty"`
 }

common/connectors/storageconn/structs.go

@@ -66,14 +66,17 @@ type ResponseFiles struct {
 	Size      uint64
 }
 
-func (r *ResponseFiles) File() string {
-	return filepath.Join(r.R.BaseFolder, r.fileNames[0])
+func (r *ResponseFiles) File() (string, bool) {
+	if len(r.fileNames) == 0 {
+		return "", false
+	}
+	return filepath.Join(r.R.BaseFolder, r.fileNames[0]), true
 }
 
 func (r *ResponseFiles) Get(fileName string) (string, bool) {
 	for _, f := range r.fileNames {
 		if fileName == f {
-			return filepath.Join(r.R.BaseFolder, r.fileNames[0]), true
+			return filepath.Join(r.R.BaseFolder, f), true
 		}
 	}
 	return "", false

invoker/storage/cache.go

@@ -6,6 +6,16 @@ import (
 	"testing_system/lib/logger"
 )
 
+// The main idea of cache is following: We have one cache of specific size to hold all types of files.
+// However, we have multiple file types so we must have different getter for each file.
+//
+// So we have single LRUSizeCache that holds all file types. Key for this cache is cacheKey, the internal struct with which we can determine the file type and it's keys.
+//
+// To access cache we have cacheGetter structs. Each cacheGetter responds for single file type.
+// Each cacheGetter accepts any number of uint64 that are transformed to cacheKey struct using cacheGetter.keyGen func.
+//
+// So to access files, we call methods of cacheGetter, that transforms our request to request for LRUSizeCache and LRUSizeCache then does all the cache work.
+
 type commonCache = cache.LRUSizeCache[cacheKey, storageconn.ResponseFiles]
 
 type cacheKey struct {