refactor: unify the mock file generation

1. Mock remote interface for distribution manifest by mockery package
feature.
2. Refactor hand-generated mock files to automated management
   generation.
3. Clean useless mocks.

Signed-off-by: chlins <chlins.zhang@gmail.com>

src/.mockery.yaml

@@ -9,6 +9,16 @@ packages:
       Controller:
         config:
           dir: testing/controller/artifact
+  github.com/goharbor/harbor/src/controller/artifact/processor:
+    interfaces:
+      Processor:
+        config:
+          dir: testing/pkg/processor
+  github.com/goharbor/harbor/src/controller/artifact/annotation:
+    interfaces:
+      Parser:
+        config:
+          dir: testing/pkg/parser
   github.com/goharbor/harbor/src/controller/blob:
     interfaces:
       Controller:
@@ -188,6 +198,11 @@ packages:
       Manager:
         config:
           dir: testing/pkg/artifact
+  github.com/goharbor/harbor/src/pkg/artifactrash:
+    interfaces:
+      Manager:
+        config:
+          dir: testing/pkg/artifactrash
   github.com/goharbor/harbor/src/pkg/blob:
     interfaces:
       Manager:
@@ -218,6 +233,11 @@ packages:
       Handler:
         config:
           dir: testing/pkg/scan
+  github.com/goharbor/harbor/src/pkg/scan/postprocessors:
+    interfaces:
+      NativeScanReportConverter:
+        config:
+          dir: testing/pkg/scan/postprocessors
   github.com/goharbor/harbor/src/pkg/scan/report:
     interfaces:
       Manager:
@@ -238,7 +258,7 @@ packages:
           dir: pkg/scheduler
           outpkg: scheduler
           mockname: mockDAO
-          filename:  mock_dao_test.go
+          filename: mock_dao_test.go
           inpackage: True
       Scheduler:
         config:
@@ -342,6 +362,14 @@ packages:
       DAO:
         config:
           dir: testing/pkg/immutable/dao
+  github.com/goharbor/harbor/src/pkg/immutable/match:
+    interfaces:
+      ImmutableTagMatcher:
+        config:
+          dir: testing/pkg/immutable
+          filename: matcher.go
+          outpkg: immutable
+          mockname: FakeMatcher
   github.com/goharbor/harbor/src/pkg/ldap:
     interfaces:
       Manager:
@@ -505,20 +533,36 @@ packages:
       Manager:
         config:
           dir: testing/pkg/securityhub
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
+  github.com/goharbor/harbor/src/pkg/tag:
+    interfaces:
+      Manager:
+        config:
+          dir: testing/pkg/tag
+  github.com/goharbor/harbor/src/pkg/p2p/preheat/policy:
+    interfaces:
+      Manager:
+        config:
+          dir: testing/pkg/p2p/preheat/policy
+  github.com/goharbor/harbor/src/pkg/p2p/preheat/instance:
+    interfaces:
+      Manager:
+        config:
+          dir: testing/pkg/p2p/preheat/instance
+  github.com/goharbor/harbor/src/pkg/chart:
+    interfaces:
+      Operator:
+        config:
+          dir: testing/pkg/chart
+  # registryctl related mocks
+  github.com/goharbor/harbor/src/registryctl/client:
+    interfaces:
+      Client:
+        config:
+          dir: testing/registryctl
+          outpkg: registryctl
+  # remote interfaces
+  github.com/docker/distribution:
+    interfaces:
+      Manifest:
+        config:
+          dir: testing/pkg/distribution

src/controller/artifact/processor/chart/chart_test.go

@@ -64,12 +64,12 @@ type processorTestSuite struct {
 	suite.Suite
 	processor *processor
 	regCli    *registry.Client
-	chartOptr *chart.FakeOpertaor
+	chartOptr *chart.Operator
 }
 
 func (p *processorTestSuite) SetupTest() {
 	p.regCli = &registry.Client{}
-	p.chartOptr = &chart.FakeOpertaor{}
+	p.chartOptr = &chart.Operator{}
 	p.processor = &processor{
 		chartOperator: p.chartOptr,
 	}

src/controller/tag/controller_test.go

@@ -38,14 +38,14 @@ type controllerTestSuite struct {
 	ctl          *controller
 	repoMgr      *repository.Manager
 	artMgr       *artifact.Manager
-	tagMgr       *tagtesting.FakeManager
+	tagMgr       *tagtesting.Manager
 	immutableMtr *immutable.FakeMatcher
 }
 
 func (c *controllerTestSuite) SetupTest() {
 	c.repoMgr = &repository.Manager{}
 	c.artMgr = &artifact.Manager{}
-	c.tagMgr = &tagtesting.FakeManager{}
+	c.tagMgr = &tagtesting.Manager{}
 	c.immutableMtr = &immutable.FakeMatcher{}
 	c.ctl = &controller{
 		tagMgr:       c.tagMgr,

src/jobservice/job/impl/gc/garbage_collection_test.go

@@ -42,8 +42,8 @@ import (
 type gcTestSuite struct {
 	htesting.Suite
 	artifactCtl       *artifacttesting.Controller
-	artrashMgr        *trashtesting.FakeManager
-	registryCtlClient *registryctl.Mockclient
+	artrashMgr        *trashtesting.Manager
+	registryCtlClient *registryctl.Client
 	projectCtl        *projecttesting.Controller
 	blobMgr           *blob.Manager
 
@@ -54,8 +54,8 @@ type gcTestSuite struct {
 
 func (suite *gcTestSuite) SetupTest() {
 	suite.artifactCtl = &artifacttesting.Controller{}
-	suite.artrashMgr = &trashtesting.FakeManager{}
-	suite.registryCtlClient = &registryctl.Mockclient{}
+	suite.artrashMgr = &trashtesting.Manager{}
+	suite.registryCtlClient = &registryctl.Client{}
 	suite.blobMgr = &blob.Manager{}
 	suite.projectCtl = &projecttesting.Controller{}