remove the test
This commit is contained in:
		
							parent
							
								
									36a5f52b24
								
							
						
					
					
						commit
						76a7c5abfe
					
				| 
						 | 
					@ -557,95 +557,6 @@ func TestExtractPgVersionFromBinPath(t *testing.T) {
 | 
				
			||||||
	}
 | 
						}
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
func GetPgVersion(t *testing.T) {
 | 
					 | 
				
			||||||
	testName := "TestGetPgVersion"
 | 
					 | 
				
			||||||
	tests := []struct {
 | 
					 | 
				
			||||||
		subTest          string
 | 
					 | 
				
			||||||
		pgContainer      v1.Container
 | 
					 | 
				
			||||||
		currentPgVersion string
 | 
					 | 
				
			||||||
		newPgVersion     string
 | 
					 | 
				
			||||||
	}{
 | 
					 | 
				
			||||||
		{
 | 
					 | 
				
			||||||
			subTest: "new version with decimal point differs from current SPILO_CONFIGURATION",
 | 
					 | 
				
			||||||
			pgContainer: v1.Container{
 | 
					 | 
				
			||||||
				Name: "postgres",
 | 
					 | 
				
			||||||
				Env: []v1.EnvVar{
 | 
					 | 
				
			||||||
					{
 | 
					 | 
				
			||||||
						Name:  "SPILO_CONFIGURATION",
 | 
					 | 
				
			||||||
						Value: "{\"postgresql\": {\"bin_dir\": \"/usr/lib/postgresql/9.6/bin\"}}",
 | 
					 | 
				
			||||||
					},
 | 
					 | 
				
			||||||
				},
 | 
					 | 
				
			||||||
			},
 | 
					 | 
				
			||||||
			currentPgVersion: "9.6",
 | 
					 | 
				
			||||||
			newPgVersion:     "12",
 | 
					 | 
				
			||||||
		},
 | 
					 | 
				
			||||||
		{
 | 
					 | 
				
			||||||
			subTest: "new version differs from current SPILO_CONFIGURATION",
 | 
					 | 
				
			||||||
			pgContainer: v1.Container{
 | 
					 | 
				
			||||||
				Name: "postgres",
 | 
					 | 
				
			||||||
				Env: []v1.EnvVar{
 | 
					 | 
				
			||||||
					{
 | 
					 | 
				
			||||||
						Name:  "SPILO_CONFIGURATION",
 | 
					 | 
				
			||||||
						Value: "{\"postgresql\": {\"bin_dir\": \"/usr/lib/postgresql/11/bin\"}}",
 | 
					 | 
				
			||||||
					},
 | 
					 | 
				
			||||||
				},
 | 
					 | 
				
			||||||
			},
 | 
					 | 
				
			||||||
			currentPgVersion: "11",
 | 
					 | 
				
			||||||
			newPgVersion:     "12",
 | 
					 | 
				
			||||||
		},
 | 
					 | 
				
			||||||
		{
 | 
					 | 
				
			||||||
			subTest: "new version is lower than the one found in current SPILO_CONFIGURATION",
 | 
					 | 
				
			||||||
			pgContainer: v1.Container{
 | 
					 | 
				
			||||||
				Name: "postgres",
 | 
					 | 
				
			||||||
				Env: []v1.EnvVar{
 | 
					 | 
				
			||||||
					{
 | 
					 | 
				
			||||||
						Name:  "SPILO_CONFIGURATION",
 | 
					 | 
				
			||||||
						Value: "{\"postgresql\": {\"bin_dir\": \"/usr/lib/postgresql/12/bin\"}}",
 | 
					 | 
				
			||||||
					},
 | 
					 | 
				
			||||||
				},
 | 
					 | 
				
			||||||
			},
 | 
					 | 
				
			||||||
			currentPgVersion: "12",
 | 
					 | 
				
			||||||
			newPgVersion:     "11",
 | 
					 | 
				
			||||||
		},
 | 
					 | 
				
			||||||
		{
 | 
					 | 
				
			||||||
			subTest: "new version is the same like in the current SPILO_CONFIGURATION",
 | 
					 | 
				
			||||||
			pgContainer: v1.Container{
 | 
					 | 
				
			||||||
				Name: "postgres",
 | 
					 | 
				
			||||||
				Env: []v1.EnvVar{
 | 
					 | 
				
			||||||
					{
 | 
					 | 
				
			||||||
						Name:  "SPILO_CONFIGURATION",
 | 
					 | 
				
			||||||
						Value: "{\"postgresql\": {\"bin_dir\": \"/usr/lib/postgresql/12/bin\"}}",
 | 
					 | 
				
			||||||
					},
 | 
					 | 
				
			||||||
				},
 | 
					 | 
				
			||||||
			},
 | 
					 | 
				
			||||||
			currentPgVersion: "12",
 | 
					 | 
				
			||||||
			newPgVersion:     "12",
 | 
					 | 
				
			||||||
		},
 | 
					 | 
				
			||||||
	}
 | 
					 | 
				
			||||||
 | 
					 | 
				
			||||||
	var cluster = New(
 | 
					 | 
				
			||||||
		Config{
 | 
					 | 
				
			||||||
			OpConfig: config.Config{
 | 
					 | 
				
			||||||
				ProtectedRoles: []string{"admin"},
 | 
					 | 
				
			||||||
				Auth: config.Auth{
 | 
					 | 
				
			||||||
					SuperUsername:       superUserName,
 | 
					 | 
				
			||||||
					ReplicationUsername: replicationUserName,
 | 
					 | 
				
			||||||
				},
 | 
					 | 
				
			||||||
			},
 | 
					 | 
				
			||||||
		}, k8sutil.KubernetesClient{}, acidv1.Postgresql{}, logger, eventRecorder)
 | 
					 | 
				
			||||||
 | 
					 | 
				
			||||||
	for _, tt := range tests {
 | 
					 | 
				
			||||||
		pgVersion, err := cluster.getNewPgVersion(tt.pgContainer, tt.newPgVersion)
 | 
					 | 
				
			||||||
		if err != nil {
 | 
					 | 
				
			||||||
			t.Errorf("Unexpected error: %v", err)
 | 
					 | 
				
			||||||
		}
 | 
					 | 
				
			||||||
		if pgVersion != tt.currentPgVersion {
 | 
					 | 
				
			||||||
			t.Errorf("%s %s: Expected version %s, have %s instead",
 | 
					 | 
				
			||||||
				testName, tt.subTest, tt.currentPgVersion, pgVersion)
 | 
					 | 
				
			||||||
		}
 | 
					 | 
				
			||||||
	}
 | 
					 | 
				
			||||||
}
 | 
					 | 
				
			||||||
 | 
					 | 
				
			||||||
func TestSecretVolume(t *testing.T) {
 | 
					func TestSecretVolume(t *testing.T) {
 | 
				
			||||||
	testName := "TestSecretVolume"
 | 
						testName := "TestSecretVolume"
 | 
				
			||||||
	tests := []struct {
 | 
						tests := []struct {
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
		Reference in New Issue