#1: Remove comments about compatibility check with exists Authlibs

elyby · Apr 28, 2019 · 9250d53 · 9250d53
1 parent 2c7a162
commit 9250d53
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 5 deletions.
diff --git a/http/textures.go b/http/textures.go
@@ -41,14 +41,12 @@ func (cfg *Config) Textures(response http.ResponseWriter, request *http.Request)
 	} else {
 		mojangTextures := <-cfg.MojangTexturesQueue.GetTexturesForUsername(username)
 		if mojangTextures == nil {
-			// TODO: test compatibility with exists authlibs
 			response.WriteHeader(http.StatusNoContent)
 			return
 		}
 
 		texturesProp := mojangTextures.DecodeTextures()
 		if texturesProp == nil {
-			// TODO: test compatibility with exists authlibs
 			response.WriteHeader(http.StatusInternalServerError)
 			cfg.Logger.Error("Unable to find textures property")
 			return

diff --git a/http/textures_test.go b/http/textures_test.go
@@ -189,8 +189,6 @@ func TestConfig_Textures(t *testing.T) {
 		config.CreateHandler().ServeHTTP(w, req)
 
 		resp := w.Result()
-		assert.Equal(204, resp.StatusCode) // TODO: this is not confirmed behavior
+		assert.Equal(204, resp.StatusCode)
 	})
 }
-
-