fix: Checking the artifact cache only if the resource artifact status is set (#98)

Signed-off-by: Rashed Kamal <[email protected]>

The MavenArtifact API status condition was set incorrectly during the artifact cache check. This PR adds a check for the resource `status.artifact` while verifying the artifact cache.

Author: rashedkvm

controllers/mavenartifact_controller.go

@@ -321,8 +321,8 @@ func MavenArtifactDownloadSyncReconciler(httpRootDir, httpHost string, now func(
 				return err
 			}
 
-			// Compare checksum with cache
-			if cache != nil {
+			// Compare checksum with cache if the resource status.artifact is set
+			if cache != nil && parent.Status.Artifact != nil {
 				if cache.checksum == remoteChecksum && cache.source == artifactInfo.ArtifactDownloadURL {
 					log.Info("download skipped", "checksum matched on disc", cache.checksum, "checksum from remote repository", remoteChecksum)
 					return nil