Refactor controller code

[amirbavand]

This PR aims to refactor DynamicReconciler.go
The reason for changing other files is to make the targeted file (DynamicReconciler.go) testable. The other files were mainly changed to do dependency injection for DynamicReconciler.
The main refactoring for other files inside controller package will be done in the future PRs

[github-actions]

⬇️ Go test coverage decreased from 71.2% to 71.0% compared to fe0e695 (1 ignored files)

⚠️ 2 of 12 packages have zero coverage.

• kumquat
• kumquat/cmd

Updated Package Coverages:

# Package Name                |  Prior |    New
+ kumquat/internal/controller |  72.6% |  73.3%
+ kumquat/repository          |  91.8% |  92.8%

View coverage for all packages

# Package Name                | Coverage
- kumquat                     |     0.0%
+ kumquat/api/v1beta1         |   100.0%
- kumquat/cmd                 |     0.0%
+ kumquat/internal/controller |    73.3%
+ kumquat/renderer            |   100.0%
+ kumquat/renderer/cue        |    88.4%
+ kumquat/renderer/gotemplate |    95.8%
+ kumquat/renderer/jsonnet    |    87.9%
+ kumquat/repository          |    92.8%
+ kumquat/store               |    75.0%
+ kumquat/template            |    93.9%
+ kumquat/test/utils          |    10.8%

[jamesdobson]

This also leaks the abstraction because it returns table names. Also, we will have to copy it for each implementation of Repository.

What do you think about the following?

We can fix this with a Query interface:

type Query interface {
    SQLQueryString() string
    GetKinds() []string
}

and a concrete implementation SQLQuery. The GetKinds() method can contain the logic for ExtractTableNamesFromQuery().

We can update Repository.Query() to accept a Query as its arguments. The implementation of Query() in SQLiteRepository can call SQLQueryString() to get the string to use.

[jamesdobson]

If you are able to change DropTable to be DeleteAll, then you can eliminate this case: DeleteAll will return success even if there is nothing to delete.