Prefer using testify/assert over dynassert by default (#3310)

## Changes
- Replace dynassert with testify/assert everywhere where testify/assert
works.
- Clean up dynassert proxy functions. We should use testify/assert
directly.
- Do not alias dynassert as assert.

## Why
- testify/assert has more clear semantics. Its usage is also understood
and fixed by linters.
- It should be obvious to reader that this test uses equality with
additional semantics (Mapping are compared unordered and locations are
compared via Location() function), so explicit dynassert.Equal signals
that. The custom equality should not mask standard equality.
- By default we should be testing equality fully, with mapping order and
full locations. Custom comparison in asserts should be an exception, not
the default.

## Tests
Existing tests.

To ensure updated tests did not become flaky I ran `go test
./libs/dyn/... ./bundle/config/mutator/... -count=1000`

Author: denik

.golangci.yaml

@@ -65,6 +65,7 @@ linters:
         - empty
         - len
         - equal-values
+        - encoded-compare # Always replaces Equal() with JSONEq, but sometimes exact string match is wanted for JSON
   exclusions:
     generated: lax
     presets:

bundle/config/mutator/normalize_paths_test.go

@@ -5,13 +5,12 @@ import (
 	"path/filepath"
 	"testing"
 
-	assert "github.com/databricks/cli/libs/dyn/dynassert"
-
 	"github.com/databricks/cli/bundle"
 	"github.com/databricks/cli/bundle/config"
 	"github.com/databricks/cli/bundle/config/resources"
 	"github.com/databricks/cli/libs/dyn"
 	"github.com/databricks/databricks-sdk-go/service/jobs"
+	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"
 )
 

bundle/config/mutator/paths/app_paths_visitor_test.go

@@ -6,7 +6,7 @@ import (
 	"github.com/databricks/cli/bundle/config"
 	"github.com/databricks/cli/bundle/config/resources"
 	"github.com/databricks/cli/libs/dyn"
-	assert "github.com/databricks/cli/libs/dyn/dynassert"
+	"github.com/stretchr/testify/assert"
 )
 
 func TestAppPathsVisitor(t *testing.T) {

bundle/config/mutator/paths/dashboard_paths_visitor_test.go

@@ -6,7 +6,7 @@ import (
 	"github.com/databricks/cli/bundle/config"
 	"github.com/databricks/cli/bundle/config/resources"
 	"github.com/databricks/cli/libs/dyn"
-	assert "github.com/databricks/cli/libs/dyn/dynassert"
+	"github.com/stretchr/testify/assert"
 )
 
 func TestVisitDashboardPaths(t *testing.T) {

bundle/config/mutator/paths/job_paths_visitor_test.go

@@ -6,9 +6,9 @@ import (
 	"github.com/databricks/cli/bundle/config"
 	"github.com/databricks/cli/bundle/config/resources"
 	"github.com/databricks/cli/libs/dyn"
-	assert "github.com/databricks/cli/libs/dyn/dynassert"
 	"github.com/databricks/databricks-sdk-go/service/compute"
 	"github.com/databricks/databricks-sdk-go/service/jobs"
+	"github.com/stretchr/testify/assert"
 )
 
 func TestVisitJobPaths(t *testing.T) {

bundle/config/mutator/paths/pipeline_paths_visitor_test.go

@@ -6,8 +6,8 @@ import (
 	"github.com/databricks/cli/bundle/config"
 	"github.com/databricks/cli/bundle/config/resources"
 	"github.com/databricks/cli/libs/dyn"
-	assert "github.com/databricks/cli/libs/dyn/dynassert"
 	"github.com/databricks/databricks-sdk-go/service/pipelines"
+	"github.com/stretchr/testify/assert"
 )
 
 func TestVisitPipelinePaths(t *testing.T) {

bundle/config/mutator/python/apply_python_output_test.go

@@ -8,7 +8,7 @@ import (
 	"github.com/databricks/cli/libs/dyn/merge"
 
 	"github.com/databricks/cli/libs/dyn"
-	assert "github.com/databricks/cli/libs/dyn/dynassert"
+	"github.com/stretchr/testify/assert"
 )
 
 type applyPythonOutputTestCase struct {

bundle/config/mutator/python/python_diagnostics_test.go

@@ -6,7 +6,7 @@ import (
 
 	"github.com/databricks/cli/libs/diag"
 	"github.com/databricks/cli/libs/dyn"
-	assert "github.com/databricks/cli/libs/dyn/dynassert"
+	"github.com/stretchr/testify/assert"
 )
 
 func TestConvertPythonLocation(t *testing.T) {

bundle/config/mutator/python/python_locations_test.go

@@ -6,10 +6,10 @@ import (
 	"testing"
 
 	"github.com/databricks/cli/libs/diag"
-	"github.com/stretchr/testify/require"
-
 	"github.com/databricks/cli/libs/dyn"
-	assert "github.com/databricks/cli/libs/dyn/dynassert"
+	"github.com/databricks/cli/libs/dyn/dynassert"
+	"github.com/stretchr/testify/assert"
+	"github.com/stretchr/testify/require"
 )
 
 func TestMergeLocations(t *testing.T) {
@@ -53,7 +53,7 @@ func TestMergeLocations(t *testing.T) {
 	actual, err := mergePythonLocations(input, locations)
 
 	assert.NoError(t, err)
-	assert.Equal(t, expected, actual)
+	dynassert.Equal(t, expected, actual)
 }
 
 func TestFindLocation(t *testing.T) {

bundle/config/mutator/python/python_mutator_test.go

@@ -20,8 +20,8 @@ import (
 
 	"github.com/databricks/cli/bundle"
 	"github.com/databricks/cli/bundle/config"
-	assert "github.com/databricks/cli/libs/dyn/dynassert"
 	"github.com/databricks/cli/libs/process"
+	"github.com/stretchr/testify/assert"
 )
 
 func TestPythonMutator_Name_loadResources(t *testing.T) {