Browse Source

settings: fixes test

For some reason, the url parse does not fail anymore for curly
braces. Add a colon in the first segment to make sure the url
parse fails.
Daniel Lee 7 years ago
parent
commit
533f3a9e8c
1 changed files with 2 additions and 2 deletions
  1. 2 2
      pkg/setting/setting_test.go

+ 2 - 2
pkg/setting/setting_test.go

@@ -37,8 +37,8 @@ func TestLoadingSettings(t *testing.T) {
 			So(appliedEnvOverrides, ShouldContain, "GF_SECURITY_ADMIN_PASSWORD=*********")
 		})
 
-		Convey("Should replace password when defined in environment2", func() {
-			os.Setenv("GF_DATABASE_URL", "postgres://grafana:sec{ret@postgres:5432/grafana")
+		Convey("Should return an error when url is invalid", func() {
+			os.Setenv("GF_DATABASE_URL", "postgres.%31://grafana:secret@postgres:5432/grafana")
 			err := NewConfigContext(&CommandLineArgs{HomePath: "../../"})
 
 			So(err, ShouldNotBeNil)